@cortexkit/aft 0.51.0 → 0.51.2

package/dist/index.js

@@ -764,7 +764,7 @@ function coerceConfigureDroppedKeys(value) {
 function isBridgeTransportTimeout(err) {
   return err instanceof Error && err.code === "transport_timeout";
 }
-var DEFAULT_BRIDGE_TIMEOUT_MS = 30000, BRIDGE_HANG_TIMEOUT_THRESHOLD = 2, MAX_STDOUT_BUFFER, STDOUT_BUFFER_COMPACT_THRESHOLD, HASHLINE_REGISTRATION_LOG_INTERVAL_MS = 60000, HASHLINE_REGISTRATION_LOG_STATE_LIMIT = 256, TERMINAL_BASH_STATUSES, binaryFingerprintReader, BENIGN_CPUINFO_PROC_CPUINFO_PARSE_FAILURE = "failed to parse processor information from /proc/cpuinfo", BridgeReplacedDuringVersionCheck, BridgeTransportTimeoutError, BridgeTransportUnavailableError, BinaryBridge;
+var DEFAULT_BRIDGE_TIMEOUT_MS = 30000, BRIDGE_HANG_TIMEOUT_THRESHOLD = 2, MAX_STDOUT_BUFFER, STDOUT_BUFFER_COMPACT_THRESHOLD, HASHLINE_REGISTRATION_LOG_INTERVAL_MS = 60000, HASHLINE_REGISTRATION_LOG_STATE_LIMIT = 256, TERMINAL_BASH_STATUSES, binaryFingerprintReader, BENIGN_CPUINFO_PROC_CPUINFO_PARSE_FAILURE = "failed to parse processor information from /proc/cpuinfo", BridgeReplacedDuringVersionCheck, BridgeTransportTimeoutError, BridgeTransportUnavailableError, BridgeTransportUnknownOutcomeError, BinaryBridge;
 var init_bridge = __esm(() => {
   init_active_logger();
   init_command_timeouts();
@@ -805,6 +805,12 @@ var init_bridge = __esm(() => {
       this.name = "BridgeTransportUnavailableError";
     }
   };
+  BridgeTransportUnknownOutcomeError = class BridgeTransportUnknownOutcomeError extends BridgeTransportUnavailableError {
+    constructor(message, options) {
+      super(message, options);
+      this.name = "BridgeTransportUnknownOutcomeError";
+    }
+  };
   BinaryBridge = class BinaryBridge {
     static RESTART_RESET_MS = 5 * 60 * 1000;
     static STDERR_TAIL_MAX = 20;
@@ -942,6 +948,20 @@ var init_bridge = __esm(() => {
     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;
     }
@@ -1063,6 +1083,12 @@ var init_bridge = __esm(() => {
         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 {
@@ -1071,11 +1097,14 @@ var init_bridge = __esm(() => {
                     ...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()}`);
                   }
@@ -1107,6 +1136,7 @@ var init_bridge = __esm(() => {
 `;
         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);
@@ -1143,23 +1173,36 @@ var init_bridge = __esm(() => {
             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;
@@ -1317,8 +1360,11 @@ var init_bridge = __esm(() => {
       });
     }
     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) {
@@ -1371,9 +1417,13 @@ var init_bridge = __esm(() => {
       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);
@@ -1381,9 +1431,13 @@ var init_bridge = __esm(() => {
       });
       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);
@@ -1406,7 +1460,7 @@ var init_bridge = __esm(() => {
           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();
@@ -1584,7 +1638,7 @@ var init_bridge = __esm(() => {
     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");
@@ -1621,7 +1675,7 @@ var init_bridge = __esm(() => {
       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;
@@ -6569,6 +6623,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) {
@@ -6577,6 +6633,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;
@@ -6590,7 +6652,7 @@ function adaptToolError(command, error2) {
   error2.message = error2.message ? `${error2.message} ${BASH_TRANSPORT_DISPOSITION}` : BASH_TRANSPORT_DISPOSITION;
   return error2;
 }
-var AftToolError, 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.", 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 AftToolError, 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.", 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.", 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.";
 var init_error_contract = __esm(() => {
   init_dist();
   init_bridge();
@@ -9721,10 +9783,12 @@ __export(exports_dist, {
   HomeProjectRootError: () => HomeProjectRootError,
   DEFAULT_LOG_GENERATIONS: () => DEFAULT_LOG_GENERATIONS,
   DEFAULT_LOG_BYTES: () => DEFAULT_LOG_BYTES,
+  BridgeTransportUnknownOutcomeError: () => BridgeTransportUnknownOutcomeError,
   BridgeTransportUnavailableError: () => BridgeTransportUnavailableError,
   BridgeTransportTimeoutError: () => BridgeTransportTimeoutError,
   BridgePool: () => BridgePool,
   BinaryBridge: () => BinaryBridge,
+  BRIDGE_TRANSPORT_UNKNOWN_OUTCOME_DISPOSITION: () => BRIDGE_TRANSPORT_UNKNOWN_OUTCOME_DISPOSITION,
   BASH_TRANSPORT_DISPOSITION: () => BASH_TRANSPORT_DISPOSITION,
   BASH_HOST_FALLBACK_REFUSAL: () => BASH_HOST_FALLBACK_REFUSAL,
   BASH_HOST_FALLBACK_MAX_TIMEOUT_MS: () => BASH_HOST_FALLBACK_MAX_TIMEOUT_MS,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cortexkit/aft",
-  "version": "0.51.0",
+  "version": "0.51.2",
   "type": "module",
   "description": "Unified CLI for Agent File Tools (AFT) — setup, doctor, and diagnostics across supported agent harnesses (OpenCode, Pi)",
   "license": "MIT",
@@ -24,7 +24,7 @@
   },
   "dependencies": {
     "@clack/prompts": "^1.6.0",
-    "@cortexkit/aft-bridge": "0.51.0",
+    "@cortexkit/aft-bridge": "0.51.2",
     "comment-json": "^4.6.2"
   },
   "devDependencies": {