revert: move promise resolution back to try block

Moving the promise resolution from finally block back to try block.
This ensures promises are only resolved on successful loader completion,
not on errors or redirects.

Resolving on redirect was incorrect because we're navigating away from
the route, so head() re-execution for the old route doesn't make sense.

The null check is kept as a safety measure since this code still runs
in an async callback where navigation could theoretically change.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>

Author: yanghuidong

packages/router-core/src/load-matches.ts

@@ -839,21 +839,17 @@ const loadRouteMatch = async (
           try {
             await runLoader(inner, matchId, index, route)
             commitContext()
-          } catch (err) {
-            if (isRedirect(err)) {
-              await inner.router.navigate(err.options)
-            }
-            // Errors are already stored in match by runLoader
-            // Continue to resolve promises so head() can execute
-          } finally {
-            // Always resolve promises (success or error) to allow head() execution
             // Match might be undefined if navigation changed while async loader was running
             const match = inner.router.getMatch(matchId)
             if (match) {
               match._nonReactive.loaderPromise?.resolve()
               match._nonReactive.loadPromise?.resolve()
               match._nonReactive.loaderPromise = undefined
             }
+          } catch (err) {
+            if (isRedirect(err)) {
+              await inner.router.navigate(err.options)
+            }
           }
         })()
       } else if (status !== 'success' || (loaderShouldRunAsync && inner.sync)) {