fix: fix external behavior with deps.optimizer (#9125)

Author: hi-ogawa

packages/vitest/src/node/resolver.ts

@@ -3,6 +3,7 @@ import { existsSync, promises as fsp } from 'node:fs'
 import { isBuiltin } from 'node:module'
 import { pathToFileURL } from 'node:url'
 import { KNOWN_ASSET_RE } from '@vitest/utils/constants'
+import { cleanUrl } from '@vitest/utils/helpers'
 import { findNearestPackageData } from '@vitest/utils/resolver'
 import * as esModuleLexer from 'es-module-lexer'
 import { dirname, extname, join, resolve } from 'pathe'
@@ -109,6 +110,10 @@ export function guessCJSversion(id: string): string | undefined {
 
 // The code from https://github.com/unjs/mlly/blob/c5bcca0cda175921344fd6de1bc0c499e73e5dac/src/syntax.ts#L51-L98
 async function isValidNodeImport(id: string) {
+  // clean url to strip off `?v=...` query etc.
+  // node can natively import files with query params, so externalizing them is safe.
+  id = cleanUrl(id)
+
   const extension = extname(id)
 
   if (BUILTIN_EXTENSIONS.has(extension)) {

pnpm-lock.yaml

@@ -0,0 +1 @@
+export default {}

test/config/deps/optimizer/external/index.js

@@ -0,0 +1,6 @@
+{
+  "name": "@vitest/test-dep-optimizer-external",
+  "type": "module",
+  "private": true,
+  "main": "./index.js"
+}

test/config/deps/optimizer/external/package.json

@@ -0,0 +1 @@
+export default {}

test/config/deps/optimizer/optimized/index.js

@@ -0,0 +1,6 @@
+{
+  "name": "@vitest/test-dep-optimizer-optimized",
+  "type": "module",
+  "private": true,
+  "main": "./index.js"
+}

test/config/deps/optimizer/optimized/package.json

@@ -0,0 +1,7 @@
+import { it, expect } from "vitest";
+// @ts-ignore
+import * as testDep from "@vitest/test-dep-optimizer-external"
+
+it("passes", () => {
+  expect(testDep.default).toBeDefined();
+});

test/config/fixtures/optimizer/external/basic.test.ts

@@ -0,0 +1,25 @@
+import { defineConfig } from "vitest/config";
+
+export default defineConfig({
+  test: {
+    deps: {
+      optimizer: {
+        ssr: {
+          enabled: true,
+          // need at least one to trigger dep optimization
+          include: ["@vitest/test-dep-optimizer-optimized"],
+        },
+      },
+    },
+  },
+  plugins: [
+    {
+      name: 'test-external',
+      transform(_code, id, _options) {
+        if (id.includes('test-dep-optimizer-external')) {
+          this.error('"@vitest/test-dep-optimizer-external" is expected to be externalized');
+        }
+      },
+    }
+  ]
+});

test/config/fixtures/optimizer/external/vitest.config.ts

@@ -11,6 +11,8 @@
     "@vitest/browser-preview": "workspace:*",
     "@vitest/browser-webdriverio": "workspace:*",
     "@vitest/test-dep-conditions": "file:./deps/test-dep-conditions",
+    "@vitest/test-dep-optimizer-external": "file:./deps/optimizer/external",
+    "@vitest/test-dep-optimizer-optimized": "file:./deps/optimizer/optimized",
     "inline-dep": "file:./deps/vite-ssr-resolve/inline-dep",
     "other-dep": "file:./deps/vite-ssr-resolve/other-dep",
     "ssr-no-external-dep": "file:./deps/vite-ssr-resolve/ssr-no-external-dep",

test/config/package.json

@@ -0,0 +1,15 @@
+import { expect, it } from 'vitest'
+import { runVitest } from '../../test-utils'
+
+it('external works with optimizer', async () => {
+  const { errorTree } = await runVitest({
+    root: 'fixtures/optimizer/external',
+  })
+  expect(errorTree()).toMatchInlineSnapshot(`
+    {
+      "basic.test.ts": {
+        "passes": "passed",
+      },
+    }
+  `)
+})