@arbidocs/blocks 0.4.83 → 0.4.85

package/dist/{chunk-RYYYHPFU.cjs → chunk-RZUMQM7W.cjs}

@@ -680,10 +680,12 @@ function stepStatesFromActivity(events) {
     const approved = Array.isArray(detail.approved) ? detail.approved.filter(asRecord) : void 0;
     const finished2 = status === "done" || status === "skipped" || status === "failed";
     const declinedBy = str2(detail.declined_by);
+    const afterFailure = status === "skipped" && (before?.status === "failed" || !!before?.skippedAfterFailure);
     states[step] = {
       status,
       responseId,
-      error: str2(detail.error) ?? null,
+      error: str2(detail.error) ?? (afterFailure ? before?.error ?? null : null),
+      ...afterFailure ? { skippedAfterFailure: true } : {},
       ...startedAt ? { startedAt } : {},
       ...action ? { action } : {},
       ...documents?.length ? { documents } : {},
@@ -717,7 +719,7 @@ var isRunFinished = (steps, states) => steps.length > 0 && steps.every((s) => fi
 var isPersonStep = (step) => step.kind !== "agent" && step.kind !== "action";
 var isPersonAction = (step) => step.kind === "action" && (step.action.type === "sign" || step.action.type === "email_draft");
 var canAssign = (step) => isPersonStep(step) || isPersonAction(step);
-function nextAction(steps, states, current) {
+function nextAction(steps, states, current, { needsStart = false, canRetry = false } = {}) {
   const i = steps.findIndex((s) => s.id === current);
   const step = steps[i];
   if (!step) return "wait";
@@ -732,9 +734,17 @@ function nextAction(steps, states, current) {
   if (isPersonStep(step)) return "complete";
   if (phase === "waiting" && step.kind === "action" && step.action.type === "email_draft")
     return "complete";
+  if (phase === "ready" && needsStart) return "start";
   if (phase !== "failed") return independentNext ? "move" : "wait";
   const waitedFor = steps.slice(i + 1).some((s) => dependsOn(steps, s.id, step.id));
-  return waitedFor || !following ? "skip" : "move";
+  if (!waitedFor && following) return "move";
+  return canRetry ? "retry" : "wait";
+}
+function failedUpstream(steps, states, id) {
+  return upstreamSteps(steps, id).flatMap((s) => {
+    const state = states[s.id];
+    return state && (state.status === "failed" || state.skippedAfterFailure) ? [{ step: s, state }] : [];
+  });
 }
 function validateStep(step, inputs) {
   const fields = stepFields(step);
@@ -1131,26 +1141,33 @@ function useWorkflowView(options) {
     },
     [following]
   );
-  const action = busy ? "wait" : nextAction(steps, states, current);
+  const actionOptions = react.useCallback(
+    (o) => ({
+      needsStart: !!o.start && !!o.needsStart,
+      canRetry: !!o.retry
+    }),
+    []
+  );
+  const action = busy ? "wait" : nextAction(steps, states, current, actionOptions(options));
   const next = react.useCallback(async () => {
     const { steps: s, states: st, inputs: values } = optionsRef.current;
     const at = s.find((x) => x.id === current);
-    const act2 = nextAction(s, st, current);
+    const act2 = nextAction(s, st, current, actionOptions(optionsRef.current));
     if (!at || act2 === "wait" || busy) return;
     setActionError(null);
     if (act2 === "finish") return setCurrent(null);
     if (act2 === "move") return setCurrent(following(current));
     setBusy(true);
     try {
-      if (act2 === "skip") {
-        await optionsRef.current.skip(at.id);
-      } else {
-        if (Object.keys(validateStep(at, values)).length > 0) {
-          setShowErrors(true);
-          return;
-        }
-        await optionsRef.current.complete(at);
+      if (act2 === "start" || act2 === "retry") {
+        await (act2 === "start" ? optionsRef.current.start?.() : optionsRef.current.retry?.(at.id));
+        return;
+      }
+      if (Object.keys(validateStep(at, values)).length > 0) {
+        setShowErrors(true);
+        return;
       }
+      await optionsRef.current.complete(at);
       setShowErrors(false);
       setCurrent(onward(at.id) ?? current);
     } catch (e) {
@@ -1158,7 +1175,7 @@ function useWorkflowView(options) {
     } finally {
       setBusy(false);
     }
-  }, [current, busy, following, onward]);
+  }, [current, busy, following, onward, actionOptions]);
   const goTo = react.useCallback((stepId) => {
     setShowErrors(false);
     setActionError(null);
@@ -1248,11 +1265,14 @@ function useWorkflowView(options) {
     setInput,
     stateOf: (id) => states[id],
     phaseOf: (id) => stepPhase(steps, states, id),
+    failedUpstreamOf: (id) => failedUpstream(steps, states, id),
     isLocked,
     errors: showErrors && step && !isLocked(step.id) ? validateStep(step, inputs) : {},
     busy,
     actionError,
     isRunning: Object.values(states).some((s) => s.status === "running"),
+    needsStart: !!options.start && !!options.needsStart,
+    runError: options.runError ?? null,
     action,
     canBack: (current === null || index > 0) && !busy,
     next,
@@ -1347,6 +1367,7 @@ function stepWindow({
     canBack: start > 0,
     canForward: end < n,
     runningHidden: all.filter((s) => phaseOf(s.id) === "running" && !shown.has(s.id)),
+    failedHidden: all.filter((s) => phaseOf(s.id) === "failed" && !shown.has(s.id)),
     position: Math.max(1, all.findIndex((s) => s.id === at) + 1),
     total: all.length
   };
@@ -1370,6 +1391,7 @@ var DEFAULT_STEP_BAR_LABELS = {
   inParallel: "In parallel",
   working: "Working: {title}",
   moreWorking: "+{count} working",
+  failedStep: "Failed: {title}",
   waitingFor: "Waiting for {name}",
   phases: {
     blocked: "Not started",
@@ -1591,6 +1613,21 @@ function StepBarFrame({
     },
     s.id
   );
+  const failedPill = (s) => /* @__PURE__ */ jsxRuntime.jsxs(
+    "button",
+    {
+      type: "button",
+      onClick: () => select(s.id),
+      className: "flex min-w-0 max-w-[14rem] shrink items-center gap-1.5 rounded-full border border-destructive/40 bg-destructive/5 px-2 py-1 text-xs text-destructive hover:bg-destructive/10",
+      title: fill(l.failedStep, { title: s.title }),
+      "data-testid": tid(`failed-${s.id}`),
+      children: [
+        /* @__PURE__ */ jsxRuntime.jsx(lucideReact.AlertTriangle, { className: "size-3.5 shrink-0" }),
+        /* @__PURE__ */ jsxRuntime.jsx("span", { className: "truncate", children: fill(l.failedStep, { title: s.title }) })
+      ]
+    },
+    s.id
+  );
   return /* @__PURE__ */ jsxRuntime.jsxs(
     "nav",
     {
@@ -1678,6 +1715,17 @@ function StepBarFrame({
               ] })
             ]
           }
+        ),
+        win.failedHidden.length > 0 && /* @__PURE__ */ jsxRuntime.jsx(
+          "div",
+          {
+            className: ui.cn(
+              "flex min-w-[6.5rem] shrink-[1000] items-center gap-1",
+              narrow && "basis-full"
+            ),
+            "data-testid": tid("failed"),
+            children: win.failedHidden.map(failedPill)
+          }
         )
       ]
     }
@@ -2011,6 +2059,7 @@ function ActionStepBody({
 }) {
   const state = run.stateOf(step.id);
   const phase = run.phaseOf(step.id);
+  const underWay = phase === "running" || phase === "ready" && !run.needsStart;
   const name = (id) => run.nameOf?.(id) ?? id;
   const [declining, setDeclining] = react.useState(false);
   const [note, setNote] = react.useState("");
@@ -2035,7 +2084,7 @@ function ActionStepBody({
     const mine = !actions?.me || signers.includes(actions.me) && !signed.has(actions.me);
     const signedRef = state?.status === "done" ? state.documents?.[0] : void 0;
     body = /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
-      (phase === "ready" || phase === "running") && working(l.preparingSign),
+      underWay && working(l.preparingSign),
       phase === "waiting" && waiting?.doc_ext_id && /* @__PURE__ */ jsxRuntime.jsx(
         DocChip,
         {
@@ -2145,7 +2194,7 @@ function ActionStepBody({
   } else if (a.type === "email_draft") {
     const draftId = state?.waiting?.draft_id ?? (typeof state?.output?.draft_id === "string" ? state.output.draft_id : void 0);
     body = /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
-      (phase === "ready" || phase === "running") && working(l.preparingDraft),
+      underWay && working(l.preparingDraft),
       phase === "waiting" && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-muted-foreground", children: l.draftReady }),
       phase === "done" && /* @__PURE__ */ jsxRuntime.jsxs("p", { className: "flex items-center gap-1.5 text-sm", "data-testid": `${testId}-sent`, children: [
         /* @__PURE__ */ jsxRuntime.jsx(lucideReact.Check, { className: "size-4 text-primary" }),
@@ -2171,7 +2220,7 @@ function ActionStepBody({
     const tags = state?.output?.tag_ext_ids ?? a.tags;
     const docs = state?.output?.doc_ext_ids ?? [];
     body = /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
-      (phase === "ready" || phase === "running") && working(l.reviewRunning),
+      underWay && working(l.reviewRunning),
       phase === "done" && /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
         /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-muted-foreground", "data-testid": `${testId}-reviewed`, children: l.reviewDone.replace("{docs}", String(docs.length)).replace("{cols}", String(tags.length)) }),
         actions?.openReview && /* @__PURE__ */ jsxRuntime.jsxs(
@@ -2222,7 +2271,9 @@ function WorkflowStepBar({
     const isCurrent = s.id === current;
     const assignee = ["ready", "waiting"].includes(phase) ? run.assigneeOf(s.id) : void 0;
     const waiting = assignee ? l.waitingFor.replace("{name}", assignee) : void 0;
-    const title = waiting ? `${s.title} \xB7 ${waiting}` : s.title;
+    const state = s.id === DONE ? void 0 : run.stateOf(s.id);
+    const failure = (phase === "failed" || state?.skippedAfterFailure) && state?.error ? `${l.phases.failed}: ${state.error}` : void 0;
+    const title = [s.title, waiting, failure].filter(Boolean).join(" \xB7 ");
     const name = `${title} (${l.phases[phase]})`;
     if (s.id === DONE)
       return /* @__PURE__ */ jsxRuntime.jsxs(
@@ -2298,15 +2349,19 @@ function WorkflowStepBar({
 var DEFAULT_WORKFLOW_RUNNER_LABELS = {
   back: "Back",
   next: "Continue",
-  skip: "Skip",
   skipStep: "Skip this step",
   finish: "Finish",
   approve: "Approve",
   working: "Working\u2026",
   stepWorking: "{step}\u2026 {elapsed}",
   starting: "Starting\u2026",
+  start: "Start",
+  notStarted: "Not started yet. Press Start to run this step.",
   retry: "Retry",
   failed: "This step did not finish.",
+  upstreamFailed: "\u201C{step}\u201D failed{error}. Retry it before this step can go on.",
+  upstreamSkippedAfterFailure: "\u201C{step}\u201D failed{error} and was skipped: this step goes on without its result.",
+  skippedAfterFailure: "Skipped after it failed{error}.",
   blocked: "Starts after {steps}.",
   awaitingReview: "Waiting for approval.",
   approved: "Approved.",
@@ -2375,7 +2430,7 @@ function WorkflowRunner({
   const unsent = added.filter((f) => f.status === "upload-failed").length;
   const uploadHold = step?.kind === "upload" && (sending > 0 || uploading || unsent > 0);
   const docsWait = !!step && waitingDocuments(run.stateOf(step.id)).length > 0;
-  const nextLabel = sending ? sending === 1 ? l.uploadingFile : l.uploadingFiles.replace("{count}", String(sending)) : uploadHold && uploading ? l.uploadingAny : uploadHold ? unsent === 1 ? l.uploadFailed : l.uploadsFailed.replace("{count}", String(unsent)) : reading ? l.readingFiles.replace("{current}", String(Math.min(added.length - unread + 1, added.length))).replace("{count}", String(added.length)) : action === "skip" ? l.skip : action === "finish" ? l.finish : action === "complete" && step?.kind === "review" ? l.approve : action === "complete" && step?.kind === "action" ? l.markSent : action === "wait" && (run.isRunning || run.busy || docsWait) ? l.working : (step && !run.isLocked(step.id) ? step.nextLabel : void 0) ?? l.next;
+  const nextLabel = sending ? sending === 1 ? l.uploadingFile : l.uploadingFiles.replace("{count}", String(sending)) : uploadHold && uploading ? l.uploadingAny : uploadHold ? unsent === 1 ? l.uploadFailed : l.uploadsFailed.replace("{count}", String(unsent)) : reading ? l.readingFiles.replace("{current}", String(Math.min(added.length - unread + 1, added.length))).replace("{count}", String(added.length)) : action === "start" ? l.start : action === "retry" ? l.retry : action === "finish" ? l.finish : action === "complete" && step?.kind === "review" ? l.approve : action === "complete" && step?.kind === "action" ? l.markSent : action === "wait" && (run.isRunning || run.busy || docsWait) ? l.working : (step && !run.isLocked(step.id) ? step.nextLabel : void 0) ?? l.next;
   return /* @__PURE__ */ jsxRuntime.jsxs("div", { className: ui.cn(compact3 ? "space-y-3" : "space-y-6", className), "data-testid": testIdPrefix, children: [
     !hideStepBar && /* @__PURE__ */ jsxRuntime.jsx(
       WorkflowStepBar,
@@ -2402,138 +2457,141 @@ function WorkflowRunner({
           children: l.restart
         }
       )
-    ] }) : step ? /* @__PURE__ */ jsxRuntime.jsx("div", { className: "space-y-3", children: run.phaseOf(step.id) === "blocked" ? /* @__PURE__ */ jsxRuntime.jsx(ui.Card, { className: ui.cn(box, "p-6"), "data-testid": tid(step.id), children: /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-muted-foreground", "data-testid": tid(`${step.id}-blocked`), children: l.blocked.replace(
-      "{steps}",
-      stepDependencies(steps, step.id).map((d) => steps.find((s) => s.id === d)?.title ?? d).join(", ")
-    ) }) }) : step.kind === "form" ? /* @__PURE__ */ jsxRuntime.jsxs(ui.Card, { className: ui.cn(box, "space-y-4"), "data-testid": tid(step.id), children: [
-      /* @__PURE__ */ jsxRuntime.jsx(StepHeading, { step, icon: icon(step.icon), testId: tid(`${step.id}-title`) }),
-      /* @__PURE__ */ jsxRuntime.jsx(SentBackNote, { state: run.stateOf(step.id), l: pl, testId: tid(step.id) }),
-      /* @__PURE__ */ jsxRuntime.jsx(AssigneeLine, { run, stepId: step.id, l: pl, testId: tid(step.id) }),
-      /* @__PURE__ */ jsxRuntime.jsx("fieldset", { disabled: run.isLocked(step.id), className: "contents", children: /* @__PURE__ */ jsxRuntime.jsx(
-        SchemaForm,
-        {
-          fields: step.inputs,
-          values: run.inputs,
-          onChange: run.setInput,
-          errors: run.errors,
-          renderScope,
-          renderDocuments,
-          renderIcon,
-          people,
-          onEnterSubmit: () => void run.next(),
-          testIdPrefix: tid("field")
-        }
-      ) }),
-      run.actionError && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-destructive", "data-testid": tid("action-error"), children: run.actionError }),
-      renderStepNote?.(step, run)
-    ] }) : step.kind === "upload" ? /* @__PURE__ */ jsxRuntime.jsxs(ui.Card, { className: ui.cn(box, "space-y-4"), "data-testid": tid(step.id), children: [
-      /* @__PURE__ */ jsxRuntime.jsx(SentBackNote, { state: run.stateOf(step.id), l: pl, testId: tid(step.id) }),
-      /* @__PURE__ */ jsxRuntime.jsx(AssigneeLine, { run, stepId: step.id, l: pl, testId: tid(step.id) }),
-      renderUpload?.(step) ?? /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
+    ] }) : step ? /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "space-y-3", children: [
+      /* @__PURE__ */ jsxRuntime.jsx(StepNotices, { run, step, l, tid }),
+      run.phaseOf(step.id) === "blocked" ? /* @__PURE__ */ jsxRuntime.jsx(ui.Card, { className: ui.cn(box, "p-6"), "data-testid": tid(step.id), children: /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-muted-foreground", "data-testid": tid(`${step.id}-blocked`), children: l.blocked.replace(
+        "{steps}",
+        stepDependencies(steps, step.id).map((d) => steps.find((s) => s.id === d)?.title ?? d).join(", ")
+      ) }) }) : step.kind === "form" ? /* @__PURE__ */ jsxRuntime.jsxs(ui.Card, { className: ui.cn(box, "space-y-4"), "data-testid": tid(step.id), children: [
+        /* @__PURE__ */ jsxRuntime.jsx(StepHeading, { step, icon: icon(step.icon), testId: tid(`${step.id}-title`) }),
+        /* @__PURE__ */ jsxRuntime.jsx(SentBackNote, { state: run.stateOf(step.id), l: pl, testId: tid(step.id) }),
+        /* @__PURE__ */ jsxRuntime.jsx(AssigneeLine, { run, stepId: step.id, l: pl, testId: tid(step.id) }),
+        /* @__PURE__ */ jsxRuntime.jsx("fieldset", { disabled: run.isLocked(step.id), className: "contents", children: /* @__PURE__ */ jsxRuntime.jsx(
+          SchemaForm,
+          {
+            fields: step.inputs,
+            values: run.inputs,
+            onChange: run.setInput,
+            errors: run.errors,
+            renderScope,
+            renderDocuments,
+            renderIcon,
+            people,
+            onEnterSubmit: () => void run.next(),
+            testIdPrefix: tid("field")
+          }
+        ) }),
+        run.actionError && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-destructive", "data-testid": tid("action-error"), children: run.actionError }),
+        renderStepNote?.(step, run)
+      ] }) : step.kind === "upload" ? /* @__PURE__ */ jsxRuntime.jsxs(ui.Card, { className: ui.cn(box, "space-y-4"), "data-testid": tid(step.id), children: [
+        /* @__PURE__ */ jsxRuntime.jsx(SentBackNote, { state: run.stateOf(step.id), l: pl, testId: tid(step.id) }),
+        /* @__PURE__ */ jsxRuntime.jsx(AssigneeLine, { run, stepId: step.id, l: pl, testId: tid(step.id) }),
+        renderUpload?.(step) ?? /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
+          /* @__PURE__ */ jsxRuntime.jsx(
+            chunkDETZZBG2_cjs.UploadDropzone,
+            {
+              onFiles: (picked) => void onUpload?.(picked, step),
+              uploading,
+              disabled: !onUpload,
+              accept: step.accept,
+              title: step.title,
+              description: step.description,
+              hint: step.hint,
+              testId: tid(`${step.id}-drop`)
+            }
+          ),
+          files && /* @__PURE__ */ jsxRuntime.jsx(
+            UploadFiles,
+            {
+              onRetry: onRetryUpload,
+              onRemove: onRemoveUpload,
+              added,
+              others: files.filter((f) => f.addedFor !== step.id),
+              l,
+              testId: tid(`${step.id}-files`)
+            }
+          )
+        ] }),
+        run.actionError && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-destructive", children: run.actionError }),
+        renderStepNote?.(step, run)
+      ] }) : step.kind === "review" ? /* @__PURE__ */ jsxRuntime.jsxs(ui.Card, { className: ui.cn(box, "space-y-4"), "data-testid": tid(step.id), children: [
+        /* @__PURE__ */ jsxRuntime.jsx(StepHeading, { step, icon: icon(step.icon) }),
+        /* @__PURE__ */ jsxRuntime.jsx(SentBackNote, { state: run.stateOf(step.id), l: pl, testId: tid(step.id) }),
+        /* @__PURE__ */ jsxRuntime.jsx("p", { className: "whitespace-pre-line text-sm text-muted-foreground", children: reviewInstructions(step, run.inputs) }),
+        /* @__PURE__ */ jsxRuntime.jsx(AssigneeLine, { run, stepId: step.id, l: pl, testId: tid(step.id) }),
+        !(run.phaseOf(step.id) === "done" && run.stateOf(step.id)?.by) && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm", "data-testid": tid(`${step.id}-status`), children: run.phaseOf(step.id) === "done" ? l.approved : run.phaseOf(step.id) === "skipped" ? l.skipped : l.awaitingReview }),
+        (step.fields ?? []).length > 0 && run.phaseOf(step.id) === "ready" && /* @__PURE__ */ jsxRuntime.jsx(
+          SchemaForm,
+          {
+            fields: step.fields ?? [],
+            values: run.inputs,
+            onChange: run.setInput,
+            errors: run.errors,
+            renderScope,
+            renderDocuments,
+            renderIcon,
+            people,
+            testIdPrefix: tid("field")
+          }
+        ),
         /* @__PURE__ */ jsxRuntime.jsx(
-          chunkDETZZBG2_cjs.UploadDropzone,
+          ApprovalEvidence,
           {
-            onFiles: (picked) => void onUpload?.(picked, step),
-            uploading,
-            disabled: !onUpload,
-            accept: step.accept,
-            title: step.title,
-            description: step.description,
-            hint: step.hint,
-            testId: tid(`${step.id}-drop`)
+            run,
+            step,
+            actions,
+            nameOfDoc,
+            l: pl,
+            testId: tid(step.id)
           }
         ),
-        files && /* @__PURE__ */ jsxRuntime.jsx(
-          UploadFiles,
+        run.phaseOf(step.id) === "ready" && renderReview?.(step),
+        /* @__PURE__ */ jsxRuntime.jsx(SendBackControl, { run, step, l: pl, testId: tid(step.id) }),
+        run.actionError && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-destructive", children: run.actionError }),
+        renderStepNote?.(step, run)
+      ] }) : step.kind === "action" ? /* @__PURE__ */ jsxRuntime.jsxs(ui.Card, { className: ui.cn(box, "space-y-4"), "data-testid": tid(step.id), children: [
+        /* @__PURE__ */ jsxRuntime.jsx(
+          ActionStepBody,
           {
-            onRetry: onRetryUpload,
-            onRemove: onRemoveUpload,
-            added,
-            others: files.filter((f) => f.addedFor !== step.id),
-            l,
-            testId: tid(`${step.id}-files`)
+            step,
+            run,
+            actions,
+            nameOfDoc,
+            heading: /* @__PURE__ */ jsxRuntime.jsx(StepHeading, { step, icon: icon(step.icon) }),
+            l: pl,
+            testId: tid(step.id)
           }
-        )
-      ] }),
-      run.actionError && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-destructive", children: run.actionError }),
-      renderStepNote?.(step, run)
-    ] }) : step.kind === "review" ? /* @__PURE__ */ jsxRuntime.jsxs(ui.Card, { className: ui.cn(box, "space-y-4"), "data-testid": tid(step.id), children: [
-      /* @__PURE__ */ jsxRuntime.jsx(StepHeading, { step, icon: icon(step.icon) }),
-      /* @__PURE__ */ jsxRuntime.jsx(SentBackNote, { state: run.stateOf(step.id), l: pl, testId: tid(step.id) }),
-      /* @__PURE__ */ jsxRuntime.jsx("p", { className: "whitespace-pre-line text-sm text-muted-foreground", children: reviewInstructions(step, run.inputs) }),
-      /* @__PURE__ */ jsxRuntime.jsx(AssigneeLine, { run, stepId: step.id, l: pl, testId: tid(step.id) }),
-      !(run.phaseOf(step.id) === "done" && run.stateOf(step.id)?.by) && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm", "data-testid": tid(`${step.id}-status`), children: run.phaseOf(step.id) === "done" ? l.approved : run.phaseOf(step.id) === "skipped" ? l.skipped : l.awaitingReview }),
-      (step.fields ?? []).length > 0 && run.phaseOf(step.id) === "ready" && /* @__PURE__ */ jsxRuntime.jsx(
-        SchemaForm,
-        {
-          fields: step.fields ?? [],
-          values: run.inputs,
-          onChange: run.setInput,
-          errors: run.errors,
-          renderScope,
-          renderDocuments,
-          renderIcon,
-          people,
-          testIdPrefix: tid("field")
-        }
-      ),
-      /* @__PURE__ */ jsxRuntime.jsx(
-        ApprovalEvidence,
-        {
-          run,
-          step,
-          actions,
-          nameOfDoc,
-          l: pl,
-          testId: tid(step.id)
-        }
-      ),
-      run.phaseOf(step.id) === "ready" && renderReview?.(step),
-      /* @__PURE__ */ jsxRuntime.jsx(SendBackControl, { run, step, l: pl, testId: tid(step.id) }),
-      run.actionError && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-destructive", children: run.actionError }),
-      renderStepNote?.(step, run)
-    ] }) : step.kind === "action" ? /* @__PURE__ */ jsxRuntime.jsxs(ui.Card, { className: ui.cn(box, "space-y-4"), "data-testid": tid(step.id), children: [
-      /* @__PURE__ */ jsxRuntime.jsx(
-        ActionStepBody,
+        ),
+        run.actionError && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-destructive", children: run.actionError }),
+        renderStepNote?.(step, run)
+      ] }) : /* @__PURE__ */ jsxRuntime.jsx(
+        AgentStepBody,
         {
           step,
           run,
-          actions,
+          renderAgentOutput,
+          icon: icon(step.icon),
+          renderStepNote,
+          compact: compact3,
           nameOfDoc,
-          heading: /* @__PURE__ */ jsxRuntime.jsx(StepHeading, { step, icon: icon(step.icon) }),
-          l: pl,
-          testId: tid(step.id)
+          extra: /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
+            /* @__PURE__ */ jsxRuntime.jsx(SentBackNote, { state: run.stateOf(step.id), l: pl, testId: tid(step.id) }),
+            /* @__PURE__ */ jsxRuntime.jsx(
+              RedlineOpen,
+              {
+                step,
+                state: run.stateOf(step.id),
+                actions,
+                l: pl,
+                testId: tid(step.id)
+              }
+            )
+          ] }),
+          l,
+          tid
         }
-      ),
-      run.actionError && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-destructive", children: run.actionError }),
-      renderStepNote?.(step, run)
-    ] }) : /* @__PURE__ */ jsxRuntime.jsx(
-      AgentStepBody,
-      {
-        step,
-        run,
-        renderAgentOutput,
-        icon: icon(step.icon),
-        renderStepNote,
-        compact: compact3,
-        nameOfDoc,
-        extra: /* @__PURE__ */ jsxRuntime.jsxs(jsxRuntime.Fragment, { children: [
-          /* @__PURE__ */ jsxRuntime.jsx(SentBackNote, { state: run.stateOf(step.id), l: pl, testId: tid(step.id) }),
-          /* @__PURE__ */ jsxRuntime.jsx(
-            RedlineOpen,
-            {
-              step,
-              state: run.stateOf(step.id),
-              actions,
-              l: pl,
-              testId: tid(step.id)
-            }
-          )
-        ] }),
-        l,
-        tid
-      }
-    ) }, step.id) : null,
+      )
+    ] }, step.id) : null,
     !isComplete && /* @__PURE__ */ jsxRuntime.jsxs(
       "div",
       {
@@ -2561,7 +2619,7 @@ function WorkflowRunner({
             {
               className: ui.cn("flex items-center gap-2", compact3 && "ml-auto flex-wrap justify-end"),
               children: [
-                step && action !== "skip" && // A step that commits (creates the matter, starts the thread) is what the run stands on.
+                step && // A step that commits (creates the matter, starts the thread) is what the run stands on.
                 !step.commit && ["ready", "blocked", "failed"].includes(run.phaseOf(step.id)) && /* @__PURE__ */ jsxRuntime.jsx(
                   ui.Button,
                   {
@@ -2752,7 +2810,7 @@ function AgentStepBody({
       " ",
       l.branch
     ] }),
-    (phase === "ready" || phase === "running") && !state?.text && /* @__PURE__ */ jsxRuntime.jsxs(
+    (phase === "running" || phase === "ready" && !run.needsStart) && !state?.text && /* @__PURE__ */ jsxRuntime.jsxs(
       "p",
       {
         className: "flex items-center gap-2 text-sm text-muted-foreground",
@@ -2779,7 +2837,7 @@ function AgentStepBody({
       }
     ),
     phase === "failed" && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-destructive", "data-testid": tid(`${step.id}-failed`), children: state?.error || l.failed }),
-    phase === "skipped" && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-muted-foreground", children: l.skipped }),
+    phase === "skipped" && /* @__PURE__ */ jsxRuntime.jsx("p", { className: "text-sm text-muted-foreground", "data-testid": tid(`${step.id}-skipped`), children: state?.skippedAfterFailure ? l.skippedAfterFailure.replace("{error}", state.error ? `: ${state.error}` : "") : l.skipped }),
     extra
   ] });
   if (step.output === "artifact" && !compact3) {
@@ -2806,6 +2864,48 @@ function AgentStepBody({
     renderStepNote?.(step, run)
   ] });
 }
+function StepNotices({
+  run,
+  step,
+  l,
+  tid
+}) {
+  const notStarted = !isPersonStep(step) && run.needsStart && run.phaseOf(step.id) === "ready";
+  const failed = run.failedUpstreamOf(step.id);
+  if (!notStarted && failed.length === 0) return null;
+  const why = (error) => error ? `: ${error}` : "";
+  return /* @__PURE__ */ jsxRuntime.jsxs("div", { className: "space-y-2", "data-testid": tid(`${step.id}-notices`), children: [
+    notStarted && /* @__PURE__ */ jsxRuntime.jsxs(
+      "p",
+      {
+        className: ui.cn(
+          "flex items-start gap-2 text-sm",
+          run.runError ? "text-destructive" : "text-muted-foreground"
+        ),
+        "data-testid": tid(`${step.id}-not-started`),
+        children: [
+          run.runError && /* @__PURE__ */ jsxRuntime.jsx(lucideReact.AlertTriangle, { className: "mt-0.5 size-4 shrink-0" }),
+          run.runError || l.notStarted
+        ]
+      }
+    ),
+    failed.map(({ step: s, state }) => /* @__PURE__ */ jsxRuntime.jsxs(
+      "p",
+      {
+        className: ui.cn(
+          "flex items-start gap-2 rounded-md border px-3 py-2 text-sm",
+          state.status === "failed" ? "border-destructive/40 text-destructive" : "border-amber-500/40 text-amber-700 dark:text-amber-300"
+        ),
+        "data-testid": tid(`${step.id}-upstream-${s.id}`),
+        children: [
+          /* @__PURE__ */ jsxRuntime.jsx(lucideReact.AlertTriangle, { className: "mt-0.5 size-4 shrink-0" }),
+          (state.status === "failed" ? l.upstreamFailed : l.upstreamSkippedAfterFailure).replace("{step}", s.title).replace("{error}", why(state.error))
+        ]
+      },
+      s.id
+    ))
+  ] });
+}
 
 exports.ActionStepBody = ActionStepBody;
 exports.ApprovalEvidence = ApprovalEvidence;
@@ -2887,5 +2987,5 @@ exports.workflowFromFrontmatter = workflowFromFrontmatter;
 exports.workflowSteps = workflowSteps;
 exports.workflowsAccepting = workflowsAccepting;
 exports.workflowsForHost = workflowsForHost;
-//# sourceMappingURL=chunk-RYYYHPFU.cjs.map
-//# sourceMappingURL=chunk-RYYYHPFU.cjs.map
+//# sourceMappingURL=chunk-RZUMQM7W.cjs.map
+//# sourceMappingURL=chunk-RZUMQM7W.cjs.map