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-structured orchestration skill: the 9-step nightly pipeline is concrete and sequenced, with preflight gates, a dry-run-before-apply filing pattern, exit-code-driven feedback loops, and an explicit decisions ledger. Detail is appropriately delegated one level deep to four named sub-skills. The only gap is that two script invocations ($H/pr.ts, $P/new-plan.ts) are named without argument syntax.
Suggestions
Show the invocation shape for $H/pr.ts and $P/new-plan.ts (e.g. '$H/pr.ts --run $RUN --pr <number>'), even in one-line examples, since they are the only commands used without any argument syntax.
Define or link <dataDir> where it is first used in step 7, since its value is not stated in this file and is only implicitly owned by fragility-common.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and operational throughout — e.g. 'Planning only. Never change product code, open a PR, or push' and 'Exit 2 is never a pass' — with zero padding and no explanation of concepts Claude already knows. Every token earns its place; not even minor trimming is apparent (anchor 4), so anchor 5 fits. | 5 / 5 |
Actionability | Most commands are concrete and flag-complete ('$H/collect.ts --run $RUN', '$J/jira-upsert.ts --run $RUN --key <KEY> --system <system> --apply', '$S/summarize.ts --run $RUN'), matching anchor 4's 'concrete code or commands with minor gaps'. It falls short of anchor 5 ('copy-paste ready... common cases') because two invocations are named without any argument syntax — 'read 3 to 6 fix PRs with `$H/pr.ts`' and 'Scaffold it with `$P/new-plan.ts`' — and `<dataDir>` is never defined in this file. | 4 / 5 |
Workflow Clarity | Nine numbered steps in a clear sequence with explicit validation checkpoints and feedback loops: preflight doctors with 'Stop on any nonzero exit', '$J/jira-upsert.ts --plan <file>` (a dry run), then add `--apply'', and 'Exit 1 means some hot system is still undecided, so go back to step 7'. This matches anchor 5 (explicit validation steps, error-recovery loops, checklist-style hard rules) rather than anchor 4, which has 'minor validation gaps'. | 5 / 5 |
Progressive Disclosure | The body is a clear orchestrator overview that states 'each step's details live in that skill's SKILL.md' and maps exactly which of the four sub-skills owns what (history/scoring, Jira dedup, plans, run format), with one-level-deep pointers like '.agents/skills/refactor-plan/references/rubric.md'. The bundle's own script (scripts/summarize.ts) is referenced in the body as '$S/summarize.ts', so referenced paths resolve; this matches anchor 5 (clear overview, well-signaled one-level-deep references, content appropriately split) rather than anchor 4, which would leave organization gaps. | 5 / 5 |
Total | 19 / 20 Passed |