refactor(router-core): redirect skips creating a URL object if href is not a string (#4986)

Creating a `URL` object is a little expensive, and we know it's going to
throw if `href` is not a string. So we can skip it entirely in that case
(which is probably most cases?).

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Bug Fixes**
* Improved redirect handling to avoid errors when a URL is missing or
invalid.
* Added safeguards to prevent runtime crashes when redirect data is
null.
  * Enhanced URL parsing reliability during redirects.
  * No changes to public APIs.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Author: Sheraff

packages/router-core/src/redirect.ts

@@ -65,9 +65,9 @@ export function redirect<
 ): Redirect<TRouter, TFrom, TTo, TMaskFrom, TMaskTo> {
   opts.statusCode = opts.statusCode || opts.code || 307
 
-  if (!opts.reloadDocument) {
+  if (!opts.reloadDocument && typeof opts.href === 'string') {
     try {
-      new URL(`${opts.href}`)
+      new URL(opts.href)
       opts.reloadDocument = true
     } catch {}
   }
@@ -103,7 +103,7 @@ export function isResolvedRedirect(
 }
 
 export function parseRedirect(obj: any) {
-  if (typeof obj === 'object' && obj.isSerializedRedirect) {
+  if (obj !== null && typeof obj === 'object' && obj.isSerializedRedirect) {
     return redirect(obj)
   }