Content
88%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 body: fully executable discovery and example commands, a coherent seven-step workflow with strategy-awareness and a genuine failure-recovery loop, and verified one-level-deep references. The main weaknesses are mild redundancy — an oversized site table with many content-free '(see --help)' rows that duplicates the reference file, a repeated write-command list, and one non-executable template block in Step 2.
Suggestions
Trim the Step 3 site table to the sources with known commands and point the '(see --help)' tail at references/finance-sources.md or `opencli list -f json`, removing ~20 rows that duplicate the reference file and the description.
State the write-command exclusion once (e.g., keep only Key rule 5's enumerated list) and reference it from the read-only banner, and rewrite the Step 2 readiness check as a plain executable bash block instead of the `!`(...)` template artifact.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Efficient and dense — commands, tables, and rules with essentially no explanation of concepts Claude already knows. Kept below 5 by padding: roughly twenty site-table rows carry no information beyond a slug ("(see `--help`)"), the source list duplicates both the frontmatter description and references/finance-sources.md, and the write-command list is stated twice (the read-only banner and Key rule 5). | 4 / 5 |
Actionability | Copy-paste-ready commands throughout ("opencli yahoo-finance quote AAPL -f json", "opencli reddit subreddit wallstreetbets --limit 20 -f json"), a strategy/precondition table, universal-flag table, and twelve concrete examples covering the common cases, plus exact discovery commands to resolve the rest. The Step 2 readiness check uses a non-executable template artifact rather than plain bash, but the embedded command is directly runnable — a blemish, not a gap that drops it to anchor 4's 'minor gaps'. | 5 / 5 |
Workflow Clarity | A clear seven-step sequence (decide → ensure ready → discover command → check strategy → execute → handle failures → present) with explicit checkpoints: readiness via `opencli doctor`, strategy check before browser-backed adapters, small `--limit` validation runs, and a full failure feedback loop (OPENCLI_DIAGNOSTIC=1 → autofix skill / file an issue / never scrape ad-hoc, plus rate-limit retry). The skill is read-only, so the destructive/batch validation cap does not apply. | 5 / 5 |
Progressive Disclosure | Good structure: the body is a step-wise overview and the two reference files (references/discovery.md, references/finance-sources.md) exist, are one level deep, and are clearly signaled with an annotated description of each. Not a 5 because the 33-row site table inlines per-site detail that overlaps references/finance-sources.md — content that should live in the separate file is partially inline in SKILL.md. | 4 / 5 |
Total | 18 / 20 Passed |