Skip to content

fix(ci): add electron-rebuild step to Linux release build#1069

Merged
arnestrickmann merged 1 commit intogeneralaction:mainfrom
jschwxrz:fix/linux-native-module-rebuild
Feb 24, 2026
Merged

fix(ci): add electron-rebuild step to Linux release build#1069
arnestrickmann merged 1 commit intogeneralaction:mainfrom
jschwxrz:fix/linux-native-module-rebuild

Conversation

@jschwxrz
Copy link
Collaborator

summary:

  • fixes PTY unavailable error on Linux since 0.4.16 (NODE_MODULE_VERSION 127 vs. 123

cause:

  • native modules (node-pty, sqlite3, keytar) are compiled for CI runners node 22 (ABI 127)
  • those modules need to be recompiled for electrons node version (ABI 123)
  • in 0.4.15, postinstall script handled this rebuild, but was changed to be skipped in CI in 0.4.16 (mac build needs to be per architecture, postinstall could not handle that)
  • each platform is now expected to run "electron-rebuild" explicitly in the workflow
  • mac and windows already had that step, linux was still missing it

fix:

  • added electron-rebuild step for linux

fixes #1068

@vercel
Copy link

vercel bot commented Feb 24, 2026

@jschwxrz is attempting to deploy a commit to the General Action Team on Vercel.

A member of the Team first needs to authorize it.

@greptile-apps
Copy link

greptile-apps bot commented Feb 24, 2026

Greptile Summary

This PR fixes the "PTY unavailable" error on Linux builds since version 0.4.16 by adding the missing electron-rebuild step to the Linux release workflow. Native modules (node-pty, sqlite3, keytar) are compiled for Node.js 22 (ABI 127) during CI install, but need to be recompiled for Electron's Node.js version (ABI 123).

The postinstall script that previously handled this rebuild was changed in 0.4.16 to skip execution in CI environments because Mac builds require per-architecture rebuilds. Mac and Windows workflows were already updated with explicit electron-rebuild steps, but Linux was missed.

Changes:

  • Added electron-rebuild step before Linux package build
  • Command matches Mac/Windows patterns: rebuilds for x64 architecture with explicit Electron version
  • Targets the same three native modules: sqlite3, node-pty, keytar

Confidence Score: 5/5

  • Safe to merge - minimal change that fixes a critical Linux runtime error
  • The fix is a straightforward addition of a missing build step that already exists in the Mac and Windows workflows. The command syntax, module list, and architecture all match the established pattern. This directly addresses issue [bug]: critical: Github Copilot Chat UI does not work anymore since 0.4.16 #1068 with a well-understood root cause.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/release.yml Added missing electron-rebuild step for Linux to fix PTY errors caused by Node.js ABI version mismatch (127 vs 123)

Last reviewed commit: 4aaedc8

@arnestrickmann
Copy link
Contributor

@jschwxrz thanks!

@arnestrickmann arnestrickmann merged commit 0a8b0db into generalaction:main Feb 24, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug]: critical: Github Copilot Chat UI does not work anymore since 0.4.16

2 participants