Skip to content

chore(mapgen-studio): simplify dev host process - #1699

Open
mateicanavra wants to merge 1 commit into
codex/run-in-game-selected-map-idfrom
codex/simplify-mapgen-studio-dev-host
Open

chore(mapgen-studio): simplify dev host process#1699
mateicanavra wants to merge 1 commit into
codex/run-in-game-selected-map-idfrom
codex/simplify-mapgen-studio-dev-host

Conversation

@mateicanavra

Copy link
Copy Markdown
Owner

The dev script now runs the daemon and Vite frontend in a single Bun process rather than spawning them as separate child processes. bun --watch is added to the dev script so that Bun's file watcher owns backend restarts via the devLive.ts entrypoint, while Vite continues to handle frontend HMR. Vite is started in-process using createServer from the Vite API instead of being spawned as a subprocess.

The DevLivePlan type is updated to reflect this: the daemon and vite fields now describe configuration values (host, port, config file path) rather than subprocess commands and arguments. The DevLiveCommand and RunningChild types are removed entirely, along with the startChild and waitForFirstExit helpers. Signal handling is updated to cleanly shut down the in-process Vite server and daemon on SIGINT/SIGTERM and exit with the appropriate code.

The runbook is updated to document the direct watcher invocation (bun --watch src/server/daemon/devLive.ts) as the preferred command for detached local sessions where dependencies are already prepared.

mateicanavra commented Jun 14, 2026

Copy link
Copy Markdown
Owner Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@mateicanavra mateicanavra changed the title refactor(mapgen-studio): collapse dev-live into single Bun watch process with in-process Vite chore(mapgen-studio): simplify dev host process Jun 14, 2026
@mateicanavra
mateicanavra force-pushed the codex/simplify-mapgen-studio-dev-host branch 2 times, most recently from a5cd771 to 3dab7a3 Compare June 14, 2026 07:56
@mateicanavra
mateicanavra force-pushed the codex/simplify-mapgen-studio-dev-host branch from 3dab7a3 to 201719b Compare June 14, 2026 08:07
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