@stigmer/runner-slim 3.12.6 → 3.12.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@stigmer/runner-slim",
-  "version": "3.12.6",
+  "version": "3.12.7",
   "description": "Self-contained Stigmer runner build for embedding in desktop apps — the bundle-friendly @stigmer/runner",
   "license": "Apache-2.0",
   "engines": {
@@ -16,11 +16,11 @@
     "jq-wasm": "^1.1.0-jq-1.8.1"
   },
   "optionalDependencies": {
-    "@stigmer/runner-slim-darwin-arm64": "3.12.6",
-    "@stigmer/runner-slim-darwin-x64": "3.12.6",
-    "@stigmer/runner-slim-linux-x64": "3.12.6",
-    "@stigmer/runner-slim-linux-arm64": "3.12.6",
-    "@stigmer/runner-slim-win32-x64": "3.12.6"
+    "@stigmer/runner-slim-darwin-arm64": "3.12.7",
+    "@stigmer/runner-slim-darwin-x64": "3.12.7",
+    "@stigmer/runner-slim-linux-x64": "3.12.7",
+    "@stigmer/runner-slim-linux-arm64": "3.12.7",
+    "@stigmer/runner-slim-win32-x64": "3.12.7"
   },
   "keywords": [
     "stigmer",

package/workflow-bundle.js

@@ -27951,6 +27951,17 @@ __webpack_require__.r(__webpack_exports__);
 // ─────────────────────────────────────────────────────────────────────
 // Signal Definitions
 // ─────────────────────────────────────────────────────────────────────
+/**
+ * Identity-only "go look" trigger (DD-012, stigmer-cloud#509): the child's
+ * server signals just the gated child's execution id, and this workflow
+ * derives the gate from the child's persisted record. The payload MUST stay a
+ * bare string: it crosses the polyglot boundary from the Java server, whose
+ * client serializes proto messages as `json/protobuf` — an encoding this
+ * worker's default converter cannot decode, which poisoned the workflow task
+ * in a permanent retry loop (the original cloud#509 failure). The object
+ * shape is tolerated for a future Go sender's natural `{executionId}` JSON,
+ * mirroring child_execution_started's both-shapes handling below.
+ */
 const childApprovalRequired = (0,_temporalio_workflow__WEBPACK_IMPORTED_MODULE_0__.defineSignal)("child_approval_required");
 /** Sent by the platform immediately after the child AgentExecution starts. */
 const childExecutionStarted = (0,_temporalio_workflow__WEBPACK_IMPORTED_MODULE_0__.defineSignal)("child_execution_started");
@@ -27992,14 +28003,26 @@ async function orchestrateAgentCall(input) {
     let activityDone = false;
     let activityResult = {};
     let activityError = undefined;
-    let pendingNotification;
+    // Child ids whose approval gates await derivation. A set (not a flag)
+    // because one workflow has ONE live handler per signal name: with parallel
+    // agent_call tasks, whichever orchestration registered last receives every
+    // child's signal, and each gate must be derived under its OWN child id for
+    // the per-child status merge to file it correctly.
+    const pendingApprovalChildIds = new Set();
     let childExecId;
     let initialProgressEmitted = false;
-    (0,_temporalio_workflow__WEBPACK_IMPORTED_MODULE_0__.setHandler)(childApprovalRequired, (notification) => {
-        pendingNotification = notification;
-        if (!childExecId && notification.executionId) {
-            childExecId = notification.executionId;
+    (0,_temporalio_workflow__WEBPACK_IMPORTED_MODULE_0__.setHandler)(childApprovalRequired, (payload) => {
+        // Identity-only signal (see the definition above): note the child and
+        // mark its gate for derivation in the main loop — approval details never
+        // travel through the signal itself.
+        const signaledId = typeof payload === "string" ? payload : payload?.executionId;
+        if (!signaledId) {
+            return;
+        }
+        if (!childExecId) {
+            childExecId = signaledId;
         }
+        pendingApprovalChildIds.add(signaledId);
     });
     (0,_temporalio_workflow__WEBPACK_IMPORTED_MODULE_0__.setHandler)(childExecutionStarted, (payload) => {
         // The Go server sends { executionId: "aex_xxx" } (struct with json tag).
@@ -28090,7 +28113,7 @@ async function orchestrateAgentCall(input) {
     while (!activityDone) {
         // Wait for a signal, activity completion, or periodic timeout for progress polling.
         // condition() returns false on timeout, true when the predicate became true.
-        const conditionMet = await (0,_temporalio_workflow__WEBPACK_IMPORTED_MODULE_0__.condition)(() => activityDone || pendingNotification !== undefined || (!!childExecId && !initialProgressEmitted), PROGRESS_POLL_INTERVAL);
+        const conditionMet = await (0,_temporalio_workflow__WEBPACK_IMPORTED_MODULE_0__.condition)(() => activityDone || pendingApprovalChildIds.size > 0 || (!!childExecId && !initialProgressEmitted), PROGRESS_POLL_INTERVAL);
         if (activityDone)
             break;
         // Emit initial progress with childExecutionId as soon as it's known
@@ -28116,18 +28139,32 @@ async function orchestrateAgentCall(input) {
             }
             await syncFileReviews(childExecId);
         }
-        // Handle HITL approval notification
-        if (pendingNotification) {
-            const notification = pendingNotification;
-            pendingNotification = undefined;
-            try {
-                await statusProxy.UpdateWorkflowTaskApprovalStatus(input.workflowExecutionId, input.taskName, notification);
-            }
-            catch (statusErr) {
-                _temporalio_workflow__WEBPACK_IMPORTED_MODULE_0__.log.warn("Failed to update workflow approval status (non-fatal)", {
-                    error: String(statusErr),
-                    taskName: input.taskName,
-                });
+        // Handle HITL approval notifications: derive each signaled child's gate
+        // from its persisted record (identity-only signal, DD-012). The child's
+        // server persists the gate BEFORE signaling, so an empty derivation means
+        // the gate already resolved — the activity answers false and there is
+        // deliberately no retry (see updateWorkflowTaskApprovalStatus).
+        if (pendingApprovalChildIds.size > 0) {
+            // Drain a deterministic snapshot: insertion order is replay-stable, and
+            // ids signaled during the awaits below land in the set for the next pass.
+            const toDerive = [...pendingApprovalChildIds];
+            pendingApprovalChildIds.clear();
+            for (const signaledChildId of toDerive) {
+                try {
+                    const surfaced = await statusProxy.UpdateWorkflowTaskApprovalStatus(input.workflowExecutionId, input.taskName, signaledChildId);
+                    if (!surfaced) {
+                        _temporalio_workflow__WEBPACK_IMPORTED_MODULE_0__.log.info("Child approval gate already resolved before derivation; nothing surfaced", {
+                            taskName: input.taskName,
+                            childExecId: signaledChildId,
+                        });
+                    }
+                }
+                catch (statusErr) {
+                    _temporalio_workflow__WEBPACK_IMPORTED_MODULE_0__.log.warn("Failed to update workflow approval status (non-fatal)", {
+                        error: String(statusErr),
+                        taskName: input.taskName,
+                    });
+                }
             }
         }
     }