@hotmeshio/hotmesh 0.7.0 → 0.9.0

package/build/services/engine/index.js

@@ -183,7 +183,7 @@ class EngineService {
      * @private
      */
     async processWebHooks() {
-        this.taskService.processWebHooks(this.hook.bind(this));
+        this.taskService.processWebHooks(this.signal.bind(this));
     }
     /**
      * @private
@@ -381,7 +381,7 @@ class EngineService {
     /**
      * @private
      */
-    async execAdjacentParent(context, jobOutput, emit = false) {
+    async execAdjacentParent(context, jobOutput, emit = false, transaction) {
         if (this.hasParentJob(context)) {
             //errors are stringified `StreamError` objects
             const error = this.resolveError(jobOutput.metadata);
@@ -414,7 +414,7 @@ class EngineService {
                 streamData.status = stream_1.StreamStatus.SUCCESS;
                 streamData.code = enums_1.HMSH_CODE_SUCCESS;
             }
-            return (await this.router?.publishMessage(null, streamData));
+            return (await this.router?.publishMessage(null, streamData, transaction));
         }
     }
     /**
@@ -457,11 +457,17 @@ class EngineService {
         //todo: do not allow scrubbing of non-existent or actively running job
         await this.store.scrub(jobId);
     }
-    // ****************** `HOOK` ACTIVITY RE-ENTRY POINT *****************
+    // ****************** `SIGNAL` ACTIVITY RE-ENTRY POINT ****************
     /**
+     * Delivers a signal (data payload) to a paused hook activity,
+     * resuming its Leg 2 execution. The `topic` must match a hook rule
+     * defined in the YAML graph's `hooks` section. The engine locates
+     * the target activity and dimension for reentry based on the hook
+     * rule's match conditions.
+     *
      * @private
      */
-    async hook(topic, data, status = stream_1.StreamStatus.SUCCESS, code = 200) {
+    async signal(topic, data, status = stream_1.StreamStatus.SUCCESS, code = 200, transaction) {
         const hookRule = await this.taskService.getHookRule(topic);
         const [aid] = await this.getSchema(`.${hookRule.to}`);
         const streamData = {
@@ -475,7 +481,7 @@ class EngineService {
             },
             data,
         };
-        return (await this.router?.publishMessage(null, streamData));
+        return (await this.router?.publishMessage(null, streamData, transaction));
     }
     /**
      * @private
@@ -503,9 +509,13 @@ class EngineService {
         await this.router?.publishMessage(null, streamData);
     }
     /**
+     * Fan-out variant of `signal()` that delivers data to **all**
+     * paused workflows matching a search query. Useful for resuming
+     * a batch of workflows waiting on the same external event.
+     *
      * @private
      */
-    async hookAll(hookTopic, data, keyResolver, queryFacets = []) {
+    async signalAll(hookTopic, data, keyResolver, queryFacets = []) {
         const config = await this.getVID();
         const hookRule = await this.taskService.getHookRule(hookTopic);
         if (hookRule) {
@@ -621,7 +631,7 @@ class EngineService {
     /**
      * @private
      */
-    async pubOneTimeSubs(context, jobOutput, emit = false) {
+    async pubOneTimeSubs(context, jobOutput, emit = false, transaction) {
         //todo: subscriber should query for the job...only publish minimum context needed
         if (this.hasOneTimeSubscription(context)) {
             const message = {
@@ -629,7 +639,7 @@ class EngineService {
                 topic: context.metadata.jid,
                 job: (0, utils_1.restoreHierarchy)(jobOutput),
             };
-            this.subscribe.publish(key_1.KeyType.QUORUM, message, this.appId, context.metadata.ngn);
+            await this.subscribe.publish(key_1.KeyType.QUORUM, message, this.appId, context.metadata.ngn, transaction);
         }
     }
     /**
@@ -644,7 +654,7 @@ class EngineService {
     /**
      * @private
      */
-    async pubPermSubs(context, jobOutput, emit = false) {
+    async pubPermSubs(context, jobOutput, emit = false, transaction) {
         const topic = await this.getPublishesTopic(context);
         if (topic) {
             const message = {
@@ -652,7 +662,7 @@ class EngineService {
                 topic,
                 job: (0, utils_1.restoreHierarchy)(jobOutput),
             };
-            this.subscribe.publish(key_1.KeyType.QUORUM, message, this.appId, `${topic}.${context.metadata.jid}`);
+            await this.subscribe.publish(key_1.KeyType.QUORUM, message, this.appId, `${topic}.${context.metadata.jid}`, transaction);
         }
     }
     /**
@@ -683,20 +693,41 @@ class EngineService {
     /**
      * @private
      */
-    async runJobCompletionTasks(context, options = {}) {
+    async runJobCompletionTasks(context, options = {}, transaction) {
         //'emit' indicates the job is still active
         const isAwait = this.hasParentJob(context, true);
         const isOneTimeSub = this.hasOneTimeSubscription(context);
         const topic = await this.getPublishesTopic(context);
         let msgId;
+        let jobOutput;
         if (isAwait || isOneTimeSub || topic) {
-            const jobOutput = await this.getState(context.metadata.tpc, context.metadata.jid);
-            msgId = await this.execAdjacentParent(context, jobOutput, options.emit);
-            this.pubOneTimeSubs(context, jobOutput, options.emit);
-            this.pubPermSubs(context, jobOutput, options.emit);
+            jobOutput = await this.getState(context.metadata.tpc, context.metadata.jid);
+            //only send RESULT to parent for non-severed children (execChild).
+            //startChild (px=true) children already sent RESULT at spawn time
+            //via Trigger.execAdjacentParent; sending again here would cause
+            //a duplicate semaphore decrement in collation workflows.
+            if (isAwait) {
+                msgId = await this.execAdjacentParent(context, jobOutput, options.emit, transaction);
+            }
+            if (transaction) {
+                //transactional: await to queue NOTIFY in the transaction
+                await this.pubOneTimeSubs(context, jobOutput, options.emit, transaction);
+                await this.pubPermSubs(context, jobOutput, options.emit, transaction);
+            }
+            else {
+                //non-transactional: fire-and-forget to avoid race with inline
+                //trigger processing (callback registered after pub() returns)
+                this.pubOneTimeSubs(context, jobOutput, options.emit);
+                this.pubPermSubs(context, jobOutput, options.emit);
+            }
         }
         if (!options.emit) {
-            this.taskService.registerJobForCleanup(context.metadata.jid, this.resolveExpires(context, options), options);
+            if (transaction) {
+                await this.taskService.registerJobForCleanup(context.metadata.jid, this.resolveExpires(context, options), options, transaction);
+            }
+            else {
+                this.taskService.registerJobForCleanup(context.metadata.jid, this.resolveExpires(context, options), options);
+            }
         }
         return msgId;
     }
@@ -745,7 +776,7 @@ class EngineService {
         }
         const [state, status] = output;
         const stateTree = (0, utils_1.restoreHierarchy)(state);
-        if (status && stateTree.metadata) {
+        if (status != null && stateTree.metadata) {
             stateTree.metadata.js = status;
         }
         return stateTree;

package/build/services/exporter/index.d.ts

@@ -10,8 +10,10 @@ import { StringStringType, Symbols } from '../../types/serializer';
 declare class ExporterService {
     appId: string;
     logger: ILogger;
+    /** @hidden */
     store: StoreService<ProviderClient, ProviderTransaction>;
     symbols: Promise<Symbols> | Symbols;
+    /** @hidden */
     constructor(appId: string, store: StoreService<ProviderClient, ProviderTransaction>, logger: ILogger);
     /**
      * Convert the job hash into a JobExport object.

package/build/services/exporter/index.js

@@ -9,6 +9,7 @@ const serializer_1 = require("../serializer");
  * includes dependency list
  */
 class ExporterService {
+    /** @hidden */
     constructor(appId, store, logger) {
         this.appId = appId;
         this.logger = logger;