@ycodium-ai/agent-acp 0.2.3192 → 0.2.3208

package/dist/index.js

@@ -27199,6 +27199,9 @@ function createAcpSessionRegistry(threads, stopBooked2) {
     if (!booked) return Promise.resolve();
     booked.closing ??= Promise.resolve().then(() => stopBooked2(booked)).then(() => {
       if (threads.get(threadId) === booked) threads.delete(threadId);
+    }).catch((cause) => {
+      if (threads.get(threadId) === booked) delete booked.closing;
+      throw cause;
     });
     return booked.closing;
   };
@@ -27208,6 +27211,61 @@ function createAcpSessionRegistry(threads, stopBooked2) {
   return { requireThread, stopSession, stopAll };
 }
 
+// ../../packages/ycodium-acp-executor/src/session-cleanup.ts
+var reportedForwardingFailures = /* @__PURE__ */ new WeakSet();
+async function cleanupAcpSession(input2) {
+  const failures = [];
+  const forwarding = input2.forwarding.catch((cause) => {
+    if (reportedForwardingFailures.has(input2.forwarding)) return;
+    reportedForwardingFailures.add(input2.forwarding);
+    failures.push(new Error(`forwarding failed: ${String(cause)}`, { cause }));
+  });
+  for (const [step, cleanup] of [
+    ["executor.dispose", () => input2.executor.dispose()],
+    ["stopProcess", input2.stopProcess],
+    ["forwarding", () => forwarding]
+  ]) {
+    try {
+      await cleanup();
+    } catch (cause) {
+      failures.push(new Error(`${step} failed: ${String(cause)}`, { cause }));
+    }
+  }
+  if (failures.length > 0) {
+    throw new AggregateError(failures, failures.map((error62) => error62.message).join("; "));
+  }
+}
+async function cleanupAcpStartup(input2) {
+  try {
+    await cleanupAcpSession(input2);
+    return true;
+  } catch (cleanupError) {
+    input2.log.warn("ACP session startup cleanup failed", {
+      threadId: input2.threadId,
+      error: String(cleanupError)
+    });
+    return false;
+  }
+}
+function createRetryableClose(run) {
+  let task;
+  return () => {
+    if (task) return task;
+    const done = Promise.withResolvers();
+    const current = done.promise;
+    task = current;
+    void current.catch(() => {
+      if (task === current) task = void 0;
+    });
+    try {
+      run().then(done.resolve, done.reject);
+    } catch (cause) {
+      done.reject(cause);
+    }
+    return current;
+  };
+}
+
 // ../../packages/ycodium-acp-executor/src/prompt-dispatch.ts
 var dispatchAcpPrompt = (executor, input2) => {
   const { promise: dispatched, resolve: resolve2 } = Promise.withResolvers();
@@ -27728,17 +27786,22 @@ function acpContinuationKey(recipe) {
   return `acp:home:${recipe.binaryPath.trim()}\0${resolvedHomePath}`;
 }
 async function stopBooked(booked) {
-  await booked.executor.dispose().catch(() => void 0);
-  await booked.handle.kill("SIGTERM").catch(() => void 0);
-  await booked.forwarding.catch(() => void 0);
+  await cleanupAcpSession({
+    executor: booked.executor,
+    stopProcess: () => booked.handle.kill("SIGTERM"),
+    forwarding: booked.forwarding
+  });
 }
 var createAcpInstance = (config2, host) => {
   const recipe = parseAcpInstanceConfig(config2);
   const events = new EventBus();
   const threads = /* @__PURE__ */ new Map();
   let disposed = false;
-  let disposeTask;
   const { requireThread, stopSession, stopAll } = createAcpSessionRegistry(threads, stopBooked);
+  const disposeOnce = createRetryableClose(async () => {
+    await stopAll();
+    events.close();
+  });
   const instance = {
     probe: async () => {
       if (recipe.binaryPath.length === 0) {
@@ -27899,9 +27962,13 @@ var createAcpInstance = (config2, host) => {
         });
         return session;
       } catch (error62) {
-        await executor.dispose().catch(() => void 0);
-        await handle.kill("SIGTERM").catch(() => void 0);
-        await forwarding.catch(() => void 0);
+        await cleanupAcpStartup({
+          executor,
+          stopProcess: () => handle.kill("SIGTERM"),
+          forwarding,
+          log: host.log,
+          threadId: input2.threadId
+        });
         throw error62;
       }
     },
@@ -27981,13 +28048,8 @@ var createAcpInstance = (config2, host) => {
     events,
     continuationKey: acpContinuationKey(recipe),
     dispose: () => {
-      if (disposeTask) return disposeTask;
       disposed = true;
-      disposeTask = (async () => {
-        await stopAll();
-        events.close();
-      })();
-      return disposeTask;
+      return disposeOnce();
     }
   };
   return instance;
@@ -28005,7 +28067,7 @@ var executors = [
 ];
 var index_default = definePlugin({
   id: "agent-acp",
-  apiVersion: 17,
+  apiVersion: 19,
   executors,
   activate(ctx) {
     ctx.log.info("generic ACP executor ready");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ycodium-ai/agent-acp",
-  "version": "0.2.3192",
+  "version": "0.2.3208",
   "description": "Generic ACP executor: user-supplied CLI over Agent Client Protocol stdio.",
   "keywords": [
     "ycodium",
@@ -21,9 +21,9 @@
     "@types/node": "24.12.4",
     "esbuild": "0.28.1",
     "vite-plus": "0.2.2",
-    "@ycodium-ai/plugin-api": "0.2.3192",
-    "ycodium-acp-executor": "0.2.3192",
-    "ycodium-executor-testing": "0.0.0"
+    "ycodium-executor-testing": "0.0.0",
+    "ycodium-acp-executor": "0.2.3208",
+    "@ycodium-ai/plugin-api": "0.2.3208"
   },
   "ycodium": {
     "plugin": "./dist/index.js",