refactor(nx-plugin): remove options merge

e2e/nx-plugin-e2e/tests/executor-cli.e2e.test.ts

@@ -204,7 +204,9 @@ describe('executor command', () => {
     expect(cleanStdout).toContain('Code PushUp CLI');
 
     await expect(
-      readJsonFile(path.join(cwd, '.reports', 'terminal-report.json')),
+      readJsonFile(
+        path.join(cwd, '.code-pushup', project, 'terminal-report.json'),
+      ),
     ).resolves.not.toThrow();
   });
 

packages/nx-plugin/src/executors/cli/executor.ts

@@ -6,7 +6,7 @@ import {
 } from '../internal/cli.js';
 import { normalizeContext } from '../internal/context.js';
 import type { AutorunCommandExecutorOptions } from './schema.js';
-import { mergeExecutorOptions, parseAutorunExecutorOptions } from './utils.js';
+import { parseAutorunExecutorOptions } from './utils.js';
 
 export type ExecutorOutput = {
   success: boolean;
@@ -19,15 +19,11 @@ export default async function runAutorunExecutor(
   context: ExecutorContext,
 ): Promise<ExecutorOutput> {
   const normalizedContext = normalizeContext(context);
-  const mergedOptions = mergeExecutorOptions(
-    context.target?.options,
-    terminalAndExecutorOptions,
-  );
   const cliArgumentObject = parseAutorunExecutorOptions(
-    mergedOptions,
+    terminalAndExecutorOptions,
     normalizedContext,
   );
-  const { dryRun, verbose, command, bin } = mergedOptions;
+  const { dryRun, verbose, command, bin } = terminalAndExecutorOptions;
   const commandString = createCliCommandString({
     command,
     args: cliArgumentObject,

packages/nx-plugin/src/executors/cli/utils.ts

@@ -34,7 +34,7 @@ export function parseAutorunExecutorOptions(
   options: Partial<AutorunCommandExecutorOptions>,
   normalizedContext: NormalizedExecutorContext,
 ): AutorunCommandExecutorOptions {
-  const { projectPrefix, persist, upload, command } = options;
+  const { projectPrefix, persist, upload, command, output } = options;
   const needsUploadParams =
     command === 'upload' || command === 'autorun' || command === undefined;
   const uploadCfg = uploadConfig(
@@ -46,48 +46,11 @@ export function parseAutorunExecutorOptions(
     ...parsePrintConfigExecutorOptions(options),
     ...parseAutorunExecutorOnlyOptions(options),
     ...globalConfig(options, normalizedContext),
+    ...(output ? { output } : {}),
     persist: persistConfig({ projectPrefix, ...persist }, normalizedContext),
     // @TODO This is a hack to avoid validation errors of upload config for commands that dont need it.
     // Fix: use utils and execute the core logic directly
     // Blocked by Nx plugins can't compile to es6
     ...(needsUploadParams && hasApiToken ? { upload: uploadCfg } : {}),
   };
 }
-
-/**
- * Deeply merges executor options.
- *
- * @param targetOptions - The original options from the target configuration.
- * @param cliOptions - The options from Nx, combining target options and CLI arguments.
- * @returns A new object with deeply merged properties.
- *
- * Nx performs a shallow merge by default, where command-line arguments can override entire objects
- * (e.g., `--persist.filename` replaces the entire `persist` object).
- * This function ensures that nested properties are deeply merged,
- * preserving the original target options where CLI arguments are not provided.
- */
-export function mergeExecutorOptions(
-  targetOptions: Partial<AutorunCommandExecutorOptions>,
-  cliOptions: Partial<AutorunCommandExecutorOptions>,
-): AutorunCommandExecutorOptions {
-  return {
-    ...targetOptions,
-    ...cliOptions,
-    ...(targetOptions?.persist || cliOptions?.persist
-      ? {
-          persist: {
-            ...targetOptions?.persist,
-            ...cliOptions?.persist,
-          },
-        }
-      : {}),
-    ...(targetOptions?.upload || cliOptions?.upload
-      ? {
-          upload: {
-            ...targetOptions?.upload,
-            ...cliOptions?.upload,
-          },
-        }
-      : {}),
-  };
-}

packages/nx-plugin/src/executors/cli/utils.unit.test.ts

@@ -2,20 +2,19 @@
 import { osAgnosticPath } from '@code-pushup/test-utils';
 import type { Command } from '../internal/types.js';
 import {
-  mergeExecutorOptions,
   parseAutorunExecutorOnlyOptions,
   parseAutorunExecutorOptions,
   parsePrintConfigExecutorOptions,
 } from './utils.js';
 
 describe('parsePrintConfigExecutorOptions', () => {
   it('should provide NO default output path', () => {
     expect(parsePrintConfigExecutorOptions({})).toStrictEqual(
       expect.not.objectContaining({ output: expect.anything() }),
     );
   });
 
   it('should process given output path', () => {
     expect(
       parsePrintConfigExecutorOptions({ output: 'code-pushup.config.json' }),
     ).toStrictEqual(
@@ -24,38 +23,38 @@
   });
 });
 
 describe('parseAutorunExecutorOnlyOptions', () => {
   it('should provide NO default projectPrefix', () => {
     expect(parseAutorunExecutorOnlyOptions({})).toStrictEqual(
       expect.not.objectContaining({ projectPrefix: expect.anything() }),
     );
   });
 
   it('should process given projectPrefix', () => {
     expect(
       parseAutorunExecutorOnlyOptions({ projectPrefix: 'cli' }),
     ).toStrictEqual(expect.objectContaining({ projectPrefix: 'cli' }));
   });
 
   it('should provide NO default dryRun', () => {
     expect(parseAutorunExecutorOnlyOptions({})).toStrictEqual(
       expect.not.objectContaining({ dryRun: expect.anything() }),
     );
   });
 
   it('should process given dryRun', () => {
     expect(parseAutorunExecutorOnlyOptions({ dryRun: false })).toStrictEqual(
       expect.objectContaining({ dryRun: false }),
     );
   });
 
   it('should provide default onlyPlugins', () => {
     expect(parseAutorunExecutorOnlyOptions({})).toStrictEqual(
       expect.not.objectContaining({ onlyPlugins: ['json'] }),
     );
   });
 
   it('should process given onlyPlugins', () => {
     expect(
       parseAutorunExecutorOnlyOptions({ onlyPlugins: ['md', 'json'] }),
     ).toStrictEqual(expect.objectContaining({ onlyPlugins: ['md', 'json'] }));
@@ -170,26 +169,3 @@
     },
   );
 });
-
-describe('mergeExecutorOptions', () => {
-  it('should deeply merge target and CLI options', () => {
-    const targetOptions = {
-      persist: {
-        outputDir: '.reports',
-        filename: 'report',
-      },
-    };
-    const cliOptions = {
-      persist: {
-        filename: 'report-file',
-      },
-    };
-    const expected = {
-      persist: {
-        outputDir: '.reports',
-        filename: 'report-file',
-      },
-    };
-    expect(mergeExecutorOptions(targetOptions, cliOptions)).toEqual(expected);
-  });
-});