ix(lint): fix 10 TypeScript errors across 3 test files and example

sandbox-handle.test.ts: remove stale mockCtx second argument from all
 execute() calls — ToolDef.execute was refactored to single-arg in v1.3.

 cli-coordination-view.test.ts + cli-ui.test.ts: type stderrSpy as
 MockInstance<(...args: any[]) => any> to resolve overload conflict on
 process.stderr.write.

 examples/consensus-debate.ts: update to current v1.3 API —
 createOrchestrator returns { engine } not { orchestrator }, constructor
 no longer takes positional args, runDebate → executeClashDebate,
 engine.formatReport → formatDebateReport, provider 'grok' → 'xai'.

Author: marceloceccon

examples/consensus-debate.ts

@@ -1,7 +1,7 @@
 /**
  * consensus-debate.ts — run a multi-persona debate programmatically.
  *
- * Runs ClashEngine.runDebate with 3 personas × 2 rounds on a topic
+ * Runs ClashEngine.executeClashDebate with 3 personas × 2 rounds on a topic
  * passed as argv[2]. Prints the structured report at the end.
  *
  * ⚠ This calls the real LLM provider and will cost tokens.
@@ -12,7 +12,12 @@
  *   pnpm tsx examples/consensus-debate.ts "Should we adopt Rust?"
  */
 
-import { ClashEngine, BUILT_IN_PERSONAS, createOrchestrator, resolveApiKey } from '../src/index.js'
+import {
+  createOrchestrator,
+  formatDebateReport,
+  BUILT_IN_PERSONAS,
+  resolveApiKey,
+} from '../src/index.js'
 
 async function main(): Promise<void> {
   const topic = process.argv[2] ?? 'Should we prefer SQL over NoSQL for a new social network?'
@@ -27,23 +32,19 @@ async function main(): Promise<void> {
   console.log('Personas available:', Object.keys(BUILT_IN_PERSONAS).join(', '))
   console.log()
 
-  const { orchestrator } = createOrchestrator({
+  const { engine } = createOrchestrator({
     defaultModel: 'grok-4',
-    defaultProvider: 'grok',
+    defaultProvider: 'xai',
     defaultApiKey: apiKey,
   })
-  const engine = new ClashEngine('grok-4', 'grok')
-
-  const result = await engine.runDebate(
-    {
-      topic,
-      rounds: 2,
-      personas: ['pragmatist', 'elegance-purist', 'security-maximalist'],
-    },
-    orchestrator,
-  )
-
-  console.log('\n' + engine.formatReport(result))
+
+  const result = await engine.executeClashDebate({
+    topic,
+    rounds: 2,
+    personas: ['pragmatist', 'elegance-purist', 'security-maximalist'],
+  })
+
+  console.log('\n' + formatDebateReport(result))
 }
 
 main().catch((err) => {

test/cli-coordination-view.test.ts

@@ -6,7 +6,7 @@
  * freezeCoordinationView) in non-interactive mode where output goes to
  * stderr as line logs.
  */
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { describe, it, expect, vi, beforeEach, afterEach, type MockInstance } from 'vitest'
 
 // Force non-interactive mode so no ANSI cursor control happens.
 process.env['CLASHCODE_NO_TUI'] = '1'
@@ -21,7 +21,8 @@ import {
 } from '../src/cli/coordination-view.js'
 
 describe('coordination-view (non-interactive)', () => {
-  let stderrSpy: ReturnType<typeof vi.spyOn>
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  let stderrSpy: MockInstance<(...args: any[]) => any>
 
   beforeEach(() => {
     stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => true)

test/cli-ui.test.ts

@@ -1,7 +1,7 @@
 /**
  * Unit tests for src/cli/ui.ts — ANSI helpers, Spinner, box, formatResponse.
  */
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
+import { describe, it, expect, vi, beforeEach, afterEach, type MockInstance } from 'vitest'
 import {
   c,
   VERSION,
@@ -44,7 +44,8 @@ describe('banner', () => {
 })
 
 describe('Spinner', () => {
-  let stderrSpy: ReturnType<typeof vi.spyOn>
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  let stderrSpy: MockInstance<(...args: any[]) => any>
 
   beforeEach(() => {
     stderrSpy = vi.spyOn(process.stderr, 'write').mockImplementation(() => true)

test/sandbox-handle.test.ts

@@ -9,9 +9,6 @@ vi.mock('../src/sandbox/backends/shuru-bootstrap.js', () => ({
   ensureBootstrapCheckpoint: vi.fn().mockResolvedValue(undefined),
 }))
 
-// Mock ToolUseContext for execute() calls
-const mockCtx = { agent: { name: 'test', model: 'test' } } as any
-
 vi.mock('../src/logger.js', () => ({
   logger: {
     info: vi.fn(),
@@ -262,7 +259,7 @@ describe('createSandboxTools — tool definitions', () => {
 
     const handle = new SandboxHandle()
     const tools = createSandboxTools(handle)
-    const result = await tools.sandboxExecTool.execute({ command: 'echo hello world' }, mockCtx)
+    const result = await tools.sandboxExecTool.execute({ command: 'echo hello world' })
 
     expect(result.data).toContain('hello world')
     expect(result.isError).toBeUndefined()
@@ -276,7 +273,7 @@ describe('createSandboxTools — tool definitions', () => {
 
     const handle = new SandboxHandle()
     const tools = createSandboxTools(handle)
-    const result = await tools.sandboxExecTool.execute({ command: 'cmd' }, mockCtx)
+    const result = await tools.sandboxExecTool.execute({ command: 'cmd' })
 
     expect(result.data).toContain('out')
     expect(result.data).toContain('err')
@@ -290,7 +287,7 @@ describe('createSandboxTools — tool definitions', () => {
 
     const handle = new SandboxHandle()
     const tools = createSandboxTools(handle)
-    const result = await tools.sandboxExecTool.execute({ command: 'true' }, mockCtx)
+    const result = await tools.sandboxExecTool.execute({ command: 'true' })
 
     expect(result.data).toBe('(no output)')
   })
@@ -306,7 +303,7 @@ describe('createSandboxTools — tool definitions', () => {
     await handle.get()
 
     const tools = createSandboxTools(handle)
-    const result = await tools.sandboxExecTool.execute({ command: 'fail' }, mockCtx)
+    const result = await tools.sandboxExecTool.execute({ command: 'fail' })
 
     expect(result.isError).toBe(true)
     expect(result.data).toContain('container died')
@@ -318,13 +315,10 @@ describe('createSandboxTools — tool definitions', () => {
 
     const handle = new SandboxHandle()
     const tools = createSandboxTools(handle)
-    const result = await tools.sandboxWriteTool.execute(
-      {
-        path: '/workspace/test.txt',
-        content: 'hello',
-      },
-      mockCtx,
-    )
+    const result = await tools.sandboxWriteTool.execute({
+      path: '/workspace/test.txt',
+      content: 'hello',
+    })
 
     expect(result.data).toContain('5 bytes')
     expect(result.data).toContain('/workspace/test.txt')
@@ -339,13 +333,10 @@ describe('createSandboxTools — tool definitions', () => {
     const handle = new SandboxHandle()
     await handle.get()
     const tools = createSandboxTools(handle)
-    const result = await tools.sandboxWriteTool.execute(
-      {
-        path: '/tmp/x',
-        content: 'data',
-      },
-      mockCtx,
-    )
+    const result = await tools.sandboxWriteTool.execute({
+      path: '/tmp/x',
+      content: 'data',
+    })
 
     expect(result.isError).toBe(true)
     expect(result.data).toContain('write failed')
@@ -359,7 +350,7 @@ describe('createSandboxTools — tool definitions', () => {
 
     const handle = new SandboxHandle()
     const tools = createSandboxTools(handle)
-    const result = await tools.sandboxReadTool.execute({ path: '/workspace/file.txt' }, mockCtx)
+    const result = await tools.sandboxReadTool.execute({ path: '/workspace/file.txt' })
 
     expect(result.data).toBe('the file contents')
   })
@@ -373,7 +364,7 @@ describe('createSandboxTools — tool definitions', () => {
     const handle = new SandboxHandle()
     await handle.get()
     const tools = createSandboxTools(handle)
-    const result = await tools.sandboxReadTool.execute({ path: '/tmp/nope' }, mockCtx)
+    const result = await tools.sandboxReadTool.execute({ path: '/tmp/nope' })
 
     expect(result.isError).toBe(true)
     expect(result.data).toContain('no such file')