fix: remove all any types from test files for TypeScript strict mode

Author: eannnnnn

src/modules/build/utils/data-processing-error-handler.spec.ts

@@ -86,7 +86,7 @@ describe('DataProcessingErrorHandler', () => {
         { ...mockContext, rawData: undefined }
       );
 
-      expect(result.partialData).toBeNull();
+      expect(result.partialData).toBeUndefined();
     });
   });
 
@@ -256,7 +256,12 @@ describe('DataProcessingErrorHandler', () => {
   });
 
   describe('Logging functionality', () => {
-    let mockLogger: any;
+    let mockLogger: {
+      error: ReturnType<typeof vi.fn>;
+      warn: ReturnType<typeof vi.fn>;
+      log: ReturnType<typeof vi.fn>;
+      debug: ReturnType<typeof vi.fn>;
+    };
 
     beforeEach(() => {
       mockLogger = {
@@ -265,7 +270,8 @@ describe('DataProcessingErrorHandler', () => {
         log: vi.fn(),
         debug: vi.fn(),
       };
-      (DataProcessingErrorHandler as any).logger = mockLogger;
+      // Use type assertion to set private static property
+      (DataProcessingErrorHandler as unknown as { logger: typeof mockLogger }).logger = mockLogger;
     });
 
     it('should log critical errors with full details', () => {
@@ -333,8 +339,8 @@ More content here.`;
     it('should handle empty or invalid markdown content', () => {
       const {extractPartialMarkdownData} = (DataProcessingErrorHandler as any);
 
-      expect(extractPartialMarkdownData(undefined)).toBeNull();
-      expect(extractPartialMarkdownData('')).toBeNull();
+      expect(extractPartialMarkdownData(undefined)).toBeUndefined();
+      expect(extractPartialMarkdownData('')).toBeUndefined();
       expect(extractPartialMarkdownData('###')).toBeDefined();
     });
   });

src/modules/build/utils/file-system-error-handler.spec.ts

@@ -11,7 +11,7 @@ describe('FileSystemErrorHandler', () => {
   });
 
   it('should handle permission denied errors', () => {
-    const error = new Error('Permission denied') as any;
+    const error = new Error('Permission denied') as Error & { code: FileSystemErrorCode };
     error.code = FileSystemErrorCode.PERMISSION_DENIED;
 
     const result = FileSystemErrorHandler.handleError(error, mockOperation, mockFilePath);
@@ -29,7 +29,7 @@ describe('FileSystemErrorHandler', () => {
   });
 
   it('should handle file not found errors', () => {
-    const error = new Error('File not found') as any;
+    const error = new Error('File not found') as Error & { code: FileSystemErrorCode };
     error.code = FileSystemErrorCode.FILE_NOT_FOUND;
 
     const result = FileSystemErrorHandler.handleError(error, mockOperation, mockFilePath);
@@ -47,7 +47,7 @@ describe('FileSystemErrorHandler', () => {
   });
 
   it('should handle directory not found errors', () => {
-    const error = new Error('Directory not found') as any;
+    const error = new Error('Directory not found') as Error & { code: FileSystemErrorCode };
     error.code = FileSystemErrorCode.DIRECTORY_NOT_FOUND;
 
     const result = FileSystemErrorHandler.handleError(error, mockOperation, mockFilePath);
@@ -59,7 +59,7 @@ describe('FileSystemErrorHandler', () => {
   });
 
   it('should handle no space left on device errors', () => {
-    const error = new Error('No space left on device') as any;
+    const error = new Error('No space left on device') as Error & { code: FileSystemErrorCode };
     error.code = FileSystemErrorCode.NO_SPACE_LEFT;
 
     const result = FileSystemErrorHandler.handleError(error, mockOperation, mockFilePath);
@@ -75,7 +75,7 @@ describe('FileSystemErrorHandler', () => {
   });
 
   it('should handle read-only file system errors', () => {
-    const error = new Error('Read-only file system') as any;
+    const error = new Error('Read-only file system') as Error & { code: FileSystemErrorCode };
     error.code = FileSystemErrorCode.READ_ONLY_FILE_SYSTEM;
 
     const result = FileSystemErrorHandler.handleError(error, mockOperation, mockFilePath);
@@ -91,7 +91,7 @@ describe('FileSystemErrorHandler', () => {
   });
 
   it('should handle too many open files errors', () => {
-    const error = new Error('Too many open files') as any;
+    const error = new Error('Too many open files') as Error & { code: FileSystemErrorCode };
     error.code = FileSystemErrorCode.TOO_MANY_OPEN_FILES;
 
     const result = FileSystemErrorHandler.handleError(error, mockOperation, mockFilePath);
@@ -107,7 +107,7 @@ describe('FileSystemErrorHandler', () => {
   });
 
   it('should handle invalid path errors', () => {
-    const error = new Error('Invalid path') as any;
+    const error = new Error('Invalid path') as Error & { code: FileSystemErrorCode };
     error.code = FileSystemErrorCode.INVALID_PATH;
 
     const result = FileSystemErrorHandler.handleError(error, mockOperation, mockFilePath);
@@ -154,15 +154,20 @@ describe('FileSystemErrorHandler', () => {
   });
 
   describe('logErrorResult', () => {
-    let mockLogger: any;
+    let mockLogger: {
+      error: ReturnType<typeof vi.fn>;
+      warn: ReturnType<typeof vi.fn>;
+      log: ReturnType<typeof vi.fn>;
+    };
 
     beforeEach(() => {
       mockLogger = {
         error: vi.fn(),
         warn: vi.fn(),
         log: vi.fn(),
       };
-      (FileSystemErrorHandler as any).logger = mockLogger;
+      // Use type assertion to set private static property
+      (FileSystemErrorHandler as unknown as { logger: typeof mockLogger }).logger = mockLogger;
     });
 
     it('should log critical errors with suggestions', () => {

src/modules/build/utils/file-system-error-handler.ts

@@ -106,8 +106,7 @@ export class FileSystemErrorHandler {
     }
 
     // Handle generic errors without specific error codes
-    const genericError = error instanceof Error ? error : new Error(String(error));
-    return this.handleGenericError(genericError, operation, filePath);
+    return this.handleGenericError(error, operation, filePath);
   }
 
   /**
@@ -239,11 +238,32 @@ export class FileSystemErrorHandler {
   /**
    * Handle generic/unknown file system errors
    */
-  private static handleGenericError(error: Error, operation: string, filePath: string): FileSystemErrorResult {
-    const errorMessage = error.message || 'Unknown error';
+  private static handleGenericError(error: unknown, operation: string, filePath: string): FileSystemErrorResult {
+    // Extract error message from various error formats
+    let errorMessage = 'Unknown error';
+    let errorStack: string | undefined;
+
+    if (error instanceof Error) {
+      errorMessage = error.message || 'Unknown error';
+      errorStack = error.stack;
+    } else if (error && typeof error === 'object') {
+      // Handle plain objects with message property
+      const errorObj = error as Record<string, unknown>;
+      if (typeof errorObj.message === 'string') {
+        errorMessage = errorObj.message;
+      }
+      if (typeof errorObj.stack === 'string') {
+        errorStack = errorObj.stack;
+      }
+    } else if (typeof error === 'string') {
+      errorMessage = error;
+    } else {
+      errorMessage = String(error);
+    }
+
     const userMessage = `File system error when ${operation}: ${filePath} - ${errorMessage}`;
 
-    this.logger.error(userMessage, error.stack);
+    this.logger.error(userMessage, errorStack);
 
     return {
       shouldContinue: false,