@aklinker1/check 2.0.0-alpha.2 → 2.1.0

package/README.md

@@ -13,7 +13,7 @@ pnpm check --fix
 To enable checks for any of the following modules, just install them:
 
 ```sh
-pnpm i -D typescript oxlint prettier publint eslint
+pnpm i -D typescript oxlint prettier publint eslint markdownlint-cli
 ```
 
 ## Contributing
@@ -30,10 +30,14 @@ bun run build
 # Run checks
 bun check --help
 bun check
+
+# Debug commands used
+DEBUG=1 bun check
 ```
 
 ### Adding Tools
 
-I've added everything I use, so if you want to add support for another tool, feel free.
-
-Just copy `src/tools/prettier.ts` and `src/tools/prettier.test.ts`, update the implementations (yes, tests are required), and add your new tool to `src/tools/index.ts`'s `ALL_TOOLS` export.
+1. Copy and rename `src/tools/prettier.ts` and `src/tools/prettier.test.ts` accordingly
+2. Implement and update tests for your new tool
+3. Add your tool to the `ALL_TOOLS` array in `src/tools/index.ts`
+4. Add the tool's NPM package to the first section of this README

package/dist/tools/index.mjs

@@ -1,6 +1,14 @@
 import { eslint } from "./eslint.mjs";
+import { markdownlint } from "./markdownlint.mjs";
 import { oxlint } from "./oxlint.mjs";
 import { prettier } from "./prettier.mjs";
 import { publint } from "./publint.mjs";
 import { typescript } from "./typescript.mjs";
-export const ALL_TOOLS = [publint, prettier, typescript, oxlint, eslint];
+export const ALL_TOOLS = [
+  eslint,
+  markdownlint,
+  oxlint,
+  prettier,
+  publint,
+  typescript
+];

package/dist/tools/markdownlint.d.ts

@@ -0,0 +1,3 @@
+import type { OutputParser, ToolDefinition } from "../types";
+export declare const markdownlint: ToolDefinition;
+export declare const parseOutput: OutputParser;

package/dist/tools/markdownlint.mjs

@@ -0,0 +1,34 @@
+import { execAndParse } from "../utils.mjs";
+export const markdownlint = ({ root }) => {
+  const bin = "markdownlint";
+  const checkArgs = [
+    ".",
+    "--json",
+    "--ignore='**/dist/**'",
+    "--ignore='**/node_modules/**'",
+    "--ignore='**/.output/**'",
+    "--ignore='**/coverage/**'"
+  ];
+  const fixArgs = [...checkArgs, "--fix"];
+  return {
+    name: "Markdownlint",
+    packageName: "markdownlint-cli",
+    check: () => execAndParse(bin, checkArgs, root, parseOutput),
+    fix: () => execAndParse(bin, fixArgs, root, parseOutput)
+  };
+};
+export const parseOutput = ({ stderr: _stderr }) => {
+  const stderr = _stderr.trim();
+  if (!stderr)
+    return [];
+  return JSON.parse(stderr).map((warning) => ({
+    location: {
+      line: warning.lineNumber,
+      column: warning.errorRange?.[0] ?? 0
+    },
+    message: warning.ruleDescription,
+    file: warning.fileName,
+    kind: "warning",
+    rule: warning.ruleNames[0]
+  }));
+};

package/dist/tools/markdownlint.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/tools/markdownlint.test.mjs

@@ -0,0 +1,72 @@
+import { describe, it, expect } from "bun:test";
+import { parseOutput } from "./markdownlint.mjs";
+describe("Markdownlint", () => {
+  it("should properly parse output", async () => {
+    const stdout = "";
+    const stderr = `
+[
+  {
+    "fileName": "docs/guide/resources/upgrading.md",
+    "lineNumber": 59,
+    "ruleNames": [
+      "MD031",
+      "blanks-around-fences"
+    ],
+    "ruleDescription": "Fenced code blocks should be surrounded by blank lines",
+    "ruleInformation": "https://github.com/DavidAnson/markdownlint/blob/v0.38.0/doc/md031.md",
+    "errorDetail": null,
+    "errorContext": "\`\`\`ts",
+    "errorRange": null,
+    "fixInfo": {
+      "lineNumber": 59,
+      "insertText": "\\n"
+    }
+  },
+  {
+    "fileName": "CODE_OF_CONDUCT.md",
+    "lineNumber": 63,
+    "ruleNames": [
+      "MD034",
+      "no-bare-urls"
+    ],
+    "ruleDescription": "Bare URL used",
+    "ruleInformation": "https://github.com/DavidAnson/markdownlint/blob/v0.38.0/doc/md034.md",
+    "errorDetail": null,
+    "errorContext": "example@gmail.com",
+    "errorRange": [
+      5,
+      23
+    ],
+    "fixInfo": {
+      "editColumn": 1,
+      "deleteCount": 23,
+      "insertText": "<example@gmail.com>"
+    }
+  }
+]
+`;
+    const code = 1;
+    expect(parseOutput({ code, stdout, stderr })).toEqual([
+      {
+        file: "docs/guide/resources/upgrading.md",
+        message: "Fenced code blocks should be surrounded by blank lines",
+        location: {
+          line: 59,
+          column: 0
+        },
+        rule: "MD031",
+        kind: "warning"
+      },
+      {
+        file: "CODE_OF_CONDUCT.md",
+        message: "Bare URL used",
+        location: {
+          line: 63,
+          column: 5
+        },
+        rule: "MD034",
+        kind: "warning"
+      }
+    ]);
+  });
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aklinker1/check",
-  "version": "2.0.0-alpha.2",
+  "version": "2.1.0",
   "packageManager": "bun@1.2.11",
   "repository": {
     "type": "git",
@@ -36,12 +36,6 @@
   "bin": {
     "check": "bin/check.mjs"
   },
-  "scripts": {
-    "build": "bunx --bun unbuild",
-    "check": "bun src/cli.ts",
-    "prepack": "bun run build",
-    "prepublish": "bun run build"
-  },
   "dependencies": {
     "@antfu/utils": "^0.7.7",
     "citty": "^0.1.6",
@@ -67,5 +61,10 @@
     "excludeAuthors": [
       "aaronklinker1@gmail.com"
     ]
+  },
+  "scripts": {
+    "build": "bunx --bun unbuild",
+    "check": "bun src/cli.ts",
+    "prepublish": "bun run build"
   }
-}
+}