@dynamatix/cat-shared 0.0.9 → 0.0.11

package/middlewares/audit.middleware.js

@@ -1,8 +1,9 @@
 import AuditConfigModel from '../models/audit-config.model.js';
-
+import AuditLog from '../models/audit.model.js';
 
 function applyAuditMiddleware(schema, collectionName) {
-    schema.pre(['save','updateOne'], async function (next) {
+    // Handle create and save
+    schema.pre('save', async function (next) {
         const doc = this;
         const isNewDoc = doc.isNew;
         const auditConfig = await AuditConfigModel.findOne({ collectionName });
@@ -12,24 +13,42 @@ function applyAuditMiddleware(schema, collectionName) {
 
         if (isNewDoc && auditConfig.trackCreation) {
             logs.push({
-                name: collectionName + ' created',
+                name: `${collectionName} created`,
                 documentId: doc._id,
                 oldValue: null,
                 newValue: 'Document created',
                 createdBy: doc._updatedBy || null
             });
-        } else if (!isNewDoc && auditConfig.fields?.length) {
-            const original = await doc.constructor.findById(doc._id).lean();
-            if (!original) return next();
+        }
 
-            for (const field of auditConfig.fields) {
-                if (doc.isModified(field)) {
+        if (logs.length) await AuditLog.insertMany(logs);
+        next();
+    });
+
+    // Handle update queries like findByIdAndUpdate
+    schema.pre(['findOneAndUpdate', 'findByIdAndUpdate'], async function (next) {
+        const query = this;
+        const update = query.getUpdate();
+        const auditConfig = await AuditConfigModel.findOne({ collectionName });
+        if (!auditConfig || !auditConfig.fields?.length) return next();
+
+        const original = await query.model.findOne(query.getQuery()).lean();
+        if (!original) return next();
+
+        const logs = [];
+
+        for (const field of auditConfig.fields) {
+            if (update?.$set?.hasOwnProperty(field) || update?.hasOwnProperty(field)) {
+                const newValue = update?.$set?.[field] ?? update?.[field];
+                const oldValue = original[field];
+
+                if (oldValue !== newValue) {
                     logs.push({
-                        name: collectionName + '.'+ field,
-                        documentId: doc._id,
-                        oldValue: original[field],
-                        newValue: doc[field],
-                        createdBy: doc._updatedBy || null
+                        name: `${collectionName}.${field}`,
+                        documentId: original._id,
+                        oldValue,
+                        newValue,
+                        createdBy: update._updatedBy || null
                     });
                 }
             }
@@ -40,5 +59,4 @@ function applyAuditMiddleware(schema, collectionName) {
     });
 }
 
-
 export default applyAuditMiddleware;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dynamatix/cat-shared",
-  "version": "0.0.9",
+  "version": "0.0.11",
   "main": "index.js",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1"