← ClaudeAtlas

readability-sweeplisted

Three-pass readability cleanup of application source: remove inline comments, extract cohesive blocks into intent-named private methods, and challenge every class and method name. Use whenever the user says "readability sweep", "no inline comments", "remove comments", "check naming", "make the code readable", "extract private methods", or asks to scan the project for readability violations. Edits allowed. Scope is application source only (not tests/config) unless told otherwise. Runs lint and the full test suite after changes.
digitaldreams/tuhin · ★ 0 · Code & Development · score 70
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