sass-loader 13.3.0 → 13.3.2

package/README.md

@@ -98,6 +98,10 @@ module.exports = {
 
 Finally run `webpack` via your preferred method.
 
+### The `outputStyle` (old API) and `style` (new API) options in `production` mode
+
+For `production` mode, the `outputStyle` (old API) and `style` (new API) options default to `compressed` unless otherwise specified in `sassOptions`.
+
 ### Resolving `import` at-rules
 
 Webpack provides an [advanced mechanism to resolve files](https://webpack.js.org/concepts/module-resolution/).

package/dist/index.js

@@ -8,7 +8,6 @@ var _url = _interopRequireDefault(require("url"));
 var _path = _interopRequireDefault(require("path"));
 var _options = _interopRequireDefault(require("./options.json"));
 var _utils = require("./utils");
-var _SassError = _interopRequireDefault(require("./SassError"));
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 /**
  * The sass-loader makes node-sass and dart-sass available to webpack modules.
@@ -19,9 +18,11 @@ function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { de
 async function loader(content) {
   const options = this.getOptions(_options.default);
   const callback = this.async();
-  const implementation = (0, _utils.getSassImplementation)(this, options.implementation);
-  if (!implementation) {
-    callback();
+  let implementation;
+  try {
+    implementation = (0, _utils.getSassImplementation)(this, options.implementation);
+  } catch (error) {
+    callback(error);
     return;
   }
   const useSourceMap = typeof options.sourceMap === "boolean" ? options.sourceMap : this.sourceMap;
@@ -38,7 +39,13 @@ async function loader(content) {
       sassOptions.importers.push((0, _utils.getModernWebpackImporter)(this, implementation));
     }
   }
-  const compile = (0, _utils.getCompileFn)(implementation, options);
+  let compile;
+  try {
+    compile = (0, _utils.getCompileFn)(implementation, options);
+  } catch (error) {
+    callback(error);
+    return;
+  }
   let result;
   try {
     result = await compile(sassOptions, options);
@@ -53,7 +60,7 @@ async function loader(content) {
       // `node-sass` returns POSIX paths
       this.addDependency(_path.default.normalize(error.file));
     }
-    callback(new _SassError.default(error));
+    callback((0, _utils.errorFactory)(error));
     return;
   }
   let map =

package/dist/utils.js

@@ -3,6 +3,7 @@
 Object.defineProperty(exports, "__esModule", {
   value: true
 });
+exports.errorFactory = errorFactory;
 exports.getCompileFn = getCompileFn;
 exports.getModernWebpackImporter = getModernWebpackImporter;
 exports.getSassImplementation = getSassImplementation;
@@ -13,9 +14,6 @@ exports.isSupportedFibers = isSupportedFibers;
 exports.normalizeSourceMap = normalizeSourceMap;
 var _url = _interopRequireDefault(require("url"));
 var _path = _interopRequireDefault(require("path"));
-var _full = require("klona/full");
-var _neoAsync = _interopRequireDefault(require("neo-async"));
-var _SassWarning = _interopRequireDefault(require("./SassWarning"));
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 function getDefaultSassImplementation() {
   let sassImplPkg = "sass";
@@ -45,35 +43,21 @@ function getDefaultSassImplementation() {
 function getSassImplementation(loaderContext, implementation) {
   let resolvedImplementation = implementation;
   if (!resolvedImplementation) {
-    try {
-      resolvedImplementation = getDefaultSassImplementation();
-    } catch (error) {
-      loaderContext.emitError(error);
-      return;
-    }
+    resolvedImplementation = getDefaultSassImplementation();
   }
   if (typeof resolvedImplementation === "string") {
-    try {
-      // eslint-disable-next-line import/no-dynamic-require, global-require
-      resolvedImplementation = require(resolvedImplementation);
-    } catch (error) {
-      loaderContext.emitError(error);
-
-      // eslint-disable-next-line consistent-return
-      return;
-    }
+    // eslint-disable-next-line import/no-dynamic-require, global-require
+    resolvedImplementation = require(resolvedImplementation);
   }
   const {
     info
   } = resolvedImplementation;
   if (!info) {
-    loaderContext.emitError(new Error("Unknown Sass implementation."));
-    return;
+    throw new Error("Unknown Sass implementation.");
   }
   const infoParts = info.split("\t");
   if (infoParts.length < 2) {
-    loaderContext.emitError(new Error(`Unknown Sass implementation "${info}".`));
-    return;
+    throw new Error(`Unknown Sass implementation "${info}".`);
   }
   const [implementationName] = infoParts;
   if (implementationName === "dart-sass") {
@@ -86,7 +70,7 @@ function getSassImplementation(loaderContext, implementation) {
     // eslint-disable-next-line consistent-return
     return resolvedImplementation;
   }
-  loaderContext.emitError(new Error(`Unknown Sass implementation "${implementationName}".`));
+  throw new Error(`Unknown Sass implementation "${implementationName}".`);
 }
 
 /**
@@ -121,16 +105,17 @@ function isSupportedFibers() {
  * @returns {Object}
  */
 async function getSassOptions(loaderContext, loaderOptions, content, implementation, useSourceMap) {
-  const options = (0, _full.klona)(loaderOptions.sassOptions ? typeof loaderOptions.sassOptions === "function" ? loaderOptions.sassOptions(loaderContext) || {} : loaderOptions.sassOptions : {});
-  const isDartSass = implementation.info.includes("dart-sass");
-  const isModernAPI = loaderOptions.api === "modern";
-  options.data = loaderOptions.additionalData ? typeof loaderOptions.additionalData === "function" ? await loaderOptions.additionalData(content, loaderContext) : `${loaderOptions.additionalData}\n${content}` : content;
-  if (!options.logger) {
+  const options = loaderOptions.sassOptions ? typeof loaderOptions.sassOptions === "function" ? loaderOptions.sassOptions(loaderContext) || {} : loaderOptions.sassOptions : {};
+  const sassOptions = {
+    ...options,
+    data: loaderOptions.additionalData ? typeof loaderOptions.additionalData === "function" ? await loaderOptions.additionalData(content, loaderContext) : `${loaderOptions.additionalData}\n${content}` : content
+  };
+  if (!sassOptions.logger) {
     const needEmitWarning = loaderOptions.warnRuleAsWarning !== false;
     const logger = loaderContext.getLogger("sass-loader");
     const formatSpan = span => `${span.url || "-"}:${span.start.line}:${span.start.column}: `;
     const formatDebugSpan = span => `[debug:${span.start.line}:${span.start.column}] `;
-    options.logger = {
+    sassOptions.logger = {
       debug(message, loggerOptions) {
         let builtMessage = "";
         if (loggerOptions.span) {
@@ -152,43 +137,48 @@ async function getSassOptions(loaderContext, loaderOptions, content, implementat
           builtMessage += `\n\n${loggerOptions.stack}`;
         }
         if (needEmitWarning) {
-          loaderContext.emitWarning(new _SassWarning.default(builtMessage, loggerOptions));
+          const warning = new Error(builtMessage);
+          warning.name = "SassWarning";
+          warning.stack = null;
+          loaderContext.emitWarning(warning);
         } else {
           logger.warn(builtMessage);
         }
       }
     };
   }
+  const isModernAPI = loaderOptions.api === "modern";
   const {
     resourcePath
   } = loaderContext;
   if (isModernAPI) {
-    options.url = _url.default.pathToFileURL(resourcePath);
+    sassOptions.url = _url.default.pathToFileURL(resourcePath);
 
     // opt.outputStyle
-    if (!options.style && isProductionLikeMode(loaderContext)) {
-      options.style = "compressed";
+    if (!sassOptions.style && isProductionLikeMode(loaderContext)) {
+      sassOptions.style = "compressed";
     }
     if (useSourceMap) {
-      options.sourceMap = true;
+      sassOptions.sourceMap = true;
     }
 
     // If we are compiling sass and indentedSyntax isn't set, automatically set it.
-    if (typeof options.syntax === "undefined") {
+    if (typeof sassOptions.syntax === "undefined") {
       const ext = _path.default.extname(resourcePath);
       if (ext && ext.toLowerCase() === ".scss") {
-        options.syntax = "scss";
+        sassOptions.syntax = "scss";
       } else if (ext && ext.toLowerCase() === ".sass") {
-        options.syntax = "indented";
+        sassOptions.syntax = "indented";
       } else if (ext && ext.toLowerCase() === ".css") {
-        options.syntax = "css";
+        sassOptions.syntax = "css";
       }
     }
-    options.importers = options.importers ? Array.isArray(options.importers) ? options.importers : [options.importers] : [];
+    sassOptions.importers = sassOptions.importers ? Array.isArray(sassOptions.importers) ? sassOptions.importers.slice() : [sassOptions.importers] : [];
   } else {
-    options.file = resourcePath;
+    sassOptions.file = resourcePath;
+    const isDartSass = implementation.info.includes("dart-sass");
     if (isDartSass && isSupportedFibers()) {
-      const shouldTryToResolveFibers = !options.fiber && options.fiber !== false;
+      const shouldTryToResolveFibers = !sassOptions.fiber && sassOptions.fiber !== false;
       if (shouldTryToResolveFibers) {
         let fibers;
         try {
@@ -198,20 +188,20 @@ async function getSassOptions(loaderContext, loaderOptions, content, implementat
         }
         if (fibers) {
           // eslint-disable-next-line global-require, import/no-dynamic-require
-          options.fiber = require(fibers);
+          sassOptions.fiber = require(fibers);
         }
-      } else if (options.fiber === false) {
+      } else if (sassOptions.fiber === false) {
         // Don't pass the `fiber` option for `sass` (`Dart Sass`)
-        delete options.fiber;
+        delete sassOptions.fiber;
       }
     } else {
       // Don't pass the `fiber` option for `node-sass`
-      delete options.fiber;
+      delete sassOptions.fiber;
     }
 
     // opt.outputStyle
-    if (!options.outputStyle && isProductionLikeMode(loaderContext)) {
-      options.outputStyle = "compressed";
+    if (!sassOptions.outputStyle && isProductionLikeMode(loaderContext)) {
+      sassOptions.outputStyle = "compressed";
     }
     if (useSourceMap) {
       // Deliberately overriding the sourceMap option here.
@@ -220,31 +210,31 @@ async function getSassOptions(loaderContext, loaderOptions, content, implementat
       // But since we're using the data option, the source map will not actually be written, but
       // all paths in sourceMap.sources will be relative to that path.
       // Pretty complicated... :(
-      options.sourceMap = true;
-      options.outFile = _path.default.join(loaderContext.rootContext, "style.css.map");
-      options.sourceMapContents = true;
-      options.omitSourceMapUrl = true;
-      options.sourceMapEmbed = false;
+      sassOptions.sourceMap = true;
+      sassOptions.outFile = _path.default.join(loaderContext.rootContext, "style.css.map");
+      sassOptions.sourceMapContents = true;
+      sassOptions.omitSourceMapUrl = true;
+      sassOptions.sourceMapEmbed = false;
     }
     const ext = _path.default.extname(resourcePath);
 
     // If we are compiling sass and indentedSyntax isn't set, automatically set it.
-    if (ext && ext.toLowerCase() === ".sass" && typeof options.indentedSyntax === "undefined") {
-      options.indentedSyntax = true;
+    if (ext && ext.toLowerCase() === ".sass" && typeof sassOptions.indentedSyntax === "undefined") {
+      sassOptions.indentedSyntax = true;
     } else {
-      options.indentedSyntax = Boolean(options.indentedSyntax);
+      sassOptions.indentedSyntax = Boolean(sassOptions.indentedSyntax);
     }
 
     // Allow passing custom importers to `sass`/`node-sass`. Accepts `Function` or an array of `Function`s.
-    options.importer = options.importer ? proxyCustomImporters(Array.isArray(options.importer) ? options.importer : [options.importer], loaderContext) : [];
-    options.includePaths = [].concat(process.cwd()).concat(
+    sassOptions.importer = sassOptions.importer ? proxyCustomImporters(Array.isArray(sassOptions.importer) ? sassOptions.importer.slice() : [sassOptions.importer], loaderContext) : [];
+    sassOptions.includePaths = [].concat(process.cwd()).concat(
     // We use `includePaths` in context for resolver, so it should be always absolute
-    (options.includePaths || []).map(includePath => _path.default.isAbsolute(includePath) ? includePath : _path.default.join(process.cwd(), includePath))).concat(process.env.SASS_PATH ? process.env.SASS_PATH.split(process.platform === "win32" ? ";" : ":") : []);
-    if (typeof options.charset === "undefined") {
-      options.charset = true;
+    (sassOptions.includePaths ? sassOptions.includePaths.slice() : []).map(includePath => _path.default.isAbsolute(includePath) ? includePath : _path.default.join(process.cwd(), includePath))).concat(process.env.SASS_PATH ? process.env.SASS_PATH.split(process.platform === "win32" ? ";" : ":") : []);
+    if (typeof sassOptions.charset === "undefined") {
+      sassOptions.charset = true;
     }
   }
-  return options;
+  return sassOptions;
 }
 const MODULE_REQUEST_REGEX = /^[^?]*~/;
 
@@ -564,7 +554,10 @@ function getCompileFn(implementation, options) {
   // We need to use a job queue to make sure that one thread is always available to the UV lib
   if (nodeSassJobQueue === null) {
     const threadPoolSize = Number(process.env.UV_THREADPOOL_SIZE || 4);
-    nodeSassJobQueue = _neoAsync.default.queue(implementation.render.bind(implementation), threadPoolSize - 1);
+    // Only used for `node-sass`, so let's load it lazily
+    // eslint-disable-next-line global-require
+    const async = require("neo-async");
+    nodeSassJobQueue = async.queue(implementation.render.bind(implementation), threadPoolSize - 1);
   }
   return sassOptions => new Promise((resolve, reject) => {
     nodeSassJobQueue.push.bind(nodeSassJobQueue)(sassOptions, (error, result) => {
@@ -623,4 +616,20 @@ function normalizeSourceMap(map, rootContext) {
     return source;
   });
   return newMap;
+}
+function errorFactory(error) {
+  let message;
+  if (error.formatted) {
+    message = error.formatted.replace(/^Error: /, "");
+  } else {
+    // Keep original error if `sassError.formatted` is unavailable
+    ({
+      message
+    } = error);
+  }
+  const obj = new Error(message, {
+    cause: error
+  });
+  obj.stack = null;
+  return obj;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sass-loader",
-  "version": "13.3.0",
+  "version": "13.3.2",
   "description": "Sass loader for webpack",
   "license": "MIT",
   "repository": "webpack-contrib/sass-loader",
@@ -63,7 +63,6 @@
     }
   },
   "dependencies": {
-    "klona": "^2.0.6",
     "neo-async": "^2.6.2"
   },
   "devDependencies": {

package/dist/SassError.js

@@ -1,31 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.default = void 0;
-class SassError extends Error {
-  constructor(sassError) {
-    super();
-    this.name = "SassError";
-
-    // Instruct webpack to hide the JS stack from the console.
-    // Usually you're only interested in the SASS error in this case.
-    this.hideStack = true;
-    Error.captureStackTrace(this, this.constructor);
-    if (typeof sassError.line !== "undefined" || typeof sassError.column !== "undefined") {
-      this.loc = {
-        line: sassError.line,
-        column: sassError.column
-      };
-    }
-
-    // Keep original error if `sassError.formatted` is unavailable
-    this.message = `${this.name}: ${typeof sassError.message !== "undefined" ? sassError.message : sassError}`;
-    if (sassError.formatted) {
-      this.message = `${this.name}: ${sassError.formatted.replace(/^Error: /, "")}`;
-    }
-  }
-}
-var _default = SassError;
-exports.default = _default;

package/dist/SassWarning.js

@@ -1,21 +0,0 @@
-"use strict";
-
-Object.defineProperty(exports, "__esModule", {
-  value: true
-});
-exports.default = void 0;
-class SassWarning extends Error {
-  constructor(warning, options) {
-    super(warning);
-    this.name = "SassWarning";
-    this.hideStack = true;
-    if (options.span) {
-      this.loc = {
-        line: options.span.start.line,
-        column: options.span.start.column
-      };
-    }
-  }
-}
-var _default = SassWarning;
-exports.default = _default;