CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-review-excellence

Transform code reviews from gatekeeping to knowledge sharing through constructive feedback, systematic analysis, and collaborative improvement.

Quality

60%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Low

Low-risk findings worth noting

Fix and improve this skill with Tessl

tessl review fix ./plugins/AI-Agents-Safe-Coding-Skills/skills/code-review-excellence/SKILL.md

The canonical home for this skill is code-review-excellence in rmyndharis/antigravity-skills

SKILL.md
Quality
Evals
Security

Quality

Content

57%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 clean, well-organized, token-efficient instruction skill with good scoping sections and a concrete output format, undermined by thin actionable detail and a dangling reference: the referenced playbook (resources/implementation-playbook.md) does not exist in the bundle, and the workflow lacks any verification checkpoints. It scores just above the scale midpoint.

Suggestions

Fix the broken reference: either create resources/implementation-playbook.md (with the review checklists and feedback templates the Instructions promise) or remove the reference and inline the essential checklist.

Add a verification checkpoint to the workflow, e.g., 'Before reporting a finding, re-confirm it against the actual diff lines and cite file:line for each issue.'

Include one short worked example of feedback (issue → severity → rationale → suggested fix) so the severity tiers and 'actionable feedback with severity and rationale' become concrete rather than aspirational.

DimensionReasoningScore

Conciseness

The body is lean and assumes Claude's competence — terse bullet instructions, no explanation of what code review is, and a short scoping section. It falls short of anchor 5 only because of two redundancies: the opening paragraph repeats the frontmatter description verbatim, and the playbook reference is stated twice ('If detailed checklists are required, open resources/implementation-playbook.md' and again in Resources).

4 / 5

Actionability

There is some concrete guidance — the four review dimensions (correctness, security, performance, maintainability), named severity tiers (blocking, important, minor), an ordered first step ('Read context, requirements, and test signals first'), and a defined output format — but key details are missing: no example of well-formed feedback, no severity definitions, and the promised detail lives in resources/implementation-playbook.md, which does not exist in the bundle. This matches anchor 3 ('Some concrete guidance but incomplete; missing key details') rather than 4, since the instructions are directives without executable specifics.

3 / 5

Workflow Clarity

The instructions imply a sequence (read context → review → give severity-ranked feedback → ask clarifying questions), but the steps are an unordered bullet list with no validation checkpoints — e.g., nothing like 're-verify each finding against the actual diff before reporting' or 'confirm the issue reproduces'. This fits anchor 3 ('Steps listed but validation gaps; sequence present but checkpoints missing or implicit') rather than 4, which requires most checkpoints to be present.

3 / 5

Progressive Disclosure

The skill is under 50 lines with well-organized sections (Use when / Do not use / Instructions / Output Format / Resources), which would ordinarily merit a 5 under the simple-skill exception — but it references resources/implementation-playbook.md in two places, and no resources/ directory (or the file) exists anywhere in the bundle, so navigation to the only referenced material fails entirely. A dangling reference is worse than the 'minor organization gaps' of anchor 4 and matches anchor 3's level of structural deficiency ('Some structure but could be better organized; references present but not clearly signaled' — here clearly signaled but broken).

3 / 5

Total

13

/

20

Passed

Description

45%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 identifies a real, distinct domain (code review) but reads as a philosophy statement rather than a capability description: actions are abstract, trigger terms are thin (no 'pull request', 'PR', 'diff'), and there is no explicit 'Use when' guidance. It sits just below the midpoint of the scale.

Suggestions

Add an explicit 'Use when...' clause naming concrete triggers, e.g., 'Use when reviewing pull requests, PR feedback, or code diffs, or when establishing team review standards.'

Replace abstract phrasing ('transform... through collaborative improvement') with concrete capabilities: 'Reviews pull requests for correctness, security, and performance; delivers severity-ranked feedback; mentors authors via constructive comments.'

Include the natural synonyms users actually say — 'pull request', 'PR', 'diff', 'review comments' — alongside 'code reviews'.

DimensionReasoningScore

Specificity

The description names the domain ("code reviews") but its actions are abstract means rather than concrete capabilities — "Transform code reviews from gatekeeping to knowledge sharing through constructive feedback, systematic analysis, and collaborative improvement" reads as a mission statement, with no specific operations (e.g., 'Review pull requests', 'Audit diffs for security issues'). It is below anchor 3 because no 1-2 truly concrete actions are named, matching anchor 2 ('Names the domain but actions are minimal or generic').

2 / 5

Completeness

The 'what' is stated (transform reviews into knowledge sharing via constructive feedback and systematic analysis), but there is no 'Use when...' clause or equivalent explicit trigger guidance, capping completeness at 3 per the judging guidelines. It is not a 2 because the 'what' is clear and domain-specific, not vague, matching anchor 3 ('clear what but when is missing').

3 / 5

Trigger Term Quality

"code reviews" and "feedback" are natural phrases a user would say, but common variations and synonyms are missing: no "pull requests", "PR", "diff", "code changes", or "review my code". This fits anchor 3 ('Some relevant keywords but missing common variations or synonyms') rather than 4, whose example shows a fuller spread of natural terms.

3 / 5

Distinctiveness Conflict Risk

"Code reviews" carves out a recognizable niche, but generic phrases like "systematic analysis" and "collaborative improvement" overlap with general coding-assistance, refactoring, and mentoring skills, and the sparse trigger terms increase misfire risk. This matches anchor 3 ('Somewhat specific but could still overlap with similar skills') rather than 4, which requires being mostly distinct with only minor overlap against closely related skills.

3 / 5

Total

11

/

20

Passed

Validation

93%

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

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

frontmatter_unknown_keys

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

Warning

Total

15

/

16

Passed

Repository

administrakt0r/AI-Agents-Safe-Coding-Skills

Path

plugins/AI-Agents-Safe-Coding-Skills/skills/code-review-excellence/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.