Content
70%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 well-structured orchestrator skill: the phase workflow is clearly sequenced with explicit checkpoints, validation rounds, and state-persistence recovery, and sub-skill invocations are concrete. Its weaknesses are redundancy (duplicated pipeline diagrams, repeated stop-boilerplate), an unverifiable external reference for the reviewer protocol, and inlined detail (patent-type and jurisdiction-format tables) that would fit better in reference files.
Suggestions
Remove one of the two pipeline diagrams (the sub-skill chain and the idea-discovery track diagram show the same pipeline) and state the "STOP HERE and wait for user confirmation" rule once in Key Rules instead of repeating the full boilerplate at every phase checkpoint.
Move the reviewer-routing blockquote protocol into the referenced reviewer-backends file (kept as a one-line pointer), and relocate the patent-type specification tables and per-jurisdiction deliverables table into a references/ file so SKILL.md stays a lean overview.
Make the reviewer slot self-contained: either verify `../shared-references/reviewer-backends.md` ships with the bundle or inline one concrete example invocation per reviewer route, since the current blockquote defers all executable detail to an unresolvable path.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly task-specific (phase structure, patent rules, state schema), but the pipeline is diagrammed twice (the sub-skill chain and the idea-discovery track diagram convey overlapping information), the "⛔ STOP HERE and wait for user response. Do NOT auto-proceed unless AUTO_PROCEED=true" boilerplate is repeated at each phase, and the opening reviewer-routing blockquote is dense meta-protocol text. It fits 'mostly efficient but could be tightened' — not 2 because no space is spent explaining concepts Claude already knows, and not 4 because the duplication and repeated boilerplate are trimmable. | 3 / 5 |
Actionability | Concrete, copy-paste-ready sub-skill invocations with exact arguments (e.g., `/prior-art-search "patent/INVENTION_BRIEF.md"`), exact output paths, a concrete PATENT_STATE.json schema, and ready-to-fill checkpoint templates. It falls short of 5 because the reviewer-slot instructions delegate all operational detail to an external reference and the checkpoint templates remain placeholder outlines rather than worked examples. | 4 / 5 |
Workflow Clarity | Phases 0-5 are clearly sequenced, each phase has an explicit checkpoint ("⛔ STOP HERE and wait for user response") with go/adjust/stop options, an explicit validation feedback loop exists (2 examiner review rounds via `/patent-review` bounded by MAX_REVIEW_ROUNDS with revision cycles), and state persistence with resume/staleness rules supports error recovery. This matches the top anchor: clear sequence, explicit validation, feedback loops, and recovery handling. | 5 / 5 |
Progressive Disclosure | There are no bundle files (no references/, scripts/, or assets/), and the single file reference in the body (`../shared-references/reviewer-backends.md`) points outside the bundle and does not exist in the workspace, so it cannot be verified. Material that could live in separate files — the patent-type specification tables, the jurisdiction deliverables table, and the reviewer-routing protocol — is inlined in one ~340-line SKILL.md. Sections are well-organized with clear headers (above anchor 2), but the one reference is not resolvable within the bundle and separable content is inline (below anchor 4). | 3 / 5 |
Total | 15 / 20 Passed |