ci: use pnpm node_modules directly for bun compile (match local build)
Some checks failed
CI/CD / lint (push) Successful in 41s
CI/CD / test (push) Successful in 54s
CI/CD / typecheck (push) Successful in 10m56s
CI/CD / build (push) Successful in 1m10s
CI/CD / docker (push) Failing after 27s
CI/CD / publish-rpm (push) Successful in 36s
CI/CD / deploy (push) Has been skipped
Some checks failed
CI/CD / lint (push) Successful in 41s
CI/CD / test (push) Successful in 54s
CI/CD / typecheck (push) Successful in 10m56s
CI/CD / build (push) Successful in 1m10s
CI/CD / docker (push) Failing after 27s
CI/CD / publish-rpm (push) Successful in 36s
CI/CD / deploy (push) Has been skipped
The local build-rpm.sh successfully uses pnpm's node_modules with bun compile. The CI was unnecessarily replacing node_modules with bun install, which broke transitive workspace dependency resolution. Match the working local approach instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -116,11 +116,7 @@ jobs:
|
|||||||
- name: Bundle standalone binaries
|
- name: Bundle standalone binaries
|
||||||
run: |
|
run: |
|
||||||
mkdir -p dist
|
mkdir -p dist
|
||||||
# bun needs a flat node_modules (pnpm's strict layout breaks resolution)
|
# Stub for optional dep that Ink tries to import (only used when DEV=true)
|
||||||
# Remove pnpm's nested node_modules first so bun resolves fresh
|
|
||||||
rm -rf node_modules
|
|
||||||
bun install
|
|
||||||
# Stub for optional dep that bun tries to resolve
|
|
||||||
if [ ! -e node_modules/react-devtools-core ]; then
|
if [ ! -e node_modules/react-devtools-core ]; then
|
||||||
ln -s ../src/cli/stubs/react-devtools-core node_modules/react-devtools-core
|
ln -s ../src/cli/stubs/react-devtools-core node_modules/react-devtools-core
|
||||||
fi
|
fi
|
||||||
|
|||||||
Reference in New Issue
Block a user