Content
92%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 well-engineered skill body: executable commands with flag-by-flag guidance, an explicit error-recovery table, honest output caveats, and a clean three-file reference structure that is verifiably present in the bundle. The only flaw found is a dangling reference to a `tests/` directory that is not shipped, which alone keeps actionability just below perfect.
Suggestions
Ship the `tests/` directory alongside the skill or remove the "Tests: python3 -m unittest discover -s tests -v" line from Resources, since running that command in the current bundle fails.
implementation.md also asserts "`tests/` checks that" no order/account code exists — if tests are dropped, reword that guarantee so the reference and the bundle stay consistent.
Consider moving the fee-source dates ("dates and sources in `arb_finder.py venues`") fully into references/implementation.md so the overview only carries the pointer, trimming a few tokens from the Output section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body assumes Claude's competence — it never explains what arbitrage, order books, or exchanges are — and every section adds skill-specific information: exact commands with flag semantics, fee-source caveats ("KuCoin and Coinbase do not show a base-tier fee without a login, so their net column reads `n/a`"), and an exclusion list. This matches anchor 5 ('lean and efficient; every token earns its place'); anchor 4 would require over-explanation that could be trimmed, and none is evident. | 5 / 5 |
Actionability | Every command is copy-paste ready with real flags, sample output, and an offline `--demo` path, covering scan, fees, sizing, triangular, and monitor use cases — near anchor 5. However, the Resources section points to a test suite ("python3 -m unittest discover -s tests -v") whose `tests/` directory is absent from the bundle, a minor gap that fits anchor 4 ('concrete commands with minor gaps') better than 5. | 4 / 5 |
Workflow Clarity | The numbered Instructions sequence each operation unambiguously, and the Error Handling table provides explicit feedback loops — message → cause → what to do (e.g., "no confirmed fee for kucoin" → "Pass `--fee kucoin=PERCENT`") — plus exit-code semantics and a reporting contract. For a read-only skill with no destructive or batch operations, this matches anchor 5's explicit validation and error-recovery structure; anchor 4's 'minor validation gaps' is not supported. | 5 / 5 |
Progressive Disclosure | SKILL.md is a clean overview whose details are appropriately split into three one-level-deep references, all present in the bundle and clearly signaled inline ("More in [`references/errors.md`]", "Details: [`references/implementation.md`]", full runs in `references/examples.md`) and again in a Resources section. This matches anchor 5; no content that belongs in a reference is inlined and no nested references exist. | 5 / 5 |
Total | 19 / 20 Passed |