Copilot instructions

1---
2applyTo: "src/__tests__/**/*.ts"
3---
4 
5# Test Review Guidelines
6 
7## Fixture-Based Testing
8 
9- Tests must NOT require real session files on the local machine — use fixture factories from `src/__tests__/fixtures/index.ts`
10- Each tool has a `create<Tool>Fixture()` factory that creates a temp directory with realistic session data
11- Ground fixture schemas in real session file formats — verify field names against actual tool storage before creating fixtures; do not invent schemas
12 
13## Conversion Coverage
14 
15- `unit-conversions.test.ts` is the primary suite — it must cover all N tools × (N-1) target conversion paths
16- Adding a new tool requires: a fixture factory AND conversion tests covering all N-1 directions (as both source and target)
17- PRs that add a new parser but do not update `unit-conversions.test.ts` are incomplete
18 
19## Test Quality
20 
21- Each test asserts ONE behavior — not multiple unrelated assertions bundled into a single test case
22- Test names should describe the scenario: `should extract session summary from Claude JSONL`
23- Tests must be independent — no shared mutable state between test cases
24- Use `beforeAll` / `afterAll` for fixture setup and cleanup (create temp dir → run tests → delete temp dir)
25 
26## Regression Tests
27 
28- Bug fixes must include a regression test that fails before the fix and passes after
29- PRs that modify parser logic without touching any test file should be flagged — the CI `test-quality` job will also flag this
30 
31## Performance
32 
33- Test timeout is 30 seconds (`vitest.config.ts`) — a test that times out indicates a parser with blocking or synchronous I/O
34- Excluded by vitest config: `e2e*`, `real-e2e*`, `stress*`, `injection*`, `parsers.test*` — these require a real environment and are not run in CI
35- Node.js 22+ is required; `node:sqlite` built-in is used by OpenCode/Crush fixtures — do not add third-party SQLite deps
36