Content
82%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.
The body is a lean, highly actionable instruction skill: exact gh/git commands with field lists, a concrete output template, and decision branches for PR and issue review. Its only real weaknesses are mild redundancy in the Provenance section and an implied (rather than explicit) end-to-end workflow order across sections.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes Claude's competence — no concept explanations, no padding — and commands are given as bare blocks. However, the Provenance section restates the shallow/grafted-history point several times across overlapping sentences (e.g. the long boundary sentence and "A genuine root needs raw-header proof that it has no parents"), which is trimmable without losing guidance. This puts it at anchor 4 (efficient, minor over-explanation) rather than 5. | 4 / 5 |
Actionability | Guidance is copy-paste ready throughout: exact `gh issue view`/`gh pr view` commands with full `--json` field lists, `gh pr diff`, concrete `git log -S/-G`, `git --no-replace-objects cat-file`, and `git diff --no-ext-diff` provenance commands, plus a fill-in output template. The common cases (issue review, PR review, diff inspection, provenance tracing) are each covered by an executable command, matching anchor 5. | 5 / 5 |
Workflow Clarity | The skill is organized into a clear sequence (Start → Review Contract → Code Reading Depth → Provenance → Fix Quality Bar → PR/Issue Review Shape → Output Template), and Issue Review Shape gives numbered steps with decision branches and an explicit checkpoint ("Check whether current `main` already fixes it") plus failure handling ("If reproduction is not feasible, say exactly what blocks it"). It sits at anchor 4 rather than 5 because the overall cross-section flow is implied by section order rather than stated as one explicit sequenced workflow with validation checkpoints per stage. | 4 / 5 |
Progressive Disclosure | No bundle files exist; all content lives in this single well-sectioned file, and the one external dependency (the author-context workflow at `~/Projects/agent-scripts/skills/github-author-context/SKILL.md`) is clearly signaled by absolute path with explicit when-to-use conditions. This matches anchor 4 (good structure, most content appropriately placed): the dense Provenance section is arguably material that could live in a separate reference file, which keeps it from anchor 5. | 4 / 5 |
Total | 17 / 20 Passed |