@aklinker1/check 1.0.5 → 1.1.1

package/dist/index.d.ts

@@ -1,4 +1,5 @@
 import type { CheckOptions, Problem } from "./types";
 export type * from "./types";
 export declare function check(options?: CheckOptions): Promise<void>;
+export declare function renderProblemGroup(problems: Problem[]): string;
 export declare function renderProblem(problem: Problem): string;

package/dist/index.mjs

@@ -37,7 +37,13 @@ export async function check(options = {}) {
       return problems2;
     }
   );
-  const problems = results.flat().sort((l, r) => {
+  const problems = results.flat();
+  console.log();
+  if (problems.length === 0) {
+    process.exit(0);
+  }
+  console.log(plural(problems.length, "Problem:", "Problems:"));
+  problems.sort((l, r) => {
     const nameCompare = l.file.localeCompare(r.file);
     if (nameCompare !== 0)
       return nameCompare;
@@ -46,14 +52,14 @@ export async function check(options = {}) {
       return lineCompare;
     return (l.location?.column ?? 0) - (r.location?.column ?? 0);
   });
-  console.log();
-  if (problems.length === 0) {
-    process.exit(0);
-  }
-  console.log(plural(problems.length, "Problem:", "Problems:"));
-  problems.forEach((problem) => {
-    console.log(renderProblem(problem));
-  });
+  const groupedProblems = problems.reduce((acc, problem) => {
+    const locationHash = `${problem.file}:${problem.location?.line}:${problem.location?.column}`;
+    const list = acc.get(locationHash) ?? [];
+    list.push(problem);
+    acc.set(locationHash, list);
+    return acc;
+  }, /* @__PURE__ */ new Map());
+  console.log([...groupedProblems.values()].map(renderProblemGroup).join("\n"));
   const files = Object.entries(
     problems.reduce((acc, problem) => {
       const file = "." + sep + relative(process.cwd(), problem.file);
@@ -91,12 +97,17 @@ async function findInstalledTools(root) {
 function plural(count, singular, plural2) {
   return `${count} ${count === 1 ? singular : plural2} `;
 }
-export function renderProblem(problem) {
-  const icon = problem.kind === "warning" ? bold(yellow("\u26A0")) : bold(red("\u2717"));
+export function renderProblemGroup(problems) {
+  const renderedProblems = problems.map(renderProblem);
+  const problem = problems[0];
   const path = relative(process.cwd(), problem.file);
   const location = problem.location ? `${path}:${problem.location.line}:${problem.location.column}` : path;
-  const source = problem.rule ? dim(` (${problem.rule})`) : "";
   const link = dim(`\u2192 .${sep}${location}`);
-  return `${icon} ${problem.message}${source}
+  return `${renderedProblems.join("\n")}
   ${link}`;
 }
+export function renderProblem(problem) {
+  const icon = problem.kind === "warning" ? bold(yellow("\u26A0")) : bold(red("\u2717"));
+  const source = problem.rule ? dim(` (${problem.rule})`) : "";
+  return `${icon} ${problem.message}${source}`;
+}

package/dist/tools/eslint.mjs

@@ -18,19 +18,19 @@ export const eslint = {
 export const parseOuptut = ({ stdout, stderr }) => {
   return `${stdout}
 ${stderr}`.split(/\r?\n/).reduce((acc, line) => {
-    const match = /^(.*?): line ([0-9]+), col ([0-9]+), (\S+) - (.*?) \((\S*?)\)$/.exec(
+    const groups = /^(?<file>.*?): line (?<line>[0-9]+), col (?<column>[0-9]+), (?<kind>\S+) - (?<message>.*?) \((?<rule>\S*?)\)$/.exec(
       line
-    );
-    if (match) {
+    )?.groups;
+    if (groups) {
       acc.push({
-        file: match[1],
-        kind: match[4] === "Warning" ? "warning" : "error",
-        message: match[5],
+        file: groups.file,
+        kind: groups.kind === "Warning" ? "warning" : "error",
+        message: groups.message,
         location: {
-          line: parseInt(match[2], 10),
-          column: parseInt(match[3], 10)
+          line: parseInt(groups.line, 10),
+          column: parseInt(groups.column, 10)
         },
-        rule: match[6]
+        rule: groups.rule
       });
     }
     return acc;

package/dist/tools/prettier.mjs

@@ -11,17 +11,17 @@ export const prettier = {
 export const parseOuptut = ({ stdout, stderr }) => {
   if (stderr.trim()) {
     return stderr.split(/\r?\n/).reduce((acc, line) => {
-      const match = /^\[(.+?)\]\s?(.+?):\s?(.*?)\s?\(([0-9]+):([0-9])\)$/.exec(
+      const groups = /^\[(?<kind>.+?)\]\s?(?<file>.+?):\s?(?<message>.*?)\s?\((?<line>[0-9]+):(?<column>[0-9])\)$/.exec(
         line
-      );
-      if (match) {
+      )?.groups;
+      if (groups) {
         acc.push({
-          file: match[2],
-          kind: match[1] === "error" ? "error" : "warning",
-          message: match[3],
+          file: groups.file,
+          kind: groups.kind === "error" ? "error" : "warning",
+          message: groups.message,
           location: {
-            line: parseInt(match[4], 10),
-            column: parseInt(match[5], 10)
+            line: parseInt(groups.line, 10),
+            column: parseInt(groups.column, 10)
           }
         });
       }

package/dist/tools/publint.mjs

@@ -13,12 +13,12 @@ export const parseOuptut = ({ stdout }) => {
       kind = "error";
       return acc;
     }
-    const match = /^[0-9]+\.\s?(.*)$/.exec(line);
-    if (match == null)
+    const groups = /^[0-9]+\.\s?(?<message>.*)$/.exec(line)?.groups;
+    if (groups == null)
       return acc;
     acc.push({
       kind,
-      message: match[1],
+      message: groups.message,
       file: "package.json"
     });
     return acc;

package/dist/tools/typescript.mjs

@@ -8,19 +8,19 @@ export const typescript = {
 };
 export const parseOuptut = ({ stdout }) => {
   return stdout.split(/\r?\n/).reduce((acc, line) => {
-    const match = /^(\S+?)\(([0-9]+),([0-9]+)\): \w+? (TS[0-9]+): (.*)$/.exec(
+    const groups = /^(?<file>\S+?)\((?<line>[0-9]+),(?<column>[0-9]+)\): \w+? (?<rule>TS[0-9]+): (?<message>.*)$/.exec(
       line
-    );
-    if (match) {
+    )?.groups;
+    if (groups) {
       acc.push({
-        file: match[1],
+        file: groups.file,
         kind: "error",
-        message: match[5],
+        message: groups.message,
         location: {
-          line: parseInt(match[2], 10),
-          column: parseInt(match[3], 10)
+          line: parseInt(groups.line, 10),
+          column: parseInt(groups.column, 10)
         },
-        rule: match[4]
+        rule: groups.rule
       });
     }
     return acc;

package/dist/utils.mjs

@@ -17,7 +17,7 @@ export async function isBinInstalled(bin, root) {
 }
 function exec(cmd, args, opts) {
   return new Promise((resolve2, reject) => {
-    const child = spawn(cmd, args, opts);
+    const child = spawn(cmd, args, { ...opts, shell: true });
     let stderr = "";
     let stdout = "";
     child.stdout.on("data", (data) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aklinker1/check",
-  "version": "1.0.5",
+  "version": "1.1.1",
   "repository": {
     "type": "git",
     "url": "https://github.com/aklinker1/check"