@xulongzhe/clawbench 0.99.1 → 0.100.0

package/bin/clawbench.js

@@ -35,9 +35,11 @@ if (process.env.CLAWBENCH_BINARY_PATH) {
   }
 }
 
-// detached: true makes the server a session leader in its own process group,
-// so a terminal hangup cannot reach it. See resolveSpawnOptions for the full
-// rationale. Signals are forwarded explicitly below, so Ctrl+C still stops it.
+// detached: true (POSIX only — see resolveSpawnOptions) makes the server a
+// session leader in its own process group, so a terminal hangup cannot reach
+// it. Windows deliberately opts out: there detached means DETACHED_PROCESS,
+// which strips the console and silently swallowed all server output. Signals
+// are forwarded explicitly below, so Ctrl+C still stops it.
 const child = spawn(binPath, process.argv.slice(2), resolveSpawnOptions({ ...process.env }));
 
 // 转发信号到子进程

package/bin/platform.js

@@ -44,10 +44,21 @@ export function resolveBinName(platform) {
 //
 // Signals are still forwarded explicitly by the launcher, so Ctrl+C and
 // `kill <launcher-pid>` keep stopping the server.
-export function resolveSpawnOptions(env) {
+//
+// Windows is excluded: there `detached` means DETACHED_PROCESS (libuv
+// src/win/process.c), i.e. the child inherits no console, while `stdio:
+// "inherit"` still hands it the launcher's console handles. The server writes
+// everything through them (slog -> stderr, startup banner -> stdout), so its
+// entire console output — including the auto-generated password banner — was
+// silently discarded. Nothing about the SIGHUP rationale applies there, so
+// Windows keeps the default non-detached spawn and the output stays visible.
+// The cost is that libuv assigns the child to a job object
+// (JOB_OBJECT_LIMIT_KILL_ON_JOB_CLOSE), so closing the terminal window also
+// stops the server; use the Windows service path for unattended runs.
+export function resolveSpawnOptions(env, platform = process.platform) {
   return {
     stdio: "inherit",
     env,
-    detached: true,
+    detached: platform !== "win32",
   };
 }

package/bin/platform.test.js

@@ -61,22 +61,39 @@ describe('resolveSpawnOptions', () => {
   // shields the launcher — Node resets SIG_IGN to SIG_DFL for itself, so the
   // SIGHUP still reaches the Go binary. The directly-installed binary has no
   // launcher layer, which is why only the npm install was affected.
-  it('detaches the server so a terminal hangup cannot kill it', () => {
-    expect(resolveSpawnOptions({}).detached).toBe(true)
+  it('detaches the server on POSIX so a terminal hangup cannot kill it', () => {
+    expect(resolveSpawnOptions({}, 'linux').detached).toBe(true)
+    expect(resolveSpawnOptions({}, 'darwin').detached).toBe(true)
+    expect(resolveSpawnOptions({}, 'android').detached).toBe(true)
+  })
+
+  // Regression guard: on Windows detached maps to DETACHED_PROCESS, which
+  // strips the child's console while `stdio: "inherit"` still passes the
+  // launcher's console handles. The server writes all of its console output
+  // (slog -> stderr, startup banner -> stdout) through those handles, so the
+  // combination silently discarded every line — including the first-run
+  // password banner. Windows must spawn without detached.
+  it('does not detach on Windows, where it would silently discard all output', () => {
+    expect(resolveSpawnOptions({}, 'win32').detached).toBe(false)
+  })
+
+  it('defaults to the running platform', () => {
+    expect(resolveSpawnOptions({}).detached).toBe(process.platform !== 'win32')
   })
 
   it('passes the environment through to the server', () => {
     const env = { PATH: '/usr/bin', CLAWBENCH_CHILD: '1' }
-    expect(resolveSpawnOptions(env).env).toEqual(env)
+    expect(resolveSpawnOptions(env, 'linux').env).toEqual(env)
   })
 
   it('keeps stdio inherited so the server logs stay visible', () => {
-    expect(resolveSpawnOptions({}).stdio).toBe('inherit')
+    expect(resolveSpawnOptions({}, 'linux').stdio).toBe('inherit')
+    expect(resolveSpawnOptions({}, 'win32').stdio).toBe('inherit')
   })
 
   it('returns a fresh object so callers cannot mutate shared state', () => {
-    const first = resolveSpawnOptions({ A: '1' })
-    const second = resolveSpawnOptions({ A: '2' })
+    const first = resolveSpawnOptions({ A: '1' }, 'linux')
+    const second = resolveSpawnOptions({ A: '2' }, 'linux')
     expect(first).not.toBe(second)
     expect(first.env).not.toBe(second.env)
   })

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xulongzhe/clawbench",
-  "version": "0.99.1",
+  "version": "0.100.0",
   "description": "ClawBench — AI Workbench, United Across Devices. Go backend + Vue3 frontend, runs on PC, Android, and Termux",
   "type": "module",
   "bin": {
@@ -14,12 +14,12 @@
     "postinstall": "node install.js"
   },
   "optionalDependencies": {
-    "@xulongzhe/clawbench-linux-x64": "0.99.1",
-    "@xulongzhe/clawbench-linux-arm64": "0.99.1",
-    "@xulongzhe/clawbench-android-arm64": "0.99.1",
-    "@xulongzhe/clawbench-darwin-x64": "0.99.1",
-    "@xulongzhe/clawbench-darwin-arm64": "0.99.1",
-    "@xulongzhe/clawbench-win32-x64": "0.99.1"
+    "@xulongzhe/clawbench-linux-x64": "0.100.0",
+    "@xulongzhe/clawbench-linux-arm64": "0.100.0",
+    "@xulongzhe/clawbench-android-arm64": "0.100.0",
+    "@xulongzhe/clawbench-darwin-x64": "0.100.0",
+    "@xulongzhe/clawbench-darwin-arm64": "0.100.0",
+    "@xulongzhe/clawbench-win32-x64": "0.100.0"
   },
   "engines": {
     "node": ">=14"