feat(ci): use nx cache friendly output path for print-config command

Author: matejchalk

packages/ci/src/lib/cli/commands/print-config.ts

@@ -2,7 +2,6 @@ import { rm } from 'node:fs/promises';
 import path from 'node:path';
 import {
   executeProcess,
-  generateRandomId,
   isVerbose,
   readJsonFile,
   stringifyError,
@@ -13,11 +12,19 @@ export async function runPrintConfig({
   bin,
   config,
   directory,
+  project,
   observer,
 }: CommandContext): Promise<unknown> {
-  // random file name so command can be run in parallel
-  const outputFile = `code-pushup.${generateRandomId()}.config.json`;
-  const outputPath = path.resolve(directory, outputFile);
+  // unique file name per project so command can be run in parallel
+  const outputFile = ['code-pushup', 'config', project, 'json']
+    .filter(Boolean)
+    .join('.');
+  const outputPath =
+    project && directory === process.cwd()
+      ? // cache-friendly path for Nx projects (assuming {workspaceRoot}/.code-pushup/{projectName})
+        path.join(process.cwd(), '.code-pushup', project, outputFile)
+      : // absolute path
+        path.resolve(directory, '.code-pushup', outputFile);
 
   await executeProcess({
     command: bin,

packages/ci/src/lib/cli/context.ts

@@ -4,6 +4,7 @@ import type { Settings } from '../models.js';
 import type { ProjectConfig } from '../monorepo/index.js';
 
 export type CommandContext = Pick<Settings, 'bin' | 'config' | 'directory'> & {
+  project?: string;
   observer?: ProcessObserver;
 };
 
@@ -15,6 +16,7 @@ export function createCommandContext(
     bin: project?.bin ?? bin,
     directory: project?.directory ?? directory,
     config,
+    ...(project?.name && { project: project.name }),
     observer: createExecutionObserver({ silent }),
   };
 }

packages/ci/src/lib/cli/context.unit.test.ts

@@ -63,6 +63,7 @@ describe('createCommandContext', () => {
       bin: 'yarn code-pushup',
       directory: '/test/ui',
       config: null,
+      project: 'ui',
       observer: expectedObserver,
     });
   });

packages/ci/src/lib/run.integration.test.ts

@@ -132,7 +132,9 @@ describe('runInCI', () => {
           ?.find(arg => arg.startsWith('--output='))
           ?.split('=')[1];
         if (outputFile) {
-          await writeFile(path.resolve(cwd as string, outputFile), content);
+          const outputPath = path.resolve(cwd as string, outputFile);
+          await mkdir(path.dirname(outputPath), { recursive: true });
+          await writeFile(outputPath, content);
         } else {
           stdout = content;
         }