Repository navigation
evidence(OMN-11061): OCC contract and receipts for topic wiring PR #657 - #1044
Merged
Merged
Conversation
Adds ModelTicketContract + two dod receipts (sweep-clean, deploy) for OMN-11061 — wire 49 undeclared topics into node_emit_daemon contract, fix bifrost_delegation.yaml annotations, add handler block to node_monitor_alert_responder. UNDECLARED_TOPIC reduced from 75 to 0.
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
jonahgabriel
enabled auto-merge (squash)
May 15, 2026 21:52
Adds self-binding receipt with pr_number: 1044 to clear OCC eligibility pr_ticket_mismatch failure.
This was referenced Aug 28, 2026
jonahgabriel
added a commit
that referenced
this pull request
Aug 28, 2026
#7420) * evidence(OMN-16799): OCC companion for OmniNode-ai/omninode_infra#1044 Hand-authored companion, not an autobind. node_occ_companion_compute's F-17 guard deliberately suppresses companion authoring while the product PR is a draft or is title-marked do-not-merge/WIP (handler_occ_companion_compute.py, added after incident occ#4333). omninode_infra#1044 is BOTH -- DRAFT, and its title carries "[DO NOT MERGE -- awaiting grant]" -- and both are correct: #1044 must not become mergeable before the OMN-16799 prod-promotion grant lands on onex_change_control@main. So the autobind will not mint this until the operator un-drafts AND retitles #1044, which is exactly the step the Evidence-Source line is supposed to unblock. This PR supplies it without touching #1044's draft state. Appends two dod_evidence items to the existing OMN-16799 contract (append-only; per-entry contract_entry_sha256 binding means entries 1..N keep their hashes): - dod-OmniNode-ai-omninode_infra-pr-1044: Evidence-Source bind, literal PR number per the OMN-15382 class-3 mis-binding rule. - dod-omn-16799-reclaim-behavior-ratchet: a real behavior-class check on the changed surface, not a ticket-independent suite standing in for one (OMN-16434). #1044 ships tests/k8s/test_omn_16799_api_hostname_reclaim.py, 8 tests pinning the fail-closed shape of the reclaim_api_hostname input. The existing OMN-16799 entries from OCC#7342 (the #1034 autobind) are untouched. Evidence-Ticket: OMN-16799 * fix(OMN-16799): use pinned product probe for reclaim evidence
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
ModelTicketContractfor OMN-11061 (wire undeclared topic literals to contract constants)dod-sweep-cleanreceipt: 3967 tests pass, UNDECLARED_TOPIC reduced 75 → 0dod-deployreceipt: static YAML + pure-Python analysis change, no live deploy requiredTicket
OMN-11061
Evidence
contracts/OMN-11061.yaml— validated against ModelTicketContract schemadrift/dod_receipts/OMN-11061/dod-sweep-clean/command.yaml— sweep receiptsdrift/dod_receipts/OMN-11061/dod-deploy/command.yaml— deploy assessmentRelated
omnimarket PR #657