camunda-cli 0.3.0 → 0.3.1

package/bin/camunda.js

@@ -24,7 +24,7 @@ program
       'Start with "camunda inspect <key>" to read a deployed model, and\n' +
       '"camunda diagnose <instanceId>" when an instance misbehaves.'
   )
-  .version('0.3.0')
+  .version('0.3.1')
   .option('--json', 'print the raw API payload instead of a formatted view')
   .option('--no-color', 'never emit colour, even on a terminal')
   .showHelpAfterError()

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "camunda-cli",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Command-line client for self-hosted Camunda 7: inspect and lint deployed BPMN models, and diagnose why an instance is stuck",
   "license": "MIT",
   "type": "module",

package/src/lint.js

@@ -108,7 +108,8 @@ export function lintProcess(model, { engineVariables = [] } = {}) {
         n.id,
         `Every outgoing flow of "${n.name || n.id}" has a condition and there is no default flow. ` +
           `If they all evaluate false at runtime the engine raises ENGINE-02004 and the instance stops. ` +
-          `Mark one flow as the default.`
+          `Mark one flow as the default and remove its condition: a default flow carrying a condition is ` +
+          `rejected at deploy time with ENGINE-09005.`
       );
 
       // Numeric gap: a > N and a < N leave a == N with nowhere to go.
@@ -134,6 +135,29 @@ export function lintProcess(model, { engineVariables = [] } = {}) {
       }
     }
 
+    // The engine refuses to parse this rather than failing at runtime, so a model that is
+    // otherwise fine is rejected outright at deploy time.
+    if (n.defaultFlow) {
+      const target = outs.find((f) => f.id === n.defaultFlow);
+      if (!target) {
+        add(
+          'error',
+          'default-flow-missing',
+          n.id,
+          `"${n.name || n.id}" names "${n.defaultFlow}" as its default flow, but that is not one of its outgoing flows.`
+        );
+      } else if (target.condition) {
+        add(
+          'error',
+          'default-flow-with-condition',
+          n.id,
+          `The default flow of "${n.name || n.id}" ("${target.name || target.id}") also carries the condition ` +
+            `${target.condition}. A default flow is the branch taken when nothing else matches, so it cannot have ` +
+            `one: the engine rejects this at deploy time with ENGINE-09005. Remove the condition from that flow.`
+        );
+      }
+    }
+
     if (n.type === 'exclusiveGateway' && unconditional.length > 1) {
       add(
         'warning',