Skip to content

Commit 98bd469

Browse files
committed
fix(build): resolve SEA build failures across platforms
- Add missing @socketsecurity/build-infra dependency to node-sea-builder - Fix Windows PowerShell syntax error in build-sea workflow by adding shell: bash directive
1 parent 9e95043 commit 98bd469

File tree

3 files changed

+8
-0
lines changed

3 files changed

+8
-0
lines changed

.github/workflows/build-sea.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,7 @@ jobs:
126126

127127
- name: Build WASM assets on cache miss
128128
if: steps.yoga-cache.outputs.cache-hit != 'true' || steps.ai-cache.outputs.cache-hit != 'true' || steps.onnx-cache.outputs.cache-hit != 'true'
129+
shell: bash
129130
run: |
130131
echo "⚠️ WASM cache miss detected - building from source"
131132
echo "This will take 30-60 minutes. Consider running build-wasm.yml workflow to prime cache."

packages/node-sea-builder/package.json

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
11
{
2+
"dependencies": {
3+
"@socketsecurity/build-infra": "workspace:*"
4+
},
25
"description": "Native Node.js SEA binary builder (fallback)",
36
"devDependencies": {
47
"vitest": "3.0.0"

pnpm-lock.yaml

Lines changed: 4 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)