Repository navigation
Conversation
…adata A step's tool context carries the user message the step answers (promptID, the newest user message in the loaded history) and the Session's parent when it is a subagent's (parentID). The shell tool echoes both onto the shell's metadata, so a shell.create.before hook can attribute the shell to its turn without a lookup: today it sees only the assistant row and the tool call.
|
This PR doesn't fully meet our contributing guidelines and PR template. What needs to be fixed:
Please edit this PR description to address the above within 2 hours, or it will be automatically closed. If you believe this was flagged incorrectly, please let a maintainer know. |
Collaborator
Author
|
Superseded: the trace plugin can read the prompt itself through the plugin session API (ctx.session.get / ctx.session.context), which it now does. No fork change needed. |
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
1 participant
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.
A step's tool context now carries
promptID(the newest user message in the loaded history, i.e. the prompt the step answers) andparentID(the Session's parent, for a subagent). The shell tool echoes both onto the shell's metadata, so ashell.create.beforehook can attribute a shell to its turn directly. Today the hook sees only the assistant row and the tool call, and Script.it's trace plugin has to hand the assistant row to the bridge to find the prompt.Both fields are optional additions to
Tool.Context; no protocol or client surface changes.Tested:
bun typecheckin core, schema and plugin;bun run lint;tool-shell(background shell metadata),session-runner(tool context carries the prompt, no parent on a top-level Session). The 6 shell-syntax failures intool-shell.test.tson this Mac fail on the unmodified branch too.A complementary Script.it change makes the trace plugin write
message_idfrompromptIDfor a top-level session, and keeps the assistant-row path for subagents.