Skip to content

fix: handle missing electron executable gracefully#747

Merged
jackwener merged 2 commits intomainfrom
mbp-codex-pr0/fix-antigravity-launch-enoent
Apr 3, 2026
Merged

fix: handle missing electron executable gracefully#747
jackwener merged 2 commits intomainfrom
mbp-codex-pr0/fix-antigravity-launch-enoent

Conversation

@jackwener
Copy link
Copy Markdown
Owner

Intent

Fix the Electron app launch path so a missing executable does not crash the CLI with an unhandled child-process error.

What changed

  • add a guarded launchDetachedApp() helper in src/launcher.ts
  • convert spawn ENOENT into a controlled CommandExecutionError with an actionable hint
  • cover successful launch and ENOENT failure in src/launcher.test.ts

Verification

  • npx vitest run src/launcher.test.ts src/electron-apps.test.ts src/engine.test.ts
  • npm run typecheck
  • npm run build
  • node dist/main.js antigravity new now returns a controlled launch error instead of an unhandled crash

@jackwener jackwener merged commit 748b092 into main Apr 3, 2026
10 of 11 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.

1 participant