Content
62%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The content has a well-sequenced, checkpoint-validated workflow and concrete per-finding rules, but it leans on a verbose Haskell pseudocode Spec, repeats the core guardrails across several sections, and points to a reference.md that is not bundled. Strong on workflow clarity, moderate on conciseness, actionability, and progressive disclosure.
Suggestions
Consolidate the repeated '只发现不定修 / 不盲扫 / 不凭记忆判架构' guardrails into one '守护规则' section and have Failure Behavior/退出条件 reference it, to cut redundancy and tighten conciseness.
Either ship reference.md in a references/ bundle (so the index/finding templates are real one-level-deep material) or inline the actual index.md/finding-NN.md templates directly in the body so the skill is fully actionable without a missing file.
Compress the Haskell Spec to the essential branch shape and move the full ADT/resume definitions into reference.md, keeping the SKILL.md overview lean.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is ~232 lines and assumes Claude's competence in most places (no 'what a PDF is' padding), but it repeats the core guardrails several times — '只发现、不定修' / '只发现不定修' appears ~6 times, and the Haskell Spec plus a separate '守护规则' plus 'Failure Behavior' plus '退出条件' all restate the same scope/no-blind-scan/no-ADR guards. Some tightening would remove redundancy without losing clarity. Not 3 because of this repetition; not 1 because nothing explains basic concepts Claude already knows and the guidance is substantive, not padded fluff. | 2 / 3 |
Actionability | Guidance is concrete in places — '每条发现必须记录 文件:行号 + 具体代码片段', per-dimension thresholds ('每种维度最多报 5 条'), and a defined output directory layout — but the main executable artifact is a Haskell pseudocode Spec rather than copy-paste-ready commands, and the index/finding templates live only in an unreferenced-as-existing reference.md. It instructs well but is not fully copy-paste ready. Not 3 because the spec is pseudocode and templates are external; not 1 because it does give specific, executable per-finding and per-dimension rules. | 2 / 3 |
Workflow Clarity | A clear four-phase sequence (范围收敛 → 扫描 → 定级+产出 → 建议下一步) with explicit checkpoints — ConfirmScope / WholeRepoRefused / ArchDriftNoAdr — and a closing checklist ('退出条件') plus failure/reporting behavior. Validation feedback is built in (resume only accepts the matching reason, else Blocked; report blocked state before retrying). Matches the anchor with explicit validation steps and an error-recovery loop; not 2 because checkpoints are explicit, not implicit. | 3 / 3 |
Progressive Disclosure | Structure is reasonable (sections for workflow, dimension matrix, guards, failure behavior) and it references detail files one level deep (reference.md for templates, shared-conventions.md, adrs/). However reference.md is referenced but not present as a bundle file (no references/ dir and no reference.md exists), and the large Haskell Spec + dimension matrix live inline where the index/finding templates arguably belong in reference.md. Some structure but references are not clearly backed and some split-out content is inline. Not 3 because the referenced detail file does not exist and inline content could be better separated; not 1 because there is real sectioning and one-level-deep references rather than a monolithic wall or nested references. | 2 / 3 |
Total | 9 / 12 Passed |