Skip to content

[codex] Require structured headless review results#1

Draft
romgenie wants to merge 3 commits into
mainfrom
codex/headless-review-contract-v2
Draft

[codex] Require structured headless review results#1
romgenie wants to merge 3 commits into
mainfrom
codex/headless-review-contract-v2

Conversation

@romgenie

@romgenie romgenie commented Jul 6, 2026

Copy link
Copy Markdown

What changed

Adds headless contract v2 for issue #119.

  • Bumps the headless contract schemas and fixtures to version 2.
  • Requires a structured review object in runtime results.
  • Adds typed adapter-side review result models for mode, evidence status, reviewed files, findings, tests run, limitations, and no-findings rationale.
  • Updates worker/API tests that deserialize or construct SessionResult.

Why

Dogfooding showed that hosted review evidence needs to be preserved as structured data. The adapter can prove a PR checkout and file list were supplied, but contract v1 had no required field for that proof in result.json.

Validation

  • cargo test -p coven-github-worker contract
  • cargo test -p coven-github-api
  • cargo test -p coven-github-worker
  • cargo test

romgenie added 3 commits July 5, 2026 20:54
Signed-off-by: Timothy Wayne Gregg <Timothy.Gregg@complete.tech>
Signed-off-by: Timothy Wayne Gregg <Timothy.Gregg@complete.tech>
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.

1 participant