Use isRouteLocation

Author: skirtles-code

packages/router/src/RouterLink.ts

@@ -27,6 +27,7 @@ import {
   ComponentOptionsMixin,
 } from 'vue'
 import {
+  isRouteLocation,
   RouteLocationRaw,
   VueUseOptions,
   RouteLocation,
@@ -36,7 +37,7 @@ import { isSameRouteLocationParams, isSameRouteRecord } from './location'
 import { routerKey, routeLocationKey } from './injectionSymbols'
 import { RouteRecord } from './matcher/types'
 import { NavigationFailure } from './errors'
-import { isArray, isBrowser, isObject, noop } from './utils'
+import { isArray, isBrowser, noop } from './utils'
 import { warn } from './warning'
 
 export interface RouterLinkOptions {
@@ -95,15 +96,14 @@ export function useLink(props: UseLinkOptions) {
   const router = inject(routerKey)!
   const currentRoute = inject(routeLocationKey)!
 
-  const isValidTo = (to: unknown) => typeof to === 'string' || isObject(to)
   let hasPrevious = false
   let previousTo: unknown = null
 
   const route = computed(() => {
     const to = unref(props.to)
 
     if (__DEV__ && (!hasPrevious || to !== previousTo)) {
-      if (!isValidTo(to)) {
+      if (!isRouteLocation(to)) {
         if (hasPrevious) {
           warn(
             `Invalid value for prop "to" in useLink()\n- to:`,
@@ -204,7 +204,7 @@ export function useLink(props: UseLinkOptions) {
           linkContextDevtools.route = route.value
           linkContextDevtools.isActive = isActive.value
           linkContextDevtools.isExactActive = isExactActive.value
-          linkContextDevtools.error = isValidTo(unref(props.to))
+          linkContextDevtools.error = isRouteLocation(unref(props.to))
             ? null
             : 'Invalid "to" value'
         },

packages/router/src/router.ts

@@ -8,6 +8,7 @@ import {
   RouteLocationNormalizedLoaded,
   RouteLocation,
   RouteRecordName,
+  isRouteLocation,
   isRouteName,
   NavigationGuardWithThis,
   RouteLocationOptions,
@@ -32,14 +33,7 @@ import {
   NavigationRedirectError,
   isNavigationFailure,
 } from './errors'
-import {
-  applyToParams,
-  isBrowser,
-  assign,
-  noop,
-  isArray,
-  isObject,
-} from './utils'
+import { applyToParams, isBrowser, assign, noop, isArray } from './utils'
 import { useCallbacks } from './utils/callbacks'
 import { encodeParam, decode, encodeHash } from './encoding'
 import {
@@ -467,7 +461,7 @@ export function createRouter(options: RouterOptions): Router {
       })
     }
 
-    if (__DEV__ && !isObject(rawLocation)) {
+    if (__DEV__ && !isRouteLocation(rawLocation)) {
       warn(
         `router.resolve() was passed an invalid location. This will fail in production.\n- Location:`,
         rawLocation