fix: resolve all ESLint errors failing CI

Remove unused imports/variables, fix inline import() type annotations,
replace unnecessary type assertions, and add eslint-disable for
intentional require() calls in tree-sitter/sqlite native module loading.

Author: sunilp

src/commands/run.ts

@@ -716,7 +716,7 @@ async function legacyRun(instruction: string, options: RunOptions): Promise<void
         // ── Duplicate write nudge ──────────────────────────────────────
         // If the model has written the same file 3+ times, nudge it to move on.
         if (!wasError && tc.name === 'write_file' && typeof tc.arguments['path'] === 'string') {
-          const wPath = tc.arguments['path'] as string;
+          const wPath = tc.arguments['path'];
           fileWriteCounts.set(wPath, (fileWriteCounts.get(wPath) ?? 0) + 1);
           if (fileWriteCounts.get(wPath)! >= 3) {
             stderrLog(formatInternalStatus(

src/commands/trace.ts

@@ -24,6 +24,7 @@ import {
   formatImpactReport,
 } from '../trace/index.js';
 import type { CliOverrides } from '../config/schema.js';
+import type { ChalkInstance } from 'chalk';
 
 export interface TraceOptions extends CliOverrides {
   depth?: number;
@@ -78,7 +79,7 @@ async function runTraceV2(
   symbolName: string,
   options: TraceOptions,
   workspaceRoot: string,
-  chalk: typeof import('chalk').default,
+  chalk: ChalkInstance,
   write: (msg: string) => void,
 ): Promise<void> {
   const indexDir = join(workspaceRoot, '.jam', 'trace-index');
@@ -119,7 +120,7 @@ async function runTraceV2(
     const result = traceSymbol(store, symbolName, { depth });
 
     if (result.notFound) {
-      let errMsg = `Symbol "${symbolName}" not found in the workspace.`;
+      const errMsg = `Symbol "${symbolName}" not found in the workspace.`;
       let hint = 'Check the spelling, or try --reindex to rebuild the index.';
       if (result.candidates && result.candidates.length > 0) {
         hint += '\n\nDid you mean:';
@@ -248,7 +249,7 @@ async function runTraceLegacy(
   symbolName: string,
   options: TraceOptions,
   workspaceRoot: string,
-  chalk: typeof import('chalk').default,
+  chalk: ChalkInstance,
   write: (msg: string) => void,
 ): Promise<void> {
   // Build call graph

src/providers/copilot-e2e.test.ts

@@ -39,7 +39,7 @@ describe('E2E: jam run with mock Copilot proxy', () => {
   });
 
   it('creates all requested Spring Boot files via tool calls', async () => {
-    const { stdout, stderr } = await execFileAsync(
+    const { stdout: _stdout, stderr: _stderr } = await execFileAsync(
       'node',
       [
         CLI_PATH,

src/providers/copilot-integration.test.ts

@@ -112,7 +112,7 @@ describe('CopilotProxyBackend integration (text response)', () => {
   it('returns text content with no tool calls and correct usage stats', async () => {
     const backend = new CopilotProxyBackend({ baseUrl: `http://127.0.0.1:${port}` });
 
-    const response = await backend.chatWithTools!(
+    const response = await backend.chatWithTools(
       [{ role: 'user', content: 'Say hello' }],
       [] // no tools — plain text scenario
     );

src/providers/copilot-sdk-backend.ts

@@ -325,7 +325,7 @@ function parseToolCalls(text: string): ToolCall[] | undefined {
       return [{
         id: `call_${Date.now()}`,
         name: parsed.tool,
-        arguments: (parsed.arguments ?? {}) as Record<string, unknown>,
+        arguments: parsed.arguments ?? {},
       }];
     }
   } catch {

src/trace/cross-language.test.ts

@@ -51,7 +51,7 @@ function populateStore(store: TraceStore): void {
   store.beginTransaction();
 
   // ── PaymentService.java symbols ──────────────────────────────────────────
-  const paymentServiceId = store.insertSymbol({
+  store.insertSymbol({
     name: 'PaymentService',
     kind: 'class',
     file: 'PaymentService.java',

src/trace/extractors/base.ts

@@ -1,5 +1,5 @@
 import type Parser from 'tree-sitter';
-import type { SymbolRecord, CallRecord, ImportRecord, ColumnRecord } from '../store.js';
+import type { SymbolRecord, ImportRecord } from '../store.js';
 
 /** Records extracted from a single file. */
 export interface ExtractionResult {

src/trace/extractors/java.test.ts

@@ -1,5 +1,5 @@
 // src/trace/extractors/java.test.ts
-import { describe, it, expect, beforeAll } from 'vitest';
+import { describe, it, expect } from 'vitest';
 import { isTreeSitterAvailable, parseSource } from '../parser.js';
 import './java.js'; // registers JavaExtractor
 import { getExtractor } from './base.js';

src/trace/extractors/python.test.ts

@@ -1,5 +1,5 @@
 // src/trace/extractors/python.test.ts
-import { describe, it, expect, beforeAll } from 'vitest';
+import { describe, it, expect } from 'vitest';
 import { isTreeSitterAvailable, parseSource } from '../parser.js';
 import './python.js'; // registers PythonExtractor
 import { getExtractor } from './base.js';

src/trace/extractors/python.ts

@@ -1,6 +1,6 @@
 // src/trace/extractors/python.ts
 import type Parser from 'tree-sitter';
-import { registerExtractor, findNodes, findNodesByTypes } from './base.js';
+import { registerExtractor, findNodes } from './base.js';
 import type { Extractor, ExtractionResult } from './base.js';
 
 /** Walk upward from a node to find the nearest enclosing function_definition. */