From 4b78d33a3f4f1128e6f71da3be256aae05f328a8 Mon Sep 17 00:00:00 2001
From: proc07 <790847400@qq.com>
Date: Sat, 23 Mar 2024 09:47:56 +0800
Subject: [PATCH 1/6] fix: externalize the asset js file type, but enter
 vite:dep-pre-bundle

---
 packages/vite/src/node/optimizer/esbuildDepPlugin.ts | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/packages/vite/src/node/optimizer/esbuildDepPlugin.ts b/packages/vite/src/node/optimizer/esbuildDepPlugin.ts
index 08b1abb72d48e3..1f4c4dab16748d 100644
--- a/packages/vite/src/node/optimizer/esbuildDepPlugin.ts
+++ b/packages/vite/src/node/optimizer/esbuildDepPlugin.ts
@@ -155,6 +155,14 @@ export function esbuildDepPlugin(
           const resolved = await resolve(id, importer, kind)
           if (resolved) {
             if (kind === 'require-call') {
+              // #16116 fix: Import the module.scss path, which is actually module.scss.js
+              if (resolved.endsWith('.js')) {
+                return {
+                  path: resolved,
+                  external: false,
+                }
+              }
+
               // here it is not set to `external: true` to convert `require` to `import`
               return {
                 path: resolved,

From cf0e3f147a60b1f2166dabf1a3f26411c10f5526 Mon Sep 17 00:00:00 2001
From: proc07 <790847400@qq.com>
Date: Sun, 24 Mar 2024 17:57:02 +0800
Subject: [PATCH 2/6] test: externalize the asset js file type (#16116)

---
 .../__tests__/optimize-deps.spec.ts           | 15 +++++++++++++++
 .../index.js                                  |  6 ++++++
 .../package.json                              |  5 +++++
 .../test.astro.js                             |  5 +++++
 .../test.okay.js                              |  5 +++++
 .../test.scss.js                              |  5 +++++
 .../test.tsx.js                               |  5 +++++
 playground/optimize-deps/index.html           | 19 +++++++++++++++++++
 playground/optimize-deps/package.json         |  1 +
 playground/optimize-deps/vite.config.js       |  4 ++++
 10 files changed, 70 insertions(+)
 create mode 100644 playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/index.js
 create mode 100644 playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/package.json
 create mode 100644 playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.astro.js
 create mode 100644 playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.okay.js
 create mode 100644 playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.scss.js
 create mode 100644 playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.tsx.js

diff --git a/playground/optimize-deps/__tests__/optimize-deps.spec.ts b/playground/optimize-deps/__tests__/optimize-deps.spec.ts
index ee691fddc4c686..d6c328b9216009 100644
--- a/playground/optimize-deps/__tests__/optimize-deps.spec.ts
+++ b/playground/optimize-deps/__tests__/optimize-deps.spec.ts
@@ -331,3 +331,18 @@ test.runIf(isServe)('warn on incompatible dependency', () => {
     ),
   )
 })
+
+test('import the CommonJS external package that omits the js suffix', async () => {
+  await expectWithRetry(() => page.textContent('.external-package-js')).toBe(
+    'okay',
+  )
+  await expectWithRetry(() =>
+    page.textContent('.external-package-scss-js'),
+  ).toBe('scss')
+  await expectWithRetry(() =>
+    page.textContent('.external-package-astro-js'),
+  ).toBe('astro')
+  await expectWithRetry(() =>
+    page.textContent('.external-package-tsx-js'),
+  ).toBe('tsx')
+})
diff --git a/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/index.js b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/index.js
new file mode 100644
index 00000000000000..c3bcc276762093
--- /dev/null
+++ b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/index.js
@@ -0,0 +1,6 @@
+const { okay } = require('./test.okay')
+const { scss } = require('./test.scss')
+const { astro } = require('./test.astro')
+const { tsx } = require('./test.tsx')
+
+module.exports = { okay, scss, astro, tsx }
diff --git a/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/package.json b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/package.json
new file mode 100644
index 00000000000000..22a91fc97ea119
--- /dev/null
+++ b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/package.json
@@ -0,0 +1,5 @@
+{
+  "name": "@vitejs/test-dep-cjs-external-package-omit-js-suffix",
+  "version": "0.0.1",
+  "main": "index.js"
+}
diff --git a/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.astro.js b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.astro.js
new file mode 100644
index 00000000000000..337124ff2857bc
--- /dev/null
+++ b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.astro.js
@@ -0,0 +1,5 @@
+function astro() {
+  return 'astro'
+}
+
+module.exports = { astro }
diff --git a/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.okay.js b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.okay.js
new file mode 100644
index 00000000000000..b8913f7c835d9c
--- /dev/null
+++ b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.okay.js
@@ -0,0 +1,5 @@
+function okay() {
+  return 'okay'
+}
+
+module.exports = { okay }
diff --git a/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.scss.js b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.scss.js
new file mode 100644
index 00000000000000..507ebe60f429d3
--- /dev/null
+++ b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.scss.js
@@ -0,0 +1,5 @@
+function scss() {
+  return 'scss'
+}
+
+module.exports = { scss }
diff --git a/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.tsx.js b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.tsx.js
new file mode 100644
index 00000000000000..6edc349a1880c3
--- /dev/null
+++ b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/test.tsx.js
@@ -0,0 +1,5 @@
+function tsx() {
+  return 'tsx'
+}
+
+module.exports = { tsx }
diff --git a/playground/optimize-deps/index.html b/playground/optimize-deps/index.html
index 9b992b7dd4892a..304eb0628522b1 100644
--- a/playground/optimize-deps/index.html
+++ b/playground/optimize-deps/index.html
@@ -110,6 +110,25 @@ <h2>Long file name import works</h2>
 
 <script type="module" src="./long-file-name.js"></script>
 
+<h2>Import the CommonJS external package that omits the js suffix</h2>
+<div class="external-package-js"></div>
+<div class="external-package-scss-js"></div>
+<div class="external-package-astro-js"></div>
+<div class="external-package-tsx-js"></div>
+<script type="module">
+  import {
+    astro,
+    okay,
+    scss,
+    tsx,
+  } from '@vitejs/test-dep-cjs-external-package-omit-js-suffix'
+  console.log(astro(), 'astroastroastroastro')
+  text('.external-package-js', okay())
+  text('.external-package-scss-js', scss())
+  text('.external-package-astro-js', astro())
+  text('.external-package-tsx-js', tsx())
+</script>
+
 <script>
   function text(el, text) {
     document.querySelector(el).textContent = text
diff --git a/playground/optimize-deps/package.json b/playground/optimize-deps/package.json
index e5f8d8a22dfcef..e728e0fc30aa48 100644
--- a/playground/optimize-deps/package.json
+++ b/playground/optimize-deps/package.json
@@ -36,6 +36,7 @@
     "@vitejs/test-dep-with-optional-peer-dep-submodule": "file:./dep-with-optional-peer-dep-submodule",
     "@vitejs/test-dep-non-optimized": "file:./dep-non-optimized",
     "@vitejs/test-added-in-entries": "file:./added-in-entries",
+    "@vitejs/test-dep-cjs-external-package-omit-js-suffix": "file:./dep-cjs-external-package-omit-js-suffix",
     "lodash-es": "^4.17.21",
     "@vitejs/test-nested-exclude": "file:./nested-exclude",
     "phoenix": "^1.7.11",
diff --git a/playground/optimize-deps/vite.config.js b/playground/optimize-deps/vite.config.js
index a2c2fdf3c6d5e7..3930b1103a9b6b 100644
--- a/playground/optimize-deps/vite.config.js
+++ b/playground/optimize-deps/vite.config.js
@@ -17,6 +17,7 @@ export default defineConfig({
     include: [
       '@vitejs/test-dep-linked-include',
       '@vitejs/test-nested-exclude > @vitejs/test-nested-include',
+      '@vitejs/test-dep-cjs-external-package-omit-js-suffix',
       // will throw if optimized (should log warning instead)
       '@vitejs/test-non-optimizable-include',
       '@vitejs/test-dep-optimize-exports-with-glob/**/*',
@@ -53,6 +54,9 @@ export default defineConfig({
         warn(msg)
       },
     },
+    commonjsOptions: {
+      include: ['./dep-cjs-external-package-omit-js-suffix'],
+    },
   },
 
   plugins: [

From 4fea3eb533b3de3ec41b468e2668d9afcd69d765 Mon Sep 17 00:00:00 2001
From: proc07 <790847400@qq.com>
Date: Sun, 24 Mar 2024 17:59:19 +0800
Subject: [PATCH 3/6] fix: del log

---
 playground/optimize-deps/index.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/playground/optimize-deps/index.html b/playground/optimize-deps/index.html
index 304eb0628522b1..b98a3174497ef6 100644
--- a/playground/optimize-deps/index.html
+++ b/playground/optimize-deps/index.html
@@ -122,7 +122,7 @@ <h2>Import the CommonJS external package that omits the js suffix</h2>
     scss,
     tsx,
   } from '@vitejs/test-dep-cjs-external-package-omit-js-suffix'
-  console.log(astro(), 'astroastroastroastro')
+
   text('.external-package-js', okay())
   text('.external-package-scss-js', scss())
   text('.external-package-astro-js', astro())

From 7b37ca1d7bafcd368408043f34dff2d1d25522e5 Mon Sep 17 00:00:00 2001
From: proc07 <790847400@qq.com>
Date: Sun, 24 Mar 2024 21:11:33 +0800
Subject: [PATCH 4/6] fix: save lock package

---
 pnpm-lock.yaml | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
index 1bdb5276b02cae..12b1da065a860d 100644
--- a/pnpm-lock.yaml
+++ b/pnpm-lock.yaml
@@ -906,6 +906,9 @@ importers:
       '@vitejs/test-dep-cjs-compiled-from-esm':
         specifier: file:./dep-cjs-compiled-from-esm
         version: file:playground/optimize-deps/dep-cjs-compiled-from-esm
+      '@vitejs/test-dep-cjs-external-package-omit-js-suffix':
+        specifier: file:./dep-cjs-external-package-omit-js-suffix
+        version: file:playground/optimize-deps/dep-cjs-external-package-omit-js-suffix
       '@vitejs/test-dep-cjs-with-assets':
         specifier: file:./dep-cjs-with-assets
         version: file:playground/optimize-deps/dep-cjs-with-assets
@@ -1028,6 +1031,8 @@ importers:
 
   playground/optimize-deps/dep-cjs-compiled-from-esm: {}
 
+  playground/optimize-deps/dep-cjs-external-package-omit-js-suffix: {}
+
   playground/optimize-deps/dep-cjs-with-assets: {}
 
   playground/optimize-deps/dep-css-require: {}
@@ -10551,6 +10556,11 @@ packages:
     name: '@vitejs/test-dep-cjs-compiled-from-esm'
     dev: false
 
+  file:playground/optimize-deps/dep-cjs-external-package-omit-js-suffix:
+    resolution: {directory: playground/optimize-deps/dep-cjs-external-package-omit-js-suffix, type: directory}
+    name: '@vitejs/test-dep-cjs-external-package-omit-js-suffix'
+    dev: false
+
   file:playground/optimize-deps/dep-cjs-with-assets:
     resolution: {directory: playground/optimize-deps/dep-cjs-with-assets, type: directory}
     name: '@vitejs/test-dep-cjs-with-assets'

From 5f16c7de0b2b0b8d21c602d5c7f3b34d2be350b7 Mon Sep 17 00:00:00 2001
From: proc07 <790847400@qq.com>
Date: Sun, 24 Mar 2024 21:26:17 +0800
Subject: [PATCH 5/6] fix: set private

---
 .../dep-cjs-external-package-omit-js-suffix/package.json       | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/package.json b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/package.json
index 22a91fc97ea119..c0e4afb3259b07 100644
--- a/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/package.json
+++ b/playground/optimize-deps/dep-cjs-external-package-omit-js-suffix/package.json
@@ -1,5 +1,6 @@
 {
   "name": "@vitejs/test-dep-cjs-external-package-omit-js-suffix",
-  "version": "0.0.1",
+  "private": true,
+  "version": "0.0.0",
   "main": "index.js"
 }

From cd02ca30817f857dd0d023909e4b95cac751fbfd Mon Sep 17 00:00:00 2001
From: proc07 <790847400@qq.com>
Date: Tue, 26 Mar 2024 22:27:33 +0800
Subject: [PATCH 6/6] fix: del config commonjsOptions

---
 playground/optimize-deps/vite.config.js | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/playground/optimize-deps/vite.config.js b/playground/optimize-deps/vite.config.js
index 3930b1103a9b6b..6ef09488556cc1 100644
--- a/playground/optimize-deps/vite.config.js
+++ b/playground/optimize-deps/vite.config.js
@@ -54,9 +54,6 @@ export default defineConfig({
         warn(msg)
       },
     },
-    commonjsOptions: {
-      include: ['./dep-cjs-external-package-omit-js-suffix'],
-    },
   },
 
   plugins: [