feat: add eslint prefix to CLI output for better debugging (#155)

* feat: add eslint prefix to CLI output for better debugging

* fix: message

* test: remove eslint from parse message

* chore: update lock file

Author: ricardogobbosouza

package-lock.json

@@ -3,7 +3,7 @@ class ESLintError extends Error {
    * @param {string=} messages
    */
   constructor(messages) {
-    super(messages);
+    super(`[eslint] ${messages}`);
     this.name = 'ESLintError';
     this.stack = '';
   }

src/ESLintError.js

@@ -64,7 +64,8 @@ function linter(key, options, compilation) {
     }
     rawResults.push(
       lintFiles(files).catch((e) => {
-        compilation.errors.push(e);
+        // @ts-ignore
+        compilation.errors.push(new ESLintError(e.message));
         return [];
       })
     );

src/linter.js

@@ -10,7 +10,9 @@ describe('formatter eslint', () => {
       expect(stats.hasWarnings()).toBe(false);
       expect(stats.hasErrors()).toBe(true);
       expect(stats.compilation.errors[0].message).toBeTruthy();
-      const message = JSON.parse(stats.compilation.errors[0].message);
+      const message = JSON.parse(
+        stats.compilation.errors[0].message.replace('[eslint] ', '')
+      );
       expect(message.formatter).toEqual('mock');
       expect(message.results).toBeTruthy();
       done();
@@ -26,7 +28,9 @@ describe('formatter eslint', () => {
       expect(stats.hasWarnings()).toBe(false);
       expect(stats.hasErrors()).toBe(true);
       expect(stats.compilation.errors[0].message).toBeTruthy();
-      const message = JSON.parse(stats.compilation.errors[0].message);
+      const message = JSON.parse(
+        stats.compilation.errors[0].message.replace('[eslint] ', '')
+      );
       expect(message.formatter).toEqual('mock');
       expect(message.results).toBeTruthy();
       done();

test/formatter-custom.test.js

@@ -26,7 +26,7 @@ describe('formatter write', () => {
       expect(err).toBeNull();
       expect(stats.hasWarnings()).toBe(false);
       expect(stats.hasErrors()).toBe(true);
-      expect(stats.compilation.errors[0].message).toBe(contents);
+      expect(stats.compilation.errors[0].message).toBe(`[eslint] ${contents}`);
       done();
     });
   });
@@ -49,7 +49,7 @@ describe('formatter write', () => {
       expect(err).toBeNull();
       expect(stats.hasWarnings()).toBe(false);
       expect(stats.hasErrors()).toBe(true);
-      expect(stats.compilation.errors[0].message).toBe(contents);
+      expect(stats.compilation.errors[0].message).toBe(`[eslint] ${contents}`);
       done();
     });
   });