CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

pr-review

Comprehensive PR review across six dimensions — cost impact, environment drift, ownership gaps, SOC 2 compliance, deprecated API / version hygiene, and rollback feasibility. Each mode inspects the diff and current file state, reports findings with severity, and recommends concrete fixes. Use when preparing a PR for merge, conducting a pre-deployment readiness check, or performing a post-merge risk assessment.

Quality

87%

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

78%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-structured review playbook with concrete commands, detection patterns, and output templates. Its main defect is progressive disclosure: all three cited reference files are missing from the bundle, and the six per-mode checklists (400+ lines) could be offloaded to the reference layer the skill already points to.

Suggestions

Ship the cited bundle files (references/pr-review.md, references/compliance.md, references/kubescape.md) or remove the dangling 'Reference:' lines — every mode currently points to a file that does not exist in the skill directory.

Move the per-mode 'What to check' checklists into references/pr-review.md (which each mode already cites by section), keeping SKILL.md as mode selection, output formats, and the summary structure to cut context weight.

Trim the Kubernetes deprecation table to entries Claude would not reliably recall (e.g., the kyverno.io/v1 → policies.kyverno.io/v1 migration with its 1.17/1.20 dates) since the standard removal timeline is well known.

DimensionReasoningScore

Conciseness

The body is dense operational checklists with essentially no concept explanation Claude already knows — pricing figures, exact match patterns, and flag-level commands earn their tokens. Minor trimmable material exists (the Kubernetes deprecation table largely restates well-known removal versions, and the severity scales repeat across output-format blocks), keeping it below the lean score-5 anchor but above score 3's 'could be tightened'.

4 / 5

Actionability

Copy-paste-ready commands with exact flags ("infracost diff --path=. --compare-to=<base.json>", "kubescape scan framework nsa <path> --honor-inline-exceptions=false --baseline base.json"), concrete pricing ("~$32/month per AZ plus $0.045/GB"), exact detection patterns ("Action: \"*\"", "prevent_destroy = true"), and complete output templates cover the common cases — matching the fully executable top anchor.

5 / 5

Workflow Clarity

Mode selection is well handled (interactive wizard, and the input guard "If no PR number or diff is provided, ask the user to paste the diff" before proceeding), 'full' mode gives an explicit ordered sequence, and the Merge Readiness Summary triages blockers vs. recommended vs. informational. Per-mode checklists are unordered lists rather than sequenced analysis steps, leaving minor checkpoint gaps that keep it below the score-5 anchor.

4 / 5

Progressive Disclosure

Section structure and reference signaling are good — each mode ends with "Reference: references/pr-review.md → <section>" at one level of depth — but none of the cited files (references/pr-review.md, references/compliance.md, references/kubescape.md) exist in the bundle, so every reference dangles. Additionally, 400+ lines of per-mode checklists are inlined when they could live in the reference file each mode already names. This fits the score-3 anchor (structure present, references signaled, but content that should be separate is inline); it is above score 2 because the per-mode sections are well organized and clearly navigable, but below score 4 because the references resolve to nothing.

3 / 5

Total

16

/

20

Passed

Description

92%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 names a distinct niche, lists its six dimensions concretely, and provides an explicit 'Use when...' clause with three concrete scenarios. The only weakness is modest synonym coverage of trigger terms.

DimensionReasoningScore

Specificity

Enumerates six concrete review dimensions ("cost impact, environment drift, ownership gaps, SOC 2 compliance, deprecated API / version hygiene, and rollback feasibility") and three specific actions ("inspects the diff and current file state, reports findings with severity, and recommends concrete fixes") in third-person voice. Coverage is comprehensive rather than having minor gaps, so it matches the top anchor over the score-4 anchor.

5 / 5

Completeness

Explicitly answers what (six named dimensions, per-mode behavior) and when: "Use when preparing a PR for merge, conducting a pre-deployment readiness check, or performing a post-merge risk assessment" — concrete trigger scenarios matching the top anchor exactly.

5 / 5

Trigger Term Quality

Natural phrases users would say are well covered — "PR review", "cost impact", "environment drift", "SOC 2 compliance", "rollback", "pre-deployment readiness check" — but there is no synonym expansion (e.g., "pull request", "pre-merge check", "merge readiness"), which keeps it below the score-5 anchor's comprehensive synonym coverage while clearly above score 3's missing-variation level.

4 / 5

Distinctiveness Conflict Risk

A clear niche (SOC 2 control mapping, rollback feasibility scoring, environment drift) with distinct trigger scenarios gives minimal conflict risk with generic code-review skills; the specialization is specific enough that wrong-skill triggering is unlikely.

5 / 5

Total

19

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

frontmatter_unknown_keys

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

Warning

referenced_paths_exist

Referenced path issues: 9 missing

Warning

Total

14

/

16

Passed

Repository

nitinjain999/platform-skills

Path

skills/pr-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.