CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-refactoring-refactor-clean

You are a code refactoring expert specializing in clean code principles, SOLID design patterns, and modern software engineering best practices. Analyze and refactor the provided code to improve its quality, maintainability, and performance.

Quality

54%

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 ./plugins/AI-Agents-Safe-Coding-Skills/skills/code-refactoring-refactor-clean/SKILL.md

The canonical home for this skill is code-refactoring-refactor-clean in rmyndharis/antigravity-skills

SKILL.md
Quality
Evals
Security

Quality

Content

42%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.

The skill body is well-sectioned and concise in length, but its guidance stays at the directive level with all concrete detail delegated to a playbook file that is missing from the bundle. Fixing the dangling reference (or inlining the detail) and adding explicit validation checkpoints are the highest-impact improvements.

Suggestions

Fix the dangling reference: either bundle `resources/implementation-playbook.md` with the skill or remove both mentions of it and inline the key patterns (e.g., specific refactoring techniques, how to slice changes safely) directly in the Instructions.

Add an explicit validation checkpoint and feedback loop to the workflow, e.g., "After each slice, run the test suite; if failures appear, revert the slice and re-plan before proceeding."

Remove the verbatim repetition of the frontmatter description and the redundant "Context" section, and merge the Safety lines with the related Instructions to eliminate duplicated guidance.

DimensionReasoningScore

Conciseness

The body is short and mostly efficient, but it repeats the frontmatter description verbatim as an opening persona paragraph, and the "Context" section ("The user needs help refactoring code to make it cleaner...") plus the Safety line "Keep diffs reviewable and ensure tests pass" largely restate what the Instructions already say. This is unnecessary repetition that could be tightened, matching the "mostly efficient but some unnecessary explanation" anchor rather than the lean anchor.

3 / 5

Actionability

The Instructions are high-level directives ("Assess code smells, dependencies, and risky hotspots", "Apply changes in small slices and keep behavior stable") with no commands, techniques, code examples, or concrete criteria for any step. The specifics are deferred to `resources/implementation-playbook.md`, but that file does not exist in the bundle, so the executable detail the skill promises is unavailable — matching the "minimal concrete guidance; high-level hints" anchor.

2 / 5

Workflow Clarity

A rough sequence is present (assess → propose plan → apply in small slices → update tests and verify regressions), so it is above the missing-or-incoherent anchors. However, validation is implicit — there is no checkpoint structure (e.g., verify behavior after each slice) and no feedback loop saying what to do when tests fail or behavior changes, matching the "sequence present but checkpoints missing or implicit" anchor.

3 / 5

Progressive Disclosure

The body is under 50 lines with well-organized sections, which would normally score well under the simple-skill note, but it twice references `resources/implementation-playbook.md` and no such file (or resources directory) exists in the bundle. Scored against the actual bundle structure, this is a dangling reference that breaks navigation, so it cannot reach the well-signaled one-level-deep anchors (4-5).

3 / 5

Total

11

/

20

Passed

Description

45%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.

The description communicates a clear domain and purpose but is written in second person, padded with buzzwords, and entirely missing a "Use when..." trigger clause, which caps completeness and weakens trigger-term quality. Rewriting in third person with explicit trigger phrases would lift most dimensions.

Suggestions

Rewrite in third person voice (e.g., "Refactors code by applying clean code principles and SOLID design patterns...") to avoid the specificity penalty and remove the buzzword phrase "modern software engineering best practices".

Append an explicit trigger clause such as "Use when the user asks to refactor, clean up, simplify, or reduce duplication in code, or mentions code smells or hard-to-maintain modules."

Add the natural synonyms users actually say ("clean up code", "reduce duplication", "code smells", "improve readability") to improve trigger-term coverage and distinctiveness.

DimensionReasoningScore

Specificity

The description names the domain and two actions ("Analyze and refactor the provided code to improve its quality, maintainability, and performance") which fits anchor 3, but it opens in second person ("You are a code refactoring expert") and the judging guidelines mandate a 1-point specificity penalty for non-third-person voice, landing it at 2. The buzzword phrase "modern software engineering best practices" further dilutes concreteness.

2 / 5

Completeness

It clearly answers "what" (analyze and refactor code to improve quality, maintainability, performance) but contains no "Use when..." clause or equivalent trigger guidance, so per the judging guidelines completeness is capped at 3. It is not a 2 because the "what" is explicit rather than vague.

3 / 5

Trigger Term Quality

It includes some natural keywords users would say ("code refactoring", "clean code", "SOLID", "design patterns") but misses common variations and synonyms such as "clean up code", "reduce duplication", "code smells", or "improve readability". This matches the anchor for relevant keywords missing common variations, not the good-coverage anchor above it.

3 / 5

Distinctiveness Conflict Risk

The refactoring/clean-code niche is somewhat specific but the description's broad goals ("quality, maintainability, and performance") and generic expertise framing overlap heavily with code-review, simplify, and general coding-assistance skills. It lacks the distinct trigger phrases needed for the mostly-distinct anchors (4-5).

3 / 5

Total

11

/

20

Passed

Validation

93%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

Total

15

/

16

Passed

Repository

administrakt0r/AI-Agents-Safe-Coding-Skills

Path

plugins/AI-Agents-Safe-Coding-Skills/skills/code-refactoring-refactor-clean/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.