fix(resolve): exclude empty optional arguments from params

packages/playground/src/App.vue

@@ -158,6 +158,15 @@
       <li>
         <router-link to="/p_1/absolute-a">/p_1/absolute-a</router-link>
       </li>
+      <li>
+        <router-link :to="{ name: 'features' }">Go to Features (name)</router-link>
+      </li>
+      <li>
+        <router-link to="/features">Go to Features (string)</router-link>
+      </li>
+      <li>
+        <router-link to="/features/one">Go to Feature one</router-link>
+      </li>
     </ul>
     <button @click="toggleViewName">Toggle view</button>
     <RouterView :name="viewName" v-slot="{ Component, route }">

packages/playground/src/router.ts

@@ -159,6 +159,11 @@ export const router = createRouter({
         { path: 'settings', component },
       ],
     },
+    {
+      path: '/features/:pathMatch(.*)*',
+      name: 'features',
+      component: User,
+    },
   ],
   async scrollBehavior(to, from, savedPosition) {
     await scrollWaiter.wait()

packages/router/__tests__/matcher/resolve.spec.ts

@@ -685,12 +685,12 @@ describe('RouterMatcher.resolve', () => {
       assertRecordMatch(
         { path: '/:a?', components, name: 'a' },
         { path: '/' },
-        { path: '/', params: { a: '' }, name: 'a' }
+        { path: '/', params: {}, name: 'a' }
       )
       assertRecordMatch(
         { path: '/a/:a?', components, name: 'a' },
         { path: '/a/' },
-        { path: '/a/', params: { a: '' }, name: 'a' }
+        { path: '/a/', params: {}, name: 'a' }
       )
     })
 

packages/router/src/matcher/index.ts

@@ -310,6 +310,12 @@ export function createRouterMatcher(
         // we know the matcher works because we tested the regexp
         params = matcher.parse(path)!
         name = matcher.record.name
+
+        matcher?.keys.forEach(key => {
+          if (key.optional && params[key.name] === '') {
+            delete params[key.name]
+          }
+        })
       }
       // location is a relative path
     } else {