Skip to content

fix(dev): use current Node runtime in worktrees - #94

Merged
boybook merged 1 commit into
mainfrom
codex/dev-runtime-node-path
Aug 28, 2026
Merged

fix(dev): use current Node runtime in worktrees#94
boybook merged 1 commit into
mainfrom
codex/dev-runtime-node-path

Conversation

@boybook

@boybook boybook commented Aug 28, 2026

Copy link
Copy Markdown
Owner

Summary

  • pass the Node executable running dev-runtime.js to Electron through TX5DR_DEV_NODE_PATH
  • allow Turbo to forward the development-only environment variable
  • use the override only in the unpackaged development runtime

Context

Source worktrees do not necessarily contain resources/bin/<triplet>/node because packaged runtime binaries are ignored build resources. dev:electron previously tried to launch that missing packaged path and failed with ENOENT even though the development process already had a working Node executable.

The startup script now defaults the override to process.execPath, while still allowing an explicit caller-provided path. Packaged and production runtimes continue to resolve Node from application resources.

Validation

  • yarn workspace @tx5dr/electron-main build
  • yarn workspace @tx5dr/electron-main lint
  • yarn workspace @tx5dr/electron-main test (10 files, 45 tests)
  • node --check scripts/dev-runtime.js
  • Turbo JSON parse check
  • git diff --check

@boybook
boybook merged commit 1063301 into main Aug 28, 2026
9 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