@roots/bud-compiler 2023.7.20-1957 → 2023.7.21-611

package/lib/service.js

@@ -9,6 +9,8 @@ import { bind } from '@roots/bud-support/decorators/bind';
 import { BudError } from '@roots/bud-support/errors';
 import { duration } from '@roots/bud-support/human-readable';
 import { render } from '@roots/bud-support/ink';
+import isNull from '@roots/bud-support/lodash/isNull';
+import isString from '@roots/bud-support/lodash/isString';
 import stripAnsi from '@roots/bud-support/strip-ansi';
 import webpack from '@roots/bud-support/webpack';
 /**
@@ -156,27 +158,38 @@ export class Compiler extends Service {
         if (!errors || !errors.length)
             return [];
         try {
-            const parseError = (error) => {
+            return errors
+                ?.map((error) => {
                 let file;
-                const moduleIdent = error.moduleId ?? error.moduleName;
+                let module;
+                const ident = error.moduleId ?? error.moduleName;
                 /**
                  * 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 (ident) {
+                    module = this.compilationStats.children
+                        .flatMap(child => child?.modules)
+                        .find(module => [module?.id, module?.name].includes(ident));
+                }
                 /**
                  * 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 (!ident && error.message?.includes(`[stylelint]`)) {
+                    // try to get the origin of the stylelint error,
+                    // which is contained in the second line of the error message
+                    const unparsedOrigin = error.message?.split(`\n`)?.[1];
+                    // if the origin is not a string or too long, we return the error as-is
+                    if (!isString(unparsedOrigin) || unparsedOrigin.length > 100)
+                        return error;
+                    // extract absolute path and context relative name of module
+                    const styleError = unparsedOrigin.match(/file:\/\/(.*)\x07(.*)\x1B]8;;/);
+                    if (isNull(styleError))
+                        return error;
+                    // get parts of matched error
+                    const [, file, name] = styleError;
+                    // return enriched error
+                    return { ...error, file, name, nameForCondition: file };
                 }
                 /**
                  * If the module is still not found, we return the error as-is
@@ -193,14 +206,13 @@ export class Compiler extends Service {
                 else if (module.name) {
                     file = this.app.path(`@src`, module.name);
                 }
-                return !file
-                    ? { ...error, name: module.name ?? error.name }
-                    : { ...error, file, name: module.name ?? error.name };
-            };
-            return errors?.map(parseError).filter(Boolean);
+                const name = module.name ?? error.name ?? `error`;
+                return { ...error, file, name };
+            })
+                .filter(Boolean);
         }
         catch (error) {
-            this.logger.warn(`error parsing errors`, error);
+            this.logger.warn(`Problem parsing errors. This probably won't break anything but please report it: https://github.com/roots/bud/issues/new`, error);
             return errors;
         }
     }
@@ -244,16 +256,12 @@ const statsOptions = {
         cachedAssets: true,
         cachedModules: true,
         entrypoints: true,
-        errorDetails: false,
         errors: true,
         errorsCount: true,
-        errorStack: false,
         hash: true,
         modules: true,
         name: true,
         outputPath: true,
-        reasons: false,
-        runtime: true,
         timings: true,
         warnings: true,
         warningsCount: true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@roots/bud-compiler",
-  "version": "2023.7.20-1957",
+  "version": "2023.7.21-611",
   "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.20-1957",
+    "@roots/bud-api": "2023.7.21-611",
     "@skypack/package-check": "0.2.2",
     "@types/node": "18.16.19",
     "@types/react": "18.2.15"
   },
   "dependencies": {
-    "@roots/bud-dashboard": "2023.7.20-1957",
-    "@roots/bud-framework": "2023.7.20-1957",
-    "@roots/bud-support": "2023.7.20-1957",
+    "@roots/bud-dashboard": "2023.7.21-611",
+    "@roots/bud-framework": "2023.7.21-611",
+    "@roots/bud-support": "2023.7.21-611",
     "react": "18.2.0",
     "tslib": "2.6.0"
   },

package/src/service.tsx

@@ -23,6 +23,8 @@ import {bind} from '@roots/bud-support/decorators/bind'
 import {BudError, type BudErrorClass} from '@roots/bud-support/errors'
 import {duration} from '@roots/bud-support/human-readable'
 import {render} from '@roots/bud-support/ink'
+import isNull from '@roots/bud-support/lodash/isNull'
+import isString from '@roots/bud-support/lodash/isString'
 import stripAnsi from '@roots/bud-support/strip-ansi'
 import webpack from '@roots/bud-support/webpack'
 
@@ -210,63 +212,71 @@ export class Compiler extends Service implements BudCompiler {
     if (!errors || !errors.length) return []
 
     try {
-      const parseError = (
-        error: StatsError,
-      ): ErrorWithSourceFile | StatsError => {
-        let file: SourceFile[`file`] | undefined
-
-        const moduleIdent = error.moduleId ?? error.moduleName
-
-        /**
-         * 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],
-            }
+      return errors
+        ?.map((error: StatsError): ErrorWithSourceFile | StatsError => {
+          let file: SourceFile[`file`] | undefined
+          let module: undefined | Webpack.StatsModule
+
+          const ident = error.moduleId ?? error.moduleName
+
+          /**
+           * In a perfect world webpack plugins would use the
+           * `nameForCondition` property to identify the module.
+           */
+          if (ident) {
+            module = this.compilationStats.children
+              .flatMap(child => child?.modules)
+              .find(module => [module?.id, module?.name].includes(ident))
           }
-        }
 
-        /**
-         * 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) {
-          file = this.app.path(`@src`, module.name)
-        }
+          /**
+           * If the module is not found, we try to parse the error message
+           */
+          if (!ident && error.message?.includes(`[stylelint]`)) {
+            // try to get the origin of the stylelint error,
+            // which is contained in the second line of the error message
+            const unparsedOrigin = error.message?.split(`\n`)?.[1]
+
+            // if the origin is not a string or too long, we return the error as-is
+            if (!isString(unparsedOrigin) || unparsedOrigin.length > 100)
+              return error
+
+            // extract absolute path and context relative name of module
+            const styleError = unparsedOrigin.match(
+              /file:\/\/(.*)\x07(.*)\x1B]8;;/,
+            )
+            if (isNull(styleError)) return error
+
+            // get parts of matched error
+            const [, file, name] = styleError
+            // return enriched error
+            return {...error, file, name, nameForCondition: file}
+          }
 
-        return !file
-          ? {...error, name: module.name ?? error.name}
-          : {...error, file, name: module.name ?? error.name}
-      }
+          /**
+           * 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) {
+            file = this.app.path(`@src`, module.name)
+          }
 
-      return errors?.map(parseError).filter(Boolean)
+          const name = module.name ?? error.name ?? `error`
+          return {...error, file, name}
+        })
+        .filter(Boolean)
     } catch (error) {
-      this.logger.warn(`error parsing errors`, error)
+      this.logger.warn(
+        `Problem parsing errors. This probably won't break anything but please report it: https://github.com/roots/bud/issues/new`,
+        error,
+      )
       return errors
     }
   }
@@ -281,16 +291,12 @@ const statsOptions = {
     cachedAssets: true,
     cachedModules: true,
     entrypoints: true,
-    errorDetails: false,
     errors: true,
     errorsCount: true,
-    errorStack: false,
     hash: true,
     modules: true,
     name: true,
     outputPath: true,
-    reasons: false,
-    runtime: true,
     timings: true,
     warnings: true,
     warningsCount: true,