Skip to content

fix(test): declare release verifier helpers#107444

Merged
steipete merged 1 commit into
mainfrom
fix/release-verifier-test-declarations
Jul 14, 2026
Merged

fix(test): declare release verifier helpers#107444
steipete merged 1 commit into
mainfrom
fix/release-verifier-test-declarations

Conversation

@steipete

Copy link
Copy Markdown
Contributor

What Problem This Solves

The full test typecheck on current main fails because three release-verifier helpers added by a2790f10af1 are exported and used by tests but absent from the ambient declaration shim.

Why This Change Was Made

Declare the existing exports in the owning external-script module shim. This restores the test graph without changing release runtime behavior.

User Impact

No product behavior change. Maintainer CI and release-note verifier tests typecheck again.

Evidence

  • Blacksmith Testbox tbx_01kxg8268xrrtc6dt2w24f04q8: pnpm check:test-types passed.
  • Same Testbox: pnpm test test/scripts/verify-release-notes.test.ts passed, 28/28 tests.
  • git diff --check passed.
  • Fresh autoreview: no accepted/actionable findings.

@openclaw-barnacle openclaw-barnacle Bot added size: XS maintainer Maintainer-authored PR labels Jul 14, 2026
@steipete

Copy link
Copy Markdown
Contributor Author

Land-ready proof:

  • Exact-head check-test-types passed in CI run 29331239976.
  • Blacksmith Testbox tbx_01kxg8268xrrtc6dt2w24f04q8: full pnpm check:test-types passed; focused release-verifier suite passed 28/28.
  • git diff --check passed.
  • Fresh autoreview: no accepted/actionable findings.
  • Native review artifacts: READY FOR /prepare-pr, zero findings.

The run's only failure is check-lint in three files already present on the PR base (packages/sdk/src/index.e2e.test.ts, packages/gateway-client/src/client.ts, and scripts/lib/gateway-ws-client.ts). This one-file declaration PR does not touch them. Admin squash merge is therefore bounded to the proven test-type repair.

@steipete
steipete merged commit dde6f81 into main Jul 14, 2026
88 of 92 checks passed
@steipete
steipete deleted the fix/release-verifier-test-declarations branch July 14, 2026 12:12
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request Jul 15, 2026
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