@nanobpm/urban 0.32.0 → 0.32.1

package/dist/runtime/core/modules/pages.js

@@ -686,12 +686,21 @@ function renderActionForm(node) {
           msg.textContent = "Field '" + p.action.correlationKeyField + "' is required";
           return;
         }
+        // Send the trimmed key back as the variable too, so the value a worker
+        // reads matches the key the message was correlated on (no leading/trailing
+        // whitespace divergence between correlationKey and variables[field]).
+        variables[p.action.correlationKeyField] = correlationKey;
         await getJSON("/app/actions/message", { method: "POST",
           headers: { "content-type": "application/json" },
           body: JSON.stringify({ name: p.action.message, correlationKey, variables }) });
         msg.className = "pc-msg ok";
         msg.textContent = "Message published";
       } else {
+        if (!p.action || !p.action.process) {
+          msg.className = "pc-msg err";
+          msg.textContent = "This form has no action configured";
+          return;
+        }
         const res = await getJSON("/app/actions/start/" + encodeURIComponent(p.action.process),
           { method: "POST", headers: { "content-type": "application/json" }, body: JSON.stringify({ variables }) });
         msg.className = "pc-msg ok";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nanobpm/urban",
-  "version": "0.32.0",
+  "version": "0.32.1",
   "description": "Urban: build and run code-first apps on Nano — runtime, derivation toolkit, and CLI in one. Author durable processes with @nanobpm/workflow (defineFlow), a typed datasource, forms, triggers and surfaces from a nano.app.json manifest, on Node or Deno.",
   "license": "Apache-2.0",
   "type": "module",

package/src/runtime/core/modules/pages.test.ts

@@ -146,8 +146,14 @@ test("renderActionForm supports a publishMessage action (message-started process
   assert.match(js, /p\.action\.kind === "publishMessage"/);
   assert.match(js, /variables\[p\.action\.correlationKeyField\]/);
   assert.match(js, /"\/app\/actions\/message"/);
+  // The trimmed correlation key is written back as the variable, so the value a
+  // worker reads matches the key the message was correlated on.
+  assert.match(js, /variables\[p\.action\.correlationKeyField\] = correlationKey/);
   // startProcess remains the default/back-compat branch.
   assert.match(js, /"\/app\/actions\/start\/" \+ encodeURIComponent\(p\.action\.process\)/);
+  // A missing action is reported clearly instead of throwing on p.action.process.
+  assert.match(js, /if \(!p\.action \|\| !p\.action\.process\)/);
+  assert.match(js, /no action configured/);
 });
 
 test("the renderer bridges the console theme over postMessage", async () => {

package/src/runtime/core/modules/pages.ts

@@ -733,12 +733,21 @@ function renderActionForm(node) {
           msg.textContent = "Field '" + p.action.correlationKeyField + "' is required";
           return;
         }
+        // Send the trimmed key back as the variable too, so the value a worker
+        // reads matches the key the message was correlated on (no leading/trailing
+        // whitespace divergence between correlationKey and variables[field]).
+        variables[p.action.correlationKeyField] = correlationKey;
         await getJSON("/app/actions/message", { method: "POST",
           headers: { "content-type": "application/json" },
           body: JSON.stringify({ name: p.action.message, correlationKey, variables }) });
         msg.className = "pc-msg ok";
         msg.textContent = "Message published";
       } else {
+        if (!p.action || !p.action.process) {
+          msg.className = "pc-msg err";
+          msg.textContent = "This form has no action configured";
+          return;
+        }
         const res = await getJSON("/app/actions/start/" + encodeURIComponent(p.action.process),
           { method: "POST", headers: { "content-type": "application/json" }, body: JSON.stringify({ variables }) });
         msg.className = "pc-msg ok";