ci(sdk-smoke): pin the nightly to the published v17 SDK 4.3.0 (fixes #389) - #395
Open
dcccrypto wants to merge 1 commit into
Open
ci(sdk-smoke): pin the nightly to the published v17 SDK 4.3.0 (fixes #389)#395dcccrypto wants to merge 1 commit into
dcccrypto wants to merge 1 commit into
Conversation
The nightly has failed every night since at least 23 July — 15 of 42 smoke
tests, with CrankAction undefined ('Cannot read properties of undefined
(reading FeeSweep)') and the v17 helpers missing entirely.
Cause: the job pinned SDK_VERSION to 2.0.5 while tests/sdk-smoke.test.ts targets
the v17 API surface. #389 recorded this as unfixable at the time — 'tests target
v17 SDK 3.0.0, which was never published (latest npm is 2.0.9)'. That premise
has since expired: **4.3.0 is now on npm** (percolator-launch already installs
it from npm rather than a git SHA), so the smoke test finally has a real
published v17 version to run against.
Verified before changing CI rather than bumping and hoping: installed
@percolatorct/sdk@4.3.0 from npm into a scratch project and ran the smoke file
against it — **42/42 pass**.
Bumps the env default and the job-name fallback. The line-28 comment keeps 2.0.5
as a repository_dispatch example — it is illustrative, not a live default.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe SDK smoke-test workflow’s default SDK version and displayed fallback version were updated from ChangesSDK smoke version update
Estimated code review effort: 1 (Trivial) | ~2 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Fixes #389.
The failure
The nightly SDK publish smoke has failed every night since at least 23 July — 6 consecutive runs, all
scheduleonmain:CrankActionis undefined and the v17 helpers are missing, because the job pinnedSDK_VERSION: "2.0.5"whiletests/sdk-smoke.test.tstargets the v17 API surface.Why it is fixable now
#389 recorded this as blocked — "tests target v17 SDK 3.0.0, which was never published (latest npm is 2.0.9)". That premise has expired.
npm view @percolatorct/sdk versionsnow returns:4.3.0is published (percolator-launch already installs it from npm instead of a pinned git SHA), so the smoke job finally has a real published v17 version to validate against.Verification — checked before touching CI, not after
Installed
@percolatorct/sdk@4.3.0from npm into a scratch project and ran the smoke file against it:versus 15 failures on the pinned 2.0.5. So this turns the nightly green and makes it actually validate the version the keeper depends on, instead of a pre-v17 artefact.
Scope
Two lines: the
SDK_VERSIONenv default and the job-name fallback. The2.0.5on line 28 is deliberately left — it is arepository_dispatchusage example in a comment, not a live default.🤖 Generated with Claude Code
Summary by CodeRabbit