Content
92%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-structured, highly actionable skill body: concrete copy-paste commands, a gated 8-step workflow with validation checkpoints and fallbacks, and clean one-level-deep disclosure to a real bundled script. The only notable weakness is repetition of the script invocation across three sections.
Suggestions
State the inspect_pr_checks.py invocation once (e.g., in Bundled Resources) and reference it from Quick start and Workflow step 3 instead of repeating the full command three times.
Drop the "Prereq: authenticate..." line from Overview since Workflow step 1 already covers auth verification and recovery in full.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Efficient and assumes competence (no explanation of what gh or GitHub Actions is), but the script invocation `python "<path-to-skill>/scripts/inspect_pr_checks.py" --repo ... --pr ...` appears three times (Quick start, Workflow step 3, Bundled Resources) and the "Prereq: authenticate" line in Overview duplicates Workflow step 1. Minor trimming would get it to anchor 5. | 4 / 5 |
Actionability | Fully executable commands throughout: `gh pr view --json number,url`, `gh pr checks <pr> --json name,state,bucket,link,...`, `gh run view <run_id> --log`, `gh api "/repos/<owner>/<repo>/actions/jobs/<job_id>/logs"`, plus a bundled script whose documented flags (--repo, --pr, --json, --max-lines, --context) match the actual argparse in scripts/inspect_pr_checks.py. Fallbacks for gh field drift and in-progress runs cover the common failure cases. | 5 / 5 |
Workflow Clarity | Eight clearly sequenced steps with explicit validation and recovery: step 1 gates on `gh auth status` with recovery ("If unauthenticated, ask the user to run `gh auth login`"), step 3 has fallbacks for rejected fields and in-progress logs, step 6-7 gate implementation on explicit approval, and step 8 rechecks status after changes. Only a minor gap: step 8 suggests re-running rather than mandating an iterate-if-still-failing loop. | 5 / 5 |
Progressive Disclosure | A lean ~70-line body with well-organized sections (Overview, Inputs, Quick start, Workflow, Bundled Resources) pointing to a single one-level-deep bundle file, scripts/inspect_pr_checks.py, which exists and matches the documented usage. Navigation is easy and nothing that belongs in a separate file is inlined. | 5 / 5 |
Total | 19 / 20 Passed |