Content
75%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, actionable single-file skill with a clear sequenced workflow and concrete executable commands. Its main weaknesses are mild redundancy (the Example Commands section repeats workflow commands) and the absence of error-recovery feedback loops in the workflow.
Suggestions
Trim or merge the "Example Commands" section, since its entries duplicate commands already shown in the workflow steps; keep only the grep-based log-search example that adds new value.
Add a brief recovery step for when log extraction fails or the run ID cannot be resolved from the status check rollup (e.g., fall back to `gh run list --branch <branch-name> --limit 5`).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is efficient and assumes Claude's competence (no explanations of what CI or gh is), but the "Example Commands" section largely duplicates commands already shown in the workflow (gh pr view --json, gh run view --log-failed). This matches anchor 4 (minor trimmable redundancy) rather than 5 (every token earning its place). | 4 / 5 |
Actionability | Concrete, mostly copy-paste-ready commands appear throughout ("GH_PAGER=cat gh pr view <branch-name> --json statusCheckRollup", "gh run view <run-id> --log-failed 2>&1 | grep -A 5 \"error:\""), with clear parsing targets. Minor gaps (the create_plan tool reference and placeholder values) keep it at anchor 4 rather than 5. | 4 / 5 |
Workflow Clarity | A clearly sequenced 5-step workflow (verify PR, check CI status, extract logs, categorize errors, generate plan) with explicit conditional handling ("If no PR exists...", "If CI is still running..."). The skill is read-only diagnosis so the destructive/batch validation cap does not apply; it stays at 4 rather than 5 because there are no feedback loops for error recovery (e.g., what to do if log extraction fails or the run ID is missing). | 4 / 5 |
Progressive Disclosure | No bundle files exist, and the ~110-line body is well organized into labeled sections with references to companion skills (fix-errors, create-pr) clearly signaled. It matches anchor 4 (good structure, minor organization gaps); the under-50-line simple-skill exception for a 5 does not apply, and sections like "Common CI Check Names" and "Example Commands" are modest inline reference material that could be split out. | 4 / 5 |
Total | 16 / 20 Passed |