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 dense, highly actionable body with an exemplary gated workflow and valuable domain-specific empirical data, but it is oversized for a single SKILL.md: plotting boilerplate and reference implementations belong in bundle files, and minor import/dependency gaps keep some code from being copy-paste ready.
Suggestions
Split the cointegration framework, visualization templates, and cross-market empirical tables into references/ files (e.g. references/cointegration.md, references/plots.md), keeping SKILL.md as a navigable overview.
Trim or externalize the matplotlib boilerplate (heatmap, dendrogram, signal plots) — Claude can write plotting code on demand; keep only the domain-specific annotation logic.
Make code snippets self-contained: define or stub the src.quantlib.timeseries functions, and add the missing adfuller/np imports used in monitor_spread_health and correlation_breakdown_test.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~1100 lines with dense, mostly non-redundant analytics code and genuinely non-obvious empirical tables (regime correlation behavior, half-life ranges, cross-market lag patterns), but it also carries content Claude does not need — full matplotlib boilerplate for heatmaps, dendrograms, and signal plots, plus a long worked report example. It is not 4 because the visualization templates and several verbose docstrings could be trimmed or externalized; it is not 2 because there is little conceptual padding and the domain-specific content earns its place. | 3 / 5 |
Actionability | Nearly all guidance is executable: complete function implementations with defaults and docstrings, concrete thresholds ('keep pairs with p < 0.05', 'half-life between 5 and 60 days'), a pip install command, and a report output template. It falls short of 5 because some snippets are not copy-paste ready — 'from src.quantlib.timeseries import ...' references a project module not in this bundle, and monitor_spread_health uses 'adfuller' and 'np' without importing them in that scope. | 4 / 5 |
Workflow Clarity | The 'Full Workflow From Correlation to Signal' section gives five clearly sequenced steps with explicit validation criteria at each gate (Pearson > 0.6, cointegration p < 0.05, half-life 5-60 days, ADF p < 0.05), and Step 5 monitoring defines feedback loops (recompute Z-Score daily, re-test cointegration monthly, 'Warn if half-life exceeds 2× the original'), reinforced by monitor_spread_health's status-to-action mapping ('exit_now'). Mode 1's numbered scan-then-test workflow mirrors this structure. | 5 / 5 |
Progressive Disclosure | The single SKILL.md is a monolith: the cointegration framework, plotting templates, and cross-market empirical tables are all inline with no references/, scripts/, or assets/ files at all, so content that clearly belongs in separate files is inlined. It is not 2 because internal structure is strong — clear mode sections, tables, an overview, and a notes checklist that make navigation easy despite the size. | 3 / 5 |
Total | 15 / 20 Passed |