Repository navigation
fix(mcp): agent toolkit fixes from the stage test - #303
Merged
Merged
Conversation
The 10-task test through an OAuth client on stage found four gaps: - A wait after t3_answer, t3_send_message or t3_start_session missed a reply that came before the wait began. Their results now carry the session's event cursor taken before the write, and the next-call hint passes it. - An approval showed the agent's description, not the command. It now shows the exact command (or file) from the run's unfinished tool item. - A turn that ended on background work the agent waits for (a background command, a subagent, a monitor) read as unread; it counts as running, as in the sidebar. - The OAuth short list was about 12,700 tokens. It keeps t3_ui_list_tools and t3_ui_call and drops t3_ui_get_tool and t3_ui_batch (opt in for them), about 8,200 tokens. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
github-actions
Bot
added
vouch:trusted
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
A 10-task test of the agent toolkit, through a real OAuth client on stage, found four gaps:
t3_waitaftert3_answermissed the reply, because the agent finished before the wait began. The next-call hint had no cursor.date > …/date.txt).Fix
t3_answer,t3_send_messageandt3_start_sessiontake the session's event cursor before the write, and return it in the next-call hint and in the JSON.command_executionorfile_changeitem.isAgentActivecounts pending background work that wakes the agent, using the sidebar's own helperbackgroundWorkHoldsCompletion.t3_ui_list_toolsandt3_ui_call(about 8,200 tokens).t3_ui_get_toolandt3_ui_batchare still available withx-t3-mcp-tools: all.node scripts/check-fork-markers.tspasses locally.🤖 Generated with Claude Code