Print documentation links of rule

package.json

@@ -43,7 +43,7 @@
     "typescript": "^4.1.2"
   },
   "dependencies": {
-    "cli-table": "^0.3.6",
+    "cli-table": "msabramo/cli-table#GH-147",
     "enquirer": "^2.3.6",
     "eslint-formatter-codeframe": "^7.32.1",
     "node-pager": "^0.3.6",

src/eslint.ts

@@ -7,6 +7,7 @@ import { DisableTarget, Option } from './rules/add-disable-comment';
 import { ApplySuggestionOption } from './rules/apply-suggestion';
 import { DisplayMode } from './types';
 import { groupBy } from './util/array';
+import { scanUsedPluginsFromResults } from './util/eslint';
 import { notEmpty } from './util/filter';
 
 function filterResultsByRuleId(results: ESLint.LintResult[], ruleIds: string[]): ESLint.LintResult[] {
@@ -72,7 +73,6 @@ function createApplySuggestionESLint(
   });
   return eslint;
 }
-
 type CachedESLintOptions = {
   rulePaths?: string[];
   extensions?: string[];
@@ -103,7 +103,18 @@ export class CachedESLint {
   }
 
   printResults(results: ESLint.LintResult[]): void {
-    const resultText = format(results);
+    // get used plugins from `results`
+    const plugins = scanUsedPluginsFromResults(results);
+
+    // get `rulesMeta` from `results`
+    const eslint = new ESLint({
+      ...this.defaultOptions,
+      overrideConfig: { plugins },
+    });
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    const rulesMeta: ESLint.LintResultData['rulesMeta'] = (eslint as any).getRulesMetaForResults?.(results);
+
+    const resultText = format(results, { rulesMeta });
     console.log(resultText);
   }
 

src/util/eslint.ts

@@ -0,0 +1,20 @@
+import { ESLint } from 'eslint';
+import { unique } from './array';
+import { notEmpty } from './filter';
+
+/** `results` 内で使われているプラグインの名前のリストを洗い出して返す */
+export function scanUsedPluginsFromResults(results: ESLint.LintResult[]): string[] {
+  const plugins = results
+    .flatMap((result) => result.messages) // messages: Linter.LintMessage[]
+    .map((message) => message.ruleId) // ruleIds: (string | undefined)[]
+    .filter(notEmpty) // ruleIds: string[]
+    .map((ruleId) => {
+      const parts = ruleId.split('/');
+      if (parts.length === 1) return undefined; // ex: 'rule-a'
+      if (parts.length === 2) return parts[0]; // ex: 'plugin/rule-a'
+      if (parts.length === 3) return `${parts[0]}/${parts[1]}`; // ex: '@scoped/plugin/rule-a'
+      return undefined; // invalid ruleId
+    }) // plugins: string[]
+    .filter(notEmpty);
+  return unique(plugins);
+}

test/formatter/format-by-rules.test.ts

@@ -1,3 +1,4 @@
+/* eslint-disable no-irregular-whitespace */
 import { ESLint } from 'eslint';
 import stripAnsi from 'strip-ansi';
 import { formatByRules } from '../../src/formatter/format-by-rules';
@@ -43,6 +44,45 @@ describe('formatByRules', () => {
 ├────────┼────────────────────────────────────┼──────────────────────────────────────┤
 │ rule-b │ 1 (0/0)                            │ 0 (0/0)                              │
 └────────┴────────────────────────────────────┴──────────────────────────────────────┘"
+`);
+  });
+  test('prints link', () => {
+    const results: ESLint.LintResult[] = [
+      fakeLintResult({
+        messages: [
+          fakeLintMessage({ ruleId: 'rule-a', severity: 2 }), // link is printed
+          fakeLintMessage({ ruleId: 'rule-b', severity: 2 }), // no link is printed
+          fakeLintMessage({ ruleId: 'plugin/rule-c', severity: 2 }), // link is printed
+        ],
+      }),
+    ];
+    const formattedText = formatByRules(results, {
+      rulesMeta: {
+        'rule-a': { docs: { url: 'https://example.com/rule-a' } },
+        'plugin/rule-c': { docs: { url: 'https://example.com/plugin/rule-c' } },
+      },
+    });
+    expect(stripAnsi(formattedText)).toMatchInlineSnapshot(`
+"┌─────────────────────────────────────────────────────┬────────────────────────────────────┬──────────────────────────────────────┐
+│ Rule                                                │ Error (fixable/suggest-applicable) │ Warning (fixable/suggest-applicable) │
+├─────────────────────────────────────────────────────┼────────────────────────────────────┼──────────────────────────────────────┤
+│ rule-a (​https://example.com/rule-a​)               │ 1 (0/0)                            │ 0 (0/0)                              │
+├─────────────────────────────────────────────────────┼────────────────────────────────────┼──────────────────────────────────────┤
+│ rule-b                                              │ 1 (0/0)                            │ 0 (0/0)                              │
+├─────────────────────────────────────────────────────┼────────────────────────────────────┼──────────────────────────────────────┤
+│ plugin/rule-c (​https://example.com/plugin/rule-c​) │ 1 (0/0)                            │ 0 (0/0)                              │
+└─────────────────────────────────────────────────────┴────────────────────────────────────┴──────────────────────────────────────┘"
+`);
+    expect(formattedText).toMatchInlineSnapshot(`
+"┌─────────────────────────────────────────────────────┬────────────────────────────────────┬──────────────────────────────────────┐
+│ Rule                                                │ Error (fixable/suggest-applicable) │ Warning (fixable/suggest-applicable) │
+├─────────────────────────────────────────────────────┼────────────────────────────────────┼──────────────────────────────────────┤
+│ rule-a (​https://example.com/rule-a​)               │ 1 (0/0)                            │ 0 (0/0)                              │
+├─────────────────────────────────────────────────────┼────────────────────────────────────┼──────────────────────────────────────┤
+│ rule-b                                              │ 1 (0/0)                            │ 0 (0/0)                              │
+├─────────────────────────────────────────────────────┼────────────────────────────────────┼──────────────────────────────────────┤
+│ plugin/rule-c (​https://example.com/plugin/rule-c​) │ 1 (0/0)                            │ 0 (0/0)                              │
+└─────────────────────────────────────────────────────┴────────────────────────────────────┴──────────────────────────────────────┘"
 `);
   });
 });

test/util/eslint.test.ts

@@ -0,0 +1,17 @@
+import { ESLint } from 'eslint';
+import { scanUsedPluginsFromResults } from '../../src/util/eslint';
+import { fakeLintMessage, fakeLintResult } from '../test-util/eslint';
+
+test('scanUsedPluginsFromResults', () => {
+  const results: ESLint.LintResult[] = [
+    fakeLintResult({
+      messages: [
+        fakeLintMessage({ ruleId: 'rule', severity: 2 }),
+        fakeLintMessage({ ruleId: 'plugin/rule', severity: 2 }),
+        fakeLintMessage({ ruleId: '@scoped/plugin/rule', severity: 2 }),
+        fakeLintMessage({ ruleId: 'invalid/@scoped/plugin/rule', severity: 2 }),
+      ],
+    }),
+  ];
+  expect(scanUsedPluginsFromResults(results)).toStrictEqual(['plugin', '@scoped/plugin']);
+});

yarn.lock

@@ -839,6 +839,11 @@ ansi-regex@^5.0.0:
   resolved "https://registry.yarnpkg.com/ansi-regex/-/ansi-regex-5.0.0.tgz#388539f55179bf39339c81af30a654d69f87cb75"
   integrity sha512-bY6fj56OUQ0hU1KjFNDQuJFezqKdrAyFdIevADiqrWHwSlbmBNMHp5ak2f40Pm8JTFyM2mqxkG6ngkHO11f/lg==
 
+ansi-regex@^5.0.1:
+  version "5.0.1"
+  resolved "https://registry.yarnpkg.com/ansi-regex/-/ansi-regex-5.0.1.tgz#082cb2c89c9fe8659a311a53bd6a4dc5301db304"
+  integrity sha512-quJQXlTSUGL2LH9SUXo8VwsY4soanhgo6LNSm84E1LBcE8s3O0wpdiRzyR9z/ZZJMlMWv37qOOb9pdJlMUEKFQ==
+
 ansi-styles@^3.2.1:
   version "3.2.1"
   resolved "https://registry.yarnpkg.com/ansi-styles/-/ansi-styles-3.2.1.tgz#41fbb20243e50b12be0f04b8dedbf07520ce841d"
@@ -1096,12 +1101,12 @@ cli-spinners@^2.4.0:
   resolved "https://registry.yarnpkg.com/cli-spinners/-/cli-spinners-2.5.0.tgz#12763e47251bf951cb75c201dfa58ff1bcb2d047"
   integrity sha512-PC+AmIuK04E6aeSs/pUccSujsTzBhu4HzC2dL+CfJB/Jcc2qTRbEwZQDfIUpt2Xl8BodYBEq8w4fc0kU2I9DjQ==
 
-cli-table@^0.3.6:
-  version "0.3.6"
-  resolved "https://registry.yarnpkg.com/cli-table/-/cli-table-0.3.6.tgz#e9d6aa859c7fe636981fd3787378c2a20bce92fc"
-  integrity sha512-ZkNZbnZjKERTY5NwC2SeMeLeifSPq/pubeRoTpdr3WchLlnZg6hEgvHkK5zL7KNFdd9PmHN8lxrENUwI3cE8vQ==
+cli-table@msabramo/cli-table#GH-147:
+  version "0.4.0-dev"
+  resolved "https://codeload.github.com/msabramo/cli-table/tar.gz/631c19de8e10338ea407889eeb710f3cb422a3b8"
   dependencies:
     colors "1.0.3"
+    strip-ansi "^6.0.1"
 
 cliui@^7.0.2:
   version "7.0.4"
@@ -3485,6 +3490,13 @@ strip-ansi@^6.0.0:
   dependencies:
     ansi-regex "^5.0.0"
 
+strip-ansi@^6.0.1:
+  version "6.0.1"
+  resolved "https://registry.yarnpkg.com/strip-ansi/-/strip-ansi-6.0.1.tgz#9e26c63d30f53443e9489495b2105d37b67a85d9"
+  integrity sha512-Y38VPSHcqkFrCpFnQ9vuSXmquuv5oXOKpGeT6aGrr3o3Gc9AlVa6JBfUSOCnbxGGZF+/0ooI7KrPuUSztUdU5A==
+  dependencies:
+    ansi-regex "^5.0.1"
+
 strip-bom@^3.0.0:
   version "3.0.0"
   resolved "https://registry.yarnpkg.com/strip-bom/-/strip-bom-3.0.0.tgz#2334c18e9c759f7bdd56fdef7e9ae3d588e68ed3"