TODO: claude-historian-mcp

Code Quality

Remove dead Claude Desktop code — universal-engine.ts contains ~1000 lines of unused Desktop/DXT integration code (LevelDB, IndexedDB, LocalStorage search). The feature branch is dead and DXT builds are not used. Comment out or remove the Desktop-specific methods, keeping only the UniversalHistorySearchEngine wrapper that delegates to HistorySearchEngine.
Eliminate any types — 99 instances of any across all files (JSON parsing, dynamic data, LevelDB). Refactor to use unknown with type guards, or define proper interfaces. This would allow upgrading eslint back to recommendedTypeChecked.
Re-enable noUncheckedIndexedAccess — Currently disabled due to 45+ violations from pervasive array iteration patterns. Once any types are eliminated and array access is properly guarded, this can be re-enabled for stronger type safety.
Fix console.error misuse — Previous console.log calls were mechanically replaced with console.error. Review each call site: some are debug logging that should be removed, others are genuine error reporting that should stay.

Extract Claude Desktop to separate package — If Desktop support is needed in the future, extract it to a separate claude-historian-desktop package rather than bundling dead code in the main MCP server.

Add return types — 2 functions missing explicit return types (search-helpers.ts:360, search.ts:712). Low priority but improves readability.