Skip to content

Commit 8f01125

Browse files
committed
fix(verify): correct stale Self-Declaration location references
SE Self-Declaration is communicated verbally in the handoff message, not written to TODO.md. Three references in verify/SKILL.md still pointed to TODO.md from the v4.1 era when it was a written block.
1 parent aa1646e commit 8f01125

File tree

1 file changed

+17
-1
lines changed

1 file changed

+17
-1
lines changed

.opencode/skills/verify/SKILL.md

Lines changed: 17 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,11 @@ This skill guides the reviewer through Step 4: independent verification that the
1717

1818
**You never move `.feature` files.** After producing an APPROVED report: update TODO.md `Next:` to `Run @product-owner — accept feature <name> at Step 5.` then stop. The PO accepts the feature and moves the file.
1919

20-
The reviewer produces one written report (see template below) that includes: all gate results, the SE Self-Declaration Audit, the **Reviewer Stance Declaration**, and the final APPROVED/REJECTED verdict. Do not start until the software-engineer has committed all work and communicated the Self-Declaration verbally in the handoff message.
20+
## When to Use (Step 4)
21+
22+
After the software-engineer signals Step 3 is complete and all self-verification checks pass. Do not start verification until the software-engineer has committed all work and communicated the Self-Declaration verbally in the handoff message.
23+
24+
The reviewer produces one written report (see template below) that includes: all gate results, the SE Self-Declaration Audit, the **Reviewer Stance Declaration**, and the final APPROVED/REJECTED verdict.
2125

2226
## Step-by-Step
2327

@@ -159,6 +163,18 @@ If the feature involves user interaction: run the app, provide real input, verif
159163

160164
Record what input was given and what output was observed.
161165

166+
### 8. Self-Declaration Audit
167+
168+
Read the software-engineer's Self-Declaration from the handoff message.
169+
170+
For every **AGREE** claim:
171+
- Find the `file:line` — does it hold?
172+
173+
For every **DISAGREE** claim:
174+
- REJECT — the software-engineer must fix before requesting review again.
175+
176+
Undeclared violations → REJECT.
177+
162178
### 9. Write the Report
163179

164180
```markdown

0 commit comments

Comments
 (0)