fix(js): support bun-only environments in release-publish executor by AI-JamesHenry · Pull Request #34835 · nrwl/nx

AI-JamesHenry · 2026-03-14T09:56:28Z

Note

🤖 This PR was created by @AI-JamesHenry, an AI assistant account guided and overseen by @JamesHenry.

Current Behavior

nx release publish hardcodes npm view and npm dist-tag add commands, which fails in bun-only environments (e.g., oven/bun Docker images) where npm is not installed, with /bin/sh: 1: npm: not found.

Expected Behavior

nx release publish works in bun-only environments by:

Using bun info instead of npm view for checking existing versions/dist-tags
Gracefully skipping npm dist-tag add when npm is not available (bun has no equivalent)
Showing a clear error message when npm is missing and the package manager is not bun

Changes

npm availability check — Detects npm at executor start via execSync('npm --version'). If npm is missing and PM is not bun, returns a controlled error.
PM-specific view command — Uses bun info / yarn info / pnpm view / npm view based on detected package manager. Only npm/pnpm support field specifiers (versions dist-tags).
Dist-tag guard — Wraps npm dist-tag add section in if (isNpmInstalled) since only npm has this command.
Hardened error handling — Adds fallback parsing for bun's plain-text 404 errors (bun doesn't output JSON errors like npm/pnpm).

This is a clean reimplementation incorporating all review feedback from the original #32252.

Notable: The PackageManagerCommands interface is NOT modified — the view command is built locally in the executor as requested in review feedback.

Related Issue(s)

Fixes #32126

netlify · 2026-03-14T09:56:31Z

✅ Deploy Preview for nx-docs canceled.

Name	Link
🔨 Latest commit	`f7cbfc0`
🔍 Latest deploy log	https://app.netlify.com/projects/nx-docs/deploys/69b530ce5f079200078403bc

netlify · 2026-03-14T09:56:31Z

✅ Deploy Preview for nx-dev canceled.

Name	Link
🔨 Latest commit	`f7cbfc0`
🔍 Latest deploy log	https://app.netlify.com/projects/nx-dev/deploys/69b530cea39d9800085d5e67

nx-cloud · 2026-03-14T09:57:07Z

View your CI Pipeline Execution ↗ for commit 4acfc08

Command	Status	Duration	Result
`nx affected --targets=lint,test,build,e2e,e2e-c...`	✅ Succeeded	50m 7s	View ↗
`nx run-many -t check-imports check-lock-files c...`	✅ Succeeded	3m 45s	View ↗
`nx-cloud record -- nx-cloud conformance:check`	✅ Succeeded	8s	View ↗
`nx-cloud record -- nx format:check`	✅ Succeeded	2s	View ↗
`nx-cloud record -- nx sync:check`	✅ Succeeded	<1s	View ↗

☁️ Nx Cloud last updated this comment at 2026-03-14 16:06:41 UTC

npm version-dependent warnings (e.g., "prefer-frozen-lockfile") cause snapshot flakiness across CI environments.

This reverts commit d8de646.

nx-cloud

Important

At least one additional CI pipeline execution has run since the conclusion below was written and it may no longer be applicable.

Nx Cloud is proposing a fix for your failed CI:

We added a third condition to the bun info error handler in release-publish.impl.ts to recognize bun's plain-text 404 Not Found stderr format as a "package not yet published" signal. Without this, the inner try block would parse an empty stdout as {}, find no E404 code, and incorrectly return success: false before the publish ever ran. This fix allows first-time bun publishes to proceed as expected.

Warning

❌ We could not verify this fix.

diff --git a/packages/js/src/executors/release-publish/release-publish.impl.ts b/packages/js/src/executors/release-publish/release-publish.impl.ts
index 69fb67b0a2..748ab96099 100644
--- a/packages/js/src/executors/release-publish/release-publish.impl.ts
+++ b/packages/js/src/executors/release-publish/release-publish.impl.ts
@@ -282,6 +282,10 @@ Please update the local dependency on "${depName}" to be a valid semantic versio
           !(
             err.stderr?.toString().includes('E404') &&
             err.stderr?.toString().toLowerCase().includes('not found')
+          ) &&
+          !(
+            err.stderr?.toString().includes('404') &&
+            err.stderr?.toString().toLowerCase().includes('not found')
           )
         ) {
           console.error(

🔔 Heads up, your workspace has pending recommendations ↗ to auto-apply fixes for similar failures.

Because this branch comes from a fork, it is not possible for us to apply fixes directly, but you can apply the changes locally using the available options below.

Apply changes locally with:

npx nx-cloud apply-locally P7NN-0cbK

Apply fix locally with your editor ↗ View interactive diff ↗

_{🎓 Learn more about Self-Healing CI on nx.dev}

Only switch to bun info when pm is bun. Keep npm view for npm/pnpm/yarn since detectPackageManager() can return pnpm in CI even when the project uses npm (due to npm_config_user_agent from the parent pnpm process).

github-actions · 2026-03-24T00:14:18Z

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

fix(js): support bun-only environments in release-publish executor

f7cbfc0

This comment was marked as outdated.

Sign in to view

fix(release): filter npm warnings from e2e snapshot serializers

d8de646

npm version-dependent warnings (e.g., "prefer-frozen-lockfile") cause snapshot flakiness across CI environments.

This comment was marked as outdated.

Sign in to view

Revert "fix(release): filter npm warnings from e2e snapshot serializers"

021dbce

This reverts commit d8de646.

This comment was marked as outdated.

Sign in to view

AI-JamesHenry force-pushed the 32126 branch from 9cf748a to 679e01b Compare March 14, 2026 14:07

nx-cloud Bot reviewed Mar 14, 2026

View reviewed changes

fix(js): use npm view for all PMs except bun in release-publish

4acfc08

Only switch to bun info when pm is bun. Keep npm view for npm/pnpm/yarn since detectPackageManager() can return pnpm in CI even when the project uses npm (due to npm_config_user_agent from the parent pnpm process).

AI-JamesHenry force-pushed the 32126 branch from 679e01b to 4acfc08 Compare March 14, 2026 15:12

AI-JamesHenry marked this pull request as ready for review March 14, 2026 16:10

AI-JamesHenry requested a review from a team as a code owner March 14, 2026 16:10

AI-JamesHenry requested a review from FrozenPandaz March 14, 2026 16:10

JamesHenry approved these changes Mar 14, 2026

View reviewed changes

JamesHenry merged commit f1735bd into nrwl:master Mar 14, 2026
16 of 17 checks passed

JamesHenry mentioned this pull request Mar 14, 2026

fix(release): do not require the npm CLI to be installed for publishing JS packages with bun #32252

Closed

github-actions Bot locked as resolved and limited conversation to collaborators Mar 24, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(js): support bun-only environments in release-publish executor#34835

fix(js): support bun-only environments in release-publish executor#34835
JamesHenry merged 4 commits into
nrwl:masterfrom
AI-JamesHenry-Org:32126

AI-JamesHenry commented Mar 14, 2026

Uh oh!

netlify Bot commented Mar 14, 2026 •

edited

Loading

Uh oh!

netlify Bot commented Mar 14, 2026 •

edited

Loading

Uh oh!

nx-cloud Bot commented Mar 14, 2026 •

edited

Loading

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

nx-cloud Bot left a comment •

edited

Loading

Uh oh!

Uh oh!

github-actions Bot commented Mar 24, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

AI-JamesHenry commented Mar 14, 2026

Current Behavior

Expected Behavior

Changes

Related Issue(s)

Uh oh!

netlify Bot commented Mar 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for nx-docs canceled.

Uh oh!

netlify Bot commented Mar 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Deploy Preview for nx-dev canceled.

Uh oh!

nx-cloud Bot commented Mar 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

This comment was marked as outdated.

Uh oh!

nx-cloud Bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Nx Cloud is proposing a fix for your failed CI:

Uh oh!

Uh oh!

github-actions Bot commented Mar 24, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

netlify Bot commented Mar 14, 2026 •

edited

Loading

netlify Bot commented Mar 14, 2026 •

edited

Loading

nx-cloud Bot commented Mar 14, 2026 •

edited

Loading

nx-cloud Bot left a comment •

edited

Loading