Skip to content

Comments

🐛 fix(ci): Add binary verification step after npm ci for all platforms#153

Merged
trickpatty merged 1 commit intomainfrom
claude/fix-electron-binary-verification-I7DQZ
Jan 22, 2026
Merged

🐛 fix(ci): Add binary verification step after npm ci for all platforms#153
trickpatty merged 1 commit intomainfrom
claude/fix-electron-binary-verification-I7DQZ

Conversation

@trickpatty
Copy link
Owner

npm ci sometimes fails to extract large binary files from packages like app-builder-bin (~50MB). This fix adds a verification step that:

  1. Checks if the platform-specific binary exists after npm ci
  2. If missing, manually downloads and extracts the package using npm pack
  3. Sets execute permissions on the binary

This approach bypasses any npm extraction issues by using npm pack + tar which is more reliable for packages containing large binary files.

Platform-specific binary paths:

  • Linux: linux/x64/app-builder
  • Windows: win/x64/app-builder.exe
  • macOS Intel: mac/app-builder_amd64
  • macOS ARM: mac/app-builder_arm64

npm ci sometimes fails to extract large binary files from packages like
app-builder-bin (~50MB). This fix adds a verification step that:

1. Checks if the platform-specific binary exists after npm ci
2. If missing, manually downloads and extracts the package using npm pack
3. Sets execute permissions on the binary

This approach bypasses any npm extraction issues by using npm pack + tar
which is more reliable for packages containing large binary files.

Platform-specific binary paths:
- Linux: linux/x64/app-builder
- Windows: win/x64/app-builder.exe
- macOS Intel: mac/app-builder_amd64
- macOS ARM: mac/app-builder_arm64
@trickpatty trickpatty merged commit f6a837b into main Jan 22, 2026
12 checks passed
@trickpatty trickpatty deleted the claude/fix-electron-binary-verification-I7DQZ branch January 22, 2026 10:57
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.

2 participants