Content
71%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 maximally lean, generated pointer stub that delegates entirely to a shared skill outside the bundle. Conciseness is perfect and the single instruction is unambiguous, but actionability and structure suffer because no task guidance or overview exists in the skill itself and the delegated target could not be verified to exist.
Suggestions
Inline a minimal quick-start (e.g., where hooks live, a one-line config.yml snippet, or the load-order rule) so the skill offers actionable guidance even when the shared skill at ~/.agents/skills/omp-hooks/SKILL.md is unavailable.
Add a fallback or verification note for the delegated path (e.g., what to do if the shared skill is missing, or how to regenerate it via the harness) so the single workflow step has an error path.
Briefly state in the body what the shared skill covers (event catalog, discovery and load order, disable controls) so the body doubles as an overview and the external reference is contextually signaled rather than bare.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is two lines — a provenance comment ("Generated by scripts/agent-harnesses.py; do not edit directly.") and a single directive ("Load and follow the shared skill at ~/.agents/skills/omp-hooks/SKILL.md"). Zero padding, nothing explaining concepts Claude already knows; every token earns its place, matching the lean-and-efficient anchor. Not 4, since there is nothing that could be trimmed. | 5 / 5 |
Actionability | "Load and follow the shared skill at `~/.agents/skills/omp-hooks/SKILL.md`" is one concrete, precise directive, but it is the entirety of the guidance — no steps, commands, or examples for the actual hook/extension tasks, which are fully delegated. This matches anchor 3 ("some concrete guidance but incomplete"); not 2 because the directive is specific rather than a high-level hint, and not 4 because the gaps are total rather than minor. | 3 / 5 |
Workflow Clarity | This is a single-task, under-50-line skill whose one action (load the shared skill) is unambiguous, satisfying the simple-skill exception; however the sole step has no verification or fallback, and its target file does not resolve in this environment. Not 5 because there is no error-recovery guidance for the single step; not 3 because the action is fully specified and the destructive/batch validation cap does not apply to a pure load instruction. | 4 / 5 |
Progressive Disclosure | The single reference is clearly signaled and one level deep, but it points outside the bundle to a file that does not exist in this environment (verified: ~/.agents/skills/omp-hooks/SKILL.md is absent), the body itself contains no overview content, and the bundle has no references/, scripts/, or assets/ structure (the commented scripts/agent-harnesses.py is also absent). Not 4 because "most content is appropriately placed" fails — the skill ships zero content, only an unverifiable external pointer; not 2 because the reference is not buried and nothing is inlined. | 3 / 5 |
Total | 15 / 20 Passed |