Skip to content

fix: install Bun in Dockerfile#284

Merged
steipete merged 3 commits intoopenclaw:mainfrom
loukotal:fix/dockerfile-install-bun
Jan 6, 2026
Merged

fix: install Bun in Dockerfile#284
steipete merged 3 commits intoopenclaw:mainfrom
loukotal:fix/dockerfile-install-bun

Conversation

@loukotal
Copy link
Copy Markdown
Contributor

@loukotal loukotal commented Jan 6, 2026

The build script was updated to use Bun (bun scripts/canvas-a2ui-copy.ts) but the Dockerfile was not updated to install Bun, causing Docker builds to fail with "bun: not found".

🤖 Generated with Claude Code

Bun added in: #278

Maybe it would make sense to swap the whole thing to bun now? Now we are installing bun just to be able to run build

The build script was updated to use Bun (`bun scripts/canvas-a2ui-copy.ts`)
but the Dockerfile was not updated to install Bun, causing Docker builds to fail
with "bun: not found".

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@steipete
Copy link
Copy Markdown
Contributor

steipete commented Jan 6, 2026

Thanks Lukáš!

@steipete
Copy link
Copy Markdown
Contributor

steipete commented Jan 6, 2026

LGTM.

Did:

  • Updated branch with latest main to unbreak protocol:check CI.
  • Added changelog entry (Unreleased → Fixes) thanking @loukotal.

Merging via squash.

@steipete steipete merged commit c16510c into openclaw:main Jan 6, 2026
12 checks passed
dgarson added a commit to dgarson/clawdbot that referenced this pull request Feb 22, 2026
…), CostAllocationDashboard (write+wire), ServiceDependencyMap (write — already wired)
@huntharo
Copy link
Copy Markdown
Member

I don't think this can work anymore. bun is installed on /root but the build scripts are run as node and cannot see the /root directory.

Additionally, we're packaging bun and shipping it to every user who then cannot even use it at runtime because the directory is inaccessible.

Just putting this note here so I remember or so someone else takes a look. I plan to come back to this in a bit to see if I can make bun accessible during the build but also not ship it in the final image (I know how to do this... it is possible... just want to see how ugly it is).

zooqueen pushed a commit to hanzoai/bot that referenced this pull request Mar 6, 2026
Install Bun in Dockerfile so `pnpm build` can run Bun scripts inside Docker.

Thanks @loukotal.
dustin-olenslager pushed a commit to dustin-olenslager/ironclaw-supreme that referenced this pull request Mar 24, 2026
Fato07 pushed a commit to Fato07/elav that referenced this pull request Mar 26, 2026
Add opencode-ai to global npm install in Dockerfile
@ridhoperdana
Copy link
Copy Markdown

Straightforward addition – installs Bun in the Dockerfile so pnpm build can run Bun scripts.

Security

  • Uses official installer from bun.sh. Consider pinning to a specific version (e.g., https://bun.sh/install | bash -s "<version>") for reproducibility and supply-chain safety.

Maintainability

  • Simple change; no impact on existing logic.

Suggestion

  • Add a comment explaining why Bun is needed (e.g., references src/macos/relay.ts or specific build scripts). Future readers will appreciate the context.

LGTM with optional version pinning.

lovewanwan pushed a commit to lovewanwan/openclaw that referenced this pull request Apr 28, 2026
Install Bun in Dockerfile so `pnpm build` can run Bun scripts inside Docker.

Thanks @loukotal.
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 9, 2026
Install Bun in Dockerfile so `pnpm build` can run Bun scripts inside Docker.

Thanks @loukotal.
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.

4 participants