refactor: clean up imports and improve error handling

- Removed unused import of Memory from src/index.ts.
- Simplified error message construction in src/core/symbol-references.ts.
- Reorganized risk level calculation in src/tools/search-codebase.ts for clarity and consistency.
- Added a TODO comment for reviewing confidence calculation in src/tools/search-codebase.ts.

Author: PatrickSys

src/core/symbol-references.ts

@@ -83,8 +83,7 @@ export async function findSymbolReferences(
     chunksRaw = JSON.parse(content);
   } catch (error) {
     throw new IndexCorruptedError(
-      `Keyword index missing or unreadable (rebuild required): ${
-        error instanceof Error ? error.message : String(error)
+      `Keyword index missing or unreadable (rebuild required): ${error instanceof Error ? error.message : String(error)
       }`
     );
   }

src/index.ts

@@ -21,8 +21,7 @@ import {
 } from '@modelcontextprotocol/sdk/types.js';
 import { CodebaseIndexer } from './core/indexer.js';
 import type {
-  IndexingStats,
-  Memory
+  IndexingStats
 } from './types/index.js';
 import { analyzerRegistry } from './core/analyzer-registry.js';
 import { AngularAnalyzer } from './analyzers/angular/index.js';

src/tools/search-codebase.ts

@@ -9,7 +9,6 @@ import type { SearchResult } from '../types/index.js';
 import { buildEvidenceLock } from '../preflight/evidence-lock.js';
 import { shouldIncludePatternConflictCategory } from '../preflight/query-scope.js';
 import {
-  isComplementaryPatternCategory,
   isComplementaryPatternConflict,
   shouldSkipLegacyTestingFrameworkCategory
 } from '../patterns/semantics.js';
@@ -395,12 +394,6 @@ export async function handle(
       const resultPaths = results.map((r) => r.filePath);
       const impactCandidates = computeImpactCandidates(resultPaths);
 
-      let riskLevel: 'low' | 'medium' | 'high' = 'low';
-      if (impactCandidates.length > 10) {
-        riskLevel = 'high';
-      } else if (impactCandidates.length > 3) {
-        riskLevel = 'medium';
-      }
 
       // Use existing pattern intelligence for evidenceLock scoring, but keep the output payload lite.
       const preferredPatternsForEvidence: Array<{ pattern: string; example?: string }> = [];
@@ -417,6 +410,14 @@ export async function handle(
         }
       }
 
+
+      let riskLevel: 'low' | 'medium' | 'high' = 'low';
+      if (impactCandidates.length > 10) {
+        riskLevel = 'high';
+      } else if (impactCandidates.length > 3) {
+        riskLevel = 'medium';
+      }
+
       editPreflight = {
         mode: 'lite',
         riskLevel,
@@ -533,7 +534,8 @@ export async function handle(
       }));
 
       // --- Confidence (index freshness) ---
-      const confidence = computeIndexConfidence();
+      // TODO: Review this confidence calculation
+      //const confidence = computeIndexConfidence();
 
       // --- Failure memories (1.5x relevance boost) ---
       const failureWarnings = relatedMemories