legion-simplifylisted
Install: claude install-skill runlegion/legion
# Legion Simplify
Review the git diff on this branch for structural quality issues, and write down -- per
changed file -- what you checked and the reasoning behind your verdict. `legion quality-gate
check` refuses the gate unless every changed file is addressed with real prose, then records
it so `legion pr create` can gate on it.
This is a forcing function, not a checkbox. The old version of this skill let you record
`"clean"` without reading the diff, and the corpus proved it: simplify caught structural
issues 0.7% of the time while `pr write-check` -- the same self-review, but with validated
prose -- caught 24%. You cannot clear this gate by typing "clean". You clear it by showing,
file by file, the reasoning that says clean.
## What to Review
Inspect `git diff main...HEAD` (or `git diff origin/main...HEAD`). Use the three-dot form
(`...`) -- it diffs against the merge base, so files changed only on main since your branch
point do not appear in your coverage list. The validator uses the same three-dot range. For each changed file, look for:
- **Duplicate logic** -- same transformation or check written twice; extract a shared helper
- **Unnecessary abstraction** -- a wrapper or trait that adds indirection without buying
extensibility; flatten it
- **Stringly-typed state** -- `status: String` where `"pending" | "done"` would be an enum
- **Hand-rolled helpers** -- functions duplicating something already in `std` or a module helper
- **Unnecessary new parameters** -- a