@pylonsync/react 0.4.11 → 0.4.12

package/dist/Link.d.ts

@@ -3,7 +3,8 @@ declare global {
     interface Window {
         __pylon?: {
             prefetch: (href: string, opts?: {
-                /** Also fetch the page payload (costs a server render — hover only). */
+                /** Also fetch the page payload. Costs a server render, so only on an
+                 *  intent signal: hover, touch, or focus. */
                 document?: boolean;
             }) => Promise<void>;
             navigate: (href: string, opts?: {

package/package.json

@@ -3,7 +3,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.4.11",
+  "version": "0.4.12",
   "type": "module",
   "main": "./src/index.ts",
   "types": "./dist/index.d.ts",
@@ -14,8 +14,8 @@
     "prepack": "bun run build"
   },
   "dependencies": {
-    "@pylonsync/sdk": "0.4.11",
-    "@pylonsync/sync": "0.4.11"
+    "@pylonsync/sdk": "0.4.12",
+    "@pylonsync/sync": "0.4.12"
   },
   "peerDependencies": {
     "react": ">=19.0.0"

package/src/Link.tsx

@@ -11,9 +11,11 @@
 //   - Prefetches in two stages, because the two halves cost very
 //     different things. On viewport entry (IntersectionObserver) the
 //     runtime modulepreloads the destination's client chunks — static,
-//     immutable, cacheable. On hover/touch it also fetches the page
-//     itself, which costs a server render, so that waits for a real
-//     signal of intent rather than firing for a screenful of links.
+//     immutable, cacheable. On hover, touch, or focus it also fetches
+//     the page itself, which costs a server render, so that waits for a
+//     real signal of intent rather than firing for a screenful of links.
+//     Focus is included so a keyboard user tabbing to a link gets the
+//     same warm path a mouse user gets.
 //   - Modifier keys (cmd/ctrl/shift/alt) or middle-click fall
 //     through to the browser's default behavior (open in new tab,
 //     etc.) — matches Next.js semantics.
@@ -32,7 +34,8 @@ declare global {
       prefetch: (
         href: string,
         opts?: {
-          /** Also fetch the page payload (costs a server render — hover only). */
+          /** Also fetch the page payload. Costs a server render, so only on an
+           *  intent signal: hover, touch, or focus. */
           document?: boolean;
         },
       ) => Promise<void>;
@@ -153,11 +156,17 @@ export function Link({
 
     el.addEventListener("mouseenter", warmPage);
     el.addEventListener("touchstart", warmPage, { passive: true });
+    // Focus is the keyboard's version of hover, and a stronger signal: a
+    // cursor slides across links it isn't aiming at, but nobody tabs onto one
+    // idly. Without it, tabbing to a link and pressing Enter took the cold
+    // path while a mouse user got the warm one — the wrong way round.
+    el.addEventListener("focus", warmPage);
 
     return () => {
       io?.disconnect();
       el.removeEventListener("mouseenter", warmPage);
       el.removeEventListener("touchstart", warmPage);
+      el.removeEventListener("focus", warmPage);
     };
   }, [href, prefetch]);
 

package/src/useRouter.test.ts

@@ -48,10 +48,16 @@ describe("notFound() — branded for the SSR not-found boundary", () => {
 describe("redirect() — client-side replace navigation", () => {
   let calls: Array<{ href: string; opts?: unknown }>;
   const hadWindow = "window" in globalThis;
+  // The suite preloads happy-dom, so a real window usually EXISTS here. Put it
+  // back rather than leaving the stub in place: bun isolates globals per test
+  // file today, so nothing observable depends on this, but a test that
+  // replaces a global and never restores it is one isolation change away from
+  // breaking every file that mounts.
+  const originalWindow = (globalThis as any).window;
 
   beforeEach(() => {
     calls = [];
-    // bun:test has no DOM — stand up a minimal window the module can see.
+    // Stand up a minimal window the module can see, without assuming a DOM.
     (globalThis as any).window = {
       __pylon: {
         prefetch: async () => {},
@@ -63,8 +69,11 @@ describe("redirect() — client-side replace navigation", () => {
   });
 
   afterEach(() => {
-    if (hadWindow) return;
-    delete (globalThis as any).window;
+    if (hadWindow) {
+      (globalThis as any).window = originalWindow;
+    } else {
+      delete (globalThis as any).window;
+    }
   });
 
   test("delegates to __pylon.navigate with replace:true (no history push)", () => {