Content
47%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 harness-generated stub that is maximally token-efficient but delivers no task content: no diagnostic steps, no commands, and no workflow. Its structure also breaks navigation — the two bundled reference files are orphaned and the only pointer targets an external absolute path, so progressive disclosure fails against the actual bundle layout.
Suggestions
Reference the bundle's own files (e.g., references/ditto-ci.md for repo-specific CI context, references/preflight-recovery.md for failure-handling dialogs) from the body so they are discoverable, rather than deferring entirely to an external path.
Inline a minimal quick-start — such as the `gh run view <RUN_ID> --repo <owner/repo>` command and what failure signal to look for — so the skill is actionable even if the external shared file is unavailable.
Replace or supplement the environment-dependent absolute path (`~/.agents/skills/gha-log-reader/SKILL.md`) with a bundle-relative reference so navigation does not rely on an unverified external location.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is a single redirect line ("Load and follow the shared skill at `~/.agents/skills/gha-log-reader/SKILL.md`.") with zero padding or explanation of concepts Claude already knows. Not 4: there is literally nothing to trim — every token beyond the pointer is a one-line provenance comment. | 5 / 5 |
Actionability | The only guidance is one directive naming an external absolute path; no steps, commands, or code for the actual task of diagnosing GHA failures. Not 3: unlike pseudocode-with-gaps, the body contains no task-relevant executable guidance at all — everything is deferred to a file outside this bundle; not 1 because the single redirect instruction itself is concrete and unambiguous. | 2 / 5 |
Workflow Clarity | No sequence exists for the skill's diagnostic workflow (fetch logs, parse, identify root cause, suggest fix) — all steps are delegated to the external file, and no validation checkpoints appear in what is written. Not 1: the body is coherent and its one instruction is unambiguous rather than incoherent; not 3 because the simple-skill exception doesn't apply — diagnosing failing runs is inherently multi-step, and none of those steps are present. | 2 / 5 |
Progressive Disclosure | The bundle contains references/ditto-ci.md and references/preflight-recovery.md, but the body never mentions them — they are orphaned and undiscoverable — while its single pointer leads outside the bundle to `~/.agents/skills/gha-log-reader/SKILL.md`, an environment-dependent absolute path. Not 3: the bundle references are not merely "not clearly signaled," they are entirely unreferenced; not 1: the body is not a monolithic wall of text and its one reference is clearly stated. | 2 / 5 |
Total | 11 / 20 Passed |