Remove --output-last-message flag; generalize error msg

Remove the deprecated --output-last-message option from CodexProvider CLI invocations and simplify error messages in WaitWithSpinner and WaitWithPanel from "Claude failed" to the more generic "agent failed" for compatibility and clearer messaging. Changes in internal/agent/codex.go and internal/prd/generator.go.

Author: Simon-BEE

internal/agent/codex.go

@@ -53,7 +53,7 @@ func (p *CodexProvider) ConvertCommand(workDir, prompt string) (*exec.Cmd, loop.
 	}
 	outPath := f.Name()
 	f.Close()
-	cmd := exec.Command(p.cliPath, "exec", "--sandbox", "read-only", "--output-last-message", "-o", outPath, "-")
+	cmd := exec.Command(p.cliPath, "exec", "--sandbox", "read-only", "-o", outPath, "-")
 	cmd.Dir = workDir
 	cmd.Stdin = strings.NewReader(prompt)
 	return cmd, loop.OutputFromFile, outPath, nil
@@ -67,7 +67,7 @@ func (p *CodexProvider) FixJSONCommand(prompt string) (*exec.Cmd, loop.OutputMod
 	}
 	outPath := f.Name()
 	f.Close()
-	cmd := exec.Command(p.cliPath, "exec", "--sandbox", "read-only", "--output-last-message", "-o", outPath, "-")
+	cmd := exec.Command(p.cliPath, "exec", "--sandbox", "read-only", "-o", outPath, "-")
 	cmd.Stdin = strings.NewReader(prompt)
 	return cmd, loop.OutputFromFile, outPath, nil
 }

internal/prd/generator.go

@@ -451,7 +451,7 @@ func WaitWithSpinner(cmd *exec.Cmd, title, message string, stderr *bytes.Buffer)
 		case err := <-done:
 			clearPanelLines(prevLines)
 			if err != nil {
-				return fmt.Errorf("Claude failed: %s", stderr.String())
+				return fmt.Errorf("agent failed: %s", stderr.String())
 			}
 			return nil
 		case <-ticker.C:
@@ -491,7 +491,7 @@ func WaitWithPanel(cmd *exec.Cmd, title, activity string, stderr *bytes.Buffer)
 		case err := <-done:
 			clearPanelLines(prevLines)
 			if err != nil {
-				return fmt.Errorf("Claude failed: %s", stderr.String())
+				return fmt.Errorf("agent failed: %s", stderr.String())
 			}
 			return nil
 		case <-ticker.C: