Strict code review that ignores style
codingReviews a diff for correctness, edge cases, and regressions — and refuses to comment on formatting.
Builds the test matrix from requirements and failure modes, so it catches what the implementation forgot.
A feature spec is final and tests are about to be written from the implementation instead.
Derive a test plan from the specification below, not from any implementation.
Produce:
1. A matrix of inputs that matter: boundaries, empty values, invalid types, concurrent access, permission levels.
2. The behaviours that must hold, one line each, phrased so they can become test names.
3. Failure modes the spec does not cover, listed as questions to the author.
Rules:
- Do not describe implementation details; tests should survive a rewrite.
- Do not include tests that only assert the framework works.
- Mark anything you are guessing about with "assumption".
Spec:
<spec>
The third section is the most valuable output — it turns a test plan into a spec review.
Writing tests from the implementation reproduces the implementation's blind spots. Deriving them from the spec is the only way to catch the requirement that was never coded.
Reviews a diff for correctness, edge cases, and regressions — and refuses to comment on formatting.
Extracts or restructures code while proving the public behavior stays identical.
Writes the failing test first, then the fix, and refuses to skip the reproduction.