Skip to content

refactor(deadcode): tighten Anthropic extension roots#108547

Merged
steipete merged 1 commit into
mainfrom
codex/harden-extension-root-deadcode-anthropic
Jul 16, 2026
Merged

refactor(deadcode): tighten Anthropic extension roots#108547
steipete merged 1 commit into
mainfrom
codex/harden-extension-root-deadcode-anthropic

Conversation

@steipete

Copy link
Copy Markdown
Contributor

What Problem This Solves

The generic plugin workspace pattern treated every Anthropic source file as a Knip entry, hiding 24 unused exports from the hard-zero dead-code gate.

Why This Change Was Made

This switches Anthropic and Anthropic Vertex to explicit runtime entries, removes or privatizes the newly exposed internal surfaces, and migrates tests to retained provider, catalog, stream, and usage boundaries. No plugin API or user behavior changes.

User Impact

No user-visible impact. The production dead-code configuration now checks Anthropic implementation files instead of implicitly exempting them.

Evidence

  • Production Knip: 24 hidden Anthropic exports exposed, then exact zero with no additions
  • Anthropic and Anthropic Vertex: 239 tests passed
  • Focused regression rerun: 30 tests passed
  • Formatting and scoped type-aware lint: passed
  • Core and extension production types: passed
  • Test types: Anthropic clean; shared dirty-checkout run stopped only on unrelated concurrent OpenCode test imports
  • Fresh sanitized autoreview: clean, no findings, confidence 0.93
  • Final signed rebased head has the same stable patch ID as the reviewed commit

@steipete
steipete merged commit 169ed48 into main Jul 16, 2026
33 of 34 checks passed
@steipete
steipete deleted the codex/harden-extension-root-deadcode-anthropic branch July 16, 2026 01:35
@openclaw-barnacle openclaw-barnacle Bot added size: M maintainer Maintainer-authored PR labels Jul 16, 2026
steipete added a commit that referenced this pull request Jul 16, 2026
…oots

The strict knip workspaces landed in #108547 flagged seven files. Two are
convention-resolved and become explicit entries (discord configured-state via
the package-state probes, qa-lab cli via the SDK facade basename resolver);
the other five were dead barrels/wrappers superseded by canonical modules and
are deleted (browser bridge/cdp barrels, discord timeouts wrapper, openai
register.runtime, qa-lab model-selection wrapper).
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 16, 2026
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 16, 2026
…oots

The strict knip workspaces landed in openclaw#108547 flagged seven files. Two are
convention-resolved and become explicit entries (discord configured-state via
the package-state probes, qa-lab cli via the SDK facade basename resolver);
the other five were dead barrels/wrappers superseded by canonical modules and
are deleted (browser bridge/cdp barrels, discord timeouts wrapper, openai
register.runtime, qa-lab model-selection wrapper).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant