CtrlK
BlogDocsLog inGet started
Tessl Logo

code-review-expert

Expert code review of current git changes with a senior engineer lens. Detects SOLID violations, security risks, and proposes actionable improvements.

62

Quality

72%

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 ./frontend/magic-web/.agents/skills/code-review-expert/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 well-structured, lean, and actionable with a clear sequenced workflow and excellent progressive disclosure via one-level-deep reference files. The main gaps are minor trim opportunities and the absence of a validate/retry feedback loop.

DimensionReasoningScore

Conciseness

The body is efficient and assumes Claude's competence — it does not explain what SOLID, XSS, or SSRF are, and uses a compact severity table. It is not a 5 because a few instructional lines ('explain why it improves cohesion/coupling') and the inline-comment format could be trimmed, and not a 3 because padding is minimal.

4 / 5

Actionability

It gives concrete commands ('git status -sb', 'git diff --stat', 'rg', 'grep'), a structured output template, and a copy-ready inline-comment format. It is not a 5 because the 'look for' lists are detection prompts rather than fully executable checks, and not a 3 because the concrete commands and templates are clearly executable.

4 / 5

Workflow Clarity

A clear 7-step numbered sequence with edge cases (no changes, large diff, mixed concerns) and an explicit end confirmation checkpoint ('Do NOT implement any changes until user explicitly confirms'). It is not a 5 because there is no validate-then-retry feedback loop, and not a 3 because steps and the final checkpoint are explicit and well-defined.

4 / 5

Progressive Disclosure

SKILL.md is a clear overview that points to four real one-level-deep reference files (solid-checklist.md, security-checklist.md, code-quality-checklist.md, removal-plan.md), each loaded at the relevant workflow step and summarized in a Resources table. This matches the anchor for well-signaled, easy-to-navigate one-level references.

5 / 5

Total

17

/

20

Passed

Description

66%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 specific and uses natural trigger terms with a clear 'what', but omits any explicit 'Use when...' guidance, which caps completeness at 3. Distinctiveness and specificity are solid.

Suggestions

Add an explicit trigger clause, e.g., 'Use when the user asks to review a PR, code review their changes, or audits a git diff for SOLID, security, or quality issues.'

Include natural synonyms such as 'pull request', 'diff', and 'PR review' to broaden trigger coverage.

Mention the removal/iteration capability in the description so coverage matches the body's workflow.

DimensionReasoningScore

Specificity

Lists several concrete actions — 'code review of current git changes', 'Detects SOLID violations, security risks', 'proposes actionable improvements' — with only minor coverage gaps (e.g., removal candidates). It is not a 5 because coverage is not comprehensive, and not a 3 because it names multiple specific actions rather than just one or two.

4 / 5

Completeness

It has a clear 'what' (review git changes, detect SOLID/security issues, propose improvements) but no explicit 'Use when...' trigger clause, so the 'when' is only weakly implied. Per the rubric guideline, a missing 'Use when...' clause caps completeness at 3, so it cannot reach 4 or 5 despite the strong 'what'.

3 / 5

Trigger Term Quality

Natural user phrases 'code review' and 'git changes' appear verbatim, matching what a user would actually say. It is not a 5 because no synonyms or variations (e.g., 'review my diff', 'PR review') are included, and not a 3 because the core terms are exactly the natural phrasing rather than generic.

4 / 5

Distinctiveness Conflict Risk

The niche — reviewing current git changes with a SOLID/security lens — is mostly distinct with only minor overlap risk against general coding skills. It is not a 5 because it lacks an explicit distinct trigger phrase, and not a 3 because 'code review of git changes' is clearly more specific than broad document/code skills.

4 / 5

Total

15

/

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.

Validation16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository
dtyq/magic
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.