@vuu-ui/vuu-utils 3.5.0 → 3.5.2

package/package.json

@@ -1,5 +1,5 @@
 {
-  "version": "3.5.0",
+  "version": "3.5.2",
   "author": "heswell",
   "main": "./src/index.js",
   "license": "Apache-2.0",
@@ -15,14 +15,14 @@
   },
   "devDependencies": {
     "@dnd-kit/react": "^0.1.21",
-    "@vuu-ui/vuu-data-types": "3.5.0",
-    "@vuu-ui/vuu-table-types": "3.5.0",
-    "@vuu-ui/vuu-filter-types": "3.5.0",
-    "@vuu-ui/vuu-protocol-types": "3.5.0"
+    "@vuu-ui/vuu-data-types": "3.5.2",
+    "@vuu-ui/vuu-table-types": "3.5.2",
+    "@vuu-ui/vuu-filter-types": "3.5.2",
+    "@vuu-ui/vuu-protocol-types": "3.5.2"
   },
   "peerDependencies": {
     "@internationalized/date": "^3.0.0",
-    "@vuu-ui/vuu-filter-parser": "3.5.0",
+    "@vuu-ui/vuu-filter-parser": "3.5.2",
     "clsx": "^2.0.0",
     "react": "^19.2.3",
     "react-dom": "^19.2.3"

package/src/keyset.js

@@ -1,10 +1,14 @@
 const EMPTY = [];
 class KeySet {
     keys = new Map();
+    freeKeys = [];
     nextKeyValue = 0;
     range;
     constructor(range){
-        this.range = range;
+        this.range = {
+            from: range.from,
+            to: range.to
+        };
         this.init(range);
     }
     next(free = EMPTY) {
@@ -13,6 +17,7 @@ class KeySet {
     }
     init({ from, to }) {
         this.keys.clear();
+        this.freeKeys.length = 0;
         this.nextKeyValue = 0;
         for(let rowIndex = from; rowIndex < to; rowIndex++){
             const nextKeyValue = this.next();
@@ -20,23 +25,24 @@ class KeySet {
         }
         return true;
     }
-    reset(range) {
-        const { from, to } = range;
-        const newSize = to - from;
-        const currentSize = this.range.to - this.range.from;
-        this.range = range;
-        if (currentSize > newSize) return this.init(range);
-        const freeKeys = [];
+    reset({ from, to }) {
+        this.range = {
+            from,
+            to
+        };
+        const { freeKeys } = this;
         this.keys.forEach((keyValue, rowIndex)=>{
             if (rowIndex < from || rowIndex >= to) {
                 freeKeys.push(keyValue);
                 this.keys.delete(rowIndex);
             }
         });
+        let freeKeyIndex = 0;
         for(let rowIndex = from; rowIndex < to; rowIndex++)if (!this.keys.has(rowIndex)) {
-            const nextKeyValue = this.next(freeKeys);
+            const nextKeyValue = freeKeyIndex < freeKeys.length ? freeKeys[freeKeyIndex++] : this.nextKeyValue++;
             this.keys.set(rowIndex, nextKeyValue);
         }
+        if (freeKeyIndex > 0) freeKeys.splice(0, freeKeyIndex);
         return false;
     }
     keyFor(rowIndex) {

package/types/keyset.d.ts

@@ -5,12 +5,22 @@ export interface IKeySet {
 }
 export declare class KeySet implements IKeySet {
     private keys;
+    private freeKeys;
     private nextKeyValue;
     private range;
     constructor(range: VuuRange);
     next(free?: number[]): number;
     private init;
-    reset(range: VuuRange): boolean;
+    /**
+     * Keys assigned to rows that remain within range are never changed, keys
+     * released by rows leaving the range are recycled for rows entering the
+     * range. Keys are never re-sequenced, clients may retain rows (and their
+     * keys) that remain within range, so re-assigning keys to those rows would
+     * risk duplicate keys on the client.
+     * Note: range is copied, callers may mutate the range object they pass.
+     * Returns false, keys are never resequenced.
+     */
+    reset({ from, to }: VuuRange): boolean;
     keyFor(rowIndex: number): number;
     toDebugString(): string;
 }