@ait-co/devtools 0.1.104 → 0.1.105

package/dist/mcp/cli.js

@@ -3560,6 +3560,24 @@ function killAndWait(pid, graceMs = 2e3) {
 	} catch {}
 }
 /**
+* Reaps an orphaned cloudflared tunnel child left behind by a previous session
+* (issue #628). The normal shutdown path TERM-cascades to the child, but a
+* SIGKILL'd or crashed Node process can't run cleanup — its cloudflared child
+* keeps the (now stale) quick tunnel alive. When `acquireLock` reclaims such a
+* lock, this kills the still-alive `tunnelChildPid` so the new session starts
+* from a clean slate (companion to the #347/#571 zombie-daemon defenses).
+*
+* No-op when the lock carries no `tunnelChildPid` (older lock files) or the
+* child is already gone. SECRET-HANDLING: logs the PID only — never the tunnel
+* host/wss (those never enter the lock file or this path).
+*/
+function reapOrphanTunnelChild(existing) {
+	const childPid = existing.tunnelChildPid;
+	if (typeof childPid !== "number" || !isPidAlive(childPid)) return;
+	process.stderr.write(`[ait-debug] reaping orphaned tunnel child PID=${childPid} from previous session.\n`);
+	killAndWait(childPid);
+}
+/**
 * Reads the current lock file without acquiring it. Returns the parsed
 * `LockData` when the file exists and is valid, otherwise `null`. Used by
 * `get_debug_status` to surface the `serverLockHolder` field without
@@ -3591,13 +3609,17 @@ function acquireLock(options = {}) {
 		else if (force) {
 			process.stderr.write(`[ait-debug] --force: terminating existing session PID=${existing.pid} …\n`);
 			killAndWait(existing.pid);
+			reapOrphanTunnelChild(existing);
 			process.stderr.write(`[ait-debug] --force: PID=${existing.pid} stopped, taking over.\n`);
 		} else {
 			const urlPart = existing.wssUrl != null ? `wssUrl=${existing.wssUrl}` : "wssUrl=(tunnel starting)";
 			process.stderr.write(`[ait-debug] 기존 debug-mode 세션이 이미 실행 중 — PID=${existing.pid}, started ${existing.startedAt}, ${urlPart}\n[ait-debug] 회복: \`kill ${existing.pid}\` 또는 \`npx @ait-co/devtools devtools-mcp --force\`\n`);
 			throw new ServerLockConflictError(existing.pid, existing.wssUrl, existing.startedAt);
 		}
-	} else process.stderr.write(`[ait-debug] stale lock from PID ${existing.pid} recovered — starting fresh.\n`);
+	} else {
+		process.stderr.write(`[ait-debug] stale lock from PID ${existing.pid} recovered — starting fresh.\n`);
+		reapOrphanTunnelChild(existing);
+	}
 	const data = {
 		pid: process.pid,
 		wssUrl: null,
@@ -4833,7 +4855,7 @@ async function readMcpSdkVersion() {
 * some test environments that skip the build step).
 */
 function readDevtoolsVersion() {
-	return "0.1.104";
+	return "0.1.105";
 }
 /**
 * Derives the next recommended action from a completed diagnostics snapshot.
@@ -5425,7 +5447,7 @@ function createDebugServer(deps) {
 	const collector = collectorDep ?? new InMemoryDiagnosticsCollector();
 	const server = new Server({
 		name: "ait-debug",
-		version: "0.1.104"
+		version: "0.1.105"
 	}, { capabilities: { tools: { listChanged: true } } });
 	server.setRequestHandler(ListToolsRequestSchema, () => {
 		const conn = router.active;
@@ -7563,7 +7585,7 @@ function createDevServer(deps = {}) {
 	const aitSource = deps.aitSource ?? new HttpAitSource({ stateEndpoint });
 	const server = new Server({
 		name: "ait-devtools",
-		version: "0.1.104"
+		version: "0.1.105"
 	}, { capabilities: { tools: {} } });
 	server.setRequestHandler(ListToolsRequestSchema, () => ({ tools: DEV_TOOL_DEFINITIONS.map((tool) => ({ ...tool })) }));
 	server.setRequestHandler(CallToolRequestSchema, async (request) => {