Content
88%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 well-structured, highly actionable, and concisely written workflow with strong validation and error-handling discipline throughout. Its main limitation is that all detail lives inline in one file with no progressive disclosure to supporting reference files.
Suggestions
Extract the detailed review-metadata field reference (reviews.events, comments, unresolved_threads, diff fields) into a references/ file and link to it from SKILL.md to shorten the main body.
Consider moving the capability-negotiation and schema-validation Python snippets into a scripts/ helper so the main workflow shows intent rather than boilerplate.
Add a brief one-line 'When to use' pointer at the top of the body that mirrors the description's trigger guidance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and purposeful — it explains only Herdr-specific concepts (protocol/schema versions, manager inventory fields) Claude would not know and avoids padding generic concepts — with only minor prose that could be trimmed, fitting the 'efficient; minor over-explanation' anchor. | 4 / 5 |
Actionability | Provides fully executable, copy-paste-ready shell plus Python heredoc blocks for plugin discovery, capability negotiation, querying, and materialization, covering the common workflow cases as the level-5 anchor requires. | 5 / 5 |
Workflow Clarity | Multi-step processes (discover → negotiate capabilities → query → filter → recommend → materialize) are clearly sequenced with explicit validation checkpoints (protocol_version, schema_version, error summaries, stale-head rejection) and feedback loops for the destructive materialization operation, matching the level-5 anchor. | 5 / 5 |
Progressive Disclosure | Content is organized into clearly headed sections with no nested references and everything kept one level deep, but the ~150-line monolithic file keeps all inline code and field reference detail in SKILL.md rather than splitting detail into bundle files, which is good but sits just below the ideally-split level-5 anchor. | 4 / 5 |
Total | 18 / 20 Passed |