fix: getLogger not defined in child loader

package.json

@@ -64,6 +64,7 @@
     "eslint-plugin-import": "^2.22.1",
     "husky": "^4.3.0",
     "jest": "^26.6.1",
+    "less-loader": "^10.2.0",
     "lint-staged": "^10.5.0",
     "lodash": "^4.17.20",
     "memfs": "^3.2.0",

src/WorkerPool.js

@@ -285,6 +285,21 @@ class PoolWorker {
         finalCallback();
         break;
       }
+      case 'getLogger': {
+        // initialise logger by name in jobData
+        const { data } = message;
+        const { data: jobData } = this.jobs[id];
+        if (!Object.hasOwnProperty.call(jobData.loggers, data.name)) {
+          jobData.loggers[data.name] = jobData.getLogger(data.name);
+        }
+        break;
+      }
+      case 'logger': {
+        const { data } = message;
+        const { data: jobData } = this.jobs[id];
+        jobData.loggers[data.name][data.severity](data.message);
+        break;
+      }
       default: {
         console.error(`Unexpected worker message ${type} in WorkerPool.`);
         finalCallback();

src/index.js

@@ -25,6 +25,8 @@ function pitch() {
       sourceMap: this.sourceMap,
       emitError: this.emitError,
       emitWarning: this.emitWarning,
+      getLogger: this.getLogger,
+      loggers: {},
       loadModule: this.loadModule,
       resolve: this.resolve,
       getResolve: this.getResolve,

src/worker.js

@@ -212,6 +212,37 @@ const queue = asyncQueue(({ id, data }, taskCallback) => {
 
             return options;
           },
+          getLogger: (name) => {
+            function writeLoggerJson(severity, message) {
+              writeJson({
+                type: 'logger',
+                id,
+                data: { severity, name, message },
+              });
+            }
+            writeJson({
+              type: 'getLogger',
+              id,
+              data: { name },
+            });
+            return {
+              error(message) {
+                writeLoggerJson('error', message);
+              },
+
+              warn(message) {
+                writeLoggerJson('warn', message);
+              },
+
+              log(message) {
+                writeLoggerJson('log', message);
+              },
+
+              debug(message) {
+                writeLoggerJson('debug', message);
+              },
+            };
+          },
           emitWarning: (warning) => {
             writeJson({
               type: 'emitWarning',

test/less-loader-example/style.less

@@ -0,0 +1,3 @@
+body {
+  background: red;
+}

test/less-loader-example/webpack.config.js

@@ -0,0 +1,30 @@
+const path = require('path');
+
+const MiniCssExtractPlugin = require('mini-css-extract-plugin'); // eslint-disable-line import/no-extraneous-dependencies
+
+module.exports = {
+  mode: 'none',
+  context: __dirname,
+  entry: ['./style.less'],
+  output: {
+    path: path.resolve('dist'),
+  },
+  module: {
+    rules: [
+      {
+        test: /\.less$/,
+        use: [
+          MiniCssExtractPlugin.loader,
+          path.resolve(__dirname, '../../dist/index.js'),
+          'css-loader',
+          'less-loader',
+        ],
+      },
+    ],
+  },
+  plugins: [
+    new MiniCssExtractPlugin({
+      filename: 'style.css',
+    }),
+  ],
+};

test/sass-loader-example/assets/color_palette.scss

@@ -1 +1 @@
-$white: #FFFFFF;
+$white: #ffffff;

test/webpack.test.js

@@ -1,6 +1,7 @@
 import webpack from 'webpack';
 
 import sassLoaderConfig from './sass-loader-example/webpack.config';
+import lessLoaderConfig from './less-loader-example/webpack.config';
 
 test("Processes sass-loader's @import correctly", (done) => {
   const config = sassLoaderConfig({});
@@ -21,3 +22,21 @@ test("Processes sass-loader's @import correctly", (done) => {
     done();
   });
 }, 30000);
+
+test('Works with less-loader', (done) => {
+  webpack(lessLoaderConfig, (err, stats) => {
+    if (err) {
+      // eslint-disable-next-line no-console
+      console.error(err);
+    }
+
+    expect(err).toBe(null);
+
+    if (stats.hasErrors()) {
+      // eslint-disable-next-line no-console
+      console.error(stats.toJson().errors);
+    }
+    expect(stats.hasErrors()).toBe(false);
+    done();
+  });
+}, 30000);