CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-review

Perform a code review of the current session's changes. Use when the user requests a code review via the Run Code Review button in the Changes toolbar.

63

Quality

79%

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 ./src/vs/sessions/skills/code-review/SKILL.md
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.

The body is a tight, well-structured instruction-only skill: it defines the reviewer role, gives a concrete six-step workflow with specific commands and a named commenting tool, and sets clear priorities (high-signal comments, no file modifications, stop when done). Its only minor gaps are the lack of an example review comment and an explicit completeness check before finishing.

DimensionReasoningScore

Conciseness

The 26-line body is lean and assumes Claude's competence: "Determine the set of changed files in the current session (e.g. `git status`, `git diff`)" and "Do not modify files. Do not run commits, pushes, or other write operations" give instructions without explaining concepts Claude already knows. Every line earns its place, matching the anchor-5 example of lean, efficient content.

5 / 5

Actionability

Concrete, executable guidance throughout: specific commands (`git status`, `git diff`), a named tool ("use the `addComment` tool to attach a comment to the exact file URI and line range"), explicit comment-content rules, and a five-item review checklist. It falls short of anchor 5 only because there is no example of a well-formed comment or an exact `addComment` invocation to copy from — minor gaps consistent with anchor 4. Per the rubric's code_vs_instruction note, the absence of code in this instruction-only skill is not penalized.

4 / 5

Workflow Clarity

The six numbered steps form a clear sequence (identify changes → review against criteria → attach inline comments → stop), with an explicit end checkpoint ("When you have finished reviewing every changed file, stop") and a checklist of review dimensions. It sits at anchor 4: most checkpoints are present, but there is no explicit verification pass (e.g., re-checking `git status` to confirm no changed file was skipped). The workflow is read-only, so the destructive/batch validation cap does not apply.

4 / 5

Progressive Disclosure

The skill is under 50 lines with no need for external references (none exist in the bundle), and the content is well organized into a role statement plus a "Workflow" section with sub-bullets. Per the rubric's guideline for short skills with no external references, well-organized sections alone merit anchor 5.

5 / 5

Total

18

/

20

Passed

Description

62%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 concise, in third person, and explicitly answers both what the skill does and when to use it, anchored by a concrete UI trigger. Its main weaknesses are thin action coverage (a single "perform a code review" verb) and missing natural trigger synonyms beyond the button name. It is solid but below the exemplar level of the rubric's good examples.

Suggestions

Enumerate the concrete actions the review performs, e.g., "Review changed files for bugs, regressions, security issues, and missing error handling, and attach inline comments to the exact lines" instead of the single generic verb "perform a code review".

Broaden the "when" clause beyond the UI button to natural trigger phrases users would say, e.g., "Use when the user requests a code review, asks to review changes or the diff, or clicks the Run Code Review button in the Changes toolbar".

Add one or two synonyms such as "diff" or "review my changes" to reduce collision risk with related skills and improve natural-language triggering.

DimensionReasoningScore

Specificity

"Perform a code review of the current session's changes" names the domain and one concrete action but does not enumerate what the review covers (e.g., finding bugs, leaving inline comments). This matches the anchor for 1-2 concrete actions without comprehensive coverage; it is not generic enough for 2 and lacks the several specific actions needed for 4.

3 / 5

Completeness

It answers both "what" ("Perform a code review of the current session's changes") and "when" ("Use when the user requests a code review via the Run Code Review button") in third person. The "when" is explicit but narrow (a single UI trigger rather than natural trigger phrases), and the "what" is thin, matching anchor 4 rather than the comprehensive both-parts anchor 5.

4 / 5

Trigger Term Quality

"code review" and "changes" are relevant natural keywords, but common variations users would say ("review my changes", "diff", "look over this code") are absent, and the trigger leans heavily on the UI button phrase "Run Code Review button in the Changes toolbar". Some relevant keywords with missing synonyms fits anchor 3, short of anchor 4's good coverage.

3 / 5

Distinctiveness Conflict Risk

"code review" plus the specific UI-button trigger gives a clear niche distinct from unrelated skills, but the term overlaps with closely related review skills (security review, PR review), fitting anchor 4 ("mostly distinct; minor overlap risk") rather than anchor 5's minimal conflict risk.

4 / 5

Total

14

/

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

posit-dev/positron

Path

src/vs/sessions/skills/code-review/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.