Skip to content

refactor(qa-matrix): localize internal declarations#101974

Merged
vincentkoc merged 1 commit into
mainfrom
refactor/deadcode-all-surfaces-20260708-v75
Jul 8, 2026
Merged

refactor(qa-matrix): localize internal declarations#101974
vincentkoc merged 1 commit into
mainfrom
refactor/deadcode-all-surfaces-20260708-v75

Conversation

@vincentkoc

Copy link
Copy Markdown
Member

No description provided.

@vincentkoc vincentkoc self-assigned this Jul 8, 2026
@openclaw-barnacle openclaw-barnacle Bot added size: XS maintainer Maintainer-authored PR labels Jul 8, 2026
@vincentkoc

Copy link
Copy Markdown
Member Author

Land-ready proof for ae59847ec355ece24fbff1f8dabc7624be6399f9:

  • git diff --check: passed
  • Testbox tbx_01kwzs6nps2r158rx25zq5as97: pnpm check:changed passed
  • same Testbox: five touched-module suites passed (5 files, 103 tests)
  • same Testbox: pnpm build passed
  • fresh local autoreview: no findings, confidence 0.92
  • fresh branch autoreview: no findings, confidence 0.87
  • repo-native review-guard, review-validate-artifacts, and prepare-run: passed

The broad QA Matrix run passed 23 files and 220 tests, with one unrelated current-main failure in extensions/qa-matrix/src/cli.test.ts. Its expected list omits seven shared-flow scenarios now present on the Matrix adapter; the exact assertion failure reproduces after restoring clean origin/main.

Behavior is unchanged: all nine declarations remain intact for their local callers; only unused export modifiers are removed.

@vincentkoc
vincentkoc marked this pull request as ready for review July 8, 2026 02:43
@vincentkoc
vincentkoc merged commit 4bdf224 into main Jul 8, 2026
103 of 105 checks passed
@vincentkoc
vincentkoc deleted the refactor/deadcode-all-surfaces-20260708-v75 branch July 8, 2026 02:43
@vincentkoc

Copy link
Copy Markdown
Member Author

Merged via squash.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintainer Maintainer-authored PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant