fix: update history only after partials apply successfully

bartlomieju · claude · bartlomieju · commit 931b8d6b5fac · 2026-03-29T21:36:51.000+02:00
Move maybeUpdateHistory() after applyPartials() so that when a
redirect falls back to full navigation, we don't leave a duplicate
history entry that requires pressing back twice.

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/packages/fresh/src/runtime/client/partials.ts b/packages/fresh/src/runtime/client/partials.ts
@@ -297,10 +297,6 @@ async function fetchPartials(
     }
   }
 
-  if (shouldNavigate) {
-    maybeUpdateHistory(actualUrl);
-  }
-
   try {
     await applyPartials(res);
   } catch (err) {
@@ -312,6 +308,10 @@ async function fetchPartials(
     }
     throw err;
   }
+
+  if (shouldNavigate) {
+    maybeUpdateHistory(actualUrl);
+  }
 }
 
 interface PartialReviveCtx {

Original file line number	Diff line number	Diff line change
`@@ -297,10 +297,6 @@ async function fetchPartials(`
`297`	`297`	`}`
`298`	`298`	`}`
`299`	`299`
`300`		`- if (shouldNavigate) {`
`301`		`- maybeUpdateHistory(actualUrl);`
`302`		`- }`
`303`		`-`
`304`	`300`	`try {`
`305`	`301`	`await applyPartials(res);`
`306`	`302`	`} catch (err) {`
`@@ -312,6 +308,10 @@ async function fetchPartials(`
`312`	`308`	`}`
`313`	`309`	`throw err;`
`314`	`310`	`}`
	`311`	`+`
	`312`	`+ if (shouldNavigate) {`
	`313`	`+ maybeUpdateHistory(actualUrl);`
	`314`	`+ }`
`315`	`315`	`}`
`316`	`316`
`317`	`317`	`interface PartialReviveCtx {`