Content
45%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-structured with a working reference and a sequenced workflow, but it is padded with generic boilerplate and its code examples are fabricated against the actual bundled script, making the seemingly concrete guidance non-executable. Fixing the code examples and trimming boilerplate would materially raise quality.
Suggestions
Rewrite the code/CLI examples to match the real bundled script: import from `scripts.main`, use `PatentClaimMapper.analyze_infringement(patent_claims, product_description, ...)`, and show the actual CLI (`python scripts/main.py --patent-claims ... --product-description ... --output ...`). Remove the nonexistent `parse_claims`/`map_to_product`/`identify_gaps` methods and `scripts/claim_mapper.py`.
Delete the generic boilerplate sections (Output Requirements, Response Template, Error Handling, Input Validation, Key Features, Implementation Details) that restate practices Claude already knows, and merge the duplicated Quick Check and Audit-Ready Commands into one block.
Replace the verbatim description echo in "When to Use" with patent-specific guidance (e.g., limitation-by-limitation comparison, claim chart construction, design-around gap analysis) instead of generic "evidence insight"/"documented fallback path" phrasing.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Noticeably verbose: generic boilerplate sections (Output Requirements, Response Template, Error Handling, Input Validation, Key Features, Implementation Details) restate good-practice concepts Claude already knows, and Quick Check / Audit-Ready Commands duplicate the same two commands. | 2 / 5 |
Actionability | The code examples look concrete but are non-executable fabrications — Quick Start imports `scripts.claim_mapper.ClaimMapper` and calls `mapper.analyze`, Core Capabilities uses `parse_claims`/`map_to_product`/`identify_gaps`, and the CLI uses `scripts/claim_mapper.py --patent/--product`, none of which exist in the bundled `scripts/main.py` (real class `PatentClaimMapper`, method `analyze_infringement`, args `--patent-claims`/`--product-description`/`--output`). | 2 / 5 |
Workflow Clarity | A clear sequenced Workflow and Example run plan are present with explicit checkpoints (validate the request, stop early if out of scope, py_compile check, fallback path), though the steps are generic rather than patent-specific. | 4 / 5 |
Progressive Disclosure | Good structure with a clearly signaled, one-level-deep reference (`references/audit-reference.md`, a real file) and a real bundled script (`scripts/main.py`); minor gaps from inlined generic content that a clean overview-pointer split would remove. | 4 / 5 |
Total | 12 / 20 Passed |