fix(plugin-eslint): artifacts options

Author: BioPhoton

packages/plugin-eslint/src/lib/__snapshots__/eslint-plugin.int.test.ts.snap

@@ -346,16 +346,7 @@ exports[`eslintPlugin > should initialize ESLint plugin for React application 1`
   ],
   "icon": "eslint",
   "packageName": "@code-pushup/eslint-plugin",
-  "runner": {
-    "args": [
-      ""<dirname>/bin.js"",
-      "--runnerConfigPath="node_modules/.code-pushup/eslint/<timestamp>/plugin-config.json"",
-      "--runnerOutputPath="node_modules/.code-pushup/eslint/<timestamp>/runner-output.json"",
-    ],
-    "command": "node",
-    "configFile": "node_modules/.code-pushup/eslint/<timestamp>/plugin-config.json",
-    "outputFile": "node_modules/.code-pushup/eslint/<timestamp>/runner-output.json",
-  },
+  "runner": [Function],
   "slug": "eslint",
   "title": "ESLint",
   "version": Any<String>,

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

@@ -3,8 +3,7 @@ import path from 'node:path';
 import process from 'node:process';
 import { fileURLToPath } from 'node:url';
 import type { MockInstance } from 'vitest';
-import type { Audit, PluginConfig, RunnerConfig } from '@code-pushup/models';
-import { toUnixPath } from '@code-pushup/utils';
+import type { Audit, PluginConfig } from '@code-pushup/models';
 import { eslintPlugin } from './eslint-plugin.js';
 
 describe('eslintPlugin', () => {
@@ -15,26 +14,7 @@ describe('eslintPlugin', () => {
   let cwdSpy: MockInstance<[], string>;
   let platformSpy: MockInstance<[], NodeJS.Platform>;
 
-  const replaceAbsolutePath = (plugin: PluginConfig): PluginConfig => ({
-    ...plugin,
-    runner: {
-      ...(plugin.runner as RunnerConfig),
-      args: (plugin.runner as RunnerConfig).args?.map(arg =>
-        toUnixPath(arg.replace(path.dirname(thisDir), '<dirname>')).replace(
-          /\/eslint\/\d+\//,
-          '/eslint/<timestamp>/',
-        ),
-      ),
-      ...((plugin.runner as RunnerConfig).configFile && {
-        configFile: toUnixPath(
-          (plugin.runner as RunnerConfig).configFile!,
-        ).replace(/\/eslint\/\d+\//, '/eslint/<timestamp>/'),
-      }),
-      outputFile: toUnixPath(
-        (plugin.runner as RunnerConfig).outputFile,
-      ).replace(/\/eslint\/\d+\//, '/eslint/<timestamp>/'),
-    },
-  });
+  const replaceAbsolutePath = (plugin: PluginConfig): PluginConfig => plugin;
 
   beforeAll(() => {
     cwdSpy = vi.spyOn(process, 'cwd');

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

@@ -3,8 +3,12 @@ import path from 'node:path';
 import type {
   Audit,
   AuditOutput,
+  AuditOutputs,
+  PersistConfig,
+  PluginArtifactOptions,
   RunnerConfig,
   RunnerFilesPaths,
+  RunnerFunction,
 } from '@code-pushup/models';
 import {
   asyncSequential,
@@ -18,6 +22,7 @@ import {
 import type { ESLintPluginRunnerConfig, ESLintTarget } from '../config.js';
 import { lint } from './lint.js';
 import { lintResultsToAudits, mergeLinterOutputs } from './transform.js';
+import { loadArtifacts } from './utils';
 
 export async function executeRunner({
   runnerConfigPath,
@@ -78,3 +83,42 @@ export async function createRunnerConfig(
     outputFile: runnerOutputPath,
   };
 }
+
+export async function createRunnerFunction(options: {
+  audits: Audit[];
+  targets: ESLintTarget[];
+  artifacts?: PluginArtifactOptions;
+}): Promise<RunnerFunction> {
+  const { audits, targets, artifacts } = options;
+  const config: ESLintPluginRunnerConfig = {
+    targets,
+    slugs: audits.map(audit => audit.slug),
+  };
+
+  return async ({ outputDir }: PersistConfig): Promise<AuditOutputs> => {
+    ui().logger.log(`ESLint plugin executing ${targets.length} lint targets`);
+
+    const linterOutputs = artifacts
+      ? await loadArtifacts(artifacts)
+      : await asyncSequential(
+          targets.map(target => ({
+            ...target,
+            outputDir,
+          })),
+          lint,
+        );
+    const lintResults = mergeLinterOutputs(linterOutputs);
+    const failedAudits = lintResultsToAudits(lintResults);
+
+    return config.slugs.map(
+      (slug): AuditOutput =>
+        failedAudits.find(audit => audit.slug === slug) ?? {
+          slug,
+          score: 1,
+          value: 0,
+          displayValue: 'passed',
+          details: { issues: [] },
+        },
+    );
+  };
+}