@nanobpm/nano-workforce 0.167.0 → 0.167.2

package/resources/processes/feature.bpmn

@@ -13,6 +13,9 @@
           <nano:extend name="featureKey" type="string" />
           <nano:extend name="question" type="string" optional="true" />
         </nano:shape>
+        <nano:shape id="RecordFeatureImplementingIn" name="Record feature implementing — input">
+          <nano:extend name="featureKey" type="string" />
+        </nano:shape>
         <nano:shape id="ConvergeFeatureIn" name="Converge feature PR — input">
           <nano:extend name="featureKey" type="string" />
           <nano:extend name="prKey" type="string" optional="true" />
@@ -183,9 +186,19 @@
           <zeebe:output source="=if (is defined(transcriptUrl)) then transcriptUrl else null" target="transcriptUrl" />
         </zeebe:ioMapping>
       </bpmn:extensionElements>
+      <bpmn:incoming>f_implementing</bpmn:incoming>
+      <bpmn:outgoing>w_toGw</bpmn:outgoing>
+    </bpmn:serviceTask>
+    <bpmn:serviceTask id="record-feature-implementing" name="Record implementing">
+      <bpmn:extensionElements>
+        <zeebe:taskDefinition type="pr.record-feature-implementing" />
+        <zeebe:properties>
+          <zeebe:property name="io.nanobpm.dataEnvelope.in" value="RecordFeatureImplementingIn" />
+        </zeebe:properties>
+      </bpmn:extensionElements>
       <bpmn:incoming>f_toImplement</bpmn:incoming>
       <bpmn:incoming>w_answerLoop</bpmn:incoming>
-      <bpmn:outgoing>w_toGw</bpmn:outgoing>
+      <bpmn:outgoing>f_implementing</bpmn:outgoing>
     </bpmn:serviceTask>
     <bpmn:exclusiveGateway id="w_gw" name="escalated?" default="w_done">
       <bpmn:incoming>w_toGw</bpmn:incoming>
@@ -287,7 +300,8 @@
     </bpmn:sequenceFlow>
     <bpmn:sequenceFlow id="f_readiness_skip" name="no gate" sourceRef="gw-readiness" targetRef="ensure-base-branch" />
     <bpmn:sequenceFlow id="f_readiness_done" sourceRef="readiness-preflight" targetRef="ensure-base-branch" />
-    <bpmn:sequenceFlow id="f_toImplement" sourceRef="ensure-base-branch" targetRef="implement-task" />
+    <bpmn:sequenceFlow id="f_toImplement" sourceRef="ensure-base-branch" targetRef="record-feature-implementing" />
+    <bpmn:sequenceFlow id="f_implementing" sourceRef="record-feature-implementing" targetRef="implement-task" />
     <bpmn:sequenceFlow id="w_toGw" sourceRef="implement-task" targetRef="w_gw" />
     <bpmn:sequenceFlow id="w_escalate" name="escalated" sourceRef="w_gw" targetRef="record-feature-escalation">
       <bpmn:conditionExpression xsi:type="bpmn:tFormalExpression">=status = "escalated" and question != null and trim(question) != ""</bpmn:conditionExpression>
@@ -295,7 +309,7 @@
     <bpmn:sequenceFlow id="w_toEscalationTask" sourceRef="record-feature-escalation" targetRef="feature-escalation" />
     <bpmn:sequenceFlow id="w_done" name="done" sourceRef="w_gw" targetRef="record-feature" />
     <bpmn:sequenceFlow id="w_toAnswerGw" sourceRef="feature-escalation" targetRef="w_gw_answer" />
-    <bpmn:sequenceFlow id="w_answerLoop" name="answer" sourceRef="w_gw_answer" targetRef="implement-task">
+    <bpmn:sequenceFlow id="w_answerLoop" name="answer" sourceRef="w_gw_answer" targetRef="record-feature-implementing">
       <bpmn:conditionExpression xsi:type="bpmn:tFormalExpression">=resolution = "answer"</bpmn:conditionExpression>
     </bpmn:sequenceFlow>
     <bpmn:sequenceFlow id="w_abandon" name="abandon" sourceRef="w_gw_answer" targetRef="record-feature" />
@@ -334,60 +348,63 @@
         <dc:Bounds x="566" y="80" width="100" height="80" />
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_implement-task" bpmnElement="implement-task">
+        <dc:Bounds x="966" y="80" width="100" height="80" />
+      </bpmndi:BPMNShape>
+      <bpmndi:BPMNShape id="BPMNShape_record-feature-implementing" bpmnElement="record-feature-implementing">
         <dc:Bounds x="766" y="80" width="100" height="80" />
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_w_gw" bpmnElement="w_gw" isMarkerVisible="true">
-        <dc:Bounds x="966" y="95" width="50" height="50" />
+        <dc:Bounds x="1166" y="95" width="50" height="50" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="954" y="76" width="74" height="14" />
+          <dc:Bounds x="1154" y="76" width="74" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_record-feature-escalation" bpmnElement="record-feature-escalation">
-        <dc:Bounds x="1116" y="240" width="100" height="80" />
+        <dc:Bounds x="1316" y="240" width="100" height="80" />
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_feature-escalation" bpmnElement="feature-escalation">
-        <dc:Bounds x="1316" y="240" width="100" height="80" />
+        <dc:Bounds x="1516" y="240" width="100" height="80" />
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_w_gw_answer" bpmnElement="w_gw_answer" isMarkerVisible="true">
-        <dc:Bounds x="1516" y="255" width="50" height="50" />
+        <dc:Bounds x="1716" y="255" width="50" height="50" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="1513" y="236" width="56" height="14" />
+          <dc:Bounds x="1713" y="236" width="56" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_record-feature" bpmnElement="record-feature">
-        <dc:Bounds x="1666" y="80" width="100" height="80" />
+        <dc:Bounds x="1866" y="80" width="100" height="80" />
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_gw-blocked" bpmnElement="gw-blocked" isMarkerVisible="true">
-        <dc:Bounds x="1866" y="95" width="50" height="50" />
+        <dc:Bounds x="2066" y="95" width="50" height="50" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="1861" y="76" width="60" height="14" />
+          <dc:Bounds x="2061" y="76" width="60" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_feature-blocked" bpmnElement="feature-blocked">
-        <dc:Bounds x="2016" y="400" width="100" height="80" />
+        <dc:Bounds x="2216" y="400" width="100" height="80" />
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_record-blocked-ack" bpmnElement="record-blocked-ack">
-        <dc:Bounds x="2216" y="400" width="100" height="80" />
+        <dc:Bounds x="2416" y="400" width="100" height="80" />
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_gw-converge" bpmnElement="gw-converge" isMarkerVisible="true">
-        <dc:Bounds x="2041" y="95" width="50" height="50" />
+        <dc:Bounds x="2241" y="95" width="50" height="50" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="2033" y="76" width="67" height="14" />
+          <dc:Bounds x="2233" y="76" width="67" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_converge" bpmnElement="converge">
-        <dc:Bounds x="2216" y="240" width="100" height="80" />
+        <dc:Bounds x="2416" y="240" width="100" height="80" />
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_End" bpmnElement="End">
-        <dc:Bounds x="2416" y="102" width="36" height="36" />
+        <dc:Bounds x="2616" y="102" width="36" height="36" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="2417" y="83" width="34" height="14" />
+          <dc:Bounds x="2617" y="83" width="34" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNShape>
       <bpmndi:BPMNShape id="BPMNShape_be_feature_sla" bpmnElement="be_feature_sla">
-        <dc:Bounds x="1348" y="302" width="36" height="36" />
+        <dc:Bounds x="1548" y="302" width="36" height="36" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="1324" y="383" width="84" height="14" />
+          <dc:Bounds x="1524" y="383" width="84" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNShape>
       <bpmndi:BPMNEdge id="BPMNEdge_f_toReadinessGw" bpmnElement="f_toReadinessGw">
@@ -405,41 +422,45 @@
         <di:waypoint x="666" y="120" />
         <di:waypoint x="766" y="120" />
       </bpmndi:BPMNEdge>
-      <bpmndi:BPMNEdge id="BPMNEdge_w_toGw" bpmnElement="w_toGw">
+      <bpmndi:BPMNEdge id="BPMNEdge_f_implementing" bpmnElement="f_implementing">
         <di:waypoint x="866" y="120" />
         <di:waypoint x="966" y="120" />
       </bpmndi:BPMNEdge>
+      <bpmndi:BPMNEdge id="BPMNEdge_w_toGw" bpmnElement="w_toGw">
+        <di:waypoint x="1066" y="120" />
+        <di:waypoint x="1166" y="120" />
+      </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_w_done" bpmnElement="w_done">
-        <di:waypoint x="1016" y="120" />
-        <di:waypoint x="1666" y="120" />
+        <di:waypoint x="1216" y="120" />
+        <di:waypoint x="1866" y="120" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="1325" y="98" width="32" height="14" />
+          <dc:Bounds x="1525" y="98" width="32" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_f_toBlockedGw" bpmnElement="f_toBlockedGw">
-        <di:waypoint x="1766" y="120" />
-        <di:waypoint x="1866" y="120" />
+        <di:waypoint x="1966" y="120" />
+        <di:waypoint x="2066" y="120" />
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_f_notBlocked" bpmnElement="f_notBlocked">
-        <di:waypoint x="1916" y="120" />
-        <di:waypoint x="2041" y="120" />
+        <di:waypoint x="2116" y="120" />
+        <di:waypoint x="2241" y="120" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="1940" y="98" width="78" height="14" />
+          <dc:Bounds x="2140" y="98" width="78" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_f_noConverge" bpmnElement="f_noConverge">
-        <di:waypoint x="2091" y="120" />
-        <di:waypoint x="2416" y="120" />
+        <di:waypoint x="2291" y="120" />
+        <di:waypoint x="2616" y="120" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="2218" y="98" width="71" height="14" />
+          <dc:Bounds x="2418" y="98" width="71" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_w_abandon" bpmnElement="w_abandon">
-        <di:waypoint x="1566" y="280" />
-        <di:waypoint x="1716" y="280" />
-        <di:waypoint x="1716" y="160" />
+        <di:waypoint x="1766" y="280" />
+        <di:waypoint x="1916" y="280" />
+        <di:waypoint x="1916" y="160" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="1615" y="258" width="53" height="14" />
+          <dc:Bounds x="1815" y="258" width="53" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_f_readiness_gate" bpmnElement="f_readiness_gate">
@@ -451,27 +472,27 @@
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_w_escalate" bpmnElement="w_escalate">
-        <di:waypoint x="991" y="145" />
-        <di:waypoint x="991" y="280" />
-        <di:waypoint x="1116" y="280" />
+        <di:waypoint x="1191" y="145" />
+        <di:waypoint x="1191" y="280" />
+        <di:waypoint x="1316" y="280" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="996" y="206" width="67" height="14" />
+          <dc:Bounds x="1196" y="206" width="67" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_f_blocked" bpmnElement="f_blocked">
-        <di:waypoint x="1891" y="145" />
-        <di:waypoint x="1891" y="440" />
-        <di:waypoint x="2016" y="440" />
+        <di:waypoint x="2091" y="145" />
+        <di:waypoint x="2091" y="440" />
+        <di:waypoint x="2216" y="440" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="1896" y="286" width="53" height="14" />
+          <dc:Bounds x="2096" y="286" width="53" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_f_converge" bpmnElement="f_converge">
-        <di:waypoint x="2066" y="145" />
-        <di:waypoint x="2066" y="280" />
-        <di:waypoint x="2216" y="280" />
+        <di:waypoint x="2266" y="145" />
+        <di:waypoint x="2266" y="280" />
+        <di:waypoint x="2416" y="280" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="2071" y="206" width="60" height="14" />
+          <dc:Bounds x="2271" y="206" width="60" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_f_readiness_done" bpmnElement="f_readiness_done">
@@ -480,43 +501,43 @@
         <di:waypoint x="616" y="160" />
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_w_toEscalationTask" bpmnElement="w_toEscalationTask">
-        <di:waypoint x="1216" y="280" />
-        <di:waypoint x="1316" y="280" />
-      </bpmndi:BPMNEdge>
-      <bpmndi:BPMNEdge id="BPMNEdge_w_toAnswerGw" bpmnElement="w_toAnswerGw">
         <di:waypoint x="1416" y="280" />
         <di:waypoint x="1516" y="280" />
       </bpmndi:BPMNEdge>
+      <bpmndi:BPMNEdge id="BPMNEdge_w_toAnswerGw" bpmnElement="w_toAnswerGw">
+        <di:waypoint x="1616" y="280" />
+        <di:waypoint x="1716" y="280" />
+      </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_f_toBlockedAck" bpmnElement="f_toBlockedAck">
-        <di:waypoint x="2116" y="440" />
-        <di:waypoint x="2216" y="440" />
+        <di:waypoint x="2316" y="440" />
+        <di:waypoint x="2416" y="440" />
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_f_blockedEnd" bpmnElement="f_blockedEnd">
-        <di:waypoint x="2316" y="440" />
-        <di:waypoint x="2434" y="440" />
-        <di:waypoint x="2434" y="138" />
+        <di:waypoint x="2516" y="440" />
+        <di:waypoint x="2634" y="440" />
+        <di:waypoint x="2634" y="138" />
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_f_toEndConverged" bpmnElement="f_toEndConverged">
-        <di:waypoint x="2316" y="280" />
-        <di:waypoint x="2434" y="280" />
-        <di:waypoint x="2434" y="138" />
+        <di:waypoint x="2516" y="280" />
+        <di:waypoint x="2634" y="280" />
+        <di:waypoint x="2634" y="138" />
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_w_sla" bpmnElement="w_sla">
-        <di:waypoint x="1366" y="338" />
-        <di:waypoint x="1366" y="358" />
-        <di:waypoint x="1716" y="358" />
-        <di:waypoint x="1716" y="160" />
+        <di:waypoint x="1566" y="338" />
+        <di:waypoint x="1566" y="358" />
+        <di:waypoint x="1916" y="358" />
+        <di:waypoint x="1916" y="160" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="1557" y="325" width="88" height="28" />
+          <dc:Bounds x="1757" y="325" width="88" height="28" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNEdge>
       <bpmndi:BPMNEdge id="BPMNEdge_w_answerLoop" bpmnElement="w_answerLoop">
-        <di:waypoint x="1541" y="305" />
-        <di:waypoint x="1541" y="360" />
+        <di:waypoint x="1741" y="305" />
+        <di:waypoint x="1741" y="360" />
         <di:waypoint x="816" y="360" />
         <di:waypoint x="816" y="160" />
         <bpmndi:BPMNLabel>
-          <dc:Bounds x="1154" y="338" width="49" height="14" />
+          <dc:Bounds x="1254" y="338" width="49" height="14" />
         </bpmndi:BPMNLabel>
       </bpmndi:BPMNEdge>
     </bpmndi:BPMNPlane>

package/test/delivery-graphs-ack-or-timeout.test.ts

@@ -0,0 +1,280 @@
+// Red/green behavioural guard for the delivery-graph Preview & Reuse "honest toast" fix (issue #645).
+//
+// The DEFECT: both cross-frame producers printed an UNCONDITIONAL "✓ …" success toast synchronously
+// right after `postMessage`, even though the message can be dropped by the host relay boundary — so the
+// UI claimed success while nothing happened. The FIX makes the success EARNED, not assumed:
+//   • Library "Reuse" (library.mount.js `doReuse`) posts the compose-fill UP to the host (relayed across
+//     to the compose sibling App-View by nano-ide #518), shows a NEUTRAL in-progress status, and only
+//     renders "✓ Loaded…" when the compose mount acks the fill (matching a correlation token) — a clear
+//     "Couldn't reach the composer" on a short timeout.
+//   • Compose "Preview generated DI" (mount.js `doPreviewDi`) posts `nano-navigate` UP to the console,
+//     shows a NEUTRAL in-progress status, and only renders "✓ Opened…" on a host `nano-navigate-ack` —
+//     "Couldn't reach the console explorer" on a short timeout.
+//
+// These tests drive the REAL mounts over a linkedom DOM inside a fake EMBEDDED App-View window whose
+// `parent.postMessage` is recorded and whose `message` listeners can be fed synthetic acks. They fail
+// against the pre-fix mounts (which render "✓" synchronously with no ack) and pass after.
+import { test } from "node:test";
+import { assert, assertEquals } from "#test-assert";
+import { parseHTML } from "linkedom";
+import { mountDeliveryGraphs, DG_COMPOSE_FILL_ACK_MESSAGE, NANO_NAVIGATE_ACK_MESSAGE } from "../pages/delivery-graphs/mount.js";
+import { mountDeliveryGraphLibrary } from "../pages/delivery-graphs/library.mount.js";
+
+const ORIGIN = "https://app.test";
+const CHECK = "\u2713"; // the "✓" a success toast must EARN, never assume
+
+interface PostedMessage {
+  data: { type?: string; graphJson?: string; token?: string | null; target?: string; params?: unknown };
+  targetOrigin: string;
+}
+
+/** Boot a mount inside a fake EMBEDDED App-View window (parent !== window) with a recording
+ *  parent.postMessage and a capturable `message` listener set, over a linkedom DOM. */
+function harness(
+  mountFn: (host: unknown, config: Record<string, unknown>) => () => void,
+  config: Record<string, unknown>,
+  fetchImpl?: (input: unknown, init?: unknown) => Promise<Response>,
+) {
+  const { window: domWindow, document } = parseHTML("<!doctype html><html><body><div id='host'></div></body></html>");
+  const host = document.getElementById("host");
+  assert(host, "harness host element must exist");
+
+  const posted: PostedMessage[] = [];
+  const messageListeners: Array<(ev: unknown) => void> = [];
+  const parentWindow = {
+    postMessage: (data: PostedMessage["data"], targetOrigin: string) => {
+      posted.push({ data, targetOrigin });
+    },
+  };
+  const fakeWindow = {
+    location: { origin: ORIGIN, href: `${ORIGIN}/delivery-graphs/` },
+    parent: parentWindow, // parent !== fakeWindow ⇒ embedded
+    addEventListener: (type: string, fn: (ev: unknown) => void) => {
+      if (type === "message") messageListeners.push(fn);
+    },
+    removeEventListener: (type: string, fn: (ev: unknown) => void) => {
+      if (type !== "message") return;
+      const i = messageListeners.indexOf(fn);
+      if (i >= 0) messageListeners.splice(i, 1);
+    },
+  };
+
+  const origFetch = globalThis.fetch;
+  if (fetchImpl) globalThis.fetch = fetchImpl as typeof fetch;
+  const origWindow = Reflect.get(globalThis, "window");
+  Reflect.set(globalThis, "window", fakeWindow);
+
+  // Restore the mutated globals if the mount itself throws, so a failing mount can't
+  // leak `fetch`/`window` into later tests and make them fail in confusing ways.
+  let dispose: () => void;
+  try {
+    dispose = mountFn(host, config);
+  } catch (err) {
+    if (fetchImpl) globalThis.fetch = origFetch;
+    Reflect.set(globalThis, "window", origWindow);
+    throw err;
+  }
+
+  const teardown = () => {
+    dispose();
+    if (fetchImpl) globalThis.fetch = origFetch;
+    Reflect.set(globalThis, "window", origWindow);
+  };
+  const flush = async () => {
+    for (let i = 0; i < 4; i++) await new Promise((r) => setTimeout(r, 0));
+  };
+  const click = (el: { dispatchEvent: (ev: unknown) => boolean } | null) => {
+    assert(el, "click target must exist");
+    el!.dispatchEvent(new domWindow.Event("click", { bubbles: true, cancelable: true }));
+  };
+  // Deliver a synthetic same-origin message from the parent (the console relay) to the mount's listeners.
+  const emitFromParent = (data: unknown) => {
+    for (const fn of [...messageListeners]) fn({ origin: ORIGIN, source: parentWindow, data });
+  };
+  const status = () => host!.querySelector(".status") as { textContent: string | null; className: string } | null;
+
+  return { host, posted, teardown, flush, click, emitFromParent, status };
+}
+
+const wait = (ms: number) => new Promise((r) => setTimeout(r, ms));
+
+// ── Library "Reuse" ────────────────────────────────────────────────────────────────────────────────
+const LIBRARY_URL = `${ORIGIN}/app/api/delivery-graph/library`;
+const ENTRY = { id: "lib-1", name: "Onboarding", graph: '{"nodes":[],"edges":[]}' };
+const libraryFetch = async (input: unknown) => {
+  if (String(input).startsWith(LIBRARY_URL)) return new Response(JSON.stringify({ entries: [ENTRY] }), { status: 200 });
+  return new Response(JSON.stringify({}), { status: 404 });
+};
+
+function libraryHarness() {
+  return harness(
+    mountDeliveryGraphLibrary as never,
+    { libraryUrl: LIBRARY_URL, refreshMs: 1_000_000_000, ackTimeoutMs: 40 },
+    libraryFetch,
+  );
+}
+
+test("#645: Reuse shows NO optimistic ✓ toast — a neutral in-progress status until the compose acks", async () => {
+  const h = libraryHarness();
+  try {
+    await h.flush();
+    h.click(h.host!.querySelector("[data-reuse]"));
+    // Synchronously after the post: the fill message went UP to the host, but there is NO success yet.
+    const fill = h.posted.find((p) => p.data.type === "nano-delivery-graph-compose-fill");
+    assert(fill, "Reuse must post the compose-fill message up to the host");
+    assertEquals(fill!.data.graphJson, ENTRY.graph, "the fill carries the saved graph JSON");
+    assert(typeof fill!.data.token === "string" && fill!.data.token, "the fill carries a correlation token for the ack");
+    const s = h.status();
+    assert(s && !s.textContent!.includes(CHECK), "Reuse must NOT render a ✓ success toast synchronously (the #645 defect)");
+    assert(s && !/status-ok/.test(s.className), "the pre-ack Reuse status must not carry the success (ok) tone");
+  } finally {
+    h.teardown();
+  }
+});
+
+test("#645: Reuse renders ✓ Loaded only once the compose acks the matching fill token", async () => {
+  const h = libraryHarness();
+  try {
+    await h.flush();
+    h.click(h.host!.querySelector("[data-reuse]"));
+    const fill = h.posted.find((p) => p.data.type === "nano-delivery-graph-compose-fill");
+    assert(fill, "Reuse must post the compose-fill message");
+    // A stale/mismatched-token ack must NOT complete this Reuse.
+    h.emitFromParent({ type: DG_COMPOSE_FILL_ACK_MESSAGE, token: "some-other-token" });
+    assert(!h.status()!.textContent!.includes(CHECK), "an ack for a different token must not satisfy this Reuse");
+    // The real ack (echoing our token) earns the success toast.
+    h.emitFromParent({ type: DG_COMPOSE_FILL_ACK_MESSAGE, token: fill!.data.token });
+    const s = h.status();
+    assert(s!.textContent!.includes(CHECK) && /Loaded/.test(s!.textContent!), "on ack Reuse renders the ✓ Loaded toast");
+    assert(/status-ok/.test(s!.className), "the acked Reuse status carries the success tone");
+  } finally {
+    h.teardown();
+  }
+});
+
+test("#645: Reuse ignores an ack with a null/absent token — it can't be correlated, so no ✓", async () => {
+  const h = libraryHarness();
+  try {
+    await h.flush();
+    h.click(h.host!.querySelector("[data-reuse]"));
+    const fill = h.posted.find((p) => p.data.type === "nano-delivery-graph-compose-fill");
+    assert(fill, "Reuse must post the compose-fill message");
+    // The compose mount emits `token: null` when a fill arrives without a token; such an ack can't be
+    // correlated to this Reuse and must NOT satisfy it (else the #645 false-positive toast returns).
+    h.emitFromParent({ type: DG_COMPOSE_FILL_ACK_MESSAGE, token: null });
+    assert(!h.status()!.textContent!.includes(CHECK), "a null-token ack must not satisfy this Reuse");
+    h.emitFromParent({ type: DG_COMPOSE_FILL_ACK_MESSAGE });
+    assert(!h.status()!.textContent!.includes(CHECK), "a token-less ack must not satisfy this Reuse");
+    // The real ack (echoing our token) still earns the success toast.
+    h.emitFromParent({ type: DG_COMPOSE_FILL_ACK_MESSAGE, token: fill!.data.token });
+    assert(h.status()!.textContent!.includes(CHECK), "the matching-token ack still resolves the Reuse");
+  } finally {
+    h.teardown();
+  }
+});
+
+test("#645: Reuse surfaces an honest error when the compose never acks (short timeout)", async () => {
+  const h = libraryHarness();
+  try {
+    await h.flush();
+    h.click(h.host!.querySelector("[data-reuse]"));
+    assert(!h.status()!.textContent!.includes(CHECK), "no success before the timeout");
+    await wait(80); // > ackTimeoutMs (40)
+    const s = h.status();
+    assert(s && /Couldn't reach the composer/.test(s.textContent!), "on timeout Reuse surfaces 'Couldn't reach the composer'");
+    assert(!s!.textContent!.includes(CHECK) && /status-err/.test(s!.className), "the timed-out Reuse is an error, never a ✓ success");
+  } finally {
+    h.teardown();
+  }
+});
+
+// ── Compose "Preview generated DI" ───────────────────────────────────────────────────────────────────
+const PREVIEW_URL = `${ORIGIN}/app/api/actions/delivery-graph/preview`;
+const STAGE_URL = `${ORIGIN}/app/api/actions/delivery-graph/stage`;
+const IMPORT_URL = `${ORIGIN}/app/api/actions/delivery-graph/library/import`;
+const composeFetch = async (input: unknown) => {
+  if (String(input) === PREVIEW_URL) {
+    return new Response(
+      JSON.stringify({ ok: true, title: "T", sideEffecting: false, nodeCount: 1, humanNodeCount: 0, sideEffectCount: 0, digest: "d1", sideEffects: [], mermaid: "graph TD", bpmn: "<xml/>" }),
+      { status: 200 },
+    );
+  }
+  return new Response(JSON.stringify({}), { status: 404 });
+};
+
+/** Boot the compose mount and drive Preview so a laid-out BPMN exists, then click "Preview generated DI". */
+async function composePreviewHarness() {
+  const h = harness(
+    mountDeliveryGraphs as never,
+    { previewUrl: PREVIEW_URL, stageUrl: STAGE_URL, importUrl: IMPORT_URL, ackTimeoutMs: 40 },
+    composeFetch,
+  );
+  const jsonEl = h.host!.querySelector("#dg-json") as { value: string } | null;
+  assert(jsonEl, "compose mount must render the #dg-json textarea");
+  jsonEl!.value = '{"nodes":[],"edges":[]}';
+  h.click(h.host!.querySelector("#dg-preview"));
+  await h.flush(); // resolves the preview door → lastBpmn set → renders the "Preview generated DI" button
+  return h;
+}
+
+test("#645: 'Preview generated DI' shows NO optimistic ✓ toast — a neutral status until the host acks", async () => {
+  const h = await composePreviewHarness();
+  try {
+    const before = h.posted.length;
+    h.click(h.host!.querySelector("[data-preview-di]"));
+    const nav = h.posted.slice(before).find((p) => p.data.type === "nano-navigate");
+    assert(nav, "Preview DI must post nano-navigate up to the console");
+    assertEquals(nav!.data.target, "definitionPreview", "the nano-navigate targets the definitionPreview view");
+    const s = h.status();
+    assert(s && !s.textContent!.includes(CHECK), "Preview DI must NOT render a ✓ 'Opening…' toast synchronously (the #645 defect)");
+    assert(s && !/status-ok/.test(s.className), "the pre-ack Preview status must not carry the success (ok) tone");
+  } finally {
+    h.teardown();
+  }
+});
+
+test("#645: 'Preview generated DI' renders ✓ Opened only once the host acks the nano-navigate", async () => {
+  const h = await composePreviewHarness();
+  try {
+    h.click(h.host!.querySelector("[data-preview-di]"));
+    assert(!h.status()!.textContent!.includes(CHECK), "no success before the host ack");
+    h.emitFromParent({ type: NANO_NAVIGATE_ACK_MESSAGE, target: "definitionPreview" });
+    const s = h.status();
+    assert(s!.textContent!.includes(CHECK) && /Opened/.test(s!.textContent!), "on host ack Preview DI renders the ✓ Opened toast");
+    assert(/status-ok/.test(s!.className), "the acked Preview status carries the success tone");
+  } finally {
+    h.teardown();
+  }
+});
+
+test("#645: 'Preview generated DI' ignores a target-less nav ack — no forged ✓ Opened", async () => {
+  const h = await composePreviewHarness();
+  try {
+    h.click(h.host!.querySelector("[data-preview-di]"));
+    // An ack that omits `target` (or names a different one) could be any unrelated same-origin parent
+    // ack — it must NOT resolve the pending Preview (else the #645 false-positive toast returns).
+    h.emitFromParent({ type: NANO_NAVIGATE_ACK_MESSAGE });
+    assert(!h.status()!.textContent!.includes(CHECK), "a target-less nav ack must not satisfy the Preview");
+    h.emitFromParent({ type: NANO_NAVIGATE_ACK_MESSAGE, target: "someOtherView" });
+    assert(!h.status()!.textContent!.includes(CHECK), "an ack for a different target must not satisfy the Preview");
+    // The real ack (matching target) still earns the success toast.
+    h.emitFromParent({ type: NANO_NAVIGATE_ACK_MESSAGE, target: "definitionPreview" });
+    assert(h.status()!.textContent!.includes(CHECK), "the matching-target ack still resolves the Preview");
+  } finally {
+    h.teardown();
+  }
+});
+
+test("#645: 'Preview generated DI' surfaces an honest error when the host never acks (short timeout)", async () => {
+  const h = await composePreviewHarness();
+  try {
+    h.click(h.host!.querySelector("[data-preview-di]"));
+    assert(!h.status()!.textContent!.includes(CHECK), "no success before the timeout");
+    await wait(80); // > ackTimeoutMs (40)
+    const s = h.status();
+    assert(s && /Couldn't reach the console explorer/.test(s.textContent!), "on timeout Preview DI surfaces 'Couldn't reach the console explorer'");
+    assert(!s!.textContent!.includes(CHECK) && /status-err/.test(s!.className), "the timed-out Preview is an error, never a ✓ success");
+  } finally {
+    h.teardown();
+  }
+});

package/test/delivery-graphs-library-embed.test.ts

@@ -71,7 +71,7 @@ test("#523: Reuse drives the compose fill seam over the shared host-bridge messa
   // Reuse loads the saved graph back into the SEPARATE compose App-View, so it posts the ONE shared
   // fill message (its type imported from ./mount.js, never re-declared) UP over the App-View boundary.
   assert(
-    /import \{ DG_COMPOSE_FILL_MESSAGE \} from "\.\/mount\.js"/.test(LIBRARY_JS),
+    /import \{[^}]*\bDG_COMPOSE_FILL_MESSAGE\b[^}]*\} from "\.\/mount\.js"/.test(LIBRARY_JS),
     "library.mount.js must import DG_COMPOSE_FILL_MESSAGE from ./mount.js (the ONE source of truth for the fill message type)",
   );
   assert(/data-reuse=/.test(LIBRARY_JS), "library.mount.js must render a per-row Reuse affordance carrying the entry id");