Content
88%Weight 40%Scale 1-5Reviews 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.
| Dimension | Reasoning | Score |
|---|---|---|
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 |