@uipath/maestro-builder-sdk 6.15.1 → 6.15.3

package/dist/serialize.js

@@ -16,7 +16,8 @@ import { connectorRawNodeRefusal } from './core/connector-raw-node.js';
 import { dataFabricUnknownInputs } from './core/typed-raw-node.js';
 import { isAnswerField } from './core/hitl-answers.js';
 import { connectionConnectors } from './bindings-v2.js';
-import { BindingRefusal, GUID_RE, bindingSelfNameMessage, bindingUndeclaredMessage, undeclaredBinding } from './core/binding-messages.js';
+import { globalNameCollisionMessage, globalNameCollisions } from './core/global-names.js';
+import { BindingRefusal, GUID_RE, bindingMissingMessage, bindingSelfNameMessage, bindingUndeclaredMessage, missingBindingLabels, undeclaredBinding } from './core/binding-messages.js';
 import { ByoaRefusal, byoaUnverified } from './core/byoa.js';
 import { OUTCOME_HANDLE_NODE_TYPES, assertHitlExitInvariant, hitlExitModel, hitlExitViolation, outcomeSlug } from './core/hitl-routing.js';
 import { declaredExits, hasExits, stopsAtEnd } from './core/step-ports.js';
@@ -3284,7 +3285,7 @@ function clone(x) {
 function subNodeId(obj, nodeId) {
     return JSON.parse(JSON.stringify(obj).split('{nodeId}').join(nodeId));
 }
-const NO_RENAME = { nodeIds: new Map(), runtimeOutputs: new Map(), errorEnvelopes: new Map() };
+const NO_RENAME = { nodeIds: new Map(), errorEnvelopes: new Map() };
 /**
  * Which runtime variable holds a failed node's envelope, by node type.
  *
@@ -3484,6 +3485,27 @@ function refuseUnverifiedByoa(opts, where, sub) {
     if (message)
         throw new ByoaRefusal(message);
 }
+/** Refuse a name declared twice across `.input()` / `.output()` / `.var()` — one `variables.globals[]` id each. */
+function refuseGlobalNameCollisions(inputs, outputs, vars) {
+    const [first] = globalNameCollisions(inputs, outputs, vars);
+    if (first)
+        throw new Error(globalNameCollisionMessage(first));
+}
+/**
+ * Refuse a connector-bound node (connector step, event trigger, `.entryPoint`
+ * onEvent, `waitForEvent`) that names no `connection` or no `folder` label.
+ * (An inline agent's connector tool is #875's: `check` refuses it as
+ * INLINE_AGENT_TOOL_CONNECTOR_NO_CONNECTION, and serialize leaves it unbound.)
+ * `resolveBinding` returns `undefined` for an absent label, which used to reach
+ * the node as `detail.connectionId: null` with no binding — valid to compile,
+ * refused by `flow validate` (flow-builder-sdk#890). `check` reports the same
+ * sentence first (BINDING_MISSING).
+ */
+function requireBindingPair(where, labels, target) {
+    const missing = missingBindingLabels(labels);
+    if (missing.length > 0)
+        throw new BindingRefusal('BINDING_MISSING', bindingMissingMessage(where, missing, target));
+}
 /**
  * Resolve a symbolic connection/folder name to the resource key that is inlined
  * as `connectionId` / `connectionFolderKey`.
@@ -3709,7 +3731,8 @@ function flattenWhere(where, declared) {
     walk(where, '');
     return out;
 }
-function eventDetail(rc, sub, nodeId, opts) {
+function eventDetail(rc, sub, nodeId, opts, site) {
+    requireBindingPair(site, sub, eventTarget(sub));
     const conn = resolveBinding(opts, sub.connection, 'connection', `event node "${nodeId}"`, eventTarget(sub));
     const folder = resolveBinding(opts, sub.folder, 'folder', `event node "${nodeId}"`, eventTarget(sub));
     const op = (rc.entry?.operation ?? {});
@@ -3944,7 +3967,7 @@ function newScope(outputs, vars = []) {
             ...outputs.map((o) => [o.name, o.type]),
             ...vars.map((vd) => [vd.name, vd.type]),
         ]),
-        rename: { nodeIds: new Map(), runtimeOutputs: new Map(), errorEnvelopes: new Map() },
+        rename: { nodeIds: new Map(), errorEnvelopes: new Map() },
         pending: new Map(),
         agentSchemas: [],
         stepNodeId: new Map(),
@@ -4190,22 +4213,18 @@ function terminateOutputs(scope) {
  * same name (an error in its own right — DUP_STEP).
  */
 function renameRefs(text, rename) {
-    if (rename.nodeIds.size === 0 && rename.runtimeOutputs.size === 0 && rename.errorEnvelopes.size === 0)
+    if (rename.nodeIds.size === 0 && rename.errorEnvelopes.size === 0)
         return text;
     // `.error` matches only with a FIELD behind it: a bare `$vars.<step>.error` is
     // the boolean "did this fail" (`err('<step>')`), which is truthy on every
     // family — the envelope object included — and must be left alone.
-    return text.replace(/\$vars\.([A-Za-z0-9_]+)(\.output\b|\.error(?=\.[A-Za-z0-9_]))?/g, (m, name, tail) => {
+    return text.replace(/\$vars\.([A-Za-z0-9_]+)(\.error(?=\.[A-Za-z0-9_]))?/g, (m, name, tail) => {
         const to = rename.nodeIds.get(name);
         const nodeId = to && to.length > 0 ? to[0] : name;
         if (!tail)
             return nodeId === name ? m : `$vars.${nodeId}`;
-        if (tail === '.error') {
-            const envelope = rename.errorEnvelopes.get(name) ?? 'error';
-            return `$vars.${nodeId}.${envelope}`;
-        }
-        const runtimeOutput = rename.runtimeOutputs.get(name) ?? 'output';
-        return `$vars.${nodeId}.${runtimeOutput}`;
+        const envelope = rename.errorEnvelopes.get(name) ?? 'error';
+        return `$vars.${nodeId}.${envelope}`;
     });
 }
 /** Every step name a flow declares in its own scope (not its children's). */
@@ -4424,6 +4443,7 @@ export function serialize(built, opts = {}) {
             for (const t of tails)
                 addEdge(scope, t, endId);
         }
+        refuseGlobalNameCollisions(child.inputs, child.outputs, child.vars);
         const globals = [];
         for (const inp of child.inputs)
             globals.push({ id: inp.name, name: inp.name, direction: 'in', type: inp.type, triggerNodeId: startId, ...varMeta(inp) });
@@ -4512,7 +4532,7 @@ export function serialize(built, opts = {}) {
                 // `detail` (see eventDetail).
                 entryPointId: stableId(`${built.id}:start:entryPoint`),
                 isDefaultEntryPoint: true,
-                detail: eventDetail(eventTriggerRc, built.trigger.subscription, triggerId, opts),
+                detail: eventDetail(eventTriggerRc, built.trigger.subscription, triggerId, opts, 'event trigger'),
             }
             : built.trigger?.kind === 'scheduled'
                 ? scheduleInputs(built.trigger.inputs, triggerId)
@@ -4662,6 +4682,7 @@ export function serialize(built, opts = {}) {
             const id = stepUid(scope, step.name);
             const node = makeNode(rc.nodeType, id, step.name, { parentId, requestedVersion: step.options?.version, def: rc.v1def });
             declareListOutputType(node, rc);
+            requireBindingPair(`connector "${step.name}"`, spec, { key: spec.key, action: spec.action });
             const conn = resolveBinding(opts, spec.connection, 'connection', `connector "${step.name}"`, { key: spec.key, action: spec.action });
             const folder = resolveBinding(opts, spec.folder, 'folder', `connector "${step.name}"`, { key: spec.key, action: spec.action });
             // The platform reaches the connection through the flow's `bindings[]`,
@@ -5310,12 +5331,12 @@ export function serialize(built, opts = {}) {
                 resourceBindings.set(b.id, b);
             const def = queueItemDef(s, step.name, ids);
             const id = stepUid(scope, step.name);
-            // Queue manifests describe the logical result as `output`, which is what
-            // out('step', path) type-checks against. The product's BPMN serializer,
-            // however, publishes both queue variants under `$vars.<step>.response`.
-            // Record that boundary alias once so every SDK expression renderer uses
-            // the runtime path while the public output contract stays unchanged.
-            scope.rename.runtimeOutputs.set(step.name, 'response');
+            // The result is read as `$vars.<step>.output`, like every other family: the
+            // node's own output block maps the runtime `response` onto the variable
+            // `output` (`source: "=response", var: "output"`), and the product honours
+            // that mapping for both queue variants. (An alias to `.response` lived here
+            // from #273 until #891; a live debug run on 2026-10-01 measured
+            // `.output.Key` populated and `.response` undefined.)
             const node = makeNode(def.nodeType, id, step.name, { parentId, requestedVersion: step.options?.version, def });
             // The node KEEPS its copy of the model here, unlike the rpa families. Two
             // reasons: it is what the one queue instance the repo has carries, and it is
@@ -5618,7 +5639,7 @@ export function serialize(built, opts = {}) {
             refuseUnverifiedByoa(opts, `waitForEvent "${step.name}"`, sub);
             const id = stepUid(scope, step.name);
             const node = makeNode(rc.nodeType, id, step.name, { parentId, requestedVersion: step.options?.version, def: rc.v1def });
-            node.inputs = { detail: eventDetail(rc, sub, id, opts) };
+            node.inputs = { detail: eventDetail(rc, sub, id, opts, `waitForEvent "${step.name}"`) };
             // Same connection-binding closure as the connector branch (G-19).
             const { entries: waitConnBindings } = connectionResourceBindings({
                 def: rc.v1def,
@@ -5856,7 +5877,7 @@ export function serialize(built, opts = {}) {
         // Record the failure-envelope boundary BEFORE any expression is rendered, so
         // a handler that reads this step resolves against the right variable. The
         // handler is emitted after the step it guards, so recording it here is early
-        // enough — the same ordering the queue family's `runtimeOutputs` alias relies on.
+        // enough.
         const envelope = errorEnvelopeVar(type);
         if (envelope)
             scope.rename.errorEnvelopes.set(step.name, envelope);
@@ -6475,7 +6496,7 @@ export function serialize(built, opts = {}) {
                 node = makeNode(rc.nodeType, ep.id, 'Event trigger', { def: rc.v1def, requestedVersion });
                 node.inputs = {
                     entryPointId: stableId(`${built.id}:${ep.id}:entryPoint`),
-                    detail: eventDetail(rc, sub, ep.id, opts),
+                    detail: eventDetail(rc, sub, ep.id, opts, `onEvent() entry point "${ep.id}"`),
                 };
                 const { entries } = connectionResourceBindings({
                     def: rc.v1def,
@@ -6510,6 +6531,9 @@ export function serialize(built, opts = {}) {
             }
         }
     }
+    // Entry-point inputs are rows of the same globals[] (pushed below), so they
+    // share the namespace; `check` reports these as ENTRY_POINT_INPUT_COLLISION.
+    refuseGlobalNameCollisions([...built.inputs, ...entryPoints.flatMap((ep) => ep.inputs)], built.outputs, built.vars);
     const globals = [];
     const triggerNodeId = built.triggerId ?? 'start';
     for (const inp of built.inputs)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uipath/maestro-builder-sdk",
-  "version": "6.15.1",
+  "version": "6.15.3",
   "description": "Build UiPath Flow, Case, and BPMN artifacts by writing TypeScript.",
   "license": "MIT",
   "homepage": "https://docs.uipath.com/maestro",
@@ -92,5 +92,5 @@
     "@types/node": "^22.7.0",
     "esbuild": "^0.28.1"
   },
-  "gitref": "a3707b159a49ed3b6611c55bed66622cb6dd1972"
+  "gitref": "a85d11af254f270d645ec46d57994ccb800ebc1f"
 }