@uipath/maestro-builder-sdk 6.15.4 → 6.15.5

package/dist/registry/path-cli.js

@@ -1,4 +1,4 @@
-import { reportResult } from '../cli-result.js';
+import { printResult, reportResult } from '../cli-result.js';
 import { join } from "node:path";
 import { CURRENT_LINK, PULL_COMMAND, currentEntrySha, libraryCacheRoot, resolveCachedLibrary, resolveCachedMarkdown, } from "./cache.js";
 /**
@@ -37,7 +37,7 @@ export async function run(argv) {
             console.error(`No connector library in the cache yet. Run \`${PULL_COMMAND}\`.`);
             return 1;
         }
-        console.log(value);
+        printResult(value);
         return 0;
     };
     if (argv.includes("--library-json"))
@@ -56,7 +56,7 @@ export async function run(argv) {
         libraryMarkdown: libraryMarkdown ?? null,
     };
     reportResult(all);
-    console.log(JSON.stringify(all, null, 2));
+    printResult(JSON.stringify(all, null, 2));
     // Not an error: a caller asking for the whole picture is entitled to be told
     // the cache is empty, rather than getting a non-zero exit for a plain report.
     return 0;

package/dist/registry/search-cli.js

@@ -1,4 +1,4 @@
-import { reportResult } from '../cli-result.js';
+import { printResult, reportResult } from '../cli-result.js';
 import { readFile, stat } from "node:fs/promises";
 import { PULL_COMMAND, resolveLibraryDir } from "./cache.js";
 import { dirname, join } from "node:path";
@@ -155,7 +155,7 @@ export async function run(argv) {
             limit,
         });
         reportResult(result);
-        console.log(JSON.stringify(result, null, 2));
+        printResult(JSON.stringify(result, null, 2));
         return 0;
     }
     catch (error) {

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.5",
   "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": "7c0103ce6f6d970156e15bbc3bae0a3f88ca4195"
 }