Skip to content

Commit d53cd0b

Browse files
committed
fix: revert npm ci to npm install --include=dev in [build] command
npm ci failed in Cloudflare Workers Builds CI — the same failure was encountered and fixed in commit 629dbf7. Their build environment appears to omit devDependencies during npm ci, which causes the react-scripts build to fail (react-scripts is a devDep). npm install --include=dev explicitly forces devDep installation and was the confirmed-working form. https://claude.ai/code/session_013cFiQZ1hnb4JrBj9DvDgip
1 parent ff4d3d3 commit d53cd0b

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

wrangler.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ compatibility_flags = ["nodejs_compat"]
99
# Installs frontend deps and builds the SPA so frontend/build exists before
1010
# `wrangler deploy` uploads assets. NODE_OPTIONS is required for
1111
# react-scripts 5.x + Node 18+ (webpack 4 OpenSSL 3.0 compatibility).
12-
command = "cd frontend && npm ci && GENERATE_SOURCEMAP=false CI=false NODE_OPTIONS=--openssl-legacy-provider npm run build"
12+
command = "cd frontend && npm install --include=dev && GENERATE_SOURCEMAP=false CI=false NODE_OPTIONS=--openssl-legacy-provider npm run build"
1313

1414
# ─── Static Assets (React SPA build output) ──────────────────────────────────
1515
# Built by `npm run build` (root package.json) before every `wrangler deploy`.

0 commit comments

Comments
 (0)