@uipath/maestro-builder-sdk 6.31.8 → 6.32.0

package/dist/nodes/hitl/flow/check.js

@@ -1,6 +1,6 @@
 import { Expr } from '../../../core/expr.js';
 import { nearestName } from '../../../core/edit-distance.js';
-import { FORM_FIELD_TYPES, formFieldTypeHint, formFieldTypeWhy, validFormFieldTypesText } from '../../../core/form-field-type.js';
+import { FORM_FIELD_TYPES, formFieldOptionsProblem, formFieldOptionsWhy, formFieldTypeHint, formFieldTypeWhy, validFormFieldTypesText } from '../../../core/form-field-type.js';
 import { isAnswerField } from '../../../core/hitl-answers.js';
 import { hitlDerivesOutcomeHandles, outcomeIdOf } from '../../../core/hitl-routing.js';
 export const HITL_DIAGNOSTICS = {
@@ -23,6 +23,8 @@ export const HITL_DIAGNOSTICS = {
     HITL_DUP_FIELD: { severity: 'error', summary: 'a form field id declared twice' },
     HITL_FIELD_RESERVED_ID: { severity: 'error', summary: 'a form field id the result already uses' },
     HITL_FIELD_TYPE_UNKNOWN: { severity: 'error', summary: 'a form field type the platform does not recognize' },
+    HITL_FIELD_OPTIONS: { severity: 'error', summary: 'a picker with no choices, or choices that are not { id, value, label } strings' },
+    HITL_FIELD_OPTIONS_IGNORED: { severity: 'warning', summary: 'choices on a field type that does not render them' },
     HITL_FIELD_NO_VALUE: { severity: 'error', summary: 'a shown field with no value' },
     HITL_FIELD_VALUE_NOT_REF: { severity: 'error', summary: 'a shown field filled from a literal' },
     HITL_FIELD_VALUE_NOT_PATH: { severity: 'error', summary: 'a shown field filled from a computed expression' },
@@ -215,6 +217,10 @@ export function checkHitl(inputs, ctx) {
             at('HITL_FIELD_TYPE_UNKNOWN', `Human task "${step}" gives the form field "${id}" the type ${JSON.stringify(f?.type)}, `
                 + `${formFieldTypeWhy(f?.type)}. Valid: ${validFormFieldTypesText()}.`, formFieldTypeHint(f?.type));
         }
+        const optionsProblem = formFieldOptionsProblem(f?.type, f.options);
+        if (optionsProblem !== undefined) {
+            at(optionsProblem === 'ignored' ? 'HITL_FIELD_OPTIONS_IGNORED' : 'HITL_FIELD_OPTIONS', `Human task "${step}"'s form field "${id}" ${formFieldOptionsWhy(optionsProblem, f?.type)}.`, `options: [{ id: 'opt-1', value: 'high', label: 'High' }, { id: 'opt-2', value: 'low', label: 'Low' }]`);
+        }
         if (isAnswerField(f))
             answers++; // an `output`, or an `inOut` whose reviewed value comes back
         if (f?.direction === 'input' || f?.direction === 'inOut') {

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

@@ -13,7 +13,7 @@
  * every edge out of an outcome-handle node to.
  */
 import { Expr, toExpr } from '../../../core/expr.js';
-import { formFieldJsonSchema, sourceFormFieldType } from '../../../core/form-field-type.js';
+import { formFieldJsonSchema, formFieldOptionsWire, liftFormFieldOptions, sourceFormFieldType } from '../../../core/form-field-type.js';
 import { isAnswerField } from '../../../core/hitl-answers.js';
 import { OUTCOME_HANDLE_NODE_TYPES, hitlExitModel, outcomeIdOf, outcomeSlug } from '../../../core/hitl-routing.js';
 import { jsonClone } from '../../../core/json-clone.js';
@@ -86,6 +86,11 @@ function hitlInputs(inputs, nodeType, ctx) {
             if (f.required)
                 base.required = true;
         }
+        // A picker's choices (a `singleSelect` dropdown, a `multiSelect`), as the
+        // canvas stores them on the row (#1235).
+        const options = formFieldOptionsWire(f.options);
+        if (options !== undefined)
+            base.options = options;
         return base;
     });
     const outcomes = (inputs?.outcomes ?? []).map((o, i) => {
@@ -326,6 +331,7 @@ function liftHitl(node, typeName, ctx) {
         const id = String(f.id ?? '');
         const direction = String(f.direction ?? 'output');
         const value = direction === 'input' || direction === 'inOut' ? bindingExpr(f.binding) : undefined;
+        const options = liftFormFieldOptions(f.options);
         return {
             id,
             // serialize defaults a row's label to its id.
@@ -336,6 +342,7 @@ function liftHitl(node, typeName, ctx) {
             direction,
             ...(value !== undefined ? { value } : {}),
             ...(f.required === true ? { required: true } : {}),
+            ...(options !== undefined ? { options } : {}),
         };
     });
     const outcomeRows = (Array.isArray(schema.outcomes) ? schema.outcomes : []);
@@ -498,6 +505,11 @@ function printHitl({ inputs }, ctx) {
         ['direction', ctx.string(f.direction)],
         ['value', f.value !== undefined ? ctx.value(f.value) : undefined],
         ['required', f.required ? 'true' : undefined],
+        ['options', f.options !== undefined ? `[${f.options.map((o) => obj([
+                ['id', ctx.string(o.id)],
+                ['value', ctx.string(o.value)],
+                ['label', ctx.string(o.label)],
+            ], ctx)).join(', ')}]` : undefined],
     ], ctx));
     const outcomes = h.outcomes.map((o) => (typeof o === 'string' ? ctx.string(o) : obj([
         ['name', ctx.string(o.name)],

package/dist/serialize.js

@@ -20,7 +20,7 @@ import { FLOW_NODE_TYPES } from './flow-node-registry.js';
 import { stepExits } from './flow-step-exits.js';
 import { resolveVersionCarried } from './core/nodes/versioned.js';
 import { indexCarriedDefinitions } from './core/carried-definitions.js';
-import { formFieldJsonSchema } from './core/form-field-type.js';
+import { formFieldJsonSchema, formFieldOptionsWire } from './core/form-field-type.js';
 import { ConnectorResolveError } from './core/connector-resolve-error.js';
 import { connectionConnectors } from './bindings-v2.js';
 import { BindingRefusal, GUID_RE, bindingLabelRefusal, bindingMissingMessage, missingBindingLabels } from './core/binding-messages.js';
@@ -3862,6 +3862,7 @@ function newScope(outputs, vars = [], backing = new Map(), backingReads = backin
         stepNodeId: new Map(),
         pendingEdges: [],
         openTails: false,
+        endNodeId: new Map(),
     };
 }
 /**
@@ -3905,6 +3906,7 @@ function newScope(outputs, vars = [], backing = new Map(), backingReads = backin
  */
 /**
  * Wire every parked `.stepToRef('<step>')` edge, now that the whole scope exists.
+ * The target is a step, or else a named End (`.return({}, { name })`, #1223).
  *
  * An unresolved target should be impossible — `check`'s REF_UNKNOWN_TARGET
  * runs first and names the steps that do exist — but a serializer that silently
@@ -3914,10 +3916,12 @@ function newScope(outputs, vars = [], backing = new Map(), backingReads = backin
  */
 function resolveRejoins(scope, addEdge) {
     for (const { from, target } of scope.pendingEdges) {
-        const id = scope.stepNodeId.get(target);
+        const id = scope.stepNodeId.get(target) ?? scope.endNodeId.get(target);
         if (!id) {
+            const ends = [...scope.endNodeId.keys()];
             throw new Error(`.stepToRef('${target}') names a step this flow does not have. Known steps: `
-                + `${[...scope.stepNodeId.keys()].join(', ') || '(none)'}.`);
+                + `${[...scope.stepNodeId.keys()].join(', ') || '(none)'}`
+                + `${ends.length ? `; named Ends: ${ends.join(', ')}` : ''}.`);
         }
         // Through the serializer's own `addEdge`, so the id comes from the one
         // file-wide counter. A per-scope count (`edge_ref_${scope.edges.length}`)
@@ -5533,12 +5537,17 @@ export function serialize(built, opts = {}) {
                     const qnode = makeNode(qdef.nodeType, qid, esc.name, { parentId, def: qdef });
                     qnode.display = { ...qnode.display, shape: 'circle' };
                     delete qnode.model;
-                    const rows = (esc.fields ?? []).map((f) => ({
-                        id: f.id,
-                        label: f.label ?? f.id,
-                        type: f.type,
-                        direction: f.direction,
-                    }));
+                    const rows = (esc.fields ?? []).map((f) => {
+                        const options = formFieldOptionsWire(f.options);
+                        return {
+                            id: f.id,
+                            label: f.label ?? f.id,
+                            type: f.type,
+                            direction: f.direction,
+                            // A picker's choices, as the human task's rows carry them (#1235).
+                            ...(options !== undefined ? { options } : {}),
+                        };
+                    });
                     const qsource = fnv1aX4Uuid(`${built.id}/${step.name}/escalation/${esc.name}`);
                     qnode.inputs = {
                         source: qsource,
@@ -6500,6 +6509,11 @@ export function serialize(built, opts = {}) {
                         scope.variableUpdates[id] = updates;
                     if (name !== undefined && !prior) {
                         (namedEnds.get(scope) ?? namedEnds.set(scope, new Map()).get(scope)).set(name, { id, sig });
+                        // A NAMED End can be a `.stepToRef` target (#1223); an unnamed one
+                        // cannot, since its id (`end`, `end2`, …) depends on emission order.
+                        // First wins, like `stepUid`: two Ends sharing a name with different
+                        // content are `check`'s REF_AMBIGUOUS_TARGET when a ref names them.
+                        scope.endNodeId.set(name, id);
                     }
                 }
                 tails = [];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uipath/maestro-builder-sdk",
-  "version": "6.31.8",
+  "version": "6.32.0",
   "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": "294a7da19cbb9eaad2ae8b97b35fdc07923d50ac"
+  "gitref": "83ae5bb02e8784f1a445c0377a09daae9ffc5b26"
 }