Content
52%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 highly actionable, well-sequenced project style guide with concrete templates and a solid review checklist, but it is held back by heavy repetition, a Zod primer that teaches things Claude already knows, and a complete lack of progressive disclosure — everything lives in one ~790-line file with no bundle resources.
Suggestions
Split the body into reference files (e.g., references/zod-patterns.md for the Zod method catalog and tag rules, references/examples.md for worked examples) and keep SKILL.md as a concise overview with well-signaled links.
Deduplicate: show the API header-comment template once and reference it thereafter; merge the duplicate meta good/bad comparison (字段定义规范 vs 问题 3) into a single occurrence; drop the annotated Zod basics primer.
Resolve the parseApiInput inconsistency — either show parseApiInput in the route template and worked example, or drop it from the mandatory rules and checklist in favor of Schema.parse(req.body).
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The ~790-line body is noticeably verbose with several padded sections: the API header-comment template is repeated verbatim four times (step 1, multi-API example, 快速参考, 问题 1); the meta good/bad comparison appears twice (字段定义规范 and 问题 3); and the '常用 Zod 验证方法' section annotates basics Claude already knows ("z.string() // 字符串", ".min(2) // 最小长度", "z.number() // 数字"). Not score 1 because the material is not concept-explanation padding throughout — most sections convey project-specific rules; not score 3 because the repetition and the Zod primer alone represent well more than 'minor' trimming opportunities. | 2 / 5 |
Actionability | Guidance is highly executable: exact file locations ("packages/global/openapi/[module]/[api].ts", "projects/app/src/pages/api/[path]/[route].ts"), copy-paste route templates with imports, a complete worked create-user example, and a structured review checklist. Not score 5 because the mandatory rule "实际存在的入参必须使用 parseApiInput 验证" references a helper that never appears in any code example (examples use GetAppChatLogsBodySchema.parse(req.body) instead), leaving a key detail unresolved. | 4 / 5 |
Workflow Clarity | A clear four-step development sequence (define schema → implement route → auth → error handling) with 🔴 mandatory checkpoints, a blocking review checklist, and a data-cleaning section that mandates dry-run by default with explicit confirmation plus success/skip/failure stats — a real feedback loop for the riskiest batch operations. Not score 5 because there is no run-and-verify step after implementing a route (e.g., typecheck or test the endpoint) to close the loop. | 4 / 5 |
Progressive Disclosure | The skill is a monolithic single file: no references/, scripts/, or assets/ exist, and ~790 lines of Zod API reference, tag catalogs, and worked examples that clearly belong in separate reference files are inlined. It also points to paths that are not part of the bundle ("/Volumes/code/fastgpt-pro/FastGPT/..." machine-specific paths and ".claude/skills/pr-review/..."). Not score 1 because the body has clear section headers and is navigable; not score 3 because essentially all detail content is inline rather than just some. | 2 / 5 |
Total | 12 / 20 Passed |