Skip to content

fix(ci): replace non-resolvable Bun build pin in setup-node-env#34637

Closed
sonwr wants to merge 2 commits into
openclaw:mainfrom
sonwr:fix/ci-bun-version-404
Closed

fix(ci): replace non-resolvable Bun build pin in setup-node-env#34637
sonwr wants to merge 2 commits into
openclaw:mainfrom
sonwr:fix/ci-bun-version-404

Conversation

@sonwr

@sonwr sonwr commented Mar 4, 2026

Copy link
Copy Markdown

Problem

CI job checks (bun, test, pnpm canvas:a2ui:bundle && bunx vitest run --config vitest.unit.config.ts) can fail before tests with:

  • oven-sh/setup-bun@v2
  • bun-version: 1.3.9+cf6cdbbba
  • download URL returns 404

Root cause

The pinned Bun build suffix (+cf6cdbbba) is currently not resolvable from setup-bun release assets.

Fix

Use a resolvable stable version string in the shared setup action:

  • .github/actions/setup-node-env/action.yml
  • bun-version: "1.3.9"

Impact

Restores Bun-based CI jobs so PRs are not blocked by setup-time 404 errors unrelated to code changes.

Ref: #34622 CI failure thread

@greptile-apps

greptile-apps Bot commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR fixes a CI setup failure by stripping a non-resolvable build-commit suffix from the pinned Bun version in the shared setup-node-env composite action, changing "1.3.9+cf6cdbbba""1.3.9".

The fix is correct and minimal — oven-sh/setup-bun@v2 only exposes stable release assets, so the +cf6cdbbba build identifier caused a 404 at download time and blocked every Bun-dependent CI job. No other inputs, steps, or downstream workflows are affected.

Confidence Score: 5/5

  • Safe to merge — this is a one-line, targeted fix to a broken CI version string with no functional impact on application code.
  • The change removes a non-publishable build suffix from a version pin, restoring the correct stable release string. The diff is trivial and the root cause is well-understood. No application code, dependencies, or other workflows are touched.
  • No files require special attention.

Last reviewed commit: ef18f55

@bmendonca3

Copy link
Copy Markdown
Contributor

Objective status snapshot:

  • Current state is CLEAN with no failing checks and high review confidence (5/5).

Actionable next steps:

  1. Consider adding one guard test in the composite action workflow to prevent reintroducing non-resolvable Bun build suffix pins.
  2. If no new failures appear, this is merge-ready from my side.

@dunamismax

Copy link
Copy Markdown
Contributor

This fix would unblock the checks (bun, test, ...) CI job that's currently failing across all open PRs due to the unresolvable Bun build pin. The 404 from oven-sh/setup-bun on 1.3.9+cf6cdbbba is a straightforward version resolution issue — stripping the build suffix to 1.3.9 is the right call. Would love to see this merged so CI goes green again.

@sonwr

sonwr commented Mar 7, 2026

Copy link
Copy Markdown
Author

Thanks for the detailed feedback. I kept this PR intentionally minimal to unblock CI quickly with the Bun version fix; I’m happy to add a separate guard test if maintainers want that follow-up.

@vincentkoc

Copy link
Copy Markdown
Member

Looks like its been fixed already

@vincentkoc vincentkoc closed this Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants