Skip to content

Expose mediated tool hook surfaces#280

Merged
chubes4 merged 2 commits into
mainfrom
issue/259-mediated-tool-hooks
Jun 3, 2026
Merged

Expose mediated tool hook surfaces#280
chubes4 merged 2 commits into
mainfrom
issue/259-mediated-tool-hooks

Conversation

@chubes4
Copy link
Copy Markdown
Contributor

@chubes4 chubes4 commented Jun 3, 2026

Summary

  • Adds a host-neutral agents_api_pre_tool_call_decision filter around mediated tool preflight while preserving the existing pre_tool_mediator option path.
  • Adds explicit pending preflight decisions that normalize to the canonical runtime-tool pending request/status and stop the loop as incomplete.
  • Adds post_tool_result_diagnostics plus agents_api_mediated_tool_result_diagnostics for audit-oriented host diagnostics without storing raw tool parameters.

Closes #259.

Testing

  • php tests/conversation-loop-tool-execution-smoke.php
  • composer phpstan
  • composer smoke

AI assistance

  • AI assistance: Yes
  • Tool(s): OpenCode (openai/gpt-5.5)
  • Used for: Implemented the mediated-loop hook surfaces, pending external-tool decision handling, diagnostics seam, and smoke test coverage; Chris remains responsible for review and merge.

@chubes4 chubes4 merged commit d9ba419 into main Jun 3, 2026
2 checks passed
@chubes4 chubes4 deleted the issue/259-mediated-tool-hooks branch June 3, 2026 17:20
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.

Expose mediated tool preflight and external-tool pending hooks

1 participant