@kayord/ui 4.0.2 → 4.0.3

package/dist/components/custom/data-table/table-url-sync.svelte.js

@@ -1,5 +1,5 @@
 import { untrack } from "svelte";
-import { beforeNavigate } from "$app/navigation";
+import { afterNavigate, beforeNavigate } from "$app/navigation";
 import { useSearchParams } from "runed/kit";
 import { defaultSearchParamSchema } from "./types";
 import { decodeColumnFilters, decodeGlobalFilter, decodePageIndex, decodeSorting, encodeColumnFilters, encodeSorting, } from "./table-search-params";
@@ -43,22 +43,35 @@ export function useTableUrlSync(table, options) {
         table.setPageIndex(decodePageIndex());
     if (enabled.columnFilters)
         table.setColumnFilters(decodeColumnFilters() ?? []);
-    // When navigating with new sort/search/filter params, snap back to the first page
-    // unless the target URL explicitly points at a later page.
+    // When navigating with new sort/search/filter params, snap back to the first
+    // page unless the target URL explicitly points at a later page. Only same-route
+    // query changes count: resetting on a cross-route navigation would mutate state
+    // mid-navigation, and the write-back effect below would then issue a competing
+    // `goto("?…")` that cancels the in-flight navigation (links off the table page
+    // would "do nothing" whenever the table isn't already on page 1).
+    let leavePage = false;
     beforeNavigate((navigation) => {
+        const from = navigation.from?.url;
+        const to = navigation.to?.url;
+        if (!from || !to)
+            return;
+        if (from.pathname !== to.pathname) {
+            leavePage = true;
+            return;
+        }
         if (!enabled.pagination)
             return;
-        const targetPage = Number(navigation.to?.url.searchParams.get("page") ?? "0");
-        const queryChanged = (enabled.sorting &&
-            navigation.from?.url.searchParams.get("sort") != navigation.to?.url.searchParams.get("sort")) ||
-            (enabled.globalFilter &&
-                navigation.from?.url.searchParams.get("search") != navigation.to?.url.searchParams.get("search")) ||
-            (enabled.columnFilters &&
-                navigation.from?.url.searchParams.get("filter") != navigation.to?.url.searchParams.get("filter"));
+        const targetPage = Number(to.searchParams.get("page") ?? "0");
+        const queryChanged = (enabled.sorting && from.searchParams.get("sort") != to.searchParams.get("sort")) ||
+            (enabled.globalFilter && from.searchParams.get("search") != to.searchParams.get("search")) ||
+            (enabled.columnFilters && from.searchParams.get("filter") != to.searchParams.get("filter"));
         if (queryChanged && (!Number.isFinite(targetPage) || targetPage <= 0)) {
             table.resetPageIndex();
         }
     });
+    afterNavigate(() => {
+        leavePage = false;
+    });
     // Write atoms back to the URL. Only the enabled atoms are read (and thus
     // tracked) so the effect doesn't re-run on unrelated state (e.g. row
     // selection), and disabled params are left untouched in the URL.
@@ -68,6 +81,10 @@ export function useTableUrlSync(table, options) {
         const sorting = enabled.sorting ? table.atoms.sorting.get() : undefined;
         const columnFilters = enabled.columnFilters ? table.atoms.columnFilters.get() : undefined;
         untrack(() => {
+            // Skip writes while leaving the route: state mutating mid-navigation
+            // (e.g. a debounced filter landing) must not `goto` over the target URL.
+            if (leavePage)
+                return;
             if (enabled.globalFilter)
                 params.search = search;
             if (enabled.pagination && page !== undefined)

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@kayord/ui",
   "private": false,
-  "version": "4.0.2",
+  "version": "4.0.3",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/kayordDX/ui.git"
@@ -112,19 +112,19 @@
     "runed": "^0.37.1",
     "tailwind-merge": "^3.6.0",
     "tailwind-variants": "^3.3.1",
-    "zod": "4.4.3"
+    "zod": "4.5.2"
   },
   "devDependencies": {
     "@eslint/compat": "^2.1.0",
     "@eslint/js": "^10.0.1",
     "@internationalized/date": "^3.12.3",
-    "@lucide/svelte": "^1.34.0",
+    "@lucide/svelte": "^1.37.0",
     "@sveltejs/adapter-auto": "^7.0.1",
     "@sveltejs/kit": "^2.70.3",
     "@sveltejs/package": "^2.5.8",
     "@sveltejs/vite-plugin-svelte": "7.3.0",
     "@tailwindcss/vite": "^4.3.3",
-    "@tanstack/svelte-table": "^9.1.2",
+    "@tanstack/svelte-table": "^9.2.4",
     "@types/d3-scale": "^4.0.9",
     "@types/d3-shape": "^3.2.0",
     "@vitest/browser": "^4.1.11",
@@ -137,7 +137,7 @@
     "eslint-plugin-svelte": "^3.23.0",
     "formsnap": "^2.0.1",
     "globals": "^17.11.0",
-    "layerchart": "^2.3.0",
+    "layerchart": "^2.3.1",
     "mode-watcher": "^1.1.0",
     "paneforge": "^1.0.2",
     "playwright": "^1.62.1",
@@ -145,8 +145,8 @@
     "prettier-plugin-svelte": "4.1.1",
     "prettier-plugin-tailwindcss": "^0.8.1",
     "publint": "^0.3.24",
-    "shadcn-svelte": "^1.5.0",
-    "svelte": "^5.56.10",
+    "shadcn-svelte": "^1.5.1",
+    "svelte": "^5.57.0",
     "svelte-check": "^4.7.6",
     "svelte-sonner": "^1.2.1",
     "sveltekit-superforms": "^2.30.2",