Skip to content

docs(architecture): propose validation review cycles#93

Merged
ncolesummers merged 2 commits into
mainfrom
codex/49-review-agent
Jul 21, 2026
Merged

docs(architecture): propose validation review cycles#93
ncolesummers merged 2 commits into
mainfrom
codex/49-review-agent

Conversation

@ncolesummers

Copy link
Copy Markdown
Owner

Summary

  • update Proposed ADR 0015 with validation-owned screenshot evidence and the validation-review sibling
  • document typed findings, exact evidence citations, and root-only routing
  • define bounded development and test-writing cycles over reused stage rows
  • update architecture and loop-manifest guidance for the new artifact contracts

Why

The implementation introduces a meaningful orchestration policy: validation owns responsive evidence, the reviewer remains advisory, and only the root can advance or requeue durable stages. This final slice records that policy independently from the runtime changes.

Stack

  1. feat(validation): persist review evidence and routing #91 — durable evidence, persistence, and routing core
  2. feat(agent): add validation review sibling #92 — Eve reviewer, tools, and eval
  3. This PR: ADR and architecture documentation

Merge in order. After #92 merges, retarget this PR from codex/49-validation-reviewer to main.

Validation

  • bun run validate
  • bun run build
  • markdownlint included in the aggregate gate

Closes #49.

Co-authored with OpenAI Codex (GPT-5).

Update ADR 0015, architecture, and loop-manifest guidance for validation-owned screenshots and root-controlled backward routing.

Co-Authored-By: OpenAI Codex GPT-5 <noreply@openai.com>
@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
loopworks Ready Ready Preview, Comment Jul 14, 2026 5:56pm

Request Review

Co-Authored-By: OpenAI Codex GPT-5 <noreply@openai.com>
@ncolesummers
ncolesummers marked this pull request as ready for review July 16, 2026 15:35
@ncolesummers
ncolesummers changed the base branch from codex/49-validation-reviewer to main July 21, 2026 04:24
@ncolesummers
ncolesummers merged commit 21e7ced into main Jul 21, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Loopworks Backlog Jul 21, 2026
@ncolesummers
ncolesummers deleted the codex/49-review-agent branch July 21, 2026 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Validation review subagent for the code review stage

2 participants