readability-sweeplisted
Install: claude install-skill digitaldreams/tuhin
# Readability Sweep — Three Passes
Code must read through its names, not through narration. Run the passes in order, report per pass. Scope: application source directories only — leave tests, config, and vendor code alone unless explicitly included.
## Pass A — Comments
1. Find every inline `//` and `/* */` comment in scope.
2. Apply per comment:
- **What-comment** (restates the next line) → delete.
- **Genuine why** (race condition, idempotency gate, security rationale, external quirk, legal constraint) → lift into the enclosing **method PHPDoc/docstring**, then delete the inline form.
- **Commented-out code** → delete outright.
- **Bare `//` placeholders and trailing data annotations** → delete; move rationale to class doc if it matters.
3. Class-level doc blocks always stay. Method-level docs only where they carry a why or a non-obvious contract.
## Pass B — Extract Method
1. Inside each method, find self-contained blocks of **4+ lines** that take input → transform → produce one output.
2. Extract each into a `private` method with an **intent name** — the name states what comes out, not how (`priorComposites`, `buildRequestLine`, `advanceStatus`).
3. **Do NOT extract** — over-extraction reads worse than inline code:
- guard clauses and early returns
- single statements or simple assignments
- blocks mutating two or more enclosing-scope locals
4. Keep paired classes symmetric: if two sibling drivers/paths share a shape, give both the same extractio