Skip to content

fix(pre-open): stamp artifact generator provenance - #463

Merged
Wibias merged 3 commits into
mainfrom
fix/preopen-artifact-provenance
Sep 20, 2026
Merged

Wibias merged 3 commits into
mainfrom
fix/preopen-artifact-provenance

Conversation

@Wibias

@Wibias Wibias commented Sep 20, 2026

Copy link
Copy Markdown
Owner

Summary

Stamp full and compact pre-open gate artifacts with the github-delivery package version plus SHA-256 digests of the gate and review-scope implementations that produced them.

Trace evidence

The newest Grok run inherited an old preopen-full.json where large_diff still implied incomplete review. Current source had already changed that semantic. Because the artifact carried no generator provenance, Grok had to re-read implementation files and reverse-engineer why the saved JSON disagreed with current behavior.

Fix

Every newly emitted pre-open artifact includes:

"generator": {
  "githubDeliveryVersion": "...",
  "preOpenGateDigest": "sha256:...",
  "reviewScopeDigest": "sha256:..."
}

Compact summaries preserve the same identity.

This does not trust old artifacts more; it makes stale/unknown-generation artifacts immediately recognizable.

No merge or release publication is authorized by this PR.

@coderabbitai

coderabbitai Bot commented Sep 20, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 65d24b86-21f0-42d2-963a-c0e54124a784


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Wibias
Wibias marked this pull request as ready for review September 20, 2026 22:43
@Wibias
Wibias enabled auto-merge September 20, 2026 22:43
@Wibias
Wibias merged commit 2efb556 into main Sep 20, 2026
13 checks passed
@Wibias
Wibias deleted the fix/preopen-artifact-provenance branch September 20, 2026 22:47
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