@decocms/tanstack 7.77.7 → 7.77.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@decocms/tanstack",
-  "version": "7.77.7",
+  "version": "7.77.9",
   "type": "module",
   "description": "Deco framework binding for TanStack Start + Cloudflare Workers",
   "repository": {
@@ -27,9 +27,9 @@
     "lint:unused": "knip"
   },
   "dependencies": {
-    "@decocms/blocks": "7.77.7",
-    "@decocms/blocks-admin": "7.77.7",
-    "@decocms/blocks-cli": "7.77.7",
+    "@decocms/blocks": "7.77.9",
+    "@decocms/blocks-admin": "7.77.9",
+    "@decocms/blocks-cli": "7.77.9",
     "@deco-cx/warp-node": "^0.3.16",
     "fast-json-patch": "^3.1.0",
     "ws": "^8.18.0"

package/src/sdk/router.test.ts

@@ -0,0 +1,34 @@
+import { describe, expect, it } from "vitest";
+import { decoParseSearch, decoStringifySearch } from "./router";
+
+describe("decoStringifySearch ∘ decoParseSearch is a fixed point", () => {
+  // TanStack Router redirects (307) on the server when
+  // stringifySearch(parseSearch(search)) differs from the request's search —
+  // every query string that round-trips unchanged is a request that never
+  // redirects.
+  it.each([
+    "?q=camisa&sort=",
+    "?utm_source=",
+    "?a=&b=",
+    "?q=camisa",
+    "?filter.brand=Nike&filter.brand=Adidas",
+    // (`%20` is re-encoded as `+` by URLSearchParams — a pre-existing
+    // normalization, out of this fix's scope.)
+    "?q=a+b&page=2",
+  ])("round-trips %s unchanged", (search) => {
+    expect(decoStringifySearch(decoParseSearch(search))).toBe(search);
+  });
+
+  it("parses a value-less param to an empty string", () => {
+    expect(decoParseSearch("?q=camisa&sort=")).toEqual({ q: "camisa", sort: "" });
+  });
+
+  it("still drops undefined and null (how navigate() removes a param)", () => {
+    expect(decoStringifySearch({ q: "camisa", sort: undefined, page: null })).toBe("?q=camisa");
+    expect(decoStringifySearch({ q: undefined })).toBe("");
+  });
+
+  it("keeps an explicit empty string as key= and arrays as repeated keys", () => {
+    expect(decoStringifySearch({ sort: "", f: ["a", "b"] })).toBe("?sort=&f=a&f=b");
+  });
+});

package/src/sdk/router.ts

@@ -30,10 +30,22 @@ export const decoParseSearch: SearchParser = (searchStr) => {
   return result;
 };
 
+/**
+ * Serializes the router's search object back to a query string.
+ *
+ * Must be a fixed point of {@link decoParseSearch}: on the server, TanStack
+ * Router compares the request's href with `stringifySearch(parseSearch(...))`
+ * (`router.beforeLoad`) and answers a 307 to the normalized URL when they
+ * differ. A value-less parameter (`?sort=`, `?utm_source=`) parses to `""`,
+ * so dropping `""` here turned every such request into a redirect that
+ * Fresh — and TanStack's own default serializer, which only drops
+ * `undefined` — never issued. `undefined`/`null` still remove a param, which
+ * is how `navigate({ search: { q: undefined } })` clears one.
+ */
 export const decoStringifySearch: SearchSerializer = (search) => {
   const params = new URLSearchParams();
   for (const [key, value] of Object.entries(search)) {
-    if (value === undefined || value === null || value === "") continue;
+    if (value === undefined || value === null) continue;
     if (Array.isArray(value)) {
       for (const v of value) params.append(key, String(v));
     } else {