Content
70%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 delivers exceptionally concrete, executable guidance with a rigorously sequenced workflow and validation loops at every fork — top-anchor actionability and workflow clarity. Its weaknesses are structural: extreme paragraph density and ~50KB of inlined operational protocol that a reference bundle (which already exists for category guidance) should absorb.
Suggestions
Move the Phase 6 publishing protocol (verdict edit-vs-post rules, author-identity handling, follow-up PR steps) and the Phase 3 test-check/test-report procedures into reference files under references/, keeping a short procedure summary in SKILL.md.
Break the 400-600-word single-paragraph bullets (e.g. the test-check and test-report bullets in Phase 3) into numbered steps or tables so each instruction is scannable.
Deduplicate verdict-publication mechanics, which are restated across the Phase 6 bullets, into one authoritative sequence.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~50KB of unbroken mega-paragraphs — the Phase 3 test-check bullet alone is a ~600-word single paragraph, and verdict-publication mechanics are restated across multiple Phase 6 bullets. Not 1 because nothing explains concepts Claude already knows and there is no fluff padding; not 3 because the verbosity and density are pervasive throughout, not occasional — the rubric penalizes verbosity even when accurate. | 2 / 5 |
Actionability | Fully executable throughout: exact `gh` commands with flags, a complete GraphQL reviewThreads query with pagination logic, exact tool invocations (`factory_review_source`, `source_control_publish_test_check`, `factory_record_review_verdict`) with their argument shapes, exact approval gates, and concrete fallback commands for every failure path. Copy-paste ready and covers the common cases. | 5 / 5 |
Workflow Clarity | Six phases are clearly sequenced (context → existing signal → quality gate → history/architecture → verdict → handoff/publish) with explicit validation checkpoints and feedback loops: adversarial check before every approve, the in-run cross-check with stop-and-retry on mismatch, the head-moved check before publishing, gate-the-file checks before posting, and retry-only-after-rejection for the terminal call. This matches the top anchor's validate-fix-revalidate pattern. | 5 / 5 |
Progressive Disclosure | The references that exist are real, clearly signaled, and one level deep (`references/categories/README.md` with 15 category pages, `references/archaeology.md` for command recipes), and the body points to them at the right moment. But the main file still inlines the bulk of the operational protocol — the entire Phase 6 publishing/edit/fallback rules and the test-check procedure are exactly the content that belongs in a reference file the body could point to. Not 4 because 'most content appropriately placed' fails for a 50KB monolith; not 2 because the references that do exist are well-used and correctly signaled. | 3 / 5 |
Total | 15 / 20 Passed |