← ClaudeAtlas

legion-simplifylisted

Review the current branch diff for code quality issues: duplicate logic, unnecessary abstraction, stringly-typed state, and other structural problems. Articulate, per changed file, what you checked and why the verdict holds; `legion quality-gate check` validates the articulation (coverage + non-boilerplate + located evidence) and records the gate so `legion pr create` can verify clean state before opening a PR. Run this on every feature branch before a PR.
runlegion/legion · ★ 3 · AI & Automation · score 75
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