Skip to content

feat(agent): add validation review sibling#92

Merged
ncolesummers merged 2 commits into
mainfrom
codex/49-validation-reviewer
Jul 21, 2026
Merged

feat(agent): add validation review sibling#92
ncolesummers merged 2 commits into
mainfrom
codex/49-validation-reviewer

Conversation

@ncolesummers

@ncolesummers ncolesummers commented Jul 14, 2026

Copy link
Copy Markdown
Owner

Summary

  • add the isolated validation-reviewer Eve sibling on openai/gpt-5.6-terra with xhigh reasoning
  • expose narrow digest-bound readers for patches, test plans, validation results, screenshots, and commit-pinned repository files
  • disable generic read, write, shell, web, question, and task tools
  • add the root-only apply_validation_review_result persistence boundary
  • add fixture routing and the discoverable validation-review/issue-49-review eval

Why

This slice puts the model-facing reviewer on top of the deterministic evidence and transition contracts introduced by #91. The sibling can emit typed findings and recommendations but cannot mutate durable state or GitHub.

Stack

  1. feat(validation): persist review evidence and routing #91 — durable evidence, persistence, and routing core
  2. This PR: Eve reviewer, tools, and eval
  3. docs(architecture): propose validation review cycles #93 — ADR and architecture documentation

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

Validation

  • bun run validate
  • bun run build
  • bunx eve info — compile ready, zero diagnostics
  • bunx eve eval --list — validation-review eval discovered
  • bunx eve eval validation-review/issue-49-review — explicitly invoked; local provider execution remains environment-blocked at listen EPERM 127.0.0.1

Part of #49.

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

Configure the Terra xhigh reviewer, narrow evidence and repository tools, root apply boundary, fixture routing, and discoverable Eve eval.

Co-Authored-By: OpenAI Codex GPT-5 <noreply@openai.com>
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:54pm

Request Review

@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-evidence to main July 21, 2026 04:23
@ncolesummers
ncolesummers merged commit 05f9244 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-validation-reviewer 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.

2 participants