CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

refactor-safely

Plan and execute safe refactoring using dependency analysis

Quality

73%

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 ./skills/refactor-safely/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

78%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 body is concise, concrete, and well-structured with tool-call-level actionability and validation checkpoints for a destructive operation. The main gap is the absence of an explicit fix-and-re-validate feedback loop after detect_changes_tool, which would lift workflow clarity to the top anchor.

Suggestions

Add an explicit feedback loop after step 6: if detect_changes_tool shows impact beyond the plan, revert or adjust and re-run get_impact_radius_tool before re-applying.

Provide one short worked example (e.g. a rename flow) showing the suggest -> preview edit list -> apply -> detect_changes sequence end-to-end.

Trim 'The graph narrows scope; it does not replace the source.' or fold it into the budget line to reach a fully lean conciseness score.

DimensionReasoningScore

Conciseness

The body is lean and tool-call driven with little padding; the only mild over-explanation is 'The graph narrows scope; it does not replace the source.', placing it just below the fully lean score-5 anchor at 'efficient with minor trimmable instances'.

4 / 5

Actionability

It specifies concrete tool invocations with modes and parameters (mode="suggest", mode="rename" with old_name/new_name, apply_refactor_tool with refactor_id, detail_level="minimal"), giving mostly executable guidance; it lacks a fully worked example sequence, keeping it just under 5.

4 / 5

Workflow Clarity

Steps 1-6 are clearly sequenced and include validation checkpoints ('Check the returned edit list before applying' and step 6's detect_changes_tool to confirm impact matches the plan), satisfying the destructive-operation validation rule; the missing explicit fix->re-validate loop keeps it at 4 rather than 5.

4 / 5

Progressive Disclosure

This is a simple, short (~25 lines) single-purpose skill with no external references needed; it is organized into clear sections (Steps, Token Efficiency Rules), which per the simple-skills note merits a 5.

5 / 5

Total

17

/

20

Passed

Description

53%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 is concise and names a clear, distinct capability, but it lacks an explicit 'Use when...' trigger clause and only lightly enumerates actions, capping completeness and trigger quality at the midpoint. Adding concrete trigger phrases and one or two more specific refactor operations would lift it toward the top anchors.

Suggestions

Add an explicit 'Use when...' clause naming natural triggers, e.g. 'Use when the user asks to refactor, rename, or remove code safely, or to find dead code before changes.'

Expand the action list with concrete operations such as 'suggest refactors, find dead code, rename symbols, and preview impact' to improve specificity and trigger coverage.

Include common synonyms users say ('refactor', 'safe rename', 'dead code') to broaden trigger term quality.

DimensionReasoningScore

Specificity

Names the domain ('refactoring') and two concrete actions ('Plan and execute') grounded in 'dependency analysis', but stops short of enumerating multiple specific refactor operations, matching the score-3 anchor of domain plus 1-2 actions.

3 / 5

Completeness

It gives a clear 'what' (plan and execute safe refactoring) but has no 'Use when...' or equivalent trigger clause, so per the judging guideline completeness is capped at 3 with 'when' only weakly implied.

3 / 5

Trigger Term Quality

'refactoring' and 'dependency analysis' are natural terms a user might say, but common synonyms ('refactor', 'rename', 'safe rename') and variations are absent, fitting the 'some relevant keywords but missing variations' anchor.

3 / 5

Distinctiveness Conflict Risk

'safe refactoring using dependency analysis' carves a fairly distinct niche with low overlap risk against generic code skills; it is not quite a comprehensive trigger set, so it sits at 'mostly distinct' rather than 5.

4 / 5

Total

13

/

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

tirth8205/code-review-graph

Path

skills/refactor-safely/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.