Fix ESLint warning and error reporting (#826)

.changeset/silly-countries-drive.md

@@ -0,0 +1,5 @@
+---
+'sku': patch
+---
+
+Fix ESLint warning and error reporting

packages/sku/lib/runESLint.js

@@ -1,3 +1,4 @@
+// @ts-check
 const { yellow, cyan, gray } = require('chalk');
 const { ESLint } = require('eslint');
 const eslintConfig = require('../config/eslint/eslintConfig');
@@ -6,7 +7,7 @@ const { lintExtensions } = require('./lint');
 const extensions = lintExtensions.map((ext) => `.${ext}`);
 
 /**
- * @param {{ fix?: boolean, paths?: string[] }}
+ * @param {{ fix?: boolean, paths?: string[] }} options
  */
 const runESLint = async ({ fix = false, paths }) => {
   console.log(cyan(`${fix ? 'Fixing' : 'Checking'} code with ESLint`));
@@ -30,16 +31,24 @@ const runESLint = async ({ fix = false, paths }) => {
   } else {
     console.log(gray(`Paths: ${filteredFilePaths.join(' ')}`));
     try {
-      const report = await eslint.lintFiles(filteredFilePaths);
+      const lintResults = await eslint.lintFiles(filteredFilePaths);
 
       if (fix) {
-        ESLint.outputFixes(report);
+        ESLint.outputFixes(lintResults);
       } else {
-        const { errorCount, warningCount, results } = report;
+        const { warningCount, errorCount } = lintResults.reduce(
+          (acc, result) => {
+            return {
+              warningCount: acc.warningCount + result.warningCount,
+              errorCount: acc.errorCount + result.errorCount,
+            };
+          },
+          { warningCount: 0, errorCount: 0 },
+        );
 
         if (errorCount || warningCount) {
           const formatter = await eslint.loadFormatter();
-          console.log(formatter(results));
+          console.log(await formatter.format(lintResults));
         }
 
         if (errorCount > 0) {
@@ -57,6 +66,8 @@ const runESLint = async ({ fix = false, paths }) => {
 };
 
 module.exports = {
+  /** @param {string[] | undefined} paths */
   check: (paths) => runESLint({ paths }),
+  /** @param {string[] | undefined} paths */
   fix: (paths) => runESLint({ fix: true, paths }),
 };