Skip to content

Fix publish of CLI through npm#9573

Merged
tobias-tengler merged 1 commit intomainfrom
tte/fix-npm-publish
Apr 17, 2026
Merged

Fix publish of CLI through npm#9573
tobias-tengler merged 1 commit intomainfrom
tte/fix-npm-publish

Conversation

@tobias-tengler
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings April 17, 2026 11:04
@tobias-tengler tobias-tengler merged commit 8ae8c59 into main Apr 17, 2026
132 checks passed
@tobias-tengler tobias-tengler deleted the tte/fix-npm-publish branch April 17, 2026 11:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Nitro CLI npm packaging docs and adjusts the release workflow steps used to publish the CLI and its platform-specific binary packages to npm.

Changes:

  • Expanded @chillicream/nitro README with usage examples and links to the official GitHub Actions.
  • Added minimal READMEs to each platform-specific @chillicream/nitro-<rid> npm package directory.
  • Simplified the release workflow’s npm publish matrix and publishing commands.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/Nitro/CommandLine/src/npm/chillicream-nitro/README.md Expands CLI usage documentation and points users to GitHub Actions.
src/Nitro/CommandLine/src/npm/chillicream-nitro-win-x86/README.md Adds a README clarifying it’s a platform binary package.
src/Nitro/CommandLine/src/npm/chillicream-nitro-win-x64/README.md Adds a README clarifying it’s a platform binary package.
src/Nitro/CommandLine/src/npm/chillicream-nitro-osx-x64/README.md Adds a README clarifying it’s a platform binary package.
src/Nitro/CommandLine/src/npm/chillicream-nitro-osx-arm64/README.md Adds a README clarifying it’s a platform binary package.
src/Nitro/CommandLine/src/npm/chillicream-nitro-linux-x64/README.md Adds a README clarifying it’s a platform binary package.
src/Nitro/CommandLine/src/npm/chillicream-nitro-linux-musl-x64/README.md Adds a README clarifying it’s a platform binary package.
src/Nitro/CommandLine/src/npm/chillicream-nitro-linux-arm64/README.md Adds a README clarifying it’s a platform binary package.
.github/workflows/release.yml Adjusts npm publishing job matrix and dist-tag behavior.
Comments suppressed due to low confidence (1)

.github/workflows/release.yml:448

  • The publish-nitro-cli-npm job now always publishes @chillicream/nitro with --tag latest. Since the workflow triggers on any 16.* tag (including pre-releases like 16.0.0-rc.1), this would publish pre-releases as the latest dist-tag and effectively override the stable channel. Reintroduce dist-tag computation based on GIT_TAG (stable -> latest, -rc.* -> rc, otherwise -> preview) and publish with that computed tag.
            }
          ' package.json > package.json.tmp
          mv package.json.tmp package.json

      - name: 📦 Create tarball

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 365 to +369
run: |
npm publish "./chillicream-nitro-${{ matrix.rid }}-${GIT_TAG}.tgz" \
--access public \
--registry="${{ vars.NPM_REGISTRY_URL }}"
--registry="${{ vars.NPM_REGISTRY_URL }}" \
--tag latest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants