fix(agent): fix OpenCode double-prompt, Claude ConvertCommand, and parser event timing

- Remove duplicate stdin in OpenCode ConvertCommand and FixJSONCommand
  where the prompt was passed both as a CLI argument and via stdin
- Revert Claude ConvertCommand to use `-p` with stdin instead of the
  incorrect `-p --tools ""` which passed an empty string as a separate arg
- Fix OpenCode parser to emit EventToolStart when tools begin (not only
  on completion) and EventToolResult when completed, so the TUI shows
  activity during tool execution

Author: MiniCodeMonkey

internal/agent/claude.go

@@ -49,7 +49,7 @@ func (p *ClaudeProvider) InteractiveCommand(workDir, prompt string) *exec.Cmd {
 
 // ConvertCommand implements loop.Provider.
 func (p *ClaudeProvider) ConvertCommand(workDir, prompt string) (*exec.Cmd, loop.OutputMode, string, error) {
-	cmd := exec.Command(p.cliPath, "-p", "--tools", "")
+	cmd := exec.Command(p.cliPath, "-p")
 	cmd.Dir = workDir
 	cmd.Stdin = strings.NewReader(prompt)
 	return cmd, loop.OutputStdout, "", nil

internal/agent/opencode.go

@@ -39,13 +39,11 @@ func (p *OpenCodeProvider) InteractiveCommand(workDir, prompt string) *exec.Cmd
 func (p *OpenCodeProvider) ConvertCommand(workDir, prompt string) (*exec.Cmd, loop.OutputMode, string, error) {
 	cmd := exec.Command(p.cliPath, "run", "--format", "json", "--", prompt)
 	cmd.Dir = workDir
-	cmd.Stdin = strings.NewReader(prompt)
 	return cmd, loop.OutputStdout, "", nil
 }
 
 func (p *OpenCodeProvider) FixJSONCommand(prompt string) (*exec.Cmd, loop.OutputMode, string, error) {
 	cmd := exec.Command(p.cliPath, "run", "--format", "json", "--", prompt)
-	cmd.Stdin = strings.NewReader(prompt)
 	return cmd, loop.OutputStdout, "", nil
 }
 
@@ -56,30 +54,32 @@ func (p *OpenCodeProvider) ParseLine(line string) *loop.Event {
 func (p *OpenCodeProvider) LogFileName() string { return "opencode.log" }
 
 // CleanOutput extracts JSON from opencode's NDJSON output format.
+// It looks for the last "text" event line and returns its part.text content.
 func (p *OpenCodeProvider) CleanOutput(output string) string {
 	output = strings.TrimSpace(output)
-
-	if !strings.Contains(output, "\n") || !strings.Contains(output, `"type":"step_start"`) || !strings.Contains(output, `"type":"text"`) {
+	if !strings.Contains(output, "\n") {
 		return output
 	}
 
-	lines := strings.Split(output, "\n")
-	for _, line := range lines {
+	// Parse each line as JSON to find text events (last one wins).
+	var lastText string
+	for _, line := range strings.Split(output, "\n") {
 		line = strings.TrimSpace(line)
 		if line == "" {
 			continue
 		}
-		if strings.Contains(line, `"type":"text"`) {
-			var ev struct {
-				Type string `json:"type"`
-				Part struct {
-					Text string `json:"text"`
-				} `json:"part"`
-			}
-			if json.Unmarshal([]byte(line), &ev) == nil && ev.Part.Text != "" {
-				return ev.Part.Text
-			}
+		var ev struct {
+			Type string `json:"type"`
+			Part struct {
+				Text string `json:"text"`
+			} `json:"part"`
+		}
+		if json.Unmarshal([]byte(line), &ev) == nil && ev.Type == "text" && ev.Part.Text != "" {
+			lastText = ev.Part.Text
 		}
 	}
+	if lastText != "" {
+		return lastText
+	}
 	return output
 }

internal/loop/opencode_parser.go

@@ -78,16 +78,21 @@ func ParseLineOpenCode(line string) *Event {
 		return &Event{Type: EventIterationStart}
 
 	case "tool_use":
-		if ev.Part == nil || ev.Part.State == nil {
+		if ev.Part == nil {
 			return nil
 		}
-		if ev.Part.State.Status == "completed" {
+		if ev.Part.State != nil && ev.Part.State.Status == "completed" {
 			return &Event{
-				Type: EventToolStart,
+				Type: EventToolResult,
 				Tool: ev.Part.Tool,
+				Text: ev.Part.State.Output,
 			}
 		}
-		return nil
+		// Tool starting or in-progress
+		return &Event{
+			Type: EventToolStart,
+			Tool: ev.Part.Tool,
+		}
 
 	case "text":
 		if ev.Part == nil {