triflux 10.17.1 → 10.17.3

package/.claude-plugin/marketplace.json

@@ -9,7 +9,7 @@
     {
       "name": "triflux",
       "description": "Tri-CLI orchestrator for Claude Code. Routes tasks across Claude + Codex + Gemini with consensus intelligence, natural language routing, 42 skills, and cross-model review.",
-      "version": "10.17.1",
+      "version": "10.17.3",
       "author": {
         "name": "tellang"
       },
@@ -30,5 +30,5 @@
       ]
     }
   ],
-  "version": "10.17.1"
+  "version": "10.17.3"
 }

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "triflux",
-  "version": "10.17.1",
+  "version": "10.17.3",
   "description": "CLI-first multi-model orchestrator for Claude Code — route tasks to Codex, Gemini, and Claude",
   "author": {
     "name": "tellang"

package/hub/lib/process-utils.mjs

@@ -233,6 +233,29 @@ function hasLiveAncestorChain(pid, procMap, protectedPids) {
   return false;
 }
 
+function hasLiveCliDescendant(pid, procMap) {
+  const children = new Map();
+  for (const proc of procMap.values()) {
+    if (!Number.isFinite(proc.ppid) || proc.ppid <= 0) continue;
+    const list = children.get(proc.ppid) || [];
+    list.push(proc);
+    children.set(proc.ppid, list);
+  }
+
+  const visited = new Set();
+  const stack = [...(children.get(pid) || [])];
+  while (stack.length > 0) {
+    const proc = stack.pop();
+    if (!proc || visited.has(proc.pid)) continue;
+    visited.add(proc.pid);
+    if (LIVE_CLI_SESSION_ROOT_NAMES.has(normalizeName(proc.name))) {
+      return true;
+    }
+    stack.push(...(children.get(proc.pid) || []));
+  }
+  return false;
+}
+
 /**
  * Legacy wrapper for scoped orphan node runtime cleanup.
  * @param {Parameters<typeof cleanupOrphanRuntimeProcesses>[0]} opts
@@ -808,7 +831,8 @@ export function cleanupOrphanRuntimeProcesses({
     if (legacy) {
       if (
         LEGACY_ORPHAN_KILLABLE_NAMES.has(name) &&
-        !hasLiveAncestorChain(proc.pid, procMap, protectedSet)
+        !hasLiveAncestorChain(proc.pid, procMap, protectedSet) &&
+        !hasLiveCliDescendant(proc.pid, procMap)
       ) {
         shouldKill = true;
         killReason = "legacy_orphan_ancestor_chain_dead";
@@ -816,7 +840,8 @@ export function cleanupOrphanRuntimeProcesses({
     } else if (name === "bun.exe") {
       if (
         hasExactGbrainServe(proc.commandLine) &&
-        !hasLiveAncestorChain(proc.pid, procMap, protectedSet)
+        !hasLiveAncestorChain(proc.pid, procMap, protectedSet) &&
+        !hasLiveCliDescendant(proc.pid, procMap)
       ) {
         shouldKill = true;
         killReason = "bun_gbrain_serve_orphan";

package/hub/server.mjs

@@ -962,6 +962,38 @@ export async function startHub({
   const tools = createTools(store, router, hitl, pipe);
   const transports = new Map();
 
+  async function closeMcpTransportSession(sid, session, reason) {
+    if (!sid) return;
+    if (!session) {
+      transports.delete(sid);
+      return;
+    }
+    if (session.closing) {
+      transports.delete(sid);
+      return;
+    }
+
+    session.closing = true;
+    transports.delete(sid);
+
+    try {
+      await session.mcp.close();
+    } catch (error) {
+      hubLog.debug(
+        { sid, reason, err: String(error?.message || error) },
+        "hub.mcp_session_close_mcp_failed",
+      );
+    }
+    try {
+      await session.transport.close();
+    } catch (error) {
+      hubLog.debug(
+        { sid, reason, err: String(error?.message || error) },
+        "hub.mcp_session_close_transport_failed",
+      );
+    }
+  }
+
   function createMcpForSession() {
     const mcp = new Server(
       { name: "tfx-hub", version: "1.0.0" },
@@ -1705,19 +1737,17 @@ export async function startHub({
               sessionIdGenerator: () => randomUUID(),
               onsessioninitialized: (sid) => {
                 transport._lastActivity = Date.now();
-                transports.set(sid, { transport, mcp });
+                transports.set(sid, { transport, mcp, closing: false });
               },
             });
             transport.onclose = () => {
-              if (transport.sessionId) {
-                const session = transports.get(transport.sessionId);
-                if (session) {
-                  try {
-                    session.mcp.close();
-                  } catch {}
-                }
-                transports.delete(transport.sessionId);
-              }
+              const sid = transport.sessionId;
+              if (sid)
+                void closeMcpTransportSession(
+                  sid,
+                  transports.get(sid),
+                  "transport.onclose",
+                );
             };
             const mcp = createMcpForSession();
             await mcp.connect(transport);
@@ -1808,13 +1838,7 @@ export async function startHub({
     for (const [sid, session] of transports) {
       if (now - (session.transport._lastActivity || 0) <= SESSION_TTL_MS)
         continue;
-      try {
-        session.mcp.close();
-      } catch {}
-      try {
-        session.transport.close();
-      } catch {}
-      transports.delete(sid);
+      void closeMcpTransportSession(sid, session, "session_ttl");
     }
   }, 60000);
   sessionTimer.unref();
@@ -2058,13 +2082,8 @@ export async function startHub({
               if (idleTimer) {
                 clearInterval(idleTimer);
               }
-              for (const [, session] of transports) {
-                try {
-                  await session.mcp.close();
-                } catch {}
-                try {
-                  await session.transport.close();
-                } catch {}
+              for (const [sid, session] of Array.from(transports)) {
+                await closeMcpTransportSession(sid, session, "hub.stop");
               }
               transports.clear();
               await pipe.stop();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "triflux",
-  "version": "10.17.1",
+  "version": "10.17.3",
   "description": "CLI-first multi-model orchestrator for Claude Code — route tasks to Codex, Gemini, and Claude",
   "type": "module",
   "bin": {

package/scripts/session-stale-cleanup.mjs

@@ -90,6 +90,27 @@ function hasProtectedAncestor(pid, procMap) {
   return false;
 }
 
+function hasProtectedDescendant(pid, procMap) {
+  const children = new Map();
+  for (const proc of procMap.values()) {
+    if (!Number.isFinite(proc.ppid) || proc.ppid <= 0) continue;
+    const list = children.get(proc.ppid) || [];
+    list.push(proc);
+    children.set(proc.ppid, list);
+  }
+
+  const seen = new Set();
+  const stack = [...(children.get(Number(pid)) || [])];
+  while (stack.length > 0) {
+    const proc = stack.pop();
+    if (!proc || seen.has(proc.pid)) continue;
+    seen.add(proc.pid);
+    if (PROTECTED_ANCESTOR_NAMES.has(proc.name)) return true;
+    stack.push(...(children.get(proc.pid) || []));
+  }
+  return false;
+}
+
 export function shouldKillTrackedPid({
   pid,
   pidFileMtimeMs,
@@ -110,6 +131,7 @@ export function shouldKillTrackedPid({
   }
 
   if (hasProtectedAncestor(pid, procMap)) return false;
+  if (hasProtectedDescendant(pid, procMap)) return false;
 
   return true;
 }