Skip to content

feat(agents): non-Anthropic tool-call validation + repair layer#68

Merged
dgarson merged 1 commit intodgarson/forkfrom
roman/tool-reliability
Feb 23, 2026
Merged

feat(agents): non-Anthropic tool-call validation + repair layer#68
dgarson merged 1 commit intodgarson/forkfrom
roman/tool-reliability

Conversation

@dgarson
Copy link
Owner

@dgarson dgarson commented Feb 22, 2026

Summary

Adds a non-Anthropic compatibility layer for tool calls by introducing:

  • src/agents/tool-call-validator.ts
  • src/agents/tool-call-repair.ts

These are wired to normalize and recover malformed tool-call payloads seen in non-Anthropic providers.

What quirks this handles

  • MiniMax M2.5: provider responses that emit incomplete or schema-divergent tool-call objects.
  • GLM-5: argument formatting and shape inconsistencies in tool-call payloads.
  • Grok 4: tool-call envelope/field irregularities that break strict downstream parsing.

Why this PR

This work is already implemented and committed on roman/tool-reliability; this PR is for review/merge into the active megabranch.

Verification

  • Run Monday discovery verification before merge.
  • Confirm tool-call execution succeeds across MiniMax M2.5, GLM-5, and Grok 4 on representative scenarios.

Notes

  • Scope is limited to the compatibility validation + repair layer (no unrelated code changes).

dgarson added a commit that referenced this pull request Feb 22, 2026
- AgentDiffViewer: unified/split diff of agent SOUL snapshots, computed line diff, hide unchanged toggle, swap controls
- MCPInspector: 8 MCP tools with schema browser, param types, invocation history, error rate, kind filters
@dgarson dgarson changed the base branch from feat/tool-reliability-layer to dgarson/fork February 22, 2026 19:15
@dgarson dgarson force-pushed the roman/tool-reliability branch from 5620cf7 to f0fc617 Compare February 23, 2026 00:28
@dgarson dgarson merged commit 2558a97 into dgarson/fork Feb 23, 2026
2 of 9 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.

1 participant