fix(agents): declare each tool name once per request #96

Merged

peersky merged 2 commits from fix/duplicate-tool-declaration into dev 2026-09-05 04:14:25 +00:00

Conversation 0 Commits 2 Files changed 5 +284 -2

peersky commented 2026-09-05 03:52:42 +00:00

Owner

Copy link

• Fixed Bug two sources could claim one name, a room's user tool called search_web beside the agent's own, and Gemini refuses such a request with 400 "Duplicate function declaration found", failing the round
• Added Redundancy the first registration wins, so a user tool cannot quietly replace what the agent already serves under that name, and the dropped one is logged
• Added Function a test that a colliding user tool leaves exactly one declaration standing

- Fixed Bug two sources could claim one name, a room's user tool called search_web beside the agent's own, and Gemini refuses such a request with 400 "Duplicate function declaration found", failing the round - Added Redundancy the first registration wins, so a user tool cannot quietly replace what the agent already serves under that name, and the dropped one is logged - Added Function a test that a colliding user tool leaves exactly one declaration standing

peersky added 1 commit 2026-09-05 03:52:42 +00:00

fix(agents): declare each tool name once per request ... 031c6d30f9

- Fixed Bug two sources could claim one name, a room's user tool called search_web beside the agent's own, and Gemini refuses such a request with 400 "Duplicate function declaration found", failing the round
- Added Redundancy the first registration wins, so a user tool cannot quietly replace what the agent already serves under that name, and the dropped one is logged
- Added Function a test that a colliding user tool leaves exactly one declaration standing

peersky added 1 commit 2026-09-05 04:12:03 +00:00

fix(agents): refuse a config that declares one search tool twice ... 91f004b17f

- Fixed Bug where a seat setting both `delegated_search` and `provider_executed_tools` declared one tool name twice in the request, so the backend answered `Duplicate function declaration found` and every task failed, not only the searches
- Added Function `AgentConfig::validate_search_tools`, refusing the pair rather than deduplicating it: the two entries mean different things, so dropping one silently yields an agent that starts, offers the name and calls the other thing
- Added Ability to catch the same pair in the launch path, because an agent configured from the registry never passes through the config loader
- Added Function `validate_tool_names`, refusing an agent whose extra-context, sandbox and delegated-search tools collide on a name
- Improved Interface docs in `reference/provider-tools.md` with the refusal text and what to clear

peersky merged commit df1c252f4f into dev 2026-09-05 04:14:25 +00:00

peersky deleted branch fix/duplicate-tool-declaration 2026-09-05 04:14:26 +00:00

peersky referenced this pull request from a commit 2026-09-05 04:14:26 +00:00

Merge pull request 'fix(agents): declare each tool name once per request' (#96) from fix/duplicate-tool-declaration into dev

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

release

Automated release PR

No labels

release

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

peeramid-labs/quorum-rs!96

No description provided.