@camstack/server 1.1.25 → 1.1.26

package/dist/core/agent/agent-registry.service.js

@@ -243,6 +243,14 @@ class AgentRegistryService {
      * match — a single package can ship multiple addons with distinct ids
      * and placements.
      *
+     * Version-skew exception: a decl-id the hub has never heard of is NOT
+     * stale when the hub already ships that addon's PACKAGE. A newer agent
+     * bundle can add a decl-id within a package the hub also ships (e.g. a
+     * new `decoder-nodeav` inside `@camstack/addon-pipeline`); undeploying
+     * it would delete the whole shared bundle dir on the agent and crash
+     * every sibling runner. Only a decl-id whose PACKAGE is absent from the
+     * hub is treated as genuinely stale.
+     *
      * All errors are caught and logged so a single bad agent never breaks
      * the caller (connect handler or boot pass).
      */
@@ -268,14 +276,20 @@ class AgentRegistryService {
             if (agentAddons.length === 0)
                 return;
             // Build the hub's placement map: decl id → placement. Absence from
-            // this map means "not installed on the hub".
+            // this map means "not installed on the hub". Also collect the set of
+            // PACKAGE names the hub ships — used by the version-skew guard below.
             const hubPlacements = new Map();
+            const hubPackages = new Set();
             for (const row of this.addonRegistry.listAddons()) {
                 const declId = row.manifest.id;
                 if (typeof declId !== 'string')
                     continue;
                 const decl = row.declaration ?? row.manifest;
                 hubPlacements.set(declId, (0, types_1.resolveAddonPlacement)(decl));
+                const packageName = row.manifest.packageName;
+                if (typeof packageName === 'string' && packageName.length > 0) {
+                    hubPackages.add(packageName);
+                }
             }
             const stale = agentAddons.filter((addon) => {
                 // Agent bootstrap infrastructure (storage/settings/metrics/logging from
@@ -289,9 +303,16 @@ class AgentRegistryService {
                     return false;
                 }
                 const placement = hubPlacements.get(addon.id);
-                // Not installed on the hub → stale.
-                if (placement === undefined)
+                if (placement === undefined) {
+                    // decl-id unknown to the hub. If the hub ships this PACKAGE, this is a
+                    // version-skew sibling (a newer agent bundle added a decl-id within a
+                    // package the hub already ships) — undeploying it would delete the whole
+                    // shared bundle on the agent. Only undeploy when the PACKAGE itself is
+                    // absent from the hub.
+                    if (addon.packageName !== undefined && hubPackages.has(addon.packageName))
+                        return false;
                     return true;
+                }
                 // Installed but pinned to the hub → must not run on an agent.
                 return placement === 'hub-only';
             });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camstack/server",
-  "version": "1.1.25",
+  "version": "1.1.26",
   "private": false,
   "files": [
     "dist",