evidence(OMN-13097): re-bind OCC receipts to gap-skill follow-up PRs by jonahgabriel · Pull Request #2823 · OmniNode-ai/onex_change_control

jonahgabriel · 2026-06-19T20:33:55Z

Summary

Refreshes the OMN-13097 OCC receipts that unblock the gap-skill follow-up PRs.
The product PRs cite OMN-13097 and require PASS receipts bound to their current
heads, including omnimarket #1292 after its latest CI nudge commit.

Paired downstream PRs:

omnibase_infra evidence(OMN-12570): DoD receipts for orchestrator phase separation #2045 — register gap -> node_gap_compute CLI mapping
omnimarket evidence(OMN-11350): bind infra topic startup noise proof #1292 — declare operation: gap on the node_gap_compute
operation_match handler

Changes

drift/dod_receipts/OMN-13097/dod-skill-tests/command.yaml: bound to
omnibase_infra evidence(OMN-12570): DoD receipts for orchestrator phase separation #2045; probe = test_cli_skill.py 21 passed, 1 skipped.
drift/dod_receipts/OMN-13097/dod-mypy-strict/command.yaml: bound to
omnimarket evidence(OMN-11350): bind infra topic startup noise proof #1292 head 380a5ce; probe = node_gap_compute mypy strict pass.
drift/dod_receipts/OMN-13097/dod-deploy-user-gated/command.yaml: bound to
omnimarket evidence(OMN-11350): bind infra topic startup noise proof #1292 head 380a5ce; probe = deploy gate is user-gated and no live
runtime mutation occurred.

dod_evidence

Local proof from this worktree:

uv run validate-yaml contracts/OMN-13097.yaml
pre-commit run --files contracts/OMN-13097.yaml drift/dod_receipts/OMN-13097/dod-mypy-strict/command.yaml drift/dod_receipts/OMN-13097/dod-deploy-user-gated/command.yaml
uv run python scripts/ci/run_contract_compliance_check.py --pr 2823 --repo OmniNode-ai/onex_change_control --contracts-dir contracts => OMN-13097: 6/6 PASS

Evidence-Source: OCC#2823
Evidence-Ticket: OMN-13097

Summary by CodeRabbit

Tests
- Test execution count increased from 20 to 21 passed tests.
Chores
- Updated validation records with latest execution metadata and checksums.
- Added new deployment gate validation check for quality assurance.

coderabbitai · 2026-06-19T20:34:07Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 799e46be-7a2c-42d5-a62e-89a9f40e2839

📥 Commits

Reviewing files that changed from the base of the PR and between 4bae6df and d3cbf6e.

📒 Files selected for processing (7)

contracts/OMN-13097.yaml
drift/dod_receipts/OMN-13097/dod-deploy-user-gated/command.yaml
drift/dod_receipts/OMN-13097/dod-mypy-strict/command.yaml
drift/dod_receipts/OMN-13097/dod-occ-pr-self/command.yaml
drift/dod_receipts/OMN-13097/dod-omniclaude-precommit/command.yaml
drift/dod_receipts/OMN-13097/dod-omniclaude-shim-migrations/command.yaml
drift/dod_receipts/OMN-13097/dod-skill-tests/command.yaml

📝 Walkthrough

Walkthrough

The OMN-13097 contract is updated to expect 21 skill tests (up from 20) and gains a new dod-deploy-user-gated evidence block. A corresponding new receipt file is created for that evidence item. Five existing DoD receipt files are refreshed with updated run metadata, commit SHAs, probe outputs, and contract hashes.

Changes

OMN-13097 DoD contract and receipts update

Layer / File(s)	Summary
Contract update and new deploy-user-gated receipt `contracts/OMN-13097.yaml`, `drift/dod_receipts/OMN-13097/dod-deploy-user-gated/command.yaml`	`dod-skill-tests` evidence updated to assert `21 passed`; new `dod-deploy-user-gated` evidence block added asserting deploy gate behavior for gap-skill follow-up PRs; matching `command.yaml` receipt created with full PASS run record.
Refreshed run metadata in existing receipts `drift/dod_receipts/OMN-13097/dod-skill-tests/command.yaml`, `drift/dod_receipts/OMN-13097/dod-mypy-strict/command.yaml`, `drift/dod_receipts/OMN-13097/dod-occ-pr-self/command.yaml`, `drift/dod_receipts/OMN-13097/dod-omniclaude-precommit/command.yaml`, `drift/dod_receipts/OMN-13097/dod-omniclaude-shim-migrations/command.yaml`	Five existing receipts refreshed with new `run_timestamp`, `commit_sha`, `runner`, `probe_stdout`, `actual_output`, `contract_sha256`, `pr_number`, and `branch` values; all retain `status: PASS`.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

OmniNode-ai/onex_change_control#1300: Adds contract_sha256 fields across drift/dod_receipts/*/dod-*/command.yaml evidence records, the same field being updated in this PR's receipt refresh.
OmniNode-ai/onex_change_control#2027: Updates dod-occ-pr-self/command.yaml receipt metadata including contract_sha256, directly matching the same file modified here.
OmniNode-ai/onex_change_control#2046: Updates contract_sha256 values across DoD receipt command files in the same receipt directory pattern as this PR.

Poem

🐇 A receipt refreshed, a new gate in place,
Twenty-one tests now pass with grace.
The deploy guard hops into the chain,
SHA hashes stamped, no drift remains.
All evidence green — the rabbit approves! ✅

🚥 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 title accurately describes the main change: re-binding OCC receipts to gap-skill follow-up PRs as evidenced by the receipt file updates and the explicit 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-13097-occ-gap-skill-binding

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

Fix the OCC PR's own OCC eligibility (pr_ticket_mismatch on PR #2823) and keep omnimarket #1292 bound. Distribute three of the five PASS receipts across the three citing PRs so each has a PR-bound PASS receipt: - dod-skill-tests -> omnibase_infra #2045 (commit 014fa5e5) - dod-mypy-strict -> omnimarket #1292 (commit b0762f65; mypy --strict clean on node_gap_compute, 7 source files) - dod-occ-pr-self -> onex_change_control #2823 self-binding (commit 75e0c34) Contract unchanged (sha256:3f321090...); all receipts keep matching contract_sha256; verifier != runner. Local proof: validator_occ_merge_eligibility eligible=true for #2045, #1292, and #2823.

The gap-skill follow-up PRs (omnibase_infra #2045 register gap->node_gap_compute CLI mapping; omnimarket #1292 declare node_gap_compute operation_match operation) cite OMN-13097 but the existing OMN-13097 OCC receipts bound to the original implementation PRs, so validator_occ_merge_eligibility reported pr_ticket_mismatch ('no PASS receipt ... binds to PR #2045/#1292'). Re-bind two of the five PASS receipts to the new PR commit SHAs so each PR has a PR-bound PASS receipt (eligibility requires >=1 bound receipt per ticket per PR): - dod-skill-tests -> omnibase_infra #2045 (commit 014fa5e5; test_cli_skill.py 21 passed, 1 skipped after registering the gap mapping) - dod-occ-pr-self -> omnimarket #1292 (commit b0762f65; OMN-13097 contract validates against ModelTicketContract) Contract OMN-13097.yaml is unchanged (hash sha256:3f321090...); all five receipts keep matching contract_sha256. verifier (receipt-gate-local) != runner (omn-13097-gap-skill-impl), so no self-attestation. Local proof: validator_occ_merge_eligibility eligible=true for both PRs; validator_receipt_gate_cli RECEIPT GATE PASSED (5 checks) for both repos.

Fix the OCC PR's own OCC eligibility (pr_ticket_mismatch on PR #2823) and keep omnimarket #1292 bound. Distribute three of the five PASS receipts across the three citing PRs so each has a PR-bound PASS receipt: - dod-skill-tests -> omnibase_infra #2045 (commit 014fa5e5) - dod-mypy-strict -> omnimarket #1292 (commit b0762f65; mypy --strict clean on node_gap_compute, 7 source files) - dod-occ-pr-self -> onex_change_control #2823 self-binding (commit 75e0c34) Contract unchanged (sha256:3f321090...); all receipts keep matching contract_sha256; verifier != runner. Local proof: validator_occ_merge_eligibility eligible=true for #2045, #1292, and #2823.

jonahgabriel added 4 commits June 19, 2026 16:50

evidence(OMN-13097): refresh gap follow-up receipts

72ce446

evidence(OMN-13097): refresh occ-self receipt commit_sha binding

3a4843e

jonahgabriel force-pushed the jonah/omn-13097-occ-gap-skill-binding branch from a66db79 to 3a4843e Compare June 19, 2026 20:51

jonahgabriel added 2 commits June 19, 2026 17:02

evidence(OMN-13097): refresh omnimarket gap head

b0a2534

evidence(OMN-13097): refresh omnimarket head binding

d3cbf6e

jonahgabriel added this pull request to the merge queue Jun 19, 2026

Merged via the queue into dev with commit b967ff3 Jun 19, 2026
4 checks passed

jonahgabriel deleted the jonah/omn-13097-occ-gap-skill-binding branch June 19, 2026 21:15

Navigation Menu

Repository navigation

evidence(OMN-13097): re-bind OCC receipts to gap-skill follow-up PRs - #2823

evidence(OMN-13097): re-bind OCC receipts to gap-skill follow-up PRs#2823
jonahgabriel merged 6 commits into
devfrom
jonah/omn-13097-occ-gap-skill-binding

jonahgabriel commented Jun 19, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Jun 19, 2026 •

edited

Loading

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Poem

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

jonahgabriel commented Jun 19, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

dod_evidence

Summary by CodeRabbit

Uh oh!

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

Uh oh!

Walkthrough

Changes

Estimated code review effort

Possibly related PRs

Poem

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

jonahgabriel commented Jun 19, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jun 19, 2026 •

edited

Loading