@uipath/maestro-builder-sdk 6.15.4 → 6.15.6

package/dist/serialize.js

@@ -16,7 +16,6 @@ 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 { 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';
@@ -46,6 +45,7 @@ import { declaredScriptShape, inferScriptShape } from './core/script-return.js';
 import { migrateToSchemaVersion } from './migrate.js';
 import { FLOW_SCHEMA_VERSIONS } from './format-profile.js';
 import { resolveSchemaVersion } from './schema-version.js';
+import { planFlowGlobals } from './flow-variables.js';
 import { prepareCommand } from './core/cli-spelling.js';
 import { QUEUE_ITEM_RECORD_SCHEMA } from './core/queue-record.js';
 const HERE = dirname(fileURLToPath(import.meta.url));
@@ -3485,12 +3485,6 @@ 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.
@@ -3957,6 +3951,24 @@ function misplacedLookupMessage(token) {
         `a lookup() must be the whole value of a connector input, or an element of an array input. ` +
         `Emitting it here would send a raw {"__lookup": …} object where the connector expects an id.`);
 }
+/**
+ * The flow's declarations as the globals it will emit: a same-named `.output()`
+ * and `.var()` folded into one `out` global (flow-builder-sdk#899), every other
+ * repeated name refused. `check()` reports the same problems as
+ * VARIABLE_NAME_COLLISION / OUTPUT_VAR_MISMATCH; this throws too, so a path that skips the static pass
+ * (`compile --no-check`, `emitFlow`, a direct `serialize()`) cannot write a
+ * file `flow validate` rejects for a duplicate variable id.
+ */
+function emittableGlobals(built, where) {
+    // Entry-point inputs are rows of the same globals[], so they share the
+    // namespace; `check` reports those as ENTRY_POINT_INPUT_COLLISION.
+    const inputs = [...built.inputs, ...(built.entryPoints ?? []).flatMap((ep) => ep.inputs)];
+    const plan = planFlowGlobals({ inputs, outputs: built.outputs, vars: built.vars });
+    if (plan.problems.length) {
+        throw new Error(`${where}: ${plan.problems.map((p) => `[${p.code}] ${p.message}`).join(' ')}`);
+    }
+    return plan;
+}
 function newScope(outputs, vars = []) {
     return {
         nodes: [],
@@ -4280,7 +4292,8 @@ export function serialize(built, opts = {}) {
      */
     const sidecars = opts.sidecars ?? [];
     const subflows = {};
-    const root = newScope(built.outputs, built.vars);
+    const rootGlobals = emittableGlobals(built, `flow "${built.id}"`);
+    const root = newScope(rootGlobals.outputs, rootGlobals.vars);
     // `input('x')` lowers eagerly to `$vars.start.output.x` — `start` being the
     // trigger node's default id. When the flow renames the trigger, register that
     // rename so the SAME pass that rewrites step-name references rewrites these
@@ -4404,7 +4417,8 @@ export function serialize(built, opts = {}) {
      * a null. It is emitted here, always, so it cannot be forgotten.
      */
     function emitSubflowBody(child, nodeId) {
-        const scope = newScope(child.outputs, child.vars);
+        const childGlobals = emittableGlobals(child, `subflow "${nodeId}"`);
+        const scope = newScope(childGlobals.outputs, childGlobals.vars);
         // Claim this copy's node ids UP FRONT, so the rename map is complete before
         // the first expression renders. A child's step names are free the first
         // time it runs; if any of them is already taken — which is what a second
@@ -4443,13 +4457,12 @@ 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) });
-        for (const o of child.outputs)
+        for (const o of childGlobals.outputs)
             globals.push({ id: o.name, name: o.name, direction: 'out', type: o.type, ...varMeta(o) });
-        for (const vd of child.vars)
+        for (const vd of childGlobals.vars)
             globals.push({ id: vd.name, name: vd.name, direction: 'inout', type: vd.type, ...varMeta(vd) });
         resolveRejoins(scope);
         assertHitlExitInvariant(scope.nodes, scope.edges);
@@ -6531,16 +6544,13 @@ 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)
         globals.push({ id: inp.name, name: inp.name, direction: 'in', type: inp.type, triggerNodeId, ...varMeta(inp) });
-    for (const o of built.outputs)
+    for (const o of rootGlobals.outputs)
         globals.push({ id: o.name, name: o.name, direction: 'out', type: o.type, ...varMeta(o) });
-    for (const vd of built.vars)
+    for (const vd of rootGlobals.vars)
         globals.push({ id: vd.name, name: vd.name, direction: 'inout', type: vd.type, ...varMeta(vd) });
     for (const ep of entryPoints) {
         for (const inp of ep.inputs) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uipath/maestro-builder-sdk",
-  "version": "6.15.4",
+  "version": "6.15.6",
   "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": "37cc4f8cdcd0cfae5ec65c17d389c762e74f4a7c"
+  "gitref": "d8d72580682f02833d28d720ee382ffe9d346f6e"
 }