wip: use builtin css handling

rolldown · Sep 26, 2024 · 1f97ea1 · 1f97ea1
1 parent 8be28c1
commit 1f97ea1
Show file tree

Hide file tree

Showing 5 changed files with 16 additions and 18 deletions.
diff --git a/packages/vite/src/node/build.ts b/packages/vite/src/node/build.ts
@@ -649,7 +649,19 @@ export async function buildEnvironment(
     // TODO: remove this and enable rolldown's CSS support later
     moduleTypes: {
       ...options.rollupOptions.moduleTypes,
-      '.css': 'js',
+      // https://github.com/rolldown/rolldown/blob/4020de442a8ab0f7973794ead3b8aa04e316d558/crates/rolldown/src/module_loader/module_task.rs#L120
+      // @ts-expect-error css
+      '.sass': 'css',
+      // @ts-expect-error css
+      '.scss': 'css',
+      // @ts-expect-error css
+      '.sss': 'css',
+      // @ts-expect-error css
+      '.styl': 'css',
+      // @ts-expect-error css
+      '.stylus': 'css',
+      // @ts-expect-error css
+      '.less': 'css',
     },
   }
 

diff --git a/packages/vite/src/node/optimizer/index.ts b/packages/vite/src/node/optimizer/index.ts
@@ -781,11 +781,6 @@ async function prepareRolldownOptimizerRun(
         conditionNames: ['browser'],
       },
       ...rollupOptions,
-      // TODO: remove this and enable rolldown's CSS support later
-      moduleTypes: {
-        '.css': 'js',
-        ...rollupOptions.moduleTypes,
-      },
     })
     if (canceled) {
       await bundle.close()
@@ -1067,11 +1062,6 @@ export async function extractExportsData(
       ...remainingRollupOptions,
       plugins,
       input: [filePath],
-      // TODO: remove this and enable rolldown's CSS support later
-      moduleTypes: {
-        '.css': 'js',
-        ...remainingRollupOptions.moduleTypes,
-      },
     })
     const result = await build.generate({
       ...rollupOptions.output,

diff --git a/packages/vite/src/node/plugins/css.ts b/packages/vite/src/node/plugins/css.ts
@@ -577,7 +577,7 @@ export function cssPostPlugin(config: ResolvedConfig): RolldownPlugin {
           code = `export default ${JSON.stringify(content)}`
         } else {
           // empty module when it's not a CSS module nor `?inline`
-          code = ''
+          code = css
         }
 
         return {

diff --git a/packages/vite/src/node/plugins/importAnalysisBuild.ts b/packages/vite/src/node/plugins/importAnalysisBuild.ts
@@ -221,7 +221,8 @@ export function buildImportAnalysisPlugin(config: ResolvedConfig): [Plugin] {
       }
     },
 
-    async transform(source, importer) {
+    async transform(source, importer, opts) {
+      if (opts?.moduleType === 'css') return
       if (isInNodeModules(importer) && !dynamicImportPrefixRE.test(source)) {
         return
       }

diff --git a/packages/vite/src/node/plugins/worker.ts b/packages/vite/src/node/plugins/worker.ts
@@ -85,11 +85,6 @@ async function bundleWorkerEntry(
     onwarn(warning, warn) {
       onRollupWarning(warning, warn, workerEnvironment)
     },
-    // TODO: remove this and enable rolldown's CSS support later
-    moduleTypes: {
-      '.css': 'js',
-      ...rollupOptions.moduleTypes,
-    },
     // preserveEntrySignatures: false,
   })
   let chunk: OutputChunk