CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-pr

Review CausalPy pull requests end-to-end by classifying PR type, checking branch freshness, mergeability, remote CI, correctness, security, tests, docs, and maintainer concerns. Use when asked to review a PR, assess a branch before merge, summarize PR risks, or request changes.

Quality

92%

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.

A well-crafted maintainer-review skill: concrete commands, repo-specific norms, explicit falsification probes, and a copy-paste output template make it highly actionable with a clearly sequenced, validated workflow. Its main weaknesses are minor redundancy and abstract framing that could be trimmed, and the fact that every detail file it dispatches to is missing from the staged bundle, leaving the otherwise-exemplary progressive-disclosure structure unverifiable.

Suggestions

Trim duplicated and speculative content for conciseness: the human-approval rule is stated in both Boundary and Review Output, and the 'if the repository later adds a dedicated monitoring skill' bullet is hypothetical — state it once and drop the future-conditional.

Include (or verify) the ten referenced `resources/*.md` files in the skill bundle so the progressive-disclosure paths actually resolve for a reader of SKILL.md.

Compress the Review Stance prose ('state a private review brief', 'trace both directions across its seam') into a short concrete checklist of probes so every line is an instruction rather than framing.

DimensionReasoningScore

Conciseness

The body is dense and assumes Claude's competence (no explaining what a PR or git is), with concrete commands like `gh pr checks`, `uv run <command>`, and `prek run --all-files`. It falls short of the lean-everywhere anchor only in minor spots: the human-approval rule is stated twice ("Never post review comments... without explicit human approval" in Boundary and "show the draft to the user first and wait for approval" in Review Output), the speculative "If the repository later adds a dedicated monitoring skill, prefer that" bullet, and some abstract framing prose in Review Stance that could be trimmed to a checklist.

4 / 5

Actionability

Guidance is concretely executable throughout for an instruction-only skill: named commands (`gh pr checks`, `uv run <command>`, `$CONDA_EXE run -n CausalPy <command>`, `prek run`), exact paths (`causalpy/tests/`, `docs/source/notebooks/index.md`, `AGENTS.md`, `ARCHITECTURE.md`), a seven-step numbered intake, and a copy-paste-ready review output template. Specific examples cover the common cases (failed/pending/skipped/missing CI checks, behind-base branches, conflict handling).

5 / 5

Workflow Clarity

The multi-step process is clearly sequenced with explicit validation checkpoints: intake steps 2–5 verify the working tree, branch freshness, conflicts, and CI before diff review; "Verify contributor claims against code, tests, and branch history before accepting them"; "Do not report 'no findings' until these probes have either found a concrete issue or produced evidence that the concern is handled"; and "run `prek run` during iteration and `prek run --all-files` before handoff" provides a fix→re-check feedback loop. The Universal Checks section is exactly the checklist-for-complex-processes pattern the top anchor rewards.

5 / 5

Progressive Disclosure

The structure as written matches the top anchor — a concise overview with one-level-deep, conditionally-signaled references ("Feature implementation: read `resources/pr-type-features.md`", "Read these when the PR touches the relevant surface"). However, all ten referenced `resources/*.md` files (workflow.md, six pr-type files, code-patterns.md, docs-patterns.md, review-patterns.md, review-comments.md, maintenance.md) are absent from the bundle, so the navigation cannot actually be completed; that unresolvable reference gap fits the good-structure/minor-gaps anchor rather than a clean 5.

4 / 5

Total

18

/

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 in third person: it answers what and when explicitly, names the domain and a comprehensive list of concrete review actions, and provides natural maintainer-voice triggers. The only gap is a handful of missing trigger synonyms like 'code review' or 'ready to merge'.

DimensionReasoningScore

Specificity

The description enumerates the full concrete review surface: "classifying PR type, checking branch freshness, mergeability, remote CI, correctness, security, tests, docs, and maintainer concerns" — multiple specific actions with comprehensive coverage for a PR-review skill, matching the top anchor. Only "maintainer concerns" is generic; the other eight named actions are concrete.

5 / 5

Completeness

Both questions are answered explicitly: the first sentence states what the skill does ("Review CausalPy pull requests end-to-end by classifying PR type, checking...") and the second gives concrete 'Use when' trigger phrases. It is written in third person with no fluff, matching the top anchor exactly.

5 / 5

Trigger Term Quality

"Use when asked to review a PR, assess a branch before merge, summarize PR risks, or request changes" gives good, natural trigger coverage a maintainer would actually say. A few common variations are missing — e.g., 'code review', 'is this ready to merge', 'merge readiness' — which fits the good-coverage/few-terms-missing anchor rather than the comprehensive-synonyms anchor.

4 / 5

Distinctiveness Conflict Risk

The description carves a clear niche — CausalPy pull-request review — with triggers ('review a PR', 'summarize PR risks', 'request changes') that are distinct from fixing/greening work, and the repo name plus PR framing separates it from generic code-review and document skills. Overlap risk with a sibling greening skill is minor because the description stays on the assess/summarize side.

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

pymc-labs/CausalPy

Path

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