@kasperrt/smolter 0.3.2 → 0.3.4

package/.generated/client/deployment-check.ts

@@ -36,27 +36,24 @@ export function deploymentCheck(options: DeploymentOptions) {
     return pending;
   }
 
-  async function reload(target: URL) {
-    // If storage is unavailable, do not risk reloading the same cached document forever.
-    const allowed = await Promise.resolve()
+  function reload(target: URL, current: () => boolean = () => true) {
+    // Validate and claim recovery in the same task: stale checks must not consume
+    // the marker or reload after a newer navigation or mutation has started.
+    return Promise.resolve()
       .then(() => {
-        if (sessionStorage.getItem(key)) return false;
+        if (stopped || !current() || sessionStorage.getItem(key)) return false;
 
         sessionStorage.setItem(key, "1");
 
+        if (options.reload) options.reload(target);
+        if (!options.reload) {
+          if (target.href === location.href) location.reload();
+          if (target.href !== location.href) location.assign(target.href);
+        }
+
         return true;
       })
       .catch(() => false);
-
-    if (allowed && !stopped) {
-      if (options?.reload) options.reload(target);
-      if (!options?.reload) {
-        if (target.href === location.href) location.reload();
-        if (target.href !== location.href) location.assign(target.href);
-      }
-    }
-
-    return allowed && !stopped;
   }
 
   return {

package/.generated/client/deployment.ts

@@ -20,18 +20,21 @@ export function watchDeployment(
 
   const check = deploymentCheck(options);
   const { changed, reload } = check;
+  const read = router.getSnapshot;
   const navigate = router.navigate;
   const dispose = router.dispose;
   let sequence = 0;
   let stopped = false;
   let replay = false;
-  let destination = new URL(location.href);
+  const valid = (current: number) => current === sequence && !stopped;
+  const unsubscribe = router.subscribe(() => {
+    if (read().state === "submitting") sequence++;
+  });
 
   const guarded: NavigationRouter["navigate"] = async (href, operation) => {
     const target = new URL(href, router.getSnapshot().url);
     const currentURL = new URL(router.getSnapshot().url);
     const current = ++sequence;
-    destination = target;
 
     if (
       target.origin === location.origin &&
@@ -39,7 +42,8 @@ export function watchDeployment(
         target.search !== currentURL.search) &&
       (await changed())
     ) {
-      if (current !== sequence || stopped || (await reload(target))) return;
+      if (!valid(current) || (await reload(target, () => valid(current))))
+        return;
     }
 
     if (current === sequence && !stopped) await navigate(href, operation);
@@ -49,7 +53,8 @@ export function watchDeployment(
     if (replay) return;
 
     const target = new URL(location.href);
-    const previous = new URL(router.getSnapshot().url);
+    const previous = new URL(read().url);
+    const current = ++sequence;
 
     if (
       target.pathname === previous.pathname &&
@@ -58,11 +63,10 @@ export function watchDeployment(
       return;
 
     event.stopImmediatePropagation();
-    const current = ++sequence;
-    destination = target;
     void changed().then(async (updated) => {
-      if (stopped || current !== sequence) return;
-      if (updated && (await reload(target))) return;
+      if (!valid(current)) return;
+      if (updated && (await reload(target, () => valid(current)))) return;
+      if (!valid(current)) return;
 
       replay = true;
       window.dispatchEvent(
@@ -73,12 +77,21 @@ export function watchDeployment(
   };
 
   const preloadError = () => {
-    if (router.getSnapshot().state === "submitting") return;
+    // Intent prefetch is speculative: it must not reload the current page or
+    // consume the recovery marker before a click records its destination.
+    if (read().state !== "loading") return;
 
     const current = sequence;
-    const target = destination;
+    const target = new URL(read().url);
     void changed().then((updated) => {
-      if (updated && current === sequence && !stopped) return reload(target);
+      if (updated)
+        return reload(
+          target,
+          () =>
+            valid(current) &&
+            read().state !== "submitting" &&
+            read().url === target.href,
+        );
     });
   };
 
@@ -86,6 +99,7 @@ export function watchDeployment(
     stopped = true;
     sequence++;
     check.dispose();
+    unsubscribe();
     window.removeEventListener("popstate", pop, true);
     window.removeEventListener("vite:preloadError", preloadError);
 

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 0.3.4
+
+- Cancel stale deployment checks when newer Back/Forward operations return to the current route, including hash changes.
+- Recover missing action-redirect chunks at the committed destination, preserving query/hash and avoiding mutation replay.
+- Invalidate pending chunk recovery when a mutation starts; validate recovery ownership before claiming the reload marker.
+- Expand production-browser coverage for scoped application paths, separate CDN asset/manifest origins, overlapping navigation, Chromium offline/online recovery and history races. Manual and file-route HMR remain covered.
+
+## 0.3.3
+
+- Ignore idle intent-prefetch chunk failures in deployment recovery so the next click retains its destination and reload marker.
+- Keep active-navigation chunk recovery and latest-navigation wins behavior, with unit and production-browser regressions.
+
 ## 0.3.2
 
 - Configure transport endpoints consistently across browser handlers, file routes, Express and development middleware for multiple apps sharing one origin.

package/README.md

@@ -1,8 +1,8 @@
 # smolter _
 
 <!-- bundlejs:start -->
-[![react gzip size](https://deno.bundlejs.com/?q=%40kasperrt%2Fsmolter%400.3.2%2Freact&treeshake=%5B*%5D&config=%7B%22compression%22%3A%22gzip%22%2C%22esbuild%22%3A%7B%22external%22%3A%5B%22react%22%2C%22react%2F*%22%5D%7D%7D&badge)](https://bundlejs.com/?q=%40kasperrt%2Fsmolter%400.3.2%2Freact&treeshake=%5B*%5D&config=%7B%22compression%22%3A%22gzip%22%2C%22esbuild%22%3A%7B%22external%22%3A%5B%22react%22%2C%22react%2F*%22%5D%7D%7D&bundle)
-[![preact gzip size](https://deno.bundlejs.com/?q=%40kasperrt%2Fsmolter%400.3.2%2Fpreact&treeshake=%5B*%5D&config=%7B%22compression%22%3A%22gzip%22%2C%22esbuild%22%3A%7B%22external%22%3A%5B%22preact%22%2C%22preact%2F*%22%5D%7D%7D&badge)](https://bundlejs.com/?q=%40kasperrt%2Fsmolter%400.3.2%2Fpreact&treeshake=%5B*%5D&config=%7B%22compression%22%3A%22gzip%22%2C%22esbuild%22%3A%7B%22external%22%3A%5B%22preact%22%2C%22preact%2F*%22%5D%7D%7D&bundle)
+[![react gzip size](https://deno.bundlejs.com/?q=%40kasperrt%2Fsmolter%400.3.4%2Freact&treeshake=%5B*%5D&config=%7B%22compression%22%3A%22gzip%22%2C%22esbuild%22%3A%7B%22external%22%3A%5B%22react%22%2C%22react%2F*%22%5D%7D%7D&badge)](https://bundlejs.com/?q=%40kasperrt%2Fsmolter%400.3.4%2Freact&treeshake=%5B*%5D&config=%7B%22compression%22%3A%22gzip%22%2C%22esbuild%22%3A%7B%22external%22%3A%5B%22react%22%2C%22react%2F*%22%5D%7D%7D&bundle)
+[![preact gzip size](https://deno.bundlejs.com/?q=%40kasperrt%2Fsmolter%400.3.4%2Fpreact&treeshake=%5B*%5D&config=%7B%22compression%22%3A%22gzip%22%2C%22esbuild%22%3A%7B%22external%22%3A%5B%22preact%22%2C%22preact%2F*%22%5D%7D%7D&badge)](https://bundlejs.com/?q=%40kasperrt%2Fsmolter%400.3.4%2Fpreact&treeshake=%5B*%5D&config=%7B%22compression%22%3A%22gzip%22%2C%22esbuild%22%3A%7B%22external%22%3A%5B%22preact%22%2C%22preact%2F*%22%5D%7D%7D&bundle)
 <!-- bundlejs:end -->
 
 A typed data router for React and Preact. **Zero runtime dependencies.**
@@ -605,10 +605,10 @@ a claim of identical features.
 | `Preact: routing-only named exports` | 1405 |
 | `@kasperrt/smolter/react` | 2948 |
 | `@kasperrt/smolter/preact` | 4842 |
-| `@kasperrt/smolter/react + @kasperrt/smolter/react/client` | 4046 |
-| `@kasperrt/smolter/preact + server transport` | 5886 |
-| `@kasperrt/smolter/preact + client + document (application without lazy routes)` | 8418 |
-| `@kasperrt/smolter/preact + client + document + lazy-route (file-routed application)` | 8704 |
+| `@kasperrt/smolter/react + @kasperrt/smolter/react/client` | 4104 |
+| `@kasperrt/smolter/preact + server transport` | 5950 |
+| `@kasperrt/smolter/preact + client + document (application without lazy routes)` | 8478 |
+| `@kasperrt/smolter/preact + client + document + lazy-route (file-routed application)` | 8761 |
 | `@kasperrt/smolter/react/data (optional resource API)` | 2962 |
 | `React: common routing/data exports` | 2761 |
 | `Preact: common routing/data exports` | 3046 |

package/dist/client/deployment-check.d.ts

@@ -1,6 +1,6 @@
 import type { DeploymentOptions } from "./deployment.ts";
 export declare function deploymentCheck(options: DeploymentOptions): {
     changed: () => Promise<boolean>;
-    reload: (target: URL) => Promise<boolean>;
+    reload: (target: URL, current?: () => boolean) => Promise<boolean>;
     dispose(): void;
 };

package/dist/client/deployment-check.js

@@ -23,20 +23,17 @@ function deploymentCheck(options) {
 		});
 		return pending;
 	}
-	async function reload(target) {
-		const allowed = await Promise.resolve().then(() => {
-			if (sessionStorage.getItem(key)) return false;
+	function reload(target, current = () => true) {
+		return Promise.resolve().then(() => {
+			if (stopped || !current() || sessionStorage.getItem(key)) return false;
 			sessionStorage.setItem(key, "1");
-			return true;
-		}).catch(() => false);
-		if (allowed && !stopped) {
-			if (options?.reload) options.reload(target);
-			if (!options?.reload) {
+			if (options.reload) options.reload(target);
+			if (!options.reload) {
 				if (target.href === location.href) location.reload();
 				if (target.href !== location.href) location.assign(target.href);
 			}
-		}
-		return allowed && !stopped;
+			return true;
+		}).catch(() => false);
 	}
 	return {
 		changed,

package/dist/client/deployment.js

@@ -5,50 +5,54 @@ function watchDeployment(router, options) {
 	if (!options || typeof window === "undefined") return () => {};
 	const check = deploymentCheck(options);
 	const { changed, reload } = check;
+	const read = router.getSnapshot;
 	const navigate = router.navigate;
 	const dispose = router.dispose;
 	let sequence = 0;
 	let stopped = false;
 	let replay = false;
-	let destination = new URL(location.href);
+	const valid = (current) => current === sequence && !stopped;
+	const unsubscribe = router.subscribe(() => {
+		if (read().state === "submitting") sequence++;
+	});
 	const guarded = async (href, operation) => {
 		const target = new URL(href, router.getSnapshot().url);
 		const currentURL = new URL(router.getSnapshot().url);
 		const current = ++sequence;
-		destination = target;
 		if (target.origin === location.origin && (target.pathname !== currentURL.pathname || target.search !== currentURL.search) && await changed()) {
-			if (current !== sequence || stopped || await reload(target)) return;
+			if (!valid(current) || await reload(target, () => valid(current))) return;
 		}
 		if (current === sequence && !stopped) await navigate(href, operation);
 	};
 	const pop = (event) => {
 		if (replay) return;
 		const target = new URL(location.href);
-		const previous = new URL(router.getSnapshot().url);
+		const previous = new URL(read().url);
+		const current = ++sequence;
 		if (target.pathname === previous.pathname && target.search === previous.search) return;
 		event.stopImmediatePropagation();
-		const current = ++sequence;
-		destination = target;
 		changed().then(async (updated) => {
-			if (stopped || current !== sequence) return;
-			if (updated && await reload(target)) return;
+			if (!valid(current)) return;
+			if (updated && await reload(target, () => valid(current))) return;
+			if (!valid(current)) return;
 			replay = true;
 			window.dispatchEvent(new PopStateEvent("popstate", { state: event.state }));
 			replay = false;
 		});
 	};
 	const preloadError = () => {
-		if (router.getSnapshot().state === "submitting") return;
+		if (read().state !== "loading") return;
 		const current = sequence;
-		const target = destination;
+		const target = new URL(read().url);
 		changed().then((updated) => {
-			if (updated && current === sequence && !stopped) return reload(target);
+			if (updated) return reload(target, () => valid(current) && read().state !== "submitting" && read().url === target.href);
 		});
 	};
 	function cleanup() {
 		stopped = true;
 		sequence++;
 		check.dispose();
+		unsubscribe();
 		window.removeEventListener("popstate", pop, true);
 		window.removeEventListener("vite:preloadError", preloadError);
 		if (router.navigate === guarded) router.navigate = navigate;

package/jsr.json

@@ -1,6 +1,6 @@
 {
   "name": "@kasperrt/smolter",
-  "version": "0.3.2",
+  "version": "0.3.4",
   "exports": {
     "./react": "./.generated/react/index.ts",
     "./preact": "./.generated/preact/index.ts",

package/package.json

@@ -56,7 +56,7 @@
     "wouter": "3.13.0",
     "wouter-preact": "3.13.0"
   },
-  "version": "0.3.2",
+  "version": "0.3.4",
   "description": "A tiny typed data router for React and Preact.",
   "sideEffects": false,
   "license": "MIT",