@objectstack/metadata 5.0.0 → 5.2.0

package/dist/index.cjs

@@ -799,7 +799,6 @@ var DatabaseLoader = class {
   /**
    * Create a history record for a metadata change.
    *
-   * @param metadataId - The metadata record ID
    * @param type - Metadata type
    * @param name - Metadata name
    * @param version - Version number
@@ -809,7 +808,7 @@ var DatabaseLoader = class {
    * @param changeNote - Optional change description
    * @param recordedBy - Optional user who made the change
    */
-  async createHistoryRecord(metadataId, type, name, version, metadata, operationType, previousChecksum, changeNote, recordedBy) {
+  async createHistoryRecord(type, name, version, metadata, operationType, previousChecksum, changeNote, recordedBy) {
     if (!this.trackHistory) return;
     await this.ensureHistorySchema();
     const now = (/* @__PURE__ */ new Date()).toISOString();
@@ -822,7 +821,6 @@ var DatabaseLoader = class {
     const eventSeq = await this.nextEventSeq();
     const historyRecord = {
       id: historyId,
-      metadataId,
       name,
       type,
       version,
@@ -839,7 +837,6 @@ var DatabaseLoader = class {
       await this._create(this.historyTableName, {
         id: historyRecord.id,
         event_seq: eventSeq,
-        metadata_id: historyRecord.metadataId,
         name: historyRecord.name,
         type: historyRecord.type,
         version: historyRecord.version,
@@ -1027,12 +1024,9 @@ var DatabaseLoader = class {
   async getHistoryRecord(type, name, version) {
     if (!this.trackHistory) return null;
     await this.ensureHistorySchema();
-    const metadataRow = await this._findOne(this.tableName, {
-      where: this.baseFilter(type, name)
-    });
-    if (!metadataRow) return null;
     const filter = {
-      metadata_id: metadataRow.id,
+      type,
+      name,
       version
     };
     if (this.organizationId) {
@@ -1044,7 +1038,6 @@ var DatabaseLoader = class {
     if (!row) return null;
     return {
       id: row.id,
-      metadataId: row.metadata_id,
       name: row.name,
       type: row.type,
       version: row.version,
@@ -1069,14 +1062,9 @@ var DatabaseLoader = class {
     }
     await this.ensureSchema();
     await this.ensureHistorySchema();
-    const filter = { type, name };
-    if (this.organizationId) filter.organization_id = this.organizationId;
-    const metadataRecord = await this._findOne(this.tableName, { where: filter });
-    if (!metadataRecord) {
-      return { records: [], total: 0, hasMore: false };
-    }
     const historyFilter = {
-      metadata_id: metadataRecord.id
+      type,
+      name
     };
     if (this.organizationId) historyFilter.organization_id = this.organizationId;
     if (options?.operationType) historyFilter.operation_type = options.operationType;
@@ -1107,7 +1095,6 @@ var DatabaseLoader = class {
       const parsedMetadata = typeof row.metadata === "string" ? JSON.parse(row.metadata) : row.metadata;
       return {
         id: row.id,
-        metadataId: row.metadata_id,
         name: row.name,
         type: row.type,
         version: row.version,
@@ -1152,7 +1139,6 @@ var DatabaseLoader = class {
     });
     this.invalidate(type, name);
     await this.createHistoryRecord(
-      existing.id,
       type,
       name,
       newVersion,
@@ -1194,7 +1180,6 @@ var DatabaseLoader = class {
         });
         this.invalidate(type, name);
         await this.createHistoryRecord(
-          existing.id,
           type,
           name,
           version,
@@ -1228,7 +1213,6 @@ var DatabaseLoader = class {
         });
         this.invalidate(type, name);
         await this.createHistoryRecord(
-          id,
           type,
           name,
           1,
@@ -2408,6 +2392,23 @@ var _MetadataManager = class _MetadataManager {
     this.notifyWatchers(type, legacyEvent);
   }
   notifyWatchers(type, event) {
+    this.notifyWatchersLocal(type, event);
+    if (this.clusterPubSub) {
+      const payload = {
+        originNode: this.clusterNodeId,
+        type,
+        event
+      };
+      const key = `${type}:${event.name ?? ""}`;
+      void this.clusterPubSub.publish(_MetadataManager.CLUSTER_CHANNEL, payload, { partitionKey: key }).catch((err) => {
+        this.logger.error("Cluster metadata publish failed", void 0, {
+          type,
+          error: err instanceof Error ? err.message : String(err)
+        });
+      });
+    }
+  }
+  notifyWatchersLocal(type, event) {
     const callbacks = this.watchCallbacks.get(type);
     if (!callbacks) return;
     for (const callback of callbacks) {
@@ -2421,6 +2422,63 @@ var _MetadataManager = class _MetadataManager {
       }
     }
   }
+  /**
+   * Attach a cluster pub/sub transport so metadata-change events fan
+   * out to peer nodes and remote events replay into local watchers.
+   *
+   * The bridge plugin in @objectstack/service-cluster calls this once
+   * per kernel boot after both cluster and metadata services are
+   * registered. Passing the same MetadataManager twice no-ops; passing
+   * a different transport replaces the prior subscription.
+   *
+   * Pass `nodeId` matching the local cluster's nodeId so loopback
+   * suppression works.
+   *
+   * @returns disposer that unsubscribes from cluster events.
+   */
+  attachClusterPubSub(pubsub, nodeId) {
+    if (this.clusterPubSub === pubsub && this.clusterNodeId === nodeId) {
+      return () => this.detachClusterPubSub();
+    }
+    this.detachClusterPubSub();
+    this.clusterPubSub = pubsub;
+    this.clusterNodeId = nodeId;
+    this.clusterUnsubscribe = pubsub.subscribe(
+      _MetadataManager.CLUSTER_CHANNEL,
+      (msg) => {
+        const p = msg.payload;
+        if (p?.originNode && p.originNode === this.clusterNodeId) return;
+        if (!p?.type || !p.event) return;
+        setImmediate(() => {
+          try {
+            this.notifyWatchersLocal(p.type, p.event);
+          } catch (err) {
+            this.logger.error("Cluster remote replay failed", void 0, {
+              type: p.type,
+              error: err instanceof Error ? err.message : String(err)
+            });
+          }
+        });
+      }
+    );
+    this.logger.info("MetadataManager attached to cluster pubsub", {
+      nodeId,
+      channel: _MetadataManager.CLUSTER_CHANNEL
+    });
+    return () => this.detachClusterPubSub();
+  }
+  /** Tear down cluster wiring. Safe to call multiple times. */
+  detachClusterPubSub() {
+    if (this.clusterUnsubscribe) {
+      try {
+        this.clusterUnsubscribe();
+      } catch {
+      }
+      this.clusterUnsubscribe = void 0;
+    }
+    this.clusterPubSub = void 0;
+    this.clusterNodeId = void 0;
+  }
   // ==========================================
   // Version History & Rollback
   // ==========================================
@@ -2521,6 +2579,7 @@ var _MetadataManager = class _MetadataManager {
   }
 };
 _MetadataManager.LIST_CACHE_TTL_MS = 3e4;
+_MetadataManager.CLUSTER_CHANNEL = "metadata.changed";
 var MetadataManager = _MetadataManager;
 
 // src/plugin.ts
@@ -3644,6 +3703,10 @@ function registerMetadataHistoryRoutes(app, metadataService) {
 }
 
 // src/utils/history-cleanup.ts
+var import_kernel2 = require("@objectstack/spec/kernel");
+function executionPinnedTypes() {
+  return import_kernel2.DEFAULT_METADATA_TYPE_REGISTRY.filter((entry) => entry.executionPinned).map((entry) => entry.type);
+}
 var HistoryCleanupManager = class {
   constructor(policy, dbLoader) {
     this.policy = policy;
@@ -3681,6 +3744,8 @@ var HistoryCleanupManager = class {
     const organizationId = this.dbLoader.organizationId;
     let deleted = 0;
     let errors = 0;
+    const pinnedTypes = executionPinnedTypes();
+    const isPinned = (t) => !!t && pinnedTypes.includes(t);
     try {
       if (this.policy.maxAgeDays) {
         const cutoffDate = /* @__PURE__ */ new Date();
@@ -3692,6 +3757,9 @@ var HistoryCleanupManager = class {
         if (organizationId) {
           filter.organization_id = organizationId;
         }
+        if (pinnedTypes.length > 0) {
+          filter.type = { $nin: pinnedTypes };
+        }
         try {
           const result = await this.bulkDeleteByFilter(driver, historyTableName, filter);
           deleted += result.deleted;
@@ -3704,19 +3772,22 @@ var HistoryCleanupManager = class {
         try {
           const baseWhere = {};
           if (organizationId) baseWhere.organization_id = organizationId;
-          const metadataIds = await driver.find(historyTableName, {
+          const metaItems = await driver.find(historyTableName, {
             object: historyTableName,
             where: baseWhere,
-            fields: ["metadata_id"]
+            fields: ["type", "name"]
           });
-          const uniqueIds = /* @__PURE__ */ new Set();
-          for (const record of metadataIds) {
-            if (record.metadata_id) {
-              uniqueIds.add(record.metadata_id);
+          const uniqueKeys = /* @__PURE__ */ new Set();
+          for (const record of metaItems) {
+            const t = record.type;
+            const n = record.name;
+            if (t && n && !isPinned(t)) {
+              uniqueKeys.add(`${t}${n}`);
             }
           }
-          for (const metadataId of uniqueIds) {
-            const filter = { metadata_id: metadataId, ...baseWhere };
+          for (const key of uniqueKeys) {
+            const [type, name] = key.split("");
+            const filter = { type, name, ...baseWhere };
             try {
               const historyRecords = await driver.find(historyTableName, {
                 object: historyTableName,
@@ -3793,6 +3864,8 @@ var HistoryCleanupManager = class {
     const organizationId = this.dbLoader.organizationId;
     let recordsByAge = 0;
     let recordsByCount = 0;
+    const pinnedTypes = executionPinnedTypes();
+    const isPinned = (t) => !!t && pinnedTypes.includes(t);
     try {
       const baseWhere = {};
       if (organizationId) baseWhere.organization_id = organizationId;
@@ -3804,25 +3877,31 @@ var HistoryCleanupManager = class {
           recorded_at: { $lt: cutoffISO },
           ...baseWhere
         };
+        if (pinnedTypes.length > 0) {
+          filter.type = { $nin: pinnedTypes };
+        }
         recordsByAge = await driver.count(historyTableName, {
           object: historyTableName,
           where: filter
         });
       }
       if (this.policy.maxVersions) {
-        const metadataIds = await driver.find(historyTableName, {
+        const metaItems = await driver.find(historyTableName, {
           object: historyTableName,
           where: baseWhere,
-          fields: ["metadata_id"]
+          fields: ["type", "name"]
         });
-        const uniqueIds = /* @__PURE__ */ new Set();
-        for (const record of metadataIds) {
-          if (record.metadata_id) {
-            uniqueIds.add(record.metadata_id);
+        const uniqueKeys = /* @__PURE__ */ new Set();
+        for (const record of metaItems) {
+          const t = record.type;
+          const n = record.name;
+          if (t && n && !isPinned(t)) {
+            uniqueKeys.add(`${t}${n}`);
           }
         }
-        for (const metadataId of uniqueIds) {
-          const filter = { metadata_id: metadataId, ...baseWhere };
+        for (const key of uniqueKeys) {
+          const [type, name] = key.split("");
+          const filter = { type, name, ...baseWhere };
           const count = await driver.count(historyTableName, {
             object: historyTableName,
             where: filter