@xulongzhe/clawbench 0.96.0 → 0.97.0

package/bin/clawbench.js

@@ -4,7 +4,7 @@ import { resolve, dirname } from "path";
 import { fileURLToPath } from "url";
 import { createRequire } from "module";
 import os from "os";
-import { resolvePlatformPackage, resolveBinName } from "./platform.js";
+import { resolvePlatformPackage, resolveBinName, resolveSpawnOptions } from "./platform.js";
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 const require = createRequire(import.meta.url);
@@ -35,10 +35,10 @@ if (process.env.CLAWBENCH_BINARY_PATH) {
   }
 }
 
-const child = spawn(binPath, process.argv.slice(2), {
-  stdio: "inherit",
-  env: { ...process.env },
-});
+// 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.
+const child = spawn(binPath, process.argv.slice(2), resolveSpawnOptions({ ...process.env }));
 
 // 转发信号到子进程
 process.on("SIGINT", () => child.kill("SIGINT"));

package/bin/platform.js

@@ -30,3 +30,24 @@ export function resolvePlatformPackage(platform, arch) {
 export function resolveBinName(platform) {
   return platform === "win32" ? "clawbench.exe" : "clawbench";
 }
+
+// Returns the spawn options used to launch the server binary.
+//
+// `detached: true` puts the server in its own session/process group. Without
+// it the server inherits the launcher's group, so a terminal hangup (SSH
+// disconnect, closing the shell) delivers SIGHUP to the whole group and the
+// server dies. `nohup` cannot prevent this: it only sets SIG_IGN on the
+// launcher, and Node resets that disposition to SIG_DFL for itself, so the
+// signal reaches the binary regardless. The directly-installed binary has no
+// launcher layer and survives a hangup, which is why only the npm install
+// showed the problem.
+//
+// Signals are still forwarded explicitly by the launcher, so Ctrl+C and
+// `kill <launcher-pid>` keep stopping the server.
+export function resolveSpawnOptions(env) {
+  return {
+    stdio: "inherit",
+    env,
+    detached: true,
+  };
+}

package/bin/platform.test.js

@@ -3,6 +3,7 @@ import {
   PLATFORM_MAP,
   resolvePlatformPackage,
   resolveBinName,
+  resolveSpawnOptions,
 } from './platform.js'
 
 describe('resolvePlatformPackage', () => {
@@ -53,3 +54,30 @@ describe('PLATFORM_MAP', () => {
     expect(PLATFORM_MAP['android-arm64']).toBe('@xulongzhe/clawbench-android-arm64')
   })
 })
+
+describe('resolveSpawnOptions', () => {
+  // Regression guard: without detached the server joins the launcher's process
+  // group and dies on terminal hangup (SSH disconnect), because `nohup` only
+  // 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('passes the environment through to the server', () => {
+    const env = { PATH: '/usr/bin', CLAWBENCH_CHILD: '1' }
+    expect(resolveSpawnOptions(env).env).toEqual(env)
+  })
+
+  it('keeps stdio inherited so the server logs stay visible', () => {
+    expect(resolveSpawnOptions({}).stdio).toBe('inherit')
+  })
+
+  it('returns a fresh object so callers cannot mutate shared state', () => {
+    const first = resolveSpawnOptions({ A: '1' })
+    const second = resolveSpawnOptions({ A: '2' })
+    expect(first).not.toBe(second)
+    expect(first.env).not.toBe(second.env)
+  })
+})

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xulongzhe/clawbench",
-  "version": "0.96.0",
+  "version": "0.97.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.96.0",
-    "@xulongzhe/clawbench-linux-arm64": "0.96.0",
-    "@xulongzhe/clawbench-android-arm64": "0.96.0",
-    "@xulongzhe/clawbench-darwin-x64": "0.96.0",
-    "@xulongzhe/clawbench-darwin-arm64": "0.96.0",
-    "@xulongzhe/clawbench-win32-x64": "0.96.0"
+    "@xulongzhe/clawbench-linux-x64": "0.97.0",
+    "@xulongzhe/clawbench-linux-arm64": "0.97.0",
+    "@xulongzhe/clawbench-android-arm64": "0.97.0",
+    "@xulongzhe/clawbench-darwin-x64": "0.97.0",
+    "@xulongzhe/clawbench-darwin-arm64": "0.97.0",
+    "@xulongzhe/clawbench-win32-x64": "0.97.0"
   },
   "engines": {
     "node": ">=14"