@uipath/maestro-builder-sdk 6.44.2 → 6.44.3

package/dist/check.js

@@ -22,6 +22,7 @@ import { FLOW_SCHEMA_VERSIONS } from './format-profile.js';
 import { compareSchemaVersions, schemaVersionRefusal } from './schema-version.js';
 import { planFlowGlobals, planSharedInputs, sharedInputRoots } from './flow-variables.js';
 import { planOutputBacking } from './flow-output-backing.js';
+import { VARS_READ_RE } from './core/vars-read.js';
 import { terminateStepClashMessage, terminateTwiceMessage } from './flow-terminate-names.js';
 import { endContentKey } from './flow-end-content.js';
 import { FLOW_NODE_TYPES } from './flow-node-registry.js';
@@ -1056,7 +1057,7 @@ function checkFlow(built, ancestors, opts = {}, ledger = newResourceIdLedger())
         // subflow read, where the value lives one level deeper
         // (`$vars.<call>.output.<childOutput>`). Every other rule reads only the
         // first two segments, exactly as before.
-        for (const m of js.matchAll(/\$vars\.([A-Za-z0-9_]+)(?:\??\.([A-Za-z0-9_]+))?((?:\??\.[A-Za-z0-9_]+)*)/g)) {
+        for (const m of js.matchAll(VARS_READ_RE)) {
             const [, name, field] = m;
             // `?.` reads the same path as `.`, and the rules below parse the tail as `.a.b`.
             const rest = m[3].replace(/\?\./g, '.');

package/dist/core/vars-read.d.ts

@@ -0,0 +1,6 @@
+/**
+ * A dotted `$vars` read: `$vars.<name>`, an optional `.<field>`, then the dotted
+ * `rest`, each dot optionally `?.`. It stops at the first bracket. Check and
+ * decompile both scan with it, so decompile declares the body fields check reads.
+ */
+export declare const VARS_READ_RE: RegExp;

package/dist/core/vars-read.js

@@ -0,0 +1,6 @@
+/**
+ * A dotted `$vars` read: `$vars.<name>`, an optional `.<field>`, then the dotted
+ * `rest`, each dot optionally `?.`. It stops at the first bracket. Check and
+ * decompile both scan with it, so decompile declares the body fields check reads.
+ */
+export const VARS_READ_RE = /\$vars\.([A-Za-z0-9_]+)(?:\??\.([A-Za-z0-9_]+))?((?:\??\.[A-Za-z0-9_]+)*)/g;

package/dist/decompile.js

@@ -42,6 +42,7 @@ import { parseIxpProjectType } from './core/actions.js';
 import { readFlowGlobals } from './flow-variables.js';
 import { outputBackingGlobal, outputBackingName } from './flow-output-backing.js';
 import { mapJsCode } from './core/js-code-spans.js';
+import { VARS_READ_RE } from './core/vars-read.js';
 import { missingBindingLabels } from './core/binding-messages.js';
 import { prepareCommand } from './core/cli-spelling.js';
 import { OUTCOME_HANDLE_NODE_TYPES } from './core/hitl-routing.js';
@@ -459,6 +460,30 @@ function emitValMap(m, imp, inputNames) {
  */
 const ARTIFACT_PORTS = new Set(['tool', 'context', 'escalation', 'memory']);
 // ─── Graph model ──────────────────────────────────────────────────────────────
+/** An index right after a read: `[0]`, `[i]`, `?.["k"]`. */
+const INDEX_NEXT_RE = /^\s*(?:\?\.)?\[/;
+/** A read's tail when an index follows it, whatever the index: `.output.body.items[]`. */
+const INDEX_MARK = '[]';
+/** Every `$vars` read in `values`' strings, by id, with `?.` read as `.`. */
+function scopeReads(values) {
+    const reads = new Map();
+    const visit = (v) => {
+        if (typeof v === 'string') {
+            for (const m of v.matchAll(VARS_READ_RE)) {
+                const [read, id, field, rest] = m;
+                const path = field === undefined ? '' : `.${field}${rest.replace(/\?\./g, '.')}`;
+                const indexed = INDEX_NEXT_RE.test(v.slice(m.index + read.length));
+                (reads.get(id) ?? reads.set(id, []).get(id)).push(indexed ? path + INDEX_MARK : path);
+            }
+            return;
+        }
+        if (v && typeof v === 'object')
+            for (const x of Object.values(v))
+                visit(x);
+    };
+    visit(values);
+    return reads;
+}
 class Graph {
     byId = new Map();
     out = new Map();
@@ -474,8 +499,11 @@ class Graph {
      */
     accounted = new Set();
     edges;
-    constructor(nodes, edges) {
+    variables;
+    reads;
+    constructor(nodes, edges, variables) {
         this.edges = edges;
+        this.variables = variables;
         for (const n of nodes)
             this.byId.set(n.id, n);
         for (const e of edges) {
@@ -483,6 +511,11 @@ class Graph {
             (this.inc.get(e.targetNodeId) ?? this.inc.set(e.targetNodeId, []).get(e.targetNodeId)).push(e);
         }
     }
+    /** The tails this scope's inputs, outputs and updates read off `$vars.<id>`, e.g. `.output.body.current`. */
+    readsOf(id) {
+        this.reads ??= scopeReads([...[...this.byId.values()].flatMap((n) => [n.inputs, n.outputs]), this.variables?.variableUpdates]);
+        return this.reads.get(id) ?? [];
+    }
     outEdges(id) {
         return this.out.get(id) ?? [];
     }
@@ -666,6 +699,7 @@ function liftRegistered(node, impl = registeredImpl(node), graph) {
         absorb: (id, compiledId) => { runAbsorbed.set(id, compiledId); },
         outPorts: () => (graph?.outEdges(node.id) ?? []).map((e) => e.sourcePort),
         convergedFanOut: () => graph?.convergedFanOut(node.id) === true,
+        reads: () => graph?.readsOf(node.id) ?? [],
     };
     const spec = impl.deserialize(node, ctx);
     return spec === undefined ? undefined : { impl, spec };
@@ -2556,7 +2590,7 @@ function readChildStartAsStart(child, startId) {
 function emitChildFlowIn(name, child, trigger, ctx) {
     const globals = readFlowGlobals(child.variables, runStrict);
     const childCtx = {
-        graph: new Graph(child.nodes ?? [], child.edges ?? []),
+        graph: new Graph(child.nodes ?? [], child.edges ?? [], child.variables),
         flow: child,
         imp: ctx.imp,
         inputNames: new Set(globals.filter((g) => g.direction === 'in').map((g) => g.name ?? g.id)),
@@ -2721,7 +2755,7 @@ export function decompile(flow, options = {}) {
     }
     runCarried = carriedPairs(flow);
     const imp = new Imports();
-    const graph = new Graph(flow.nodes ?? [], flow.edges ?? []);
+    const graph = new Graph(flow.nodes ?? [], flow.edges ?? [], flow.variables);
     const globals = readFlowGlobals(flow.variables, runStrict);
     const inputNames = new Set(globals.filter((g) => g.direction === 'in').map((g) => g.name ?? g.id));
     // Trigger-bound globals name the node they came from; fall back to the graph's

package/dist/flow-node-type.d.ts

@@ -281,6 +281,8 @@ export interface FlowDeserializeContext {
      * the type's `exits()`, so it is `false` for any other node.
      */
     convergedFanOut(): boolean;
+    /** The tails the node's scope reads off `$vars.<nodeId>`, e.g. `.output.body.current`, `.output.body.items[]` for an indexed read. */
+    reads(): readonly string[];
 }
 /** What `print` may ask of the decompiler. Every method returns source text. */
 export interface FlowPrintContext {

package/dist/nodes/http/flow/node-type.js

@@ -245,6 +245,27 @@ function liftReturns(node) {
         .map(([k, v]) => [k, v.type]);
     return entries.length ? Object.fromEntries(entries) : undefined;
 }
+/** A read of a body field: the field, then whatever the read goes on to. */
+const BODY_READ_RE = /^\.output\.body\.([A-Za-z0-9_]+)(.*)$/;
+/**
+ * The `returns` a node with no body schema needs for the body fields its flow
+ * already reads, so the source decompile prints compiles. A field read further
+ * in (`body.current.time`) is an `object`; a field read whole is a `string`, the
+ * type the check suggests. Undefined when nothing reads the body.
+ */
+function returnsFromReads(reads) {
+    const fields = new Map();
+    for (const read of reads) {
+        const m = BODY_READ_RE.exec(read);
+        if (!m)
+            continue;
+        if (m[2] !== '')
+            fields.set(m[1], 'object');
+        else if (!fields.has(m[1]))
+            fields.set(m[1], 'string');
+    }
+    return fields.size ? Object.fromEntries(fields) : undefined;
+}
 /**
  * Invert an HTTP node. The plain node keeps its request at the top level
  * (`url` / `method` / `headers` / `queryParams` / `body` / `contentType`); the
@@ -275,7 +296,7 @@ function liftHttp(node, ctx, managed) {
         throw new Error(`decompile: expected a number, got ${JSON.stringify(inputs.retryCount)}.`);
     }
     const branchRows = Array.isArray(inputs.branches) ? inputs.branches : [];
-    const returns = liftReturns(node);
+    const returns = liftReturns(node) ?? returnsFromReads(ctx.reads());
     const spec = {
         url: value(req.url ?? ''),
         ...(text(req.method) !== undefined && req.method !== 'GET' ? { method: req.method } : {}),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uipath/maestro-builder-sdk",
-  "version": "6.44.2",
+  "version": "6.44.3",
   "description": "Build UiPath Flow, Case, and BPMN artifacts by writing TypeScript.",
   "license": "MIT",
   "homepage": "https://docs.uipath.com/maestro",
@@ -93,5 +93,5 @@
     "@types/node": "^22.7.0",
     "esbuild": "^0.28.1"
   },
-  "gitref": "7afed05cfe5a6856deb6fe7984a5cf2b31a2dd20"
+  "gitref": "89512b0912180900bd6b3c62191b7beb090af3cc"
 }