CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-fix-violations

Use when architecture violations were found and need automated fixing by priority.

Quality

81%

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

SKILL.md
Quality
Evals
Security

Quality

Content

93%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 an exemplary lean, actionable fix-by-priority playbook: concrete pattern-to-action rules, a real validation command run after each fix, and clean structure. The only gap is the absence of an explicit error-recovery loop when validation fails.

DimensionReasoningScore

Conciseness

The body is a lean imperative rule list ("try/catch in services -> remove, move error handling to page frontmatter or endpoint"; "Raw <img> tags -> use <Image /> from astro:assets") with no explanation of concepts Claude already knows; every token earns its place.

5 / 5

Actionability

Fully executable guidance: a copy-paste-ready validation command ("npx astro check && npx astro build && npx vitest run --passWithNoTests") and specific pattern-to-action mappings covering the common violation cases; no pseudocode or missing key details.

5 / 5

Workflow Clarity

Clear 4-step sequence with an explicit per-fix validation checkpoint and concrete command, so it clears the 3 anchor (validation is present, not missing, for this batch-edit skill). It falls short of 5 only because the error-recovery feedback loop (what to do when validation fails) is implicit rather than an explicit "if errors: fix and re-validate" step.

4 / 5

Progressive Disclosure

The skill is under 50 lines with no external references needed, and the content is well organized (intro, module argument, numbered steps with bolded priority tiers); per the rubric's simple-skill guideline this earns a 5, and no content that belongs in a separate file is inlined.

5 / 5

Total

19

/

20

Passed

Description

62%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 has an explicit 'Use when' trigger and a clear niche, but it is thin: only one action verb and one trigger phrase, with the when-clause circularly restating the what. It is serviceable but below the bar set by the reference good examples.

Suggestions

List 2-3 concrete fix actions in the description, e.g., "Fixes architecture violations by priority: removes try/catch from services, converts islands to .astro components, replaces raw <img> with <Image />".

Broaden trigger coverage with natural variations, e.g., "Use when architecture violations were found (e.g., by /review-check-architecture) and need automated fixing by priority, or when the user asks to fix arch violations or clean up architecture issues".

Make the 'when' clause additive rather than circular by tying it to a concrete trigger context, such as "Use after running /review-check-architecture or when the user mentions architecture violations in a module".

DimensionReasoningScore

Specificity

Names the domain ("architecture violations") and one concrete action ("automated fixing by priority"), but does not list multiple specific actions like the 5-anchor example; it matches "names domain and 1-2 concrete actions, but not comprehensive".

3 / 5

Completeness

Both 'what' ("automated fixing by priority" of architecture violations) and 'when' (explicit "Use when..." clause) are present, but the when-clause merely restates the what rather than adding concrete trigger contexts, so it falls short of the 5 anchor's explicit trigger phrases.

4 / 5

Trigger Term Quality

"architecture violations" is a relevant keyword users of this workflow would say, but the description contains only this single trigger phrase with no synonyms or variations (e.g., "arch violations", "fix architecture issues"), matching the 3 anchor.

3 / 5

Distinctiveness Conflict Risk

The niche (fixing architecture violations by priority) is mostly distinct from generic code-fixing skills, with only minor overlap risk against the closely related companion review skill; it lacks the fully distinct trigger phrases of the 5 anchor.

4 / 5

Total

14

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

frontmatter_unknown_keys

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

Warning

Total

14

/

16

Passed

Repository

fernandezbaptiste/specialist-agent

Path

packs/astro/skills/review-fix-violations/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.