fix(tools): namespace the delegated-search tool so nothing else can claim its name #97

Merged

peersky merged 1 commit from fix/delegated-search-tool-name into dev 2026-09-05 06:01:24 +00:00

Conversation 0 Commits 1 Files changed 2 +52 -10

peersky commented 2026-09-05 06:01:05 +00:00

Owner

Copy link

• Fixed Bug where the delegated-search tool was named search_web, the same name this workspace's job fixtures use as their example user tool, so a room copying them sent two functions of one name and every round 400'd
• Improved Interface by renaming the tool to nsed_delegated_search, which neither a gateway translating a backend's own search tool nor a room naming its user_tools will reach for
• Added Function test pinning the name as namespaced and distinct from names a backend or a room may already use
• Improved Interface docs, which named the tool web_search and had never matched the code

- Fixed Bug where the delegated-search tool was named `search_web`, the same name this workspace's job fixtures use as their example user tool, so a room copying them sent two functions of one name and every round 400'd - Improved Interface by renaming the tool to `nsed_delegated_search`, which neither a gateway translating a backend's own search tool nor a room naming its `user_tools` will reach for - Added Function test pinning the name as namespaced and distinct from names a backend or a room may already use - Improved Interface docs, which named the tool `web_search` and had never matched the code

peersky added 1 commit 2026-09-05 06:01:05 +00:00

fix(tools): namespace the delegated-search tool so nothing else can claim its name ... cdd80e4612

- Fixed Bug where the delegated-search tool was named `search_web`, the same name this workspace's job fixtures use as their example user tool, so a room copying them sent two functions of one name and every round 400'd
- Improved Interface by renaming the tool to `nsed_delegated_search`, which neither a gateway translating a backend's own search tool nor a room naming its `user_tools` will reach for
- Added Function test pinning the name as namespaced and distinct from names a backend or a room may already use
- Improved Interface docs, which named the tool `web_search` and had never matched the code

peersky merged commit ffb1da4547 into dev 2026-09-05 06:01:24 +00:00

peersky deleted branch fix/delegated-search-tool-name 2026-09-05 06:01:24 +00:00

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

Merge pull request 'fix(tools): namespace the delegated-search tool so nothing else can claim its name' (#97) from fix/delegated-search-tool-name 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!97

No description provided.