@skaile/workspaces 3.1.0 → 3.2.0

package/dist/{chunk-H26KDA7J.js → chunk-RW4I42MW.js}

@@ -4,10 +4,10 @@ import { resolveModelTier } from './chunk-V34ZEXSE.js';
 import { createDriver } from './chunk-QFO4IYBZ.js';
 import { pluginRegistry } from './chunk-6E6PKKAD.js';
 import { computeStimulus } from './chunk-IFGTRL2I.js';
-import { getConsumedOutputSchema, validateConsumedNodeOutput, appendAttempt, remainingActiveExecutionMs, resolveNodeBindings, interpolateNodeInstruction, deriveCheckEvidence, computeFlowStateFromSnapshots, availableRouterFrontier, evaluateRouterSelection, routerSelectionOutput, planRouterSkips, unsettledRouterBranchIds, countFailedAttempts, hasAutomaticRetryBudget, persistedRouterSelection } from './chunk-S5FV6ISY.js';
-import { parseFlowDefinition } from './chunk-5QGMEPUX.js';
-import { renderSubpromptPrompt, renderStimulusPrompt } from './chunk-7KFN2EPR.js';
-import { deterministicNodeKindLabel, flowDefaults, isAgentNode, isGateNode, isCheckNode, nodeRequiresApproval, isSubFlowNode, isFunctionNode, isSubpromptNode, nodeIsOptional, nodeInstruction, isRouterNode, isActiveRuntimeNode, nodeLabel, nodeAssets } from './chunk-F5VREBV2.js';
+import { resolveNodeBindings, getConsumedOutputSchema, validateConsumedNodeOutput, appendAttempt, resolveTerminalFailureStatus, remainingActiveExecutionMs, interpolateNodeInstruction, deriveCheckEvidence, computeFlowStateFromSnapshots, availableRouterFrontier, evaluateRouterSelection, routerSelectionOutput, planRouterSkips, unsettledRouterBranchIds, countFailedAttempts, hasAutomaticRetryBudget, persistedRouterSelection } from './chunk-EYBGQJNF.js';
+import { parseFlowDefinition } from './chunk-B4J6TV4G.js';
+import { renderSubpromptPrompt, renderStimulusPrompt } from './chunk-FLBDLF3I.js';
+import { deterministicNodeKindLabel, flowDefaults, isAgentNode, isGateNode, isCheckNode, nodeRequiresApproval, isSubFlowNode, isFunctionNode, isSubpromptNode, nodeInstruction, isRouterNode, isActiveRuntimeNode, nodeIsOptional, nodeLabel, nodeAssets } from './chunk-F5VREBV2.js';
 import { portableSpawn } from './chunk-P43HBWKK.js';
 import { withFsDeadline, tryWithFsDeadline, FsDeadlineError, fsDeadlineMs } from './chunk-DYMWJAL4.js';
 import { createLogger } from './chunk-7GOCNYUV.js';
@@ -7640,7 +7640,8 @@ function toSnapshot(exec) {
     input: exec.input,
     output: exec.output,
     error: exec.error,
-    failedPreconditions: exec.failedPreconditions
+    failedPreconditions: exec.failedPreconditions,
+    inputOrigins: exec.inputOrigins
   };
 }
 function snapshotMap(execution) {
@@ -7687,7 +7688,10 @@ function prepareExecutionSnapshot(flowDef, execution) {
         output: void 0,
         approval: void 0,
         input: void 0,
-        error: void 0
+        error: void 0,
+        // Moves with `output`: the node restarts from scratch, so origins that
+        // described the discarded attempt would be stale.
+        inputOrigins: void 0
       };
       continue;
     }
@@ -7985,7 +7989,7 @@ var FlowAdapter = class extends AbstractConnector {
         },
         {
           name: "fail_node",
-          description: "Mark a node as failed. `recoverable: true` transitions back to `available` so the runner can retry; `recoverable: false` marks the node and flow as failed.",
+          description: "Mark a node as failed. `recoverable: true` transitions back to `available` so the runner can retry; `recoverable: false` is terminal \u2014 an optional node becomes `skipped` so its dependents still proceed, any other node becomes `failed`.",
           accessLevel: "write",
           args: [
             { name: "nodeId", description: "Node ID", type: "string", required: true },
@@ -8312,17 +8316,27 @@ var FlowAdapter = class extends AbstractConnector {
   // ── Agent-side mutations (tool calls) ─────────────────────────────────────
   doStartNode(handle, nodeId) {
     return this.mutate(handle, (s) => {
-      assertLegal(
-        isAgentNode(findNode(s.flowDef, nodeId)),
-        `node is not an active agent node: ${nodeId}`
-      );
+      const definition = findNode(s.flowDef, nodeId);
+      assertLegal(isAgentNode(definition), `node is not an active agent node: ${nodeId}`);
       const node = s.execution.nodes[nodeId];
       assertLegal(node, `node not in execution: ${nodeId}`);
       assertLegal(node.status === "available", `cannot start ${nodeId}: status is ${node.status}`);
+      let inputOrigins;
+      try {
+        inputOrigins = resolveNodeBindings(
+          s.flowDef,
+          definition,
+          snapshotMap(s.execution),
+          s.flowInputOrigins
+        ).origins;
+      } catch {
+        inputOrigins = void 0;
+      }
       s.execution.nodes[nodeId] = {
         ...node,
         status: "running",
-        startedAt: now()
+        startedAt: now(),
+        inputOrigins
       };
       return JSON.stringify({ ok: true, status: "running" });
     });
@@ -8435,7 +8449,7 @@ var FlowAdapter = class extends AbstractConnector {
         ) : execution.outputHistory;
         current.execution.nodes[nodeId] = {
           ...execution,
-          status: "failed",
+          status: resolveTerminalFailureStatus(findNode(current.flowDef, nodeId)),
           completedAt,
           error,
           errorHistory: [...execution.errorHistory, error],
@@ -8538,7 +8552,7 @@ var FlowAdapter = class extends AbstractConnector {
         ) : node.outputHistory;
         s.execution.nodes[nodeId] = {
           ...node,
-          status: "failed",
+          status: resolveTerminalFailureStatus(findNode(s.flowDef, nodeId)),
           completedAt,
           error,
           errorHistory: [...node.errorHistory, error],
@@ -8606,7 +8620,7 @@ var FlowAdapter = class extends AbstractConnector {
       } else if (isCheckNode(definition)) {
         s.execution.nodes[nodeId] = {
           ...node,
-          status: "failed",
+          status: resolveTerminalFailureStatus(definition),
           completedAt: now(),
           approval,
           approvalHistory: history
@@ -8660,11 +8674,21 @@ var FlowAdapter = class extends AbstractConnector {
    * Transition a recoverable failed node back to `available` for retry.
    * Preserves the failure in `errorHistory` so the next turn's agent can
    * see the prior failure context.
+   *
+   * `failed` is the only status accepted; a `skipped` node is refused
+   * deliberately, because a skip has already released its dependents and this
+   * operation owns no cascade machinery to unwind the ones that ran without
+   * the producer's output. Reasoning in
+   * `_devlog/entries/2026-08-15-optional-node-skip-semantics.md`.
    */
   retryNode(handle, nodeId) {
     this.mutate(handle, (s) => {
       const node = s.execution.nodes[nodeId];
       assertLegal(node, `retryNode: unknown node ${nodeId}`);
+      assertLegal(
+        node.status !== "skipped",
+        `retryNode: node ${nodeId} was skipped, and a skip is not recoverable \u2014 it already released its dependents, which may have run without this node's output`
+      );
       assertLegal(
         node.status === "failed" && node.error?.recoverable === true,
         `retryNode: node ${nodeId} is not a recoverable failure (status=${node.status}, recoverable=${node.error?.recoverable})`
@@ -8837,6 +8861,7 @@ var FlowAdapter = class extends AbstractConnector {
         const runningExecution = state.execution.nodes[node.id];
         const remainingMs = runningExecution ? remainingActiveExecutionMs(runningExecution, timeoutSec, Date.now()) : void 0;
         let result;
+        let subFlowInputOrigins;
         try {
           const runner = state.inlineSubFlowRunner;
           if (!runner) {
@@ -8848,6 +8873,7 @@ var FlowAdapter = class extends AbstractConnector {
             snapshotMap(state.execution),
             state.flowInputOrigins
           );
+          subFlowInputOrigins = resolved.origins;
           const runPromise = runner.run({
             node,
             flowId: node.run.flow,
@@ -8919,7 +8945,8 @@ var FlowAdapter = class extends AbstractConnector {
                 status: "complete",
                 completedAt,
                 output: result.output,
-                outputHistory
+                outputHistory,
+                inputOrigins: subFlowInputOrigins
               };
               return;
             }
@@ -8935,7 +8962,7 @@ var FlowAdapter = class extends AbstractConnector {
             current.execution.nodes[node.id] = retry ? { ...execution2, ...patch, status: "available", error: void 0 } : {
               ...execution2,
               ...patch,
-              status: nodeIsOptional(node) ? "skipped" : "failed",
+              status: resolveTerminalFailureStatus(node),
               completedAt
             };
           },
@@ -8967,6 +8994,7 @@ var FlowAdapter = class extends AbstractConnector {
         const subpromptRunningExecution = state.execution.nodes[node.id];
         const subpromptRemainingMs = subpromptRunningExecution ? remainingActiveExecutionMs(subpromptRunningExecution, subpromptTimeoutSec, Date.now()) : void 0;
         let subpromptOutput;
+        let subpromptInputOrigins;
         let subpromptOutcome;
         try {
           const runner = state.subpromptRunner;
@@ -8979,6 +9007,7 @@ var FlowAdapter = class extends AbstractConnector {
             snapshotMap(state.execution),
             state.flowInputOrigins
           );
+          subpromptInputOrigins = resolved.origins;
           const instruction = nodeInstruction(node);
           assertLegal(instruction, `subprompt node ${node.id} has no instruction`);
           const outputSchema = node.contract?.output?.fields;
@@ -9036,7 +9065,8 @@ var FlowAdapter = class extends AbstractConnector {
                 status: "complete",
                 completedAt,
                 output: subpromptOutput,
-                outputHistory
+                outputHistory,
+                inputOrigins: subpromptInputOrigins
               };
             },
             false
@@ -9059,7 +9089,7 @@ var FlowAdapter = class extends AbstractConnector {
               current.execution.nodes[node.id] = retry ? { ...execution, ...patch, status: "available", error: void 0 } : {
                 ...execution,
                 ...patch,
-                status: nodeIsOptional(node) ? "skipped" : "failed",
+                status: resolveTerminalFailureStatus(node),
                 completedAt
               };
             },
@@ -9096,6 +9126,7 @@ var FlowAdapter = class extends AbstractConnector {
       ) : void 0;
       const deterministicKindLabel = deterministicNodeKindLabel(node.run.kind);
       let capturedOutput;
+      let capturedInputOrigins;
       let checkEvidence;
       let deterministicOutcome;
       try {
@@ -9108,6 +9139,7 @@ var FlowAdapter = class extends AbstractConnector {
           snapshotMap(state.execution),
           state.flowInputOrigins
         );
+        capturedInputOrigins = resolved.origins;
         const runPromise = executor.execute({
           node,
           inputs: resolved.values,
@@ -9188,7 +9220,8 @@ var FlowAdapter = class extends AbstractConnector {
               status: "complete",
               completedAt,
               output: capturedOutput,
-              outputHistory
+              outputHistory,
+              inputOrigins: capturedInputOrigins
             };
           },
           false
@@ -9222,7 +9255,7 @@ var FlowAdapter = class extends AbstractConnector {
               current.execution.nodes[node.id] = {
                 ...execution,
                 ...patch,
-                status: "failed",
+                status: resolveTerminalFailureStatus(node),
                 completedAt
               };
             } else {
@@ -9257,7 +9290,7 @@ var FlowAdapter = class extends AbstractConnector {
               completedAt,
               capturedOutput
             );
-            current.execution.nodes[node.id] = retry ? { ...execution, ...patch, status: "available", error: void 0 } : { ...execution, ...patch, status: "failed", completedAt };
+            current.execution.nodes[node.id] = retry ? { ...execution, ...patch, status: "available", error: void 0 } : { ...execution, ...patch, status: resolveTerminalFailureStatus(node), completedAt };
           },
           false
         );
@@ -9337,7 +9370,9 @@ var FlowAdapter = class extends AbstractConnector {
             };
             current.execution.nodes[settlement.routerNodeId] = {
               ...execution,
-              status: "failed",
+              status: resolveTerminalFailureStatus(
+                findNode(current.flowDef, settlement.routerNodeId)
+              ),
               startedAt: execution.startedAt ?? settledAt,
               completedAt: settledAt,
               output,
@@ -9563,7 +9598,7 @@ var FlowAdapter = class extends AbstractConnector {
             completedAt,
             execution.output
           );
-          current.execution.nodes[node.id] = retry ? { ...execution, ...patch, status: "available", error: void 0 } : { ...execution, ...patch, status: "failed", completedAt };
+          current.execution.nodes[node.id] = retry ? { ...execution, ...patch, status: "available", error: void 0 } : { ...execution, ...patch, status: resolveTerminalFailureStatus(node), completedAt };
         },
         true
       );
@@ -9579,13 +9614,15 @@ var FlowAdapter = class extends AbstractConnector {
    * once, the same way, everywhere.
    *
    * Returns the `NodeExecution` patch (error/errorHistory/outputHistory/
-   * output) plus whether budget remains; the caller decides the resulting
-   * `status` because that differs by kind (e.g. sub-flow degrades an
-   * optional node's exhausted failure to `skipped`; agent/function do not,
-   * preserving their existing behavior). A terminal failure whose node has
-   * any `control.retries` configured carries `recoverable: true` so the
-   * manual `retryNode` operator override — which sits outside this budget
-   * entirely — remains usable once the automatic budget runs out.
+   * output) plus whether budget remains. The terminal *failure* status is
+   * uniform across kinds and computed by {@link resolveTerminalFailureStatus};
+   * the caller still writes the status because its non-failure outcomes differ
+   * (a check's escalation parks at `awaiting_approval`, a surviving budget
+   * returns to `available`). A terminal failure whose node has any
+   * `control.retries` configured carries `recoverable: true` so the manual
+   * `retryNode` operator override — which sits outside this budget entirely —
+   * remains usable once the automatic budget runs out; a node that will be
+   * written `skipped` never claims it, because a skip is not retryable.
    *
    * @param sourceErrorAtIso - The failure's own timestamp, when the caller has
    *   one more precise than `completedAtIso` (e.g. a sub-flow child reports
@@ -9602,7 +9639,7 @@ var FlowAdapter = class extends AbstractConnector {
     const retry = hasAutomaticRetryBudget(priorFailures, retries);
     const error = {
       message,
-      recoverable: !retry && !!retries,
+      recoverable: !retry && !!retries && resolveTerminalFailureStatus(node) === "failed",
       at: sourceErrorAtIso ?? completedAtIso
     };
     const outputHistory = appendAttempt(
@@ -9822,5 +9859,5 @@ function createConnector() {
 }
 
 export { AbstractConnector, BUILTIN_CONNECTOR_CATALOG, BoxConnector, ConnectorFieldMissingError, ConnectorManager, ConnectorStartupError, DeployConnector, DevServerConnector, FlowAdapter, GitConnector, GmailConnector, GoogleDriveConnector, LocalConnector, LogBuffer, ManagedGitconfigCollisionError, MattermostConnector, MemoryConnector, MinIOConnector, PortPool, PostgresConnector, S3Connector, SQLiteConnector, SessionInlineSubFlowRunner, SessionNodeExecutor, SessionSubpromptRunner, SharePointConnector, StaticServerConnector, TunnelConnector, WebDAVConnector, XStateConnector, XStateStoreConnector, __test, atomicReplaceCredential, buildConnectorPromptSection, buildSdkConnectorTools, buildSdkFlowTools, createConnector, createConnector10, createConnector11, createConnector12, createConnector13, createConnector14, createConnector15, createConnector16, createConnector17, createConnector18, createConnector19, createConnector2, createConnector20, createConnector3, createConnector4, createConnector5, createConnector6, createConnector7, createConnector8, createConnector9, createFsWatcher, createWorktree, defaultSpawn, ensureDirMode, ensureFleetMounted, findGitRoot, findMissingPackages, getConnector, installNpmPackages, isFleetManaged, isPackageResolvable, listConnectors, listWorktrees, readGitSyncStatus, registerBuiltinConnectors, removeMountBlock, renderCredentialHelperScript, tryGetConnector, worktreeRegistry, writeHelperScript, writeMountBlock };
-//# sourceMappingURL=chunk-H26KDA7J.js.map
-//# sourceMappingURL=chunk-H26KDA7J.js.map
+//# sourceMappingURL=chunk-RW4I42MW.js.map
+//# sourceMappingURL=chunk-RW4I42MW.js.map