@uipath/maestro-builder-sdk 6.16.6 → 6.16.8

package/dist/flow-step-tree.d.ts

@@ -0,0 +1,25 @@
+/**
+ * flow-step-tree — walkers over a Flow's `Step` tree. They are typed by Flow's
+ * `Step` union (`childStepLists` is exhaustive over `Step['kind']`), so they are
+ * Flow code; the family-neutral exit table stays in `core/step-ports.ts`.
+ */
+import type { Step } from './flow-sdk.js';
+/**
+ * Whether a step list that hands on no tail got there by reaching an End — a
+ * `.return()`, on some path — rather than by stopping the whole run
+ * (`.terminate`) or leaving a loop (`.break`). Only the first leaves a sibling
+ * `.parallel` join waiting: a terminate ends the run, join included.
+ */
+export declare function stopsAtEnd(steps: readonly Step[]): boolean;
+/**
+ * The step lists nested DIRECTLY inside `step` — the bodies a walk over this
+ * flow's own tree must descend into. Exhaustive over `Step['kind']`, so a new
+ * container kind fails to compile here instead of being silently skipped by
+ * every walker. That is how `check`'s return scan came to miss `.stepSwitch`
+ * arms and `.stepToList` / `.onError` bodies, and warned NO_RETURN on flows
+ * whose every path ends in `.return()` (UV-16855).
+ *
+ * A subflow's child flow is deliberately NOT a child here: it is another
+ * flow's tree, and its returns say nothing about whether THIS flow ends.
+ */
+export declare function childStepLists(step: Step): readonly Step[][];

package/dist/flow-step-tree.js

@@ -0,0 +1,50 @@
+/**
+ * Whether a step list that hands on no tail got there by reaching an End — a
+ * `.return()`, on some path — rather than by stopping the whole run
+ * (`.terminate`) or leaving a loop (`.break`). Only the first leaves a sibling
+ * `.parallel` join waiting: a terminate ends the run, join included.
+ */
+export function stopsAtEnd(steps) {
+    const has = (list, kinds) => list.some((st) => kinds.has(st.kind)
+        || (st.kind === 'branch' && (has(st.then, kinds) || has(st.otherwise, kinds)))
+        || ((st.kind === 'switch' || st.kind === 'stepSwitch') && st.cases.some((c) => has(c.body, kinds)))
+        || (st.kind === 'switch' && st.default !== undefined && has(st.default, kinds)));
+    return has(steps, new Set(['return'])) && !has(steps, new Set(['terminate', 'break']));
+}
+/**
+ * The step lists nested DIRECTLY inside `step` — the bodies a walk over this
+ * flow's own tree must descend into. Exhaustive over `Step['kind']`, so a new
+ * container kind fails to compile here instead of being silently skipped by
+ * every walker. That is how `check`'s return scan came to miss `.stepSwitch`
+ * arms and `.stepToList` / `.onError` bodies, and warned NO_RETURN on flows
+ * whose every path ends in `.return()` (UV-16855).
+ *
+ * A subflow's child flow is deliberately NOT a child here: it is another
+ * flow's tree, and its returns say nothing about whether THIS flow ends.
+ */
+export function childStepLists(step) {
+    switch (step.kind) {
+        case 'branch':
+            return [step.then, step.otherwise];
+        case 'switch':
+            return [...step.cases.map((c) => c.body), ...(step.default ? [step.default] : [])];
+        case 'stepSwitch':
+            return step.cases.map((c) => c.body);
+        case 'loop':
+        case 'doWhile':
+        case 'stepToList':
+            return [step.body];
+        case 'parallel':
+            return step.arms;
+        case 'action':
+        case 'break':
+        case 'terminate':
+        case 'stepToRef':
+        case 'return':
+            return [];
+        default: {
+            const unhandled = step;
+            throw new Error(`childStepLists: unhandled step kind ${JSON.stringify(unhandled.kind)}`);
+        }
+    }
+}

package/dist/registry/path-cli.js

@@ -1,6 +1,6 @@
 import { printResult, reportResult } from '../cli-result.js';
 import { join } from "node:path";
-import { CURRENT_LINK, PULL_COMMAND, currentEntrySha, libraryCacheRoot, resolveCachedLibrary, resolveCachedMarkdown, } from "./cache.js";
+import { CURRENT_LINK, PULL_COMMAND, currentEntrySha, libraryCacheRoot, resolveCachedLibrary, resolveCachedMarkdown, } from "../core/library-cache.js";
 /**
  * Say where the connector library is.
  *

package/dist/registry/pull-cli.js

@@ -2,7 +2,7 @@ import { reportResult } from '../cli-result.js';
 import { createHash } from "node:crypto";
 import { mkdir, readFile, rename, rm, stat, symlink, writeFile, } from "node:fs/promises";
 import { join, resolve } from "node:path";
-import { CURRENT_LINK, LIBRARY_JSON_DIR, LIBRARY_MARKDOWN_DIR, cacheEntry, libraryCacheRoot, referencedConnectorKeys, } from "./cache.js";
+import { CURRENT_LINK, LIBRARY_JSON_DIR, LIBRARY_MARKDOWN_DIR, cacheEntry, libraryCacheRoot, referencedConnectorKeys, } from "../core/library-cache.js";
 import { extractLibrary } from "./extractor.js";
 import { materializeRegistry } from "./materializer.js";
 import { curatedOverlayIdentity } from "./curated-overlay.js";

package/dist/registry/search-cli.js

@@ -1,6 +1,6 @@
 import { printResult, reportResult } from '../cli-result.js';
 import { readFile, stat } from "node:fs/promises";
-import { PULL_COMMAND, resolveLibraryDir } from "./cache.js";
+import { PULL_COMMAND, resolveLibraryDir } from "../core/library-cache.js";
 import { dirname, join } from "node:path";
 export const DEFAULT_SEARCH_LIMIT = 20;
 const USAGE = "Usage: flow-sdk registry search [query] [--library <dir>] [--library-md <dir>] [--connector <key>] [--limit <n>]";

package/dist/serialize.js

@@ -19,10 +19,11 @@ import { connectionConnectors } from './bindings-v2.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';
+import { declaredExits, hasExits } from './core/step-ports.js';
+import { stopsAtEnd } from './flow-step-tree.js';
 import { assertNodeEnvelope } from './core/node-envelope.js';
 import { buildConfiguration } from './config.js';
-import { stableId } from './core/stable-id.js';
+import { fnv1aX4Uuid, stableId } from './core/stable-id.js';
 import { readEventFilter, eventFilterProblem, eventFilterJmes, eventFilterTreeLeaf } from './core/event-filters.js';
 import { FLOW_EXPR_DIALECT } from './core/connector-checks.js';
 import { buildConnectorInputs, connectorInputErrors as coreConnectorInputErrors, multipartBodyFieldName, transportHttpMethod } from './core/connector-inputs.js';
@@ -3428,37 +3429,6 @@ export function serializeWithSidecars(built, opts = {}) {
     const flow = serialize(built, { ...opts, sidecars });
     return { flow, sidecars };
 }
-/**
- * A deterministic v4-shaped uuid from a string.
- *
- * Used for an inline agent's `source` when the author does not pass one. It has to
- * LOOK like a uuid (the platform treats it as an opaque directory name and the
- * designer shows it as a folder) and it has to be STABLE, because a random one would
- * make every recompile a diff of the `.flow` and a rename of a directory. FNV-1a
- * ×4 with the version/variant nibbles pinned — not a cryptographic hash, and it does
- * not need to be: the only requirement is that two different (flow, step) pairs do
- * not collide within one project.
- */
-function stableUuid(seed) {
-    const words = [];
-    for (let i = 0; i < 4; i++) {
-        let h = 0x811c9dc5 ^ (i * 0x9e3779b9);
-        const s = `${seed}#${i}`;
-        for (let j = 0; j < s.length; j++) {
-            h ^= s.charCodeAt(j);
-            h = Math.imul(h, 0x01000193) >>> 0;
-        }
-        words.push(h >>> 0);
-    }
-    const hex = words.map((w) => w.toString(16).padStart(8, '0')).join('');
-    return [
-        hex.slice(0, 8),
-        hex.slice(8, 12),
-        `4${hex.slice(13, 16)}`,
-        `${((parseInt(hex.slice(16, 17), 16) & 0x3) | 0x8).toString(16)}${hex.slice(17, 20)}`,
-        hex.slice(20, 32),
-    ].join('-');
-}
 /**
  * `variables.nodes` — one entry per node OUTPUT, derived from the nodes we just
  * emitted.
@@ -5080,7 +5050,7 @@ export function serialize(built, opts = {}) {
             // name rather than random so a recompile is byte-identical — a v4-shaped uuid
             // over a stable hash, because the platform treats it as an opaque id and the
             // designer shows it as a folder.
-            const source = s.source ?? stableUuid(`${built.id}/${step.name}`);
+            const source = s.source ?? fnv1aX4Uuid(`${built.id}/${step.name}`);
             const node = makeNode(def.nodeType, id, step.name, { parentId, requestedVersion: step.options?.version, def });
             // Unlike the published families the inline agent KEEPS no node-level model:
             // 29 of 29 deployed instances carry none (the def's `model.source: true` is
@@ -5192,7 +5162,7 @@ export function serialize(built, opts = {}) {
                 // resource's `id` are the same uuid, because that is what joins them. The
                 // platform validator says so in its own refusal text — *"the resource id
                 // from the parent inline agent's `resources[]`"*.
-                const contextResourceId = stableUuid(`${built.id}/${step.name}/context/${cx.id}`);
+                const contextResourceId = fnv1aX4Uuid(`${built.id}/${step.name}/context/${cx.id}`);
                 const contextQuery = renderContextQuery(cx.query, scope.rename);
                 cnode.inputs = {
                     // REQUIRED, and the platform's validator is what says so — measured
@@ -5406,7 +5376,7 @@ export function serialize(built, opts = {}) {
                             : tr.kind === 'clientside' ? `clientside.${tr.name}`
                                 : tr.kind === 'httpRequest' ? `httprequest.${tr.name ?? 'default'}`
                                     : tr.key;
-                const toolResourceId = stableUuid(`${built.id}/${step.name}/tool/${toolSeed}`);
+                const toolResourceId = fnv1aX4Uuid(`${built.id}/${step.name}/tool/${toolSeed}`);
                 tnode.inputs = {
                     source: toolResourceId,
                     ...extraInputs,
@@ -5449,7 +5419,7 @@ export function serialize(built, opts = {}) {
                         type: f.type,
                         direction: f.direction,
                     }));
-                    const qsource = stableUuid(`${built.id}/${step.name}/escalation/${esc.name}`);
+                    const qsource = fnv1aX4Uuid(`${built.id}/${step.name}/escalation/${esc.name}`);
                     qnode.inputs = {
                         source: qsource,
                         name: esc.name,
@@ -5485,7 +5455,7 @@ export function serialize(built, opts = {}) {
                         path: `${source}/resources/${qsource}/resource.json`,
                         json: escalationResourceJson(esc, {
                             id: qsource,
-                            channelId: stableUuid(`${built.id}/${step.name}/escalation/${esc.name}/channel`),
+                            channelId: fnv1aX4Uuid(`${built.id}/${step.name}/escalation/${esc.name}/channel`),
                             quickFormSchema: qnode.inputs.schema,
                         }),
                     });
@@ -5497,7 +5467,7 @@ export function serialize(built, opts = {}) {
                 const enode = makeNode(edef.nodeType, eid, esc.name, { parentId, def: edef });
                 enode.display = { ...enode.display, shape: 'circle' };
                 delete enode.model;
-                const esource = stableUuid(`${built.id}/${step.name}/escalation/${esc.name}`);
+                const esource = fnv1aX4Uuid(`${built.id}/${step.name}/escalation/${esc.name}`);
                 enode.inputs = {
                     source: esource,
                     name: esc.name,
@@ -5534,7 +5504,7 @@ export function serialize(built, opts = {}) {
                     path: `${source}/resources/${esource}/resource.json`,
                     json: escalationResourceJson(esc, {
                         id: esource,
-                        channelId: stableUuid(`${built.id}/${step.name}/escalation/${esc.name}/channel`),
+                        channelId: fnv1aX4Uuid(`${built.id}/${step.name}/escalation/${esc.name}/channel`),
                     }),
                 });
             }
@@ -5548,7 +5518,7 @@ export function serialize(built, opts = {}) {
                 mnode.display = { ...mnode.display, shape: 'circle' };
                 delete mnode.model;
                 mnode.inputs = {
-                    source: stableUuid(`${built.id}/${step.name}/memory/${m.id}`),
+                    source: fnv1aX4Uuid(`${built.id}/${step.name}/memory/${m.id}`),
                     name: m.name,
                     description: m.description ?? '',
                     dynamicFewShotLearning: m.dynamicFewShotLearning ?? true,
@@ -5664,7 +5634,7 @@ export function serialize(built, opts = {}) {
             // lets the platform do that translation.
             const s = spec.inputs;
             const id = stepUid(scope, step.name);
-            const source = s.source ?? stableUuid(`${built.id}/${step.name}`);
+            const source = s.source ?? fnv1aX4Uuid(`${built.id}/${step.name}`);
             const node = makeNode(NODE_TYPE.conversationalAgent, id, step.name, { parentId, requestedVersion: step.options?.version });
             delete node.model;
             const st = s.settings ?? {};
@@ -5757,7 +5727,7 @@ export function serialize(built, opts = {}) {
             // definition declares what comes back.
             const s = spec.inputs;
             const id = stepUid(scope, step.name);
-            const source = s.source ?? stableUuid(`${built.id}/${step.name}`);
+            const source = s.source ?? fnv1aX4Uuid(`${built.id}/${step.name}`);
             const node = makeNode(NODE_TYPE.voiceAgent, id, step.name, { parentId, requestedVersion: step.options?.version });
             // Same rule as the inline agent's: the resource model lives on the
             // definition (its `model.source: true` is what makes `source` an input).

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uipath/maestro-builder-sdk",
-  "version": "6.16.6",
+  "version": "6.16.8",
   "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": "36462ef66bdae191ba107334d402dddaed067b2e"
+  "gitref": "77b292a40160cde6e025bbeb534397e5e930bae1"
 }