Content
92%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A high-quality, token-efficient routing reference with concrete tools, env keys, decision rules, and explicit validation/feedback guidance for batch and verification workflows. Its one real weakness is progressive disclosure: the detailed source and capability tables live inline in SKILL.md with no bundle-file split.
Suggestions
Move the 20-row Source Overview and 19-row Capability→Tool tables into a reference file (e.g. references/SOURCES.md), keeping the Decision Tree, Source priority, and Ban-Risk & Fallback Notes inline so SKILL.md reads as a lean overview with one-level-deep pointers.
Add a short 'References' section that explicitly signals the per-source skills (tushare, akshare, yfinance, qveris, etc.) and the `financial_rigor` tool as off-file destinations, making navigation more discoverable than the inline 'Skill' column alone.
If the tables must stay inline (since the router loads on every data task), add a one-line 'How to read this table' caption so the dense reference is navigable at a glance.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is almost entirely dense reference tables (Source Overview, Capability→Tool, Symbol Format) and concrete routing rules, assuming Claude's competence (e.g. it never explains what OHLCV or a backtest is). The redundancy between 'Source priority' and 'Ban-Risk & Fallback Notes' is minor, so it stays at the lean/efficient anchor rather than dropping to 2. | 3 / 3 |
Actionability | It names exact tools (`get_market_data`, `cross_validate`), env keys (`TUSHARE_TOKEN`, `FRED_API_KEY`), symbol formats, and a concrete `pass {source: value, ...}` usage. As an instruction-only router it carries no code blocks, which the rubric's scoring note says is not penalized when guidance is this specific — so it is not capped at 2. | 3 / 3 |
Workflow Clarity | The Decision Tree gives a clear numbered sequence for the analysis scenario with an explicit validation checkpoint ('if [the env key] is missing, report the missing key rather than failing silently') plus feedback loops (fallback on throttle/timeout, error envelopes that 'never abort the surrounding batch'). Batch operations have explicit validation, so it is not capped at 2. | 3 / 3 |
Progressive Disclosure | Sections are well organized and off-skill references (per-source skills, `financial_rigor`, test files) are one level deep, but the body is ~140 lines dominated by two large inline tables (20-row Source Overview, 19-row Capability table) that fit the 'content that should be separate is inline' anchor. It does not reach the 3 anchor's 'content appropriately split' because nothing is split into bundle files. | 2 / 3 |
Total | 11 / 12 Passed |