@uipath/maestro-builder-sdk 6.51.0 → 6.51.2

package/dist/case/check.js

@@ -9,6 +9,17 @@ import { checkEventResolution } from '../core/event-checks.js';
 import { EXTERNAL_TASK_DESCRIPTOR_CONTRACT, EXTERNAL_TASK_HYBRID_UNSUPPORTED, externalHybridRefusal, isExternalExecutionMode } from './external.js';
 import { isResourceLabelRefusal, resolveResourceRef } from './resource-identity.js';
 const TYPE_NAMES = new Set(Object.values(types));
+/**
+ * Every key an `ActionField` takes. `compile` writes only these onto the form
+ * row, so any other key would be dropped without a word (builder-sdk#1422): an
+ * author who adds the Designer's row keys (`id`, `var`, `source`) sees `check`
+ * and `compile` pass and the row come out unchanged. The `satisfies` keeps this
+ * list and the interface in step: a key added to one and not the other fails to
+ * type-check.
+ */
+const ACTION_FIELD_KEYS = new Set(Object.keys({
+    name: true, type: true, subType: true, displayName: true, value: true, required: true, options: true,
+}));
 // audit-case-coverage: rule-slot table start
 // This runtime table must equal the committed matrix's rejected cells. The audit
 // parses these object literals; the product-pin matrix currently rejects none.
@@ -628,6 +639,24 @@ function checkStage(stage, diags, refStage, refPlacement, bindings) {
                 message: `Action field "${emptyRequiredField.name}" in task "${t.name}" is required but has no value. Set its value or remove required: true; required does not mean reviewer-must-fill.`,
                 where: t.name,
             });
+        for (const [side, fields] of [['input', t.action?.inputs], ['output', t.action?.outputs]])
+            for (const field of fields ?? []) {
+                const unknown = Object.keys(field)
+                    .filter((key) => !ACTION_FIELD_KEYS.has(key) && field[key] !== undefined);
+                if (!unknown.length)
+                    continue;
+                diags.push({
+                    level: 'error',
+                    code: 'ACTION_FIELD_UNKNOWN_KEY',
+                    message: `Action ${side} field "${String(field.name)}" in task "${t.name}" has ${unknown.length === 1 ? 'a key' : 'keys'} ` +
+                        `${unknown.map((key) => `"${key}"`).join(', ')} that an action field does not take, so compile would drop ${unknown.length === 1 ? 'it' : 'them'}. ` +
+                        `An action field takes ${[...ACTION_FIELD_KEYS].join(', ')}. ` +
+                        (side === 'output'
+                            ? `To choose the case variable an output writes, set value: '<variable>'; compile writes the row's id, var, source and target from it.`
+                            : `An input's value is the literal or =vars.<name> expression the assignee sees.`),
+                    where: t.name,
+                });
+            }
         if (t.entry.length === 0)
             diags.push({
                 level: 'error',

package/dist/case/serialize.js

@@ -989,11 +989,15 @@ function taskData(task, resolver, taskId, stageId, ids) {
             }
             return data;
         }
+        // The io rows carry the task's own `elementId` ("<stageId>-<taskId>"), as
+        // every other kind's do. They once got the bare task id, so the converter's
+        // `task.elementId === output.elementId` filter matched none of them and the
+        // task's `response` / `Error` never reached a later task (#1419).
         case 'connector':
-            return connectorTaskData(task.connector, resolver, taskId, `Connector task "${task.name}"`);
+            return connectorTaskData(task.connector, resolver, `${stageId}-${taskId}`, `Connector task "${task.name}"`);
         case 'external-agent':
         case 'external-workflow':
-            return externalTaskData(task.kind, task.external, resolver, taskId, `${task.kind === 'external-agent' ? 'External Agent' : 'External Workflow'} task "${task.name}"`);
+            return externalTaskData(task.kind, task.external, resolver, `${stageId}-${taskId}`, `${task.kind === 'external-agent' ? 'External Agent' : 'External Workflow'} task "${task.name}"`);
         case 'wait-for-timer': {
             if (task.timer?.duration)
                 return { timerType: 'timeDuration', timeDuration: task.timer.duration };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uipath/maestro-builder-sdk",
-  "version": "6.51.0",
+  "version": "6.51.2",
   "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": "5647b4660231b68f1a5c44badd9edc030e36729a"
+  "gitref": "19785af943dff491f68e984f2440a196f95bc2c6"
 }