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.
A well-engineered skill body: terse operational instructions, an explicit validation checklist with blocked-path feedback loops, and clean delegation of artifact formats to a single real reference file. The only notable improvements are eliminating the thrice-stated GitHub no-copy rule and making the fstate CLI path fully concrete.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes Claude's competence — no concept padding, every section gives operational instruction. It is not a 5 because the 'do not publish a second copy of a GitHub issue into .factory/tickets when GitHub is the ticket source' rule is stated three times (Start, the to-tickets bullet, and the Output section), which is trimmable redundancy. | 4 / 5 |
Actionability | Mostly executable guidance: concrete commands ('git rev-parse --show-toplevel'), exact paths ('.factory/FACTORY.json', '.factory/tickets/<ticket-id>/'), named tool calls (caller_ping, subagent_resume), and a fully specified output template. Minor gaps keep it from 5: 'node …/factory-supervise/scripts/fstate/cli.mjs' has an elided path, and several integration steps depend on optional external skills and environment state. | 4 / 5 |
Workflow Clarity | The multi-step process is clearly sequenced (Start → Resolve uncertainty → Build the plan → Validate and hand off → Output) with an explicit pre-finish validation checklist ('Before finishing, verify that: every success condition maps to at least one task acceptance criterion...') and genuine feedback loops (ask → waiting_for_user → subagent_resume; blocked status naming the unresolved decision). This matches the level-5 anchor. | 5 / 5 |
Progressive Disclosure | The body is a clear overview that appropriately moves artifact file templates (plan.md, task files, adr.md formats) into a single real, one-level-deep reference, [references/artifacts.md](references/artifacts.md), referenced from clearly relevant sections. Navigation is easy and nothing is buried or nested, matching the level-5 anchor. | 5 / 5 |
Total | 18 / 20 Passed |