release(v8.4.1): activate the doc-verifier spawn in /audit + /refactor - #90
Merged
Conversation
v8.4.0 shipped agents/doc-verifier.md (the EC-SELFDOC doc/interface-truthfulness verifier) but no skill spawned it — EC-SELFDOC fired only via the inline ac-evaluator duty. This wires the actual spawn so the cross-agent handoff runs at review time: - /audit Step 2 and /refactor Phase 3 Step 6 now invoke simple-workflow:doc-verifier in parallel with their other read-only reviewers, gated on constraints.selfdoc_verification being active AND a documentation / advertised-interface surface being touched (README / *.md / --help / man-page, or a Gate 9 R3/R4 AC). A non-reproducing example or advertised-vs-enforced boundary mismatch is a Critical that feeds the audit aggregation / refactor loop-exit tally; an unrunnable build is a fail-open PASS-WITH-CAVEATS. - agents/doc-verifier.md gains a ## When spawned (input contract) section. - constraints.selfdoc_verification: off now also skips the spawn (no new kill switch). doc-verifier is wired into the two review skills only — /impl Step 15 already grades EC-SELFDOC inline via the ac-evaluator duty, and /create-ticket is authoring-side. Additive + kill-switched; with the switch off or no doc surface touched the review flow is byte-for-byte pre-v8.4.1. Guarded by CT-EV-SELFDOC-6; skill-contracts 824/824, path-consistency 144/144, full sweep green; ac-evaluator <-> ac-evaluator-hi byte-identical. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
TL;DR
v8.4.0 shipped
agents/doc-verifier.md(the EC-SELFDOC doc/interface-truthfulness verifier) but no skill spawned it — EC-SELFDOC fired only via the inlineac-evaluatorduty. v8.4.1 wires the actual spawn so the cross-agent handoff runs at review time.What changed
/auditStep 2 and/refactorPhase 3 Step 6 now invokesimple-workflow:doc-verifierin parallel with their other read-only reviewers, gated onconstraints.selfdoc_verificationbeing active AND a documentation / advertised-interface surface being touched (aREADME/*.md/--help/ man-page, or a Gate 9 R3DESCRIPTION-MATCHES-BEHAVIOR/ R4DOC/INTERFACE TRUTHFULNESSAC)..simple-workflow/scratch/carve-out; a non-reproducing example or advertised≠enforced boundary is a Critical that feeds the/auditStep 3 aggregation and the/refactorStep 7 loop-exit tally; an unrunnable build is a fail-openPASS-WITH-CAVEATS.agents/doc-verifier.mdgains a## When spawned (input contract)section; both review skills' reviewer tables + Binding-rules name doc-verifier.Scope rationale
doc-verifier is wired into the two review skills only.
/implStep 15 already grades EC-SELFDOC inline via theac-evaluatorduty (avoids hot-path duplication);/create-ticketis authoring-side. This follows CLAUDE.md## Modificationssibling-symmetry (both review orchestrators get it).Migration
Non-breaking, no new kill switch —
constraints.selfdoc_verification: offnow also skips the spawn. With the switch off, or no documentation / advertised-interface surface touched, the review flow is byte-for-byte pre-v8.4.1.Verification
tests/test-skill-contracts.sh824/824 (+ CT-EV-SELFDOC-6),tests/test-path-consistency.sh144/144 (doc-verifier already registered), fullrun-all.shALL SUITES PASSED.plugin.json8.4.1== newest CHANGELOG[8.4.1](CT-MODE-14), real ISO date (CT-MODE-13);ac-evaluator↔ac-evaluator-hibyte-identical (CT-EV-MODEL-1).See
CHANGELOG.md[8.4.1].🤖 Generated with Claude Code