CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

xp-simple-design-refactor

Refactor toward simple design and maintainability with ROI-driven prioritization. Use when the user asks to refactor, improve simple design, maintainability, or reduce accidental complexity.

Quality

76%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./.agents/skills/xp-simple-design-refactor/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

65%Weight 40%Scale 1-5

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 process skill that clearly communicates the XP simple-design approach and a sensible analyze-prioritize-implement-explain sequence. Its gaps are in executability: no examples or named techniques make the guidance abstract, and verification is implicit — the test safety net is invoked but never turned into an explicit run-the-tests checkpoint.

Suggestions

Add an explicit verification checkpoint to the workflow, e.g., "Run the test suite after the refactor and only keep the change when all tests pass" — this turns the mentioned test safety net into an actual validate step and closes the workflow_clarity gap.

Include one compact worked example — a short before/after code snippet or a sample ROI ranking of 2-3 candidate refactorings — so the analysis and prioritization steps are concrete rather than abstract.

Name specific refactoring moves to reach for (e.g., extract method, inline variable, collapse duplicate conditionals) in step 3 so implementation has concrete techniques instead of only principles.

DimensionReasoningScore

Conciseness

The ~34-line body is lean with no padded explanations of concepts Claude lacks, earning near the top anchor, but there is minor redundancy: the four XP simple-design rules are restated knowledge, and "Strict adherence to simple design principles" under step 3 repeats the section header. It is not a 5 because those tokens could be trimmed, and not a 3 because the body is otherwise efficient throughout.

4 / 5

Actionability

Concrete elements exist (ROI factors: impact, frequency of change, risk, feasibility; "tiny, reversible changes"; "add or refine tests if needed"), but the guidance stops at process description: no worked example, before/after code, or named refactoring techniques, leaving key execution details missing — the anchor-3 signature of 'some concrete guidance but incomplete'. It is not a 4 because an instruction-only skill still needs specific, example-backed guidance to be mostly executable, and not a 2 because the steps and selection criteria are explicitly enumerated.

3 / 5

Workflow Clarity

The four-step sequence (analyze, prioritize, implement, explain) is clearly ordered with sub-criteria, but validation checkpoints are only implicit: the "strong test safety net" is listed as an implementation input, and the workflow never says to run the tests to confirm green after the refactor, matching 'steps listed but validation gaps; checkpoints missing or implicit'. It is not a 4 because the critical verify step for refactoring work is absent rather than a minor gap, and not a 2 because the sequence is well defined, not rough or gapped.

3 / 5

Progressive Disclosure

The skill is under 50 lines, single-purpose, and organized into clear sections (principles, then a numbered task with sub-bullets); no bundle files exist and the body references none, so there are no dangling or nested references. Per the rubric's simple-skill note, this earns the top anchor.

5 / 5

Total

15

/

20

Passed

Description

75%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A well-formed description that explicitly states what the skill does and when to use it, with an explicit 'Use when...' clause and several natural trigger phrases. Its weaknesses are moderate: the capability list stays abstract (no concrete actions beyond 'refactor... with ROI-driven prioritization') and the broad 'refactor' trigger risks overlap with general refactoring skills.

DimensionReasoningScore

Specificity

The description names the domain ("Refactor toward simple design and maintainability") with 1-2 concrete actions ("ROI-driven prioritization", "reduce accidental complexity") but does not enumerate several distinct specific actions, matching the 'names domain and 1-2 concrete actions' anchor. It is not a 4 because there is no list of specific capabilities, and not a 2 because it goes beyond a bare domain name.

3 / 5

Completeness

It explicitly answers both questions: the what is "Refactor toward simple design and maintainability with ROI-driven prioritization" and the when is a full "Use when the user asks to refactor, improve simple design, maintainability, or reduce accidental complexity" clause listing concrete trigger phrases, matching the 5 anchor. It is not a 4 because the 'when' clause is already explicit and specific, not merely present-but-imprecise.

5 / 5

Trigger Term Quality

Good natural keyword coverage: "refactor", "improve simple design", "maintainability", and "reduce accidental complexity" are phrases users would plausibly say. It falls short of the 5 anchor because common variations like "simplify the code", "clean up code", or "reduce complexity" are missing, and it is clearly above the 3 anchor's 'some relevant keywords but missing common variations' since multiple natural trigger phrases are present.

4 / 5

Distinctiveness Conflict Risk

The XP framing (simple design, accidental complexity, ROI) carves a niche, but the trigger "Use when the user asks to refactor" is broad and would fire for nearly any general refactoring or code-quality skill, fitting the 'somewhat specific but could still overlap with similar skills' anchor. It is not a 4 because the generic 'refactor'/'maintainability' triggers create real overlap risk with closely related skills, and not a 2 because the simple-design framing meaningfully narrows the scope.

3 / 5

Total

15

/

20

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 — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

saski/arnesto

Path

.agents/skills/xp-simple-design-refactor/SKILL.md

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.