fix: resolve ESLint errors and warnings

Author: hanna-skryl

packages/ci/vitest.int.config.ts

@@ -1,16 +1,3 @@
 import { createIntTestConfig } from '../../testing/test-setup-config/src/index.js';
 
-const baseConfig = createIntTestConfig('ci');
-
-const config = {
-  ...baseConfig,
-  test: {
-    ...baseConfig.test,
-    setupFiles: [
-      ...(baseConfig.test!.setupFiles || []),
-      '../../testing/test-setup/src/lib/logger.mock.ts',
-    ],
-  },
-};
-
-export default config;
+export default createIntTestConfig('ci');

packages/ci/vitest.unit.config.ts

@@ -1,16 +1,3 @@
 import { createUnitTestConfig } from '../../testing/test-setup-config/src/index.js';
 
-const baseConfig = createUnitTestConfig('ci');
-
-const config = {
-  ...baseConfig,
-  test: {
-    ...baseConfig.test,
-    setupFiles: [
-      ...(baseConfig.test!.setupFiles || []),
-      '../../testing/test-setup/src/lib/logger.mock.ts',
-    ],
-  },
-};
-
-export default config;
+export default createUnitTestConfig('ci');

packages/utils/src/lib/file-system.ts

@@ -66,10 +66,8 @@ export function logMultipleFileResults(
     const formattedSize = size ? ` (${ansis.gray(formatBytes(size))})` : '';
     return `- ${ansis.bold(fileName)}${formattedSize}`;
   };
-  const failedTransform = (result: PromiseRejectedResult) => {
-    const reason = String(result.reason);
-    return `- ${ansis.bold(reason)}`;
-  };
+  const failedTransform = (result: PromiseRejectedResult) =>
+    `- ${ansis.bold(String(result.reason))}`;
 
   logMultipleResults<FileResult>(
     fileResults,

packages/utils/src/lib/logger.ts

@@ -1,4 +1,4 @@
-/* eslint-disable max-lines, no-console */
+/* eslint-disable max-lines, no-console, @typescript-eslint/class-methods-use-this */
 import ansis, { type AnsiColors } from 'ansis';
 import os from 'node:os';
 import path from 'node:path';
@@ -13,8 +13,12 @@ type GroupColor = Extract<AnsiColors, 'cyan' | 'magenta'>;
 type CiPlatform = 'GitHub Actions' | 'GitLab CI/CD';
 
 const HEX_RADIX = 16;
-const SIGINT_EXIT_CODE_UNIX = 130;
-const SIGINT_EXIT_CODE_WINDOWS = 2;
+
+const SIGINT_CODE = 2;
+// https://www.gnu.org/software/bash/manual/html_node/Exit-Status.html#:~:text=When%20a%20command%20terminates%20on%20a%20fatal%20signal%20whose%20number%20is%20N%2C%20Bash%20uses%20the%20value%20128%2BN%20as%20the%20exit%20status.
+const SIGNALS_CODE_OFFSET_UNIX = 128;
+const SIGINT_EXIT_CODE_UNIX = SIGNALS_CODE_OFFSET_UNIX + SIGINT_CODE;
+const SIGINT_EXIT_CODE_WINDOWS = SIGINT_CODE;
 
 /**
  * Rich logging implementation for Code PushUp CLI, plugins, etc.
@@ -196,10 +200,7 @@ export class Logger {
     await this.#spinner(worker, {
       pending: title,
       success: value => value,
-      failure: error => {
-        const errorMessage = String(error);
-        return `${title} → ${ansis.red(errorMessage)}`;
-      },
+      failure: error => `${title} → ${ansis.red(String(error))}`,
     });
   }
 
@@ -476,15 +477,13 @@ export class Logger {
     );
   }
 
-  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   #colorize(text: string, color: AnsiColors | undefined): string {
     if (!color) {
       return text;
     }
     return ansis[color](text);
   }
 
-  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   #formatDurationSuffix({
     start,
     end,

testing/test-setup-config/src/lib/vitest-setup-files.ts

@@ -1,3 +1,4 @@
+/* eslint-disable sonarjs/no-duplicate-string */
 import type { TestKind } from './vitest-config-factory.js';
 
 /**
@@ -19,7 +20,6 @@ const CUSTOM_MATCHERS = [
  * which is why they use `../../` to navigate to the workspace root first.
  */
 const UNIT_TEST_SETUP_FILES = [
-  // eslint-disable-next-line sonarjs/no-duplicate-string
   '../../testing/test-setup/src/lib/reset.mocks.ts',
   '../../testing/test-setup/src/lib/fs.mock.ts',
   '../../testing/test-setup/src/lib/logger.mock.ts',