@dynamatix/cat-shared 0.0.140 → 0.0.149

package/middlewares/audit.middleware.js

@@ -316,8 +316,23 @@ function applyAuditMiddleware(schema, collectionName) {
                 // --- Add pound prefix if needed ---
                 const dataType = fieldTypeMap[field];
                 if (dataType === 'pound') {
-                    if (displayOldValue != null && displayOldValue !== '') displayOldValue = `£${displayOldValue}`;
-                    if (displayNewValue != null && displayNewValue !== '') displayNewValue = `£${displayNewValue}`;
+                    const formatPoundValue = (rawValue) => {
+                        if (rawValue == null || rawValue === '') return rawValue;
+                        const stringValue = typeof rawValue === 'string' ? rawValue.trim() : String(rawValue);
+                        if (stringValue === '') return rawValue;
+                        const valueSansSymbol = stringValue.startsWith('£') ? stringValue.slice(1) : stringValue;
+                        const numericString = valueSansSymbol.replace(/,/g, '');
+                        const numericValue = Number(numericString);
+                        if (!Number.isFinite(numericValue)) return `£${valueSansSymbol}`;
+                        const decimalPart = numericString.split('.')[1];
+                        const formatted = numericValue.toLocaleString('en-GB', {
+                            minimumFractionDigits: decimalPart ? decimalPart.length : 0,
+                            maximumFractionDigits: decimalPart ? decimalPart.length : 20
+                        });
+                        return `£${formatted}`;
+                    };
+                    displayOldValue = formatPoundValue(displayOldValue);
+                    displayNewValue = formatPoundValue(displayNewValue);
                 }
                 if (displayOldValue !== displayNewValue) {
                     const fieldDescription = await resolveDescription(field, result);
@@ -411,12 +426,21 @@ function applyAuditMiddleware(schema, collectionName) {
 async function updateContextAuditCount(contextId, count) {
     count = Number(count);
     if (!contextId || isNaN(count)) return;
-    const model = mongoose.models['Application'];
-    await model.findByIdAndUpdate(
-        { _id: contextId },
-        { $inc: { newAuditRecordsCount: count } },
-        { new: true, upsert: true }
-    );
+    
+    try {
+        const model = mongoose.models['Application'];
+        
+        // Use direct collection access to bypass all Mongoose middleware
+        // This prevents workflow triggers from firing for audit count updates
+        await model.collection.updateOne(
+            { _id: new mongoose.Types.ObjectId(contextId) },
+            { $inc: { newAuditRecordsCount: count } },
+            { upsert: false }
+        );
+    } catch (error) {
+        console.error('Error updating audit count:', error);
+        // Don't throw - audit count update failure shouldn't break the main operation
+    }
 }
 
 export function registerAuditHook(callback) {

package/package.json

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