Skip to main content

clean: Review whether code names express their purpose

Review variable, boolean, function and type names for clarity and consistency. The criteria favor names that reveal intent, predicate names such as isActive for booleans, and function names that describe what they do.

Source facts

Repository
devinat1/engineering-skills
Last source activity
September 21, 2026 at 18:11
Detected SKILL.md language
English
Stars
0
Forks
0

How to use

Invoke /clean or ask for a code naming review. The source checks abbreviations, single-letter names outside tiny loops, and consistency with the codebase’s naming conventions.

Install options

The review-first prompt is selected by default. You can switch to a direct command or download a local copy.

Review the source files

Read SKILL.md and any companion files shown by SkillsMP before deciding whether to install.

Showing SKILL.md

SKILL.md
Source instructions · Read-only preview
name
clean
description
Review code for clean naming conventions — descriptive, intention-revealing names. Use when the user invokes /clean or asks for a clean code naming review.
disable-model-invocation
true
## Automatic Jev check During an actual `/clean` review, after proposing a naming finding against a bounded changed-code slice, read `${AGENTIC_HOME:-$HOME/.agentic}/artifacts/jev/PROTOCOL.md` and run its helper. Send only stable source IDs, the relevant declaration/use lines, and the proposed finding. Ask one **Noul** per finding: `Does this evidence show that the proposed name fails the stated clean-naming criterion?` True means the cited evidence supports that exact failure; false means it does not; unavailable context is not false. Reconcile the advisory answer with the evidence. On unavailable or ambiguous evidence, keep the ordinary review; do not invoke this check when these criteria are merely borrowed by another skill. It never suppresses an evidence-backed finding or authorizes changes. # Clean Naming - Use descriptive, intention-revealing names. A variable name should tell you why it exists, what it does, and how it's used. - Avoid abbreviations and single-letter names outside of tiny loop scopes. - Use consistent naming conventions across the codebase (e.g., `camelCase` for variables, `PascalCase` for types/classes). - Name booleans as predicates: `isActive`, `hasPermission`, `shouldRetry`. - Name functions after what they do, not how they do it.
View on GitHub