refactor-pass
Perform a refactor pass focused on simplicity after recent changes. Use when the user asks for a refactor/cleanup pass, simplification, or dead-code removal and expects build/tests to verify behavior.
85
77%
Does it follow best practices?
Impact
97%
1.00xAverage score across 3 eval scenarios
Passed
No known issues
Optimize this skill with Tessl
npx tessl skill review --optimize ./.claude/skills/refactor-pass/SKILL.mdQuality
Discovery
82%Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.
This is a solid description that clearly communicates both what the skill does and when to use it, with good natural trigger terms. Its main weaknesses are moderate specificity in the concrete actions performed and some potential overlap with other code quality or refactoring skills. Adding more specific actions (e.g., extracting functions, simplifying conditionals) would strengthen it.
Suggestions
Add more specific concrete actions like 'extract helper functions, simplify conditionals, remove unused imports and variables, reduce nesting depth' to improve specificity.
Differentiate more clearly from general code review or linting skills by emphasizing the post-change simplification focus, e.g., 'after implementing features or fixing bugs'.
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | Names the domain (refactoring) and some actions (simplification, dead-code removal, build/test verification), but doesn't list multiple concrete specific actions like 'extract methods, inline variables, remove unused imports'. The actions remain somewhat high-level. | 2 / 3 |
Completeness | Clearly answers both 'what' (perform a refactor pass focused on simplicity, verify with build/tests) and 'when' (explicit 'Use when' clause covering refactor/cleanup pass, simplification, dead-code removal, and build/test verification expectations). | 3 / 3 |
Trigger Term Quality | Includes natural keywords users would say: 'refactor', 'cleanup pass', 'simplification', 'dead-code removal'. These are terms developers naturally use when requesting this kind of work. | 3 / 3 |
Distinctiveness Conflict Risk | While 'refactor pass focused on simplicity' is somewhat specific, it could overlap with general code review skills, linting skills, or other refactoring-focused skills. The 'after recent changes' qualifier helps but the scope is still broad enough to potentially conflict. | 2 / 3 |
Total | 10 / 12 Passed |
Implementation
72%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A concise, well-structured skill that clearly communicates the refactoring workflow at a high level. Its main weakness is the lack of concrete, actionable guidance—specific examples of refactoring patterns, commands to run, or before/after code snippets would make it significantly more useful. The workflow also lacks an explicit feedback loop for test failures.
Suggestions
Add concrete before/after code examples for at least one refactoring pattern (e.g., dead code removal or logic straightening) to improve actionability.
Add an explicit feedback loop: if build/tests fail after refactoring, revert the last change and retry with a smaller refactor step.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Very lean and efficient. Every line serves a purpose, no unnecessary explanations of what refactoring is or why it matters. Assumes Claude's competence. | 3 / 3 |
Actionability | Provides a clear checklist of what to do but lacks concrete examples—no specific code patterns showing before/after refactors, no specific commands for running builds/tests, and no guidance on how to identify dead code or straighten logic flows. | 2 / 3 |
Workflow Clarity | Steps are clearly sequenced and include a build/test verification step, but there's no explicit feedback loop for what to do if tests fail after refactoring, and no validation checkpoint between individual refactor steps to catch regressions early. | 2 / 3 |
Progressive Disclosure | For a simple, single-purpose skill under 50 lines, the content is well-organized with a clear workflow section. No external references are needed given the scope. | 3 / 3 |
Total | 10 / 12 Passed |
Validation
100%Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.
Validation — 11 / 11 Passed
Validation for skill structure
No warnings or errors.
- Repository
- pgplex/pgconsole
- Commit
2b49a9b
- Reviewed
Table of Contents
Is this your skill?
If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.