Content
57%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 dense, information-rich reference with strong actionable content — executable slippage/impact functions, concrete per-market parameters, and useful decision guidance — undermined by monolithic structure and missing validation steps. Nothing is offloaded to reference files, and there is no verification loop to check that execution assumptions are realistic before conclusions are drawn.
Suggestions
Move the per-market cost/impact reference tables and the execution-algorithm details into references/ files (e.g. references/cost-tables.md, references/execution-algorithms.md) and keep SKILL.md as a concise overview with one-level-deep links.
Add explicit validation checkpoints to the workflow, e.g. 'sanity-check modeled fills against historical VWAP and reject the backtest if cost drag exceeds X% of gross return', turning the sensitivity table into a required verification step rather than an illustration.
Trim the 'Why Slippage Models Are Needed' motivation block and consolidate the three separate commission=0.001 recommendations into one, and fix the incomplete SignalEngine/delayed_execution snippets (import pandas, initialize signals) so all code is copy-paste executable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Most of the body is genuinely non-obvious domain data (per-market cost tables, impact coefficients, the model-selection tree) that earns its tokens, but it opens with a motivational 'Why Slippage Models Are Needed' block explaining basics Claude already knows ('the order book has a bid-ask spread', 'large orders push prices'), and repeats the commission=0.001 recommendation in three separate sections. It fits 'mostly efficient but includes some unnecessary explanation or could be tightened' rather than level 4's 'minor instances'. | 3 / 5 |
Actionability | The skill provides mostly executable guidance: complete Python functions (fixed_slippage, linear_impact, sqrt_impact) with documented arguments, concrete parameter ranges in tables, a capital-vs-ADV decision tree, and explicit config examples. It falls short of fully copy-paste-ready (level 5) because delayed_execution uses pd.Series without importing pandas, and the SignalEngine.generate snippet references an undefined signals dict and self._compute_signal, so it will not run as written. | 4 / 5 |
Workflow Clarity | There are sequenced processes — the three-step cost-impact analysis framework and the slippage-model selection decision tree — but no validation checkpoints or feedback loops anywhere (e.g. 'verify the fill assumptions against historical VWAP before accepting results'). The sequence is present but checkpoints are missing or only implicit, matching the level-3 anchor. | 3 / 5 |
Progressive Disclosure | The body has clear section structure but is a ~340-line monolithic document with no bundle files at all: per-market reference tables, the cost-coefficient data, and the code snippets are exactly the material that belongs in references/ files linked one level deep. It matches 'some structure but content that should be separate is inline' — better than the unstructured level-2 anchor, but with no reference architecture to reach level 4. | 3 / 5 |
Total | 13 / 20 Passed |