Content
85%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 a well-sequenced, highly actionable adversarial review procedure with explicit validation checkpoints, a finding format, and a strict verdict rule. Its main weakness is progressive disclosure: a large inlined principle-ID index duplicates the constitution file it points to, and the referenced files (e.g., review-constitution.md) live outside the skill bundle. Trimming the lens index to priorities and keeping detail in the constitution would tighten it further.
Suggestions
Reduce the 'Lens Priority for Spec Reviews' section to the priority ordering and rationale, leaving full principle-ID enumeration to the constitution file it already directs the reader to load.
Move the lens-by-lens ID checklists into a reference file in the skill bundle (e.g., references/lens-index.md) so SKILL.md stays an overview with one-level-deep references.
Ship the referenced files (review-constitution.md, documentation-style.md, spec-template.md) inside the skill bundle or note that they are repo-external, so the skill is self-contained when distributed.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, directive prose with no concept explanations Claude already knows ('The spec is wrong until proven right', 'This prevents anchoring bias' are brief purposeful rationales). Not 5 because the ~60-line lens-priority ID index partially duplicates the constitution that Step 1 already requires reading in full, and could be trimmed to priorities only. | 4 / 5 |
Actionability | Guidance is copy-paste ready for an instruction skill: an exact finding-format block ('[PRINCIPLE-ID] Severity: ... Location: ... Evidence: ...'), an exact output-structure markdown template, a binary verdict rule (PASS/BLOCK with explicit anti-hedging), and a concrete sampling rule ('minimum 3 or 20%, whichever is larger'). Everything needed to execute is specified; not 4 because there are no material gaps. | 5 / 5 |
Workflow Clarity | Eight clearly sequenced steps with explicit validation checkpoints: an anchoring-bias countermeasure (build an independent model before reading the spec), codebase verification of claims with a failure escalation rule ('If ANY verification fails, flag ALL unverified claims as suspect'), a pre-verdict completeness checklist, and a binary verdict. This matches the feedback-loop/checklist anchor exactly; not 4 because checkpoints are explicit at every stage. | 5 / 5 |
Progressive Disclosure | References are clearly signaled (read '.opencode/rules/review-constitution.md' in full; '.opencode/rules/documentation-style.md'; '.opencode/skills/ideate/spec-template.md'), but the bulk of the body is an inlined index of principle IDs (AMB-01..CPX-07) that duplicates detail belonging in the constitution file. Not 4 because roughly a third of the body is inlined material that belongs in the referenced file; not 2 because structure and navigation are otherwise sound and references are explicit, not buried. | 3 / 5 |
Total | 17 / 20 Passed |