fix: injectQuery break relative path

packages/vite/src/node/__tests__/utils.spec.ts

@@ -6,6 +6,7 @@ import {
   getPotentialTsSrcPaths,
   injectQuery,
   isWindows,
+  normalizePath,
   resolveHostname
 } from '../utils'
 
@@ -19,6 +20,32 @@ describe('injectQuery', () => {
     })
   }
 
+  test('relative path ""', () => {
+    expect(injectQuery('usr/vite/%20a%20', 'direct')).toEqual(
+      'usr/vite/%20a%20?direct'
+    )
+  })
+
+  test('relative path "./"', () => {
+    expect(injectQuery('./usr/vite/%20a%20', 'direct')).toEqual(
+      'usr/vite/%20a%20?direct'
+    )
+  })
+
+  test('file path', () => {
+    expect(injectQuery('file:///usr/vite/%20a%20', 'direct')).toMatch(
+      !isWindows
+        ? // d/a/vite/vite/file:/usr/vite/%20a%20?direct
+          new RegExp(
+            `${process.cwd().slice(1)}/file:/usr/vite/%20a%20\\?direct`
+          )
+        : // D:/a/vite/vite/file:/usr/vite/%20a%20?direct
+          new RegExp(
+            `${normalizePath(process.cwd())}/file:/usr/vite/%20a%20\\?direct`
+          )
+    )
+  })
+
   test('path with multiple spaces', () => {
     expect(injectQuery('/usr/vite/path with space', 'direct')).toEqual(
       '/usr/vite/path with space?direct'

packages/vite/src/node/utils.ts

@@ -312,7 +312,9 @@ export function injectQuery(url: string, queryToInject: string): string {
     resolvedUrl = pathToFileURL(url)
   }
   let { protocol, pathname, search, hash } = resolvedUrl
-  if (protocol === 'file:') {
+  // pathname must startWith '/'
+  // if url[0] is not '/' should be relative path
+  if (protocol === 'file:' || url[0] !== '/') {
     pathname = pathname.slice(1)
   }
   pathname = decodeURIComponent(pathname)