feat: enable writable hook properties for tool and result modification - #13
Open
github-actions[bot] wants to merge 13 commits into
Open
feat: enable writable hook properties for tool and result modification#13github-actions[bot] wants to merge 13 commits into
github-actions[bot] wants to merge 13 commits into
Conversation
Added a checklist for PR descriptions to ensure clarity and relevance.
Added a checklist for testing best practices.
Added a checklist item to ensure the PR description includes an overview of the feature and key implementation decisions.
Clarified guidelines for pull request creation and checklist inclusion.
Allow hook callbacks to modify event properties to change agent behavior, achieving parity with the Python SDK. Hook callbacks can now modify: - BeforeToolCallEvent.tool - Change which tool gets executed - BeforeToolCallEvent.toolUse - Modify tool parameters before execution - AfterToolCallEvent.result - Transform tool results before sending to model This enables use cases like tool routing, parameter validation, result transformation, and conditional behavior changes without modifying tool implementations. Resolves #12
zastrowm
force-pushed
the
main
branch
3 times, most recently
from
December 8, 2025 19:32
985eb50 to
44c2ca0
Compare
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.
Motivation
The Python SDK allows hook callbacks to modify event properties to change agent behavior (e.g., switching tools, modifying parameters, transforming results), but the TypeScript SDK marked these properties as readonly. This prevented users from implementing similar behavior modification patterns and created an API inconsistency between the two SDKs.
Resolves: #12
Public API Changes
The following hook event properties are now writable, allowing hook callbacks to modify agent behavior:
BeforeToolCallEvent:
AfterToolCallEvent:
The changes are backward compatible. Existing hooks that only read properties continue to work without modification.
Use Cases
Implementation Notes
The agent now reads potentially modified values after yielding hook events:
BeforeToolCallEventand capture modifiedtoolandtoolUseAfterToolCallEventand capture modifiedresultThis follows the same pattern as the existing
AfterModelCallEvent.retryModelCallproperty.