@hotmeshio/hotmesh 0.22.5 → 0.22.7

package/build/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@hotmeshio/hotmesh",
-    "version": "0.22.5",
+    "version": "0.22.7",
     "description": "Durable Workflow",
     "main": "./build/index.js",
     "types": "./build/index.d.ts",

package/build/services/durable/client.js

@@ -365,7 +365,10 @@ class ClientService {
                 throw error;
             }
         }
-        else if (isNaN(Number(appVersion)) || appVersion < version) {
+        else if (isNaN(Number(appVersion)) ||
+            Number(appVersion) < Number(version)) {
+            // Numeric compare: string `<` mis-orders at digit boundaries (e.g. '9' < '16'
+            // is false as strings). Redeploy + hot-swap when an older schema is deployed.
             try {
                 await hotMesh.deploy((0, factory_1.getWorkflowYAML)(appId, version));
                 await hotMesh.activate(version);

package/build/services/durable/schemas/factory.d.ts

@@ -1,4 +1,4 @@
-declare const APP_VERSION = "15";
+declare const APP_VERSION = "16";
 declare const APP_ID = "durable";
 /**
  * returns a new durable workflow schema

package/build/services/durable/schemas/factory.js

@@ -1,7 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.APP_ID = exports.APP_VERSION = exports.getWorkflowYAML = void 0;
-const APP_VERSION = '15';
+// Schema content version for the generated durable app YAML below. BUMP THIS
+// whenever `getWorkflowYAML` changes — it is how existing deployments detect a
+// schema upgrade and hot-swap to it (see WorkerService.activateWorkflow and
+// ClientService.deployAndActivate). Changing the YAML without a bump leaves
+// every already-deployed database on the old schema forever. Numeric string —
+// compared with `Number()` for ordering. (v16: condition() escalation hook.)
+const APP_VERSION = '16';
 exports.APP_VERSION = APP_VERSION;
 const APP_ID = 'durable';
 exports.APP_ID = APP_ID;

package/build/services/durable/worker.js

@@ -128,7 +128,16 @@ class WorkerService {
     static async activateWorkflow(hotMesh) {
         const app = await hotMesh.engine.store.getApp(hotMesh.engine.appId);
         const appVersion = app?.version;
-        if (!appVersion) {
+        if (!appVersion ||
+            isNaN(Number(appVersion)) ||
+            Number(appVersion) < Number(factory_1.APP_VERSION)) {
+            // Not deployed, or an OLDER schema version is deployed (the SDK was
+            // upgraded). Deploy this SDK's schema and hot-swap to it. HotMesh keeps
+            // the prior version deployed, so in-flight jobs drain on the old schema
+            // while new jobs use APP_VERSION. Without this branch an existing, active
+            // app would never pick up a new schema — atomic-escalation hooks and any
+            // other schema-encoded feature would silently never activate after an
+            // upgrade. Mirrors ClientService.deployAndActivate.
             try {
                 await hotMesh.deploy((0, factory_1.getWorkflowYAML)(hotMesh.engine.appId, factory_1.APP_VERSION));
                 await hotMesh.activate(factory_1.APP_VERSION);

package/build/services/engine/version.js

@@ -49,10 +49,21 @@ async function getVID(instance, vid) {
         }
         return { id: instance.appId, version: app.version };
     }
-    else if (!instance.apps && vid) {
-        instance.apps = {};
-        instance.apps[instance.appId] = vid;
-        return vid;
+    else if (!instance.apps) {
+        // First call — always DB-refresh to avoid locking in a stale version.
+        // The `vid` parameter originates from store.getApp() (which may be
+        // cached from before the last activation).  If a worker missed the
+        // nocache NOTIFY (startup race: LISTEN not yet established when the
+        // NOTIFY fired), it would lock in the pre-activation version for its
+        // entire lifetime, silently loading the old schema on every request.
+        // One extra DB query here, once per engine lifetime, eliminates the
+        // race regardless of NOTIFY delivery.
+        const id = vid?.id ?? instance.appId;
+        const freshApp = await instance.store.getApp(id, true);
+        if (!instance.apps)
+            instance.apps = {};
+        instance.apps[instance.appId] = freshApp;
+        return { id: freshApp.id, version: freshApp.version };
     }
     else {
         return await fetchAndVerifyVID(instance, {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hotmeshio/hotmesh",
-  "version": "0.22.5",
+  "version": "0.22.7",
   "description": "Durable Workflow",
   "main": "./build/index.js",
   "types": "./build/index.d.ts",