Content
85%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 tight, well-sequenced instruction skill: every step is actionable with real commands, and the delegation to .trellis/ project files is exactly the right structure. Its only weaknesses are slight framing redundancy and minor gaps that would make the templated cat commands fully self-contained.
Suggestions
Remove the redundant framing sentences (the opening line and the step 6 wrap-up) — the numbered steps and the closing 'mandatory' gate already convey the workflow.
Show a one-line example of the get_context.py --mode packages output so the <package>/<layer> placeholders in the cat commands can be filled in confidently.
Add a brief fallback for when ./.trellis/scripts/get_context.py is missing or fails, e.g. listing .trellis/spec/ directories directly with ls.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — no concept explanations, just commands and steps. However, minor filler could be trimmed: the opening "Read the relevant development guidelines before starting your task" and step 6's "Understand the coding standards and patterns you need to follow, then proceed with your development plan" restate what the numbered steps already convey. This sits between anchor 5 (every token earns its place) and anchor 4 (minor instances that could be trimmed), closer to 4. | 4 / 5 |
Actionability | Mostly executable guidance: "python3 ./.trellis/scripts/get_context.py --mode packages" is copy-paste ready, and "cat .trellis/spec/<package>/<layer>/index.md" plus "cat .trellis/spec/guides/index.md" are concrete. Minor gaps keep it below anchor 5: the cat commands require substituting placeholders with no sample of the get_context.py output showing what package/layer values look like, and there is no fallback if the script is absent. It is well above anchor 3 because the commands are real, not pseudocode, and the placeholder use is justified by dynamic discovery. | 4 / 5 |
Workflow Clarity | A clear 6-step numbered sequence from discovery to reading the actual guideline files, with an explicit error-prevention checkpoint in step 4 ("The index is NOT the goal — it points you to the actual guideline files") and a mandatory gate ("This step is mandatory before writing any code"). The skill is read-only (no destructive or batch operations), so validation loops are not required, and as a simple under-50-line single-purpose skill with an unambiguous sequence it matches the top anchor per the simple-skills note. | 5 / 5 |
Progressive Disclosure | The skill has no bundle files (no references/, scripts/, or assets/), and at ~30 lines it needs none — all detailed content appropriately lives in the project's .trellis/ tree, which the skill delegates to with exact paths. Per the rubric's simple-skills guidance, an under-50-line skill with no need for external references scores 5 with well-organized sections, which the numbered step structure provides. It is not score 4 because there is no content that should have been split out and every pointer is one level deep. | 5 / 5 |
Total | 18 / 20 Passed |