Content
40%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 generated stub whose entire content is a redirect to a shared skill at an absolute home-directory path that does not resolve, leaving the skill non-functional as shipped. It is maximally lean (conciseness 4) but provides no actionable guidance, no workflow, and no verifiable content structure. The leanness reflects absence of content rather than efficient delivery of it.
Suggestions
Make the pointer resolvable: ship the shared skill inside the bundle (e.g., references/main.md referenced with a relative path) instead of an absolute ~/.agents/ path, or verify the external path exists at runtime.
Inline the essential executable guidance — the clean/smudge filter script, the .gitattributes entry, and the git config commands — so the body is actionable even if the external file is unavailable.
Add a brief overview of what the shared skill covers plus a validation checkpoint (e.g., verify the filter with git check-attr on a tracked file) so the config-modifying workflow has error-recovery steps.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is two lines with zero over-explanation of known concepts, matching 'efficient; minor instances that could be trimmed' — the maintainer-facing HTML comment ('Generated by scripts/agent-harnesses.py; do not edit directly') is roughly a third of the body and serves no model-facing purpose, so 'every token earns its place' (anchor 5) does not hold. | 4 / 5 |
Actionability | The only guidance is 'Load and follow the shared skill at `~/.agents/skills/git-value-scrubbing/SKILL.md`' — a concrete directive, but it contains no task-level steps, code, or commands for the actual skill function, and the target path does not exist in the bundle or on the system. This fits 'minimal concrete guidance; missing the specific steps to execute' (anchor 2); it is not anchor 1 because the instruction is specific and executable rather than vague, and not anchor 3 because no task guidance exists in the body at all. | 2 / 5 |
Workflow Clarity | The single redirect step is unambiguous, but the actual workflow (writing filter scripts, configuring .gitattributes, renormalizing tracked files) is entirely absent with no sequence or validation checkpoints — matching 'rough sequence present but many gaps; validation absent' (anchor 2). Not anchor 1 because the one step present is coherent and clearly stated; not anchor 3 because there are no task steps whose checkpoints could be implicit. | 2 / 5 |
Progressive Disclosure | The body is a single clearly-signaled, one-level-deep reference, but it points outside the bundle to `~/.agents/skills/git-value-scrubbing/SKILL.md`, which does not exist (verified), and the bundle contains no references/, scripts/, or assets/ files while the comment cites a nonexistent `scripts/agent-harnesses.py`. This is 'minimal structure' (anchor 2); not anchor 1 because the reference is prominently signaled rather than buried or nested, and not anchor 3 because there is no overview content or in-bundle organization — the sole navigation target is dead. | 2 / 5 |
Total | 10 / 20 Passed |