Skip to content

Fix selected report reassessment provenance - #462

Merged
rcs2153 merged 1 commit into
mainfrom
codex/selected-report-adapter-provenance-fix
Aug 11, 2026
Merged

Fix selected report reassessment provenance#462
rcs2153 merged 1 commit into
mainfrom
codex/selected-report-adapter-provenance-fix

Conversation

@rcs2153

@rcs2153 rcs2153 commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Summary

Fix the selected project-validation report adapter's existing-terminal temporal provenance blocker.

  • use a fresh Core-owned evaluation time for every terminal reassessment
  • compare V3 current-runtime source, fact, and assessment commitments semantically
  • preserve the original durable governance binding and event history
  • expose reassessment time only as transient, redacted provenance
  • fail closed on current semantic drift without report regeneration or skill replay

Governed phase

  • workflow: dg/blocker
  • run: run-1786471531593349000-2
  • approval: approval/run-1786471531593349000-2/fix-approved
  • presentation: presentation/82976575ad0bc7b2
  • approval outcome: granted with persisted presentation proof
  • close: Completed, 39 events, 1 approval, 0 retries, 0 escalations

Validation

  • cargo fmt --all --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • focused selected-adapter tests: 5 passed
  • cargo test -p workflow-cli --test cli: 169 passed
  • cargo test --workspace: passed
  • npm run check:docs
  • git diff --check

Live GitHub, Jira, and DocsCheck probes remained explicitly ignored because opt-in environment flags and live credentials were not supplied. Fixture and injected-provider coverage passed.

Non-goals

No CLI cutover, selected approval envelope, schema change, provider mutation, new persistence, event mutation, reusable reassessment authority, automatic report generation, hosted behavior, or release-posture change.

Next phase

Focused blocker-fix maintainer review. The selected approval adoption envelope remains blocked until that review accepts this fix.

@rcs2153
rcs2153 marked this pull request as ready for review August 11, 2026 22:36
@rcs2153
rcs2153 merged commit c77fac7 into main Aug 11, 2026
7 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.

1 participant