CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

refactor

Refactoring process with test safety. Invoke immediately when user or document mentions refactoring, or proactively when code gets too complex or messy.

Quality

75%

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 ./.tessl/tiles/lightfactory/crafter/skills/refactor/SKILL.md

The canonical home for this skill is refactor in ericjohnolson/tessl-test

SKILL.md
Quality
Evals
Security

Quality

Content

82%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 well-structured, lean process skill: clear staged workflow with per-refactor test checkpoints, concrete commit and summary templates, and no wasted tokens. The main gaps are minor — no failure-recovery guidance for when tests fail after a change, a few abstract directives, and an unlinked references/credits.md in the bundle.

DimensionReasoningScore

Conciseness

The body is lean and imperative throughout — staged sections, exact file paths, commit formats, and summary templates, with no explanation of concepts Claude already knows. Every token earns its place, matching anchor 5 ("Lean and efficient; assumes Claude's competence"); the only near-redundancy ("Never change test assertions...") is deliberate emphasis of a critical safety rule.

5 / 5

Actionability

Guidance is highly concrete — "Find or create ./test.sh", the exact commit format "- r <refactoring>", and copy-paste-ready REFACTORING_SUMMARY.md templates. It falls at anchor 4 rather than 5 because a few directives are abstract ("Look for opportunities to simplify") and no worked example of a refactoring step is given.

4 / 5

Workflow Clarity

A clear 4-stage sequence with test-before/test-after checkpoints around every refactor and per-step commits, matching anchor 4. It misses anchor 5 because there is no error-recovery loop: nothing says what to do when tests fail after a change (revert it, amend or undo the commit?).

4 / 5

Progressive Disclosure

Sections are well-organized and all process content belongs in SKILL.md, matching anchor 4 ("Good structure; most content is appropriately placed... minor organization gaps"). The gap: the bundle contains references/credits.md, which is never linked or signaled from the body, and the ~95-line body does not qualify for the under-50-line simple-skill exemption to score 5 on sections alone.

4 / 5

Total

17

/

20

Passed

Description

57%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 answers both 'what' and 'when' with explicit trigger guidance, but the 'what' is generic: it never states what the refactoring process actually does (e.g., simplifies code, renames, extracts methods) beyond naming the domain. Trigger keyword coverage within the description itself is limited to essentially one term plus a condition clause, while natural variants sit unused in the triggers field.

Suggestions

Enumerate 2-3 concrete actions in the description, e.g., 'Simplifies code, extracts methods, renames for clarity, and removes dead code, with tests verified before and after every step.'

Fold natural trigger phrases from the triggers field into the description itself, e.g., 'Use when the user asks to refactor, clean up code, or improve messy or overly complex code.'

Tighten the proactive clause to a more distinctive condition (e.g., 'proactively when a file has grown too complex to follow') to reduce overlap with general code-editing requests.

DimensionReasoningScore

Specificity

"Refactoring process with test safety" names the domain but lists no concrete actions — no equivalents of 'rename methods, extract functions, remove dead code'. This matches anchor 2 ("Names the domain but actions are minimal or generic"); it falls short of anchor 3 because not even 1-2 concrete actions are enumerated.

2 / 5

Completeness

Both parts are present: the what ("Refactoring process with test safety") and an explicit when ("Invoke immediately when user or document mentions refactoring, or proactively when code gets too complex or messy"). It sits at anchor 4 rather than 5 because the 'what' is thin — it characterizes the process without saying what it actually does to the code.

4 / 5

Trigger Term Quality

"refactoring" and "code gets too complex or messy" are natural phrasings, but common variations like "clean up code", "tidy", or "improve readability" appear only in the separate triggers field, not in the description itself. This matches anchor 3 ("Some relevant keywords but missing common variations or synonyms") rather than anchor 4's fuller keyword coverage.

3 / 5

Distinctiveness Conflict Risk

Refactoring is a distinct niche with its own trigger term, but "proactively when code gets too complex or messy" could fire on nearly any messy-code request, giving minor overlap with general coding skills. Anchor 4 ("Mostly distinct; minor overlap risk with closely related skills") is the best fit — better than 3 since the core trigger is unambiguous, short of 5 because the proactive clause is broad.

4 / 5

Total

13

/

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

ericjohnolson/tessl-test

Path

.tessl/tiles/lightfactory/crafter/skills/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.