Skip to content

fix(mcp): split tool search registration schema - #5272

Merged
diegosouzapw merged 1 commit into
cheaper-inference:feat/mcp-tool-searchfrom
KooshaPari:fix/5269-mcp-tool-search-filesize
Jun 28, 2026
Merged

diegosouzapw merged 1 commit into
cheaper-inference:feat/mcp-tool-searchfrom
KooshaPari:fix/5269-mcp-tool-search-filesize

Conversation

@KooshaPari

Copy link
Copy Markdown
Contributor

Unblocks #5269 Fast Quality by moving the new tool_search schema/tool definition and server registration into focused modules.

Why:

  • Fast Quality failed on frozen file-size caps:
    • open-sse/mcp-server/schemas/tools.ts 1541 > 1497
    • open-sse/mcp-server/server.ts 1565 > 1555
  • This keeps exported schema names and registration behavior intact while reducing:
    • tools.ts to 1493 lines
    • server.ts to 1549 lines

Validation:

  • npm run check:file-size
  • npx vitest run open-sse/mcp-server/tests/toolSearch.catalog.test.ts open-sse/mcp-server/tests/toolSearch.search.test.ts open-sse/mcp-server/tests/toolSearch.signature.test.ts open-sse/mcp-server/tests/toolSearch.tool.test.ts (14/14 pass)
  • npx eslint open-sse/mcp-server/schemas/tools.ts open-sse/mcp-server/schemas/toolSearch.ts open-sse/mcp-server/server.ts open-sse/mcp-server/toolSearch/register.ts (0 errors; existing server any warnings only)

@KooshaPari
KooshaPari requested a review from diegosouzapw as a code owner June 28, 2026 23:31
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@diegosouzapw
diegosouzapw merged commit 3853f17 into cheaper-inference:feat/mcp-tool-search Jun 28, 2026
1 check passed
@KooshaPari
KooshaPari deleted the fix/5269-mcp-tool-search-filesize branch July 2, 2026 22:10
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