Content
65%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, domain-rich reference with executable Python helpers, concrete thresholds, and a clear output template. Its weaknesses are the absence of an explicit analysis workflow with validation checkpoints, an API section that documents endpoints as comments rather than usable code, and a monolithic ~250-line structure with no progressive disclosure into reference files.
Suggestions
Add an explicit step-by-step analysis workflow (fetch multi-exchange funding history → classify regime → compute annualized basis → evaluate carry opportunity → render the output template) with validation checkpoints such as confirming data freshness and cross-exchange consistency.
Move the detailed signal frameworks, threshold tables, and output template into a references/ file, keeping SKILL.md as a lean overview with clearly signaled links.
Replace the commented OKX endpoint URLs with a short executable request example so the data-access section is copy-paste ready.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and signal-heavy — threshold tables, compact Python functions, and an output template with almost no filler. It is not a 5 because passages like 'Perpetual futures have no expiry date. Instead, a funding rate is exchanged between longs and shorts... to keep the perpetual price anchored to the spot price' and the generic 'buy spot + sell perpetual futures → collect funding rate' explanation restate knowledge Claude already has; not a 3 because the padding is minor and most tokens carry information Claude does not have (exchange schedules, thresholds, OKX endpoints). | 4 / 5 |
Actionability | Mostly executable guidance: `funding_regime`, `annualized_basis`, `carry_trade_pnl`, and `oi_funding_matrix` are valid runnable Python, and the carry-trade execution steps name specific instruments ('Buy spot BTC-USDT on OKX spot market', 'short BTC-USDT-SWAP'). It is not a 5 because the OKX API access section shows endpoints only as comments ('# GET /api/v5/public/funding-rate-history?instId=BTC-USDT-SWAP') without a usable request example, and the historical pattern statistics ('70% probability of a 5-10% correction') are asserted without a way to verify or recompute them. | 4 / 5 |
Workflow Clarity | The carry-trade section has a clear numbered 5-step sequence with an exit condition ('Close both legs when funding rate turns negative or basis compresses'), but the overall analysis workflow — fetch rates, classify regime, check basis, assess carry, produce the output — is never laid out as a sequence and must be inferred from the section order and output template. Validation checkpoints are implicit (e.g. no step to sanity-check data freshness or cross-exchange consistency), which matches the 'steps present but checkpoints missing or implicit' anchor rather than the 'most checkpoints present' anchor above. | 3 / 5 |
Progressive Disclosure | No bundle files exist (references/, scripts/, assets/ are absent), and the single external pointer is `load_skill("okx-market")` for data retrieval — referenced but not clearly signaled as a separate navigable resource. At ~250 lines, substantial content that could live in reference files (signal frameworks, threshold tables, output template) is inlined in SKILL.md; sections are well-organized, which lifts it above a 2, but the monolithic structure without split references fits the 'could be better organized; content that should be separate is inline' anchor. | 3 / 5 |
Total | 14 / 20 Passed |