Content
86%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.
A tight, well-structured workflow that gets straight to executable commands with no wasted tokens and appropriately delegates details to the project's own spec files. The only gaps are minor: the lint/typecheck step names no concrete command, and fixing violations has no explicit re-validation loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~26-line body is lean and instructional throughout: three concrete commands and six tersely labeled steps with zero padding or explanation of concepts Claude already knows. Even "The index is NOT the goal — it points you to the actual guideline files" earns its place by guarding a real failure mode, matching 'every token earns its place'. | 5 / 5 |
Actionability | Steps 1–3 give copy-paste-ready commands ("git diff --name-only HEAD", "python3 ./.trellis/scripts/get_context.py --mode packages", "cat .trellis/spec/<package>/<layer>/index.md"), but step 5 "Run lint and typecheck for the affected package" provides no specific command or pointer to where it is defined. This is 'mostly executable guidance; concrete code or commands with minor gaps' rather than the fully copy-paste-ready profile of score 5. | 4 / 5 |
Workflow Clarity | The six steps are clearly numbered and sequenced, and the lint/typecheck run plus "Report any violations" act as checkpoints. It falls short of score 5 because "fix them if found" lacks an explicit re-validation loop (fix, then re-run lint/typecheck), leaving a minor validation gap; it is above score 3 since checkpoints are present and explicit, not merely implied. | 4 / 5 |
Progressive Disclosure | This is a simple skill under 50 lines with no bundle files and no content that belongs in separate reference files, so the simple-skill exception applies: the well-organized numbered structure with bold step labels suffices. The dynamic pointers into ".trellis/spec/" are one level deep (index.md to the guideline files it names) and explicitly signaled. | 5 / 5 |
Total | 18 / 20 Passed |