Skip to content

Add zarvis harness hooks#160

Merged
edwin-zvs merged 1 commit into
mainfrom
zarvis-hooks
May 23, 2026
Merged

Add zarvis harness hooks#160
edwin-zvs merged 1 commit into
mainfrom
zarvis-hooks

Conversation

@edwin-zvs

@edwin-zvs edwin-zvs commented May 23, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add explicit opt-in zarvis hook config via AGENTD_ZARVIS_HOOKS_CONFIG or AGENTD_ZARVIS_HOOKS_JSON
  • emit notification hooks for session_start, user_prompt_submit, pre_tool_use, post_tool_use, tool_approval_request, and session_stop from headless and interactive zarvis paths
  • add separate mutating hooks: user_prompt_mutate can rewrite the prompt and pre_tool_use_mutate can rewrite tool args before display/approval/execution
  • document the hook config and add focused tests for config loading, stdin JSON delivery, and mutating stdout

Refs #52

Tests

  • cargo test -p agentd-adapter-zarvis hooks -- --nocapture
  • cargo check -p agentd-adapter-zarvis
  • cargo build

@edwin-zvs
edwin-zvs merged commit 8e92c0c into main May 23, 2026
1 check passed
@edwin-zvs
edwin-zvs deleted the zarvis-hooks branch May 23, 2026 04:58
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.

1 participant