What problem does it solve?
Reviewing Vitest unit tests for correctness, maintainability, and adherence to project conventions is time-consuming and error-prone, especially when tests contain subtle issues like leaky spies, mis-scoped emitters, or tests that verify implementation details instead of behavior.
Core Features & Use Cases
- Structured 12-point checklist review: Evaluates test value and falsifiability, anti-patterns, setup weight, mock minimality, edge-case coverage, isolation, emitter scoping, spy cleanup, and snapshot appropriateness.
- Lint and suppression auditing: Runs ESLint with testing-library and jest-dom rules, greps for eslint-disable suppressions, wide
as unknown as casts, and private-method test seams.
- Cross-file consistency checks: Compares conventions against neighboring test files and flags inconsistent stubbing or assertion styles.
- Use Case: After writing a new
.vitest.tsx test suite for a Positron service, run this review to catch a new Emitter() created inside an it() block that would break event wiring through the test builder.
Quick Start
Review the Vitest test file at src/vs/workbench/services/example/test/myService.vitest.ts against the project checklist and report only failing items.
