support empty array/string to mean no revalidation

Author: ryansolid

.changeset/hip-weeks-rhyme.md

@@ -0,0 +1,5 @@
+---
+"@solidjs/router": patch
+---
+
+support empty array/string to mean no revalidation

src/data/cache.ts

@@ -222,7 +222,7 @@ cache.clear = () => getCache().clear();
 
 function matchKey(key: string, keys: string[]) {
   for (let k of keys) {
-    if (key.startsWith(k)) return true;
+    if (k && key.startsWith(k)) return true;
   }
   return false;
 }

src/data/response.ts

@@ -14,7 +14,7 @@ export function redirect(url: string, init: number | RouterResponseInit = 302) {
 
   const headers = new Headers(responseInit.headers);
   headers.set("Location", url);
-  revalidate && headers.set("X-Revalidate", revalidate.toString());
+  revalidate !== undefined && headers.set("X-Revalidate", revalidate.toString());
 
   const response = new Response(null, {
     ...responseInit,
@@ -27,7 +27,7 @@ export function redirect(url: string, init: number | RouterResponseInit = 302) {
 export function reload(init: RouterResponseInit = {}) {
   const { revalidate, ...responseInit } = init;
   const headers = new Headers(responseInit.headers);
-  revalidate && headers.set("X-Revalidate", revalidate.toString());
+  revalidate !== undefined && headers.set("X-Revalidate", revalidate.toString());
 
   return new Response(null, {
     ...responseInit,
@@ -38,7 +38,7 @@ export function reload(init: RouterResponseInit = {}) {
 export function json<T>(data: T, init: RouterResponseInit = {}) {
   const { revalidate, ...responseInit } = init;
   const headers = new Headers(responseInit.headers);
-  revalidate && headers.set("X-Revalidate", revalidate.toString());
+  revalidate !== undefined && headers.set("X-Revalidate", revalidate.toString());
   headers.set("Content-Type", "application/json");
 
   const response = new Response(JSON.stringify(data), {