@uipath/maestro-builder-sdk 6.8.6 → 6.9.1

package/dist/serialize.js

@@ -14,7 +14,7 @@ import { Expr, toExpr, SCHEDULE_PRESETS, DELAY_PRESETS, parseIxpProjectType, fla
 import { FlowAction, FlowTrigger } from './core/node-classes.js';
 import { connectorRawNodeRefusal } from './core/connector-raw-node.js';
 import { dataFabricUnknownInputs } from './core/typed-raw-node.js';
-import { bindingSelfNameMessage } from './core/binding-messages.js';
+import { BindingRefusal, GUID_RE, bindingSelfNameMessage, bindingUndeclaredMessage, undeclaredBinding } from './core/binding-messages.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 { buildConfiguration, transportHttpMethod } from './config.js';
@@ -3064,8 +3064,6 @@ function renderValue(val, rename = NO_RENAME) {
         return val.literal ? val.value : '=js:' + renameRefs(val.js, rename);
     return val;
 }
-/** A tenant resource key — an Integration Service connection id or an Orchestrator folder key — is a GUID. */
-const GUID_RE = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
 /**
  * The all-zero prefix authors reach for when they need "a GUID" and have none:
  * `00000000-0000-0000-0000-000000000001`. No tenant issues these. The same
@@ -3087,6 +3085,12 @@ function connectionStubReason(value) {
         return 'not a GUID';
     return undefined;
 }
+/** What `registry prepare` takes for an event subscription: its connector key and curated event name. */
+function eventTarget(sub) {
+    return typeof sub.connector === 'string' && sub.connector !== ''
+        ? { key: sub.connector, ...(typeof sub.event === 'string' && sub.event !== '' ? { action: sub.event } : {}) }
+        : undefined;
+}
 /**
  * Resolve a symbolic connection/folder name to the resource key that is inlined
  * as `connectionId` / `connectionFolderKey`.
@@ -3100,30 +3104,55 @@ function connectionStubReason(value) {
  * `bindings.json` that never declares the name, and one that declares it with
  * its own name as the `resourceKey`.
  *
- * Only the SELF-REFERENTIAL case is an error here. Emitting the symbolic name
- * when nothing resolves it is long-established and deliberate — compiling
- * against partial or absent bindings is how a flow is authored offline, and the
- * name is the only honest placeholder available. A binding that declares the
- * name and then answers with the name is different: someone wrote the entry, so
- * the intent to bind is explicit, and the one thing it cannot mean is "the
- * resource key is the string `is-sandboxes`".
+ * Two cases are errors here, and they are the two where the author's intent to
+ * bind is on record and the emitted value still cannot be a resource key:
+ *
+ * - SELF-REFERENTIAL (`BINDING_SELF_NAME`): an entry declares the name and then
+ *   answers with the name. Someone wrote the entry, and the one thing it cannot
+ *   mean is "the resource key is the string `is-sandboxes`".
+ * - UNDECLARED IN A POPULATED FILE (`BINDING_UNDECLARED`): bindings.json declares
+ *   entries, none of them answers to this label, and the label is not itself a
+ *   GUID. The file is the author's statement of what is bound, so a label it
+ *   does not carry is a mismatch between two names for one connection, not
+ *   offline authoring. Measured: the 2026-09-23 orchestrator-paths run bound
+ *   `slack` (prepare's default label) while the source said `is-sandboxes`;
+ *   compile warned, exited 0, and the Slack node answered `400 'Connection' has
+ *   an invalid GUID value: 'is-sandboxes'`. Across the eval archive this shape
+ *   warned 15 times; 13 were fixed before the final compile and the 2 that were
+ *   not are that row's failures (09-18, 09-23) — no green run shipped it.
  *
  * Everything else that cannot be a resource key is a WARNING (`CONNECTION_STUB`),
- * never an error: an unbound symbol, an all-zero GUID, an `<angle>` placeholder,
- * a label. The archive shows those in green runs of simulated and validate-only
- * tasks, where no connection exists to bind — refusing them would push an author
- * with nothing real to bind into inventing a plausible GUID instead, which is
- * worse because nothing can tell. But the 2026-09-01 orchestrator-paths run
- * compiled `00000000-0000-0000-0000-000000000001` with no signal at all and the
- * Slack node answered `401 Invalid Organization or User secret` at dispatch;
- * the author deserves to hear that BEFORE the run.
+ * never an error: a symbol when NO bindings.json is loaded or it declares no
+ * entries, an entry declared without a resourceKey, an all-zero GUID, an
+ * `<angle>` placeholder, a label. Compiling against absent bindings is how a flow
+ * is authored offline, and the name is the only honest placeholder available;
+ * the archive shows those in green runs of simulated and validate-only tasks,
+ * where no connection exists to bind. Refusing them would push an author with
+ * nothing real to bind into inventing a plausible GUID instead, which is worse
+ * because nothing can tell. That is also the route for a connector that is not
+ * bound YET in a file that binds others (a two-connector flow with one prepared,
+ * or a simulated connector next to real ones): declare its label with a
+ * placeholder resourceKey and it stays a warning. Mid-loop compiles took that
+ * state in three archived runs (ipe-drive-to-slack 09-23 `drive`, ipe-jira-
+ * lifecycle 09-04 `entra`, customer-escalation-simulated 09-03 `zendesk`), and
+ * each fixed it before the final compile, so what now costs one refused compile
+ * there was never shipped. The 2026-09-01 orchestrator-paths run compiled
+ * `00000000-0000-0000-0000-000000000001` with no signal at all and the Slack node
+ * answered `401 Invalid Organization or User secret` at dispatch; the author
+ * deserves to hear that BEFORE the run.
  */
-function resolveBinding(opts, name, attribute, where) {
+function resolveBinding(opts, name, attribute, where, target) {
     if (name === undefined)
         return undefined;
     const resolved = opts.bindings?.resolve(name);
     if (resolved === name) {
-        throw new Error(bindingSelfNameMessage(where, name, attribute));
+        throw new BindingRefusal('BINDING_SELF_NAME', bindingSelfNameMessage(where, name, attribute, target));
+    }
+    if (resolved === undefined) {
+        const declared = undeclaredBinding(opts.bindings, name);
+        if (declared) {
+            throw new BindingRefusal('BINDING_UNDECLARED', bindingUndeclaredMessage(where, name, attribute, declared, target));
+        }
     }
     const value = resolved ?? name;
     const reason = connectionStubReason(value);
@@ -3136,8 +3165,10 @@ function resolveBinding(opts, name, attribute, where) {
             + `and writes both entries into bindings.json (several candidates: pass \`--connection-id <id>\` from the list it `
             + `prints); by hand, read Id and FolderKey from \`uip is connections list --all-folders --output json\`.`;
         const message = resolved === undefined
-            ? `${where}: ${attribute} "${name}" has no bindings.json entry`
-                + `${describeDeclared(opts.bindings)}, so the .flow carries the symbolic name as its ${key} and the run `
+            ? `${where}: ${declaredWithoutKey(opts.bindings, name)
+                ? `binding "${name}" is declared in bindings.json with no resourceKey`
+                : `${attribute} "${name}" has no bindings.json entry${describeDeclared(opts.bindings)}`}`
+                + `, so the .flow carries the symbolic name as its ${key} and the run `
                 + `faults at dispatch with ${fault}. ${remedy}`
             : `${where}: binding "${name}" resolves to "${value}", which is ${reason} and so never a tenant ${key}. `
                 + `The .flow carries it and the run faults at dispatch with ${fault}. ${remedy}`;
@@ -3145,6 +3176,10 @@ function resolveBinding(opts, name, attribute, where) {
     }
     return value;
 }
+/** An entry answers to `name` but carries no resourceKey or default — the placeholder route, still a warning. */
+function declaredWithoutKey(bindings, name) {
+    return (bindings?.declared?.() ?? []).includes(name) || bindings?.entry?.(name) !== undefined;
+}
 /** " (bindings.json declares: a, b)" / " (no bindings.json was loaded)" — what the author CAN refer to. */
 function describeDeclared(bindings) {
     const names = bindings?.declared?.() ?? [];
@@ -3282,8 +3317,8 @@ function flattenWhere(where, declared) {
     return out;
 }
 function eventDetail(rc, sub, nodeId, opts) {
-    const conn = resolveBinding(opts, sub.connection, 'connection', `event node "${nodeId}"`);
-    const folder = resolveBinding(opts, sub.folder, 'folder', `event node "${nodeId}"`);
+    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 ?? {});
     const filters = sub.filters ?? [];
     // A GENERIC event (`activityType: GenericTrigger` — Data Fabric, Salesforce,
@@ -4106,8 +4141,8 @@ export function serialize(built, opts = {}) {
         const { entries } = connectionResourceBindings({
             def: eventTriggerRc.v1def,
             connectorKey: eventTriggerRc.connectorKey,
-            connectionId: resolveBinding(opts, sub.connection, 'connection', 'event trigger'),
-            folderKey: resolveBinding(opts, sub.folder, 'folder', 'event trigger'),
+            connectionId: resolveBinding(opts, sub.connection, 'connection', 'event trigger', eventTarget(sub)),
+            folderKey: resolveBinding(opts, sub.folder, 'folder', 'event trigger', eventTarget(sub)),
         }, resourceBindings);
         for (const b of entries)
             resourceBindings.set(b.id, b);
@@ -4231,8 +4266,8 @@ 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);
-            const conn = resolveBinding(opts, spec.connection, 'connection', `connector "${step.name}"`);
-            const folder = resolveBinding(opts, spec.folder, 'folder', `connector "${step.name}"`);
+            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[]`,
             // never through `inputs.detail` — without this pair the run faults with
             // `[102010]` (board G-19; see connectionResourceBindings).
@@ -4396,7 +4431,7 @@ export function serialize(built, opts = {}) {
                 ...(s.guardrails === undefined ? {} : { guardrails: s.guardrails }),
                 // The two DESCRIPTOR ARRAYS. `agentInputVariables` entries carry the
                 // binding expression; `agentOutputVariables` entries are name + type only,
-                // and a runner coerces the model's completion to exactly these keys.
+                // and the runtime's structured final call fills exactly these keys.
                 //
                 // `id` is the FLAT NAME, not the author's — the platform re-derives this
                 // array from the node's prompts and keeps a pre-existing entry only when
@@ -5131,8 +5166,8 @@ export function serialize(built, opts = {}) {
             const { entries: waitConnBindings } = connectionResourceBindings({
                 def: rc.v1def,
                 connectorKey: rc.connectorKey,
-                connectionId: resolveBinding(opts, sub.connection, 'connection', `waitForEvent "${step.name}"`),
-                folderKey: resolveBinding(opts, sub.folder, 'folder', `waitForEvent "${step.name}"`),
+                connectionId: resolveBinding(opts, sub.connection, 'connection', `waitForEvent "${step.name}"`, eventTarget(sub)),
+                folderKey: resolveBinding(opts, sub.folder, 'folder', `waitForEvent "${step.name}"`, eventTarget(sub)),
             }, resourceBindings);
             for (const b of waitConnBindings)
                 resourceBindings.set(b.id, b);
@@ -5986,8 +6021,8 @@ export function serialize(built, opts = {}) {
                 const { entries } = connectionResourceBindings({
                     def: rc.v1def,
                     connectorKey: rc.connectorKey,
-                    connectionId: resolveBinding(opts, sub.connection, 'connection', 'onEvent() entry point'),
-                    folderKey: resolveBinding(opts, sub.folder, 'folder', 'onEvent() entry point'),
+                    connectionId: resolveBinding(opts, sub.connection, 'connection', 'onEvent() entry point', eventTarget(sub)),
+                    folderKey: resolveBinding(opts, sub.folder, 'folder', 'onEvent() entry point', eventTarget(sub)),
                 }, resourceBindings);
                 for (const b of entries)
                     resourceBindings.set(b.id, b);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uipath/maestro-builder-sdk",
-  "version": "6.8.6",
+  "version": "6.9.1",
   "description": "Build UiPath Flow, Case, and BPMN artifacts by writing TypeScript.",
   "license": "MIT",
   "homepage": "https://docs.uipath.com/maestro",
@@ -87,5 +87,5 @@
     "@types/node": "^22.7.0",
     "esbuild": "^0.28.1"
   },
-  "gitref": "503ff8140144536c938e0b0ad0356b6ccb41b9f9"
+  "gitref": "f637225ed0a30014831ffab8733374625126f6fb"
 }