Fix indexing failure tracking

Author: peteretelej

src/cli.ts

@@ -34,9 +34,13 @@ export async function main() {
         const config = await loadConfig({ verbose: options.verbose });
         console.log(`Indexing ${paths.length} ${paths.length === 1 ? 'directory' : 'directories'}: ${paths.join(', ')}`);
         const result = await indexDirectories(paths, config);
-        console.log(`Indexed ${result.indexed} files, skipped ${result.skipped} files, ${result.errors.length} errors`);
-        if (result.errors.length > 0 && config.verbose) {
-          console.log('Errors:', result.errors);
+        console.log(`Indexed ${result.indexed} files, skipped ${result.skipped} files, ${result.failed} failed`);
+        if (result.errors.length > 0) {
+          console.log(`Errors: [`);
+          result.errors.forEach(error => {
+            console.log(`  '${error}'`);
+          });
+          console.log(`]`);
         }
       } catch (error) {
         console.error('Error indexing directories:', error);

src/indexing.ts

@@ -22,6 +22,7 @@ export interface ScanOptions {
 export interface IndexResult {
   indexed: number;
   skipped: number;
+  failed: number;
   errors: string[];
 }
 
@@ -164,6 +165,7 @@ async function shouldReprocessFile(filePath: string, existingRecord: FileRecord,
 export async function indexDirectories(paths: string[], config: Config): Promise<IndexResult> {
   let indexed = 0;
   let skipped = 0;
+  let failed = 0;
   const errors: string[] = [];
 
   const scanOptions: ScanOptions = {
@@ -253,7 +255,12 @@ export async function indexDirectories(paths: string[], config: Config): Promise
         } catch (error) {
           const errorMessage = error instanceof Error ? error.message : String(error);
           const causeMessage = error instanceof Error && error.cause ? `: ${(error.cause as Error).message}` : '';
-          errors.push(`Failed to process ${file.path}: ${errorMessage}${causeMessage}`);
+          const fullError = `Failed to process ${file.path}: ${errorMessage}${causeMessage}`;
+          errors.push(fullError);
+          failed++;
+          
+          // Print error immediately during processing (not just in verbose mode)
+          console.error(`❌ ${fullError}`);
         }
       }
 
@@ -269,5 +276,5 @@ export async function indexDirectories(paths: string[], config: Config): Promise
     }
   }
 
-  return { indexed, skipped, errors };
+  return { indexed, skipped, failed, errors };
 }

src/mcp-handlers.ts

@@ -66,11 +66,21 @@ export async function handleIndexTool(args: unknown, config: Config): Promise<Ca
   const paths = args.directory_path.split(',').map((p: string) => p.trim());
   const result = await indexDirectories(paths, config);
 
+  let responseText = `Indexed ${result.indexed} files, skipped ${result.skipped} files, ${result.failed} failed`;
+  
+  if (result.errors.length > 0) {
+    responseText += `\nErrors: [\n`;
+    result.errors.forEach(error => {
+      responseText += `  '${error}'\n`;
+    });
+    responseText += `]`;
+  }
+  
   return {
     content: [
       {
         type: 'text',
-        text: `Indexed ${result.indexed} files, skipped ${result.skipped} files, ${result.errors.length} errors`
+        text: responseText
       }
     ]
   };

tests/mcp-handlers.test.ts

@@ -37,6 +37,7 @@ describe('MCP Handlers Unit Tests', () => {
       vi.mocked(indexDirectories).mockResolvedValue({
         indexed: 5,
         skipped: 2,
+        failed: 1,
         errors: ['error1']
       });
 
@@ -49,7 +50,10 @@ describe('MCP Handlers Unit Tests', () => {
       expect(result).toEqual({
         content: [{
           type: 'text',
-          text: 'Indexed 5 files, skipped 2 files, 1 errors'
+          text: `Indexed 5 files, skipped 2 files, 1 failed
+Errors: [
+  'error1'
+]`
         }]
       });
     });