@roots/bud-compiler 2023.7.18-645 → 2023.7.20-622

package/lib/service.js

@@ -44,7 +44,11 @@ export class Compiler extends Service {
             : await Promise.all(Object.values(bud.children).map(async (child) => await child.build.make().catch(error => {
                 throw error;
             })));
-        this.config.parallelism = Math.max(cpus().length - 1, 1);
+        const cores = Math.max(cpus().length, 1);
+        const compilations = Math.max(Object.keys(bud.children ?? []).length, 1);
+        const parallelism = Math.max(Math.floor(cores / compilations), 1);
+        this.config.parallelism = parallelism;
+        this.logger.info(`parallel compilations: ${this.config.parallelism}`);
         await bud.hooks.fire(`compiler.before`, bud).catch(error => {
             throw error;
         });
@@ -156,11 +160,34 @@ export class Compiler extends Service {
             const parseError = (error) => {
                 let file;
                 const moduleIdent = error.moduleId ?? error.moduleName;
-                const module = this.compilationStats.children
+                /**
+                 * In a perfect world webpack plugins would use the
+                 * `nameForCondition` property to identify the module.
+                 */
+                let module = this.compilationStats.children
                     .flatMap(child => child?.modules)
                     .find(module => module?.id === moduleIdent || module?.name === moduleIdent);
+                /**
+                 * If the module is not found, we try to parse the error message
+                 */
+                if (!moduleIdent) {
+                    const stylelintExtracted = error.message.match(/file:\/\/(.*)\x07(.*)\x1B]8;;/);
+                    if (stylelintExtracted?.[1]) {
+                        module = {
+                            name: stylelintExtracted[2] ?? stylelintExtracted[1],
+                            nameForCondition: stylelintExtracted[1],
+                        };
+                    }
+                }
+                /**
+                 * If the module is still not found, we return the error as-is
+                 */
                 if (!module)
                     return error;
+                /**
+                 * We'll prefer the `nameForCondition` property if it exists,
+                 * otherwise we'll use the `name` property.
+                 */
                 if (module.nameForCondition) {
                     file = module.nameForCondition;
                 }
@@ -174,8 +201,8 @@ export class Compiler extends Service {
             return errors?.map(parseError).filter(Boolean);
         }
         catch (error) {
-            this.app.warn(`error parsing errors`, error);
-            return [];
+            this.logger.warn(`error parsing errors`, error);
+            return errors;
         }
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@roots/bud-compiler",
-  "version": "2023.7.18-645",
+  "version": "2023.7.20-622",
   "description": "Compilation handler",
   "engines": {
     "node": ">=16"
@@ -61,15 +61,15 @@
   "types": "./lib/index.d.ts",
   "module": "./lib/index.js",
   "devDependencies": {
-    "@roots/bud-api": "2023.7.18-645",
+    "@roots/bud-api": "2023.7.20-622",
     "@skypack/package-check": "0.2.2",
     "@types/node": "18.16.19",
     "@types/react": "18.2.15"
   },
   "dependencies": {
-    "@roots/bud-dashboard": "2023.7.18-645",
-    "@roots/bud-framework": "2023.7.18-645",
-    "@roots/bud-support": "2023.7.18-645",
+    "@roots/bud-dashboard": "2023.7.20-622",
+    "@roots/bud-framework": "2023.7.20-622",
+    "@roots/bud-support": "2023.7.20-622",
     "react": "18.2.0",
     "tslib": "2.6.0"
   },

package/src/service.tsx

@@ -70,7 +70,16 @@ export class Compiler extends Service implements BudCompiler {
               }),
           ),
         )
-    this.config.parallelism = Math.max(cpus().length - 1, 1)
+
+    const cores = Math.max(cpus().length, 1)
+    const compilations = Math.max(
+      Object.keys(bud.children ?? []).length,
+      1,
+    )
+    const parallelism = Math.max(Math.floor(cores / compilations), 1)
+
+    this.config.parallelism = parallelism
+    this.logger.info(`parallel compilations: ${this.config.parallelism}`)
 
     await bud.hooks.fire(`compiler.before`, bud).catch(error => {
       throw error
@@ -158,6 +167,7 @@ export class Compiler extends Service implements BudCompiler {
               subtitle: error.file ? `Error in ${error.name}` : error.name,
               title: makeNoticeTitle(child),
             })
+
             this.app.notifier.openEditor(error.file)
           } catch (error) {
             this.logger.error(error)
@@ -214,15 +224,42 @@ export class Compiler extends Service implements BudCompiler {
 
         const moduleIdent = error.moduleId ?? error.moduleName
 
-        const module = this.compilationStats.children
+        /**
+         * In a perfect world webpack plugins would use the
+         * `nameForCondition` property to identify the module.
+         */
+        let module = this.compilationStats.children
           .flatMap(child => child?.modules)
           .find(
             module =>
               module?.id === moduleIdent || module?.name === moduleIdent,
           )
 
+        /**
+         * If the module is not found, we try to parse the error message
+         */
+        if (!moduleIdent) {
+          const stylelintExtracted = error.message.match(
+            /file:\/\/(.*)\x07(.*)\x1B]8;;/,
+          )
+
+          if (stylelintExtracted?.[1]) {
+            module = {
+              name: stylelintExtracted[2] ?? stylelintExtracted[1],
+              nameForCondition: stylelintExtracted[1],
+            }
+          }
+        }
+
+        /**
+         * If the module is still not found, we return the error as-is
+         */
         if (!module) return error
 
+        /**
+         * We'll prefer the `nameForCondition` property if it exists,
+         * otherwise we'll use the `name` property.
+         */
         if (module.nameForCondition) {
           file = module.nameForCondition
         } else if (module.name) {
@@ -236,8 +273,8 @@ export class Compiler extends Service implements BudCompiler {
 
       return errors?.map(parseError).filter(Boolean)
     } catch (error) {
-      this.app.warn(`error parsing errors`, error)
-      return []
+      this.logger.warn(`error parsing errors`, error)
+      return errors
     }
   }
 }