refactor: remove all any types and fix TypeScript strict mode compliance

- Replace ALL 'any' types with proper TypeScript types across codebase
- Add proper interfaces for complex data structures
- Use 'unknown' type with type guards for truly unknown data
- Fix all ESLint errors and warnings
- Improve type safety in test fixtures and utility files
- Add proper type assertions where needed
- Fix import statements for vitest in test files
- Resolve all TypeScript strict mode compilation errors

All files now pass ESLint with zero errors and TypeScript strict mode

Author: eannnnnn

src/modules/build/services/collection/collection.service.claude-code.spec.ts

@@ -8,14 +8,14 @@ import * as path from 'node:path';
 
 import { Test, TestingModule } from '@nestjs/testing';
 
-import { describe, it, expect, beforeEach, vi } from 'vitest';
+import { beforeEach, describe, expect, it, vi } from 'vitest';
 
 import { CollectionService } from './collection.service';
 
 import type {
   ClaudeCodeLocalSettingsData,
   ClaudeCodeGlobalSettingsData,
-  McpServerConfig
+  McpServerConfig,
 } from '../../interfaces/claude-code.interfaces';
 
 // Mock os module

src/modules/build/services/transformation/transformation.service.ts

@@ -389,7 +389,7 @@ export class TransformationService {
       DataProcessingErrorHandler.logErrorResult(errorResult);
 
       // Return partial data if available, otherwise empty object
-      return errorResult.partialData || {};
+      return (errorResult.partialData as ParsedData) || ({} as ParsedData);
     }
   }
 

src/modules/build/test-fixtures/advanced-error-scenarios.ts

@@ -445,7 +445,7 @@ export class ErrorScenarioFactory {
     return new AdvancedMockFileSystem(
       {
         files: Object.fromEntries(Array.from({ length: 100 }, (_, i) => [`/file${i}.txt`, `content ${i}`])
-          .map(( [path, content]) => [path, content])),
+          .map(([path, content]) => [path, content])),
         directories: ['/'],
       },
       { concurrentOperationLimit: 3 }
@@ -569,7 +569,7 @@ export class ErrorScenarioTestUtilities {
    * Test that a function properly handles and retries on EAGAIN errors
    */
   static async testRetryOnEAGAIN(
-    operation: () => Promise<any>,
+    operation: () => Promise<unknown>,
     maxRetries = 3
   ): Promise<{ success: boolean; attempts: number; error?: Error }> {
     let attempts = 0;
@@ -580,14 +580,15 @@ export class ErrorScenarioTestUtilities {
         // eslint-disable-next-line no-await-in-loop
         await operation();
         return { success: true, attempts };
-      } catch (error: any) {
-        if (error.code === 'EAGAIN' && i < maxRetries) {
+      } catch (error: unknown) {
+        const err = error as { code?: string };
+        if (err.code === 'EAGAIN' && i < maxRetries) {
           // Wait before retry
           // eslint-disable-next-line no-await-in-loop
           await new Promise(resolve => setTimeout(resolve, Math.pow(2, i) * 100));
           continue;
         }
-        return { success: false, attempts, error };
+        return { success: false, attempts, error: error as Error };
       }
     }
 
@@ -598,16 +599,17 @@ export class ErrorScenarioTestUtilities {
    * Test that a function properly handles disk space errors
    */
   static async testDiskSpaceHandling(
-    writeOperation: () => Promise<any>
+    writeOperation: () => Promise<unknown>
   ): Promise<{ handledCorrectly: boolean; errorType?: string }> {
     try {
       await writeOperation();
       return { handledCorrectly: false }; // Should have failed
-    } catch (error: any) {
-      const isDiskSpaceError = error.code === 'ENOSPC' || error.code === 'EFBIG';
+    } catch (error: unknown) {
+      const err = error as { code?: string };
+      const isDiskSpaceError = err.code === 'ENOSPC' || err.code === 'EFBIG';
       return { 
         handledCorrectly: isDiskSpaceError, 
-        errorType: error.code 
+        errorType: err.code 
       };
     }
   }
@@ -616,7 +618,7 @@ export class ErrorScenarioTestUtilities {
    * Test concurrent operation limits
    */
   static async testConcurrencyLimits(
-    operations: (() => Promise<any>)[],
+    operations: (() => Promise<unknown>)[],
     _expectedFailures = 0  
   ): Promise<{ successes: number; failures: number; errors: Error[] }> {
     const results = await Promise.allSettled(
@@ -627,7 +629,7 @@ export class ErrorScenarioTestUtilities {
     const failures = results.filter(r => r.status === 'rejected').length;
     const errors = results
       .filter((r): r is PromiseRejectedResult => r.status === 'rejected')
-      .map(r => r.reason);
+      .map(r => r.reason as Error);
 
     return { successes, failures, errors };
   }
@@ -636,7 +638,7 @@ export class ErrorScenarioTestUtilities {
    * Performance benchmark for file operations
    */
   static async benchmarkOperation(
-    operation: () => Promise<any>,
+    operation: () => Promise<unknown>,
     iterations = 10
   ): Promise<{ 
     averageTime: number; 

src/modules/build/test-fixtures/mock-file-system.ts

@@ -3,6 +3,25 @@
  * Mock file system utilities for testing with advanced error scenarios
  */
 
+/**
+ * File system error with code property
+ */
+interface FileSystemError extends Error {
+  code: string;
+}
+
+/**
+ * File system stats interface
+ */
+export interface MockFileStats {
+  isDirectory(): boolean;
+  isFile(): boolean;
+  size: number;
+}
+
+/**
+ * Mock file system configuration
+ */
 export interface MockFileSystemConfig {
   files: Record<string, string>;
   directories: string[];
@@ -51,15 +70,19 @@ export class MockFileSystem {
     // Check permissions
     const perms = this.permissions.get(filePath);
     if (perms && !perms.readable) {
-      const error = new Error(`EACCES: permission denied, open '${filePath}'`);
-      (error as any).code = 'EACCES';
+      const error: FileSystemError = Object.assign(
+        new Error(`EACCES: permission denied, open '${filePath}'`),
+        { code: 'EACCES' }
+      );
       throw error;
     }
 
     // Check if file exists
     if (!this.files.has(filePath)) {
-      const error = new Error(`ENOENT: no such file or directory, open '${filePath}'`);
-      (error as any).code = 'ENOENT';
+      const error: FileSystemError = Object.assign(
+        new Error(`ENOENT: no such file or directory, open '${filePath}'`),
+        { code: 'ENOENT' }
+      );
       throw error;
     }
 
@@ -75,8 +98,10 @@ export class MockFileSystem {
     // Check permissions
     const perms = this.permissions.get(filePath);
     if (perms && !perms.writable) {
-      const error = new Error(`EACCES: permission denied, open '${filePath}'`);
-      (error as any).code = 'EACCES';
+      const error: FileSystemError = Object.assign(
+        new Error(`EACCES: permission denied, open '${filePath}'`),
+        { code: 'EACCES' }
+      );
       throw error;
     }
 
@@ -91,14 +116,17 @@ export class MockFileSystem {
 
     // Check if directory exists
     if (!this.directories.has(directoryPath)) {
-      const error = new Error(`ENOENT: no such file or directory, scandir '${directoryPath}'`);
-      (error as any).code = 'ENOENT';
+      const error: FileSystemError = Object.assign(
+        new Error(`ENOENT: no such file or directory, scandir '${directoryPath}'`),
+        { code: 'ENOENT' }
+      );
       throw error;
     }
 
     // Return files in directory
     const filesInDirectory: string[] = [];
-    for (const filePath of this.files.keys()) {
+    const filePathsArray = Array.from(this.files.keys());
+    for (const filePath of filePathsArray) {
       if (filePath.startsWith(`${directoryPath  }/`)) {
         const relativePath = filePath.slice(Math.max(0, directoryPath.length + 1));
         if (!relativePath.includes('/')) {
@@ -119,8 +147,10 @@ export class MockFileSystem {
     // Check permissions
     const perms = this.permissions.get(directoryPath);
     if (perms && !perms.writable) {
-      const error = new Error(`EACCES: permission denied, mkdir '${directoryPath}'`);
-      (error as any).code = 'EACCES';
+      const error: FileSystemError = Object.assign(
+        new Error(`EACCES: permission denied, mkdir '${directoryPath}'`),
+        { code: 'EACCES' }
+      );
       throw error;
     }
 
@@ -139,7 +169,7 @@ export class MockFileSystem {
     }
   }
 
-  async stat(filePath: string): Promise<{ isDirectory(): boolean; isFile(): boolean; size: number }> {
+  async stat(filePath: string): Promise<MockFileStats> {
     // Check for errors first
     if (this.errors.has(filePath)) {
       throw this.errors.get(filePath);
@@ -162,8 +192,10 @@ export class MockFileSystem {
       };
     }
 
-    const error = new Error(`ENOENT: no such file or directory, stat '${filePath}'`);
-    (error as any).code = 'ENOENT';
+    const error: FileSystemError = Object.assign(
+      new Error(`ENOENT: no such file or directory, stat '${filePath}'`),
+      { code: 'ENOENT' }
+    );
     throw error;
   }