Skip to content

Remove tool_search_tool_bm25_20251119 internal tool#334

Closed
Dumbris wants to merge 1 commit intomainfrom
remove-tool-search-bm25
Closed

Remove tool_search_tool_bm25_20251119 internal tool#334
Dumbris wants to merge 1 commit intomainfrom
remove-tool-search-bm25

Conversation

@Dumbris
Copy link
Contributor

@Dumbris Dumbris commented Mar 10, 2026

Summary

  • Remove the tool_search_tool_bm25_20251119 Anthropic-specific tool search tool
  • This tool is redundant with retrieve_tools which provides richer BM25 search results (descriptions, annotations, call_with hints)
  • Pure deletion: 77 lines removed from internal/server/mcp.go, no new code

Test plan

  • go build ./cmd/mcpproxy passes
  • go test ./internal/server/... passes
  • E2E tests pass (tool was not covered by E2E tests)

🤖 Generated with Claude Code

This Anthropic-specific tool search tool is no longer needed. The
retrieve_tools built-in tool provides the same BM25 search functionality
with richer results (descriptions, annotations, call_with hints).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link

Deploying mcpproxy-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 47a50be
Status:🚫  Build failed.

View logs

@Dumbris
Copy link
Contributor Author

Dumbris commented Mar 11, 2026

Closing: this removal was included in PR #335 (commit 4753adf) which has been merged to main.

@Dumbris Dumbris closed this Mar 11, 2026
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.

2 participants