refactor: add various logs WIP

Author: vmasek

packages/core/src/lib/collect-and-persist.ts

@@ -28,6 +28,7 @@ export async function collectAndPersistReports(
 
   const report = await collect(options);
   const sortedScoredReport = sortReport(scoreReport(report));
+
   const persistResults = await persistReport(
     report,
     sortedScoredReport,

packages/core/src/lib/implementation/collect.ts

@@ -1,6 +1,6 @@
 import { createRequire } from 'node:module';
 import type { CoreConfig, Report } from '@code-pushup/models';
-import { calcDuration, getLatestCommit } from '@code-pushup/utils';
+import { calcDuration, getLatestCommit, ui } from '@code-pushup/utils';
 import type { GlobalOptions } from '../types.js';
 import { executePlugins } from './execute-plugin.js';
 
@@ -12,6 +12,7 @@ export type CollectOptions = Pick<CoreConfig, 'plugins' | 'categories'> &
  * @param options
  */
 export async function collect(options: CollectOptions): Promise<Report> {
+  ui().logger.log('Running collect');
   const { plugins, categories } = options;
   const date = new Date().toISOString();
   const start = performance.now();

packages/core/src/lib/implementation/runner.ts

@@ -4,7 +4,12 @@ import type {
   RunnerConfig,
   RunnerFunction,
 } from '@code-pushup/models';
-import { calcDuration, executeProcess, readJsonFile } from '@code-pushup/utils';
+import {
+  calcDuration,
+  executeProcess,
+  readJsonFile,
+  removeDirectoryIfExists,
+} from '@code-pushup/utils';
 
 export type RunnerResult = {
   date: string;
@@ -26,7 +31,8 @@ export async function executeRunnerConfig(
   });
 
   // read process output from file system and parse it
-  const outputs = await readJsonFile(path.join(process.cwd(), outputFile));
+  const outputs = await readJsonFile(outputFile);
+  await removeDirectoryIfExists(path.dirname(outputFile));
 
   // transform unknownAuditOutputs to auditOutputs
   const audits = outputTransform ? await outputTransform(outputs) : outputs;

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

@@ -12,6 +12,7 @@ import {
   filePathToCliArg,
   objectToCliArgs,
   readJsonFile,
+  ui,
 } from '@code-pushup/utils';
 import type { ESLintPluginRunnerConfig, ESLintTarget } from '../config.js';
 import { lint } from './lint.js';
@@ -25,6 +26,8 @@ export async function executeRunner({
   const { slugs, targets } =
     await readJsonFile<ESLintPluginRunnerConfig>(runnerConfigPath);
 
+  ui().logger.log(`ESLint plugin executing ${targets.length} lint targets`);
+
   const linterOutputs = await targets.reduce(
     async (acc, target) => [...(await acc), await lint(target)],
     Promise.resolve<LinterOutput[]>([]),

packages/utils/src/lib/execute-process.ts

@@ -6,6 +6,7 @@ import {
   spawn,
 } from 'node:child_process';
 import type { Readable, Writable } from 'node:stream';
+import { ui } from './logging.js';
 import { calcDuration } from './reports/utils.js';
 
 /**
@@ -148,6 +149,10 @@ export function executeProcess(cfg: ProcessConfig): Promise<ProcessResult> {
   const date = new Date().toISOString();
   const start = performance.now();
 
+  ui().logger.log(
+    `Executing command:\n${[command, ...(args || [])].join(' ')}`,
+  );
+
   return new Promise((resolve, reject) => {
     // shell:true tells Windows to use shell command for spawning a child process
     const spawnedProcess = spawn(command, args ?? [], {
@@ -179,6 +184,8 @@ export function executeProcess(cfg: ProcessConfig): Promise<ProcessResult> {
       const timings = { date, duration: calcDuration(start) };
       if (code === 0 || ignoreExitCode) {
         onComplete?.();
+        // todo: maybe if I update spinner instead of logging it will appear with a command it was supposed to be grouped with
+        ui().logger.log(`Command execution took ${calcDuration(start)}ms`);
         resolve({ code, stdout, stderr, ...timings });
       } else {
         const errorMsg = new ProcessError({ code, stdout, stderr, ...timings });