@alaarab/ogrid-core 2.0.4 → 2.0.6

package/dist/esm/utils/aggregationUtils.js

@@ -30,12 +30,22 @@ export function computeAggregations(items, visibleCols, selectionRange) {
     // Need at least 2 cells selected and at least 1 numeric value to show aggregation
     if (totalCells < 2 || numericValues.length === 0)
         return null;
-    const sum = numericValues.reduce((a, b) => a + b, 0);
+    let sum = 0;
+    let min = numericValues[0];
+    let max = numericValues[0];
+    for (let i = 0; i < numericValues.length; i++) {
+        const v = numericValues[i];
+        sum += v;
+        if (v < min)
+            min = v;
+        if (v > max)
+            max = v;
+    }
     return {
         sum,
         avg: sum / numericValues.length,
-        min: Math.min(...numericValues),
-        max: Math.max(...numericValues),
+        min,
+        max,
         count: numericValues.length,
     };
 }

package/dist/esm/utils/clientSideData.js

@@ -12,6 +12,11 @@ import { getFilterField } from './ogridHelpers';
  * @returns Filtered and sorted array
  */
 export function processClientSideData(data, columns, filters, sortBy, sortDirection) {
+    // Build column lookup map for O(1) access by columnId
+    const columnMap = new Map();
+    for (let i = 0; i < columns.length; i++) {
+        columnMap.set(columns[i].columnId, columns[i]);
+    }
     // --- Filtering (single-pass: build predicates, then one .filter()) ---
     const predicates = [];
     for (let i = 0; i < columns.length; i++) {
@@ -71,37 +76,59 @@ export function processClientSideData(data, columns, filters, sortBy, sortDirect
         : data.slice();
     // --- Sorting ---
     if (sortBy) {
-        const sortCol = columns.find((c) => c.columnId === sortBy);
+        const sortCol = columnMap.get(sortBy);
         const compare = sortCol?.compare;
         const dir = sortDirection === 'asc' ? 1 : -1;
-        rows.sort((a, b) => {
-            if (compare)
-                return compare(a, b) * dir;
-            const av = sortCol
-                ? getCellValue(a, sortCol)
-                : a[sortBy];
-            const bv = sortCol
-                ? getCellValue(b, sortCol)
-                : b[sortBy];
-            if (av == null && bv == null)
-                return 0;
-            if (av == null)
-                return -1 * dir;
-            if (bv == null)
-                return 1 * dir;
-            if (sortCol?.type === 'date') {
-                const at = new Date(String(av)).getTime();
-                const bt = new Date(String(bv)).getTime();
-                const aN = Number.isNaN(at) ? 0 : at;
-                const bN = Number.isNaN(bt) ? 0 : bt;
-                return aN === bN ? 0 : aN > bN ? dir : -dir;
+        const isDateSort = sortCol?.type === 'date';
+        // For date columns, pre-compute timestamps to avoid repeated new Date() in O(n log n) comparisons
+        if (isDateSort && !compare) {
+            const timestampCache = new Map();
+            for (let i = 0; i < rows.length; i++) {
+                const row = rows[i];
+                const val = sortCol ? getCellValue(row, sortCol) : row[sortBy];
+                if (val == null) {
+                    timestampCache.set(row, NaN);
+                }
+                else {
+                    const t = new Date(String(val)).getTime();
+                    timestampCache.set(row, Number.isNaN(t) ? 0 : t);
+                }
             }
-            if (typeof av === 'number' && typeof bv === 'number')
-                return av === bv ? 0 : av > bv ? dir : -dir;
-            const as = String(av).toLowerCase();
-            const bs = String(bv).toLowerCase();
-            return as === bs ? 0 : as > bs ? dir : -dir;
-        });
+            rows.sort((a, b) => {
+                const at = timestampCache.get(a);
+                const bt = timestampCache.get(b);
+                if (Number.isNaN(at) && Number.isNaN(bt))
+                    return 0;
+                if (Number.isNaN(at))
+                    return -1 * dir;
+                if (Number.isNaN(bt))
+                    return 1 * dir;
+                return at === bt ? 0 : at > bt ? dir : -dir;
+            });
+        }
+        else {
+            rows.sort((a, b) => {
+                if (compare)
+                    return compare(a, b) * dir;
+                const av = sortCol
+                    ? getCellValue(a, sortCol)
+                    : a[sortBy];
+                const bv = sortCol
+                    ? getCellValue(b, sortCol)
+                    : b[sortBy];
+                if (av == null && bv == null)
+                    return 0;
+                if (av == null)
+                    return -1 * dir;
+                if (bv == null)
+                    return 1 * dir;
+                if (typeof av === 'number' && typeof bv === 'number')
+                    return av === bv ? 0 : av > bv ? dir : -dir;
+                const as = String(av).toLowerCase();
+                const bs = String(bv).toLowerCase();
+                return as === bs ? 0 : as > bs ? dir : -dir;
+            });
+        }
     }
     return rows;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alaarab/ogrid-core",
-  "version": "2.0.4",
+  "version": "2.0.6",
   "description": "OGrid core – framework-agnostic types, algorithms, and utilities for OGrid data grids.",
   "main": "dist/esm/index.js",
   "module": "dist/esm/index.js",
@@ -21,5 +21,6 @@
   "license": "MIT",
   "files": ["dist", "README.md", "LICENSE"],
   "engines": { "node": ">=18" },
+  "sideEffects": false,
   "publishConfig": { "access": "public" }
 }