@opengeni/runtime 4.3.0-canary.36855269964001 → 4.3.0

package/src/sandbox/routing/routing-session.ts

@@ -30,17 +30,13 @@
 import { SandboxWorkspaceReadNotFoundError } from "@openai/agents/sandbox";
 import { SandboxFilesystemNotFoundError } from "modal";
 import type { ExposedPortEndpoint } from "../stream-port";
-import {
-  ModalCommandStartPreDispatchUnavailableError,
-  ModalCommandStartNotDispatchedError,
-} from "../providers/modal-command-router-wire";
+import { ModalCommandStartPreDispatchUnavailableError } from "../providers/modal-command-router-wire";
 import { isDeepStrictEqual } from "node:util";
 import {
   withProviderCommandHandle,
   withCommandSupervisionReady,
   withSupervisedLaunchReservation,
   ProviderCommandStartRejectedError,
-  ProviderCommandStartOutcomeUnknownError,
   type ProviderCommandPersistence,
   type ProviderCommandSession,
 } from "../provider-command-session";
@@ -251,11 +247,11 @@ export interface RoutingSandboxSessionDeps {
     op: string;
     backend: ResolvedActiveBackend;
     admission: unknown;
-    outcome: "resolved" | "rejected" | "outcome_unknown";
-    /** Provider result for a resolved call. Absent when Start is unknown. */
+    outcome: "resolved" | "rejected";
+    /** Provider result for a resolved call. Never present for rejection. */
     result?: unknown;
-    /** Exact candidate for a yielded process or a client-chosen unknown Start.
-     * Unknown Start settlement retains this writer; it does not prove exit. */
+    /** Stable candidate generated before durable promotion is attempted. It is
+     * supplied only when the provider returned a positive yielded-session id. */
     retainedProcess?: RoutingRetainedProcess;
   }) => Promise<void | RoutingMutationSettlementResult>;
   /** Admit one model/user-visible stdin mutation under the already-durable
@@ -525,15 +521,6 @@ export class RoutingRetainedProcessNotFoundError extends Error {
   }
 }
 
-export function renderRoutingMutationOutcomeUnknownToolResult(
-  error: RoutingMutationOutcomeUnknownError,
-): string {
-  const locator = error.retainedProcess
-    ? ` Inspect the exact invocation with write_stdin using session_id ${error.retainedProcess.providerSessionId} and empty chars.`
-    : " Inspect state before deciding how to proceed.";
-  return `Sandbox operation outcome unknown; the operation was not replayed. Do not blindly retry the command.${locator} Error: ${error}`;
-}
-
 type PendingParentPromotion = Parameters<
   NonNullable<RoutingSandboxSessionDeps["afterMutation"]>
 >[0];
@@ -1083,8 +1070,7 @@ export class RoutingSandboxSession implements RoutableBackendSession {
         { cause: error },
       );
     }
-    if (pending.result !== undefined)
-      await this.captureRetainedOutput(record, formatExecResult(pending.result));
+    await this.captureRetainedOutput(record, formatExecResult(pending.result));
   }
 
   private confirmDurableRejectedPromotion(
@@ -1724,50 +1710,9 @@ export class RoutingSandboxSession implements RoutableBackendSession {
           );
         }
       } catch (error) {
-        if (
-          !reservedProcess &&
-          error instanceof ProviderCommandStartOutcomeUnknownError &&
-          mutatesWorkspace &&
-          this.deps.afterMutation
-        ) {
-          const handle = this.deps.providerCommandHandle?.(admission);
-          if (!handle) throw error;
-          const process: RoutingRetainedProcess = {
-            id: crypto.randomUUID(),
-            providerSessionId: handle,
-            providerCommand: error.command,
-          };
-          const record = this.registerRetainedProcess(process, backend);
-          const settlement: PendingParentPromotion = {
-            op,
-            backend,
-            admission,
-            outcome: "outcome_unknown",
-            retainedProcess: process,
-          };
-          record.pendingParentPromotion = settlement;
-          try {
-            const rejected = await this.deps.afterMutation(settlement);
-            if (rejected) this.confirmDurableRejectedPromotion(record, rejected);
-            else record.durable = true;
-            record.pendingParentPromotion = null;
-          } catch (cause) {
-            throw new RoutingMutationOutcomeUnknownError(
-              op,
-              "Command Start outcome is unknown and exact invocation promotion remains unresolved; it was not replayed",
-              { cause: new AggregateError([error, cause]), retainedProcess: process },
-            );
-          }
-          throw new RoutingMutationOutcomeUnknownError(
-            op,
-            "Command Start outcome is unknown; its exact invocation is retained and was not replayed",
-            { cause: error, retainedProcess: process },
-          );
-        }
         if (reservedProcess) {
           if (
             (error instanceof ProviderCommandStartRejectedError ||
-              error instanceof ModalCommandStartNotDispatchedError ||
               error instanceof ModalCommandStartPreDispatchUnavailableError) &&
             reservedProcess.providerCommand?.kind === "modal-router-v1"
           ) {

package/src/sandbox/turn-tool-cancellation.ts

@@ -5,10 +5,7 @@ import {
   parseExecBannerExitCode,
   parseExecBannerSessionId,
 } from "./exec-banner";
-import {
-  RoutingMutationOutcomeUnknownError,
-  renderRoutingMutationOutcomeUnknownToolResult,
-} from "./routing/routing-session";
+import { RoutingMutationOutcomeUnknownError } from "./routing/routing-session";
 import { sendCommandInput } from "./command-input";
 import {
   withPendingCommandSupervision,
@@ -636,8 +633,6 @@ export function renderDirectToolFault(error: unknown, retainedProcessSessionId?:
   } catch {
     // A hostile error Proxy must not replace the original safe fault renderer.
   }
-  if (error instanceof RoutingMutationOutcomeUnknownError)
-    return renderRoutingMutationOutcomeUnknownToolResult(error);
   const details = error instanceof Error ? error.toString() : String(error);
   return `An error occurred while running the tool. Please try again. Error: ${details}`;
 }
@@ -1231,8 +1226,6 @@ class TurnToolCancellationControllerImpl implements TurnToolCancellationControll
                 });
               }
               pendingStart?.settle();
-              if (error instanceof RoutingMutationOutcomeUnknownError)
-                return renderRoutingMutationOutcomeUnknownToolResult(error);
               throw error;
             } finally {
               remoteExec?.settle();