Skip to content

feat(tools): include tool arguments in events#2724

Merged
olimorris merged 1 commit intoolimorris:developfrom
bassamsdata:tool_event_load
Feb 9, 2026
Merged

feat(tools): include tool arguments in events#2724
olimorris merged 1 commit intoolimorris:developfrom
bassamsdata:tool_event_load

Conversation

@bassamsdata
Copy link
Contributor

Description

This PR extends tool-related events to include the tool arguments in their payload. The change is additive and does not affect existing behavior.

Benefits / Use cases

  • Enables extensions such as fs-monitor.nvim to access relevant metadata (e.g. file paths) when tools operate on files.
  • Improves observability of tool execution by exposing command arguments, enabling filtering, targeted notifications, and safety checks (for example, detecting potentially dangerous commands executed via cmd_runner).

Example event output:

ToolStarted │ bufnr: 131 │ id: 5844143 │ cmd: find . -maxdepth 4 -name "*.rs" │ tool: cmd_runner
ToolStarted │ bufnr: 131 │ id: 1017944 │ filepath: hive/src/config/loader.rs │ explanation: Refactoring configuration loading to improve error handling and reduce duplicate parsing logic │ tool: insert_edit_into_file

Thank you

AI Usage

None

Related Issue(s)

None.

Screenshots

Checklist

  • I've read the contributing guidelines and have adhered to them in this PR
  • I confirm that this PR has been majority created by me, and not AI (unless stated in the "AI Usage" section above)
  • I've run make all to ensure docs are generated, tests pass and StyLua has formatted the code
  • (optional) I've added test coverage for this fix/feature
  • (optional) I've updated the README and/or relevant docs pages

@olimorris olimorris merged commit f862c5f into olimorris:develop Feb 9, 2026
5 checks passed
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