refactor(agents): trim MCP runtime exports#106520
Merged
Merged
Conversation
Contributor
Author
|
Land-ready proof for head
Known proof gap: unrelated broad hosted shards are not required under the user-approved focused CI override. |
wm0018
pushed a commit
to wm0018/openclaw
that referenced
this pull request
Jul 14, 2026
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
Jul 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related: #105595
What Problem This Solves
Resolves ten grandfathered production-unused exports in the agents MCP runtime facade while keeping the actual runtime implementations and tests intact.
Why This Change Was Made
The MCP facade no longer re-exports six types and four runtime helpers with no production consumers. Two helpers used only inside their owner module become module-local, and the focused tests import the two runtime seams they exercise directly from that owner.
User Impact
No user-visible behavior changes. The production dead-export baseline shrinks from 2,541 to 2,531 entries, and
src/agentsshrinks from 402 to 392 entries.Evidence
tbx_01kxe267fak1qjaxqcvfz17dpz, run 29263226425: deadcode regeneration byte-stable at 2,531 total / 392 agents; production deadcode green.src/agents/agent-bundle-mcp-runtime.test.ts: 48/48 tests green.pnpm check:test-typesandpnpm tsgo:coregreen.