refactor: cleanup

Author: John Doe

CONTRIBUTING.md

@@ -83,21 +83,6 @@ Therefore, PRs are merged via one of two strategies:
 - rebase - branch cannot contain merge commits ([rebase instead of merge](https://www.atlassian.com/git/tutorials/merging-vs-rebasing)),
 - squash - single commit whose message is the PR title (should be in conventional commit format).
 
-## Releases
-
-We use nx release command to create releases for GitHub as well as publish to npm.
-
-**Preconditions:**
-
-- `npm login` - Only users with write access to [code-pushup](https://www.npmjs.com/org/code-pushup) can publish
-- (optional) `GITHUB_TOKEN=ghp_...` in `.env` - [Personal access token](https://github.com/settings/personal-access-tokens/new) to create a GitHub release.
-
-**Steps:**
-
-- `git checkout main`, `git pull`
-- (recommended optional) `npx nx release --dryRun`
-- `npx nx release` and confirm publish prompt
-
 ## Project tags
 
 [Nx tags](https://nx.dev/core-features/enforce-module-boundaries) are used to enforce module boundaries in the project graph when linting.

code-pushup.config.ts

@@ -21,9 +21,6 @@ const config: CoreConfig = {
       server: 'https://api.staging.code-pushup.dev/graphql',
       apiKey: process.env['CP_API_KEY'],
     },
-    persist: {
-      outputDir: '.code-pushup',
-    },
   }),
   plugins: [],
 };

packages/nx-plugin/src/executors/internal/config.ts

@@ -27,18 +27,12 @@ export function persistConfig(
 ): Partial<PersistConfig> {
   const {
     format,
-    outputDir = '{workspaceRoot}/.code-pushup/{projectName}',
+    outputDir = '{projectRoot}/.code-pushup',
     filename,
   } = options;
 
-  const { workspaceRoot, projectConfig } = context;
-  const projectName = projectConfig?.name ?? '';
-  const resolvedOutputDir = outputDir
-    .replace('{workspaceRoot}', workspaceRoot)
-    .replace('{projectName}', projectName);
-
   return {
-    outputDir: resolvedOutputDir,
+    outputDir,
     ...(format ? { format } : {}),
     ...(filename ? { filename } : {}),
   };

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

@@ -104,7 +104,6 @@ describe('executeProcess', () => {
     expect(logger.command).toHaveBeenCalledWith(
       'echo hello',
       expect.any(Function),
-      {},
     );
   });
 

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

@@ -158,7 +158,7 @@ export function executeProcess(cfg: ProcessConfig): Promise<ProcessResult> {
   const { onStdout, onStderr, onError, onComplete } = observer ?? {};
 
   const bin = [command, ...(args ?? [])].join(' ');
-  logger.debug(JSON.stringify(options), { force: true });
+
   const worker = () =>
     new Promise<ProcessResult>((resolve, reject) => {
       const spawnedProcess = spawn(command, args ?? [], {

testing/test-utils/src/lib/utils/string.ts

@@ -1,6 +1,5 @@
-import ansis from 'ansis';
-
 // removes all color codes from the output for snapshot readability
 export function removeColorCodes(stdout: string) {
-  return ansis.strip(stdout);
+  // eslint-disable-next-line no-control-regex
+  return stdout.replace(/\u001B\[\d+m/g, '');
 }

tools/src/debug/debug.plugin.ts

@@ -1,6 +1,6 @@
 import type { CreateNodes, CreateNodesContext } from '@nx/devkit';
 import { dirname } from 'node:path';
-import { objectToCliArgs } from '@code-pushup/utils';
+import { objectToCliArgs } from '../../../packages/nx-plugin/src/executors/internal/cli.js';
 import { TOOLS_TSCONFIG_PATH } from '../constants.js';
 import { KILL_PROCESS_BIN, LIST_PROCESS_BIN } from './constants.js';