From 656e26f58e96dafe2bbef3a6bdada88a19404abf Mon Sep 17 00:00:00 2001
From: sapphi-red <green@sapphi.red>
Date: Fri, 1 Jul 2022 12:08:47 +0900
Subject: [PATCH 1/5] test: add #8245 test case

---
 packages/playground/css/__tests__/css.spec.ts  | 6 ++++++
 packages/playground/css/aliased/bar.module.css | 3 +++
 packages/playground/css/aliased/foo.css        | 3 +++
 packages/playground/css/index.html             | 5 +++++
 packages/playground/css/main.js                | 7 +++++++
 packages/playground/css/vite.config.js         | 4 +++-
 6 files changed, 27 insertions(+), 1 deletion(-)
 create mode 100644 packages/playground/css/aliased/bar.module.css
 create mode 100644 packages/playground/css/aliased/foo.css

diff --git a/packages/playground/css/__tests__/css.spec.ts b/packages/playground/css/__tests__/css.spec.ts
index e9372f67220c09..fe1d06132dcdd2 100644
--- a/packages/playground/css/__tests__/css.spec.ts
+++ b/packages/playground/css/__tests__/css.spec.ts
@@ -419,3 +419,9 @@ test('PostCSS source.input.from includes query', async () => {
   // should resolve assets
   expect(code).toContain('/postcss-source-input.css?query=foo')
 })
+
+test('aliased css has content', async () => {
+  expect(await getColor('.aliased')).toBe('blue')
+  expect(await page.textContent('.aliased-content')).toMatch('.aliased')
+  expect(await getColor('.aliased-module')).toBe('blue')
+})
diff --git a/packages/playground/css/aliased/bar.module.css b/packages/playground/css/aliased/bar.module.css
new file mode 100644
index 00000000000000..e4e46f3306a02e
--- /dev/null
+++ b/packages/playground/css/aliased/bar.module.css
@@ -0,0 +1,3 @@
+.aliasedModule {
+  color: blue;
+}
diff --git a/packages/playground/css/aliased/foo.css b/packages/playground/css/aliased/foo.css
new file mode 100644
index 00000000000000..7e32cb71a8f375
--- /dev/null
+++ b/packages/playground/css/aliased/foo.css
@@ -0,0 +1,3 @@
+.aliased {
+  color: blue;
+}
diff --git a/packages/playground/css/index.html b/packages/playground/css/index.html
index 15e81192cec7f1..be559f7af401bf 100644
--- a/packages/playground/css/index.html
+++ b/packages/playground/css/index.html
@@ -138,6 +138,11 @@ <h1>CSS</h1>
 
   <p>PostCSS source.input.from. Should include query</p>
   <pre class="postcss-source-input"></pre>
+
+  <p>Aliased</p>
+  <p class="aliased">import '#alias': this should be blue</p>
+  <pre class="aliased-content"></pre>
+  <p class="aliased-module">import '#alias-module': this should be blue</p>
 </div>
 
 <script type="module" src="./main.js"></script>
diff --git a/packages/playground/css/main.js b/packages/playground/css/main.js
index f728b0251066d1..350a117d59cd51 100644
--- a/packages/playground/css/main.js
+++ b/packages/playground/css/main.js
@@ -90,3 +90,10 @@ text('.imported-css-globEager', JSON.stringify(globEager, null, 2))
 
 import postcssSourceInput from './postcss-source-input.css?query=foo'
 text('.postcss-source-input', postcssSourceInput)
+
+import aliasContent from '#alias'
+text('.aliased-content', aliasContent)
+import aliasModule from '#alias-module'
+document
+  .querySelector('.aliased-module')
+  .classList.add(aliasModule.aliasedModule)
diff --git a/packages/playground/css/vite.config.js b/packages/playground/css/vite.config.js
index 639a1302debb88..c501213b47cded 100644
--- a/packages/playground/css/vite.config.js
+++ b/packages/playground/css/vite.config.js
@@ -10,7 +10,9 @@ module.exports = {
   resolve: {
     alias: {
       '@': __dirname,
-      spacefolder: __dirname + '/folder with space'
+      spacefolder: __dirname + '/folder with space',
+      '#alias': __dirname + '/aliased/foo.css',
+      '#alias-module': __dirname + '/aliased/bar.module.css'
     }
   },
   css: {

From 2c5aca021ccb0f0847935cdc4e67249e45d5796e Mon Sep 17 00:00:00 2001
From: sapphi-red <green@sapphi.red>
Date: Fri, 1 Jul 2022 12:17:19 +0900
Subject: [PATCH 2/5] test: add #8461 test case

---
 packages/playground/css/__tests__/css.spec.ts     | 5 +++++
 packages/playground/css/css-js-dep/bar.module.css | 3 +++
 packages/playground/css/css-js-dep/foo.css        | 3 +++
 packages/playground/css/css-js-dep/index.js       | 4 ++++
 packages/playground/css/css-js-dep/package.json   | 7 +++++++
 packages/playground/css/index.html                | 7 +++++++
 packages/playground/css/main.js                   | 5 +++++
 packages/playground/css/package.json              | 6 +++++-
 pnpm-lock.yaml                                    | 8 +++++++-
 9 files changed, 46 insertions(+), 2 deletions(-)
 create mode 100644 packages/playground/css/css-js-dep/bar.module.css
 create mode 100644 packages/playground/css/css-js-dep/foo.css
 create mode 100644 packages/playground/css/css-js-dep/index.js
 create mode 100644 packages/playground/css/css-js-dep/package.json

diff --git a/packages/playground/css/__tests__/css.spec.ts b/packages/playground/css/__tests__/css.spec.ts
index fe1d06132dcdd2..11fa77aa3f2e74 100644
--- a/packages/playground/css/__tests__/css.spec.ts
+++ b/packages/playground/css/__tests__/css.spec.ts
@@ -342,6 +342,11 @@ test('PostCSS dir-dependency', async () => {
   }
 })
 
+test('import dependency includes css import', async () => {
+  expect(await getColor('.css-js-dep')).toBe('green')
+  expect(await getColor('.css-js-dep-module')).toBe('green')
+})
+
 test('URL separation', async () => {
   const urlSeparated = await page.$('.url-separated')
   const baseUrl = 'url(images/dog.webp)'
diff --git a/packages/playground/css/css-js-dep/bar.module.css b/packages/playground/css/css-js-dep/bar.module.css
new file mode 100644
index 00000000000000..9d62f66761fa3d
--- /dev/null
+++ b/packages/playground/css/css-js-dep/bar.module.css
@@ -0,0 +1,3 @@
+.cssJsDepModule {
+  color: green;
+}
diff --git a/packages/playground/css/css-js-dep/foo.css b/packages/playground/css/css-js-dep/foo.css
new file mode 100644
index 00000000000000..515ee7693bff3f
--- /dev/null
+++ b/packages/playground/css/css-js-dep/foo.css
@@ -0,0 +1,3 @@
+.css-js-dep {
+  color: green;
+}
diff --git a/packages/playground/css/css-js-dep/index.js b/packages/playground/css/css-js-dep/index.js
new file mode 100644
index 00000000000000..853094b806fa97
--- /dev/null
+++ b/packages/playground/css/css-js-dep/index.js
@@ -0,0 +1,4 @@
+import './foo.css'
+import barModuleClasses from './bar.module.css'
+
+export { barModuleClasses }
diff --git a/packages/playground/css/css-js-dep/package.json b/packages/playground/css/css-js-dep/package.json
new file mode 100644
index 00000000000000..d762f6566fdbdc
--- /dev/null
+++ b/packages/playground/css/css-js-dep/package.json
@@ -0,0 +1,7 @@
+{
+  "name": "css-js-dep",
+  "private": true,
+  "type": "module",
+  "version": "1.0.0",
+  "main": "index.js"
+}
diff --git a/packages/playground/css/index.html b/packages/playground/css/index.html
index be559f7af401bf..0f30526cb31900 100644
--- a/packages/playground/css/index.html
+++ b/packages/playground/css/index.html
@@ -114,6 +114,13 @@ <h1>CSS</h1>
     PostCSS dir-dependency (file 2): this should be grey too
   </p>
 
+  <p class="css-js-dep">
+    import dependency includes 'import "./foo.css"': this should be green
+  </p>
+  <p class="css-js-dep-module">
+    import dependency includes 'import "./bar.module.css"': this should be green
+  </p>
+
   <p class="url-separated">
     URL separation preservation: should have valid background-image
   </p>
diff --git a/packages/playground/css/main.js b/packages/playground/css/main.js
index 350a117d59cd51..20ef6ed60fb51c 100644
--- a/packages/playground/css/main.js
+++ b/packages/playground/css/main.js
@@ -47,6 +47,11 @@ text('.charset-css', charset)
 import './dep.css'
 import './glob-dep.css'
 
+import { barModuleClasses } from 'css-js-dep'
+document
+  .querySelector('.css-js-dep-module')
+  .classList.add(barModuleClasses.cssJsDepModule)
+
 function text(el, text) {
   document.querySelector(el).textContent = text
 }
diff --git a/packages/playground/css/package.json b/packages/playground/css/package.json
index b45063100be089..d59d9108eb91c8 100644
--- a/packages/playground/css/package.json
+++ b/packages/playground/css/package.json
@@ -6,7 +6,11 @@
     "dev": "vite",
     "build": "vite build",
     "debug": "node --inspect-brk ../../vite/bin/vite",
-    "preview": "vite preview"
+    "preview": "vite preview",
+    "postinstall": "ts-node ../../../scripts/patchFileDeps.ts"
+  },
+  "dependencies": {
+    "css-js-dep": "file:./css-js-dep"
   },
   "devDependencies": {
     "css-dep": "link:./css-dep",
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
index 609e5f1870459e..48d3cf3b4b6a5c 100644
--- a/pnpm-lock.yaml
+++ b/pnpm-lock.yaml
@@ -137,11 +137,14 @@ importers:
   packages/playground/css:
     specifiers:
       css-dep: link:./css-dep
+      css-js-dep: file:./css-js-dep
       fast-glob: ^3.2.11
       less: ^4.1.2
       postcss-nested: ^5.0.6
       sass: ^1.43.4
       stylus: ^0.55.0
+    dependencies:
+      css-js-dep: link:css-js-dep
     devDependencies:
       css-dep: link:css-dep
       fast-glob: 3.2.11
@@ -171,6 +174,9 @@ importers:
   packages/playground/css/css-dep:
     specifiers: {}
 
+  packages/playground/css/css-js-dep:
+    specifiers: {}
+
   packages/playground/css/pkg-dep:
     specifiers: {}
 
@@ -5836,7 +5842,7 @@ packages:
     dev: true
 
   /image-size/0.5.5:
-    resolution: {integrity: sha1-Cd/Uq50g4p6xw+gLiZA3jfnjy5w=}
+    resolution: {integrity: sha512-6TDAlDPZxUFCv+fuOkIoXT/V/f3Qbq8e37p+YOiYrUv3v9cc3/6x78VdfPgFVaB9dZYeLUfKgHRebpkm/oP2VQ==}
     engines: {node: '>=0.10.0'}
     hasBin: true
     requiresBuild: true

From 47a9ba5517863c447fb96a49530777451f71a61e Mon Sep 17 00:00:00 2001
From: sapphi-red <green@sapphi.red>
Date: Fri, 1 Jul 2022 12:36:01 +0900
Subject: [PATCH 3/5] fix: reverts #8471

This reverts commit 8d7bac416003cc0268a269cba6630162b1ac7412.
---
 packages/playground/css/__tests__/css.spec.ts |  6 ++++-
 packages/vite/src/node/importGlob.ts          |  7 ++++--
 packages/vite/src/node/plugins/css.ts         | 24 +++++++++----------
 .../src/node/plugins/importAnalysisBuild.ts   | 22 +++++++++++++++--
 4 files changed, 42 insertions(+), 17 deletions(-)

diff --git a/packages/playground/css/__tests__/css.spec.ts b/packages/playground/css/__tests__/css.spec.ts
index 11fa77aa3f2e74..c306479f0c38ff 100644
--- a/packages/playground/css/__tests__/css.spec.ts
+++ b/packages/playground/css/__tests__/css.spec.ts
@@ -422,7 +422,11 @@ test("relative path rewritten in Less's data-uri", async () => {
 test('PostCSS source.input.from includes query', async () => {
   const code = await page.textContent('.postcss-source-input')
   // should resolve assets
-  expect(code).toContain('/postcss-source-input.css?query=foo')
+  expect(code).toContain(
+    isBuild
+      ? '/postcss-source-input.css?used&query=foo'
+      : '/postcss-source-input.css?query=foo'
+  )
 })
 
 test('aliased css has content', async () => {
diff --git a/packages/vite/src/node/importGlob.ts b/packages/vite/src/node/importGlob.ts
index 0af49b8dc51268..70ad62321236bb 100644
--- a/packages/vite/src/node/importGlob.ts
+++ b/packages/vite/src/node/importGlob.ts
@@ -9,6 +9,7 @@ import {
   preloadMarker,
   preloadMethod
 } from './plugins/importAnalysisBuild'
+import { isCSSRequest } from './plugins/css'
 import {
   blankReplacer,
   cleanUrl,
@@ -149,14 +150,16 @@ export async function transformImportGlob(
         await fsp.readFile(path.join(base, files[i]), 'utf-8')
       )},`
     } else {
+      const importeeUrl = isCSSRequest(importee) ? `${importee}?used` : importee
       if (isEager) {
         const identifier = `__glob_${importIndex}_${i}`
+        // css imports injecting a ?used query to export the css string
         importsString += `import ${
           isEagerDefault ? `` : `* as `
-        }${identifier} from ${JSON.stringify(importee)};`
+        }${identifier} from ${JSON.stringify(importeeUrl)};`
         entries += ` ${JSON.stringify(file)}: ${identifier},`
       } else {
-        let imp = `import(${JSON.stringify(importee)})`
+        let imp = `import(${JSON.stringify(importeeUrl)})`
         if (!normalizeUrl && preload) {
           imp =
             `(${isModernFlag}` +
diff --git a/packages/vite/src/node/plugins/css.ts b/packages/vite/src/node/plugins/css.ts
index 87add94d261845..e3532cec97cbf6 100644
--- a/packages/vite/src/node/plugins/css.ts
+++ b/packages/vite/src/node/plugins/css.ts
@@ -102,6 +102,7 @@ const htmlProxyRE = /(\?|&)html-proxy\b/
 const commonjsProxyRE = /\?commonjs-proxy/
 const inlineRE = /(\?|&)inline\b/
 const inlineCSSRE = /(\?|&)inline-css\b/
+const usedRE = /(\?|&)used\b/
 const varRE = /^var\(/i
 
 const enum PreprocessLang {
@@ -369,19 +370,18 @@ export function cssPostPlugin(config: ResolvedConfig): Plugin {
       }
 
       let code: string
-      if (modulesCode) {
-        code = modulesCode
-      } else {
-        let content = css
-        if (config.build.minify) {
-          content = await minifyCSS(content, config)
+      if (usedRE.test(id)) {
+        if (modulesCode) {
+          code = modulesCode
+        } else {
+          let content = css
+          if (config.build.minify) {
+            content = await minifyCSS(content, config)
+          }
+          code = `export default ${JSON.stringify(content)}`
         }
-        // marking as pure to make it tree-shakable by minifier
-        // but the module itself is still treated as a non tree-shakable module
-        // because moduleSideEffects is 'no-treeshake'
-        code = `export default /* #__PURE__ */ (() => ${JSON.stringify(
-          content
-        )})()`
+      } else {
+        code = `export default ''`
       }
 
       return {
diff --git a/packages/vite/src/node/plugins/importAnalysisBuild.ts b/packages/vite/src/node/plugins/importAnalysisBuild.ts
index dbe2ffd79770bf..effa22a0e5651a 100644
--- a/packages/vite/src/node/plugins/importAnalysisBuild.ts
+++ b/packages/vite/src/node/plugins/importAnalysisBuild.ts
@@ -5,11 +5,11 @@ import { init, parse as parseImports } from 'es-module-lexer'
 import type { OutputChunk, SourceMap } from 'rollup'
 import type { RawSourceMap } from '@ampproject/remapping'
 import { transformImportGlob } from '../importGlob'
-import { combineSourcemaps } from '../utils'
+import { bareImportRE, combineSourcemaps } from '../utils'
 import type { Plugin } from '../plugin'
 import type { ResolvedConfig } from '../config'
 import { genSourceMapUrl } from '../server/sourcemap'
-import { removedPureCssFilesCache } from './css'
+import { isCSSRequest, removedPureCssFilesCache } from './css'
 
 /**
  * A flag for injected helpers. This flag will be set to `false` if the output
@@ -148,6 +148,7 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin {
           e: end,
           ss: expStart,
           se: expEnd,
+          n: specifier,
           d: dynamicIndex
         } = imports[index]
 
@@ -194,6 +195,23 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin {
           const replacement = `${preloadMethod}(() => ${original},${isModernFlag}?"${preloadMarker}":void 0)`
           str().overwrite(expStart, expEnd, replacement, { contentOnly: true })
         }
+
+        // Differentiate CSS imports that use the default export from those that
+        // do not by injecting a ?used query - this allows us to avoid including
+        // the CSS string when unnecessary (esbuild has trouble tree-shaking
+        // them)
+        if (
+          specifier &&
+          isCSSRequest(specifier) &&
+          source.slice(expStart, start).includes('from') &&
+          // edge case for package names ending with .css (e.g normalize.css)
+          !(bareImportRE.test(specifier) && !specifier.includes('/'))
+        ) {
+          const url = specifier.replace(/\?|$/, (m) => `?used${m ? '&' : ''}`)
+          str().overwrite(start, end, dynamicIndex > -1 ? `'${url}'` : url, {
+            contentOnly: true
+          })
+        }
       }
 
       if (

From 68e1be657dbe870e11dcf1bb271c17d0d0b5f4d9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?=E7=BF=A0=20/=20green?= <green@sapphi.red>
Date: Mon, 9 May 2022 23:07:04 +0900
Subject: [PATCH 4/5] fix(css): backport #7746

---
 .../__tests__/css-codesplit.spec.ts            | 18 +++++++++++++++++-
 packages/playground/css-codesplit/async.css    |  3 +++
 packages/playground/css-codesplit/index.html   |  9 +++++++++
 packages/playground/css-codesplit/inline.css   |  3 +++
 packages/playground/css-codesplit/main.js      | 15 ++++++++++++---
 .../playground/css-codesplit/mod.module.css    |  3 +++
 .../src/node/plugins/importAnalysisBuild.ts    | 18 ++++++++++++------
 7 files changed, 59 insertions(+), 10 deletions(-)
 create mode 100644 packages/playground/css-codesplit/async.css
 create mode 100644 packages/playground/css-codesplit/inline.css
 create mode 100644 packages/playground/css-codesplit/mod.module.css

diff --git a/packages/playground/css-codesplit/__tests__/css-codesplit.spec.ts b/packages/playground/css-codesplit/__tests__/css-codesplit.spec.ts
index 95fe97a1b953ba..789adba23ae021 100644
--- a/packages/playground/css-codesplit/__tests__/css-codesplit.spec.ts
+++ b/packages/playground/css-codesplit/__tests__/css-codesplit.spec.ts
@@ -1,8 +1,23 @@
 import { findAssetFile, getColor, isBuild, readManifest } from '../../testUtils'
 
-test('should load both stylesheets', async () => {
+test('should load all stylesheets', async () => {
   expect(await getColor('h1')).toBe('red')
   expect(await getColor('h2')).toBe('blue')
+  expect(await getColor('.dynamic')).toBe('green')
+})
+
+test('should load dynamic import with inline', async () => {
+  const css = await page.textContent('.dynamic-inline')
+  expect(css).toMatch('.inline')
+
+  expect(await getColor('.inline')).not.toBe('yellow')
+})
+
+test('should load dynamic import with module', async () => {
+  const css = await page.textContent('.dynamic-module')
+  expect(css).toMatch('_mod_')
+
+  expect(await getColor('.mod')).toBe('yellow')
 })
 
 if (isBuild) {
@@ -10,6 +25,7 @@ if (isBuild) {
     expect(findAssetFile(/style.*\.js$/)).toBe('')
     expect(findAssetFile('main.*.js$')).toMatch(`/* empty css`)
     expect(findAssetFile('other.*.js$')).toMatch(`/* empty css`)
+    expect(findAssetFile(/async.*\.js$/)).toBe('')
   })
 
   test('should generate correct manifest', async () => {
diff --git a/packages/playground/css-codesplit/async.css b/packages/playground/css-codesplit/async.css
new file mode 100644
index 00000000000000..4902b2e7bee811
--- /dev/null
+++ b/packages/playground/css-codesplit/async.css
@@ -0,0 +1,3 @@
+.dynamic {
+  color: green;
+}
diff --git a/packages/playground/css-codesplit/index.html b/packages/playground/css-codesplit/index.html
index 6b7b3bb2b4dc2d..63bdb59e11dc6b 100644
--- a/packages/playground/css-codesplit/index.html
+++ b/packages/playground/css-codesplit/index.html
@@ -1,2 +1,11 @@
+<h1>This should be red</h1>
+<h2>This should be blue</h2>
+
+<p class="dynamic">This should be green</p>
+<p class="inline">This should not be yellow</p>
+<p class="dynamic-inline"></p>
+<p class="mod">This should be yellow</p>
+<p class="dynamic-module"></p>
+
 <script type="module" src="/main.js"></script>
 <div id="app"></div>
diff --git a/packages/playground/css-codesplit/inline.css b/packages/playground/css-codesplit/inline.css
new file mode 100644
index 00000000000000..b2a2b5f1ead51f
--- /dev/null
+++ b/packages/playground/css-codesplit/inline.css
@@ -0,0 +1,3 @@
+.inline {
+  color: yellow;
+}
diff --git a/packages/playground/css-codesplit/main.js b/packages/playground/css-codesplit/main.js
index 8c80df2c181511..eb6e703f79e718 100644
--- a/packages/playground/css-codesplit/main.js
+++ b/packages/playground/css-codesplit/main.js
@@ -1,6 +1,15 @@
 import './style.css'
 import './main.css'
 
-document.getElementById(
-  'app'
-).innerHTML = `<h1>This should be red</h1><h2>This should be blue</h2>`
+import('./async.css')
+
+import('./inline.css?inline').then((css) => {
+  document.querySelector('.dynamic-inline').textContent = css.default
+})
+
+import('./mod.module.css').then((css) => {
+  document.querySelector('.dynamic-module').textContent = JSON.stringify(
+    css.default
+  )
+  document.querySelector('.mod').classList.add(css.default.mod)
+})
diff --git a/packages/playground/css-codesplit/mod.module.css b/packages/playground/css-codesplit/mod.module.css
new file mode 100644
index 00000000000000..7f84410485a32c
--- /dev/null
+++ b/packages/playground/css-codesplit/mod.module.css
@@ -0,0 +1,3 @@
+.mod {
+  color: yellow;
+}
diff --git a/packages/vite/src/node/plugins/importAnalysisBuild.ts b/packages/vite/src/node/plugins/importAnalysisBuild.ts
index effa22a0e5651a..e3b42179fd6314 100644
--- a/packages/vite/src/node/plugins/importAnalysisBuild.ts
+++ b/packages/vite/src/node/plugins/importAnalysisBuild.ts
@@ -152,6 +152,8 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin {
           d: dynamicIndex
         } = imports[index]
 
+        const isDynamic = dynamicIndex > -1
+
         // import.meta.glob
         if (
           source.slice(start, end) === 'import.meta' &&
@@ -189,11 +191,13 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin {
           continue
         }
 
-        if (dynamicIndex > -1 && insertPreload) {
+        if (isDynamic && insertPreload) {
           needPreloadHelper = true
-          const original = source.slice(expStart, expEnd)
-          const replacement = `${preloadMethod}(() => ${original},${isModernFlag}?"${preloadMarker}":void 0)`
-          str().overwrite(expStart, expEnd, replacement, { contentOnly: true })
+          str().prependLeft(expStart, `${preloadMethod}(() => `)
+          str().appendRight(
+            expEnd,
+            `,${isModernFlag}?"${preloadMarker}":void 0)`
+          )
         }
 
         // Differentiate CSS imports that use the default export from those that
@@ -203,12 +207,14 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): Plugin {
         if (
           specifier &&
           isCSSRequest(specifier) &&
-          source.slice(expStart, start).includes('from') &&
+          // always inject ?used query when it is a dynamic import
+          // because there is no way to check whether the default export is used
+          (source.slice(expStart, start).includes('from') || isDynamic) &&
           // edge case for package names ending with .css (e.g normalize.css)
           !(bareImportRE.test(specifier) && !specifier.includes('/'))
         ) {
           const url = specifier.replace(/\?|$/, (m) => `?used${m ? '&' : ''}`)
-          str().overwrite(start, end, dynamicIndex > -1 ? `'${url}'` : url, {
+          str().overwrite(start, end, isDynamic ? `'${url}'` : url, {
             contentOnly: true
           })
         }

From 873d47482e9619a9c671163c9bc036c8c96625d7 Mon Sep 17 00:00:00 2001
From: sapphi-red <green@sapphi.red>
Date: Fri, 1 Jul 2022 13:08:00 +0900
Subject: [PATCH 5/5] test: skip failing test

---
 packages/playground/css/__tests__/css.spec.ts | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/packages/playground/css/__tests__/css.spec.ts b/packages/playground/css/__tests__/css.spec.ts
index c306479f0c38ff..e650670b285b94 100644
--- a/packages/playground/css/__tests__/css.spec.ts
+++ b/packages/playground/css/__tests__/css.spec.ts
@@ -342,7 +342,8 @@ test('PostCSS dir-dependency', async () => {
   }
 })
 
-test('import dependency includes css import', async () => {
+// skip because #8471 is reverted
+test.skip('import dependency includes css import', async () => {
   expect(await getColor('.css-js-dep')).toBe('green')
   expect(await getColor('.css-js-dep-module')).toBe('green')
 })
@@ -429,7 +430,8 @@ test('PostCSS source.input.from includes query', async () => {
   )
 })
 
-test('aliased css has content', async () => {
+// skip because #8471 is reverted
+test.skip('aliased css has content', async () => {
   expect(await getColor('.aliased')).toBe('blue')
   expect(await page.textContent('.aliased-content')).toMatch('.aliased')
   expect(await getColor('.aliased-module')).toBe('blue')