✨ Add chain-verifier subagent for parallel review verification#17
Merged
✨ Add chain-verifier subagent for parallel review verification#17
Conversation
Integrate subagent support into the technical-review phase: - Add agents/chain-verifier.md for tracing decisions through artifact chain - Update review skill to spawn parallel chain-verifiers for efficiency - Update review checklist with parallel verification guidance - Document agents in README and CLAUDE.md
Update chain-verifier and review skill to trace from specification onwards, not from discussion/research phases: - Specification is the validated, filtered source of truth - Earlier phases may contain rejected ideas or rough thoughts - This avoids false positives from outdated/filtered content - Aligns with how planning only references specification
Fundamental change to review approach: - Start from plan (tasks + acceptance criteria), not specification - Verify ALL tasks, not sampled requirements - Use spec for context, not as primary source - Check implementation, tests, AND code quality per task - Flag both under-testing AND over-testing - chain-verifier now receives plan task, loads spec context Code quality review as senior architect: - Check project-specific skills for conventions - Apply SOLID, DRY, low complexity principles - Verify modern language idioms used appropriately - Check readability, security, performance
leeovery
added a commit
that referenced
this pull request
Apr 11, 2026
…nals Flagged during PR #246 convention audit. ~53 skill files use bold **If ...:** for prelude and STOP-gate-adjacent conditionals where CLAUDE.md's strict reading would require H4 (#### If). Out of scope for #246 since it pre-dates the work; logged as idea #17 for a separate convention sweep or CLAUDE.md clarification. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
leeovery
added a commit
that referenced
this pull request
Apr 20, 2026
…nits) cmdList previously used the project manifest registry when populated, falling back to a filesystem scan only when the registry was empty. Work units created before the registry existed are therefore invisible to 'manifest list' — and to every knowledge-base consumer (bulk index, status, compact). Real-data test: 9 dirs on disk, 1 registered, only 2 files indexed. Union both sources so downstream tools see every unit that actually exists. Deduped via Set. Dot-prefix and malformed-manifest handling unchanged. Closes deferred-issues #17.
leeovery
added a commit
that referenced
this pull request
Apr 20, 2026
… withdrawn) The Tick data that motivated deferred-issues #17 (9 dirs on disk, 1 in registry) was a one-off project-manifest corruption, not a systemic issue. Migration 031 already populates the registry from the filesystem on first run, and work units created via 'manifest init' register atomically. The registry is the authoritative source — a filesystem scan papering over a corrupted registry masks real problems. Reverts the cmdList union added in bbe53a4. Test covering the union case is dropped. deferred-issues.md entry updated to reflect the misdiagnosis and withdraw the fix.
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
2 participants
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.
Integrate subagent support into the technical-review phase: