Skip to content

feat(validation): verify expected run observations#75

Merged
morluto merged 6 commits into
mainfrom
codex/validation-observations
Jul 21, 2026
Merged

feat(validation): verify expected run observations#75
morluto merged 6 commits into
mainfrom
codex/validation-observations

Conversation

@morluto

@morluto morluto commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • adds expected observations to validation definitions and persisted run results
  • evaluates those observations against base and candidate artifacts
  • exposes observation results through CLI and MCP output without executing corpus reads

PR #68 merged into its stacked base after that base had already landed on main. This PR carries the resulting validation-only diff onto main.

Closes #48

Review order

  1. Evidence contracts and observation evaluation
  2. Corpus persistence and artifact safety
  3. Application, CLI, and MCP integration

Verification

  • go test ./... -count=1
  • go test ./internal/evidence ./internal/corpus ./internal/app ./internal/cli ./internal/mcpserver -count=1

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@morluto
morluto merged commit e9181a6 into main Jul 21, 2026
14 checks passed
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.

[Feature] Verify expected symptoms in base/candidate validation

1 participant