Content
86%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 and well-structured, giving concrete path mappings, skip rules, commands, and a sequenced workflow with a retry loop. Its main weakness is an inconsistent coverage target (100% vs 90%) that slightly muddies the validation checkpoint.
Suggestions
Reconcile the coverage requirement: pick one authoritative threshold (rule 2's 100% or the verification step's 90%+) and state it consistently in both places.
Add one complete minimal .test.ts example (a small function plus its describe/it blocks) so the writing pattern is concrete rather than only described.
Tighten rule 7 and remove the duplicated trigger line in 'When to Use This Skill' to recover a few tokens and reach full conciseness.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes competence: direct rules, executable commands, and short code snippets with no padding about what unit tests are. It is not a 5 because a few rules (e.g. rule 7 on global mocks) and the duplicate trigger line could be tightened, but it is well above the 3 anchor that tolerates unnecessary explanation. | 4 / 5 |
Actionability | Guidance is copy-paste ready: explicit source-to-test path mappings, real import examples via '@fastgpt', concrete skip-lists (type.ts/constants.ts/schema.ts), and runnable 'pnpm test <file-path> <test-name>' commands. This matches 'Fully executable; copy-paste ready; specific examples cover the common cases.' | 5 / 5 |
Workflow Clarity | A clear three-phase sequence (任务准备 → 测例编写 → 结果验证) with an explicit feedback loop ('如需二次修改,则回到二、测例编写') and a coverage checkpoint is present. It is not a 5 because the validation threshold is contradictory (rule 2 says 100% line/branch coverage while the verification step says 90%+), leaving a minor gap about which checkpoint is authoritative. | 4 / 5 |
Progressive Disclosure | The single-file skill is well-organized into clear sections (位置/规则/流程/场景/命令) with one-level-deep references to real repo files (vitest.config.mts, test/mocks/index.ts) and no content that clearly belongs in a separate file. With no bundle to split and good navigation, this matches the well-organized single-file case. | 5 / 5 |
Total | 18 / 20 Passed |