Skip to content

evidence(OMN-12545): bind Phase 1A route PRs (#2052, #1321) to OCC PASS receipts - #2864

Merged
jonahgabriel merged 2 commits into
devfrom
jonah/omn-12545-route-occ-receipts
Jun 20, 2026
Merged

jonahgabriel merged 2 commits into
devfrom
jonah/omn-12545-route-occ-receipts

Conversation

@jonahgabriel

Copy link
Copy Markdown
Contributor

OMN-12545 Phase 1A — bind route-half OCC evidence to the two route PRs

Authors the paired OCC dod_receipts that unblock the required Receipt Gate (verify / verify,
validator_occ_merge_eligibility) on the two Phase 1A route-consolidation PRs. Both previously failed with
reason="pr_ticket_mismatch" because the existing OMN-12545 receipts bound only to the OLD enum-half PR
(omnibase_core #1187), not to the route PRs.

Evidence-Ticket: OMN-12545

New PR-bound PASS receipts

Receipt id Binds PR Head SHA
OMN-12545:dod-route-single-source-infra-pr2052:command omnibase_infra #2052 0d1861a0a85b4ded46604b859f3989cd0a6b71fb
OMN-12545:dod-route-repoint-omnimarket-pr1321:command omnimarket #1321 48f5949bce90b331413ad5c0db516a7d04644607

Route-half evidence recorded (real, independently re-verified on the PR heads)

  • omnibase_infra evidence(OMN-12583): bind omninode_infra dependency batch #2052 — deletes the infra ModelDispatchRoute duplicate
    (src/omnibase_infra/models/dispatch/model_dispatch_route.py, 279 lines) and single-sources on the canonical
    omnibase_core copy; models/dispatch/__init__.py shim + core alias retained for import-path stability.
    Re-verified: deleted file absent on branch, 0 infra-local class ModelDispatchRoute, core import live in
    message_dispatch_engine.py. Route close-out: mypy --strict clean (2493 files), full suite 6949 passed.
  • omnimarket docs(OMN-10475): bind omniclaude salvage PR 1633 #1321 — repoints node_delegation_orchestrator/wiring.py ModelDispatchRoute import
    infra→core and switches dispatcher_id= kwargs to the canonical handler_id= field.
    Re-verified: 0 remaining infra ModelDispatchRoute imports, core import live at wiring.py:123.
    Route close-out: mypy --strict clean (2428 files), full suite 7705 passed.

Contract + existing receipts

  • contracts/OMN-12545.yaml gains two route-half dod_evidence items; summary extended to describe the route
    PRs. New contract hash sha256:e05bfbc4a5e01c8762dc1e029bf30d75c70153e391ea6e1b8cafbd73123e5e1c.
  • The three pre-existing receipts (dod-core-pr-1187, dod-deploy-assessment, dod-occ-pr) have their
    contract_sha256 re-stamped to the new hash so they stay hash-bound; their PR bindings (1187 / 2000) are
    unchanged.

Local gate proof (before merge)

validator_occ_merge_eligibility returns eligible:true, reason:eligible,
detail:"OCC evidence is present, PASS, hash-bound, and PR-bound" for both route PR snapshots — all 5
receipts PASS, hash-bound, and PR-bound.

Route PRs cite this PR's merge commit as their Evidence-Source once it lands.

…SS receipts

Authors the paired OCC dod_receipts that unblock the required Receipt Gate
(verify/verify, validator_occ_merge_eligibility) on the two Phase 1A route PRs.

- dod-route-single-source-infra-pr2052: binds omnibase_infra #2052 (sha 0d1861a0a)
- dod-route-repoint-omnimarket-pr1321: binds omnimarket #1321 (sha 48f5949bc)

Contract gains two route-half dod_evidence items; the three pre-existing receipts
re-stamped to the new contract hash. Local validator_occ_merge_eligibility returns
eligible:true / hash-bound / PR-bound for both route PR snapshots.

Evidence-Ticket: OMN-12545
@coderabbitai

coderabbitai Bot commented Jun 20, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

@jonahgabriel, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 5 minutes and 3 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 941327c7-5141-48a6-b995-51179d6b22cf

📥 Commits

Reviewing files that changed from the base of the PR and between 727a90e and af03fb7.

📒 Files selected for processing (6)
  • contracts/OMN-12545.yaml
  • drift/dod_receipts/OMN-12545/dod-core-pr-1187/command.yaml
  • drift/dod_receipts/OMN-12545/dod-deploy-assessment/command.yaml
  • drift/dod_receipts/OMN-12545/dod-occ-pr/command.yaml
  • drift/dod_receipts/OMN-12545/dod-route-repoint-omnimarket-pr1321/command.yaml
  • drift/dod_receipts/OMN-12545/dod-route-single-source-infra-pr2052/command.yaml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jonah/omn-12545-route-occ-receipts

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

@jonahgabriel
jonahgabriel enabled auto-merge June 20, 2026 19:22
…igibility)

The OCC PR's own occ-preflight/verify eligibility check requires a PASS receipt
bound to this PR. Repoints dod-occ-pr from the historical #2000 binding to #2864
(pr_number-based binding). Route-PR receipts (#2052, #1321) and the enum-half
#1187 receipt are unchanged; local validator_occ_merge_eligibility returns
eligible:true for the OCC-self snapshot and both route-PR snapshots.

Evidence-Ticket: OMN-12545
@jonahgabriel
jonahgabriel added this pull request to the merge queue Jun 20, 2026
Merged via the queue into dev with commit f624fab Jun 20, 2026
38 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-12545-route-occ-receipts branch June 20, 2026 20:14
jonahgabriel pushed a commit that referenced this pull request Jun 25, 2026
…l receipts to final contract hash

The OCC-self eligibility gate (validator_occ_merge_eligibility) requires
a PASS receipt whose pr_number binds to THIS OCC PR. Repoint the
dod-occ-pr receipt + its contract check_value from the prior OCC PR
#2864 to #3087, and rebind contract_sha256 across all 6 OMN-12545
receipts to the settled post-edit contract hash
sha256:850005f63f4db1718f4ca675385427fe7bad9b49bca2c7fe3707da6e629d8d2f.
Receipt Honesty + Hardening gates green; all receipts match the live
contract hash.

Evidence-Source: 86420fd00bc76f91fb2e2e15d2f311c255f2e839
Evidence-Ticket: OMN-12545
jonahgabriel added a commit that referenced this pull request Sep 25, 2026
…11331)

* evidence(OMN-19438): hand-authored OCC companion for OmniNode-ai/omnimarket#2865

occ-autobind terminally declined omnimarket#2865 (both at head b17b39074c
and again after the branch update to 3d810c36f) with
skip:NO_RED_DERIVABLE_CHECK: the diff is a net-new SQL migration file with
no existing test to derive a RED against the merge base. Per OMN-15247 this
is the designed hand-authored-evidence remedy, safe to author now (the
producer stood down for this head, no in-flight mint to collide with) and
signed by a lane other than the one that authored the ticket.

Appends two dod_evidence entries to the existing OMN-19438 contract
(already carrying #4081's and #2864's evidence): a content-bound probe
(the migration file's header line present at the PR head, the file itself
absent/404 at the merge base, with a positive control proving the pipeline
reads real content) and a product-diff-scope check confined to the one
added migration file. Purely additive: the six pre-existing dod_evidence
entries and their receipts are untouched.

* evidence(OMN-18921): mint the owed OCC self-bind for PR #11331

---------

Co-authored-by: onexbot-occ-writer[bot] <onexbot-occ-writer[bot]@users.noreply.github.com>
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