contracts(OMN-11985): Wave 3C OCC evidence for omnimarket PRs #800 #801 #802 #803 by jonahgabriel · Pull Request #1586 · OmniNode-ai/onex_change_control

jonahgabriel · 2026-05-25T00:11:48Z

Summary

Add OCC contracts + dod_receipts for 4 declarative migration wave PRs (Wave 3C closeout)
OMN-11902 → omnimarket docs(OMN-10653): add cloud deployment OCC evidence #803 (Wave 2: Build/Dispatch, 25 nodes)
OMN-11904 → omnimarket docs(OMN-10644): add hardcoded sweep OCC evidence #800 (Wave 4: Session/Overnight, 14 nodes)
OMN-11905 → omnimarket docs(OMN-10626): add omniclaude delegation OCC evidence #802 (Wave 5: Sweeps/Verification, 22 nodes)
OMN-11908 → omnimarket docs(OMN-10655): add onboarding OCC evidence #801 (Wave 8: Remaining nodes, 33 nodes)

Rebase conflict resolutions

docs(OMN-10653): add cloud deployment OCC evidence #803: handler_model_router.py topic constant comments + verified_dispatch_orchestrator handler routing format
docs(OMN-10644): add hardcoded sweep OCC evidence #800: node_morning_handoff_generator — merged operation key from main with event_model from PR
docs(OMN-10626): add omniclaude delegation OCC evidence #802: node_golden_chain_sweep — kept sweep_golden_chains operation name from main
docs(OMN-10655): add onboarding OCC evidence #801: Evidence JSON conflicts resolved to main version; one-off fix_handler_routing.py script removed

Test plan

All 4 branches rebased onto main cleanly
uv run pytest tests/ -v -k handler_model_router — 1 passed
pre-commit run --all-files — all hooks passed on OCC files

Evidence-Ticket: OMN-11985
Evidence-Source: OCC#1585

…#801 #802 #803 Adds contracts + dod_receipts for OMN-11902 (#803), OMN-11904 (#800), OMN-11905 (#802), and OMN-11908 (#801) — the four declarative contract migration wave PRs repaired and rebased as part of OMN-11985 Wave 3C.

coderabbitai · 2026-05-25T00:11:54Z

Warning

Review limit reached

@jonahgabriel, we couldn't start this review because you've used your available PR reviews for now.

Your plan includes 5 reviews of capacity. Refill in 21 minutes and 48 seconds.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

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

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 trial, open-source, and free plans. In all cases, review capacity refills continuously over time.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 495b3783-68d4-4230-8035-dc2c59760371

📥 Commits

Reviewing files that changed from the base of the PR and between 1018de2 and 7d2c914.

📒 Files selected for processing (16)

contracts/OMN-11902.yaml
contracts/OMN-11904.yaml
contracts/OMN-11905.yaml
contracts/OMN-11908.yaml
drift/dod_receipts/OMN-11902/dod-conflict-resolution-local-gates/command.yaml
drift/dod_receipts/OMN-11902/dod-deploy-readiness/command.yaml
drift/dod_receipts/OMN-11902/dod-omnimarket-pr/command.yaml
drift/dod_receipts/OMN-11904/dod-conflict-resolution-local-gates/command.yaml
drift/dod_receipts/OMN-11904/dod-deploy-readiness/command.yaml
drift/dod_receipts/OMN-11904/dod-omnimarket-pr/command.yaml
drift/dod_receipts/OMN-11905/dod-conflict-resolution-local-gates/command.yaml
drift/dod_receipts/OMN-11905/dod-deploy-readiness/command.yaml
drift/dod_receipts/OMN-11905/dod-omnimarket-pr/command.yaml
drift/dod_receipts/OMN-11908/dod-conflict-resolution-local-gates/command.yaml
drift/dod_receipts/OMN-11908/dod-deploy-readiness/command.yaml
drift/dod_receipts/OMN-11908/dod-omnimarket-pr/command.yaml

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch jonah/omn-11985-wave3c-occ-evidence

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

…ntracts Adds dod-deploy-readiness evidence items to the 4 Wave 3C ticket contracts (OMN-11902/11904/11905/11908) so the omnimarket deploy-gate CI check passes. Each receipt confirms contract-only YAML migration with no production mutation.

jonahgabriel · 2026-05-25T01:58:26Z

Superseded by the already-merged Wave 2/4/5/8 OCC evidence on main. Current main contains OMN-11902/11904/11905/11908 contracts and deploy-readiness/local-validation/OCC receipts from the successful downstream merge path; this branch now add/add conflicts on the same evidence files. Closing as redundant rather than hand-merging stale duplicate evidence.

…and receipt bundle for 6 tickets (OmniNode-ai#999) * contract(OMN-10940,OMN-10941): add OCC contracts and receipts for infra PRs 1586 and 1587 - OMN-10940: add dod-omnibase-infra-pr-1586 evidence binding PR OmniNode-ai#1586 (E2E golden-chain task-delegated event test) to the OMN-10940 contract - OMN-10941: new contract + receipt for PR OmniNode-ai#1587 (max_tokens propagation regression tests) Both receipts are required for the receipt-gate Evidence-Source check on PRs 1586 and 1587. * contract: add OMN-10936/10942/10944 contracts + fix receipt hashes for 10940/10941/10949 - Add contracts for OMN-10936 (reviewdog-caller omniclaude), OMN-10942 (contract routing policy), OMN-10944 (dashboard projection API contract) - Update OMN-10949 contract to cover infra PR OmniNode-ai#1588 and market PR OmniNode-ai#622 in addition to market OmniNode-ai#616 - Fix contract_sha256 in OMN-10940 receipts (pr-1580, pr-1586) — contract was updated with pr-1586 evidence item, changing the hash - Add contract_sha256 to OMN-10941 pr-1587 receipt (was missing) - Update OMN-10949 pr-616 receipt contract_sha256 to reflect expanded contract - Add receipts: OMN-10936/pr-1582, OMN-10942/pr-621, OMN-10944/pr-620, OMN-10949/pr-1588, OMN-10949/pr-622 * contract(OMN-10949): add deploy evidence item + update receipt hashes - Add deploy-evidence-pricing-version dod_evidence item to satisfy deploy-gate (pricing_manifest_version is additive schema field, no runtime restart needed) - Update contract_sha256 in all OMN-10949 receipts (pr-616, pr-1588, pr-622) to reflect updated contract hash * contract: add OCC self-binding receipts (pr-999) + fix all contract_sha256 hashes - Add dod-occ-pr-999 evidence items to all 6 ticket contracts (OMN-10936/10940/10941/10942/10944/10949) - Create dod-occ-pr-999/command.yaml self-binding receipts for each ticket - Update all downstream receipt contract_sha256 fields to reflect updated contract hashes - Fixes OCC eligibility pr_ticket_mismatch failure (receipts now bind to OCC PR OmniNode-ai#999) * receipt(OMN-10949): add deploy-evidence-pricing-version receipt for deploy-gate * fix: add headRefName to probe_stdout in OMN-10942/OMN-10944 receipts (CR review) * ci(OMN-10940): suppress delegation_events migration name conflict The Migration Conflict Check on PR OmniNode-ai#999 flagged delegation_events as a NAME_CONFLICT between omnibase_infra/078_create_delegation_events.sql (Postgres projection, 22 cols, UUID PK) and omniclaude/001_create_delegation_tables.sql (SQLite hook-side capture, 15 cols, INTEGER PK). Different databases, different access patterns. The omnibase_infra side is the canonical projection consumed by HandlerProjectionDelegation from onex.evt.omniclaude.task-delegated.v1 (OMN-10526). The omniclaude side is the local hook-side store pinned to the projection contract (OMN-10604). Neither shares a Postgres instance with the other. Suppress the conflict using the existing schema_evolution reason — the same pattern already used for llm_call_metrics and savings_estimates, which have an identical infra-vs-omniclaude split. No code changes; this is a CI suppressions update only.

…try validation (OmniNode-ai#1008) * contract(OMN-10971): update skip_token_approvals schema for GHA registry validation Align allowlist field names with the validation logic in reject-deploy-gate-skip.yml: receipt_id, repo, pr_number, approved_by, expires_at, created_at, token_type, reason. Documents that approved_by != PR author is enforced by the workflow. * evidence(OMN-10971): add contract and self-binding receipt Receipt gate requires contracts/OMN-10971.yaml to exist at the PR HEAD. * evidence(OMN-10971): add omniclaude PR OmniNode-ai#1586 receipt Binds omniclaude central registry validation PR to the OMN-10971 contract.

jonahgabriel enabled auto-merge (squash) May 25, 2026 00:12

jonahgabriel closed this May 25, 2026

auto-merge was automatically disabled May 25, 2026 01:58
Pull request was closed

jonahgabriel deleted the jonah/omn-11985-wave3c-occ-evidence branch June 11, 2026 17:59

This was referenced Jul 3, 2026

receipt(OMN-13870): add infra PR 2186 evidence #3512

Merged

evidence(OMN-13859): OCC contract + DoD receipts for event-driven worktree prune #3513

Closed

Navigation Menu

Repository navigation

contracts(OMN-11985): Wave 3C OCC evidence for omnimarket PRs #800 #801 #802 #803 - #1586

contracts(OMN-11985): Wave 3C OCC evidence for omnimarket PRs #800 #801 #802 #803#1586
jonahgabriel wants to merge 2 commits into
mainfrom
jonah/omn-11985-wave3c-occ-evidence

jonahgabriel commented May 25, 2026

Uh oh!

coderabbitai Bot commented May 25, 2026 •

edited

Loading

Review limit reached

Uh oh!

jonahgabriel commented May 25, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

jonahgabriel commented May 25, 2026

Summary

Rebase conflict resolutions

Test plan

Uh oh!

coderabbitai Bot commented May 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review limit reached

Uh oh!

jonahgabriel commented May 25, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

coderabbitai Bot commented May 25, 2026 •

edited

Loading