Content
71%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 an exemplary lean router: token-efficient, rule-dense, with a well-sequenced workflow and explicit validation and failure-labeling steps. Its main weakness is bundle integrity — the manifest-driven loading scheme and the stage-gate workflow it delegates to reference files that are not present, so the skill cannot be executed as written.
Suggestions
Ship `manifest.yaml` and the `static/` tree (`core/workflow.md`, `core/output-contract.md`, `fragments/task/technical-disclosure.md`, `fragments/task/disclosure-iteration.md`) referenced in sections 1, 3, and 5, or rewrite those sections to point at files that actually exist in the bundle.
If the manifest is intentionally user-generated, state that explicitly (e.g., 'run `scripts/init_patent_project.py` to create manifest.yaml') so the load step is actionable on a fresh project.
Inline a brief checklist of the stage-gate order from `static/core/workflow.md` (source map → terminology ledger → inventories → evidence ledger → invention concept) so the gate sequence is recoverable even if the referenced file is missing.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and imperative throughout — source-ID schemes ("P001... for paper text blocks"), support-state enums, and guardrails ("Never infer inventorship, ownership, unpublished implementation details...") with zero tutorial-style explanation of concepts Claude already knows. Every token carries a rule or a pointer, matching the 5 anchor; there is no padding to push it down to 4. | 5 / 5 |
Actionability | Concrete guidance is present (source ID formats, allowed support states, copy-paste bash commands for validate/build), but the core of the workflow delegates to files that do not exist in the bundle: "Read `manifest.yaml`, then read every file under `always_load`", "complete the stages in `static/core/workflow.md`", and "`static/core/output-contract.md`" — none of these paths resolve, leaving key execution details missing per the 3 anchor. The commands that are present are executable, which keeps it above anchor 2's high-level-hints level. | 3 / 5 |
Workflow Clarity | A clear five-section sequence with explicit validation gates ("Resolve all validation `ERROR` findings. Review every `WARNING` against the source"), stage gates before formal claims, and an explicit failure labeling rule ("Label the result `incomplete draft`"). It falls short of the 5 anchor because the actual gate definitions and quality thresholds live in the missing `static/core/workflow.md` and `static/core/output-contract.md`, so the checkpoints the sequence relies on cannot actually be followed — a material gap beyond "minor validation gaps". | 4 / 5 |
Progressive Disclosure | The router pattern itself is sound — SKILL.md stays an overview and points to `references/draft-schema.md`, `references/disclosure/disclosure_self_check.md`, and named scripts, all one level deep and clearly signaled. However, per the actual bundle structure, four referenced navigation paths (`manifest.yaml`, `static/core/workflow.md`, `static/core/output-contract.md`, `static/fragments/task/*.md`) are absent, so the primary navigation mechanism (manifest-driven loading) is broken rather than merely imperfectly organized, which fits the 3 anchor better than 4. | 3 / 5 |
Total | 15 / 20 Passed |