Skip to content

ci: pin release build to Node 22; restore electron to ^42#121

Merged
eboudrant merged 1 commit into
mainfrom
ci/release-node-22
Jun 5, 2026
Merged

ci: pin release build to Node 22; restore electron to ^42#121
eboudrant merged 1 commit into
mainfrom
ci/release-node-22

Conversation

@eboudrant

Copy link
Copy Markdown
Owner

Summary

Fixes the broken v0.0.20 release (it produced no installers).

Root cause (reproduced locally both ways): electron-forge 7.11.2 — the latest stable — silently fails under Node 24.x. electron-forge make runs the package phase, stops at "Finalizing package", never runs the dmg/zip makers, and exits 0 with an empty out/make. On Node 22, the same code + deps produce the dmg + zip. ci.yml never runs make, so it only surfaced at release time.

electron was not the cause — v0.0.19 shipped on electron 42.2.0, and both 41 and 42 build fine on Node 22 / fail on Node 24. So this also reverts the misdiagnosed #119 electron pin back to ^42.

No stable electron-forge supports Node 24 yet (only 8.0.0-alpha targets it), so pinning the release build to Node 22 is the robust fix.

Changes

  • release.yml: build job setup-node '24''22' (with an explanatory comment).
  • package.json: electron ^41.7.1^42.0.0 (resolves 42.3.3).

Test plan

  • Locally, npm ci + npx electron-forge make --arch=arm64 on Node 22 + electron 42.3.3 runs both makers and emits out/make/PapaStudio.dmg + the arm64 zip. On Node 24 the same setup emits nothing.
  • After merge: re-dispatch the Release workflow for v0.0.20.

🤖 Generated with Claude Code

The v0.0.20 release produced no installers: electron-forge 7.11.2 (latest
stable) silently fails to run the makers under Node 24.x — `make` stops after
packaging and emits no .dmg/.zip. Reproduced locally: Node 24 → nothing,
Node 22 → dmg + zip. electron was never the cause (v0.0.19 shipped on 42.2.0),
so this also reverts the misdiagnosed #119 pin back to ^42.

- release.yml: build job Node 24 → 22.
- package.json: electron ^41.7.1 → ^42.0.0 (resolves 42.3.3).

Verified locally: npm ci + electron-forge make on Node 22 + electron 42.3.3
produces PapaStudio.dmg and the arm64 zip.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@eboudrant eboudrant enabled auto-merge (squash) June 5, 2026 01:51
@eboudrant eboudrant merged commit f308da7 into main Jun 5, 2026
7 checks passed
@eboudrant eboudrant deleted the ci/release-node-22 branch June 5, 2026 01:53
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.

1 participant