fix: disable parallel less loader when on linux ant node version < 20.12.0

packages/mako/package.json

@@ -70,4 +70,4 @@
     "@umijs/mako-linux-x64-gnu": "0.4.17"
   },
   "repository": "[email protected]:umijs/mako.git"
-}
+}

packages/mako/src/lessLoader/index.ts

@@ -1,5 +1,16 @@
 import url from 'url';
-import { compile, terminatePool } from './parallelLessLoader';
+/**
+ * When on platform linux ant node version is lower then 20,
+ * the worker pool may exit unexpectedly, we need to disable it.
+ * This may be is problem of nodejs, may related issue: https://github.com/nodejs/node/issues/51129.
+ */
+const [NodeMajorVersion, NodeMirrorVersion] = process.versions.node
+  .split('.')
+  .map((v) => parseInt(v));
+const DisableParallelLess =
+  process.platform === 'linux' &&
+  (NodeMajorVersion < 20 ||
+    (NodeMajorVersion === 20 && NodeMirrorVersion < 12));
 
 export interface LessLoaderOpts {
   modifyVars: Record<string, string>;
@@ -32,7 +43,9 @@ function lessLoader(fn: Function | null, opts: LessLoaderOpts) {
       return;
     }
     if (pathname?.endsWith('.less')) {
-      return compile(pathname, opts);
+      return DisableParallelLess
+        ? require('./render').render(pathname, opts)
+        : require('./parallelLessLoader').render(pathname, opts);
     } else {
       // TODO: remove this
       fn && fn(filePath);
@@ -41,7 +54,7 @@ function lessLoader(fn: Function | null, opts: LessLoaderOpts) {
 }
 
 lessLoader.terminate = () => {
-  terminatePool();
+  !DisableParallelLess && require('./parallelLessLoader').terminatePool();
 };
 
 export { lessLoader };

packages/mako/src/lessLoader/lessLoader.worker.ts

@@ -1,42 +1,4 @@
-import fs from 'fs';
-import less from 'less';
 import workerpool from 'workerpool';
-import { LessLoaderOpts } from '.';
+import { render } from './render';
 
-const lessLoader = {
-  render: async function (
-    filePath: string,
-    opts: LessLoaderOpts,
-  ): Promise<string> {
-    const { modifyVars, math, sourceMap, plugins } = opts;
-    const input = fs.readFileSync(filePath, 'utf-8');
-
-    const pluginInstances: Less.Plugin[] | undefined = plugins?.map((p) => {
-      if (Array.isArray(p)) {
-        const pluginModule = require(p[0]);
-        const PluginClass = pluginModule.default || pluginModule;
-        return new PluginClass(p[1]);
-      } else {
-        return require(p);
-      }
-    });
-
-    const result = await less
-      .render(input, {
-        filename: filePath,
-        javascriptEnabled: true,
-        math,
-        plugins: pluginInstances,
-        modifyVars,
-        sourceMap,
-        rewriteUrls: 'all',
-      } as unknown as Less.Options)
-      .catch((err) => {
-        throw new Error(err.toString());
-      });
-
-    return result.css;
-  },
-};
-
-workerpool.worker(lessLoader);
+workerpool.worker({ render });

packages/mako/src/lessLoader/parallelLessLoader.ts

@@ -15,13 +15,13 @@ export function terminatePool() {
   pool = undefined;
 }
 
-export async function compile(
+export async function render(
   filePath: string,
   opts: LessLoaderOpts,
 ): Promise<{ content: string; type: string }> {
   createPool();
 
-  const css = await pool!.exec('render', [filePath, opts]);
+  const res = await pool!.exec('render', [filePath, opts]);
 
-  return { content: css, type: 'css' };
+  return res;
 }

packages/mako/src/lessLoader/render.ts

@@ -0,0 +1,37 @@
+import fs from 'fs';
+import less from 'less';
+import { LessLoaderOpts } from '.';
+
+export async function render(
+  filePath: string,
+  opts: LessLoaderOpts,
+): Promise<{ content: string; type: 'css' }> {
+  const { modifyVars, math, sourceMap, plugins } = opts;
+  const input = fs.readFileSync(filePath, 'utf-8');
+
+  const pluginInstances: Less.Plugin[] | undefined = plugins?.map((p) => {
+    if (Array.isArray(p)) {
+      const pluginModule = require(p[0]);
+      const PluginClass = pluginModule.default || pluginModule;
+      return new PluginClass(p[1]);
+    } else {
+      return require(p);
+    }
+  });
+
+  const result = await less
+    .render(input, {
+      filename: filePath,
+      javascriptEnabled: true,
+      math,
+      plugins: pluginInstances,
+      modifyVars,
+      sourceMap,
+      rewriteUrls: 'all',
+    } as unknown as Less.Options)
+    .catch((err) => {
+      throw new Error(err.toString());
+    });
+
+  return { content: result.css, type: 'css' };
+}