From 586c57d11ba19aec2292c19a0f425f11cdde7869 Mon Sep 17 00:00:00 2001
From: Peter Dedene <peter@zenjoy.be>
Date: Mon, 16 Mar 2026 13:38:21 +0100
Subject: [PATCH] fix(daemon): skip Bun virtual entry path in detached child
 spawn

In Bun compiled binaries, process.argv[1] is a virtual /$bunfs/...
path that Bun auto-injects into every spawned child process. The
daemon launcher was including this path explicitly in the spawn args,
causing it to appear twice in the child's argv. The child then parsed
the duplicate path as the CLI command instead of "daemon", silently
failing to start.

Detect the /$bunfs/ prefix and omit the entry from spawn args, letting
Bun handle it automatically.
---
 src/daemon/launch.ts | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/daemon/launch.ts b/src/daemon/launch.ts
index 504f54f..162369e 100644
--- a/src/daemon/launch.ts
+++ b/src/daemon/launch.ts
@@ -15,7 +15,7 @@ export function launchDaemonDetached(options: DaemonLaunchOptions): void {
   const configArgs = options.configExplicit ? ['--config', options.configPath] : [];
   const args = [
     ...process.execArgv,
-    cliEntry,
+    ...(cliEntry ? [cliEntry] : []),
     ...configArgs,
     ...(options.rootDir ? ['--root', options.rootDir] : []),
     'daemon',
@@ -36,10 +36,16 @@ export function launchDaemonDetached(options: DaemonLaunchOptions): void {
   child.unref();
 }
 
-function resolveCliEntry(): string {
+function resolveCliEntry(): string | undefined {
   const entry = process.argv[1];
   if (!entry) {
     throw new Error('Unable to resolve mcporter entry script.');
   }
+  // In Bun compiled binaries, argv[1] is a virtual /$bunfs/... path that Bun
+  // auto-injects into every spawned child.  Including it explicitly would
+  // duplicate it and break CLI argument parsing in the child process.
+  if (entry.startsWith('/$bunfs/')) {
+    return undefined;
+  }
   return path.resolve(entry);
 }