Content
72%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 lean, well-sectioned body whose main weakness is workflow clarity: the body's core message is that baseline updates happen automatically, contradicting the description's download-and-commit promise, and the investigation path omits how to get the run-id or what to do with the downloaded screenshots. Progressive disclosure is excellent for a short single-purpose skill.
Suggestions
Reconcile the body with the description: either state explicitly that the skill's action when invoked is to verify/investigate (not commit), or restore download-and-commit instructions.
Add the concrete step for obtaining the run-id, e.g. `gh run list --workflow "Checking Component Screenshots" --limit 1`.
Specify what to do after downloading the artifacts (where to view diffs, when to escalate) so the investigation loop has a defined endpoint.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~30-line body is efficient — a lean explanation of the new baseline storage model plus a numbered investigation procedure with one command. Minor trims possible (e.g. "Compare locally if needed. The artifact contains the full set of captured screenshots." and some of the historical "What Changed" context), so it sits just below the 'every token earns its place' anchor. | 4 / 5 |
Actionability | The gh command is concrete and executable ("gh run download <run-id> --name screenshots --dir .tmp/screenshots") and the investigation steps are actionable, but <run-id> is never specified and 'Compare locally if needed' gives no method. Not a 3 because the guidance is genuinely executable, not pseudocode; not a 5 because obtaining the run-id and the comparison method are missing gaps. | 4 / 5 |
Workflow Clarity | The investigation sequence (check PR comment → download artifact → compare) is numbered, but the central workflow is muddied: the frontmatter promises download-and-commit while the body states "No manual baseline updates are needed — the screenshots on the main branch commit become the new baselines automatically after merge", leaving the invoked agent without a clear primary action. Gaps include how to find the run-id and what to do after comparing. Not a 4 because these gaps are more than minor; not a 2 because a rough, readable sequence is present. | 3 / 5 |
Progressive Disclosure | The skill is under 50 lines with no external references needed (no references/, scripts/, or assets/ directories exist, and none are required), and it is organized into clearly labeled sections ("What Changed", "If Screenshots Need Investigation"). Per the simple-skill guideline, well-organized sections alone merit the top score. | 5 / 5 |
Total | 16 / 20 Passed |