Content
77%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The body is a highly actionable, well-sequenced orchestrator skill with clear gates and validation feedback loops, but it is long and inlines reference-grade material that the progressive-disclosure links suggest should live in separate files. Tightening and moving bulk mechanics into the referenced files would materially improve it.
Suggestions
Move the full 'Resumable runs' and 'Overnight heartbeat: stall detection' sections into the already-referenced shared-references files, keeping only a short pointer plus the exact accept/gate rule in SKILL.md to cut the inline token load.
Collapse the Constants block into a compact table (flag | default | where it is passed through) instead of one verbose bullet per constant, preserving the override example.
Confirm the referenced shared-references and templates files actually ship with the skill bundle; the body links to ../shared-references/*.md and templates/RESEARCH_BRIEF_TEMPLATE.md but none are present in references/, scripts/, or assets/.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | It avoids explaining concepts Claude already knows, but the ~350-line body inlines substantial mechanics (the Constants block, the full Resumable runs section, and the complete heartbeat stall-detection bash) that could be tightened or offloaded to references. | 2 / 3 |
Actionability | Provides copy-paste-ready invocations ("/idea-discovery \"$ARGUMENTS\""), concrete run_state.py / iteration_log.py commands with exact arguments, and explicit artifact paths — fully executable guidance throughout. | 3 / 3 |
Workflow Clarity | Stages 1–5 are clearly sequenced with two explicit gates (Gate 1 human checkpoint, Gate 2 writing checkpoint), a per-phase accept table, and a validation/feedback loop (re-validate done-but-unaccepted stages on resume, never accept on the executor's own say-so). | 3 / 3 |
Progressive Disclosure | Shared-reference links (external-cadence.md, resumable-runs.md, output-versioning.md) are one-level-deep and clearly signaled, but no bundle files exist in references/scripts/assets and large blocks of content that belong in those references (resumable runs, heartbeat detection) are inlined rather than split out. | 2 / 3 |
Total | 10 / 12 Passed |