Noticed during a peer-review pass on main @ 6a40258.
Per AGENTS.md rule 6, after an implementing PR merges its SpecSync change should be archived (specsync change archive <id> from a clean main-based checkout, small housekeeping PR), and accepted changes should not pile up in .specsync/changes/.
The archive routine covered CHG-0037 (#81), CHG-0039 (#94), and CHG-0040 (#96), but CHG-0038 (the change landed by #91) was skipped:
.specsync/changes/CHG-0038-harden-adversarial-findings-... is still present on main
- its
state.json still says "state": "implementing" (never even transitioned to accepted)
specsync change list reports "No active SDD changes" (5.2.0 squash-merge evidence), so the archive preflight should pass; the change may need the acceptance step recorded first given the stale state field
Suggested fix: run the acceptance/archive flow for CHG-0038 from a clean main-based checkout and push it as a small housekeeping PR, same as the #94/#96 routine.
Noticed during a peer-review pass on main @ 6a40258.
Per AGENTS.md rule 6, after an implementing PR merges its SpecSync change should be archived (
specsync change archive <id>from a clean main-based checkout, small housekeeping PR), and accepted changes should not pile up in.specsync/changes/.The archive routine covered CHG-0037 (#81), CHG-0039 (#94), and CHG-0040 (#96), but CHG-0038 (the change landed by #91) was skipped:
.specsync/changes/CHG-0038-harden-adversarial-findings-...is still present on mainstate.jsonstill says"state": "implementing"(never even transitioned to accepted)specsync change listreports "No active SDD changes" (5.2.0 squash-merge evidence), so the archive preflight should pass; the change may need the acceptance step recorded first given the stale state fieldSuggested fix: run the acceptance/archive flow for CHG-0038 from a clean main-based checkout and push it as a small housekeeping PR, same as the #94/#96 routine.