Strict code review that ignores style
codingReviews a diff for correctness, edge cases, and regressions — and refuses to comment on formatting.
Plans a brownfield migration in steps small enough to revert, and names the point of no return.
Before touching a system that is currently serving traffic.
Write a migration plan from the current state to the target described below.
Requirements:
- Steps small enough that each one can be reverted in under ten minutes.
- Explicitly name the point of no return, and what must be true before crossing it.
- For each step: what to measure before and after, and the signal that means stop.
- Include a rollback section, written as commands or actions, not principles.
Rules:
- Do not plan a big-bang cutover.
- If the target cannot be reached incrementally, say so in the first line and stop.
- Assume the system is in use during the migration.
Current state:
<current>
Target state:
<target>
Constraints:
<constraints>
Fill in the constraints with real ones (freeze windows, team size, on-call). Without them the plan assumes infinite engineers.
The 'say so and stop' escape hatch catches the cases where migration is impossible incrementally — which is exactly when a confident plan is most dangerous.
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.