CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-reviewer

Review pull requests, diffs, and code changes for concrete behavior, contract, security, test, maintainability, and documentation defects. Use when the user asks for review of a PR, patch, diff, commit, or code change.

Quality

93%

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

SKILL.md
Quality
Evals
Security

Quality

Content

88%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 a highly actionable, well-sequenced review workflow with copy-paste commands, an explicit output template, and genuine validation feedback loops. Its weaknesses are minor: some duplicated staged-only guidance and a reference chain (investigation.md → review-cues.md) that adds a second level of disclosure.

Suggestions

Consolidate the staged-only review guidance that currently appears in both 'Establish the Target' (step 5) and 'Coordinator Responsibilities' into one location to trim redundant tokens.

Mention review-cues.md directly in the body (or move its pointer up a level) so the deepest reference is reachable in one hop from SKILL.md instead of only via investigation.md.

DimensionReasoningScore

Conciseness

The body is dense and assumes Claude's competence — no concept explanations, no padding — but staged-only-review guidance is repeated across 'Establish the Target' and 'Coordinator Responsibilities', and several sentences run long enough to need trimming, fitting the 4 anchor ('minor instances of over-explanation that could be trimmed') rather than the every-token-earns-its-place of 5.

4 / 5

Actionability

Guidance is fully executable: copy-paste-ready commands ('gh pr view <PR> --json title,body,files,commits,baseRefName,headRefName', 'git diff <TARGET-REF>...HEAD'), a complete markdown output template with per-section rules, and concrete severity definitions covering the common target cases (PR, working tree, branch/commit/range).

5 / 5

Workflow Clarity

The sequence (Establish Target → mode selection → coordinator prep → investigations → verification → severity → output) is clearly ordered, with explicit validation checkpoints and feedback loops: 'audit coverage against the target ... redispatch for material gaps', 'Verify each candidate against complete code paths', 'Run a focused non-destructive test, type check, build, linter, or reproduction', and 'If cleanup would disturb local work, stop and ask'. The workflow is read-only, so the destructive/batch validation cap does not apply.

5 / 5

Progressive Disclosure

Structure is good: the three mode references (standard-review.md, adversarial-review.md, investigation.md) are clearly signaled with explicit use conditions and all exist in references/. However, review-cues.md is only reachable through investigation.md (a two-level hop never signaled in the body), and the ~95-line body inlines coordination detail that could partly move to reference files — minor organization gaps matching the 4 anchor rather than the clean one-level-deep split of 5.

4 / 5

Total

18

/

20

Passed

Description

95%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.

A strong description: it states concrete capabilities, enumerates defect classes, and provides an explicit and specific 'Use when' trigger clause with natural synonyms (PR/patch/diff/commit). Its only limitation is that 'review' is a single action verb applied to many targets rather than a list of distinct actions.

DimensionReasoningScore

Specificity

The description enumerates comprehensive review targets ('pull requests, diffs, and code changes') and six concrete defect classes ('behavior, contract, security, test, maintainability, and documentation'), but rests on a single action verb rather than the multiple distinct concrete actions of the 5 anchor.

4 / 5

Completeness

It explicitly answers both questions: what it does ('Review ... for concrete behavior, contract, security, test, maintainability, and documentation defects') and when to use it ('Use when the user asks for review of a PR, patch, diff, commit, or code change') with concrete trigger phrases, matching the 5 anchor rather than the less-explicit 'when' of the 4 anchor.

5 / 5

Trigger Term Quality

It covers the natural terms users would say — 'PR', 'pull requests', 'diffs', 'patch', 'commit', 'code change', 'review' — including both the spelled-out and abbreviated forms, matching the comprehensive-synonyms anchor; nothing common is missing.

5 / 5

Distinctiveness Conflict Risk

It occupies a clear niche (defect review of code changes) with precise, unambiguous triggers ('review of a PR, patch, diff, commit, or code change'), so the risk of triggering for a wrong skill is minimal.

5 / 5

Total

19

/

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

shillem/cc-marketplace

Path

plugins/code-swiss-knife/skills/code-reviewer/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.