Skip to content

fix(perstack): handle stoppedByInteractiveTool in TUI execution loop#429

Merged
FL4TLiN3 merged 1 commit intomainfrom
fix/interactive-tool-continue
Jan 18, 2026
Merged

fix(perstack): handle stoppedByInteractiveTool in TUI execution loop#429
FL4TLiN3 merged 1 commit intomainfrom
fix/interactive-tool-continue

Conversation

@FL4TLiN3
Copy link
Contributor

Summary

  • Add stoppedByInteractiveTool to the list of statuses that allow continuing the TUI loop
  • Track when the next query should be treated as an interactive tool result
  • Pass the user's input through parseInteractiveToolCallResult when continuing from an interactive tool stop

Problem

When an expert calls an interactive tool and the run stops with status stoppedByInteractiveTool:

  1. The TUI execution loop would exit immediately instead of allowing the user to continue
  2. Even if continued via CLI flags, the user's input was treated as a regular text query instead of an interactive tool result
  3. This caused the TUI to hang or display incorrectly

Solution

  1. Added stoppedByInteractiveTool to canContinue conditions
  2. Added isNextQueryInteractiveToolResult flag that is set to true when the run stops for interactive tool
  3. Use this flag to determine whether to parse the next query as an interactive tool result

Test plan

  • Local build succeeds
  • Test with an expert that uses interactive tools (e.g., buyer-category-agent)

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@FL4TLiN3 FL4TLiN3 merged commit 9b8950e into main Jan 18, 2026
9 checks passed
@FL4TLiN3 FL4TLiN3 deleted the fix/interactive-tool-continue branch January 18, 2026 23:45
@FL4TLiN3 FL4TLiN3 mentioned this pull request Jan 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants