@aklinker1/check 1.0.1 → 1.0.3

package/dist/tools/prettier.mjs

@@ -8,8 +8,27 @@ export const prettier = {
   check: (root) => execAndParse(root, bin, checkArgs, parseOuptut),
   fix: (root) => execAndParse(root, bin, fixArgs, parseOuptut)
 };
-export const parseOuptut = ({ stdout }) => {
-  return stdout.trim().split(/\r?\n/).filter((line) => !!line).map(
+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(
+        line
+      );
+      if (match) {
+        acc.push({
+          file: match[2],
+          kind: match[1] === "error" ? "error" : "warning",
+          message: match[3],
+          location: {
+            line: parseInt(match[4], 10),
+            column: parseInt(match[5], 10)
+          }
+        });
+      }
+      return acc;
+    }, []);
+  }
+  return stdout.trim().split(/\r?\n/).map((line) => line.trim()).filter((line) => !!line && !line.includes(" ")).map(
     (line) => ({
       file: line.trim(),
       kind: "warning",

package/dist/tools/prettier.test.mjs

@@ -26,4 +26,53 @@ describe("Prettier", () => {
     const code = 1;
     expect(parseOuptut({ code, stdout, stderr })).toEqual([]);
   });
+  it("should return an error when a syntax error is reported", async () => {
+    const stderr = `[error] src/components/CommitDiff.ts: SyntaxError: Declaration or statement expected. (15:1)
+[error]   13 | });
+[error]   14 |
+[error] > 15 | }
+[error]      | ^
+[error]   16 |
+[error] src/components/CompareDiff.ts: Some other error message. (14:1)
+[error]   12 | });
+[error]   13 |
+[error] > 14 | }
+[error]      | ^
+[error]   15 |
+`;
+    const stdout = `.github/assets/privacy-policy.md 18ms
+  .github/workflows/submit.yml 20ms
+  .github/workflows/validate.yml 4ms
+  .prettierrc.yml 0ms`;
+    const code = 1;
+    expect(parseOuptut({ code, stdout, stderr })).toEqual([
+      {
+        file: "src/components/CommitDiff.ts",
+        message: "SyntaxError: Declaration or statement expected.",
+        location: {
+          line: 15,
+          column: 1
+        },
+        kind: "error"
+      },
+      {
+        file: "src/components/CompareDiff.ts",
+        message: "Some other error message.",
+        location: {
+          line: 14,
+          column: 1
+        },
+        kind: "error"
+      }
+    ]);
+  });
+  it("should not report warnings for fix output", () => {
+    const stderr = "";
+    const stdout = `.github/assets/privacy-policy.md 18ms
+.github/workflows/submit.yml 20ms
+.github/workflows/validate.yml 4ms
+.prettierrc.yml 0ms`;
+    const code = 0;
+    expect(parseOuptut({ code, stdout, stderr })).toEqual([]);
+  });
 });

package/dist/utils.mjs

@@ -38,6 +38,8 @@ export async function execAndParse(root, bin, args, parser) {
   const res = await exec(resolveRoot(root, bin), args, { cwd: root });
   if (res.exitCode == null)
     throw Error("Exit code was null");
+  if (isDebug())
+    console.debug({ bin, args, root, ...res });
   return parser({
     code: res.exitCode,
     stderr: res.stderr,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aklinker1/check",
-  "version": "1.0.1",
+  "version": "1.0.3",
   "type": "module",
   "exports": {
     ".": {
@@ -39,6 +39,11 @@
     ],
     "declaration": true
   },
+  "changelog": {
+    "excludeAuthors": [
+      "aaronklinker1@gmail.com"
+    ]
+  },
   "scripts": {
     "build": "bunx --bun unbuild",
     "check": "bun src/cli.ts",