Skip to content

Fix bundle smoke checks and cross-platform CI coverage - #2

Closed
dipeshbabu wants to merge 2 commits into
mainfrom
fix/ci-bundle-smoke
Closed

dipeshbabu wants to merge 2 commits into
mainfrom
fix/ci-bundle-smoke

Conversation

@dipeshbabu

@dipeshbabu dipeshbabu commented Oct 5, 2026 •

Copy link
Copy Markdown
Owner

Upstream replacement: subconscious-systems#2

This PR was created in the fork in error. The fix and its issue are now submitted to upstream.

The npm smoke check executed TypeScript sources through the checkout launcher, allowing broken bundles to pass. CI now runs the generated bundle's version and help commands directly and covers Node 20/22 on Ubuntu and Windows. Claude path test expectations now account for the native drive prefix.

Checks also run on fix/** pushes, allowing subsequent fixes to be verified before opening PRs.

Validation: frozen-lockfile install, typecheck, full tests, npm build, and generated bundle smoke checks. GitHub Actions must pass on this commit before this PR is opened.

Closes #1.

@dipeshbabu dipeshbabu closed this Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI smoke check runs sources instead of the built npm bundle

1 participant