@tap-payments/os-micro-frontend-shared 0.1.135-test.3 → 0.1.135-test.5

package/build/components/VirtualTables/SheetViewVirtualTable/hooks/usePerformanceOptimizations.js

@@ -9,12 +9,7 @@ export const usePerformanceOptimizations = (columns) => {
         const hasChanged = columns.length !== stableColumnsRef.current.length ||
             columns.some((col, index) => {
                 const prevCol = stableColumnsRef.current[index];
-                return (!prevCol ||
-                    col.id !== prevCol.id ||
-                    col.width !== prevCol.width ||
-                    col.pinned !== prevCol.pinned ||
-                    col.hidden !== prevCol.hidden ||
-                    col.order !== prevCol.order);
+                return !prevCol || col.id !== prevCol.id || col.filter !== prevCol.filter;
             });
         if (hasChanged) {
             stableColumnsRef.current = [...columns];
@@ -39,6 +34,39 @@ export const usePerformanceOptimizations = (columns) => {
                 currentFilters[columnId] = filterData;
             }
         });
+        // Safe comparison helper that avoids circular references
+        const safeCompare = (obj1, obj2) => {
+            // Handle primitive types
+            if (obj1 === obj2)
+                return true;
+            if (obj1 == null || obj2 == null)
+                return obj1 === obj2;
+            if (typeof obj1 !== 'object' || typeof obj2 !== 'object')
+                return obj1 === obj2;
+            // Handle arrays
+            if (Array.isArray(obj1) && Array.isArray(obj2)) {
+                if (obj1.length !== obj2.length)
+                    return false;
+                return obj1.every((item, index) => safeCompare(item, obj2[index]));
+            }
+            if (Array.isArray(obj1) || Array.isArray(obj2))
+                return false;
+            // Handle objects - only compare basic properties, avoid complex nested objects
+            const keys1 = Object.keys(obj1);
+            const keys2 = Object.keys(obj2);
+            if (keys1.length !== keys2.length)
+                return false;
+            return keys1.every((key) => {
+                const val1 = obj1[key];
+                const val2 = obj2[key];
+                // Skip complex objects that might have circular references
+                if (typeof val1 === 'object' && val1 != null && (val1.constructor === Object || Array.isArray(val1))) {
+                    return safeCompare(val1, val2);
+                }
+                // For other object types (like React contexts), just compare references
+                return val1 === val2;
+            });
+        };
         // Deep comparison of filter state
         const hasFilterChanged = () => {
             const currentKeys = Object.keys(currentFilters);
@@ -53,11 +81,10 @@ export const usePerformanceOptimizations = (columns) => {
                 // Compare filter properties that affect rendering
                 if (current.type !== prev.type || current.apiKey !== prev.apiKey)
                     return true;
-                // Deep compare options array
-                if (JSON.stringify(current.options) !== JSON.stringify(prev.options))
+                // Safe comparison for options and data
+                if (!safeCompare(current.options, prev.options))
                     return true;
-                // Deep compare filter data
-                if (JSON.stringify(current.data) !== JSON.stringify(prev.data))
+                if (!safeCompare(current.data, prev.data))
                     return true;
                 return false;
             });

package/package.json

@@ -1,8 +1,8 @@
 {
   "name": "@tap-payments/os-micro-frontend-shared",
   "description": "Shared components and utilities for Tap Payments micro frontends",
-  "version": "0.1.135-test.3",
-  "testVersion": 3,
+  "version": "0.1.135-test.5",
+  "testVersion": 5,
   "type": "module",
   "main": "build/index.js",
   "module": "build/index.js",