Fix build scripts to use bun instead of pnpm#17
Merged
Conversation
Replace pnpm install/exec calls with bun equivalents so the build works in this bun-only project. Also switch from `bunx eleventy` to `bun run build` so the Bun runtime is used and Bun.file() APIs in reviews.js are available. https://claude.ai/code/session_01C2QdU9Djkt7xZ2hxusFHsC
Replace pnpm setup/install/exec steps with bun equivalents. Also update sparse-checkout to fetch bun.lock and bunfig.toml instead of pnpm-lock.yaml, since the template is bun-only. https://claude.ai/code/session_01C2QdU9Djkt7xZ2hxusFHsC
The eleventy-build.js entrypoint lives in the template's scripts/ directory, which wasn't included in the sparse-checkout. This caused the 'Build Site' job to fail with a file-not-found error. https://claude.ai/code/session_01C2QdU9Djkt7xZ2hxusFHsC
- Add packages/ to sparse-checkout so #toolkit/* path alias resolves - Cache node_modules alongside .image-cache (keyed on bun.lock hash) so bun install is fast on repeat runs, matching chobble-client setup https://claude.ai/code/session_01C2QdU9Djkt7xZ2hxusFHsC
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
pnpm installwithbun installinscripts/prepare-dev.jspnpm exec eleventywithbun run buildinscripts/build.jsWhy
The project is bun-only (
package.jsonhas"packageManager": "bun"), but the build scripts were calling pnpm. This caused two failures:pnpm installrejected with "This project is configured to use bun"pnpm exec eleventysame rejectionSwitching to
bun run build(which runsbun scripts/eleventy-build.js) also fixes a secondary error: the template'sreviews.jscallsBun.file()— a Bun-only API — which was crashing when eleventy was invoked viabunx(Node.js runtime). Running viabun run buildensures the Bun runtime is used throughout.Test plan
bun run buildcompletes successfully and writes 42 files to_site/https://claude.ai/code/session_01C2QdU9Djkt7xZ2hxusFHsC
Generated by Claude Code