Skip to content

Update release-npm workflow for dryruns - #5634

Closed
rekhoff wants to merge 2 commits into
masterfrom
rekhoff/release-npm-dryrun-fix
Closed

Update release-npm workflow for dryruns#5634
rekhoff wants to merge 2 commits into
masterfrom
rekhoff/release-npm-dryrun-fix

Conversation

@rekhoff

@rekhoff rekhoff commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

This PR addresses the npm dry-run failures observed in recent release-dry-run CI runs, where pnpm publish --dry-run contacts the npm registry and errors out for already-published versions like spacetimedb@2.7.1.

Description of Changes

Updates tools/release/src/targets/npm.rs so that in dry-run mode the npm release target:

  1. runs pnpm run prepublishOnly to execute the build/test/size checks
  2. runs pnpm pack to create the tarball locally

The live pnpm publish --no-git-checks path is unchanged. This also makes the log and error messages accurate for whichever mode is running.

API and ABI breaking changes

No changes.

Expected complexity level and risk

Low. Only the dry-run branch of tools/release/src/targets/npm.rs is changed. The live npm publish path still runs pnpm publish.

Testing

  • cargo clippy -p spacetimedb-release passes.
  • The dry-run path should be exercised by running cargo-release release npm v2.7.1 --dry-run or the release-dry-run workflow on a branch.

@rekhoff rekhoff self-assigned this Jul 31, 2026
@bfops

bfops commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

I didn't realize you had already started a PR for this. Closing in favor of the merged #5636

@bfops bfops closed this Jul 31, 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.

2 participants