Content
78%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.
Excellent actionability and a well-validated workflow for a risky refactor operation, written almost entirely from repo-specific knowledge. The weaknesses are structural: no progressive disclosure into reference files, some triplication of the step list, and an implicit rather than explicit failure-recovery loop.
Suggestions
Move the 详细步骤 sections into a reference file (e.g. references/steps.md) and keep SKILL.md to the TODO checklist, 核心理念 table, and verification checklist — this would cut the always-loaded body substantially.
State the error-recovery loop explicitly after step 8 (e.g. 'if typecheck fails on import paths, re-check the ../ depth in the moved files and rerun') to convert the implicit retry into a feedback loop.
Deduplicate the three restatements of the step list (TODO template, detailed steps, anti-patterns) — for example, fold the anti-pattern column of the 核心理念 table into the 反模式 section.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Almost every token carries repo-specific knowledge Claude cannot infer (exact paths, moduleTemplatesFlat semantics, the dead `abandon` field warning, shared-child-node caveats). The score sits below 5 only because the 核心理念 table, TODO template, and 详细步骤 restate the same 8 steps three times at different altitudes. | 4 / 5 |
Actionability | Fully executable throughout: copy-paste grep commands with --include filters, exact source/target paths for git mv, before/after code diffs for constants.ts changes, the SoonOffline enum snippet, and pnpm lint/test commands. The <enumKey> placeholders are justified by the explicit fill-in TODO template. | 5 / 5 |
Workflow Clarity | An 8-step TODO template, matching detailed steps, a final verification checklist (template panel search, old-workflow load/run, badge rendering, lint), and a lint/typecheck/partial-test step give clear checkpoints. It misses 5 only because the error-recovery loop (what to do when lint/typecheck or a checklist item fails) is implicit rather than stated. | 4 / 5 |
Progressive Disclosure | A ~205-line monolithic SKILL.md with no bundle files at all. Sections are clearly organized, but the eight detailed step sections with code examples are exactly the deep material that belongs in a one-level reference file, leaving SKILL.md as the overview plus TODO checklist. | 3 / 5 |
Total | 16 / 20 Passed |