@uipath/maestro-builder-sdk 6.14.4 → 6.15.0

package/dist/registry/prepare/source-labels.js

@@ -46,7 +46,15 @@ import { GUID_RE } from "../../core/binding-messages.js";
 import { relativePosix } from "../compat/paths.js";
 import { canonicalPath, flowSources, isRelativeSpecifier } from "./source-import.js";
 import { inWorkDir } from "../../workdir.js";
-const CALLEES = new Set(["connector", "onEvent", "waitForEvent"]);
+const CALLEES = new Set([
+    "connector", "onEvent", "waitForEvent", "eventTrigger", "dataService",
+]);
+/**
+ * Data Fabric's connector key, spelled here rather than imported from
+ * `bpmn/data-service.ts` — `registry/` does not depend on a family's emitter.
+ * `.dataService()` is the one call that binds a connection without naming one.
+ */
+const DATA_SERVICE_KEY = "uipath-uipath-dataservice";
 function isFile(path) {
     try {
         return statSync(path).isFile();
@@ -81,7 +89,8 @@ export function sourceBindingLabels(options) {
         catch {
             continue; // unreadable: it cannot be the author's working source
         }
-        readFile(file, relativePosix(root, file), text, options.connectorKey, moduleDirs, result);
+        const family = file.endsWith(".bpmn.ts") ? "bpmn" : "flow";
+        readFile(file, relativePosix(root, file), text, options.connectorKey, moduleDirs, result, family);
     }
     return result;
 }
@@ -115,7 +124,7 @@ function propertyName(name) {
         return name.text;
     return undefined;
 }
-function readFile(file, shown, text, connectorKey, moduleDirs, result) {
+function readFile(file, shown, text, connectorKey, moduleDirs, result, family) {
     const sf = ts.createSourceFile(file, text, ts.ScriptTarget.Latest, true, ts.ScriptKind.TS);
     // `parseDiagnostics` is where the scanner records syntax problems (see core/expr-check.ts).
     const problems = sf.parseDiagnostics;
@@ -316,6 +325,24 @@ function readFile(file, shown, text, connectorKey, moduleDirs, result) {
                     unplaceable(at, args.why);
                 else if (args.length === 0)
                     unplaceable(at, "spreads an empty array");
+                else if (family === "bpmn") {
+                    // Every BPMN call leads with the ELEMENT ID, so the key is one further
+                    // along than Flow's and the options are last. `.dataService()` names no
+                    // connector at all — there is only one it can mean.
+                    if (callee === "dataService") {
+                        record(DATA_SERVICE_KEY, args[1], at);
+                    }
+                    else if (callee === "connector" || callee === "eventTrigger" || callee === "waitForEvent") {
+                        const key = constString(args[1]);
+                        // A computed key is not a reason to distrust the whole file here: a
+                        // BPMN options object is a literal at the call site, and the label
+                        // this reads is the author's own name for a binding, not a fact the
+                        // emitter depends on. Flow's `unplaceable` exists because its first
+                        // argument may be a DESCRIPTOR whose key decides everything.
+                        if (key !== undefined)
+                            record(key, args[callee === "connector" ? 4 : 3], at);
+                    }
+                }
                 else if (callee === "connector") {
                     const [first] = args;
                     const literalKey = constString(first);
@@ -325,7 +352,7 @@ function readFile(file, shown, text, connectorKey, moduleDirs, result) {
                     else
                         record(key, args[literalKey !== undefined ? 3 : 2], at);
                 }
-                else {
+                else if (callee === "onEvent" || callee === "waitForEvent") {
                     const [first] = args;
                     const subscription = constObject(first);
                     if (subscription) {

package/dist/serialize.d.ts

@@ -1,7 +1,16 @@
 import type { BuiltFlow } from './flow-sdk.js';
-import type { Library, ResolvedConnector } from './library.js';
+import type { Library } from './library.js';
 import type { Bindings } from './bindings.js';
-import { type LookupResolutions, type LookupSpec } from './core/lookups.js';
+import { connectorInputErrors as coreConnectorInputErrors } from './core/connector-inputs.js';
+/**
+ * What `compile` would refuse these connector inputs with, or `undefined` when it
+ * would accept them — Flow's binding of the shared validator.
+ *
+ * `render` is not optional here: `renderDeep` is what REFUSES a lookup token in a
+ * position no lookup is resolved from (inside an opaque `object` input, say), and
+ * dropping it would make `check` quietly accept what `compile` throws on.
+ */
+export declare function connectorInputErrors(rc: Parameters<typeof coreConnectorInputErrors>[0], inputs: Record<string, unknown>, resolutions?: Parameters<typeof coreConnectorInputErrors>[2], lookups?: Parameters<typeof coreConnectorInputErrors>[3]): string | undefined;
 /**
  * A compile-time observation that does not stop emission. `check()` has its
  * diagnostics; this is the serializer's own channel for things it can only see
@@ -82,12 +91,3 @@ export declare function serialize(built: BuiltFlow, opts?: SerializeOpts): any;
  * placeholder and the step that carries it.
  */
 export declare function assertBindingPlaceholdersResolve(flow: any): void;
-/**
- * What `compile` would refuse these connector inputs with, or `undefined` when it
- * would accept them — the SAME code path `buildConnectorInputs` runs at emit time,
- * so `check` and `compile` cannot disagree about a missing required field, an
- * unknown or mistyped input, or a container part passed as a value. Unresolved
- * lookups are the one thing skipped: `check` already reports them as
- * LOOKUP_UNRESOLVED with the resolution recipe.
- */
-export declare function connectorInputErrors(rc: ResolvedConnector, inputs: Record<string, unknown>, resolutions?: LookupResolutions, lookups?: Readonly<Record<string, LookupSpec>>): string | undefined;