@cortexkit/aft-pi 0.51.1 → 0.51.2

package/dist/index.js

@@ -8613,6 +8613,13 @@ class BridgeTransportUnavailableError extends Error {
     this.name = "BridgeTransportUnavailableError";
   }
 }
+
+class BridgeTransportUnknownOutcomeError extends BridgeTransportUnavailableError {
+  constructor(message, options) {
+    super(message, options);
+    this.name = "BridgeTransportUnknownOutcomeError";
+  }
+}
 function isBridgeTransportTimeout(err) {
   return err instanceof Error && err.code === "transport_timeout";
 }
@@ -8766,7 +8773,7 @@ class BinaryBridge {
     this.clearRestartResetTimer();
     this.configured = false;
     this.outstandingBackgroundTaskIds.clear();
-    this.rejectAllPending(error2);
+    this.rejectAllPending(error2 instanceof BridgeTransportUnknownOutcomeError ? error2 : new BridgeTransportUnknownOutcomeError(error2.message, { cause: error2 }));
   }
   hasPendingRequests() {
     return this.pending.size > 0;
@@ -8889,6 +8896,12 @@ class BinaryBridge {
       if (!this.configured) {
         if (command !== "configure" && command !== "version") {
           if (!this._configurePromise) {
+            const configuringChild = this.process;
+            const requireConfiguringChild = () => {
+              if (this.process !== configuringChild) {
+                throw new BridgeTransportUnavailableError(`${this.errorPrefix} Bridge process changed during configure; retry on replacement`);
+              }
+            };
             const sessionIdForConfigure = typeof params.session_id === "string" ? params.session_id : undefined;
             this._configurePromise = (async () => {
               try {
@@ -8897,11 +8910,14 @@ class BinaryBridge {
                   ...this.configOverrides,
                   ...sessionIdForConfigure ? { session_id: sessionIdForConfigure } : {}
                 }, implicitTransportOptions);
+                requireConfiguringChild();
                 if (configResult.success === false) {
                   throw new Error(`${this.errorPrefix} Configure failed: ${configResult.message ?? "unknown error"}`);
                 }
                 await this.deliverConfigureWarnings(configResult, params, options);
+                requireConfiguringChild();
                 await this.checkVersion(implicitTransportOptions);
+                requireConfiguringChild();
                 if (!this.isAlive()) {
                   throw new BridgeTransportUnavailableError(`${this.errorPrefix} Bridge died during version check. Check logs: ${this.getLogFilePathVia()}`);
                 }
@@ -8980,19 +8996,26 @@ class BinaryBridge {
           return;
         }
         requestSentAt = Date.now();
-        child.stdin.write(line, (err) => {
-          if (err) {
-            const error2 = new BridgeTransportUnavailableError(`${this.errorPrefix} Failed to write to stdin: ${err.message}`, { cause: err });
-            const entry = this.pending.get(id);
-            if (entry) {
-              this.pending.delete(id);
-              clearTimeout(entry.timer);
-              entry.reject(error2);
-            }
-            if (this.process === child)
-              this.invalidateTransportProcess(error2);
+        const handleWriteFailure = (cause) => {
+          const writeError = cause instanceof Error ? cause : new Error(String(cause));
+          const error2 = new BridgeTransportUnknownOutcomeError(`${this.errorPrefix} Failed to write to stdin: ${writeError.message}`, { cause: writeError });
+          const entry = this.pending.get(id);
+          if (entry) {
+            this.pending.delete(id);
+            clearTimeout(entry.timer);
+            entry.reject(error2);
           }
-        });
+          if (this.process === child)
+            this.invalidateTransportProcess(error2);
+        };
+        try {
+          child.stdin.write(line, (err) => {
+            if (err)
+              handleWriteFailure(err);
+          });
+        } catch (err) {
+          handleWriteFailure(err);
+        }
       });
       if (command === "configure" && response.success === true && options?.markConfiguredOnSuccess !== false) {
         this.configured = true;
@@ -9207,9 +9230,13 @@ class BinaryBridge {
     const currentChild = child;
     const stdoutDecoder = new StringDecoder("utf8");
     child.stdout?.on("data", (chunk) => {
+      if (this.process !== currentChild)
+        return;
       this.onStdoutData(stdoutDecoder.write(chunk));
     });
     child.stdout?.on("end", () => {
+      if (this.process !== currentChild)
+        return;
       const remaining = stdoutDecoder.end();
       if (remaining)
         this.onStdoutData(remaining);
@@ -9217,9 +9244,13 @@ class BinaryBridge {
     });
     const stderrDecoder = new StringDecoder("utf8");
     child.stderr?.on("data", (chunk) => {
+      if (this.process !== currentChild)
+        return;
       this.onStderrData(stderrDecoder.write(chunk));
     });
     child.stderr?.on("end", () => {
+      if (this.process !== currentChild)
+        return;
       const remaining = stderrDecoder.end();
       if (remaining)
         this.onStderrData(remaining);
@@ -9242,7 +9273,7 @@ class BinaryBridge {
         this.process = null;
         this.configured = false;
         this.clearRestartResetTimer();
-        this.rejectAllPending(new BridgeTransportUnavailableError(`${this.errorPrefix} Binary killed by ${signal}`));
+        this.rejectAllPending(new BridgeTransportUnknownOutcomeError(`${this.errorPrefix} Binary killed by ${signal}`));
         return;
       }
       this.handleCrash();
@@ -9420,7 +9451,7 @@ class BinaryBridge {
   handleTimeout(triggeringSessionId) {
     this.consecutiveRequestTimeouts = 0;
     this.spawnedBinaryFingerprint = null;
-    this.rejectAllPending(new Error(`${this.errorPrefix} bridge killed during sibling timeout — request aborted`));
+    this.rejectAllPending(new BridgeTransportUnknownOutcomeError(`${this.errorPrefix} bridge killed during sibling timeout — request aborted`));
     this.outstandingBackgroundTaskIds.clear();
     if (this.process) {
       this.process.kill("SIGKILL");
@@ -9457,7 +9488,7 @@ class BinaryBridge {
     if (tail) {
       this.errorVia(`Binary crashed (restarts: ${this._restartCount})${cause ? `: ${cause.message}` : ""}.${tail}`);
     }
-    this.rejectAllPending(new BridgeTransportUnavailableError(`${this.errorPrefix} Binary crashed (restarts: ${this._restartCount})${cause ? `: ${cause.message}` : ""} (see ${this.getLogFilePathVia()})`, { cause }));
+    this.rejectAllPending(new BridgeTransportUnknownOutcomeError(`${this.errorPrefix} Binary crashed (restarts: ${this._restartCount})${cause ? `: ${cause.message}` : ""} (see ${this.getLogFilePathVia()})`, { cause }));
     if (this._retiringDueToBinaryChange) {
       this.logVia("Binary exited while retiring after an on-disk update; skipping auto-restart");
       return;
@@ -14364,6 +14395,7 @@ function toolErrorFromResponse(command, response) {
 }
 var BASH_TRANSPORT_DISPOSITION = "The transport to the AFT daemon was interrupted; no background task was created for this command and no task ID exists. Re-run the command. Do not poll bash_status for it.";
 var SUBC_MODULE_RESTART_DISPOSITION = "The AFT daemon module restarted while this call was in flight, so its outcome is UNKNOWN: it may or may not have executed. Verify actual state before re-running, and never blind-retry a mutation.";
+var BRIDGE_TRANSPORT_UNKNOWN_OUTCOME_DISPOSITION = "The standalone AFT transport failed after this call may have been sent, so its outcome is UNKNOWN: it may or may not have executed. Verify actual state before re-running, and never blind-retry a mutation.";
 function isRouteGoodbyeError(error2) {
   if (!(error2 instanceof SubcError))
     return false;
@@ -14386,6 +14418,8 @@ function isBashTransportDeadError(error2) {
   if (!(error2 instanceof Error) || hasEngineResponse(error2) || isRouteGoodbyeError(error2)) {
     return false;
   }
+  if (error2 instanceof BridgeTransportUnknownOutcomeError)
+    return false;
   return error2 instanceof BridgeTransportUnavailableError || error2 instanceof SubcTransportShuttingDownError || isConsumerReconnectTransient(error2) || error2 instanceof StaleRouteHandleError || error2 instanceof SubcRootGenerationExpiredError || error2 instanceof SubcRootReapedError;
 }
 function isTransportClassError(error2) {
@@ -14394,6 +14428,12 @@ function isTransportClassError(error2) {
 function adaptToolError(command, error2) {
   if (!(error2 instanceof Error))
     return error2;
+  if (error2 instanceof BridgeTransportUnknownOutcomeError) {
+    if (error2.message.includes(BRIDGE_TRANSPORT_UNKNOWN_OUTCOME_DISPOSITION))
+      return error2;
+    error2.message = error2.message ? `${error2.message} ${BRIDGE_TRANSPORT_UNKNOWN_OUTCOME_DISPOSITION}` : BRIDGE_TRANSPORT_UNKNOWN_OUTCOME_DISPOSITION;
+    return error2;
+  }
   if (isRouteGoodbyeError(error2)) {
     if (error2.message.includes(SUBC_MODULE_RESTART_DISPOSITION))
       return error2;
@@ -17977,7 +18017,7 @@ import {
 // package.json
 var package_default = {
   name: "@cortexkit/aft-pi",
-  version: "0.51.1",
+  version: "0.51.2",
   type: "module",
   description: "Pi coding agent extension for Agent File Tools (AFT) — tree-sitter and LSP-powered code analysis",
   main: "dist/index.js",
@@ -18000,19 +18040,19 @@ var package_default = {
     "test:unit": "bun test src/__tests__ --path-ignore-patterns 'src/__tests__/e2e/**'"
   },
   dependencies: {
-    "@cortexkit/aft-bridge": "0.51.1",
+    "@cortexkit/aft-bridge": "0.51.2",
     "comment-json": "^5.0.0",
     diff: "^8.0.4",
     typebox: "1.1.38",
     zod: "^4.4.3"
   },
   optionalDependencies: {
-    "@cortexkit/aft-darwin-arm64": "0.51.1",
-    "@cortexkit/aft-darwin-x64": "0.51.1",
-    "@cortexkit/aft-linux-arm64": "0.51.1",
-    "@cortexkit/aft-linux-x64": "0.51.1",
-    "@cortexkit/aft-win32-arm64": "0.51.1",
-    "@cortexkit/aft-win32-x64": "0.51.1"
+    "@cortexkit/aft-darwin-arm64": "0.51.2",
+    "@cortexkit/aft-darwin-x64": "0.51.2",
+    "@cortexkit/aft-linux-arm64": "0.51.2",
+    "@cortexkit/aft-linux-x64": "0.51.2",
+    "@cortexkit/aft-win32-arm64": "0.51.2",
+    "@cortexkit/aft-win32-x64": "0.51.2"
   },
   devDependencies: {
     "@earendil-works/pi-coding-agent": "^0.80.2",
@@ -39275,7 +39315,7 @@ var PLUGIN_VERSION = (() => {
     return "0.0.0";
   }
 })();
-var ANNOUNCEMENT_VERSION = "0.51.1";
+var ANNOUNCEMENT_VERSION = "0.51.2";
 var ANNOUNCEMENT_FEATURES = [
   "Callgraph stays consistent under load: edits arriving mid-rebuild can no longer be silently lost from navigation results.",
   "aft_zoom handles real-world .jsonc: files with comment banners above the opening brace now resolve, and not-found errors point at the segment that actually failed (thanks @iceteaSA).",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cortexkit/aft-pi",
-  "version": "0.51.1",
+  "version": "0.51.2",
   "type": "module",
   "description": "Pi coding agent extension for Agent File Tools (AFT) — tree-sitter and LSP-powered code analysis",
   "main": "dist/index.js",
@@ -23,19 +23,19 @@
     "test:unit": "bun test src/__tests__ --path-ignore-patterns 'src/__tests__/e2e/**'"
   },
   "dependencies": {
-    "@cortexkit/aft-bridge": "0.51.1",
+    "@cortexkit/aft-bridge": "0.51.2",
     "comment-json": "^5.0.0",
     "diff": "^8.0.4",
     "typebox": "1.1.38",
     "zod": "^4.4.3"
   },
   "optionalDependencies": {
-    "@cortexkit/aft-darwin-arm64": "0.51.1",
-    "@cortexkit/aft-darwin-x64": "0.51.1",
-    "@cortexkit/aft-linux-arm64": "0.51.1",
-    "@cortexkit/aft-linux-x64": "0.51.1",
-    "@cortexkit/aft-win32-arm64": "0.51.1",
-    "@cortexkit/aft-win32-x64": "0.51.1"
+    "@cortexkit/aft-darwin-arm64": "0.51.2",
+    "@cortexkit/aft-darwin-x64": "0.51.2",
+    "@cortexkit/aft-linux-arm64": "0.51.2",
+    "@cortexkit/aft-linux-x64": "0.51.2",
+    "@cortexkit/aft-win32-arm64": "0.51.2",
+    "@cortexkit/aft-win32-x64": "0.51.2"
   },
   "devDependencies": {
     "@earendil-works/pi-coding-agent": "^0.80.2",