@boyingliu01/xp-gate 0.12.9 → 0.12.12

package/plugins/opencode/skills/sprint-flow/references/phase-2-build.md

@@ -20,6 +20,65 @@
 
 ---
 
+---
+
+## Uncommitted Changes Gate
+
+**Purpose**: Prevent entering BUILD with uncommitted changes that could mix with sprint work.
+
+**Execution**: Before entering Phase 2 BUILD, the orchestrator MUST check for uncommitted changes in the worktree.
+
+### Gate Logic
+
+```bash
+# Check for uncommitted changes
+UNCOMMITTED=$(git status --porcelain 2>/dev/null | wc -l | tr -d ' ')
+
+if [ "$SKIP_UNCOMMITTED_GATE" = "1" ]; then
+  echo "[UNCOMMITTED-GATE] Skipped (SKIP_UNCOMMITTED_GATE=1)"
+  echo "{\"skipped\":true,\"reason\":\"SKIP_UNCOMMITTED_GATE=1\",\"timestamp\":\"$(date -u +%Y-%m-%dT%H:%M:%SZ)\"}" > .sprint-state/uncommitted-gate-log.json
+elif [ "$UNCOMMITTED" -gt 0 ]; then
+  echo "⚠️ [UNCOMMITTED-GATE] Found ${UNCOMMITTED} uncommitted files in worktree:"
+  git status --short 2>/dev/null | head -20
+  echo ""
+  echo "Uncommitted changes may conflict with sprint work. Recommended actions:"
+  echo "  1. Commit changes: git add -A && git commit -m 'pre-sprint: save work before BUILD'"
+  echo "  2. Stash changes: git stash push -m 'pre-sprint stash'"
+  echo "  3. Skip gate: export SKIP_UNCOMMITTED_GATE=1 (not recommended)"
+  echo ""
+  echo "Logging to .sprint-state/uncommitted-gate-log.json"
+  echo "{\"blocked\":true,\"uncommitted_files\":${UNCOMMITTED},\"timestamp\":\"$(date -u +%Y-%m-%dT%H:%M:%SZ)\"}" > .sprint-state/uncommitted-gate-log.json
+  echo "[BLOCK] Uncommitted changes detected. Please commit, stash, or set SKIP_UNCOMMITTED_GATE=1 to continue."
+  exit 1
+else
+  echo "✅ [UNCOMMITTED-GATE] Worktree clean. Proceeding to BUILD."
+  echo "{\"clean\":true,\"timestamp\":\"$(date -u +%Y-%m-%dT%H:%M:%SZ)\"}" > .sprint-state/uncommitted-gate-log.json
+fi
+```
+
+### Escape Valve
+
+```bash
+# Skip the uncommitted changes gate (use with caution)
+SKIP_UNCOMMITTED_GATE=1
+```
+
+### Log Format (`.sprint-state/uncommitted-gate-log.json`)
+
+```json
+{
+  "clean": true,
+  "blocked": false,
+  "skipped": false,
+  "uncommitted_files": 0,
+  "timestamp": "2026-07-05T10:00:00Z"
+}
+```
+
+**Log location**: `.sprint-state/uncommitted-gate-log.json` — written on every gate execution for audit trail.
+
+---
+
 ## TDD 强制执行
 
 ### Gate 5a-BLOCK: 新增文件测试强制
@@ -41,3 +100,48 @@
 # 非 main/master 分支临时跳过 Gate 5a-BLOCK
 SKIP_GATE_5A_BLOCK=1 git commit -m "message"
 ```
+
+---
+
+## Timing & Stability
+
+This section documents expected execution times and timeout handling for each Phase 2 sub-step to reduce execution timing stddev and prevent pipeline stalls.
+
+### Expected Execution Times
+
+| Step | Description | Expected Time | Timeout | On Timeout |
+|------|-------------|--------------|---------|------------|
+| DELPHI-GATE | Verify delphi-reviewed.json exists | 1-2s | 10s | BLOCK (critical gate) |
+| ralph-loop (per REQ) | TDD + verification per requirement | 5-15 min | 30 min/REQ | Mark REQ as `timeout`, continue next REQ |
+| parallel dispatch | Multi-agent parallel build | 10-30 min | 45 min | Collect partial results, continue |
+| TDD cycle (per unit) | RED → GREEN → REFACTOR | 2-5 min | 10 min | Skip unit, log failure |
+| freeze + blind-review | Code review in isolation | 5-10 min | 20 min | WARNING, continue |
+| verification | Test suite + coverage check | 2-5 min | 15 min | Retry once, then BLOCK |
+| cost monitor | Token cost accounting | <1s | 5s | Skip, log warning |
+| Phase 2 total (lightweight) | ≤3 REQs | 15-30 min | 45 min | — |
+| Phase 2 total (standard) | 4-10 REQs | 30-120 min | 150 min | — |
+| Phase 2 total (complex) | >10 REQs | 60-240 min | 300 min | — |
+
+### Stability Guidelines
+
+1. **Timeout handling**: All Phase 2 sub-steps MUST have explicit timeouts. If a step times out, log the failure to `.sprint-state/phase-outputs/phase-2-errors.json` and continue to the next step (except DELPHI-GATE which is a hard BLOCK).
+
+2. **Retry strategy**: For recoverable failures (verification, TDD cycle):
+   - First failure: log warning, retry once
+   - Second failure: log error, BLOCK and prompt user decision
+   - Do NOT retry more than twice for any single sub-step
+
+3. **Parallel dispatch stability**: When using `--mode parallel`, if any agent fails:
+   - Collect partial results from successful agents
+   - Rerun failed agent(s) individually with `--mode ralph-loop`
+   - Do NOT rerun the entire parallel batch
+
+4. **Cost monitor thresholds**: 
+   - Token cost per REQ > 50,000 → WARNING (review REQ scope)
+   - Token cost per REQ > 100,000 → BLOCK (REQ too large, split into smaller units)
+
+5. **StdDev reduction**: To reduce execution timing variability:
+   - Cache dependency installation results between REQs
+   - Reuse TDD scaffolding across similar REQ types
+   - Pre-compute code structure analysis once at Phase start
+   - Batch lint/format operations at the end, not per REQ

package/plugins/opencode/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-07-05
-**Commit:** a620d9b
+**Generated:** 2026-07-06
+**Commit:** 132a12e
 **Branch:** main
-**Version:** 0.12.9.0
+**Version:** 0.12.12.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/plugins/qoder/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "xp-gate",
-  "version": "0.12.9",
+  "version": "0.12.12",
   "displayName": "XP-Gate",
   "description": "Extreme Programming quality gates + AI workflow skills for Qoder. Includes 10 quality gates (Gate 0-9), Sprint Flow (11 phases), and Delphi multi-expert review (>=90% consensus).",
   "author": {

package/plugins/qoder/skills/delphi-review/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/DELPHI-REVIEW KNOWLEDGE BASE
 
-**Generated:** 2026-07-05
-**Commit:** a620d9b
+**Generated:** 2026-07-06
+**Commit:** 132a12e
 **Branch:** main
-**Version:** 0.12.9.0
+**Version:** 0.12.12.0
 
 ## OVERVIEW
 Delphi Consensus Review — multi-round anonymous expert review (≥90% threshold, 3 experts from ≥2 providers, domestic models only). Supports design + code-walkthrough modes.

package/plugins/qoder/skills/sprint-flow/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/SPRINT-FLOW KNOWLEDGE BASE
 
-**Generated:** 2026-07-05
-**Commit:** a620d9b
+**Generated:** 2026-07-06
+**Commit:** 132a12e
 **Branch:** main
-**Version:** 0.12.9.0
+**Version:** 0.12.12.0
 
 ## OVERVIEW
 **11-phase** development pipeline: ISOLATE → AUTO-ESTIMATE → THINK → PLAN → BUILD → REVIEW → USER ACCEPTANCE → FEEDBACK → SHIP → LAND → CLEANUP. Phase 2 default build mode is **ralph-loop** (REQ-level iteration, 40-67% token savings vs parallel). HARD-GATE in Phase 1: design must pass Delphi review (≥90% consensus) before any coding.

package/plugins/qoder/skills/test-specification-alignment/AGENTS.md

@@ -1,9 +1,9 @@
 # SKILLS/TEST-SPECIFICATION-ALIGNMENT KNOWLEDGE BASE
 
-**Generated:** 2026-07-05
-**Commit:** a620d9b
+**Generated:** 2026-07-06
+**Commit:** 132a12e
 **Branch:** main
-**Version:** 0.12.9.0
+**Version:** 0.12.12.0
 
 ## OVERVIEW
 Test-Specification Alignment Engine — two-stage validation ensuring tests accurately reflect requirements and design specs.

package/principles/AGENTS.md

@@ -1,9 +1,9 @@
 # PRINCIPLES CHECKER MODULE
 
-**Generated:** 2026-07-05
-**Commit:** a620d9b
+**Generated:** 2026-07-06
+**Commit:** 132a12e
 **Branch:** main
-**Version:** 0.12.9.0
+**Version:** 0.12.12.0
 
 ## OVERVIEW
 Clean Code & SOLID principles checker — **Gate 4** of pre-commit. 14 rules × 9 language adapters, SARIF 2.1.0 output. Houses the **Boy Scout Rule** enforcement engine (Gate 6) and warning-baseline storage.

package/principles/__tests__/baseline-storage.test.ts

@@ -154,8 +154,9 @@ describe('Baseline Storage', () => {
     expect(stats.totalFiles).toBe(2);
     expect(stats.totalWarnings).toBe(7);
     expect(stats.averageWarningsPerFile).toBe(3.5);
-    expect(stats.eslint?.totalWarnings).toBe(6);
-    expect(stats.eslint?.totalErrors).toBe(1);
+    const richStats = stats as { totalFiles: number; totalWarnings: number; averageWarningsPerFile: number; eslint: { totalWarnings: number; totalErrors: number } };
+    expect(richStats.eslint.totalWarnings).toBe(6);
+    expect(richStats.eslint.totalErrors).toBe(1);
   });
 
   it('creates baseline from files with warning data', async () => {

package/principles/__tests__/baseline.test.ts

@@ -19,7 +19,7 @@ describe('BaselineEntry - lint tool fields (ruff, golangci, shellcheck)', () =>
         lastAnalyzed: new Date().toISOString(),
       },
     };
-    expect(() => storage.validate(entry as Record<string, import('../baseline').BaselineEntry>)).not.toThrow();
+    expect(() => storage.validate(entry as unknown as Record<string, import('../baseline').BaselineEntry>)).not.toThrow();
   });
 
   it('accepts golangci field in BaselineEntry', () => {
@@ -31,7 +31,7 @@ describe('BaselineEntry - lint tool fields (ruff, golangci, shellcheck)', () =>
         lastAnalyzed: new Date().toISOString(),
       },
     };
-    expect(() => storage.validate(entry as Record<string, import('../baseline').BaselineEntry>)).not.toThrow();
+    expect(() => storage.validate(entry as unknown as Record<string, import('../baseline').BaselineEntry>)).not.toThrow();
   });
 
   it('accepts shellcheck field in BaselineEntry', () => {
@@ -43,7 +43,7 @@ describe('BaselineEntry - lint tool fields (ruff, golangci, shellcheck)', () =>
         lastAnalyzed: new Date().toISOString(),
       },
     };
-    expect(() => storage.validate(entry as Record<string, import('../baseline').BaselineEntry>)).not.toThrow();
+    expect(() => storage.validate(entry as unknown as Record<string, import('../baseline').BaselineEntry>)).not.toThrow();
   });
 
   it('rejects ruff entry with non-numeric warnings', () => {
@@ -55,7 +55,7 @@ describe('BaselineEntry - lint tool fields (ruff, golangci, shellcheck)', () =>
         lastAnalyzed: new Date().toISOString(),
       },
     };
-    expect(() => storage.validate(entry as Record<string, import('../baseline').BaselineEntry>)).toThrow(/Invalid ruff properties/);
+    expect(() => storage.validate(entry as unknown as Record<string, import('../baseline').BaselineEntry>)).toThrow(/Invalid ruff properties/);
   });
 
   it('combines all lint tools in getSummaryStatistics', () => {
@@ -82,12 +82,13 @@ describe('BaselineEntry - lint tool fields (ruff, golangci, shellcheck)', () =>
 
     expect(stats.totalFiles).toBe(3);
     expect(stats.totalWarnings).toBe(10);
-    expect(stats.ruff?.totalWarnings).toBe(5);
-    expect(stats.ruff?.totalErrors).toBe(2);
-    expect(stats.golangci?.totalWarnings).toBe(3);
-    expect(stats.golangci?.totalErrors).toBe(1);
-    expect(stats.shellcheck?.totalWarnings).toBe(2);
-    expect(stats.shellcheck?.totalErrors).toBe(0);
+    const richStats = stats as typeof stats & { ruff: { totalWarnings: number; totalErrors: number }; golangci: { totalWarnings: number; totalErrors: number }; shellcheck: { totalWarnings: number; totalErrors: number } };
+    expect(richStats.ruff.totalWarnings).toBe(5);
+    expect(richStats.ruff.totalErrors).toBe(2);
+    expect(richStats.golangci.totalWarnings).toBe(3);
+    expect(richStats.golangci.totalErrors).toBe(1);
+    expect(richStats.shellcheck.totalWarnings).toBe(2);
+    expect(richStats.shellcheck.totalErrors).toBe(0);
   });
 });
 
@@ -121,11 +122,12 @@ describe('BaselineStorage - extended coverage', () => {
         },
       };
 
-      const stats = storage.getSummaryStatistics(baseline);
+      const rawStats = storage.getSummaryStatistics(baseline);
+      const stats = rawStats as { totalFiles: number; totalWarnings: number; averageWarningsPerFile: number; ccn: { totalWarnings: number; totalMax: number } };
 
       expect(stats.totalFiles).toBe(2);
-      expect(stats.ccn?.totalWarnings).toBe(8);
-      expect(stats.ccn?.totalMax).toBe(30);
+      expect(stats.ccn.totalWarnings).toBe(8);
+      expect(stats.ccn.totalMax).toBe(30);
     });
 
     it('returns averageWarningsPerFile of 0 for empty baseline', () => {
@@ -135,7 +137,7 @@ describe('BaselineStorage - extended coverage', () => {
       expect(stats.totalFiles).toBe(0);
       expect(stats.totalWarnings).toBe(0);
       expect(stats.averageWarningsPerFile).toBe(0);
-      expect(stats.ccn).toBeUndefined();
+      expect('ccn' in stats ? (stats as Record<string, unknown>).ccn : undefined).toBeUndefined();
     });
   });
 

package/principles/__tests__/config.test.ts

@@ -14,7 +14,7 @@ describe('config.ts - Configuration Loader', () => {
 
       // Clean code thresholds from design Section 6
       expect(config.rules['clean-code']['long-function'].threshold).toBe(50);
-      expect(config.rules['clean-code']['large-file'].threshold).toBe(650);
+      expect(config.rules['clean-code']['large-file'].threshold).toBe(1000);
       expect(config.rules['clean-code']['god-class'].threshold).toBe(15);
       expect(config.rules['clean-code']['deep-nesting'].threshold).toBe(4);
       expect(config.rules['clean-code']['too-many-params'].threshold).toBe(7);

package/principles/__tests__/types.test.ts

@@ -90,6 +90,7 @@ describe('types.ts - Core Interfaces', () => {
         parseAST: () => null,
         extractFunctions: () => [],
         extractClasses: () => [],
+        extractExports: () => [],
         countLines: () => 0,
       };
 

package/principles/adapters/__tests__/base.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, vi } from 'vitest';
+import { describe, it, expect, vi, type Mock } from 'vitest';;
 import type { Adapter } from '../../types';
 import { BaseAdapter } from '../base';
 
@@ -24,7 +24,7 @@ import { readFileSync } from 'fs';
 describe('BaseAdapter - Default Implementation', () => {
 
   it('should implement the Adapter interface', () => {
-    (readFileSync as vi.Mock).mockReturnValue('mock file content\ntest line 2');
+    (readFileSync as Mock).mockReturnValue('mock file content\ntest line 2');
     const adapter = new MockAdapter('./test.ts');
     const implemented: Adapter = adapter as unknown as Adapter;
     
@@ -37,7 +37,7 @@ describe('BaseAdapter - Default Implementation', () => {
   });
 
   it('should detect language based on file extension', () => {
-    (readFileSync as vi.Mock).mockReturnValue('file content');
+    (readFileSync as Mock).mockReturnValue('file content');
     
     const adapterTs = new MockAdapter('./test.ts');
     expect(adapterTs.detectLanguage()).toBe('typescript');
@@ -56,35 +56,35 @@ describe('BaseAdapter - Default Implementation', () => {
   });
 
   it('should parse file AST correctly', () => {
-    (readFileSync as vi.Mock).mockReturnValue('mock file content\ntest line 2');
+    (readFileSync as Mock).mockReturnValue('mock file content\ntest line 2');
     const adapter = new MockAdapter('./test.ts');
     const ast = adapter.parseAST();
     expect(ast).toEqual({ content: 'mock file content\ntest line 2', astType: 'mock' });
   });
 
   it('should extract functions from AST', () => {
-    (readFileSync as vi.Mock).mockReturnValue('');
+    (readFileSync as Mock).mockReturnValue('');
     const adapter = new MockAdapter('./test.ts');
     const functions = adapter.extractFunctions();
     expect(Array.isArray(functions)).toBe(true);
   });
 
   it('should extract classes from AST', () => {
-    (readFileSync as vi.Mock).mockReturnValue('');
+    (readFileSync as Mock).mockReturnValue('');
     const adapter = new MockAdapter('./test.ts');
     const classes = adapter.extractClasses();
     expect(Array.isArray(classes)).toBe(true);
   });
 
   it('should count physical lines correctly', () => {
-    (readFileSync as vi.Mock).mockReturnValue('line 1\nline 2\nline 3');
+    (readFileSync as Mock).mockReturnValue('line 1\nline 2\nline 3');
     const adapter = new MockAdapter('./test.ts');
     const lineCount = adapter.countLines();
     expect(lineCount).toBe(3);
   });
 
   it('should throw error for unsupported file operations', () => {
-    (readFileSync as vi.Mock).mockImplementation(() => {
+    (readFileSync as Mock).mockImplementation(() => {
       throw new Error('Could not read file');
     });
     

package/principles/adapters/__tests__/cpp.test.ts

@@ -1,4 +1,4 @@
-import { describe, it, expect, vi } from 'vitest';
+import { describe, it, expect, vi, type Mock } from 'vitest';;
 import { CppAdapter } from '../cpp';
 
 type MockFunction = Record<string, unknown>;
@@ -13,7 +13,7 @@ import { readFileSync } from 'fs';
 describe('CppAdapter', () => {
   
   it('should implement the Adapter interface', () => {
-    (readFileSync as vi.Mock).mockReturnValue('int main() { return 0; }');
+    (readFileSync as Mock).mockReturnValue('int main() { return 0; }');
     const adapter = new CppAdapter('test.cpp');
     
     expect(adapter).toHaveProperty('detectLanguage');
@@ -24,59 +24,59 @@ describe('CppAdapter', () => {
   });
 
   it('should detect language as cpp for .cpp files', () => {
-    (readFileSync as vi.Mock).mockReturnValue('int main() { return 0; }');
+    (readFileSync as Mock).mockReturnValue('int main() { return 0; }');
     const adapter = new CppAdapter('test.cpp');
     const detected = adapter.detectLanguage();
     expect(detected).toBe('cpp');
   });
 
   it('should detect language as cpp for .cxx files', () => {
-    (readFileSync as vi.Mock).mockReturnValue('int main() { return 0; }');
+    (readFileSync as Mock).mockReturnValue('int main() { return 0; }');
     const adapter = new CppAdapter('test.cxx');
     const detected = adapter.detectLanguage();
     expect(detected).toBe('cpp');
   });
 
   it('should detect language as cpp for .cc files', () => {
-    (readFileSync as vi.Mock).mockReturnValue('int main() { return 0; }');
+    (readFileSync as Mock).mockReturnValue('int main() { return 0; }');
     const adapter = new CppAdapter('test.cc');
     const detected = adapter.detectLanguage();
     expect(detected).toBe('cpp');
   });
 
   it('should detect language as cpp for .c files', () => {
-    (readFileSync as vi.Mock).mockReturnValue('int main() { return 0; }');
+    (readFileSync as Mock).mockReturnValue('int main() { return 0; }');
     const adapter = new CppAdapter('test.c');
     const detected = adapter.detectLanguage();
     expect(detected).toBe('cpp');
   });
 
   it('should detect language as cpp for .hpp files', () => {
-    (readFileSync as vi.Mock).mockReturnValue('#pragma once');
+    (readFileSync as Mock).mockReturnValue('#pragma once');
     const adapter = new CppAdapter('test.hpp');
     const detected = adapter.detectLanguage();
     expect(detected).toBe('cpp');
   });
 
   it('should detect language as cpp for .h files', () => {
-    (readFileSync as vi.Mock).mockReturnValue('#pragma once');
+    (readFileSync as Mock).mockReturnValue('#pragma once');
     const adapter = new CppAdapter('test.h');
     const detected = adapter.detectLanguage();
     expect(detected).toBe('cpp');
   });
 
   it('should parse C++ file AST correctly', () => {
-    (readFileSync as vi.Mock).mockReturnValue('int main() { return 0; }');
+    (readFileSync as Mock).mockReturnValue('int main() { return 0; }');
     const adapter = new CppAdapter('test.cpp');
     const ast = adapter.parseAST();
     expect(ast).toHaveProperty('content');
     expect(ast).toHaveProperty('language');
     expect(ast).toHaveProperty('filePath');
-    expect(ast.language).toBe('cpp');
+    expect((ast as { language: unknown }).language).toBe('cpp');
   });
 
   it('should extract functions from C++ code', () => {
-    (readFileSync as vi.Mock).mockReturnValue(`
+    (readFileSync as Mock).mockReturnValue(`
 int add(int a, int b) {
   return a + b;
 }
@@ -92,7 +92,7 @@ int main() {
   });
 
   it('should handle C++ strings with special characters', () => {
-    (readFileSync as vi.Mock).mockReturnValue(`
+    (readFileSync as Mock).mockReturnValue(`
 int main() {
   const char* str = "Hello \\"World\\"";
   char c = '\\'';
@@ -107,7 +107,7 @@ int main() {
   });
   
   it('should throw error when file cannot be read', () => {
-    (readFileSync as vi.Mock).mockImplementation(() => {
+    (readFileSync as Mock).mockImplementation(() => {
       throw new Error('Could not read file');
     });
     
@@ -125,7 +125,7 @@ int main() {
 describe('CppAdapter - extractCodeBlock edge cases', () => {
   it('should skip single-line block comment before the opening brace', () => {
     const src = `int foo() /* inline comment */ { return 1; }`;
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const block = adapter.extractCodeBlock(0);
     expect(block).toContain('{ return 1; }');
@@ -142,7 +142,7 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
       '  return 42;',
       '}',
     ].join('\n');
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const block = adapter.extractCodeBlock(0);
     expect(block).toContain('return 42;');
@@ -151,7 +151,7 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
 
   it('should ignore braces inside block comments', () => {
     const src = `int foo() /* fake { brace } here */ { return 7; }`;
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const block = adapter.extractCodeBlock(0);
     expect(block).toContain('return 7;');
@@ -160,7 +160,7 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
 
   it('should count nested braces correctly via braceCount++ path', () => {
     const src = `void foo() { if (x) { return; } }`;
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const block = adapter.extractCodeBlock(0);
     expect(block).toBe('void foo() { if (x) { return; } }');
@@ -168,7 +168,7 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
 
   it('should handle deeply nested braces (triple nesting)', () => {
     const src = `int bar() { if (a) { while (b) { c++; } } return 0; }`;
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const block = adapter.extractCodeBlock(0);
     expect(block.trim().endsWith('}')).toBe(true);
@@ -190,7 +190,7 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
       '  return 0;',
       '}',
     ].join('\n');
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const block = adapter.extractCodeBlock(0);
     expect(block.trim().endsWith('}')).toBe(true);
@@ -205,7 +205,7 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
       '  return 5;',
       '}',
     ].join('\n');
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const block = adapter.extractCodeBlock(0);
     expect(block).toContain('return 5;');
@@ -223,7 +223,7 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
       '  int value_;',
       '};',
     ].join('\n');
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const functions = adapter.extractFunctions();
     expect(Array.isArray(functions)).toBe(true);
@@ -240,7 +240,7 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
       '  int x;',
       '};',
     ].join('\n');
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const classes = adapter.extractClasses();
     expect(Array.isArray(classes)).toBe(true);
@@ -250,13 +250,13 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
   });
 
   it('should count lines correctly', () => {
-    (readFileSync as vi.Mock).mockReturnValue('line1\nline2\nline3');
+    (readFileSync as Mock).mockReturnValue('line1\nline2\nline3');
     const adapter = new CppAdapter('test.cpp');
     expect(adapter.countLines()).toBe(3);
   });
 
   it('should fall back to super.detectLanguage for non-cpp extensions', () => {
-    (readFileSync as vi.Mock).mockReturnValue('content');
+    (readFileSync as Mock).mockReturnValue('content');
     const adapter = new CppAdapter('test.txt');
     expect(adapter.detectLanguage()).toBe('unknown');
   });
@@ -272,7 +272,7 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
       '  return -x;',
       '}',
     ].join('\n');
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const functions = adapter.extractFunctions();
     expect(Array.isArray(functions)).toBe(true);
@@ -281,7 +281,7 @@ describe('CppAdapter - extractCodeBlock edge cases', () => {
 
   it('should extract functions with scope prefix (e.g. MyClass::method)', () => {
     const src = 'void MyClass::method() { return; }';
-    (readFileSync as vi.Mock).mockReturnValue(src);
+    (readFileSync as Mock).mockReturnValue(src);
     const adapter = new CppAdapter('test.cpp');
     const functions = adapter.extractFunctions();
     expect(Array.isArray(functions)).toBe(true);