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.
The body is well-sectioned with a clear process, constraints, and failure-recovery handling, but it functions as an island: it never tells Claude how to actually run the skill's script pipeline, and its only file pointer (config.yaml) is missing from the bundle. Redundant policy text also dilutes token efficiency.
Suggestions
Add a '运行入口' section that points to `scripts/skill_controller.py` (and `references/README.md`) with one concrete invocation example, e.g. `python scripts/skill_controller.py --project <path> --output-mode preview`, so the existing script bundle is discoverable from SKILL.md.
Ship `config.yaml` in the bundle or inline the key default values, since the body defers all defaults and config keys (`parameters.*`, `generation.*`, etc.) to a file that is currently absent.
Deduplicate the bug-reporting rules (stated in 执行步骤 and again in 公共硬约束) and the task-directory convention (stated in both 输出管理 and 公共硬约束) to reclaim tokens and reduce drift between the two copies.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Mostly efficient bullet-style prose with no explanations of concepts Claude already knows, but noticeably redundant: the bug-reporting rules appear twice (执行步骤 bullets and again in 公共硬约束) and the task-directory convention is stated in both 输出管理 and 公共硬约束. Not 4 because this repetition and the boilerplate constraint block could be meaningfully tightened. | 3 / 5 |
Actionability | Provides some concrete guidance — parameter enums (`minimal`/`moderate`/`comprehensive`, `preview`/`apply`/`report`), output directory layout, and section-command whitelist rules — but no executable commands or code anywhere. The body never says how to invoke the 18-script bundle (e.g., `scripts/skill_controller.py`) and defers all defaults to a `config.yaml` that is not in the bundle, so key execution details are missing. Not 4 because there is no copy-paste-ready instruction; not 2 because inputs, outputs, and constraints are concretely specified. | 3 / 5 |
Workflow Clarity | A clear 7-step sequence with validation checkpoints present: step 7 ("执行格式与结构验证,需要时生成质量报告"), a dedicated 校验 section, blacklist access control that must refuse illegal modifications, and a 失败与恢复 section for retry from the last failed step. Not 5 because the validation commands/checks are never named and there is no explicit validate→fix→re-validate loop; validation exists, so the destructive/batch cap of 3 does not apply. | 4 / 5 |
Progressive Disclosure | The bundle contains 18 Python scripts and a references/README.md, yet the body references none of them — no pointer to the main controller `scripts/skill_controller.py` or to references/README.md. The only file reference, `config.yaml`, does not exist in the bundle, making it a dangling reference. This matches anchor 2 (references buried/absent, content split not navigable) rather than 3, where references would at least be present but weakly signaled. | 2 / 5 |
Total | 12 / 20 Passed |