@uipath/maestro-builder-sdk 6.17.5 → 6.18.0

package/dist/bindings-v2.js

@@ -56,14 +56,12 @@
 import { existsSync, readdirSync, readFileSync } from 'node:fs';
 import { join } from 'node:path';
 import { BindingRefusal, connectionConnectorConflictMessage } from './core/binding-messages.js';
+import { isRecord } from './core/node-envelope.js';
 /** The document version pack emits. */
 const DOCUMENT_VERSION = '2.0';
 /** The per-resource metadata version pack stamps. */
 const BINDINGS_VERSION = '2.2';
-function isRecord(value) {
-    return typeof value === 'object' && value !== null && !Array.isArray(value);
-}
-function str(value) {
+function nonEmptyString(value) {
     return typeof value === 'string' && value.length > 0 ? value : undefined;
 }
 /** `resource` values are spelled lower-case in a `.flow` and capitalised in the BPMN projection. */
@@ -77,7 +75,7 @@ function nodeDetail(node) {
 }
 /** The connection a node runs on: `connectionId`, else `connectionResourceId`. */
 function nodeConnectionKey(detail) {
-    return str(detail.connectionId) ?? str(detail.connectionResourceId);
+    return nonEmptyString(detail.connectionId) ?? nonEmptyString(detail.connectionResourceId);
 }
 /**
  * The connector a node's CONNECTION belongs to — not the connector the node is.
@@ -92,9 +90,9 @@ function nodeConnectionKey(detail) {
  * `detail.connector` is its connection's connector.
  */
 function nodeConnectionConnector(detail) {
-    const connector = str(detail.connector);
+    const connector = nonEmptyString(detail.connector);
     if (connector === MANAGED_HTTP_CONNECTOR && isRecord(detail.bodyParameters)) {
-        return str(detail.bodyParameters.targetConnector) ?? connector;
+        return nonEmptyString(detail.bodyParameters.targetConnector) ?? connector;
     }
     return connector;
 }
@@ -130,8 +128,8 @@ export function connectionConnectors(flow) {
         return out;
     const bound = new Set();
     for (const b of Array.isArray(flow.bindings) ? flow.bindings.filter(isRecord) : []) {
-        const key = str(b.resourceKey);
-        if (key && isConnection(b.resource) && str(b.propertyAttribute)?.toLowerCase() === 'connectionid')
+        const key = nonEmptyString(b.resourceKey);
+        if (key && isConnection(b.resource) && nonEmptyString(b.propertyAttribute)?.toLowerCase() === 'connectionid')
             bound.add(key);
     }
     const users = new Map();
@@ -146,7 +144,7 @@ export function connectionConnectors(flow) {
         const byConnector = users.get(key) ?? new Map();
         users.set(key, byConnector);
         const steps = byConnector.get(connector) ?? [];
-        steps.push((isRecord(node.display) ? str(node.display.label) : undefined) ?? str(node.id) ?? '?');
+        steps.push((isRecord(node.display) ? nonEmptyString(node.display.label) : undefined) ?? nonEmptyString(node.id) ?? '?');
         byConnector.set(connector, steps);
     }
     for (const [key, byConnector] of users) {
@@ -184,10 +182,10 @@ export function connectionResources(flow) {
     // and pack does not carry it into the sidecar.
     const byKey = new Map();
     for (const binding of bindings) {
-        const key = str(binding.resourceKey);
+        const key = nonEmptyString(binding.resourceKey);
         if (!key || !isConnection(binding.resource))
             continue;
-        if (str(binding.propertyAttribute)?.toLowerCase() !== 'connectionid')
+        if (nonEmptyString(binding.propertyAttribute)?.toLowerCase() !== 'connectionid')
             continue;
         byKey.set(key, [...(byKey.get(key) ?? []), binding]);
     }
@@ -214,8 +212,8 @@ export function connectionResources(flow) {
         // maestro flow pack` (uip 1.202.0-dev) on both orders of the issue's repro: pack labels
         // the resource after the earlier step's binding, like `ActivityName`.
         const binding = candidates[0];
-        const displayLabel = str(binding.name) ?? key;
-        const label = isRecord(node.display) ? str(node.display.label) : undefined;
+        const displayLabel = nonEmptyString(binding.name) ?? key;
+        const label = isRecord(node.display) ? nonEmptyString(node.display.label) : undefined;
         resources.push({
             resource: 'connection',
             key,
@@ -223,13 +221,13 @@ export function connectionResources(flow) {
             id: `connection${key}`,
             value: {
                 ConnectionId: {
-                    defaultValue: str(binding.default) ?? key,
+                    defaultValue: nonEmptyString(binding.default) ?? key,
                     isExpression: false,
                     displayName: displayLabel,
                 },
             },
             metadata: {
-                ActivityName: label ?? str(node.id) ?? key,
+                ActivityName: label ?? nonEmptyString(node.id) ?? key,
                 BindingsVersion: BINDINGS_VERSION,
                 DisplayLabel: displayLabel,
                 UseConnectionService: 'true',
@@ -254,7 +252,7 @@ export function mergeBindingsV2(existing, connections) {
         ? existing.resources.filter(isRecord)
         : [];
     const preserved = previous.filter((r) => !isConnection(r.resource));
-    const version = (isRecord(existing) && str(existing.version)) || DOCUMENT_VERSION;
+    const version = (isRecord(existing) && nonEmptyString(existing.version)) || DOCUMENT_VERSION;
     return { version, resources: [...connections, ...preserved] };
 }
 /** Read a `bindings_v2.json`, or `undefined` when absent or unparseable. */

package/dist/bpmn/bpmn-sdk.js

@@ -4,6 +4,7 @@ import { BPMN_SCHEMA_VERSIONS } from './format-profile.js';
 import { schemaVersionRefusal } from '../schema-version.js';
 import { bindingRequiredFields, REGISTRY_GAPS, registryType } from './typed-node.js';
 import { DATA_SERVICE_ACTIVITY_TYPE, DATA_SERVICE_CONNECTOR_KEY, dataServiceWire } from './data-service.js';
+import { ownEntry } from '../core/own-entry.js';
 export { BpmnBuildError } from './bpmn-expr-check.js';
 // @internal helpers kept on the published /bpmn subpath; SDK code imports declared-vars.js.
 export { connectorOutputVars, nodeDeclaredVars, typedNodeDeclaredVars } from './declared-vars.js';
@@ -1218,7 +1219,7 @@ class ScopeBuilder {
                 resource: field.bindingInfo?.resource ?? 'process',
                 propertyAttribute: field.bindingInfo?.propertyAttribute ?? field.name,
                 // A fact the registry omits and the validator enforces — see REGISTRY_GAPS.
-                resourceSubType: REGISTRY_GAPS[type]?.resourceSubType,
+                resourceSubType: ownEntry(REGISTRY_GAPS, type)?.resourceSubType,
                 default: value,
                 resourceKey: value,
             });
@@ -1242,7 +1243,7 @@ class ScopeBuilder {
                 name: opts.process,
                 resource: 'process',
                 propertyAttribute: 'Key',
-                resourceSubType: REGISTRY_GAPS[type]?.resourceSubType,
+                resourceSubType: ownEntry(REGISTRY_GAPS, type)?.resourceSubType,
                 default: releaseKey,
                 resourceKey: releaseKey,
             });

package/dist/bpmn/check.js

@@ -3,7 +3,8 @@ import { allNodes, folderCompanionKeys, implicitNodeVars, processId } from './se
 import { KNOWN_METHOD_SET } from './known-methods.js';
 import { EQUALS_EXPR_DIALECT, checkConnectorContract, checkConnectorLookups, checkConnectorSchema, checkBindingLabels, } from '../core/connector-checks.js';
 import { checkEventResolution, checkEventSubscription, eventParamsFor, } from '../core/event-checks.js';
-import { bindingRequiredFields, missingRequiredContext, missingRequiredInputs, registryType, REGISTRY_GAPS, unsatisfiableRequiredInputs, } from './typed-node.js';
+import { bindingRequiredFields, missingRequiredContext, missingRequiredInputs, REGISTRY_GAPS, registryType, unsatisfiableRequiredInputs, } from './typed-node.js';
+import { ownEntry } from '../core/own-entry.js';
 /**
  * The job-start family — the types whose engine handler switches addressing on the
  * `uipath:type` version: v2 reads `name`/`folderPath`, v1 reads `releaseKey`/`folderKey`.
@@ -1089,7 +1090,7 @@ inEventSubProcess = false) {
             }
             // A type that permits only the output row its spec declares. Reachable via
             // the generic `.activity()`; the typed wrapper does not offer the option.
-            if (REGISTRY_GAPS[n.type]?.outputRowsFixed && Object.keys(n.outputs ?? {}).length) {
+            if (ownEntry(REGISTRY_GAPS, n.type)?.outputRowsFixed && Object.keys(n.outputs ?? {}).length) {
                 diags.push({
                     level: 'error',
                     code: 'OUTPUT_ROWS_FIXED',