@netnodeag/kraftwerk 0.15.0 → 0.16.0

package/dist/inspector/chat/sessions.d.ts

@@ -1,4 +1,6 @@
 import type { ChatAgentId, ChatMeta, ChatScope, StoredChatEvent } from "./types.js";
+/** Server shutdown: no agent subprocess may outlive the inspector. */
+export declare function disposeAllBackends(): void;
 export declare function createChat(opts: {
     agent: ChatAgentId;
     scope: ChatScope;

package/dist/inspector/chat/sessions.js

@@ -10,6 +10,32 @@ import { startAcpBackend } from "./acp.js";
 import { startPiBackend } from "./pi.js";
 import { appendEvent, listChatMetas, newChatId, readEvents, readMeta, safeChatDir, writeMeta, } from "./store.js";
 const states = new Map();
+/** Kill a chat's agent subprocess; the next message spawns a fresh one. */
+function dropBackend(state) {
+    state.backend?.dispose();
+    state.backend = null;
+}
+// Idle reaper: a finished chat must not keep its agent subprocess alive
+// forever (one ACP adapter + one harness binary per chat adds up fast —
+// hourly routines used to leak a process pair per tick). History is on
+// disk and ensureBackend re-sends scope context, so reaping just costs
+// the next message one respawn. busy=true covers the whole turn including
+// pending permission prompts, so nothing is killed mid-work.
+const IDLE_BACKEND_MS = 15 * 60_000;
+const reaper = setInterval(() => {
+    const cutoff = Date.now() - IDLE_BACKEND_MS;
+    for (const state of states.values()) {
+        if (state.backend && !state.busy && Date.parse(state.meta.updatedAt) < cutoff) {
+            dropBackend(state);
+        }
+    }
+}, 60_000);
+reaper.unref?.();
+/** Server shutdown: no agent subprocess may outlive the inspector. */
+export function disposeAllBackends() {
+    for (const state of states.values())
+        dropBackend(state);
+}
 async function loadState(id) {
     const existing = states.get(id);
     if (existing)
@@ -387,13 +413,17 @@ export async function postMessage(id, text) {
             const promptText = context ? `<context>\n${context}\n</context>\n\n${body}` : body;
             const stopReason = await backend.prompt(promptText);
             emit(state, { type: "turn_end", stopReason });
+            // Routine sessions are one-shot and unattended: release the agent
+            // process as soon as the turn ends instead of waiting for the reaper.
+            const scope = state.meta.scope;
+            if (scope.kind === "team" && scope.routine)
+                dropBackend(state);
         }
         catch (err) {
             emit(state, { type: "error", message: err.message });
             // A failed turn may mean a dead subprocess; drop it so the next
             // message spawns a fresh agent (thread history stays on disk).
-            state.backend?.dispose();
-            state.backend = null;
+            dropBackend(state);
         }
         finally {
             state.busy = false;
@@ -419,7 +449,7 @@ export async function deleteChat(id) {
         return { error: "not found" };
     for (const resolve of state.pendingPermissions.values())
         resolve(null);
-    state.backend?.dispose();
+    dropBackend(state);
     states.delete(id);
     await fs.rm(safeChatDir(id), { recursive: true, force: true });
     return {};

package/dist/inspector/server.js

@@ -6,7 +6,7 @@ import { resolveProject } from "../config.js";
 import { listRuns, getRun, readRunFile } from "./runs.js";
 import { listWorkflows, getWorkflow } from "./workflows.js";
 import { dockerStatus, triggerRun, stopRun } from "./runner.js";
-import { cancelChat, createChat, deleteChat, getChat, listChats, postMessage, resolvePermission, subscribeChat, } from "./chat/sessions.js";
+import { cancelChat, createChat, deleteChat, disposeAllBackends, getChat, listChats, postMessage, resolvePermission, subscribeChat, } from "./chat/sessions.js";
 import { bundleDetail, conceptDetail, createBundle, knowledgeIndex, putConcept, verifyFromUi, } from "./knowledge.js";
 import { deleteMember, getMember, listMembers, saveMember, teamRoot, } from "./team.js";
 import { getSkill, listSkills, skillsRoot } from "./skills.js";
@@ -488,6 +488,15 @@ export function startInspector(opts) {
     if (opts.projectRoot)
         setProjectRoot(opts.projectRoot);
     startRoutineScheduler();
+    // Chat agent subprocesses must die with the server — signals bypass
+    // "exit" handlers, so hook the signals themselves.
+    for (const sig of ["SIGINT", "SIGTERM"]) {
+        process.once(sig, () => {
+            disposeAllBackends();
+            process.exit(sig === "SIGINT" ? 130 : 143);
+        });
+    }
+    process.once("exit", disposeAllBackends);
     const server = http.createServer(async (req, res) => {
         try {
             const url = new URL(req.url ?? "/", "http://localhost");