CtrlK
BlogDocsLog inGet started
Tessl Logo

agent-reviewer

Agent skill for reviewer - invoke with $agent-reviewer

51

1.15x
Quality

31%

Does it follow best practices?

Impact

81%

1.15x

Average score across 3 eval scenarios

SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./.agents/skills/agent-reviewer/SKILL.md
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 body is well-sectioned and includes a useful feedback-format template, but it is bloated with generic code-review tutorials Claude already knows and its MCP integration examples are non-executable pseudocode. A stray duplicate YAML frontmatter block at the top of the body is a formatting defect.

Suggestions

Delete the generic educational examples (SOLID/DRY/naming illustrations, SQL injection and N+1 demos) — Claude already knows these patterns; keep only skill-specific guidance such as the review checklist items and the report format.

Make the MCP integration examples executable: show the actual tool-call format for memory_usage (action, key, namespace, value) as it would really be invoked, or replace with a working shell/HTTP example.

Remove the stray second YAML frontmatter block at the top of the body (name/type/hooks), which is a duplicate agent manifest leaking into the markdown content.

DimensionReasoningScore

Conciseness

The body pads roughly 200 lines with generic bad-example/good-example illustrations of SQL injection, N+1 queries, SOLID, DRY, and naming conventions — knowledge Claude already has. This matches 'Noticeably verbose; several unnecessary explanations or padded sections'; it avoids a 1 only because it is not written as tutorial prose explaining what code review is.

2 / 5

Actionability

There is some genuinely concrete guidance (the review feedback format template and 'npm run lint / test / security-scan' commands), but the MCP blocks are pseudocode ('mcp__claude-flow__memory_usage { ... }' is not executable syntax) and the TypeScript examples call undefined functions. This matches 'Some concrete guidance but incomplete; pseudocode instead of executable code; missing key details'.

3 / 5

Workflow Clarity

The five review areas are presented in a defined order and 'Run automated tools before manual review' establishes a sequence, but there are no validation checkpoints or feedback loops (e.g. verify automated checks pass before writing the summary). This matches 'Steps listed but validation gaps; sequence present but checkpoints missing or implicit'.

3 / 5

Progressive Disclosure

The body has clear section headers and no broken or nested references (no bundle files exist to verify), but everything — including ~200 lines of generic example content that belongs in a reference file or should be deleted — is inlined in one ~330-line SKILL.md. This matches 'Some structure but could be better organized; content that should be separate is inline'.

3 / 5

Total

11

/

20

Passed

Description

20%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 a placeholder-quality label: it identifies a domain and an invocation token but states no capabilities, no use-when triggers, and nothing that distinguishes it from other review skills. It would rarely be selected correctly by trigger-based routing.

Suggestions

State the concrete actions the skill performs, e.g. 'Reviews code for security vulnerabilities, performance bottlenecks, and maintainability; produces a structured review report with prioritized issues and action items.'

Add an explicit 'Use when...' trigger clause with natural user phrasing, e.g. 'Use when the user asks to review code, audit security, critique a PR, or assess code quality.'

Remove the invocation directive ('$agent-reviewer') in favor of user-facing trigger terms like 'code review', 'security audit', and 'PR feedback' to improve trigger-term quality and distinctiveness.

DimensionReasoningScore

Specificity

The description 'Agent skill for reviewer - invoke with $agent-reviewer' contains no concrete actions whatsoever; it only labels the domain ('reviewer') and gives an invocation directive. It matches the anchor 'Entirely vague; no concrete actions; pure abstract language' and falls short of score 2, whose example ('Processes PDF files') at least states a generic action.

1 / 5

Completeness

There is only a vague 'what' ('Agent skill for reviewer') and no 'when should Claude use it' clause at all, which is the exact score-2 anchor ('Has a vague what and no when'). It cannot be a 3 because a 3 requires a clear 'what', and 'agent skill for reviewer' does not state what the skill actually does.

2 / 5

Trigger Term Quality

It offers one generic keyword ('reviewer') plus the invocation token '$agent-reviewer', but omits the natural phrases a user would actually say ('code review', 'review my PR', 'security audit', 'review my changes'). This matches 'One or two generic keywords; missing the natural phrases users say'; it is not a 1 because 'reviewer' is at least a relevant domain term rather than pure jargon.

2 / 5

Distinctiveness Conflict Risk

'Reviewer' is a very broad label with high overlap risk against any code-review, security-review, or QA skill, matching the score-2 anchor ('Very broad; high overlap risk with many similar skills'). It is above a 1 only because it is not so generic as 'helps with code and documents'.

2 / 5

Total

7

/

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
ruvnet/ruflo
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.