CtrlK
BlogDocsLog inGet started
Tessl Logo

review-pr

Review a pull request diff and write structured feedback to review.json for the workflow to publish. Use when reviewing a checked-out PR from local artifacts like pr_diff.txt and pr_description.txt and producing machine-readable review output instead of posting directly to GitHub.

68

Quality

83%

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

82%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 strong, highly actionable instruction-only skill: exact output contract, severity labels, suggestion-block semantics, an audit checklist, and a bundled validator with an explicit fix-and-rerun loop. Its weaknesses are minor — some duplicated rules across sections that could be consolidated, and one bundle-script reference that is less clearly signaled than the other.

DimensionReasoningScore

Conciseness

The body is dense, directive, contract-style prose that assumes Claude's competence — no padding or explanations of known concepts. Minor trimming is possible: the Approve/REJECT verdict mapping is stated twice (Output Format field rules and Body Requirements), and 'put the feedback in top-level body instead of comments' repeats in Review Scope, Diff Line Annotations, and Comment Requirements. This fits anchor 4's 'minor instances of over-explanation that could be trimmed' rather than anchor 5's every-token-earns-its-place.

4 / 5

Actionability

Fully executable guidance throughout: a complete review.json example with exact field shapes, four exact severity labels with emoji prefixes, exact suggestion-block syntax with precise range semantics, and a copy-paste validator command ('python3 .agents/skills/review-pr/scripts/validate_review_json.py --review-json review.json --diff pr_diff.txt') including fallback path resolution. This matches anchor 5's copy-paste-ready coverage of common cases.

5 / 5

Workflow Clarity

Sections flow in execution order (Context → Review Scope → Repository-specific guidance → annotation/comment contracts → output format → Pre-Verdict Audit → Final Checks), with an explicit audit checklist and a validate→fix→rerun feedback loop ('If the script reports any invalid comments, fix review.json and rerun it'). It falls short of anchor 5 only because the sequence is thematic rather than explicitly numbered as a step-by-step workflow; checkpoints are otherwise present, placing it above anchor 3.

4 / 5

Progressive Disclosure

The bundle (scripts/resolve_spec_context.py, scripts/validate_review_json.py — both verified present) is referenced with exact paths and usage at one level of depth, and the body is well sectioned. The resolve_spec_context.py reference is buried mid-list in Context rather than clearly signaled like the validator's dedicated code block in Final Checks, so it matches anchor 4's 'references mostly clear; minor organization gaps' rather than anchor 5's fully well-signaled structure.

4 / 5

Total

17

/

20

Passed

Description

83%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: concrete actions, an explicit 'Use when' clause with specific trigger artifacts, and a distinctive local-artifacts/machine-readable framing that separates it from direct-to-GitHub review skills. Written in third person with no fluff. The only gaps are a few missing natural synonyms (e.g., 'code review') and slight overlap with neighboring PR-review skills.

DimensionReasoningScore

Specificity

Lists several concrete actions — 'Review a pull request diff', 'write structured feedback to review.json', 'producing machine-readable review output' — with minor gaps in coverage (e.g., no hint at the review dimensions like correctness or security). It exceeds anchor 3 because more than 1-2 concrete actions are named, but is not the comprehensive coverage of anchor 5.

4 / 5

Completeness

Explicitly answers both: 'what' ('Review a pull request diff and write structured feedback to review.json for the workflow to publish') and 'when' ('Use when reviewing a checked-out PR from local artifacts like pr_diff.txt and pr_description.txt...'). This matches anchor 5's requirement of a clear what plus concrete trigger phrases; anchor 4's weaker 'when' would understate the explicit and specific trigger guidance here.

5 / 5

Trigger Term Quality

Good natural keyword coverage — 'pull request', 'PR', 'diff', 'pr_diff.txt', 'pr_description.txt', 'review.json' — phrases a user would naturally say. A few natural variants (e.g., 'code review') are missing, so it sits at anchor 4 rather than the comprehensive synonym/extension coverage of anchor 5, and clearly above the sparse coverage of anchor 3.

4 / 5

Distinctiveness Conflict Risk

The framing 'from local artifacts like pr_diff.txt... instead of posting directly to GitHub' carves a distinct niche versus generic GitHub-review skills, with concrete artifact filenames as triggers. Minor overlap risk with closely related PR-review skills keeps it at anchor 4 rather than the minimal-conflict niche of anchor 5.

4 / 5

Total

17

/

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
warpdotdev/common-skills
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.