@qiaopeng/tanstack-query-plus 0.2.7 → 0.2.8

package/dist/utils/consistency.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"consistency.d.ts","sourceRoot":"","sources":["../../src/utils/consistency.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAC;AACnE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAIvD,wBAAgB,mBAAmB,CAAC,GAAG,EAAE,QAAQ,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAA2O;AACxT,MAAM,MAAM,gBAAgB,GAAG;IAAE,OAAO,CAAC,EAAE,MAAM,CAAC;IAAC,YAAY,CAAC,EAAE,CAAC,IAAI,EAAE,OAAO,KAAK;QAAE,KAAK,EAAE,YAAY,EAAE,CAAC;QAAC,KAAK,CAAC,EAAE,MAAM,CAAA;KAAE,GAAG,IAAI,CAAC;IAAC,SAAS,CAAC,EAAE,CAAC,GAAG,EAAE,OAAO,EAAE,KAAK,EAAE,YAAY,EAAE,EAAE,KAAK,CAAC,EAAE,MAAM,KAAK,OAAO,CAAC;IAAC,OAAO,CAAC,EAAE,MAAM,CAAC;IAAC,mBAAmB,CAAC,EAAE,KAAK,CAAC,QAAQ,GAAG,QAAQ,CAAC,CAAA;CAAE,CAAC;AAC1R,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,OAAO,GAAG;IAAE,KAAK,EAAE,YAAY,EAAE,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CASnG;AACD,wBAAgB,gBAAgB,CAAC,GAAG,EAAE,OAAO,EAAE,KAAK,EAAE,YAAY,EAAE,EAAE,KAAK,CAAC,EAAE,MAAM,GAAG,OAAO,CAe7F;AACD,eAAO,MAAM,mBAAmB,EAAE,gBAA4J,CAAC;AAC/L,wBAAgB,sBAAsB,CAAC,WAAW,EAAE,WAAW,EAAE,YAAY,EAAE,QAAQ,EAAE,GAAG,EAAE,gBAAgB,EAAE,SAAS,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,CAAC,YAAY,CAAC,GAAG,IAAI,CAqBvK;AAED,wBAAgB,gCAAgC,CAC9C,WAAW,EAAE,WAAW,EACxB,YAAY,EAAE,QAAQ,EACtB,GAAG,EAAE,gBAAgB,EACrB,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,OAAO,CAAC,YAAY,CAAC,GAC7B,KAAK,CAAC;IAAE,QAAQ,EAAE,QAAQ,CAAC;IAAC,YAAY,EAAE,OAAO,CAAA;CAAE,CAAC,CA6CtD"}
+{"version":3,"file":"consistency.d.ts","sourceRoot":"","sources":["../../src/utils/consistency.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,WAAW,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAC;AACnE,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAiDvD,wBAAgB,mBAAmB,CAAC,GAAG,EAAE,QAAQ,EAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,CAgB5E;AAED,MAAM,MAAM,gBAAgB,GAAG;IAC7B,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,YAAY,CAAC,EAAE,CAAC,IAAI,EAAE,OAAO,KAAK;QAAE,KAAK,EAAE,YAAY,EAAE,CAAC;QAAC,KAAK,CAAC,EAAE,MAAM,CAAA;KAAE,GAAG,IAAI,CAAC;IACnF,SAAS,CAAC,EAAE,CAAC,GAAG,EAAE,OAAO,EAAE,KAAK,EAAE,YAAY,EAAE,EAAE,KAAK,CAAC,EAAE,MAAM,KAAK,OAAO,CAAC;IAC7E,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,mBAAmB,CAAC,EAAE,KAAK,CAAC,QAAQ,GAAG,QAAQ,CAAC,CAAC;CAClD,CAAC;AAEF,wBAAgB,mBAAmB,CAAC,IAAI,EAAE,OAAO,GAAG;IAAE,KAAK,EAAE,YAAY,EAAE,CAAC;IAAC,KAAK,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CAiBnG;AAED,wBAAgB,gBAAgB,CAAC,GAAG,EAAE,OAAO,EAAE,KAAK,EAAE,YAAY,EAAE,EAAE,KAAK,CAAC,EAAE,MAAM,GAAG,OAAO,CAe7F;AAED,eAAO,MAAM,mBAAmB,EAAE,gBAMjC,CAAC;AAEF,wBAAgB,sBAAsB,CACpC,WAAW,EAAE,WAAW,EACxB,YAAY,EAAE,QAAQ,EACtB,GAAG,EAAE,gBAAgB,EACrB,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,OAAO,CAAC,YAAY,CAAC,GAC7B,IAAI,CAiDN;AAED,wBAAgB,gCAAgC,CAC9C,WAAW,EAAE,WAAW,EACxB,YAAY,EAAE,QAAQ,EACtB,GAAG,EAAE,gBAAgB,EACrB,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,OAAO,CAAC,YAAY,CAAC,GAC7B,KAAK,CAAC;IAAE,QAAQ,EAAE,QAAQ,CAAC;IAAC,YAAY,EAAE,OAAO,CAAA;CAAE,CAAC,CAkDtD"}

package/dist/utils/consistency.js

@@ -1,21 +1,65 @@
 import { listUpdater } from "./optimisticUtils.js";
-function ensureArray(key) { return Array.isArray(key) ? key : [key]; }
-function deepEqual(a, b) { if (a === b)
-    return true; if (typeof a === "object" && a !== null && typeof b === "object" && b !== null) {
-    try {
-        return JSON.stringify(a) === JSON.stringify(b);
+function ensureArray(key) {
+    return Array.isArray(key) ? key : [key];
+}
+/**
+ * Robust deep equality check that handles object key order and common types
+ */
+function deepEqual(a, b) {
+    if (a === b)
+        return true;
+    if (a && b && typeof a === "object" && typeof b === "object") {
+        if (Array.isArray(a)) {
+            if (!Array.isArray(b) || a.length !== b.length)
+                return false;
+            for (let i = 0; i < a.length; i++) {
+                if (!deepEqual(a[i], b[i]))
+                    return false;
+            }
+            return true;
+        }
+        // Handle plain objects
+        const keysA = Object.keys(a);
+        const keysB = Object.keys(b);
+        if (keysA.length !== keysB.length)
+            return false;
+        // Check if all keys in A exist in B and values are deep equal
+        // Key order doesn't matter here because we look up by key
+        for (const key of keysA) {
+            if (!Object.prototype.hasOwnProperty.call(b, key))
+                return false;
+            if (!deepEqual(a[key], b[key]))
+                return false;
+        }
+        return true;
     }
-    catch {
+    return false;
+}
+/**
+ * Loose equality check for IDs (string vs number)
+ */
+function idsAreEqual(a, b) {
+    if (a === b)
+        return true;
+    if (a === undefined || a === null || b === undefined || b === null)
         return false;
-    }
-} return false; }
-export function startsWithKeyPrefix(key, prefix) { const k = ensureArray(key); const p = ensureArray(prefix); if (p.length > k.length)
-    return false; for (let i = 0; i < p.length; i++) {
-    const a = k[i];
-    const b = p[i];
-    if (!(a === b || deepEqual(a, b)))
+    return String(a) === String(b);
+}
+export function startsWithKeyPrefix(key, prefix) {
+    const k = ensureArray(key);
+    const p = ensureArray(prefix);
+    if (p.length > k.length)
         return false;
-} return true; }
+    for (let i = 0; i < p.length; i++) {
+        const a = k[i];
+        const b = p[i];
+        // Check for equality using robust deepEqual
+        if (!deepEqual(a, b)) {
+            return false;
+        }
+    }
+    return true;
+}
 export function defaultListSelector(data) {
     if (!data)
         return null;
@@ -24,9 +68,15 @@ export function defaultListSelector(data) {
     if (typeof data === "object") {
         const obj = data;
         if (Array.isArray(obj.items))
-            return { items: obj.items, total: typeof obj.total === "number" ? obj.total : undefined };
+            return {
+                items: obj.items,
+                total: typeof obj.total === "number" ? obj.total : undefined
+            };
         if (Array.isArray(obj.Rows))
-            return { items: obj.Rows, total: typeof obj.Total === "number" ? obj.Total : undefined };
+            return {
+                items: obj.Rows,
+                total: typeof obj.Total === "number" ? obj.Total : undefined
+            };
     }
     return null;
 }
@@ -48,7 +98,13 @@ export function defaultWriteBack(old, items, total) {
     }
     return { items, total };
 }
-export const DEFAULT_FAMILY_SYNC = { idField: "id", listSelector: defaultListSelector, writeBack: defaultWriteBack, maxKeys: 50, enableForOperations: ["update", "delete"] };
+export const DEFAULT_FAMILY_SYNC = {
+    idField: "id",
+    listSelector: defaultListSelector,
+    writeBack: defaultWriteBack,
+    maxKeys: 50,
+    enableForOperations: ["update", "delete"]
+};
 export function syncEntityAcrossFamily(queryClient, familyPrefix, cfg, operation, payload) {
     const enabledOps = cfg.enableForOperations ?? ["update", "delete"];
     const op = String(operation).toLowerCase();
@@ -67,10 +123,28 @@ export function syncEntityAcrossFamily(queryClient, familyPrefix, cfg, operation
         const idValue = payload?.[idField];
         let items = picked.items;
         if (op === "update" && idValue !== undefined) {
-            items = listUpdater.update(items, { ...payload, id: idValue });
+            // Use custom listUpdater logic but ensure ID matching is robust if listUpdater uses strict equality
+            // Since listUpdater is imported, we should ideally modify it too or handle it here.
+            // For now, let's modify how we find the item index if we were doing it manually, 
+            // but listUpdater is a black box here.
+            // Assuming listUpdater uses strict equality, we might have a problem if we pass '1' and item has 1.
+            // But listUpdater.update usually takes the whole object.
+            // To be safe, we should ensure the payload ID matches the type of the item ID in the list if possible.
+            // Or, we rely on listUpdater being robust.
+            // Let's look at listUpdater implementation later. For now, let's assume we need to fix the ID type in payload if possible.
+            // Actually, the safer way is to find the item using loose equality, get its strict ID, and use that in payload.
+            const existingItem = items.find(item => idsAreEqual(item[idField], idValue));
+            if (existingItem) {
+                const strictId = existingItem[idField];
+                items = listUpdater.update(items, { ...payload, [idField]: strictId });
+            }
         }
         else if (op === "delete" && idValue !== undefined) {
-            items = listUpdater.remove(items, idValue);
+            const existingItem = items.find(item => idsAreEqual(item[idField], idValue));
+            if (existingItem) {
+                const strictId = existingItem[idField];
+                items = listUpdater.remove(items, strictId);
+            }
         }
         const total = typeof picked.total === "number" && op === "delete" ? Math.max(0, picked.total - 1) : picked.total;
         const next = (cfg.writeBack ?? defaultWriteBack)(old, items, total);
@@ -97,16 +171,20 @@ export function syncEntityAcrossFamilyOptimistic(queryClient, familyPrefix, cfg,
         let items = picked.items;
         let shouldUpdate = false;
         if (op === "update" && idValue !== undefined) {
-            const exists = items.some(item => item[idField] === idValue);
-            if (exists) {
-                items = listUpdater.update(items, { ...payload, id: idValue });
+            // Robust ID check
+            const existingItem = items.find(item => idsAreEqual(item[idField], idValue));
+            if (existingItem) {
+                const strictId = existingItem[idField];
+                // Ensure payload uses the correct ID type from the store
+                items = listUpdater.update(items, { ...payload, [idField]: strictId });
                 shouldUpdate = true;
             }
         }
         else if (op === "delete" && idValue !== undefined) {
-            const exists = items.some(item => item[idField] === idValue);
-            if (exists) {
-                items = listUpdater.remove(items, idValue);
+            const existingItem = items.find(item => idsAreEqual(item[idField], idValue));
+            if (existingItem) {
+                const strictId = existingItem[idField];
+                items = listUpdater.remove(items, strictId);
                 shouldUpdate = true;
             }
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@qiaopeng/tanstack-query-plus",
-  "version": "0.2.7",
+  "version": "0.2.8",
   "description": "Enhanced TanStack Query toolkit: defaults, hooks, persistence, offline, utils",
   "author": "qiaopeng",
   "license": "MIT",