Skip to content

fix(tools): enforce registry result contract - #61787

Merged
teknium1 merged 1 commit into
mainfrom
fix/tool-registry-result-contract
Jul 10, 2026
Merged

fix(tools): enforce registry result contract#61787
teknium1 merged 1 commit into
mainfrom
fix/tool-registry-result-contract

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Summary

The registry now accepts strings and the supported multimodal envelope, converting unsupported handler return types into structured JSON contract errors.

Changes

  • Implement the behavior at the existing subsystem chokepoint rather than adding a parallel runtime.
  • Preserve current prompt caching, message-role alternation, profile isolation, and provider wire compatibility.
  • Add focused regression coverage for the corrected success, failure, interruption, or restart paths.

Validation

44 registry tests + 207 model-tools/computer-use tests passed.

tests/tools/test_registry.py | 64 ++++++++++++++++++++++++++++++++++++++++++++
 tools/registry.py            | 41 +++++++++++++++++++++++++---
 2 files changed, 102 insertions(+), 3 deletions(-)

Infographic

05-registry-contract

@alt-glitch alt-glitch added type/bug Something isn't working comp/tools Tool registry, model_tools, toolsets P3 Low — cosmetic, nice to have labels Jul 10, 2026
@teknium1
teknium1 force-pushed the fix/tool-registry-result-contract branch from f763649 to 59d1bc8 Compare July 10, 2026 04:22
@teknium1
teknium1 merged commit f8361d2 into main Jul 10, 2026
31 checks passed
@teknium1
teknium1 deleted the fix/tool-registry-result-contract branch July 10, 2026 04:32
santhreal pushed a commit to santhreal/hermes-agent that referenced this pull request Jul 13, 2026
justemu pushed a commit to justemu/hermes-agent that referenced this pull request Jul 18, 2026
Gravezzz pushed a commit to Gravezzz/hermes-agent that referenced this pull request Jul 21, 2026
leewenjie pushed a commit to leewenjie/hermes-agent that referenced this pull request Aug 7, 2026
randlee pushed a commit to randlee/hermes-agent that referenced this pull request Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/tools Tool registry, model_tools, toolsets P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants