Clean up dead code and enable noUnusedLocals

Remove unused variables and dead code across the codebase:
- Remove unused imports (LiveConnectionInfo, ContextPattern, SshProfile)
- Remove dead functions (parseTimestamp, extractTimestampString in index.ts)
- Remove unused locals (componentMentions, visibleCols, endTime, etc.)
- Clean up write-only SSE state tracking in MCP server
- Enable noUnusedLocals in tsconfig to prevent future dead code

All 271 tests pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: ozgesolidkey

src/main/baselineStore.ts

@@ -147,7 +147,6 @@ export function buildFingerprint(
 
   // Single pass through the file
   let startTime: Date | null = null;
-  let endTime: Date | null = null;
   const minuteBuckets = new Map<number, number>(); // minute offset -> count
 
   const batchSize = 10000;
@@ -159,7 +158,6 @@ export function buildFingerprint(
       const ts = parseTimestamp(line.text);
       if (ts) {
         if (!startTime) startTime = ts;
-        endTime = ts;
         if (startTime) {
           const minuteOffset = Math.floor((ts.getTime() - startTime.getTime()) / 60000);
           if (minuteOffset >= 0 && minuteOffset < 1440) {

src/main/index.ts

@@ -15,7 +15,7 @@ if (process.platform !== 'linux') {
   console.warn('node-pty not available on Linux — using child_process fallback for terminal');
 }
 import { FileHandler, filterLineToVisibleColumns, ColumnConfig } from './fileHandler';
-import { IPC, SearchOptions, Bookmark, Highlight, HighlightGroup, SearchConfig, SearchConfigSession, ActivityEntry, LocalFileData, LiveConnectionInfo, ContextDefinition, ContextPattern, ContextMatchGroup, Annotation } from '../shared/types';
+import { IPC, SearchOptions, Bookmark, Highlight, HighlightGroup, SearchConfig, SearchConfigSession, ActivityEntry, LocalFileData, ContextDefinition, ContextMatchGroup, Annotation } from '../shared/types';
 import * as Diff from 'diff';
 import { analyzerRegistry, AnalyzerOptions, AnalysisResult } from './analyzers';
 import { loadDatadogConfig, saveDatadogConfig, clearDatadogConfig, fetchDatadogLogs, DatadogConfig, DatadogFetchParams } from './datadogClient';
@@ -1348,7 +1348,6 @@ app.whenReady().then(() => {
       // Scan the range
       const levelCounts: Record<string, number> = {};
       const crashes: { lineNumber: number; keyword: string; text: string }[] = [];
-      const componentMentions: Record<string, { lineCount: number; errorCount: number }> = {};
       const CRASH_KEYWORDS = ['fatal', 'panic', 'crash', 'exception', 'segfault', 'abort', 'oom', 'killed', 'core dump'];
       const timeGaps: { lineNumber: number; gapSeconds: number; prevTimestamp: string; currTimestamp: string }[] = [];
       let prevTimestamp: Date | null = null;
@@ -2593,7 +2592,6 @@ ipcMain.handle(IPC.SEARCH, async (_, options: SearchOptions) => {
 
     // Check for hidden column matches (matches in columns that are filtered out)
     if (options.columnConfig) {
-      const visibleCols = options.columnConfig.columns.filter(c => c.visible).map(c => c.index);
       const hiddenCols = options.columnConfig.columns.filter(c => !c.visible).map(c => c.index);
       if (hiddenCols.length > 0) {
         // Do a full-text search (without column config) to find matches in hidden columns
@@ -4521,70 +4519,11 @@ interface TimeGap {
   linePreview: string;
 }
 
-// Timestamp patterns for parsing
-const TIME_GAP_PATTERNS = [
-  // ISO format: 2024-01-25T10:00:01 or 2024-01-25 10:00:01
-  /(\d{4})-(\d{2})-(\d{2})[T ](\d{2}):(\d{2}):(\d{2})/,
-  // European format: DD.MM.YYYY HH:mm:ss (e.g., 02.01.2026 18:16:01.061)
-  /(\d{2})\.(\d{2})\.(\d{4})\s+(\d{2}):(\d{2}):(\d{2})/,
-  // Syslog format: Jan 25 10:00:01 (assumes current year)
-  /([A-Z][a-z]{2})\s+(\d{1,2})\s+(\d{2}):(\d{2}):(\d{2})/,
-];
-
 const MONTH_MAP: Record<string, number> = {
   'Jan': 0, 'Feb': 1, 'Mar': 2, 'Apr': 3, 'May': 4, 'Jun': 5,
   'Jul': 6, 'Aug': 7, 'Sep': 8, 'Oct': 9, 'Nov': 10, 'Dec': 11
 };
 
-function parseTimestamp(text: string): Date | null {
-  const sample = text.length > 100 ? text.substring(0, 100) : text;
-
-  // Try ISO format first
-  const isoMatch = sample.match(TIME_GAP_PATTERNS[0]);
-  if (isoMatch) {
-    const [, year, month, day, hour, min, sec] = isoMatch;
-    return new Date(
-      parseInt(year), parseInt(month) - 1, parseInt(day),
-      parseInt(hour), parseInt(min), parseInt(sec)
-    );
-  }
-
-  // Try European format: DD.MM.YYYY HH:mm:ss
-  const euroMatch = sample.match(TIME_GAP_PATTERNS[1]);
-  if (euroMatch) {
-    const [, day, month, year, hour, min, sec] = euroMatch;
-    return new Date(
-      parseInt(year), parseInt(month) - 1, parseInt(day),
-      parseInt(hour), parseInt(min), parseInt(sec)
-    );
-  }
-
-  // Try syslog format
-  const syslogMatch = sample.match(TIME_GAP_PATTERNS[2]);
-  if (syslogMatch) {
-    const [, monthStr, day, hour, min, sec] = syslogMatch;
-    const month = MONTH_MAP[monthStr];
-    if (month !== undefined) {
-      const now = new Date();
-      return new Date(
-        now.getFullYear(), month, parseInt(day),
-        parseInt(hour), parseInt(min), parseInt(sec)
-      );
-    }
-  }
-
-  return null;
-}
-
-function extractTimestampString(text: string): string | null {
-  const sample = text.length > 100 ? text.substring(0, 100) : text;
-  for (const pattern of TIME_GAP_PATTERNS) {
-    const match = sample.match(pattern);
-    if (match) return match[0];
-  }
-  return null;
-}
-
 interface TimeGapOptions {
   thresholdSeconds: number;
   startLine?: number;

src/main/sshHandler.ts

@@ -4,7 +4,7 @@ import * as path from 'path';
 import * as os from 'os';
 import { Client, SFTPWrapper } from 'ssh2';
 import type { FileEntry } from 'ssh2';
-import { SshProfile, SshStatus } from '../shared/types';
+import { SshStatus } from '../shared/types';
 import { wallClockPrefix } from './serialHandler';
 
 export interface SshHostEntry {

src/mcp-server/index.ts

@@ -91,8 +91,6 @@ interface ChatMessage {
 
 let sseBuffer: ChatMessage[] = [];
 let sseWaiters: Array<(msg: ChatMessage) => void> = [];
-let sseConnected = false;
-let sseReconnectTimer: ReturnType<typeof setTimeout> | null = null;
 
 let keepAliveTimer: ReturnType<typeof setInterval> | null = null;
 
@@ -108,14 +106,13 @@ function connectSSE(): void {
   const port = getApiPort();
   if (!port) {
     // Retry after a delay
-    sseReconnectTimer = setTimeout(connectSSE, 5000);
+    setTimeout(connectSSE, 5000);
     return;
   }
 
   startKeepAlive(port);
 
   const req = http.get(`http://127.0.0.1:${port}/api/events`, (res) => {
-    sseConnected = true;
     let buffer = '';
 
     res.on('data', (chunk: Buffer) => {
@@ -150,14 +147,12 @@ function connectSSE(): void {
     });
 
     res.on('end', () => {
-      sseConnected = false;
-      sseReconnectTimer = setTimeout(connectSSE, 3000);
+      setTimeout(connectSSE, 3000);
     });
   });
 
   req.on('error', () => {
-    sseConnected = false;
-    sseReconnectTimer = setTimeout(connectSSE, 5000);
+    setTimeout(connectSSE, 5000);
   });
 }
 

src/renderer/renderer.ts

@@ -4913,7 +4913,6 @@ function renderFolderSearchResults(pattern: string, cancelled?: boolean): void {
   `;
 
   const items = matches.map((match, index) => {
-    const relPath = match.filePath;
     const lineText = match.lineText.length > 200 ? match.lineText.substring(0, 200) + '...' : match.lineText;
 
     return `

src/tests/baseline.test.ts

@@ -165,8 +165,6 @@ describe('BaselineStore', () => {
       const id = store.save('test', '', [], fp);
       const before = store.get(id)!.updatedAt;
 
-      // Small delay to ensure different timestamp
-      const future = Date.now() + 100;
       store.update(id, { name: 'updated' });
       const after = store.get(id)!.updatedAt;
       expect(after).toBeGreaterThanOrEqual(before);

src/tests/benchmark.test.ts

@@ -155,8 +155,6 @@ describe('Logan Benchmark', () => {
       it(`should index ${size.label}`, async () => {
         const filePath = path.join(BENCH_DIR, `bench_${size.lines}.log`);
         const handler = new FileHandler();
-        const stat = fs.statSync(filePath);
-
         global.gc?.(); // Optional GC before measuring
         const memBefore = getMemoryMB();
         const start = performance.now();

src/tests/serial.test.ts

@@ -83,7 +83,7 @@ describe('FileHandler.indexNewLines', () => {
 
     // Append the rest + a new line
     fs.appendFileSync(tempFile, '-complete\nline3\n');
-    const newLines = handler.indexNewLines();
+    handler.indexNewLines();
     // The previous 'partial' line gets extended to 'partial-complete', plus 'line3'
     // So net new lines = 2 (re-parsed partial + new line3), but since the old 'partial'
     // was already counted, the total should be 3
@@ -446,7 +446,6 @@ describe('Wall clock timestamp prepending in serial data', () => {
   });
 
   it('should produce timestamps that time-gap detection can parse', async () => {
-    const buf = { value: '' };
     const handler = new FileHandler();
 
     try {

tsconfig.json

@@ -8,6 +8,7 @@
     "strict": true,
     "noImplicitReturns": true,
     "noFallthroughCasesInSwitch": true,
+    "noUnusedLocals": true,
     "esModuleInterop": true,
     "skipLibCheck": true,
     "forceConsistentCasingInFileNames": true,