@uipath/maestro-builder-sdk 6.11.5 → 6.12.0

package/dist/bpmn/bpmn-expr-check.js

@@ -148,8 +148,15 @@ function walk(nodes, flows, rootsIn, out) {
             // enclosing scope (before entry); the body and completion see the sub-scope.
             checkExpr(n.loop?.collection, `${at} loop collection`, roots, out);
             const subRoots = { vars: new Set(roots.vars), varIds: new Set(roots.varIds), bindings: roots.bindings };
-            for (const v of n.variables)
+            // By ID as well as display name, exactly as the root scope below: an expression
+            // reads `vars.<id>`. This added the name alone, so a sub-process variable whose
+            // display name differed from its id — every `errorVar`, displayed as `Error` —
+            // was unreadable inside the very scope that declares it.
+            for (const v of n.variables) {
+                subRoots.vars.add(v.id);
                 subRoots.vars.add(v.name);
+                subRoots.varIds.add(v.id);
+            }
             checkExpr(n.loop?.completion, `${at} loop completion`, subRoots, out);
             walk(n.nodes, n.flows, subRoots, out);
         }