Skip to content

Reduce the committed review receipt to a minimal attestation - #237

Merged
bigboateng merged 2 commits into
mainfrom
minimal-review-receipt
Aug 21, 2026
Merged

Reduce the committed review receipt to a minimal attestation#237
bigboateng merged 2 commits into
mainfrom
minimal-review-receipt

Conversation

@bigboateng

Copy link
Copy Markdown
Contributor

Summary

  • The artifact committed for a reviewed PR shrinks to a two-field attestation (reviewed_tree, program_fingerprint); the full sealed receipt (round trajectory, kernel receipt chain, final review bytes) is verified in full at seal time and archived only in the local review store.
  • CI verification (review-verified) now performs three deterministic checks: strict attestation decode, program fingerprint recompiled from the base-revision admitted policy, and reviewed-tree recomputation from the PR head.
  • The self-review yield skill is strictly report-only (verdict in conversation, no seal/commit/push); self-review-solve seals and commits the attestation locally but never pushes.

Verification

  • Full repository contract suite (60 tests) plus go vet ./... and go test ./... pass locally.
  • Both yield skill fixture tests pass; the branch self-review converged with the sealed attestation verifying against HEAD.

@bigboateng
bigboateng merged commit c57ae78 into main Aug 21, 2026
18 checks passed
@bigboateng
bigboateng deleted the minimal-review-receipt branch August 21, 2026 08:33
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