@objectstack/metadata 5.0.0 → 5.1.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,
@@ -3644,6 +3628,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 +3669,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 +3682,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 +3697,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 +3789,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 +3802,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