Merge branch 'main' into refactor/nx-plugin/fix-verbose-handling

# Conflicts:
#	e2e/nx-plugin-e2e/tests/plugin-create-nodes.e2e.test.ts
#	packages/nx-plugin/src/executors/cli/executor.ts
#	packages/nx-plugin/src/executors/cli/executor.unit.test.ts

Author: John Doe

CHANGELOG.md

@@ -1,3 +1,13 @@
+## 0.92.0 (2025-12-02)
+
+### 🚀 Features
+
+- **ci:** style top-level logs ([5da74215](https://github.com/code-pushup/cli/commit/5da74215))
+
+### ❤️ Thank You
+
+- Matěj Chalk
+
 ## 0.91.0 (2025-12-01)
 
 ### 🚀 Features

e2e/nx-plugin-e2e/tests/plugin-create-nodes.e2e.test.ts

@@ -1,6 +1,5 @@
 import type { Tree } from '@nx/devkit';
 import path from 'node:path';
-import * as process from 'node:process';
 import { readProjectConfiguration } from 'nx/src/generators/utils/project-configuration';
 import { afterEach, expect } from 'vitest';
 import { generateCodePushupConfig } from '@code-pushup/nx-plugin';
@@ -104,29 +103,6 @@ describe('nx-plugin', () => {
     });
   });
 
-  it('should consider plugin option bin in configuration target', async () => {
-    const cwd = path.join(testFileDir, 'configuration-option-bin');
-    registerPluginInWorkspace(tree, {
-      plugin: '@code-pushup/nx-plugin',
-      options: {
-        bin: 'XYZ',
-      },
-    });
-    await materializeTree(tree, cwd);
-
-    const { code, projectJson } = await nxShowProjectJson(cwd, project);
-
-    expect(code).toBe(0);
-
-    expect(projectJson.targets).toStrictEqual({
-      'code-pushup--configuration': expect.objectContaining({
-        options: {
-          command: `nx g XYZ:configuration --project="${project}"`,
-        },
-      }),
-    });
-  });
-
   it('should NOT add config targets dynamically if the project is configured', async () => {
     const cwd = path.join(testFileDir, 'configuration-already-configured');
     registerPluginInWorkspace(tree, '@code-pushup/nx-plugin');

packages/ci/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@code-pushup/ci",
-  "version": "0.91.0",
+  "version": "0.92.0",
   "description": "CI automation logic for Code PushUp (provider-agnostic)",
   "license": "MIT",
   "homepage": "https://github.com/code-pushup/cli/tree/main/packages/ci#readme",
@@ -26,9 +26,10 @@
   },
   "type": "module",
   "dependencies": {
-    "@code-pushup/models": "0.91.0",
+    "@code-pushup/models": "0.92.0",
     "@code-pushup/portal-client": "^0.16.0",
-    "@code-pushup/utils": "0.91.0",
+    "@code-pushup/utils": "0.92.0",
+    "ansis": "^3.3.2",
     "glob": "^11.0.1",
     "simple-git": "^3.20.0",
     "yaml": "^2.5.1",

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

@@ -8,15 +8,56 @@ import {
 } from '@code-pushup/utils';
 import type { CommandContext } from './context.js';
 
+/**
+ * Executes Code PushUp CLI command and logs output in a way that's more readable in CI.
+ * @param args Arguments for Code PushUp CLI
+ * @param context Command context
+ * @param options Optional information on whether all persist formats are set (if known)
+ */
 export async function executeCliCommand(
   args: string[],
   context: CommandContext,
   options?: { hasFormats: boolean },
 ): Promise<void> {
+  const { logOutputChunk, logOutputEnd, logSilencedOutput } =
+    createLogCallbacks(context);
+
+  const observer: ProcessObserver = {
+    onStdout: logOutputChunk,
+    onStderr: logOutputChunk,
+    onComplete: logOutputEnd,
+    onError: logOutputEnd,
+  };
+
+  const config: ProcessConfig = {
+    command: context.bin,
+    args: combineArgs(args, context, options),
+    cwd: context.directory,
+    observer,
+    silent: true,
+  };
+  const bin = serializeCommandWithArgs(config);
+
+  try {
+    await logger.command(bin, async () => {
+      try {
+        await executeProcess(config);
+      } catch (error) {
+        // ensure output of failed process is always logged for debugging
+        logSilencedOutput();
+        throw error;
+      }
+    });
+  } finally {
+    logger.newline();
+  }
+}
+
+function createLogCallbacks(context: Pick<CommandContext, 'silent'>) {
   // eslint-disable-next-line functional/no-let
   let output = '';
 
-  const logRaw = (message: string) => {
+  const logOutputChunk = (message: string) => {
     if (!context.silent) {
       if (!output) {
         logger.newline();
@@ -26,43 +67,23 @@ export async function executeCliCommand(
     output += message;
   };
 
-  const logEnd = () => {
+  const logOutputEnd = () => {
     if (!context.silent && output) {
       logger.newline();
     }
   };
 
-  const observer: ProcessObserver = {
-    onStdout: logRaw,
-    onStderr: logRaw,
-    onComplete: logEnd,
-    onError: logEnd,
-  };
-
-  const config: ProcessConfig = {
-    command: context.bin,
-    args: combineArgs(args, context, options),
-    cwd: context.directory,
-    observer,
-    silent: true,
-  };
-  const bin = serializeCommandWithArgs(config);
-
-  await logger.command(bin, async () => {
-    try {
-      await executeProcess(config);
-    } catch (error) {
-      // ensure output of failed process is always logged for debugging
-      if (context.silent) {
+  const logSilencedOutput = () => {
+    if (context.silent) {
+      logger.newline();
+      logger.info(output, { noIndent: true });
+      if (!output.endsWith('\n')) {
         logger.newline();
-        logger.info(output, { noIndent: true });
-        if (!output.endsWith('\n')) {
-          logger.newline();
-        }
       }
-      throw error;
     }
-  });
+  };
+
+  return { logOutputChunk, logOutputEnd, logSilencedOutput };
 }
 
 function combineArgs(

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

@@ -130,6 +130,7 @@ describe('executeCliCommand', () => {
       `
 - $ npx code-pushup
 ✔ $ npx code-pushup (42 ms)
+
 `.trimStart(),
     );
   });
@@ -152,6 +153,7 @@ WARN: API key is missing, skipping upload
 Collected report files in .code-pushup directory
 
 ✔ $ npx code-pushup (42 ms)
+
 `.trimStart(),
     );
   });
@@ -197,6 +199,7 @@ Collected report
 Uploaded report to portal
 
 ✔ $ npx code-pushup (42 ms)
+
 `.trimStart(),
     );
   });
@@ -222,6 +225,7 @@ Code PushUp CLI v0.42.0
 ERROR: Config file not found
 
 ✖ $ npx code-pushup
+
 `.trimStart(),
     );
   });
@@ -236,6 +240,7 @@ ERROR: Config file not found
       `
 - $ npx code-pushup
 ✔ $ npx code-pushup (42 ms)
+
 `.trimStart(),
     );
   });
@@ -261,6 +266,7 @@ Code PushUp CLI v0.42.0
 ERROR: Config file not found
 
 ✖ $ npx code-pushup
+
 `.trimStart(),
     );
   });

packages/ci/src/lib/comment.ts

@@ -1,5 +1,5 @@
 import { readFile } from 'node:fs/promises';
-import { logger } from '@code-pushup/utils';
+import { logDebug, logInfo, logWarning } from './log.js';
 import type { ProviderAPIClient, Settings } from './models.js';
 
 export async function commentOnPR(
@@ -17,32 +17,32 @@ export async function commentOnPR(
   );
 
   const comments = await api.listComments();
-  logger.debug(`Fetched ${comments.length} comments for pull request`);
+  logDebug(`Fetched ${comments.length} comments for pull request`);
 
   const prevComment = comments.find(comment =>
     comment.body.includes(identifier),
   );
-  logger.debug(
+  logDebug(
     prevComment
       ? `Found previous comment ${prevComment.id} from Code PushUp`
       : 'Previous Code PushUp comment not found',
   );
 
   if (prevComment) {
     const updatedComment = await api.updateComment(prevComment.id, body);
-    logger.info(`Updated body of comment ${updatedComment.url}`);
+    logInfo(`Updated body of comment ${updatedComment.url}`);
     return updatedComment.id;
   }
 
   const createdComment = await api.createComment(body);
-  logger.info(`Created new comment ${createdComment.url}`);
+  logInfo(`Created new comment ${createdComment.url}`);
   return createdComment.id;
 }
 
 function truncateBody(body: string, max: number): string {
   const truncateWarning = '...*[Comment body truncated]*';
   if (body.length > max) {
-    logger.warn(`Comment body is too long. Truncating to ${max} characters.`);
+    logWarning(`Comment body is too long. Truncating to ${max} characters.`);
     return body.slice(0, max - truncateWarning.length) + truncateWarning;
   }
   return body;

packages/ci/src/lib/comment.unit.test.ts

@@ -1,3 +1,4 @@
+import ansis from 'ansis';
 import { vol } from 'memfs';
 import { writeFile } from 'node:fs/promises';
 import path from 'node:path';
@@ -112,7 +113,7 @@ describe('commentOnPR', () => {
       expect.stringContaining('...*[Comment body truncated]*'),
     );
     expect(logger.warn).toHaveBeenCalledWith(
-      'Comment body is too long. Truncating to 1000000 characters.',
+      `${ansis.bold.blue('<✓>')} Comment body is too long. Truncating to 1000000 characters.\n`,
     );
   });
 });

packages/ci/src/lib/log.ts

@@ -0,0 +1,59 @@
+import ansis from 'ansis';
+import {
+  CODE_PUSHUP_UNICODE_LOGO,
+  logger,
+  transformLines,
+} from '@code-pushup/utils';
+
+const LOG_PREFIX = ansis.bold.blue(CODE_PUSHUP_UNICODE_LOGO);
+
+/**
+ * Logs error message with top-level CI log styles (lines prefixed with logo, ends in empty line).
+ * @param message Log message
+ */
+export function logError(message: string): void {
+  log('error', message);
+}
+
+/**
+ * Logs warning message with top-level CI log styles (lines prefixed with logo, ends in empty line).
+ * @param message Log message
+ */
+export function logWarning(message: string): void {
+  log('warn', message);
+}
+
+/**
+ * Logs info message with top-level CI log styles (lines prefixed with logo, ends in empty line).
+ * @param message Log message
+ */
+export function logInfo(message: string): void {
+  log('info', message);
+}
+
+/**
+ * Logs debug message with top-level CI log styles (lines prefixed with logo, ends in empty line).
+ * @param message Log message
+ */
+export function logDebug(message: string): void {
+  log('debug', message);
+}
+
+/**
+ * Prefixes CI logs with logo and ensures each CI log is followed by an empty line.
+ * This is to make top-level CI logs more visually distinct from printed process logs.
+ * @param level Log level
+ * @param message Log message
+ */
+export function log(
+  level: 'error' | 'warn' | 'info' | 'debug',
+  message: string,
+): void {
+  const prefixedLines = transformLines(
+    message.trim(),
+    line => `${LOG_PREFIX} ${line}`,
+  );
+  const styledMessage = `${prefixedLines}\n`;
+
+  logger[level](styledMessage);
+}

packages/ci/src/lib/log.unit.test.ts

@@ -0,0 +1,31 @@
+import ansis from 'ansis';
+import { logger } from '@code-pushup/utils';
+import { log } from './log.js';
+
+describe('log', () => {
+  it('should add logo prefix and ending line-break to message', () => {
+    log('info', 'Running Code PushUp in standalone mode');
+    expect(logger.info).toHaveBeenCalledWith(
+      `${ansis.bold.blue('<✓>')} Running Code PushUp in standalone mode\n`,
+    );
+  });
+
+  it('should add logo prefix to each line', () => {
+    log('debug', 'Found 3 Nx projects:\n- api\n- backoffice\n- frontoffice');
+    expect(logger.debug).toHaveBeenCalledWith(
+      `
+${ansis.bold.blue('<✓>')} Found 3 Nx projects:
+${ansis.bold.blue('<✓>')} - api
+${ansis.bold.blue('<✓>')} - backoffice
+${ansis.bold.blue('<✓>')} - frontoffice
+`.trimStart(),
+    );
+  });
+
+  it('should not add final line-break if already present', () => {
+    log('warn', 'Comment body is too long, truncating to 1000 characters\n');
+    expect(logger.warn).toHaveBeenCalledWith(
+      `${ansis.bold.blue('<✓>')} Comment body is too long, truncating to 1000 characters\n`,
+    );
+  });
+});