Skip to content

Reduce MCP tool-list trace volume - #31790

Merged
jif-oai merged 1 commit into
mainfrom
jif/reduce-mcp-tool-list-logs
Jul 9, 2026
Merged

Reduce MCP tool-list trace volume#31790
jif-oai merged 1 commit into
mainfrom
jif/reduce-mcp-tool-list-logs

Conversation

@jif-oai

@jif-oai jif-oai commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Why

Every MCP tool-list build emitted two normal-path TRACE events per configured server: one before waiting for tools and one after listing them. On active sessions this produced thousands of nearly identical SQLite rows while carrying little information beyond server readiness and tool counts.

What changed

  • Remove the two normal-path per-server TRACE events.
  • Keep the existing per-server trace span for timing and remote trace context.
  • Emit per-server details only when a server's tools are unavailable.
  • Emit one bounded summary per tool-list build with available server, unavailable server, and tool counts.

Successful builds retain the useful aggregate signal without repeating it for every server.

Related to #28224.

@jif-oai
jif-oai requested a review from a team as a code owner July 9, 2026 13:11
@jif-oai
jif-oai merged commit c8d2db9 into main Jul 9, 2026
32 of 35 checks passed
@jif-oai
jif-oai deleted the jif/reduce-mcp-tool-list-logs branch July 9, 2026 13:36
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants