spec-review
Skill16 starsChanged 6 days agoMITView on GitHub
Verify an implementation against its OpenSpec artifacts. TRIGGER when: checking completed work against design.md and tasks.md. SKIP: security-specific review (use security-review-checklists); executing tasks (use spec-develop).
47 other projects use a version of this.
Adopted by
Most recently taken up by
- utopia-rise/fmod-gdextensiontheir own version, first seen Nov 9, 2026
- crsmithdev/alephtheir own version, first seen Nov 8, 2026
- maxatwork/skillstheir own version, first seen Nov 8, 2026
- salasebas/skillstheir own version, first seen Nov 8, 2026
- rioffe/specchecktheir own version, first seen Nov 6, 2026
- celestialdust/achilles-skillstheir own version, first seen Nov 1, 2026
Used with
Projects with it also use get-pr-reviews, push-everything and 10 more.
Other files people use alongside this one.
Files found in the same public projects as a version of this one, more often than they turn up anywhere else. Each project counts once, large collections count for less, and this file's own repository is left out.
saffron-healthSkillget-pr-reviews
Get nicely formatted output of the last review comments on a PR. Used internally by other skills.
Also in 3 projects that use this
saffron-healthSkillpush-everything
Stage ALL changes in the repository (not just session changes), commit, and push. Use when you need to commit all pending changes.
Also in 3 projects that use this
NVIDIASkillgenerate-spec
Produce a feature specification covering API changes, backward compatibility, cross-language impact, performance, and a file-level change plan. Use when designing a new feature, planning a significant change, or the user asks to spec something out.
Also in 3 projects that use this
telefrekSkillspec-init
Bootstrap the spec corpus for this project
Also in 3 projects that use this
AWSSkillimplement-spec
Implement the result of /to-spec and /to-tickets in code.
Also in 8 projects that use this
ArvorCoSkillspecify
Create feature specification from natural language description. Use when starting a new feature or creating a spec. Triggers on: create spec, specify feature, new feature spec.
Also in 3 projects that use this
- +2 more from saffron-health
Other versions47 projects use a version of this.
Word for word · 0
None found.
Edited · 0
None found.
Wrote their own version · 47
- microsoft/winappCli★ 1.4ksame name
- utopia-rise/godot-kotlin-jvm★ 995same name
- utopia-rise/fmod-gdextension★ 954same name
- cashew-labs/libretto★ 898same name
- saffron-health/libretto★ 883same name
- liza-mas/liza★ 418same name
- HHU3637kr/skills★ 146same name
- saffron-health/opencode-gui★ 59same name
- agent-ix/quoin★ 20same name
- microsoft/WinAppVSCE★ 18same name
37 more not listed
How matches are found. Word for word: the exact same text. Edited: shares passages with this file (“99% same” means 99% of its five-word phrases also appear here; 10% or more counts). Their own version: similar meaning, or, weakest, the same name. We can’t tell which project had it first. Projects are counted among the 200 most-starred candidates, and mentions are links to the repository we found, up to 50 per site, so both counts are a floor.
In this repository39 other files it gives its agents.
- CLAUDE.mdCLAUDE.md
- agent-commsSkill
- agent-memorySkill
- api-designSkill
- context-engineeringSkill
- database-optimizationSkill
- distillSkill
- docker-templatesSkill
- error-handlingSkill
- frontend-designSkill
- github-actions-templateSkill
- git-operationsSkill
- logging-standardsSkill
- mcp-toolsSkill
- monitoring-observabilitySkill
- mui-stylingSkill
- necessity-firstSkill
- pattern-recognitionSkill
- planning-methodologySkill
- python-patternsSkill
- quality-validationSkill
- react-patternsSkill
- research-methodologySkill
- reverse-engineer-anythingSkill
- security-review-checklistsSkill
- semantic-memory-mcpSkill
- skill-authoringSkill
- sofa-searchSkill
- spec-designSkill
- spec-developSkill
- spec-researchSkill
- spec-workflowSkill
- state-managementSkill
- systematic-debuggingSkill
- testing-strategySkill
- ui-ux-pro-maxSkill
- verification-before-completionSkill
- watch-patternsSkill
- worktree-managementSkill
The file
We split this skill into the rules it gives an agent
9 rules. 0 are common, 9 are distinctive, and 0 are very specific to this project.
0
Common
Most of the 43 edited versions we read keep these.
9
Distinctive
This author's own choices. Worth a look.
0
Very specific
Tied to their project's names, files or tools. Probably skip.
- Common
- Distinctive
- Very specific
- Distinctive: line 72
When acting on review feedback:
- Distinctive: line 75
Disagree with evidence: - if a finding is wrong or conflicts with the spec, reply with the reason (file/line, spec reference) instead of …
- Distinctive: line 76
One fix, one check: - after each change re-run the relevant tests.
- Distinctive: line 82
Load artifacts: - Read design.md, tasks.md, proposal.md
- Distinctive: line 125
Always note which checks were skipped and why.
- Distinctive: line 130
Correctness: Use keyword search + file path analysis.
- Distinctive: line 131
Coherence: Look for glaring inconsistencies, don't nitpick style
- Distinctive: line 132
False positives: Prefer SUGGESTION over WARNING, WARNING over CRITICAL when uncertain
- Distinctive: line 133
Actionability: Every issue must have a specific recommendation with file references
“Common” is measured: kept by at least 6 in 10 of the 43 edited versions of this file we could read. “Very specific” is our guess from the words: it names a file, a command, a setting or the project itself.
Files
OpenSpec Verification
Guide for verifying that implementation matches spec artifacts.
Input Files
| File | Required | Purpose |
{specs_path}/design.md | Yes | Requirements and scenarios to verify |
{specs_path}/tasks.md | Yes | Completion checklist |
{specs_path}/proposal.md | Optional | Original intent reference |
Path Enforcement: The specs_path MUST be .scaffolding/conversations/{UUID}/specs/ where {UUID} is a valid UUID (format: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx). NEVER use descriptive folder names.
.scaffolding/conversations/ is a symlink to <share>/projects/<repo-name>/conversations (paths below are unchanged; always go through the symlink).
Three Verification Dimensions
1. Completeness
Question: Are all tasks done and all requirements covered?
| Check | Method | Issue Level |
All checkboxes marked [x] | Parse tasks.md | CRITICAL if incomplete |
| All requirements have code | Search codebase for keywords | CRITICAL if missing |
| All new files exist | Verify file paths from tasks | CRITICAL if missing |
2. Correctness
Question: Does the code do what the spec says?
| Check | Method | Issue Level |
| GIVEN/WHEN/THEN satisfied | Trace scenario through code | WARNING if divergent |
| Tests cover scenarios | Match test names to scenarios | WARNING if uncovered |
| Edge cases handled | Check error paths in code | WARNING if missing |
| Validation commands pass | Run pytest / npm run validate | CRITICAL if failing |
3. Coherence
Question: Does the code match design decisions?
| Check | Method | Issue Level |
| Design decisions followed | Compare Decisions section to code | WARNING if violated |
| Patterns consistent | Check naming, structure, style | SUGGESTION |
| No undocumented changes | Diff scope vs design scope | WARNING if extra |
| No design deviations | Cross-reference architecture | WARNING if different |
Two-Stage Review
Review in two ordered stages. Do not start stage 2 until stage 1 passes.
| Stage | Question | Basis | Blocks on |
| 1. Spec compliance | Does it build exactly what was asked - nothing missing, nothing extra? | design.md requirements/scenarios, tasks.md | Missing or divergent requirement, undocumented scope |
| 2. Code quality | Is it well built - readable, tested, secure, maintainable? | Project conventions, tests, patterns | Bugs, missing tests, unsafe code; style nits are SUGGESTION |
Why this order: polishing code that implements the wrong thing is wasted work, and quality findings on code that will change after a spec fix are noise. Stage 1 maps to the Completeness + Correctness + Coherence dimensions above; stage 2 is the code-quality pass layered on top. Report stage results separately.
Receiving Code Review (implementer side)
When acting on review feedback:
- Read all items first, then address them in order: CRITICAL, WARNING, SUGGESTION.
- Verify before changing - confirm each finding against the code; do not apply a suggestion blindly.
- Disagree with evidence - if a finding is wrong or conflicts with the spec, reply with the reason (file/line, spec reference) instead of silently ignoring or complying.
- One fix, one check - after each change re-run the relevant tests; do not batch unrelated fixes.
- Ask when unclear - an ambiguous item gets a question, not a guess.
- Report back per item: fixed / disputed (why) / deferred (why). No performative agreement; state what changed.
Verification Process
- Load artifacts - Read design.md, tasks.md, proposal.md
- Check completeness - Parse checkboxes, search for requirement implementations
- Check correctness - Trace each scenario through code, verify test coverage
- Check coherence - Compare decisions to implementation, check patterns
- Generate report - Summarize findings with issue levels
Report Format
## Verification Report
### Summary
| Dimension | Status |
|--------------|---------------------|
| Completeness | X/Y tasks, N reqs |
| Correctness | M/N scenarios pass |
| Coherence | Followed / N issues |
### Critical Issues
| # | Dimension | Issue | File | Recommendation |
|---|-----------|-------|------|----------------|
| 1 | Completeness | Task 2.3 incomplete | - | Complete or mark blocked |
### Warnings
| # | Dimension | Issue | File | Recommendation |
|---|-----------|-------|------|----------------|
| 1 | Correctness | Scenario X not tested | test_foo.py | Add test case |
### Suggestions
- [Pattern deviation details with file reference]
### Assessment
[CRITICAL: N issues | WARNINGS: N | Ready for archive: Yes/No]Graceful Degradation
| Available Artifacts | Checks Performed |
| tasks.md only | Completeness (checkboxes) only |
| tasks.md + design.md | Completeness + Correctness |
| All three | All three dimensions |
Always note which checks were skipped and why.
Verification Heuristics
- Completeness: Focus on objective items (checkboxes, requirement lists)
- Correctness: Use keyword search + file path analysis; don't require certainty
- Coherence: Look for glaring inconsistencies, don't nitpick style
- False positives: Prefer SUGGESTION over WARNING, WARNING over CRITICAL when uncertain
- Actionability: Every issue must have a specific recommendation with file references
Does it work?
Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.
No reviews yet
People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.
- It worked
- Partly
- It didn't
Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.
Your trail
Files you open will collect here, so you can find them again.
Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.
Now do this for your own.
Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.

