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.
A tight, well-sequenced instruction skill: it composes an existing review skill for its criteria, gives concrete git and tool-call commands, and builds in failure handling and a rejection feedback loop. The only shortfalls are small actionability gaps, chiefly leaving the default branch for merge-base unspecified.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~40-line body is lean and assumes competence: it delegates review criteria to factory-review via skill_read instead of re-explaining them, gives bare git commands with no tutorial padding, and every sentence instructs. It matches anchor 5 ("every token earns its place") rather than 4, since there is no over-explanation left to trim. | 5 / 5 |
Actionability | Guidance is mostly executable: copy-paste commands (`git rev-parse --abbrev-ref HEAD`, `git merge-base HEAD origin/<default branch>`, `base...HEAD`), a fully specified `factory_record_review_verdict` call with argument names, and a concrete report format. Minor gaps hold it at anchor 4 rather than 5 — the default branch is never determined for the reader, and steps like "load the category guidance" resolve only through the referenced skill's phases. | 4 / 5 |
Workflow Clarity | The sequence is explicit (role guard → load criteria → checkout → report → verdict → stop) with validation checkpoints and feedback loops: "If it does not exist on `origin`, the builder never pushed: request changes", "confirm each one is fixed on this head before looking for new problems", and the verdict "repeated only if rejected and only with the stated reason addressed". The workflow is read/test/record (pushing and committing are explicitly forbidden), so the destructive/batch cap does not apply; anchor 5 fits. | 5 / 5 |
Progressive Disclosure | The skill is under 50 lines with no bundle files, well-organized into scoped sections, and its only external dependency is a single clearly-signaled one-level reference ("Load `factory-review` with `skill_read`"). Per the rubric's simple-skill guideline this matches anchor 5. | 5 / 5 |
Total | 19 / 20 Passed |