Content
77%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.
The body is highly actionable, with executable commands, a complete typed argument contract, explicit client-resolution decision rules, and a real verification/feedback loop for writes and validation errors. Its weaknesses are structural: the action contract is fully duplicated (table plus per-action sections) and the large inline API reference makes SKILL.md monolithic rather than an overview pointing to one-level-deep reference files.
Suggestions
Remove the duplication between the action summary table and the per-action sections — keep one canonical form of the contract (e.g., the summary table plus only the rules/defaults that need elaboration).
Move the full per-action argument reference into a bundled reference file (e.g., references/actions.md) and keep a compact overview plus the fast-path rules in SKILL.md, with clearly signaled links.
Tighten the repeated guidance about not guessing or probing (the same instruction appears in the fast path, optional discovery, and contract sections) into a single statement.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly dense, useful reference material with no explanation of concepts Claude already knows, but the Complete Action Contract states all 26 actions twice — once in the summary table ("| `tasks.delete` | 删除任务... arguments: `task_id`, `task_ids`, `delete_files` |") and again in per-action sections — which is significant duplication that could be tightened. This sits between the score-4 anchor (only minor trimmable over-explanation) and score 2 (noticeably padded), matching "mostly efficient but could be tightened". | 3 / 5 |
Actionability | Commands are fully executable and copy-paste ready ("python skills/downloader-operation/scripts/mp-downloader.py call --client \"main-qb\" --action tasks.list --arguments '{\"status\":\"downloading\",\"limit\":20}'"), and every action's arguments are specified with types, defaults, enums, and required markers. Specific examples cover the common cases, matching the top anchor. | 5 / 5 |
Workflow Clarity | The workflow is clearly sequenced (resolve client → compose complete arguments → single call → verify), with an explicit Verification section ("After a write, query the smallest relevant state: `tasks.list` for status and properties, `tasks.files` for file priority") and an explicit error-recovery feedback loop ("reports all detectable argument errors in one response... correct every reported field together instead of retrying one field at a time"). Destructive and batch operations are present and verification guidance covers them, so no cap applies. | 5 / 5 |
Progressive Disclosure | The single bundle file `scripts/mp-downloader.py` is real and correctly referenced, and section headers organize the body well, but roughly 190 lines of the 340-line body are an inline per-action API reference (the score-3 anchor's "200 lines of API reference that could be in a separate file"). No reference files exist to split detail into, and the summary table duplicates the detailed sections, leaving structure present but not well split. | 3 / 5 |
Total | 16 / 20 Passed |