@cortexkit/aft-opencode 0.51.0 → 0.51.2

package/dist/index.js

@@ -8532,6 +8532,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";
 }
@@ -8673,6 +8680,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;
   }
@@ -8794,6 +8815,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 {
@@ -8802,11 +8829,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()}`);
                 }
@@ -8838,6 +8868,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);
@@ -8874,23 +8905,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;
@@ -9048,8 +9092,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) {
@@ -9102,9 +9149,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);
@@ -9112,9 +9163,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);
@@ -9137,7 +9192,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();
@@ -9315,7 +9370,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");
@@ -9352,7 +9407,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;
@@ -13945,6 +14000,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;
@@ -13967,6 +14023,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) {
@@ -13975,6 +14033,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;
@@ -39979,7 +40043,7 @@ function semanticTools(ctx) {
       query: arg3(z15.string().describe("Concept, regex, literal text, filename, or capability to find. Examples: 'fuzzy match with whitespace tolerance', '^export', 'Cargo.lock'.")),
       topK: arg3(optionalInt(1, 100).describe("Number of results (default: 10, max: 100)")),
       includeTests: arg3(z15.boolean().optional().describe("Include test files (*.test.*, *_test.rs, __tests__/, …) plus test-support, fixture, mock, snapshot, and corpus files. Defaults to false.")),
-      path: arg3(z15.string().optional().describe("Search a different project root (absolute or ~ path). Requires that project to have been indexed by AFT."))
+      path: arg3(z15.string().optional().describe("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."))
     },
     execute: async (args, context) => {
       if (isEmptyParam(args.query) || typeof args.query !== "string" || args.query.trim().length === 0) {
@@ -40182,7 +40246,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-opencode",
-  "version": "0.51.0",
+  "version": "0.51.2",
   "type": "module",
   "description": "OpenCode plugin for Agent File Tools (AFT) — tree-sitter and lsp powered code analysis",
   "main": "dist/index.js",
@@ -34,7 +34,7 @@
   },
   "dependencies": {
     "@clack/prompts": "^1.6.0",
-    "@cortexkit/aft-bridge": "0.51.0",
+    "@cortexkit/aft-bridge": "0.51.2",
     "@opentui/core": "0.4.3",
     "@opentui/solid": "0.4.3",
     "comment-json": "^4.6.2",
@@ -43,12 +43,12 @@
     "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": {
     "@opencode-ai/plugin": "^1.17.11",