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

86%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 lean, highly actionable skill body: a clear find-fix-validate-report workflow with a copy-paste validation command, a prioritized remedy per violation, and sensible delegation to companion skills. The only notable gap is the absence of an explicit failure-handling loop after validation and a couple of underspecified remedies.

DimensionReasoningScore

Conciseness

Every line is a terse, actionable directive ('try/catch in services -> remove, move error handling to component/interceptor', 'Console.log -> remove') with zero padding and no explanation of concepts Claude already knows, matching the 'lean and efficient; every token earns its place' anchor.

5 / 5

Actionability

The body provides a copy-paste validation command ('npx tsc --noEmit && ng build && ng test --watch=false'), a specific remedy for each violation type, and delegates complex migrations to named companion skills ('/migration-migrate-component'). It stops short of anchor 5 because a few remedies are thin (e.g. 'any types -> type correctly') and no file-discovery guidance is given for the module.

4 / 5

Workflow Clarity

The four-step sequence (find violations -> fix in priority order -> validate after each fix -> report) is clear with explicit validation checkpoints, matching anchor 4. It is not anchor 5 because there is no explicit error-recovery feedback loop for what to do when the validation command fails.

4 / 5

Progressive Disclosure

The skill is under 50 lines with no need for external reference files (none exist in the bundle), and its content is well-organized into a Steps section with clearly labeled Critical/Important/Improvements priority tiers — per the rubric's simple-skill guideline this earns a 5.

5 / 5

Total

18

/

20

Passed

Description

70%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 cleanly states both what the skill does and when to use it, with a reasonably distinct trigger phrase. Its main weakness is thin capability coverage — a single action ('automated fixing by priority') with no mention of the concrete domain (Angular, ARCHITECTURE.md) or the fix categories involved.

Suggestions

Add 1-2 more concrete actions or scope details, e.g. 'Fixes Angular architecture violations (error handling placement, standalone migration, signals, OnPush) in priority order' to lift specificity.

Include natural trigger variations and the governing file, e.g. 'Use when architecture violations or architectural issues are found in docs/ARCHITECTURE.md and need automated fixing by priority.'

Mention the module-scope behavior (e.g. 'in a specified module') so the 'when' clause is more explicit about invocation context.

DimensionReasoningScore

Specificity

The description names the domain ('architecture violations') and one concrete action ('automated fixing' with a 'by priority' mechanism), which matches the anchor for naming a domain with 1-2 concrete actions but not comprehensive coverage; it does not reach anchor 4 because only a single action is stated.

3 / 5

Completeness

Both parts are explicit: what ('need automated fixing by priority') and when ('Use when architecture violations were found'). It matches anchor 4 rather than 5 because the 'when' clause could be more specific about the module or project context.

4 / 5

Trigger Term Quality

'architecture violations', 'found', and 'fixing' are natural phrases a user would say when needing this skill, giving good keyword coverage; it falls short of anchor 5 because synonyms like 'architectural issues', 'fix violations', or 'ARCHITECTURE.md' are missing.

4 / 5

Distinctiveness Conflict Risk

'architecture violations' is a distinct niche trigger that clearly separates this from unrelated skills, with only minor overlap risk against closely related lint-fix or refactor skills; it does not reach 5 because the description omits the concrete domain (Angular, docs/ARCHITECTURE.md) that would make it fully unambiguous.

4 / 5

Total

15

/

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/angular/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.