Skip to content

fix(addie): Remove duplicate validate_adagents tool causing API 400 errors#1005

Merged
bokelley merged 1 commit intomainfrom
bokelley/fix-duplicate-tools
Feb 11, 2026
Merged

fix(addie): Remove duplicate validate_adagents tool causing API 400 errors#1005
bokelley merged 1 commit intomainfrom
bokelley/fix-duplicate-tools

Conversation

@bokelley
Copy link
Copy Markdown
Contributor

Summary

  • Removed duplicate validate_adagents tool definition from member-tools.ts (canonical version stays in property-tools.ts)
  • Added tool name deduplication in claude-client.ts to prevent future duplicates from reaching the API

Context

Production logs showed repeated 400 errors from the Anthropic API:

"tools: Tool names must be unique."

validate_adagents was defined in both MEMBER_TOOLS and PROPERTY_TOOLS. When bolt-app.ts combined them into allTools, the duplicate caused the API to reject every request. The member-tools handler was already dead code — the property-tools handler overwrote it in the allHandlers Map.

Test plan

  • TypeScript compiles cleanly
  • All tests pass (292/293 — 1 pre-existing failure in organization-db unrelated to this change)
  • Verified only one validate_adagents definition remains across all tool files
  • No other duplicate tool names in the codebase

🤖 Generated with Claude Code

…rrors

validate_adagents was defined in both member-tools.ts and property-tools.ts.
When combined in bolt-app.ts, the duplicate tool name caused the Anthropic API
to reject requests with "tools: Tool names must be unique."

Also adds tool deduplication in claude-client.ts to prevent future duplicates.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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