@objectstack/service-automation 9.5.1 → 9.6.0

package/dist/index.cjs

@@ -67,6 +67,8 @@ var _AutomationEngine = class _AutomationEngine {
     this.boundFlowTriggers = /* @__PURE__ */ new Map();
     /** Connectors registered by integration plugins, keyed by connector name (ADR-0018 §Addendum). */
     this.connectors = /* @__PURE__ */ new Map();
+    /** Bridge to the host function registry for `script`-node calls (#1870), if wired. */
+    this.functionResolver = null;
     this.executionLogs = [];
     /**
      * Runs paused at a node, keyed by runId (ADR-0019). In-memory hot cache —
@@ -354,6 +356,24 @@ var _AutomationEngine = class _AutomationEngine {
   resolveConnectorAction(connectorId, actionId) {
     return this.connectors.get(connectorId)?.handlers[actionId];
   }
+  /**
+   * Wire the engine to the host's named-function registry (#1870). The
+   * automation plugin calls this in `start()` with a resolver backed by
+   * ObjectQL's `resolveFunction` (populated from `bundle.functions` /
+   * `defineStack({ functions })`), so a `script` node can invoke an
+   * authored function by name. Passing `null` detaches the bridge.
+   */
+  setFunctionResolver(resolver) {
+    this.functionResolver = resolver;
+  }
+  /**
+   * Resolve a named function for a `script` node. Returns `undefined` when no
+   * resolver is wired or the name is unregistered — the node then fails the
+   * step with a clear error rather than silently no-op'ing.
+   */
+  resolveFunction(name) {
+    return this.functionResolver?.(name) ?? void 0;
+  }
   /** Get all registered connector names. */
   getRegisteredConnectors() {
     return [...this.connectors.keys()];
@@ -1897,7 +1917,7 @@ function resolveToken(token, variables, context) {
     if (path[0] === "Email") return resolvePath(context.user, ["email", ...path.slice(1)]) ?? void 0;
     return resolvePath(context.user, path);
   }
-  if (/^[A-Za-z_$][\w$]*(?:\.[A-Za-z_$][\w$]*)*$/.test(trimmed)) {
+  if (/^[A-Za-z_$][\w$]*(?:\.(?:[A-Za-z_$][\w$]*|\d+))*$/.test(trimmed)) {
     const segments = trimmed.split(".");
     const head = segments[0];
     if (variables.has(head)) {
@@ -2271,14 +2291,21 @@ function registerCrudNodes(engine, ctx) {
       const data = getData();
       if (!data) {
         ctx.logger.warn(`[create_record] no data engine; skipping ${objectName}`);
-        if (outputVariable) variables.set(outputVariable, `mock-${objectName}-${Date.now()}`);
-        return { success: true, output: { id: `mock-${objectName}-${Date.now()}`, object: objectName } };
+        const mockId = `mock-${objectName}-${Date.now()}`;
+        if (outputVariable) variables.set(outputVariable, { id: mockId });
+        return { success: true, output: { id: mockId, object: objectName } };
       }
       try {
         const created = await data.insert(objectName, fields);
-        const insertedId = Array.isArray(created) ? created[0]?.id : created?.id ?? created;
-        if (outputVariable) variables.set(outputVariable, insertedId);
-        return { success: true, output: { id: insertedId, record: created, object: objectName } };
+        const createdRecord = Array.isArray(created) ? created[0] : created;
+        const insertedId = createdRecord && typeof createdRecord === "object" ? createdRecord.id : createdRecord;
+        if (outputVariable) {
+          variables.set(
+            outputVariable,
+            createdRecord && typeof createdRecord === "object" ? createdRecord : { id: insertedId }
+          );
+        }
+        return { success: true, output: { id: insertedId, record: createdRecord, object: objectName } };
       } catch (err) {
         return { success: false, error: `create_record(${objectName}) failed: ${err.message}` };
       }
@@ -2345,6 +2372,7 @@ function registerCrudNodes(engine, ctx) {
 
 // src/builtin/screen-nodes.ts
 var import_automation7 = require("@objectstack/spec/automation");
+var SCRIPT_BUILTIN_ACTION_TYPES = /* @__PURE__ */ new Set(["email", "slack"]);
 function registerScreenNodes(engine, ctx) {
   engine.registerNodeExecutor({
     type: "screen",
@@ -2400,24 +2428,53 @@ function registerScreenNodes(engine, ctx) {
       category: "logic",
       source: "builtin"
     }),
-    async execute(node, _variables, _context) {
+    async execute(node, variables, context) {
       const cfg = node.config ?? {};
-      const actionType = cfg.actionType ?? "noop";
-      if (actionType === "email") {
+      const fnRaw = cfg.function ?? cfg.functionName;
+      const fnName = typeof fnRaw === "string" && fnRaw.trim() ? fnRaw.trim() : void 0;
+      const actionType = typeof cfg.actionType === "string" && cfg.actionType.trim() ? cfg.actionType.trim() : void 0;
+      if (!fnName && actionType && SCRIPT_BUILTIN_ACTION_TYPES.has(actionType)) {
         ctx.logger.info(
-          `[Script:email] template=${String(cfg.template)} recipients=${JSON.stringify(cfg.recipients)} vars=${JSON.stringify(cfg.variables)}`
+          `[Script:${actionType}] template=${String(cfg.template)} recipients=${JSON.stringify(cfg.recipients)} vars=${JSON.stringify(cfg.variables)}`
         );
         return {
           success: true,
-          output: {
-            actionType,
-            template: cfg.template,
-            recipients: cfg.recipients
-          }
+          output: { actionType, template: cfg.template, recipients: cfg.recipients }
+        };
+      }
+      const inlineScript = typeof cfg.script === "string" && cfg.script.trim() ? cfg.script : void 0;
+      if (!fnName && inlineScript) {
+        ctx.logger.warn(
+          `[Script] node '${node.id}': inline \`config.script\` is not executed by the built-in runtime (no server-side JS sandbox) \u2014 this node is a no-op. To run server logic, move it into a registered function and call it via \`config.function\` + \`defineStack({ functions })\`.`
+        );
+        return { success: true, output: { script: "not-executed" } };
+      }
+      const target = fnName ?? (actionType === "invoke_function" ? void 0 : actionType);
+      if (!target) {
+        return {
+          success: false,
+          error: actionType === "invoke_function" ? `script node '${node.id}': actionType 'invoke_function' requires \`config.function\` (or \`functionName\`) naming the function to call.` : `script node '${node.id}': declares neither \`actionType\` nor \`function\` \u2014 nothing to run.`
+        };
+      }
+      const handler = engine.resolveFunction(target);
+      if (!handler) {
+        return {
+          success: false,
+          error: `script node '${node.id}': '${target}' is not a built-in action (${[...SCRIPT_BUILTIN_ACTION_TYPES].join(", ")}) and no function named '${target}' is registered. Register it via \`defineStack({ functions: { '${target}': fn } })\`, or fix the name (#1870).`
+        };
+      }
+      const input = interpolate(cfg.inputs ?? cfg.input ?? {}, variables, context);
+      const outputVariable = typeof cfg.outputVariable === "string" && cfg.outputVariable.trim() ? cfg.outputVariable.trim() : void 0;
+      try {
+        const result = await handler({ input, variables, automation: context, logger: ctx.logger });
+        if (outputVariable) variables.set(outputVariable, result);
+        return { success: true, output: { function: target, result } };
+      } catch (err) {
+        return {
+          success: false,
+          error: `script function '${target}' (node '${node.id}') failed: ${err.message}`
         };
       }
-      ctx.logger.info(`[Script:${actionType}] node=${node.id} executed (no-op handler)`);
-      return { success: true, output: { actionType } };
     }
   });
   ctx.logger.info("[Screen/Script Nodes] 2 built-in node executors registered");
@@ -3067,6 +3124,18 @@ var AutomationServicePlugin = class {
         ctx.logger.info("[Automation] No ObjectQL engine \u2014 suspended runs kept in-memory only");
       }
     }
+    try {
+      const fnRegistry = ctx.getService("objectql");
+      if (fnRegistry && typeof fnRegistry.resolveFunction === "function") {
+        this.engine.setFunctionResolver((name) => {
+          const fn = fnRegistry.resolveFunction(name);
+          return typeof fn === "function" ? (fnCtx) => fn(fnCtx) : void 0;
+        });
+        ctx.logger.debug("[Automation] script-node function registry bridged to objectql.resolveFunction");
+      }
+    } catch {
+      ctx.logger.debug("[Automation] objectql not present \u2014 script-node function calls will fail loudly when used");
+    }
     try {
       const ql = ctx.getService("objectql");
       if (!ql) {