refactor: avoid using isFileServingAllowed from Vite (#9160)

Author: sapphi-red

packages/vitest/src/node/vite.ts

@@ -1,5 +1,6 @@
-import type { InlineConfig, ViteDevServer } from 'vite'
-import { createServer } from 'vite'
+import type { InlineConfig, ResolvedConfig, ViteDevServer } from 'vite'
+import { cleanUrl } from '@vitest/utils/helpers'
+import { createServer, isFileLoadingAllowed, normalizePath } from 'vite'
 
 export async function createViteServer(inlineConfig: InlineConfig): Promise<ViteDevServer> {
   // Vite prints an error (https://github.com/vitejs/vite/issues/14328)
@@ -20,3 +21,48 @@ export async function createViteServer(inlineConfig: InlineConfig): Promise<Vite
   console.error = error
   return server
 }
+
+// backward compat implementation
+/**
+ * Check if the url is allowed to be served, via the `server.fs` config.
+ * @deprecated Use the `isFileLoadingAllowed` function instead.
+ */
+export function isFileServingAllowed(
+  config: ResolvedConfig,
+  url: string,
+): boolean
+export function isFileServingAllowed(
+  url: string,
+  server: ViteDevServer,
+): boolean
+export function isFileServingAllowed(
+  configOrUrl: ResolvedConfig | string,
+  urlOrServer: string | ViteDevServer,
+): boolean {
+  const config = (
+    typeof urlOrServer === 'string' ? configOrUrl : urlOrServer.config
+  ) as ResolvedConfig
+  const url = (
+    typeof urlOrServer === 'string' ? urlOrServer : configOrUrl
+  ) as string
+
+  if (!config.server.fs.strict) {
+    return true
+  }
+  const filePath = fsPathFromUrl(url)
+  return isFileLoadingAllowed(config, filePath)
+}
+
+const FS_PREFIX = '/@fs/'
+const VOLUME_RE = /^[A-Z]:/i
+
+function fsPathFromId(id: string): string {
+  const fsPath = normalizePath(
+    id.startsWith(FS_PREFIX) ? id.slice(FS_PREFIX.length) : id,
+  )
+  return fsPath[0] === '/' || VOLUME_RE.test(fsPath) ? fsPath : `/${fsPath}`
+}
+
+function fsPathFromUrl(url: string): string {
+  return fsPathFromId(cleanUrl(url))
+}

packages/vitest/src/public/node.ts

@@ -135,6 +135,7 @@ export type {
 export type { VitestPluginContext } from '../node/types/plugin'
 export type { TestRunResult } from '../node/types/tests'
 export type { WorkerContext } from '../node/types/worker'
+export { isFileServingAllowed } from '../node/vite'
 export { createViteLogger } from '../node/viteLogger'
 export type { WatcherTriggerPattern } from '../node/watcher'
 
@@ -170,7 +171,7 @@ export type { SerializedError } from '@vitest/utils'
 export {
   esbuildVersion,
   isCSSRequest,
-  isFileServingAllowed,
+  isFileLoadingAllowed,
   parseAst,
   parseAstAsync,
   rollupVersion,

test/core/test/exports.test.ts

@@ -120,6 +120,7 @@ it('exports snapshot', async ({ skip, task }) => {
           "generateFileHash": "function",
           "getFilePoolName": "function",
           "isCSSRequest": "function",
+          "isFileLoadingAllowed": "function",
           "isFileServingAllowed": "function",
           "isValidApiRequest": "function",
           "parseAst": "function",
@@ -285,6 +286,7 @@ it('exports snapshot', async ({ skip, task }) => {
             "generateFileHash": "function",
             "getFilePoolName": "function",
             "isCSSRequest": "function",
+            "isFileLoadingAllowed": "function",
             "isFileServingAllowed": "function",
             "isValidApiRequest": "function",
             "parseAst": "function",