Content
56%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.
The body is rich in concrete, executable workflow guidance with strong validation gates and checklists, but it is severely over-long and self-duplicating, with conflicting step numbering that undermines an otherwise clear sequence. Progressive disclosure is undercut by inlining content that should live in separate reference files.
Suggestions
De-duplicate the body: keep a single canonical EXECUTION CONTRACT and remove the repeated context-detection block and the four near-identical banner copies, consolidating into one banner template.
Move the long worked examples (authentication system, API endpoint), the dev-subtype supplement table, and the banner templates into separate reference files linked one level deep from a leaner overview.
Reconcile the two conflicting step-numbering schemes into one sequence so the validation checkpoints are unambiguous.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~800-line body duplicates entire blocks — the context-detection step appears twice (EXECUTION CONTRACT Step 1 and the later 'Develop Workflow' Step 1) and the provider banner is shown four times with conflicting content — matching 'Noticeably verbose; several unnecessary explanations or padded sections'; not 1 because genuine actionable content is present underneath the padding, and not 3 because the duplication is pervasive rather than occasional. | 2 / 5 |
Actionability | It provides concrete executable commands (orchestrate.sh, state-manager.sh, octo-state.sh, check-providers.sh, the BM25 search.py invocation, git tag checkpoint) matching 'Mostly executable guidance; concrete code or commands with minor gaps'; not 5 because several commands use unresolved placeholders like '${codex_status}', '${agy_status}', and '<user\'s implementation request>'. | 4 / 5 |
Workflow Clarity | The EXECUTION CONTRACT defines a clearly sequenced 7-step mandatory flow with explicit validation gates (Step 5 synthesis verification), blocking checkpoints, per-step error handling, and before/after checklists, fitting 'Clear sequence with most checkpoints present; minor validation gaps'; not 5 because the duplicated, conflicting step numbering (two parallel 'Step 1/2/3' sequences) muddies the canonical sequence. | 4 / 5 |
Progressive Disclosure | Template placeholders ({{PREAMBLE}}, {{VISUAL_INDICATORS}}, {{QUALITY_GATES}}) and references to external plugin scripts provide some deferred content, but the SKILL.md itself is an 800-line monolith with duplicated banners and long examples inlined that belong in separate files, matching 'Some structure ... content that should be separate is inline'; not 4 because the inlining is substantial and navigation to detail is not cleanly one-level. | 3 / 5 |
Total | 13 / 20 Passed |