feat(plugin-eslint): log initializer steps (rules discovery, group creation, nx helpers)

Author: matejchalk

packages/plugin-eslint/package.json

@@ -38,6 +38,7 @@
   },
   "type": "module",
   "dependencies": {
+    "ansis": "^3.3.2",
     "glob": "^11.0.0",
     "@code-pushup/utils": "0.94.0",
     "@code-pushup/models": "0.94.0",

packages/plugin-eslint/src/lib/constants.ts

@@ -1 +1,2 @@
 export const ESLINT_PLUGIN_SLUG = 'eslint';
+export const ESLINT_PLUGIN_TITLE = 'ESLint';

packages/plugin-eslint/src/lib/eslint-plugin.ts

@@ -6,7 +6,7 @@ import {
   eslintPluginConfigSchema,
   eslintPluginOptionsSchema,
 } from './config.js';
-import { ESLINT_PLUGIN_SLUG } from './constants.js';
+import { ESLINT_PLUGIN_SLUG, ESLINT_PLUGIN_TITLE } from './constants.js';
 import { listAuditsAndGroups } from './meta/index.js';
 import { createRunnerFunction } from './runner/index.js';
 
@@ -51,7 +51,7 @@ export async function eslintPlugin(
 
   return {
     slug: ESLINT_PLUGIN_SLUG,
-    title: 'ESLint',
+    title: ESLINT_PLUGIN_TITLE,
     icon: 'eslint',
     description: 'Official Code PushUp ESLint plugin',
     docsUrl: 'https://www.npmjs.com/package/@code-pushup/eslint-plugin',
@@ -61,7 +61,7 @@ export async function eslintPlugin(
     audits,
     groups,
 
-    runner: await createRunnerFunction({
+    runner: createRunnerFunction({
       audits,
       targets,
       ...(artifacts ? { artifacts } : {}),

packages/plugin-eslint/src/lib/meta/format.ts

@@ -0,0 +1,4 @@
+import { pluginMetaLogFormatter } from '@code-pushup/utils';
+import { ESLINT_PLUGIN_TITLE } from '../constants.js';
+
+export const formatMetaLog = pluginMetaLogFormatter(ESLINT_PLUGIN_TITLE);

packages/plugin-eslint/src/lib/meta/index.ts

@@ -1,5 +1,7 @@
 import type { Audit, Group } from '@code-pushup/models';
+import { formatAsciiTable, logger, pluralizeToken } from '@code-pushup/utils';
 import type { CustomGroup, ESLintTarget } from '../config.js';
+import { formatMetaLog } from './format.js';
 import {
   groupsFromCustomConfig,
   groupsFromRuleCategories,
@@ -10,24 +12,50 @@ import { ruleToAudit } from './transform.js';
 
 export { ruleIdToSlug } from './hash.js';
 export { detectConfigVersion, type ConfigFormat } from './versions/index.js';
+export { formatMetaLog };
 
 export async function listAuditsAndGroups(
   targets: ESLintTarget[],
   customGroups?: CustomGroup[] | undefined,
 ): Promise<{ audits: Audit[]; groups: Group[] }> {
   const rules = await listRules(targets);
+  const audits = rules.map(ruleToAudit);
+
+  logger.info(
+    formatMetaLog(
+      `Found ${pluralizeToken('rule', rules.length)} in total for ${pluralizeToken('target', targets.length)}, mapped to audits`,
+    ),
+  );
 
+  const resolvedTypeGroups = groupsFromRuleTypes(rules);
+  const resolvedCategoryGroups = groupsFromRuleCategories(rules);
   const resolvedCustomGroups = customGroups
     ? groupsFromCustomConfig(rules, customGroups)
     : [];
-
-  const audits = rules.map(ruleToAudit);
-
   const groups = [
-    ...groupsFromRuleTypes(rules),
-    ...groupsFromRuleCategories(rules),
+    ...resolvedTypeGroups,
+    ...resolvedCategoryGroups,
     ...resolvedCustomGroups,
   ];
 
+  logger.info(
+    formatMetaLog(
+      `Created ${pluralizeToken('group', groups.length)} (${resolvedTypeGroups.length} from meta.type, ${resolvedCategoryGroups.length} from meta.docs.category, ${resolvedCustomGroups.length} from custom groups)`,
+    ),
+  );
+  logger.debug(
+    formatMetaLog(
+      formatAsciiTable(
+        {
+          rows: groups.map(group => [
+            `• ${group.title}`,
+            pluralizeToken('audit', group.refs.length),
+          ]),
+        },
+        { borderless: true },
+      ),
+    ),
+  );
+
   return { audits, groups };
 }

packages/plugin-eslint/src/lib/meta/rules.ts

@@ -1,4 +1,7 @@
+import ansis from 'ansis';
+import { logger, pluralize, pluralizeToken, toArray } from '@code-pushup/utils';
 import type { ESLintTarget } from '../config.js';
+import { formatMetaLog } from './format.js';
 import { jsonHash } from './hash.js';
 import type { RuleData } from './parse.js';
 import { detectConfigVersion, selectRulesLoader } from './versions/index.js';
@@ -8,10 +11,16 @@ type RulesMap = Record<string, Record<string, RuleData>>;
 export async function listRules(targets: ESLintTarget[]): Promise<RuleData[]> {
   const version = await detectConfigVersion();
   const loadRulesMap = selectRulesLoader(version);
+  logger.debug(formatMetaLog(`Detected ${version} config format`));
 
   const rulesMap = await targets.reduce(async (acc, target) => {
     const map = await acc;
     const rules = await loadRulesMap(target);
+    logger.debug(
+      formatMetaLog(
+        `Found ${pluralizeToken('rule', rules.length)} for ${formatTarget(target)}`,
+      ),
+    );
     return rules.reduce(mergeRuleIntoMap, map);
   }, Promise.resolve<RulesMap>({}));
 
@@ -28,6 +37,16 @@ function mergeRuleIntoMap(map: RulesMap, rule: RuleData): RulesMap {
   };
 }
 
+function formatTarget(target: ESLintTarget): string {
+  const patterns = toArray(target.patterns);
+  return [
+    `${pluralize('pattern', patterns.length)} ${ansis.bold(patterns.join(' '))}`,
+    target.eslintrc && `using config ${ansis.bold(target.eslintrc)}`,
+  ]
+    .filter(Boolean)
+    .join(' ');
+}
+
 export function expandWildcardRules(
   wildcard: string,
   rules: string[],

packages/plugin-eslint/src/lib/nx/find-all-projects.ts

@@ -1,5 +1,6 @@
-import { logger, stringifyError } from '@code-pushup/utils';
+import { logger, pluralizeToken, stringifyError } from '@code-pushup/utils';
 import type { ESLintTarget } from '../config.js';
+import { formatMetaLog } from '../meta/format.js';
 import { filterProjectGraph } from './filter-project-graph.js';
 import { nxProjectsToConfig } from './projects-to-config.js';
 
@@ -14,7 +15,7 @@ async function resolveCachedProjectGraph() {
   try {
     return readCachedProjectGraph();
   } catch (error) {
-    logger.info(
+    logger.warn(
       `Could not read cached project graph, falling back to async creation.\n${stringifyError(error)}`,
     );
     return await createProjectGraphAsync({ exitOnError: false });
@@ -55,7 +56,21 @@ export async function eslintConfigFromAllNxProjects(
     projectGraph,
     options.exclude,
   );
-  return nxProjectsToConfig(filteredProjectGraph);
+  const targets = await nxProjectsToConfig(filteredProjectGraph);
+
+  logger.info(
+    formatMetaLog(
+      [
+        `Inferred ${pluralizeToken('lint target', targets.length)} for all Nx projects`,
+        options.exclude?.length &&
+          `(excluding ${pluralizeToken('project', options.exclude.length)})`,
+      ]
+        .filter(Boolean)
+        .join(' '),
+    ),
+  );
+
+  return targets;
 }
 
 /**

packages/plugin-eslint/src/lib/nx/find-project-with-deps.ts

@@ -1,4 +1,7 @@
+import { logger } from '@nx/devkit';
+import { pluralizeToken } from '@code-pushup/utils';
 import type { ESLintTarget } from '../config.js';
+import { formatMetaLog } from '../meta/format.js';
 import { nxProjectsToConfig } from './projects-to-config.js';
 import { findAllDependencies } from './traverse-graph.js';
 
@@ -35,10 +38,18 @@ export async function eslintConfigFromNxProjectAndDeps(
 
   const dependencies = findAllDependencies(projectName, projectGraph);
 
-  return nxProjectsToConfig(
+  const targets = await nxProjectsToConfig(
     projectGraph,
     project =>
       !!project.name &&
       (project.name === projectName || dependencies.has(project.name)),
   );
+
+  logger.info(
+    formatMetaLog(
+      `Inferred ${pluralizeToken('lint target', targets.length)} for Nx project "${projectName}" and its dependencies`,
+    ),
+  );
+
+  return targets;
 }

packages/plugin-eslint/src/lib/nx/find-project-without-deps.ts

@@ -1,4 +1,6 @@
+import { logger } from '@code-pushup/utils';
 import type { ESLintTarget } from '../config.js';
+import { formatMetaLog } from '../meta/format.js';
 import { nxProjectsToConfig } from './projects-to-config.js';
 
 /**
@@ -41,5 +43,9 @@ export async function eslintConfigFromNxProject(
     throw new Error(`Couldn't find Nx project named "${projectName}"`);
   }
 
+  logger.info(
+    formatMetaLog(`Inferred lint target for Nx project "${projectName}"`),
+  );
+
   return project;
 }