fix: avoid passing leading ? to parseQuery

Fix #2571

Author: posva

packages/router/__tests__/location.spec.ts

@@ -248,7 +248,7 @@ describe('parseURL', () => {
     const parseQuery = vi.fn()
     originalParseURL(parseQuery, '/?é=é&é=a')
     expect(parseQuery).toHaveBeenCalledTimes(1)
-    expect(parseQuery).toHaveBeenCalledWith('?é=é&é=a')
+    expect(parseQuery).toHaveBeenCalledWith('é=é&é=a')
   })
 })
 

packages/router/__tests__/router.spec.ts

@@ -175,7 +175,7 @@ describe('Router', () => {
     const parseQuery = vi.fn(_ => ({}))
     const { router } = await newRouter({ parseQuery })
     const to = router.resolve('/foo?bar=baz')
-    expect(parseQuery).toHaveBeenCalledWith('?bar=baz')
+    expect(parseQuery).toHaveBeenCalledWith('bar=baz')
     expect(to.query).toEqual({})
   })
 

packages/router/src/location.ts

@@ -68,7 +68,10 @@ export function parseURL(
       hashPos > 0 ? hashPos : location.length
     )
 
-    query = parseQuery(searchString)
+    query = parseQuery(
+      // remove the leading ?
+      searchString.slice(1)
+    )
   }
 
   if (hashPos >= 0) {