CtrlK
BlogDocsLog inGet started
Tessl Logo

review-pr

Review a pull request against CDP architecture standards — fetches PR diff, verifies previous comments are addressed, validates PR metadata (title, branch, JIRA key, size), runs a code-standards enforcer against every file in `.claude/rules/` and `.claude/hooks/guard-protected-files.sh`, and drafts inline review comments with suggested fixes. NEVER auto-posts comments or submits reviews — always presents a draft in the terminal for user approval before any comment lands on the PR. Use when reviewing PRs, checking PR quality, validating code changes, or when the user says "review", "check this PR", or "audit code".

76

Quality

96%

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

96%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 well-engineered workflow skill: exact commands, a parallelized enforcer subagent with a hallucination guard, code-level verification of previous comments, and a strict human-approval gate before any PR mutation. The main improvements are removing duplicated content (PR-size rule, auto-post warnings) and surfacing the reference checklists as first-class navigation rather than paths buried in the agent prompt.

DimensionReasoningScore

Conciseness

The phases are dense with executable commands and templates and explain nothing Claude already knows, but there is noticeable redundancy: the PR-size rule appears in both Phase 4 ('if additions > 1000, note per commit-workflow.md's 1000-line target') and the 'Additional Rules' section, the 'NEVER auto-post' constraint is stated three times, and the Phase 5 false-positive filter repeats the enforcer prompt's 'drop the finding' instruction. This places it above level 3 ('mostly efficient... could be tightened') but short of level 5's 'every token earns its place'.

4 / 5

Actionability

Fully executable guidance throughout: exact gh/git commands with flags and --json field lists ('gh pr view <N> --json title,body,headRefName,...'), a structured agent prompt with a JSON return schema, markdown table templates, and concrete AskUserQuestion options — copy-paste ready and covering the common cases, matching the level-5 anchor.

5 / 5

Workflow Clarity

Six clearly ordered phases with an explicit background-agent wait point ('Wait for the Phase 2 enforcer Agent to complete'), a validation filter before surfacing findings, and a hard approval checkpoint for the PR-mutating batch operation ('Do NOT proceed until the user explicitly picks an option. Treat silence or ambiguous replies as "don't post"') — a full validate/approve loop matching the level-5 anchor, so the destructive/batch cap does not apply.

5 / 5

Progressive Disclosure

All four referenced checklists (backend, frontend, services, sql) are real one-level-deep bundle files referenced by explicit path, and the main body is well organized by phases — but the checklist references appear only inside the Phase 2 agent prompt rather than being clearly signaled as navigation from the main body, which keeps it at 'good structure... minor organization gaps' (4) rather than the clearly-signaled level-5 anchor.

4 / 5

Total

18

/

20

Passed

Description

96%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: third-person voice, concrete enumerated capabilities, an explicit 'Use when' clause with quoted natural trigger phrases, and a clear safety boundary (never auto-posts). The only weakness is that 'review' and 'validating code changes' are broad enough to occasionally overlap generic review skills.

DimensionReasoningScore

Specificity

Lists multiple specific concrete actions — 'fetches PR diff, verifies previous comments are addressed, validates PR metadata (title, branch, JIRA key, size), runs a code-standards enforcer... drafts inline review comments with suggested fixes' — with comprehensive coverage of the skill's capabilities, matching the level-5 anchor; it exceeds level 4, whose anchor expects only 'several specific actions' with minor gaps.

5 / 5

Completeness

Explicitly answers 'what' (a detailed list of concrete capabilities) and 'when' ('Use when reviewing PRs, checking PR quality, validating code changes, or when the user says...') with concrete trigger phrases, exactly matching the level-5 anchor; level 4 would require the 'when' clause to be less explicit.

5 / 5

Trigger Term Quality

Includes natural phrases users would actually say — '"review", "check this PR", "audit code"' — plus synonyms like 'reviewing PRs, checking PR quality, validating code changes', matching the comprehensive natural-term coverage of the level-5 anchor.

5 / 5

Distinctiveness Conflict Risk

The PR-review scope is a clear niche, but bare trigger words like 'review' and broader phrases like 'validating code changes' carry minor overlap risk with generic code-review skills, so it sits between the 'clear niche' (5) and 'mostly distinct' (4) anchors — closer to 4.

4 / 5

Total

19

/

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

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

Total

15

/

16

Passed

Repository
linuxfoundation/crowd.dev
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.