Merge remote-tracking branch 'origin/fix-sass-canonicalize-non-css' i…

…nto fix-sass-canonicalize-non-css
vitejs · Dec 16, 2024 · c28df71 · c28df71
2 parents 3e5ea27 + f0ac437
commit c28df71
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/packages/vite/src/node/plugins/css.ts b/packages/vite/src/node/plugins/css.ts
@@ -2382,6 +2382,11 @@ const makeModernScssWorker = (
             const resolved = await internalCanonicalize(url, importer)
             if (
               resolved &&
+              // only limit to these extensions because:
+              // - for the `@import`/`@use`s written in file loaded by `load` function,
+              //   the `canonicalize` function of that `importer` is called first
+              // - the `load` function of an importer is only called for the importer
+              //   that returned a non-null result from its `canonicalize` function
               (resolved.endsWith('.css') ||
                 resolved.endsWith('.scss') ||
                 resolved.endsWith('.sass'))