Skip to content

refactor(agents): trim MCP runtime exports#106520

Merged
steipete merged 2 commits into
mainfrom
codex/dead-exports-agents-mcp-runtime
Jul 13, 2026
Merged

refactor(agents): trim MCP runtime exports#106520
steipete merged 2 commits into
mainfrom
codex/dead-exports-agents-mcp-runtime

Conversation

@steipete

Copy link
Copy Markdown
Contributor

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/agents shrinks from 402 to 392 entries.

Evidence

  • Blacksmith Testbox tbx_01kxe267fak1qjaxqcvfz17dpz, run 29263226425: deadcode regeneration byte-stable at 2,531 total / 392 agents; production deadcode green.
  • Targeted oxfmt and type-aware oxlint green.
  • src/agents/agent-bundle-mcp-runtime.test.ts: 48/48 tests green.
  • pnpm check:test-types and pnpm tsgo:core green.
  • Stable patch ID unchanged across the final baseline rebase; prior fresh autoreview clean with no actionable findings, 0.96 confidence.
  • AI-assisted; implementation and proof reviewed before submission.

@openclaw-barnacle openclaw-barnacle Bot added scripts Repository scripts agents Agent runtime and tooling size: XS maintainer Maintainer-authored PR labels Jul 13, 2026
@steipete

Copy link
Copy Markdown
Contributor Author

Land-ready proof for head e9d5e058a82f54fd8a494258562fafe9a981939c:

  • Removed ten production-unused src/agents MCP facade/helper exports; integrated baseline is 2,531 total / 392 agents.
  • Canonical MCP types and runtime owner exports remain intact; live CLI, gateway app, UI-resource, materialization, and retirement callers still use the same implementations.
  • Blacksmith Testbox tbx_01kxe267fak1qjaxqcvfz17dpz, run 29263226425: regeneration byte-stable, production deadcode, targeted oxfmt, type-aware oxlint, 48/48 focused tests, pnpm check:test-types, and pnpm tsgo:core green.
  • Stable semantic patch ID survived the final baseline rebase. Fresh semantic autoreview clean at 0.96 confidence. Native review: READY, zero findings.

Known proof gap: unrelated broad hosted shards are not required under the user-approved focused CI override.

@steipete
steipete merged commit f0fe068 into main Jul 13, 2026
132 of 139 checks passed
@steipete
steipete deleted the codex/dead-exports-agents-mcp-runtime branch July 13, 2026 15:49
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents Agent runtime and tooling maintainer Maintainer-authored PR scripts Repository scripts size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant