@cortexkit/aft-pi 0.51.0 → 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";
 }
@@ -8754,6 +8761,20 @@ class BinaryBridge {
   isAlive() {
     return this.process !== null && this.process.exitCode === null && !this.process.killed;
   }
+  invalidateTransportProcess(error2) {
+    const proc = this.process;
+    if (!proc)
+      return;
+    this.process = null;
+    this.spawnedBinaryFingerprint = null;
+    if (proc.exitCode === null && !proc.killed) {
+      proc.kill("SIGKILL");
+    }
+    this.clearRestartResetTimer();
+    this.configured = false;
+    this.outstandingBackgroundTaskIds.clear();
+    this.rejectAllPending(error2 instanceof BridgeTransportUnknownOutcomeError ? error2 : new BridgeTransportUnknownOutcomeError(error2.message, { cause: error2 }));
+  }
   hasPendingRequests() {
     return this.pending.size > 0;
   }
@@ -8875,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 {
@@ -8883,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()}`);
                 }
@@ -8919,6 +8949,7 @@ class BinaryBridge {
 `;
       const keepBridgeOnTimeout = passive || options?.keepBridgeOnTimeout === true;
       let requestSentAt = Date.now();
+      const child = this.process;
       const response = await new Promise((resolve2, reject) => {
         const timer = setTimeout(() => {
           const entry = this.pending.get(id);
@@ -8955,23 +8986,36 @@ class BinaryBridge {
           this.handleTimeout(requestSessionId);
         }, effectiveTimeoutMs);
         this.pending.set(id, { resolve: resolve2, reject, timer, onProgress: options?.onProgress, command });
-        if (!this.process?.stdin?.writable) {
+        if (!child?.stdin?.writable) {
           this.pending.delete(id);
           clearTimeout(timer);
-          reject(new BridgeTransportUnavailableError(`${this.errorPrefix} stdin not writable for command "${command}"`));
+          const error2 = new BridgeTransportUnavailableError(`${this.errorPrefix} stdin not writable for command "${command}"`);
+          reject(error2);
+          if (this.process === child)
+            this.invalidateTransportProcess(error2);
           return;
         }
         requestSentAt = Date.now();
-        this.process.stdin.write(line, (err) => {
-          if (err) {
-            const entry = this.pending.get(id);
-            if (entry) {
-              this.pending.delete(id);
-              clearTimeout(entry.timer);
-              entry.reject(new BridgeTransportUnavailableError(`${this.errorPrefix} Failed to write to stdin: ${err.message}`, { cause: err }));
-            }
+        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;
@@ -9129,8 +9173,11 @@ class BinaryBridge {
     });
   }
   ensureSpawned(triggeringSessionId) {
-    if (this.isAlive())
+    if (this.isAlive() && this.process?.stdin?.writable)
       return;
+    if (this.process !== null) {
+      this.invalidateTransportProcess(new BridgeTransportUnavailableError(`${this.errorPrefix} Child stdin is not writable`));
+    }
     this.spawnProcess(triggeringSessionId);
   }
   spawnProcess(triggeringSessionId) {
@@ -9183,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);
@@ -9193,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);
@@ -9218,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();
@@ -9396,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");
@@ -9433,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;
@@ -14340,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;
@@ -14362,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) {
@@ -14370,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;
@@ -17953,7 +18017,7 @@ import {
 // package.json
 var package_default = {
   name: "@cortexkit/aft-pi",
-  version: "0.51.0",
+  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",
@@ -17976,19 +18040,19 @@ var package_default = {
     "test:unit": "bun test src/__tests__ --path-ignore-patterns 'src/__tests__/e2e/**'"
   },
   dependencies: {
-    "@cortexkit/aft-bridge": "0.51.0",
+    "@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.0",
-    "@cortexkit/aft-darwin-x64": "0.51.0",
-    "@cortexkit/aft-linux-arm64": "0.51.0",
-    "@cortexkit/aft-linux-x64": "0.51.0",
-    "@cortexkit/aft-win32-arm64": "0.51.0",
-    "@cortexkit/aft-win32-x64": "0.51.0"
+    "@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",
@@ -38880,7 +38944,7 @@ var SearchParams2 = Type13.Object({
     description: "Include test files (*.test.*, *_test.rs, __tests__/, …) plus test-support, fixture, mock, snapshot, and corpus files. Defaults to false."
   })),
   path: Type13.Optional(Type13.String({
-    description: "Search a different project root (absolute or ~ path). Requires that project to have been indexed by AFT."
+    description: "Only set this to search a different Git project (absolute or ~ path). Omit it for the current configured workspace, including non-Git workspace roots; this is not a subdirectory filter. Unindexed external projects use a bounded lexical fallback."
   }))
 });
 function buildSemanticSections(args, payload, theme) {
@@ -39251,7 +39315,7 @@ var PLUGIN_VERSION = (() => {
     return "0.0.0";
   }
 })();
-var ANNOUNCEMENT_VERSION = "0.51.0";
+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.0",
+  "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.0",
+    "@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.0",
-    "@cortexkit/aft-darwin-x64": "0.51.0",
-    "@cortexkit/aft-linux-arm64": "0.51.0",
-    "@cortexkit/aft-linux-x64": "0.51.0",
-    "@cortexkit/aft-win32-arm64": "0.51.0",
-    "@cortexkit/aft-win32-x64": "0.51.0"
+    "@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",