// ❌ BEFORE (duplicate definition)
export interface AutoTrackConfig {
  autoTrackingEnabled: boolean;
  autoTrackDecisions: boolean;
  // ...
}

const autoTrackConfigSchema = z.object({
  autoTrackingEnabled: z.boolean(),
  // ...
});
// Type inferred from schema

// ✅ AFTER (removed duplicate, use Zod-inferred type only)
const autoTrackConfigSchema = z.object({
  autoTrackingEnabled: z.boolean(),
  // ...
});
type AutoTrackConfig = z.infer<typeof autoTrackConfigSchema>;

// ❌ BEFORE (old property names)
this.config = {
  enabled: false,
  trackDecisions: true,
  trackCompletions: true,
  trackErrors: true,
  sessionSummaryInterval: 50,
};

// ✅ AFTER (new schema property names)
this.config = {
  autoTrackingEnabled: false,
  autoTrackDecisions: true,
  autoTrackCompletions: true,
  autoTrackErrors: true,
  autoSummaryInterval: 50,
};

// ❌ BEFORE
if (!this.config.enabled) { return actions; }
if (this.config.trackDecisions) { ... }

// ✅ AFTER
if (!this.config.autoTrackingEnabled) { return actions; }
if (this.config.autoTrackDecisions) { ... }

// ❌ BEFORE
confidence: decisionMatch.weight, // possibly undefined

// ✅ AFTER
confidence: decisionMatch.weight ?? 0, // nullish coalescing

// ❌ BEFORE (old property names)
autoTracker.updateConfig({
  enabled: true,
  trackDecisions: pluginConfig.get('autoTrackDecisions') ?? true,
  // ...
});

// ✅ AFTER (new schema property names only)
autoTracker.updateConfig({
  autoTrackingEnabled: true,
  autoTrackDecisions: pluginConfig.get('autoTrackDecisions') ?? true,
  autoTrackCompletions: pluginConfig.get('autoTrackCompletions') ?? true,
  autoTrackErrors: pluginConfig.get('autoTrackErrors') ?? true,
  autoSummaryInterval: pluginConfig.get('autoSummaryInterval') ?? 50,
});

// ❌ BEFORE
import type { FileHandle } from '@lmstudio/sdk'; // unused

// ✅ AFTER (removed)

// ❌ BEFORE
const buffer = await attachment.read();

// ✅ AFTER (proper FileHandle method access)
const buffer = await (attachment as any).readFile 
  ? await (attachment as any).readFile() 
  : Buffer.from(await (attachment as any).read());

// ❌ BEFORE
const result = await (mammoth as { extractRawText: ... }).extractRawText({ buffer });

// ✅ AFTER (via unknown for safety)
const result = await ((mammoth as unknown) as { 
  extractRawText: (opts: { buffer: Buffer }) => Promise<{ value: string; messages: Array<{ message: string }> }>
}).extractRawText({ buffer });

// ❌ BEFORE (SimpleGit doesn't have .remote() method)
import * as simpleGit from 'simple-git';
const git = await createGit();
const remotes = await git.remote(['--get-url', 'origin']);

// ✅ AFTER (use child_process directly)
import * as childProcess from 'child_process';
const output = childProcess.execSync('git remote get-url origin 2>/dev/null', { 
  encoding: 'utf-8',
  stdio: ['pipe', 'pipe', 'ignore']
});
const remoteUrl = (output as string).trim();

// ❌ BEFORE (missing type assertions)
liveConfig: PluginConfig = {
  searchFallbackChain: pluginConfig.get('searchFallbackChain'), // string, not enum
  safesearch: pluginConfig.get('safesearch'), // string, not enum
  language: pluginConfig.get('language'), // string, not enum
  dateFormatStyle: pluginConfig.get('dateFormatStyle'), // string, not enum
};

// ✅ AFTER (explicit type assertions)
liveConfig: PluginConfig = {
  searchFallbackChain: pluginConfig.get('searchFallbackChain') as 'ddg-api' | 'ddg-fetch' | 'google' | 'bing',
  safesearch: pluginConfig.get('safesearch') as '0' | '1' | '2',
  language: pluginConfig.get('language') as 'en' | 'de' | 'zh-CN' | 'zh-TW',
  dateFormatStyle: pluginConfig.get('dateFormatStyle') as 'standard' | 'heuteIst',
};

// ❌ BEFORE (extra argument)
registerImageProcessingTools(config, lmClient).forEach(...);

// ✅ AFTER (correct signature)
registerImageProcessingTools(config).forEach(...);

$ npx tsc --project tsconfig.json --noEmit
src/autoTracker.ts(75,7): error TS2353...
src/promptPreprocessor.ts(343,9): error TS2353...
... (14 errors total)

$ npx tsc --project tsconfig.json --noEmit
# ✅ No output — compilation successful!

const schema = z.object({ /* ... */ });
type ConfigType = z.infer<typeof schema>;
// Use ConfigType everywhere — no manual interface definitions

const result = await ((lib as unknown) as CorrectType).method();