@nanobpm/nano-workforce 0.194.0 → 0.195.0

package/test/derivation-parity/derivation-parity.test.ts

@@ -128,11 +128,11 @@ test("convergence-loop golden has arbitrary-graph features the structured builde
   // (a) the loop head is a serviceTask that MERGES its back-edges directly — after #786/#789 the
   // round-entry head capture (`capture-head`) is the loop head sitting BEFORE `review-round`, so it
   // is `capture-head` that absorbs the back-edges (f_start, the review loop, the answer resume, the
-  // #786 husk auto-retry, and the #796 bounded auto-ack re-entry f_ackRetry); `review-round` then
-  // takes its single `f_capture` in-edge. A serviceTask merging FIVE back-edges directly is the
-  // arbitrary-graph shape the structured builder cannot emit — the feature this asserts, now on
-  // `capture-head`.
-  assertEquals(between("capture-head", "serviceTask", "incoming"), 5, "capture-head should merge 5 flows on the task itself");
+  // #786 husk auto-retry, the #796 bounded auto-ack re-entry f_ackRetry, and the #826 merge-approval
+  // change request f_revise); `review-round` then takes its single `f_capture` in-edge. A serviceTask
+  // merging SIX back-edges directly is the arbitrary-graph shape the structured builder cannot emit —
+  // the feature this asserts, now on `capture-head`.
+  assertEquals(between("capture-head", "serviceTask", "incoming"), 6, "capture-head should merge 6 flows on the task itself");
   assertEquals(between("review-round", "serviceTask", "incoming"), 1, "review-round now takes the single f_capture in-edge");
   // (b) a single exclusive gateway forks FOUR heterogeneous-condition out-edges.
   assertEquals(between("gw-status", "exclusiveGateway", "outgoing"), 4, "gw-status should be a 4-way exclusive gateway");
@@ -145,8 +145,8 @@ test("convergence-loop golden has arbitrary-graph features the structured builde
 // (a): a `loop()` whose body starts with a task derives an exclusive-gateway
 // loop head that absorbs the back-edge (in>=2), leaving the task itself at
 // in=1. The golden instead merges its back-edges directly into `capture-head`
-// (capture-head in=5: f_start, f_reviewLoop, f_answerLoop, the #786 f_huskRetry, and the #796
-// f_ackRetry re-entry) with no loop-head gateway — a shape the builder cannot express.
+// (capture-head in=6: f_start, f_reviewLoop, f_answerLoop, the #786 f_huskRetry, the #796
+// f_ackRetry re-entry, and the #826 f_revise) with no loop-head gateway — a shape the builder cannot express.
 test("loop() inserts a gateway head, so back-edges cannot merge into a task", () => {
   const probe = defineFlow("loop-head-probe", (w) => {
     w.loop((b) => {

package/test/derivation-parity/flows.ts

@@ -24,7 +24,7 @@
 //         so it clears class (1) — but its topology is NOT expressible with the
 //         structured-only builder (`loop`/`switch`/`branch`), empirically proven
 //         (see `derivation-parity.test.ts`): its loop head `capture-head` is a
-//         serviceTask that MERGES five back-edges directly (in=5), whereas
+//         serviceTask that MERGES six back-edges directly (in=6), whereas
 //         `loop()` always inserts an exclusive-gateway loop head (the task stays
 //         in=1); `gw-status` is a single exclusive gateway with FOUR
 //         heterogeneous-condition out-edges (two `=x = "v"`, one complex boolean,
@@ -207,10 +207,11 @@ export const PORTS: readonly PortEntry[] = [
       "blocked (arbitrary control-flow graph): single top-level start/end, but " +
       "its topology is not expressible with the published @nanobpm/workflow's " +
       "structured-only builder (loop/switch/branch). Proven in the test suite: " +
-      "the loop head `capture-head` is a serviceTask that merges 5 back-edges " +
-      "directly (in=5: f_start, f_reviewLoop, f_answerLoop, the #786 husk " +
-      "auto-retry f_huskRetry, and the #796 bounded auto-ack re-entry " +
-      "f_ackRetry), but loop() always inserts an exclusive-gateway head " +
+      "the loop head `capture-head` is a serviceTask that merges 6 back-edges " +
+      "directly (in=6: f_start, f_reviewLoop, f_answerLoop, the #786 husk " +
+      "auto-retry f_huskRetry, the #796 bounded auto-ack re-entry " +
+      "f_ackRetry, and the #826 merge-approval change request f_revise), but " +
+      "loop() always inserts an exclusive-gateway head " +
       "(task stays in=1); `gw-status` is one gateway with 4 heterogeneous-" +
       "condition out-edges (no switch/branch emits that); `gw-escalated` is one " +
       "gateway that is at once a 6-way merge and a 2-way split. Awaits an " +

package/workers/converge-feature/worker.ts

@@ -22,6 +22,7 @@ const handler: AppJobHandler<In, Record<string, never>> = async (job, app) => {
   const featureKey = job.variables.featureKey;
   const prKey = typeof job.variables.prKey === "string" ? job.variables.prKey.trim() : "";
   const autoMerge = job.variables.autoMerge === true;
+  const humanApproval = job.variables.humanApproval === true;
   const parsed = prKey ? parsePr(prKey) : null;
   if (!parsed) {
     // Should not happen — the gateway only routes here when record-feature emitted a parseable
@@ -46,11 +47,12 @@ const handler: AppJobHandler<In, Record<string, never>> = async (job, app) => {
     pr_key: parsed.prKey,
     updated_at: new Date().toISOString(),
   });
-  await submitPr(app.data, app.engine, parsed, [], MAX_ROUNDS, !autoMerge, featureKey);
+  await submitPr(app.data, app.engine, parsed, [], MAX_ROUNDS, !autoMerge, featureKey, humanApproval);
   app.log.info("converge-feature: enrolled PR into convergence loop", {
     featureKey,
     prKey: parsed.prKey,
     convergeOnly: !autoMerge,
+    humanApproval,
   });
   return {};
 };