Skip to content

feat: integrate review suite into epic execution - #17

Merged
shaug merged 1 commit into
mainfrom
scott/issue-10-integrate-review-suite
Jul 20, 2026
Merged

feat: integrate review suite into epic execution#17
shaug merged 1 commit into
mainfrom
scott/issue-10-integrate-review-suite

Conversation

@shaug

@shaug shaug commented Jul 20, 2026

Copy link
Copy Markdown
Owner

TL;DR

Routes epic implementation through the repository-owned review suite while preserving strict validation, remote-review, merge, and cleanup gates.

Summary

  • Make review-code-change the single local adversarial-review dependency and fail closed when it or its evidence is unavailable.
  • Keep implementation, fix application, validation, publishing, CI, human and connector review, thread disposition, merge, and cleanup in the epic workflow.
  • Remove duplicated review-rubric ownership and consume the suite's aggregate verdict, findings, dispositions, limitations, and re-review instructions.
  • Replace universal base-advance invalidation with recorded merge-candidate analysis that retains evidence only for unchanged, non-overlapping candidates when repository policy permits it.
  • Add seven forward-tested integration scenarios covering fix cycles, redesign, targeted correctness, both base-drift paths, missing dependency, and separate remote policy gates.

Tickets

Fixes #10

## Summary

- Replace the epic workflow's third-party review dependency with the repository-owned review-code-change suite
- Keep fix application, validation, publishing, remote policy, merge, and cleanup mechanics in the epic workflow while removing duplicated lens judgment
- Apply risk-based base-drift invalidation and add seven forward-tested integration scenarios

## Why

- Give epic execution one stable local review interface without weakening CI, human, connector, thread, or merge gates
- Retain trustworthy head-bound evidence across unrelated base drift while restarting affected gates when the effective candidate changes
@shaug
shaug merged commit bee27f7 into main Jul 20, 2026
1 check passed
@shaug
shaug deleted the scott/issue-10-integrate-review-suite branch July 20, 2026 04:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Integrate the repository-owned review suite into implement-epic-sequence

1 participant