ros/automaker
1
0
Fork 0
mirror of https://github.com/AutoMaker-Org/automaker.git synced 2026-01-31 20:03:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(ci): rebuild node-pty after install to fix native module errors

Browse Source 
The --ignore-scripts flag also skips building native modules like
node-pty which the server needs. Added explicit rebuild step for
node-pty in test.yml and e2e-tests.yml workflows.

This is more targeted than electron-builder install-app-deps which
rebuilds ALL native modules and causes OOM.

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Kacper
2025-12-18 12:25:21 +01:00
parent 2c079623a8
commit 0d8043f1f2
3 changed files with 17 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
.github/workflows/pr-check.yml vendored
View File

@@ -46,5 +46,10 @@ jobs:
@rollup/rollup-linux-x64-gnu@4.53.3 \
@tailwindcss/oxide-linux-x64-gnu@4.1.17
- name: Rebuild native modules for Electron
# Rebuild node-pty and other native modules for Electron after npm install --ignore-scripts
# Avoids using electron-builder install-app-deps to prevent OOM issues
run: npm rebuild node-pty
- name: Run build:electron (dir only - faster CI)
run: npm run build:electron:dir