Skip to content

evidence(OMN-12564): refresh omniclaude push DoD binding - #2329

Merged
jonahgabriel merged 6 commits into
devfrom
jonah/omn-12564-omniclaude-push-dod-evidence
Jun 8, 2026
Merged

jonahgabriel merged 6 commits into
devfrom
jonah/omn-12564-omniclaude-push-dod-evidence

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented Jun 8, 2026 •

Copy link
Copy Markdown
Contributor

Evidence-Ticket: OMN-12564
Evidence-Source: OCC-local

Refreshes the central OMN-12564 evidence binding for omniclaude PR #1735 after the push-event DoD resolver fix.

Verification:

  • uv run validate-yaml contracts/OMN-12564.yaml
  • PYTHONPATH=/Users/jonah/Code/omni_home/omnibase_core/src uv run python -m omnibase_core.validation.validator_receipt_honesty --receipts-dir drift/dod_receipts drift/dod_receipts/OMN-12564/dod-omniclaude-pr-1735-main-hotfix/command.yaml
  • omniclaude: uv run pytest tests/workflows/test_deploy_gate_reusable_workflow.py tests/scripts/test_checkout_occ_contracts.py -q

Summary by CodeRabbit

  • Chores
    • Updated Definition of Done evidence verification infrastructure to align push and pull_request CI lanes to the same authoritative evidence source.
    • Refreshed verification records with updated execution metadata and checksums across multiple evidence receipts.
    • Added new verified evidence receipt for pull request 2329.

@coderabbitai

coderabbitai Bot commented Jun 8, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: abc9656c-e7ee-4975-9b03-4ce0a3df02c3

📥 Commits

Reviewing files that changed from the base of the PR and between a3e5137 and bc23ed6.

📒 Files selected for processing (11)
  • contracts/OMN-12564.yaml
  • drift/dod_receipts/OMN-12564/dod-infra-pr-1824/command.yaml
  • drift/dod_receipts/OMN-12564/dod-infra-pr-1825/command.yaml
  • drift/dod_receipts/OMN-12564/dod-local-env-proof/command.yaml
  • drift/dod_receipts/OMN-12564/dod-occ-cache-pr-self/command.yaml
  • drift/dod_receipts/OMN-12564/dod-occ-pr-2039-omniclaude-binding/command.yaml
  • drift/dod_receipts/OMN-12564/dod-occ-pr-2326-omniclaude-main-hotfix-binding/command.yaml
  • drift/dod_receipts/OMN-12564/dod-occ-pr-2329-omniclaude-push-dod-binding/command.yaml
  • drift/dod_receipts/OMN-12564/dod-occ-pr-self/command.yaml
  • drift/dod_receipts/OMN-12564/dod-omniclaude-pr-1718-checkout-hardening/command.yaml
  • drift/dod_receipts/OMN-12564/dod-omniclaude-pr-1735-main-hotfix/command.yaml

📝 Walkthrough

Walkthrough

This PR updates the OMN-12564 ticket definition and re-verifies its Definition of Done evidence. The ticket summary is clarified to note that central OCC DoD evidence resolves from live PR push events. Eight existing evidence receipts are re-bound to fresh contract digests, a new PR 2329 evidence receipt is registered, and PR 1735's evidence outputs are updated with new execution metadata and clarified resolver behavior.

Changes

OMN-12564 DoD Evidence and Contract Updates

Layer / File(s) Summary
Contract Definition and Evidence Registry
contracts/OMN-12564.yaml
Ticket summary extended to clarify that central OCC DoD evidence resolves from live PR push events with push and pull_request CI lanes aligned to the same source. Commit hash for PR 1735 evidence updated. New evidence receipt entry dod-occ-pr-2329-omniclaude-push-dod-binding registered with status: PASS and pr_number: 2329 assertions.
Evidence Receipt Hash Rebindings
drift/dod_receipts/OMN-12564/dod-infra-pr-1824/command.yaml, dod-infra-pr-1825/command.yaml, dod-local-env-proof/command.yaml, dod-occ-cache-pr-self/command.yaml, dod-occ-pr-2039-omniclaude-binding/command.yaml, dod-occ-pr-2326-omniclaude-main-hotfix-binding/command.yaml, dod-occ-pr-self/command.yaml, dod-omniclaude-pr-1718-checkout-hardening/command.yaml
Contract SHA-256 hash bindings updated across 8 existing evidence receipts; all other metadata, status assertions, timestamps, and outputs remain unchanged.
New and Updated Evidence Records
drift/dod_receipts/OMN-12564/dod-occ-pr-2329-omniclaude-push-dod-binding/command.yaml, drift/dod_receipts/OMN-12564/dod-omniclaude-pr-1735-main-hotfix/command.yaml
New PR 2329 receipt added capturing gh pr view 2329 execution with PASS status. PR 1735 receipt updated with new execution metadata (contract_sha256, run_timestamp, commit_sha), probe_stdout reflecting new headRefOid, and expanded actual_output describing DoD evidence resolver behavior for live PR push events.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • OmniNode-ai/onex_change_control#2015: Both PRs touch the same OMN-12564 evidence YAML receipts and contract file, directly updating the same evidence records and ticket definition.
  • OmniNode-ai/onex_change_control#2014: Both PRs follow the same pattern of updating drift/dod_receipts/**/command.yaml receipt records by updating the contract_sha256 field to bind evidence verification gates to fresh contract digests.
  • OmniNode-ai/onex_change_control#1300: Both PRs update drift/dod_receipts/**/command.yaml evidence receipt hash bindings to re-verify ticket Definition of Done using the same contract_sha256 rebinding mechanism.

Poem

🐰 Receipts rebound with fresh digests spun,
PR 2329 joins the DoD fun,
Evidence hash-linked, test outputs bright,
Push and pull lanes now unified tight! 🌙✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title 'evidence(OMN-12564): refresh omniclaude push DoD binding' directly and accurately reflects the main changes: updating DoD (Definition of Delivery) evidence receipts and bindings for ticket OMN-12564, specifically the omniclaude push DoD binding refresh mentioned in the PR objectives.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jonah/omn-12564-omniclaude-push-dod-evidence

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

@jonahgabriel
jonahgabriel added this pull request to the merge queue Jun 8, 2026
Merged via the queue into dev with commit e7aefac Jun 8, 2026
40 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-12564-omniclaude-push-dod-evidence branch June 8, 2026 13:00
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