@arbidocs/blocks 0.4.76 → 0.4.77

package/dist/workflow-canvas/index.js

@@ -1,9 +1,9 @@
 "use client";
 import { useConnection } from '../chunk-OYMWV45N.js';
-import { workflowWhen, isoToLocalInput, describeRRule, RecurrenceBuilder, AgentSelect, useWorkflowPreview, localInputToIso } from '../chunk-TKB7TVMW.js';
+import { workflowWhen, isoToLocalInput, describeRRule, RecurrenceBuilder, AgentSelect, useWorkflowPreview, localInputToIso } from '../chunk-HEVVT6IT.js';
 import '../chunk-CKB64Z4R.js';
 import '../chunk-P6GHJSLR.js';
-import { workflowSteps, dependsOn, stepLevels, isIncompleteSignal, stepDependencies, runFields, fromPlanSteps, layoutFromPlanSteps, validateWorkflowGraph, WorkflowRunner, skillToWorkflow, orderSteps, toPlanSteps } from '../chunk-22P5KNIM.js';
+import { workflowSteps, dependsOn, stepLevels, isIncompleteSignal, runFields, upstreamSteps, stepDependencies, canAssign, fromPlanSteps, layoutFromPlanSteps, validateWorkflowGraph, stepConfigProblems, WorkflowRunner, skillToWorkflow, assigneeName, orderSteps, toPlanSteps } from '../chunk-34UW6XUQ.js';
 import '../chunk-4WWM3GQ5.js';
 import '../chunk-RFXJYIDV.js';
 import '../chunk-DPLRHEJZ.js';
@@ -24,13 +24,23 @@ import '../chunk-TX2ZICS7.js';
 import '../chunk-7F6CBP3Z.js';
 import '../chunk-VUNV25KB.js';
 import { useMemo, useState, useEffect, useCallback, useRef } from 'react';
-import { useWorkflows, useSkills, useAgents, useConversations, useCreateWorkflow, useUpdateWorkflow, useRunWorkflow } from '@arbidocs/react';
+import { useWorkspaceUsers, useTags, useDocuments, useWorkflows, useSkills, useAgents, useConversations, useCreateWorkflow, useUpdateWorkflow, useRunWorkflow } from '@arbidocs/react';
 import { Label, Input, cn, Button, Textarea, Checkbox, Popover, PopoverTrigger, PopoverContent } from '@arbidocs/react/ui';
-import { X, Plus, Trash2, Maximize, ClipboardCheck, Bot, Upload, FileInput, PencilRuler, Eye, LayoutGrid, Settings2, Play, Loader2, Save, Zap, GitBranch, Check, AlertTriangle, MinusCircle } from 'lucide-react';
+import { X, Plus, Trash2, Maximize, Table2, Mail, PenLine, ClipboardCheck, Bot, Upload, FileInput, PencilRuler, Eye, LayoutGrid, Settings2, Play, Loader2, Save, Zap, GitBranch, FilePenLine, UserRound, Hourglass, Check, AlertTriangle, MinusCircle } from 'lucide-react';
 import { jsxs, jsx, Fragment } from 'react/jsx-runtime';
 import { MarkerType, applyNodeChanges, ReactFlow, Background, BackgroundVariant, Controls, ControlButton, MiniMap, Handle, Position } from '@xyflow/react';
 
 // src/workflow-canvas/edit.ts
+var WORKFLOW_STEP_KINDS = [
+  "form",
+  "upload",
+  "agent",
+  "review",
+  "sign",
+  "email_draft",
+  "tabular_review"
+];
+var stepKindOf = (step) => step.kind === "action" ? step.action.type : step.kind;
 function newSteppedWorkflow(id, title) {
   return {
     id,
@@ -63,7 +73,10 @@ var DEFAULT_TITLES = {
   form: "Details",
   upload: "Upload files",
   agent: "Agent step",
-  review: "Review"
+  review: "Review",
+  sign: "Sign",
+  email_draft: "Email draft",
+  tabular_review: "Tabular review"
 };
 function blankStep(kind, id, title) {
   switch (kind) {
@@ -75,6 +88,22 @@ function blankStep(kind, id, title) {
       return { id, kind, title };
     case "review":
       return { id, kind, title, instructions: "" };
+    case "sign":
+      return { id, kind: "action", title, action: { type: "sign", signers: [] } };
+    case "email_draft":
+      return {
+        id,
+        kind: "action",
+        title,
+        action: { type: "email_draft", to: [], subject: "", body: "" }
+      };
+    case "tabular_review":
+      return {
+        id,
+        kind: "action",
+        title,
+        action: { type: "tabular_review", tags: [], documents: "run_uploads" }
+      };
   }
 }
 function addStep(workflow, kind, options = {}) {
@@ -139,14 +168,49 @@ function renameInput(workflow, from, to) {
       return {
         ...s,
         with: s.with ? Object.fromEntries(Object.entries(s.with).map(([k, v]) => [k, renameIn(v, from, to)])) : s.with,
-        run: s.run?.kind === "prompt" ? { ...s.run, promptTemplate: renameIn(s.run.promptTemplate, from, to) } : s.run
+        run: s.run?.kind === "prompt" ? { ...s.run, promptTemplate: renameIn(s.run.promptTemplate, from, to) } : s.run,
+        ...s.editDocument ? { editDocument: renameRef(s.editDocument, from, to) } : {}
       };
-    if (s.kind === "review" && s.instructions)
-      return { ...s, instructions: renameIn(s.instructions, from, to) };
-    return s;
+    const assignee = s.assignee && renamePerson(s.assignee, from, to);
+    const base = assignee ? { ...s, assignee } : s;
+    if (base.kind === "review" && base.instructions)
+      return { ...base, instructions: renameIn(base.instructions, from, to) };
+    if (base.kind === "action") {
+      const a = base.action;
+      if (a.type === "sign")
+        return {
+          ...base,
+          action: {
+            ...a,
+            document: a.document && renameRef(a.document, from, to),
+            signers: a.signers.map((x) => renamePerson(x, from, to)),
+            ...a.message ? { message: renameIn(a.message, from, to) } : {}
+          }
+        };
+      if (a.type === "email_draft")
+        return {
+          ...base,
+          action: {
+            ...a,
+            to: a.to.map((t) => renameIn(t, from, to)),
+            ...a.cc ? { cc: a.cc.map((t) => renameIn(t, from, to)) } : {},
+            subject: renameIn(a.subject, from, to),
+            body: renameIn(a.body, from, to),
+            ...a.attachments ? { attachments: a.attachments.map((r) => renameRef(r, from, to)) } : {}
+          }
+        };
+      if (Array.isArray(a.documents))
+        return {
+          ...base,
+          action: { ...a, documents: a.documents.map((r) => renameRef(r, from, to)) }
+        };
+    }
+    return base;
   });
   return withSteps(workflow, steps);
 }
+var renamePerson = (a, from, to) => "input" in a && a.input === from ? { input: to } : a;
+var renameRef = (r, from, to) => "input" in r && r.input === from ? { input: to } : r;
 function moveStep(workflow, id, position) {
   return withSteps(workflow, workflowSteps(workflow), { ...layoutOf(workflow), [id]: position });
 }
@@ -381,6 +445,537 @@ function SignalsEditor({
     /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: l.hint })
   ] });
 }
+var DEFAULT_STEP_ACTION_SETTINGS_LABELS = {
+  assignee: "Who does it",
+  assigneeAnyone: "Anyone on the run",
+  assigneeRunOwner: "Whoever started the run",
+  assigneeProjectLead: "The project lead",
+  assigneeFields: "The person in a form field",
+  assigneePeople: "A person",
+  remind: "Remind after (hours)",
+  remindHint: "Left that long, they are told again. Blank: no reminder.",
+  documentFrom: "Document",
+  docFromStep: "From an earlier step",
+  docFromInput: "From a run input",
+  docById: "A document",
+  chooseStep: "Choose a step\u2026",
+  chooseInput: "Choose an input\u2026",
+  chooseDocument: "Choose a document\u2026",
+  documentIdPlaceholder: "doc-\u2026",
+  signDocument: "Document to sign",
+  signers: "Signers",
+  addSigner: "Add signer",
+  signMessage: "Message to the signers",
+  expires: "Expires after (hours)",
+  expiresHint: "Unsigned by then, the step fails. Blank: no expiry.",
+  emailTo: "To",
+  emailCc: "Cc",
+  emailSubject: "Subject",
+  emailBody: "Body",
+  addressesHint: "Separate several with commas.",
+  variablesHint: "Variables (click to add to the body):",
+  attachments: "Attachments",
+  addAttachment: "Add attachment",
+  reviewColumns: "Columns (labels)",
+  noTags: "No labels in this workspace yet.",
+  reviewDocuments: "Documents",
+  reviewRunUploads: "The files this run collects",
+  reviewWorkspace: "Every document in the workspace",
+  reviewChosen: "Chosen documents",
+  addDocument: "Add document",
+  overwrite: "Fill cells that already have a value again",
+  editDocument: "Edit this document (tracked changes)",
+  editDocumentHint: "The instruction says what to change; the edited copy is saved as its next version, to accept or reject change by change."
+};
+var selectClass2 = "flex h-9 w-full rounded-md border border-input bg-background px-2 text-sm shadow-sm focus-visible:outline-none focus-visible:ring-1 focus-visible:ring-ring";
+function Field({
+  label,
+  children,
+  hint
+}) {
+  return /* @__PURE__ */ jsxs("div", { className: "space-y-1.5", children: [
+    /* @__PURE__ */ jsx(Label, { className: "text-xs", children: label }),
+    children,
+    hint && /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: hint })
+  ] });
+}
+var assigneeValue = (a) => !a ? "" : "user" in a ? `user:${a.user}` : "input" in a ? `input:${a.input}` : "runOwner" in a ? "owner" : "lead";
+var parseAssignee = (v) => v === "owner" ? { runOwner: true } : v === "lead" ? { projectLead: true } : v.startsWith("user:") ? { user: v.slice(5) } : v.startsWith("input:") ? { input: v.slice(6) } : void 0;
+function AssigneeSelect({
+  value,
+  onChange,
+  ctx,
+  l,
+  anyone,
+  testId,
+  ariaLabel
+}) {
+  const personFields = ctx.fields.filter((f) => f.type === "person");
+  const v = assigneeValue(value);
+  const known = !v.startsWith("user:") || ctx.people.some((p) => `user:${p.value}` === v) ? null : v.slice(5);
+  return /* @__PURE__ */ jsxs(
+    "select",
+    {
+      className: selectClass2,
+      value: v,
+      "aria-label": ariaLabel,
+      onChange: (e) => onChange(parseAssignee(e.target.value)),
+      "data-testid": testId,
+      children: [
+        anyone !== void 0 ? /* @__PURE__ */ jsx("option", { value: "", children: anyone }) : /* @__PURE__ */ jsx("option", { value: "", disabled: true }),
+        /* @__PURE__ */ jsx("option", { value: "owner", children: l.assigneeRunOwner }),
+        /* @__PURE__ */ jsx("option", { value: "lead", children: l.assigneeProjectLead }),
+        personFields.length > 0 && /* @__PURE__ */ jsx("optgroup", { label: l.assigneeFields, children: personFields.map((f) => /* @__PURE__ */ jsx("option", { value: `input:${f.name}`, children: f.label }, f.name)) }),
+        (ctx.people.length > 0 || known) && /* @__PURE__ */ jsxs("optgroup", { label: l.assigneePeople, children: [
+          ctx.people.map((p) => /* @__PURE__ */ jsx("option", { value: `user:${p.value}`, children: p.label }, p.value)),
+          known && /* @__PURE__ */ jsx("option", { value: v, children: known })
+        ] })
+      ]
+    }
+  );
+}
+function AssigneeSettings({
+  step,
+  patch,
+  ctx,
+  l,
+  testId
+}) {
+  return /* @__PURE__ */ jsxs(Fragment, { children: [
+    /* @__PURE__ */ jsx(Field, { label: l.assignee, children: /* @__PURE__ */ jsx(
+      AssigneeSelect,
+      {
+        value: step.assignee,
+        onChange: (assignee) => patch({ assignee }),
+        ctx,
+        l,
+        anyone: l.assigneeAnyone,
+        testId: `${testId}-assignee`
+      }
+    ) }),
+    /* @__PURE__ */ jsx(Field, { label: l.remind, hint: l.remindHint, children: /* @__PURE__ */ jsx(
+      Input,
+      {
+        type: "number",
+        min: 1,
+        value: step.remindAfterHours ?? "",
+        onChange: (e) => {
+          const n = Number(e.target.value);
+          patch({ remindAfterHours: e.target.value && n > 0 ? n : void 0 });
+        },
+        className: "w-28",
+        "data-testid": `${testId}-remind`
+      }
+    ) })
+  ] });
+}
+var modeOf = (ref) => !ref ? "step" : "step" in ref ? "step" : "input" in ref ? "input" : "id";
+function DocumentRefPicker({
+  value,
+  onChange,
+  ctx,
+  l,
+  testId
+}) {
+  const mode = modeOf(value);
+  const inputs = ctx.fields.filter((f) => f.type === "documents" || f.type === "text");
+  return /* @__PURE__ */ jsxs("div", { className: "space-y-1.5", "data-testid": testId, children: [
+    /* @__PURE__ */ jsxs(
+      "select",
+      {
+        className: cn(selectClass2, "h-8"),
+        value: mode,
+        "aria-label": l.documentFrom,
+        onChange: (e) => {
+          const m = e.target.value;
+          onChange(m === "step" ? { step: "" } : m === "input" ? { input: "" } : { id: "" });
+        },
+        "data-testid": `${testId}-mode`,
+        children: [
+          /* @__PURE__ */ jsx("option", { value: "step", children: l.docFromStep }),
+          /* @__PURE__ */ jsx("option", { value: "input", children: l.docFromInput }),
+          /* @__PURE__ */ jsx("option", { value: "id", children: l.docById })
+        ]
+      }
+    ),
+    mode === "step" && /* @__PURE__ */ jsxs(
+      "select",
+      {
+        className: cn(selectClass2, "h-8"),
+        value: value && "step" in value ? value.step : "",
+        onChange: (e) => onChange({ step: e.target.value }),
+        "data-testid": `${testId}-step`,
+        children: [
+          /* @__PURE__ */ jsx("option", { value: "", disabled: true, children: l.chooseStep }),
+          ctx.earlier.map((s) => /* @__PURE__ */ jsx("option", { value: s.id, children: s.title }, s.id))
+        ]
+      }
+    ),
+    mode === "input" && /* @__PURE__ */ jsxs(
+      "select",
+      {
+        className: cn(selectClass2, "h-8"),
+        value: value && "input" in value ? value.input : "",
+        onChange: (e) => onChange({ input: e.target.value }),
+        "data-testid": `${testId}-input`,
+        children: [
+          /* @__PURE__ */ jsx("option", { value: "", disabled: true, children: l.chooseInput }),
+          inputs.map((f) => /* @__PURE__ */ jsx("option", { value: f.name, children: f.label }, f.name))
+        ]
+      }
+    ),
+    mode === "id" && (ctx.documents.length ? /* @__PURE__ */ jsxs(
+      "select",
+      {
+        className: cn(selectClass2, "h-8"),
+        value: value && "id" in value ? value.id : "",
+        onChange: (e) => onChange({ id: e.target.value }),
+        "data-testid": `${testId}-id`,
+        children: [
+          /* @__PURE__ */ jsx("option", { value: "", disabled: true, children: l.chooseDocument }),
+          ctx.documents.map((d) => /* @__PURE__ */ jsx("option", { value: d.value, children: d.label }, d.value))
+        ]
+      }
+    ) : /* @__PURE__ */ jsx(
+      Input,
+      {
+        className: "h-8",
+        value: value && "id" in value ? value.id : "",
+        placeholder: l.documentIdPlaceholder,
+        onChange: (e) => onChange({ id: e.target.value }),
+        "data-testid": `${testId}-id`
+      }
+    ))
+  ] });
+}
+function DocumentRefList({
+  value,
+  onChange,
+  ctx,
+  l,
+  addLabel,
+  testId
+}) {
+  return /* @__PURE__ */ jsxs("div", { className: "space-y-2", children: [
+    value.map((ref, i) => /* @__PURE__ */ jsxs("div", { className: "flex items-start gap-1.5", children: [
+      /* @__PURE__ */ jsx("div", { className: "min-w-0 flex-1", children: /* @__PURE__ */ jsx(
+        DocumentRefPicker,
+        {
+          value: ref,
+          onChange: (next) => onChange(value.map((r, j) => j === i ? next : r)),
+          ctx,
+          l,
+          testId: `${testId}-${i}`
+        }
+      ) }),
+      /* @__PURE__ */ jsx(
+        "button",
+        {
+          type: "button",
+          className: "mt-2",
+          "aria-label": `${addLabel} \xD7`,
+          onClick: () => onChange(value.filter((_, j) => j !== i)),
+          children: /* @__PURE__ */ jsx(X, { className: "size-4 text-muted-foreground" })
+        }
+      )
+    ] }, i)),
+    /* @__PURE__ */ jsxs(
+      Button,
+      {
+        variant: "outline",
+        size: "sm",
+        onClick: () => onChange([...value, { step: "" }]),
+        "data-testid": `${testId}-add`,
+        children: [
+          /* @__PURE__ */ jsx(Plus, { className: "size-4" }),
+          " ",
+          addLabel
+        ]
+      }
+    )
+  ] });
+}
+var optionalNumber = (raw) => {
+  const n = Number(raw);
+  return raw && n > 0 ? n : void 0;
+};
+function SignSettings({
+  action,
+  onChange,
+  ctx,
+  l,
+  testId
+}) {
+  const set = (p) => onChange({ ...action, ...p });
+  return /* @__PURE__ */ jsxs(Fragment, { children: [
+    /* @__PURE__ */ jsx(Field, { label: l.signDocument, children: /* @__PURE__ */ jsx(
+      DocumentRefPicker,
+      {
+        value: action.document,
+        onChange: (document) => set({ document }),
+        ctx,
+        l,
+        testId: `${testId}-document`
+      }
+    ) }),
+    /* @__PURE__ */ jsx(Field, { label: l.signers, children: /* @__PURE__ */ jsxs("div", { className: "space-y-1.5", "data-testid": `${testId}-signers`, children: [
+      action.signers.map((s, i) => /* @__PURE__ */ jsxs("div", { className: "flex items-center gap-1.5", children: [
+        /* @__PURE__ */ jsx(
+          AssigneeSelect,
+          {
+            value: s,
+            onChange: (next) => set({
+              signers: next ? action.signers.map((x, j) => j === i ? next : x) : action.signers.filter((_, j) => j !== i)
+            }),
+            ctx,
+            l,
+            ariaLabel: `${l.signers} ${i + 1}`,
+            testId: `${testId}-signer-${i}`
+          }
+        ),
+        /* @__PURE__ */ jsx(
+          "button",
+          {
+            type: "button",
+            "aria-label": `${l.signers} ${i + 1} \xD7`,
+            onClick: () => set({ signers: action.signers.filter((_, j) => j !== i) }),
+            children: /* @__PURE__ */ jsx(X, { className: "size-4 text-muted-foreground" })
+          }
+        )
+      ] }, i)),
+      /* @__PURE__ */ jsxs(
+        Button,
+        {
+          variant: "outline",
+          size: "sm",
+          onClick: () => set({ signers: [...action.signers, { runOwner: true }] }),
+          "data-testid": `${testId}-add-signer`,
+          children: [
+            /* @__PURE__ */ jsx(Plus, { className: "size-4" }),
+            " ",
+            l.addSigner
+          ]
+        }
+      )
+    ] }) }),
+    /* @__PURE__ */ jsx(Field, { label: l.signMessage, children: /* @__PURE__ */ jsx(
+      Textarea,
+      {
+        rows: 3,
+        value: action.message ?? "",
+        onChange: (e) => set({ message: e.target.value || void 0 }),
+        "data-testid": `${testId}-message`
+      }
+    ) }),
+    /* @__PURE__ */ jsx(Field, { label: l.expires, hint: l.expiresHint, children: /* @__PURE__ */ jsx(
+      Input,
+      {
+        type: "number",
+        min: 1,
+        className: "w-28",
+        value: action.expiresAfterHours ?? "",
+        onChange: (e) => set({ expiresAfterHours: optionalNumber(e.target.value) }),
+        "data-testid": `${testId}-expires`
+      }
+    ) })
+  ] });
+}
+var addresses = (list) => (list ?? []).join(", ");
+var parseAddresses = (text) => text.split(",").map((t) => t.trim());
+function templateVariables(ctx) {
+  return [
+    ...ctx.fields.map((f) => `{{${f.name}}}`),
+    ...ctx.earlier.filter((s) => s.kind === "agent").map((s) => `{{${s.id}.answer}}`)
+  ];
+}
+function EmailDraftSettings({
+  action,
+  onChange,
+  ctx,
+  l,
+  testId
+}) {
+  const set = (p) => onChange({ ...action, ...p });
+  const vars = templateVariables(ctx);
+  return /* @__PURE__ */ jsxs(Fragment, { children: [
+    /* @__PURE__ */ jsx(Field, { label: l.emailTo, hint: l.addressesHint, children: /* @__PURE__ */ jsx(
+      Input,
+      {
+        value: addresses(action.to),
+        onChange: (e) => set({ to: parseAddresses(e.target.value) }),
+        placeholder: "{{client_email}}",
+        "data-testid": `${testId}-to`
+      }
+    ) }),
+    /* @__PURE__ */ jsx(Field, { label: l.emailCc, children: /* @__PURE__ */ jsx(
+      Input,
+      {
+        value: addresses(action.cc),
+        onChange: (e) => {
+          const cc = parseAddresses(e.target.value);
+          set({ cc: cc.some(Boolean) ? cc : void 0 });
+        },
+        "data-testid": `${testId}-cc`
+      }
+    ) }),
+    /* @__PURE__ */ jsx(Field, { label: l.emailSubject, children: /* @__PURE__ */ jsx(
+      Input,
+      {
+        value: action.subject,
+        onChange: (e) => set({ subject: e.target.value }),
+        "data-testid": `${testId}-subject`
+      }
+    ) }),
+    /* @__PURE__ */ jsx(Field, { label: l.emailBody, children: /* @__PURE__ */ jsx(
+      Textarea,
+      {
+        rows: 6,
+        value: action.body,
+        onChange: (e) => set({ body: e.target.value }),
+        "data-testid": `${testId}-body`
+      }
+    ) }),
+    vars.length > 0 && /* @__PURE__ */ jsxs("div", { className: "flex flex-wrap gap-1", "data-testid": `${testId}-variables`, children: [
+      /* @__PURE__ */ jsx("span", { className: "w-full text-xs text-muted-foreground", children: l.variablesHint }),
+      vars.map((v) => /* @__PURE__ */ jsx(
+        "button",
+        {
+          type: "button",
+          onClick: () => set({ body: `${action.body}${action.body ? " " : ""}${v}` }),
+          className: "rounded bg-muted px-1.5 py-0.5 font-mono text-[11px] text-muted-foreground hover:text-foreground",
+          title: v,
+          children: v
+        },
+        v
+      ))
+    ] }),
+    /* @__PURE__ */ jsx(Field, { label: l.attachments, children: /* @__PURE__ */ jsx(
+      DocumentRefList,
+      {
+        value: action.attachments ?? [],
+        onChange: (attachments) => set({ attachments: attachments.length ? attachments : void 0 }),
+        ctx,
+        l,
+        addLabel: l.addAttachment,
+        testId: `${testId}-attachments`
+      }
+    ) })
+  ] });
+}
+function TabularReviewSettings({
+  action,
+  onChange,
+  ctx,
+  l,
+  testId
+}) {
+  const set = (p) => onChange({ ...action, ...p });
+  const mode = Array.isArray(action.documents) ? "chosen" : action.documents;
+  const toggle = (id) => set({
+    tags: action.tags.includes(id) ? action.tags.filter((t) => t !== id) : [...action.tags, id]
+  });
+  return /* @__PURE__ */ jsxs(Fragment, { children: [
+    /* @__PURE__ */ jsx(Field, { label: l.reviewColumns, children: /* @__PURE__ */ jsxs(
+      "div",
+      {
+        className: "max-h-44 space-y-1 overflow-y-auto rounded-md border border-border p-2",
+        "data-testid": `${testId}-tags`,
+        children: [
+          ctx.tags.length === 0 && /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: l.noTags }),
+          ctx.tags.map((t) => /* @__PURE__ */ jsxs("label", { className: "flex items-center gap-2 text-sm", children: [
+            /* @__PURE__ */ jsx(
+              Checkbox,
+              {
+                checked: action.tags.includes(t.value),
+                onCheckedChange: () => toggle(t.value),
+                "data-testid": `${testId}-tag-${t.value}`
+              }
+            ),
+            /* @__PURE__ */ jsx("span", { className: "truncate", children: t.label })
+          ] }, t.value)),
+          action.tags.filter((t) => !ctx.tags.some((x) => x.value === t)).map((t) => /* @__PURE__ */ jsxs("label", { className: "flex items-center gap-2 text-sm text-muted-foreground", children: [
+            /* @__PURE__ */ jsx(Checkbox, { checked: true, onCheckedChange: () => toggle(t) }),
+            /* @__PURE__ */ jsx("code", { className: "text-xs", children: t })
+          ] }, t))
+        ]
+      }
+    ) }),
+    /* @__PURE__ */ jsx(Field, { label: l.reviewDocuments, children: /* @__PURE__ */ jsxs(
+      "select",
+      {
+        className: selectClass2,
+        value: mode,
+        onChange: (e) => {
+          const v = e.target.value;
+          set({ documents: v === "chosen" ? [] : v });
+        },
+        "data-testid": `${testId}-documents`,
+        children: [
+          /* @__PURE__ */ jsx("option", { value: "run_uploads", children: l.reviewRunUploads }),
+          /* @__PURE__ */ jsx("option", { value: "workspace", children: l.reviewWorkspace }),
+          /* @__PURE__ */ jsx("option", { value: "chosen", children: l.reviewChosen })
+        ]
+      }
+    ) }),
+    Array.isArray(action.documents) && /* @__PURE__ */ jsx(
+      DocumentRefList,
+      {
+        value: action.documents,
+        onChange: (documents) => set({ documents }),
+        ctx,
+        l,
+        addLabel: l.addDocument,
+        testId: `${testId}-chosen`
+      }
+    ),
+    /* @__PURE__ */ jsxs("label", { className: "flex items-center gap-2 text-sm", children: [
+      /* @__PURE__ */ jsx(
+        Checkbox,
+        {
+          checked: !!action.overwrite,
+          onCheckedChange: (v) => set({ overwrite: v === true || void 0 })
+        }
+      ),
+      l.overwrite
+    ] })
+  ] });
+}
+function EditDocumentSettings({
+  step,
+  patch,
+  ctx,
+  l,
+  testId
+}) {
+  const on = !!step.editDocument;
+  return /* @__PURE__ */ jsxs("div", { className: "space-y-2", children: [
+    /* @__PURE__ */ jsxs("label", { className: "flex items-start gap-2 text-sm", children: [
+      /* @__PURE__ */ jsx(
+        Checkbox,
+        {
+          checked: on,
+          disabled: !!step.branch,
+          onCheckedChange: (v) => patch({ editDocument: v === true ? { step: "" } : void 0 }),
+          "data-testid": `${testId}-edit-document`
+        }
+      ),
+      /* @__PURE__ */ jsxs("span", { children: [
+        l.editDocument,
+        /* @__PURE__ */ jsx("span", { className: "block text-xs text-muted-foreground", children: l.editDocumentHint })
+      ] })
+    ] }),
+    on && /* @__PURE__ */ jsx(
+      DocumentRefPicker,
+      {
+        value: step.editDocument,
+        onChange: (editDocument) => patch({ editDocument }),
+        ctx,
+        l,
+        testId: `${testId}-edit-document-ref`
+      }
+    )
+  ] });
+}
 var DEFAULT_STEP_INSPECTOR_LABELS = {
   workflow: "Workflow",
   title: "Title",
@@ -410,17 +1005,19 @@ var DEFAULT_STEP_INSPECTOR_LABELS = {
   hint: "Hint",
   instructions: "What to review",
   instructionsHint: "Shown on the review task; {{field}} and {{steps.<id>}} work here.",
+  reviewFields: "Filled in when approving",
+  reviewFieldsHint: "E.g. a rating or a comment; the answers join the run\u2019s inputs.",
   deleteStep: "Delete step"
 };
-var FIELD_TYPES = ["text", "longtext", "boolean", "number", "date", "scope"];
-var selectClass2 = "flex h-9 w-full rounded-md border border-input bg-background px-2 text-sm shadow-sm focus-visible:outline-none focus-visible:ring-1 focus-visible:ring-ring";
-function Field({ label, children, hint }) {
-  return /* @__PURE__ */ jsxs("div", { className: "space-y-1.5", children: [
-    /* @__PURE__ */ jsx(Label, { className: "text-xs", children: label }),
-    children,
-    hint && /* @__PURE__ */ jsx("p", { className: "text-xs text-muted-foreground", children: hint })
-  ] });
-}
+var FIELD_TYPES = [
+  "text",
+  "longtext",
+  "boolean",
+  "number",
+  "date",
+  "scope",
+  "person"
+];
 var fieldName = (label, taken) => {
   const base = label.toLowerCase().replace(/[^a-z0-9]+/g, "_").replace(/^_+|_+$/g, "").replace(/^[^a-z]+/, "") || "field";
   let name = base;
@@ -434,7 +1031,11 @@ function StepInspector({
   onChange,
   onSelect,
   workflows = [],
+  people = [],
+  tags = [],
+  documents = [],
   labels,
+  actionLabels,
   signalLabels,
   testId = "step-inspector"
 }) {
@@ -464,6 +1065,30 @@ function StepInspector({
     ] });
   }
   const patch = (p) => onChange(updateStep(workflow, step.id, p));
+  const al = { ...DEFAULT_STEP_ACTION_SETTINGS_LABELS, ...actionLabels };
+  const ctx = {
+    earlier: upstreamSteps(steps, step.id),
+    fields: runFields(workflow),
+    people,
+    tags,
+    documents
+  };
+  const takenBy = (id) => steps.flatMap(
+    (s) => s.id === id ? [] : s.kind === "form" ? s.inputs : s.kind === "review" ? s.fields ?? [] : []
+  ).map((f) => f.name);
+  const setFields = (before, after, key) => {
+    const renamed = before.flatMap(
+      (f, i) => after[i] && after[i].name !== f.name ? [[f.name, after[i].name]] : []
+    );
+    onChange(
+      renamed.reduce(
+        (w, [from, to]) => renameInput(w, from, to),
+        updateStep(workflow, step.id, {
+          [key]: key === "fields" && !after.length ? void 0 : after
+        })
+      )
+    );
+  };
   const deps = stepDependencies(steps, step.id);
   const titleOf = (id) => steps.find((s) => s.id === id)?.title ?? id;
   return /* @__PURE__ */ jsxs("div", { className: "space-y-4", "data-testid": `${testId}-step`, children: [
@@ -513,18 +1138,8 @@ function StepInspector({
       FormFields,
       {
         fields: step.inputs,
-        taken: steps.flatMap((s) => s.kind === "form" && s.id !== step.id ? s.inputs : []).map((f) => f.name),
-        onChange: (inputs) => {
-          const renamed = step.inputs.flatMap(
-            (f, i) => inputs[i] && inputs[i].name !== f.name ? [[f.name, inputs[i].name]] : []
-          );
-          onChange(
-            renamed.reduce(
-              (w, [from, to]) => renameInput(w, from, to),
-              updateStep(workflow, step.id, { inputs })
-            )
-          );
-        },
+        taken: takenBy(step.id),
+        onChange: (inputs) => setFields(step.inputs, inputs, "inputs"),
         l,
         testId
       }
@@ -548,26 +1163,73 @@ function StepInspector({
         }
       ) })
     ] }),
-    step.kind === "agent" && /* @__PURE__ */ jsx(
-      AgentSettings,
+    step.kind === "agent" && /* @__PURE__ */ jsxs(Fragment, { children: [
+      /* @__PURE__ */ jsx(
+        AgentSettings,
+        {
+          step,
+          workflows,
+          fields: runFields(workflow).map((f) => f.name),
+          patch,
+          l,
+          testId
+        }
+      ),
+      /* @__PURE__ */ jsx(EditDocumentSettings, { step, patch, ctx, l: al, testId })
+    ] }),
+    step.kind === "review" && /* @__PURE__ */ jsxs(Fragment, { children: [
+      /* @__PURE__ */ jsx(Field, { label: l.instructions, hint: l.instructionsHint, children: /* @__PURE__ */ jsx(
+        Textarea,
+        {
+          value: step.instructions ?? "",
+          rows: 4,
+          onChange: (e) => patch({ instructions: e.target.value || void 0 }),
+          "data-testid": `${testId}-instructions`
+        }
+      ) }),
+      /* @__PURE__ */ jsx(
+        FormFields,
+        {
+          fields: step.fields ?? [],
+          taken: takenBy(step.id),
+          onChange: (fields) => setFields(step.fields ?? [], fields, "fields"),
+          l: { ...l, fields: l.reviewFields },
+          hint: l.reviewFieldsHint,
+          testId: `${testId}-review`
+        }
+      )
+    ] }),
+    step.kind === "action" && step.action.type === "sign" && /* @__PURE__ */ jsx(
+      SignSettings,
       {
-        step,
-        workflows,
-        fields: runFields(workflow).map((f) => f.name),
-        patch,
-        l,
-        testId
+        action: step.action,
+        onChange: (action) => patch({ action }),
+        ctx,
+        l: al,
+        testId: `${testId}-sign`
       }
     ),
-    step.kind === "review" && /* @__PURE__ */ jsx(Field, { label: l.instructions, hint: l.instructionsHint, children: /* @__PURE__ */ jsx(
-      Textarea,
+    step.kind === "action" && step.action.type === "email_draft" && /* @__PURE__ */ jsx(
+      EmailDraftSettings,
       {
-        value: step.instructions ?? "",
-        rows: 4,
-        onChange: (e) => patch({ instructions: e.target.value || void 0 }),
-        "data-testid": `${testId}-instructions`
+        action: step.action,
+        onChange: (action) => patch({ action }),
+        ctx,
+        l: al,
+        testId: `${testId}-email`
       }
-    ) }),
+    ),
+    step.kind === "action" && step.action.type === "tabular_review" && /* @__PURE__ */ jsx(
+      TabularReviewSettings,
+      {
+        action: step.action,
+        onChange: (action) => patch({ action }),
+        ctx,
+        l: al,
+        testId: `${testId}-review-grid`
+      }
+    ),
+    canAssign(step) && /* @__PURE__ */ jsx(AssigneeSettings, { step, patch, ctx, l: al, testId }),
     /* @__PURE__ */ jsx(
       SignalsEditor,
       {
@@ -602,10 +1264,11 @@ function FormFields({
   taken,
   onChange,
   l,
+  hint,
   testId
 }) {
   const set = (i, p) => onChange(fields.map((f, j) => j === i ? { ...f, ...p } : f));
-  return /* @__PURE__ */ jsx(Field, { label: l.fields, children: /* @__PURE__ */ jsxs("div", { className: "space-y-2", "data-testid": `${testId}-fields`, children: [
+  return /* @__PURE__ */ jsx(Field, { label: l.fields, hint, children: /* @__PURE__ */ jsxs("div", { className: "space-y-2", "data-testid": `${testId}-fields`, children: [
     fields.map((f, i) => /* @__PURE__ */ jsxs("div", { className: "space-y-1.5 rounded-md border border-border p-2", children: [
       /* @__PURE__ */ jsxs("div", { className: "flex items-center gap-2", children: [
         /* @__PURE__ */ jsx(
@@ -641,7 +1304,9 @@ function FormFields({
             className: cn(selectClass2, "h-8"),
             value: f.type,
             onChange: (e) => set(i, { type: e.target.value }),
-            children: FIELD_TYPES.map((t) => /* @__PURE__ */ jsx("option", { value: t, children: t }, t))
+            children: (FIELD_TYPES.includes(f.type) ? FIELD_TYPES : [...FIELD_TYPES, f.type]).map(
+              (t) => /* @__PURE__ */ jsx("option", { value: t, children: t }, t)
+            )
           }
         ),
         /* @__PURE__ */ jsxs("label", { className: "flex shrink-0 items-center gap-1.5 text-xs", children: [
@@ -1132,12 +1797,19 @@ var DEFAULT_WORKFLOW_CANVAS_LABELS = {
   upload: "Upload",
   agent: "AI step",
   review: "Approval",
+  sign: "Sign",
+  email_draft: "Email draft",
+  tabular_review: "Tabular review",
+  redline: "Tracked changes",
   branch: "Branch",
   kindHints: {
     form: "Ask for details",
     upload: "Collect files",
     agent: "Run a skill or an instruction",
-    review: "A person approves"
+    review: "A person approves",
+    sign: "People e-sign a document",
+    email_draft: "Prepare an email to send",
+    tabular_review: "Review documents in a grid"
   },
   addNext: "Add the next step",
   runs: "Runs \u201C{name}\u201D",
@@ -1145,6 +1817,14 @@ var DEFAULT_WORKFLOW_CANVAS_LABELS = {
   fields: "{count} fields",
   noFields: "No fields yet",
   approvedBy: "A person approves",
+  signers: "{count} signers",
+  chooseDocument: "Choose the document to sign",
+  emailTo: "To {to}",
+  noRecipients: "Add a recipient",
+  reviewColumns: "{count} columns",
+  noColumns: "Choose the columns",
+  assignee: "{name}",
+  remind: "Reminds after {hours}h",
   emptyTitle: "Start your workflow",
   emptyHint: "Add the first step: what should happen when it starts?",
   fitView: "Fit view"
@@ -1154,9 +1834,12 @@ var STEP_KIND_ICONS = {
   form: FileInput,
   upload: Upload,
   agent: Bot,
-  review: ClipboardCheck
+  review: ClipboardCheck,
+  sign: PenLine,
+  email_draft: Mail,
+  tabular_review: Table2
 };
-var KINDS = ["form", "upload", "agent", "review"];
+var KINDS = WORKFLOW_STEP_KINDS;
 function AddNext({
   from,
   onAdd,
@@ -1209,20 +1892,23 @@ function AddNext({
 }
 function PhaseMark({ phase }) {
   if (phase === "running") return /* @__PURE__ */ jsx(Loader2, { className: "wf-kind-text size-4 animate-spin" });
+  if (phase === "waiting") return /* @__PURE__ */ jsx(Hourglass, { className: "wf-kind-text size-4" });
   if (phase === "done") return /* @__PURE__ */ jsx(Check, { className: "wf-kind-text size-4" });
   if (phase === "failed") return /* @__PURE__ */ jsx(AlertTriangle, { className: "size-4 text-destructive" });
   if (phase === "skipped") return /* @__PURE__ */ jsx(MinusCircle, { className: "size-4 text-muted-foreground" });
   return null;
 }
 function StepNode({ data }) {
-  const { step, phase, summary, icon, l, onAdd, testId } = data;
-  const KindIcon = STEP_KIND_ICONS[step.kind];
+  const { step, phase, summary, assignee, icon, l, onAdd, testId } = data;
+  const kind = stepKindOf(step);
+  const KindIcon = STEP_KIND_ICONS[kind];
+  const redline = step.kind === "agent" && !!step.editDocument;
   return /* @__PURE__ */ jsxs(
     "div",
     {
       className: cn(
         "wf-node w-64 py-3 pl-4 pr-3 text-left",
-        `wf-kind-${step.kind}`,
+        `wf-kind-${kind}`,
         phase && `wf-phase-${phase}`
       ),
       "data-testid": `${testId}-node-${step.id}`,
@@ -1233,15 +1919,44 @@ function StepNode({ data }) {
           /* @__PURE__ */ jsx("span", { className: "wf-badge flex size-8 shrink-0 items-center justify-center rounded-lg [&_svg]:size-4", children: icon ?? /* @__PURE__ */ jsx(KindIcon, {}) }),
           /* @__PURE__ */ jsxs("div", { className: "min-w-0 flex-1", children: [
             /* @__PURE__ */ jsxs("div", { className: "flex items-center justify-between gap-2", children: [
-              /* @__PURE__ */ jsx("span", { className: "wf-kind-text text-[10px] font-semibold uppercase tracking-wider", children: l[step.kind] }),
+              /* @__PURE__ */ jsx("span", { className: "wf-kind-text text-[10px] font-semibold uppercase tracking-wider", children: l[kind] }),
               /* @__PURE__ */ jsx(PhaseMark, { phase })
             ] }),
             /* @__PURE__ */ jsx("p", { className: "truncate text-sm font-semibold leading-snug", children: step.title }),
             summary && /* @__PURE__ */ jsx("p", { className: "mt-0.5 line-clamp-2 text-xs leading-snug text-muted-foreground", children: summary }),
-            step.kind === "agent" && step.branch && /* @__PURE__ */ jsxs("span", { className: "mt-1.5 inline-flex items-center gap-1 rounded-full bg-muted px-2 py-0.5 text-[10px] font-medium text-muted-foreground", children: [
-              /* @__PURE__ */ jsx(GitBranch, { className: "size-3" }),
-              " ",
-              l.branch
+            (redline || step.kind === "agent" && step.branch || assignee) && /* @__PURE__ */ jsxs("div", { className: "mt-1.5 flex flex-wrap gap-1", children: [
+              step.kind === "agent" && step.branch && /* @__PURE__ */ jsxs("span", { className: "inline-flex items-center gap-1 rounded-full bg-muted px-2 py-0.5 text-[10px] font-medium text-muted-foreground", children: [
+                /* @__PURE__ */ jsx(GitBranch, { className: "size-3" }),
+                " ",
+                l.branch
+              ] }),
+              redline && /* @__PURE__ */ jsxs(
+                "span",
+                {
+                  className: "inline-flex items-center gap-1 rounded-full bg-muted px-2 py-0.5 text-[10px] font-medium text-muted-foreground",
+                  "data-testid": `${testId}-node-${step.id}-redline`,
+                  children: [
+                    /* @__PURE__ */ jsx(FilePenLine, { className: "size-3" }),
+                    " ",
+                    l.redline
+                  ]
+                }
+              ),
+              assignee && /* @__PURE__ */ jsxs(
+                "span",
+                {
+                  className: "inline-flex max-w-full items-center gap-1 truncate rounded-full bg-muted px-2 py-0.5 text-[10px] font-medium text-muted-foreground",
+                  title: assignee,
+                  "data-testid": `${testId}-node-${step.id}-assignee`,
+                  children: [
+                    /* @__PURE__ */ jsx(UserRound, { className: "size-3 shrink-0" }),
+                    /* @__PURE__ */ jsx("span", { className: "truncate", children: l.assignee.replace(
+                      "{name}",
+                      assignee.charAt(0).toUpperCase() + assignee.slice(1)
+                    ) })
+                  ]
+                }
+              )
             ] })
           ] })
         ] }),
@@ -1290,6 +2005,16 @@ function summaryOf(step, l, workflowTitle) {
       return step.hint;
     case "review":
       return step.instructions ? firstWords(step.instructions) : l.approvedBy;
+    case "action": {
+      const a = step.action;
+      if (a.type === "sign")
+        return !a.document ? l.chooseDocument : a.signers.length ? l.signers.replace("{count}", String(a.signers.length)) : void 0;
+      if (a.type === "email_draft") {
+        const to = a.to.find((t) => t.trim());
+        return to ? l.emailTo.replace("{to}", to) : l.noRecipients;
+      }
+      return a.tags.length ? l.reviewColumns.replace("{count}", String(a.tags.length)) : l.noColumns;
+    }
   }
 }
 function WorkflowCanvas({
@@ -1301,6 +2026,7 @@ function WorkflowCanvas({
   onAddAfter,
   phases,
   workflowTitle,
+  assigneeOf,
   trigger,
   renderIcon,
   labels,
@@ -1339,6 +2065,7 @@ function WorkflowCanvas({
           step,
           phase: phases?.[step.id],
           summary: summaryOf(step, l, workflowTitle),
+          assignee: assigneeOf?.(step),
           icon: step.icon && renderIcon ? renderIcon(step.icon) : void 0,
           l,
           onAdd: onAddAfter,
@@ -1346,7 +2073,18 @@ function WorkflowCanvas({
         }
       }))
     ];
-  }, [workflow, selected, phases, workflowTitle, trigger, renderIcon, l, onAddAfter, testId]);
+  }, [
+    workflow,
+    selected,
+    phases,
+    workflowTitle,
+    assigneeOf,
+    trigger,
+    renderIcon,
+    l,
+    onAddAfter,
+    testId
+  ]);
   const [nodes, setNodes] = useState(built);
   useEffect(() => setNodes(built), [built]);
   const edges = useMemo(
@@ -1560,10 +2298,17 @@ var DEFAULT_WORKFLOW_BUILDER_LABELS = {
     unknown: "\u201C{step}\u201D waits for a step that does not exist.",
     cycle: "\u201C{step}\u201D waits for itself (a loop).",
     order: "\u201C{step}\u201D is listed before a step it waits for."
+  },
+  missing: {
+    document: "\u201C{step}\u201D needs a document.",
+    signers: "\u201C{step}\u201D needs at least one signer.",
+    recipients: "\u201C{step}\u201D needs a recipient.",
+    tags: "\u201C{step}\u201D needs at least one column.",
+    documents: "\u201C{step}\u201D needs documents to review."
   }
 };
 var PANEL_WIDTH = 344;
-var KINDS2 = ["form", "upload", "agent", "review"];
+var KINDS2 = WORKFLOW_STEP_KINDS;
 function WorkflowBuilder({
   value,
   onChange,
@@ -1572,6 +2317,9 @@ function WorkflowBuilder({
   eventKinds = [],
   tasks = [],
   signals = [],
+  people = [],
+  tags = [],
+  documents = [],
   compose,
   renderIcon,
   actions,
@@ -1594,6 +2342,10 @@ function WorkflowBuilder({
   const { workflow, trigger } = value;
   const steps = workflowSteps(workflow);
   const problems = validateWorkflowGraph(steps);
+  const missing = stepConfigProblems(steps);
+  const nameOf = (id) => people.find((p) => p.value === id)?.label;
+  const fields = runFields(workflow);
+  const assigneeOf = (step) => canAssign(step) ? assigneeName(step.assignee, { nameOf, fields }) : void 0;
   const titles = useMemo(() => new Map(workflows.map((w) => [w.id, w.title])), [workflows]);
   const setWorkflow = (next) => onChange({ ...value, workflow: next });
   const addAfter = (from, kind) => {
@@ -1706,10 +2458,16 @@ function WorkflowBuilder({
       ] }),
       actions && /* @__PURE__ */ jsx("div", { className: "ml-auto flex flex-wrap items-center gap-2", children: actions })
     ] }),
-    problems.length > 0 && /* @__PURE__ */ jsx("ul", { className: "space-y-1 text-sm text-destructive", "data-testid": `${testId}-problems`, children: problems.map((p, i) => /* @__PURE__ */ jsx("li", { children: l.problems[p.code].replace(
-      "{step}",
-      steps.find((s) => s.id === p.step)?.title ?? p.step
-    ) }, i)) }),
+    problems.length + missing.length > 0 && /* @__PURE__ */ jsxs("ul", { className: "space-y-1 text-sm text-destructive", "data-testid": `${testId}-problems`, children: [
+      problems.map((p, i) => /* @__PURE__ */ jsx("li", { children: l.problems[p.code].replace(
+        "{step}",
+        steps.find((s) => s.id === p.step)?.title ?? p.step
+      ) }, i)),
+      missing.map((p, i) => /* @__PURE__ */ jsx("li", { "data-testid": `${testId}-missing-${p.step}-${p.missing}`, children: l.missing[p.missing].replace(
+        "{step}",
+        steps.find((s) => s.id === p.step)?.title ?? p.step
+      ) }, `m${i}`))
+    ] }),
     mode === "design" ? /* @__PURE__ */ jsxs("div", { className: "relative", children: [
       /* @__PURE__ */ jsx(
         WorkflowCanvas,
@@ -1722,6 +2480,7 @@ function WorkflowBuilder({
           onAddAfter: addAfter,
           trigger: { title: l.start, summary },
           workflowTitle: (id) => titles.get(id),
+          assigneeOf,
           renderIcon,
           labels: canvasLabels,
           reserveRight: panelOpen ? PANEL_WIDTH : 0,
@@ -1771,6 +2530,9 @@ function WorkflowBuilder({
                 onChange: setWorkflow,
                 onSelect: setSelected,
                 workflows,
+                people,
+                tags,
+                documents,
                 labels: inspectorLabels,
                 testId: `${testId}-inspector`
               }
@@ -1866,7 +2628,22 @@ function ConnectedWorkflowBuilder({
   ...builderProps
 }) {
   const l = { ...DEFAULT_CONNECTED_WORKFLOW_BUILDER_LABELS, ...connectedLabels };
-  const { workspaceId } = useConnection();
+  const { workspaceId, status, ensureWorkspaceOpen } = useConnection();
+  const members = useWorkspaceUsers(workspaceId ?? void 0, {
+    enabled: status === "authenticated" && !!workspaceId,
+    beforeFetch: ensureWorkspaceOpen
+  });
+  const tagsQuery = useTags(workspaceId ?? void 0);
+  const docsQuery = useDocuments(workspaceId ?? void 0);
+  const people = (members.data ?? []).map((m) => ({
+    value: m.user.external_id,
+    label: [m.user.given_name, m.user.family_name].filter(Boolean).join(" ") || m.user.email
+  }));
+  const tagOptions = (tagsQuery.data ?? []).map((t) => ({ value: t.external_id, label: t.name }));
+  const documentOptions = (docsQuery.data ?? []).map((d) => ({
+    value: d.external_id,
+    label: d.file_name ?? d.external_id
+  }));
   const saved = useWorkflows();
   const skills = useSkills();
   const agentsQuery = useAgents();
@@ -1905,7 +2682,7 @@ function ConnectedWorkflowBuilder({
   }));
   const tasks = (conversations.data ?? []).filter((c) => c.status && c.status !== "done").map((c) => ({ value: c.external_id, label: c.title || c.external_id }));
   const options = { resolveWorkflow: (wid) => workflows.find((w) => w.id === wid), compose };
-  const problem = triggerProblem(value.trigger) ?? (validateWorkflowGraph(workflowSteps(value.workflow)).length ? "graph" : null);
+  const problem = triggerProblem(value.trigger) ?? (validateWorkflowGraph(workflowSteps(value.workflow)).length || stepConfigProblems(workflowSteps(value.workflow)).length ? "graph" : null);
   const busy = create.isPending || update.isPending || run.isPending;
   const save = async () => {
     if (!workspaceId) return void 0;
@@ -1927,6 +2704,9 @@ function ConnectedWorkflowBuilder({
       agents,
       eventKinds,
       tasks,
+      people,
+      tags: tagOptions,
+      documents: documentOptions,
       signals: declaredSignals(saved.data ?? []),
       compose,
       testId,