Content
75%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 highly actionable, with copy-paste-ready commands for every workflow stage and a clear sequenced process backed by a real, verified bundle. Its weaknesses are token efficiency — a metric-definition glossary that re-teaches known concepts and a duplicated strategy listing — and an unreferenced implementation.md that hurts discoverability of the bundle's deeper content.
Suggestions
Trim the Performance/Risk/Trade Statistics tables to metric names plus only the non-obvious notes (e.g., the profit-factor N/A rule), dropping definitions of well-known concepts like volatility and max drawdown.
Remove the 8-strategy bullet list from the Overview since the Supported Strategies table already covers it in full detail.
Add a link to references/implementation.md (e.g., in the Files or Examples section) so users extending the framework can discover it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The 13-row metric glossary re-defines concepts Claude already knows (e.g., 'Volatility: Annualized standard deviation', 'Total Return: Overall percentage gain/loss', 'Max Drawdown: Largest peak-to-trough decline'), and the 8 strategies are listed twice (Overview bullets plus the Supported Strategies table). This is more than the 'minor instances' of the 4 anchor but the rest of the body is efficient, so it fits the 'mostly efficient but includes some unnecessary explanation' anchor. | 3 / 5 |
Actionability | Fully executable, copy-paste-ready commands covering the common cases: fetch_data.py with symbol/period/interval, backtest.py with both defaults and a custom --params JSON payload, optimize.py with a --param-grid example, plus a concrete settings.yaml. Output file naming (*_summary.txt, *_trades.csv, *_equity.csv, *_chart.png) tells Claude exactly what to inspect. | 5 / 5 |
Workflow Clarity | The fetch → backtest → analyze → optimize sequence is clearly numbered with concrete commands per step, and this is a read-only analytical workflow (no destructive/batch cap applies). It falls short of 5 because validation/verification checkpoints are delegated to references/errors.md rather than stated inline (e.g., no step to confirm data was fetched or sanity-check the trade log before optimizing). | 4 / 5 |
Progressive Disclosure | Good structure: the core workflow stays in SKILL.md, errors and examples are clearly signaled one-level-deep references, and the Files table documents the verified script bundle. It misses 5 because references/implementation.md exists in the bundle but is never referenced from the body, leaving a navigation/discovery gap. | 4 / 5 |
Total | 16 / 20 Passed |