Content
92%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.
An exceptionally lean orchestration skill: a fully sequenced workflow with an explicit pre-finish validation checklist, concrete commands/paths/fields throughout, and a clean two-file reference layer that keeps the body compact. The single weakness is minor — a handful of rules are repeated across sections and could be consolidated into one statement.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative, and assumes Claude's competence — no concept explanations, no filler. It falls short of 5 only because a few rules are stated twice (the GitHub second-copy rule appears in both Start and Output, and the no-commit rule is restated after the validation checklist), which could be consolidated. | 4 / 5 |
Actionability | For an instruction-only skill the guidance is fully executable: exact commands (`git rev-parse --show-toplevel`), exact paths (`.factory/FACTORY.json`, `.factory/tickets/<ticket-id>/`), exact frontmatter fields (`type`, `qa`, `blocked_by` with their allowed values), named tools and calls (`grill-with-docs`, `caller_ping`, `subagent_done`), and templates in references. Per the scoring notes, the absence of code is not penalized since every instruction is concrete. | 5 / 5 |
Workflow Clarity | The sequence (Start → Resolve uncertainty → Build the plan → Validate → Output) is explicit, and 'Validate and hand off' is an eight-item verification checklist (criterion mapping, acyclic dependencies, no speculative work, no commits) that gates finishing. It is not 4 because validation is explicit and error-recovery paths are covered — the caller_ping/waiting/resume loop and the 'if it replies that the session is shutting down, stop' handling — and not lower because the skill's operations are explicitly non-destructive (no commit/push/merge). | 5 / 5 |
Progressive Disclosure | The ~60-line body is a clear overview that delegates templates and output format to two well-signaled markdown links, [references/artifacts.md](references/artifacts.md) and [references/output.md](references/output.md), both of which exist and point no deeper — one level of nesting exactly. Verified against the actual bundle listing, the split is appropriate: artifact and template detail lives in references while the body keeps the workflow. | 5 / 5 |
Total | 19 / 20 Passed |