fix: Apply linting fixes for code formatting and unused variables

- Fix code formatting (indentation, trailing commas)
- Prefix unused function parameters with underscore
- Replace global isNaN with Number.isNaN
- Wrap switch clause declarations in blocks

Author: nolindnaidoo

src/commands/sort.ts

@@ -5,53 +5,49 @@ import { showProgress } from '../utils/progress'
 import { sortDates } from '../utils/sort'
 
 export async function sortDatesCommand(): Promise<void> {
-  const editor = vscode.window.activeTextEditor
-  if (!editor) {
-    vscode.window.showWarningMessage('No active editor found')
-    return
-  }
-
-  const document = editor.document
-  const content = document.getText()
-  const languageId = document.languageId
-
-  if (!content.trim()) {
-    vscode.window.showWarningMessage('Document is empty')
-    return
-  }
-
-  const config = getConfiguration()
-
-  try {
-    await showProgress('Sorting dates...', async () => {
-      const result = await extractDates(content, languageId)
-
-      if (result.dates.length === 0) {
-        vscode.window.showInformationMessage('No dates found to sort')
-        return
-      }
-
-      const sortedDates = sortDates([...result.dates], 'chronological')
-
-      // Create a new document with sorted dates
-      const sortedContent = sortedDates
-        .map((date) => `${date.value} - ${date.context || 'Extracted date'}`)
-        .join('\n')
-
-      // Open a new document with sorted results
-      const doc = await vscode.workspace.openTextDocument({
-        content: sortedContent,
-        language: 'plaintext',
-      })
-
-      await vscode.window.showTextDocument(doc)
-
-      vscode.window.showInformationMessage(`Sorted ${sortedDates.length} dates chronologically`)
-    })
-  } catch (error) {
-    console.error('[Dates-LE] Sort command error:', error)
-    vscode.window.showErrorMessage(
-      `Failed to sort dates: ${error instanceof Error ? error.message : 'Unknown error'}`,
-    )
-  }
+	const editor = vscode.window.activeTextEditor
+	if (!editor) {
+		vscode.window.showWarningMessage('No active editor found')
+		return
+	}
+
+	const document = editor.document
+	const content = document.getText()
+	const languageId = document.languageId
+
+	if (!content.trim()) {
+		vscode.window.showWarningMessage('Document is empty')
+		return
+	}
+
+	const _config = getConfiguration()
+
+	try {
+		await showProgress('Sorting dates...', async () => {
+			const result = await extractDates(content, languageId)
+
+			if (result.dates.length === 0) {
+				vscode.window.showInformationMessage('No dates found to sort')
+				return
+			}
+
+			const sortedDates = sortDates([...result.dates], 'chronological')
+
+			// Create a new document with sorted dates
+			const sortedContent = sortedDates.map((date) => `${date.value} - ${date.context || 'Extracted date'}`).join('\n')
+
+			// Open a new document with sorted results
+			const doc = await vscode.workspace.openTextDocument({
+				content: sortedContent,
+				language: 'plaintext',
+			})
+
+			await vscode.window.showTextDocument(doc)
+
+			vscode.window.showInformationMessage(`Sorted ${sortedDates.length} dates chronologically`)
+		})
+	} catch (error) {
+		console.error('[Dates-LE] Sort command error:', error)
+		vscode.window.showErrorMessage(`Failed to sort dates: ${error instanceof Error ? error.message : 'Unknown error'}`)
+	}
 }

src/config/configuration.ts

@@ -2,42 +2,39 @@ import * as vscode from 'vscode'
 import type { Configuration, OutputFormat } from '../types'
 
 export function getConfiguration(): Configuration {
-  const config = vscode.workspace.getConfiguration('dates-le')
+	const config = vscode.workspace.getConfiguration('dates-le')
 
-  return Object.freeze({
-    copyToClipboardEnabled: config.get<boolean>('copyToClipboardEnabled') ?? false,
-    dedupeEnabled: config.get<boolean>('dedupeEnabled') ?? true,
-    notificationsLevel:
-      config.get<'all' | 'important' | 'silent'>('notificationsLevel') ?? 'important',
-    postProcessOpenInNewFile: config.get<boolean>('postProcessOpenInNewFile') ?? false,
-    openResultsSideBySide: config.get<boolean>('openResultsSideBySide') ?? true,
-    safetyEnabled: config.get<boolean>('safetyEnabled') ?? true,
-    safetyFileSizeWarnBytes: config.get<number>('safetyFileSizeWarnBytes') ?? 1048576,
-    safetyLargeOutputLinesThreshold: config.get<number>('safetyLargeOutputLinesThreshold') ?? 1000,
-    safetyManyDocumentsThreshold: config.get<number>('safetyManyDocumentsThreshold') ?? 10,
-    showParseErrors: config.get<boolean>('showParseErrors') ?? true,
-    statusBarEnabled: config.get<boolean>('statusBarEnabled') ?? true,
-    telemetryEnabled: config.get<boolean>('telemetryEnabled') ?? false,
-    analysisEnabled: config.get<boolean>('analysisEnabled') ?? false,
-    analysisIncludePatterns: config.get<boolean>('analysisIncludePatterns') ?? false,
-    analysisIncludeRanges: config.get<boolean>('analysisIncludeRanges') ?? false,
-    analysisIncludeAnomalies: config.get<boolean>('analysisIncludeAnomalies') ?? false,
-    outputFormat: config.get<OutputFormat>('outputFormat') ?? 'iso',
-    timezone: config.get<string>('timezone') ?? 'UTC',
-    performanceEnabled: config.get<boolean>('performanceEnabled') ?? false,
-    performanceMaxDuration: config.get<number>('performanceMaxDuration') ?? 5000,
-    performanceMaxMemoryUsage: config.get<number>('performanceMaxMemoryUsage') ?? 100,
-    performanceMaxCpuUsage: config.get<number>('performanceMaxCpuUsage') ?? 80,
-    performanceMinThroughput: config.get<number>('performanceMinThroughput') ?? 1000,
-    performanceMaxCacheSize: config.get<number>('performanceMaxCacheSize') ?? 100,
-    keyboardShortcutsEnabled: config.get<boolean>('keyboardShortcutsEnabled') ?? true,
-    keyboardExtractShortcut: config.get<string>('keyboardExtractShortcut') ?? 'ctrl+shift+e',
-    keyboardConvertShortcut: config.get<string>('keyboardConvertShortcut') ?? 'ctrl+shift+c',
-    keyboardAnalyzeShortcut: config.get<string>('keyboardAnalyzeShortcut') ?? 'ctrl+shift+a',
-    presetsEnabled: config.get<boolean>('presetsEnabled') ?? false,
-    defaultPreset:
-      config.get<'minimal' | 'balanced' | 'comprehensive' | 'performance' | 'timezone'>(
-        'defaultPreset',
-      ) ?? 'balanced',
-  })
+	return Object.freeze({
+		copyToClipboardEnabled: config.get<boolean>('copyToClipboardEnabled') ?? false,
+		dedupeEnabled: config.get<boolean>('dedupeEnabled') ?? true,
+		notificationsLevel: config.get<'all' | 'important' | 'silent'>('notificationsLevel') ?? 'important',
+		postProcessOpenInNewFile: config.get<boolean>('postProcessOpenInNewFile') ?? false,
+		openResultsSideBySide: config.get<boolean>('openResultsSideBySide') ?? true,
+		safetyEnabled: config.get<boolean>('safetyEnabled') ?? true,
+		safetyFileSizeWarnBytes: config.get<number>('safetyFileSizeWarnBytes') ?? 1048576,
+		safetyLargeOutputLinesThreshold: config.get<number>('safetyLargeOutputLinesThreshold') ?? 1000,
+		safetyManyDocumentsThreshold: config.get<number>('safetyManyDocumentsThreshold') ?? 10,
+		showParseErrors: config.get<boolean>('showParseErrors') ?? true,
+		statusBarEnabled: config.get<boolean>('statusBarEnabled') ?? true,
+		telemetryEnabled: config.get<boolean>('telemetryEnabled') ?? false,
+		analysisEnabled: config.get<boolean>('analysisEnabled') ?? false,
+		analysisIncludePatterns: config.get<boolean>('analysisIncludePatterns') ?? false,
+		analysisIncludeRanges: config.get<boolean>('analysisIncludeRanges') ?? false,
+		analysisIncludeAnomalies: config.get<boolean>('analysisIncludeAnomalies') ?? false,
+		outputFormat: config.get<OutputFormat>('outputFormat') ?? 'iso',
+		timezone: config.get<string>('timezone') ?? 'UTC',
+		performanceEnabled: config.get<boolean>('performanceEnabled') ?? false,
+		performanceMaxDuration: config.get<number>('performanceMaxDuration') ?? 5000,
+		performanceMaxMemoryUsage: config.get<number>('performanceMaxMemoryUsage') ?? 100,
+		performanceMaxCpuUsage: config.get<number>('performanceMaxCpuUsage') ?? 80,
+		performanceMinThroughput: config.get<number>('performanceMinThroughput') ?? 1000,
+		performanceMaxCacheSize: config.get<number>('performanceMaxCacheSize') ?? 100,
+		keyboardShortcutsEnabled: config.get<boolean>('keyboardShortcutsEnabled') ?? true,
+		keyboardExtractShortcut: config.get<string>('keyboardExtractShortcut') ?? 'ctrl+shift+e',
+		keyboardConvertShortcut: config.get<string>('keyboardConvertShortcut') ?? 'ctrl+shift+c',
+		keyboardAnalyzeShortcut: config.get<string>('keyboardAnalyzeShortcut') ?? 'ctrl+shift+a',
+		presetsEnabled: config.get<boolean>('presetsEnabled') ?? false,
+		defaultPreset:
+			config.get<'minimal' | 'balanced' | 'comprehensive' | 'performance' | 'timezone'>('defaultPreset') ?? 'balanced',
+	})
 }

src/extraction/extract.ts

@@ -4,54 +4,54 @@ import { extractFromJson } from './formats/json'
 import { extractFromYaml } from './formats/yaml'
 
 export async function extractDates(content: string, languageId: string): Promise<ExtractionResult> {
-  const fileType = determineFileType(languageId)
-  const dates: DateValue[] = []
-  const errors: ParseError[] = []
+	const fileType = determineFileType(languageId)
+	const dates: DateValue[] = []
+	const errors: ParseError[] = []
 
-  try {
-    switch (fileType) {
-      case 'json':
-        dates.push(...extractFromJson(content))
-        break
-      case 'yaml':
-      case 'yml':
-        dates.push(...extractFromYaml(content))
-        break
-      case 'csv':
-        dates.push(...extractFromCsv(content))
-        break
-      default:
-        // Unsupported file type - return empty results
-        break
-    }
-  } catch (error) {
-    errors.push({
-      category: 'parsing' as const,
-      severity: 'warning' as const,
-      message: error instanceof Error ? error.message : 'Unknown parsing error',
-      recoverable: true,
-      recoveryAction: 'skip' as const,
-      timestamp: Date.now(),
-    })
-  }
+	try {
+		switch (fileType) {
+			case 'json':
+				dates.push(...extractFromJson(content))
+				break
+			case 'yaml':
+			case 'yml':
+				dates.push(...extractFromYaml(content))
+				break
+			case 'csv':
+				dates.push(...extractFromCsv(content))
+				break
+			default:
+				// Unsupported file type - return empty results
+				break
+		}
+	} catch (error) {
+		errors.push({
+			category: 'parsing' as const,
+			severity: 'warning' as const,
+			message: error instanceof Error ? error.message : 'Unknown parsing error',
+			recoverable: true,
+			recoveryAction: 'skip' as const,
+			timestamp: Date.now(),
+		})
+	}
 
-  return Object.freeze({
-    success: errors.length === 0,
-    dates: Object.freeze(dates),
-    errors: Object.freeze(errors),
-  })
+	return Object.freeze({
+		success: errors.length === 0,
+		dates: Object.freeze(dates),
+		errors: Object.freeze(errors),
+	})
 }
 
 function determineFileType(languageId: string): FileType {
-  switch (languageId) {
-    case 'json':
-      return 'json'
-    case 'yaml':
-    case 'yml':
-      return 'yaml'
-    case 'csv':
-      return 'csv'
-    default:
-      return 'unknown'
-  }
+	switch (languageId) {
+		case 'json':
+			return 'json'
+		case 'yaml':
+		case 'yml':
+			return 'yaml'
+		case 'csv':
+			return 'csv'
+		default:
+			return 'unknown'
+	}
 }