Content
88%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 actionable orchestration skill: every step carries exact commands, paths, formats, and validation/BLOCKED checkpoints with failure-recovery loops, and detail is pushed to separate config, sub-skill, and template files. The only tightening opportunities are minor repetition around Compose conditions and snapshot-validator quirks, and making file references more clearly navigable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative, and explains nothing Claude already knows — every line is operational instruction — matching 'efficient; minor instances of over-explanation that could be trimmed'. Not 5 because §6's validator-quirk paragraph is repetitive (three consecutive 'never...' formulations) and the Compose condition is stated twice (§1 and §6); not 3 because there is no padded or concept-explaining content. | 4 / 5 |
Actionability | Fully executable guidance throughout: exact shell snippets (RUN_ID="${QA_RUN_ID:-$(date +%s)-$$}", regex ^[A-Za-z0-9_-]+$), exact paths (qa-results/$RUN_ID/evidence/case-<number>.snapshot.txt), exact invocation format (env -u CI FACTORY_DISABLE_KEYRING=true ./qa-results/$RUN_ID/publish/Zipper …), and exact report formats (result-column emoji codes, link syntax [$label]($RUN_ID/evidence/...)). Matches 'copy-paste ready' with specific formats covering the common cases. | 5 / 5 |
Workflow Clarity | Eight clearly numbered stages with explicit validation checkpoints and stop conditions (symlink refusal, RUN_ID regex check, existing-run-directory BLOCKED, pre-flight prerequisite checks) and error-recovery loops ('Never silently skip a flow... Continue with other relevant flows', 'Upload failures fall back to text and artifacts'). The one destructive operation (cleanup) is explicitly validated ('after verifying qa-results, the run directory, and both targets are not symlinks'), so the destructive-operation cap does not apply; this matches the 5 anchor with feedback loops and checkpoints. | 5 / 5 |
Progressive Disclosure | Detail is appropriately delegated one level deep to clearly signaled files at point of use — config (.factory/skills/qa/config.yaml), the test menu (.factory/skills/qa-cli/SKILL.md), and the report template (.factory/skills/qa/REPORT-TEMPLATE.md) — keeping SKILL.md an orchestration overview, matching 'good structure; most content is appropriately placed'. Not 5 because the references are plain code-formatted paths rather than clearly navigable links and they point outside the skill bundle (no references/ directory exists, so they cannot be verified as bundle files). | 4 / 5 |
Total | 18 / 20 Passed |