Diff Quality Review
Review a diff for bugs, security, load, missing tests and code to delete.
Install
npx skills add DietrichGebert/ponytail --skill ponytail-review -g -a claude-code -y- Repository
- DietrichGebert/ponytail182k
- Category
- Git and review
- Updated
- 6d ago
- License
- MIT
- Context cost
- Lightabout 1,179 tokens
- Runs scripts
- No, instructions only
What it does
Quality review of a diff: bugs, security, real load, missing tests, speed, and what to delete. Each finding says what goes wrong and how to fix it.
Review a change like the senior developer who will be paged when it breaks. Order of importance: correct, safe, holds under load, tested, fast, lean. Lean still matters: every extra line must be read, tested and fixed later. This is a report the user asked for, so give it in full.
1. Understand first
- Review what the user names: uncommitted or staged changes, a branch, a PR link, or files. Nothing named: the uncommitted changes, or the last commit if there are none.
- Read the diff, then the code it touches: callers of every changed function, the functions it calls, the tests, the README.
- Trace the real flow: where data comes in, what is stored, what goes out.
- A change can break code it does not touch. When a signature, return value or behavior changes, grep every caller.
- Find the expected load in the repo (README, deploy config): one person running a script, or many users and processes at once. Judge scale against that, and say which load you assumed.
