Content
70%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 strong orchestrator skill body: routing, sequencing, validation gates, and audit rules are explicit and executable, with excellent workflow clarity. Its weaknesses are redundant restatement of the git-checkpoint and reroute rules, an undefined `$PYTHON_PATH` placeholder, and no navigation to the bundled `scripts/progress.py`.
Suggestions
Consolidate the git-commit rules into the single Git Checkpoint Discipline section and reference it once from Hard Constraints instead of restating the rules in Workspace Initialization, Stage Transitions, and Rules.
Split the reroute semantics (the 6-step numbered list and stage-state rules) and the command quick reference into a separate reference file, and link to them from the body to reduce inline bulk.
Define how `$PYTHON_PATH` should be resolved (or drop the placeholder), and reference the bundled `scripts/progress.py` so the tooling layer of the skill is discoverable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dominated by dense tables and copy-ready commands with no explanations of known concepts, but the git-commit discipline is repeated across four sections (Workspace Initialization, Stage Transitions, Rules, Hard Constraints) and reroute semantics are restated multiple times, so it could be meaningfully tightened — anchor 3 rather than 4. | 3 / 5 |
Actionability | Concrete, mostly copy-paste commands appear throughout (`research-pipeline update-stage STAGE STATUS`, the fully realized reroute example with reason and source artifact), but `$PYTHON_PATH` is never defined and `ags.py` is not part of the bundle — minor gaps that keep it below anchor 5. | 4 / 5 |
Workflow Clarity | The sequence is explicit with entry-condition routing, a produces/consumes pipeline map, and hard validation gates ("Always run `experiment-config check` before `run-experiment`", "Never run analysis before... `_schema.json`") plus error-recovery feedback loops (failed stage → owning-skill repair, run → config on failed validation, auditable reroute), matching anchor 5. | 5 / 5 |
Progressive Disclosure | Sections are clearly organized, but all detail lives inline in a ~255-line body: the bundle's `scripts/progress.py` (26 KB) is never referenced from the body, and the reroute semantics and command reference are prime candidates for a separate file — anchor 3 ("content that should be separate is inline"), not 4 given the completely un-signaled bundle file. | 3 / 5 |
Total | 15 / 20 Passed |