[jonahgabriel]

Evidence-Source: 02fde8d
Evidence-Ticket: OMN-13530

Summary

• Add central OCC contracts and PASS receipts for OMN-13530, OMN-13531, and OMN-13532.
• Add OMN-592 deploy-gate evidence receipt so downstream core PR docs(OMN-7980): bind omniclaude PR 1629 receipt evidence #1311 has deploy evidence.

Verification

• uv run validate-yaml contracts/OMN-13530.yaml contracts/OMN-13531.yaml contracts/OMN-13532.yaml contracts/OMN-592.yaml
• uv run pre-commit run --files

Summary by CodeRabbit

• Chores
  • Introduced additional pre-merge enforcement gates to prevent routing regressions, including cases where required operation-matching fields are missing.
  • Added stronger validation coverage for skill-to-node mapping consistency and CLI delegation’s live-bus behavior, with no live runtime/deploy impact required.
  • Disabled emergency bypass for these checks to improve merge safety and reduce the chance of faulty releases.

[coderabbitai]

📝 Walkthrough

Walkthrough

Three new ticket contracts (OMN-13530, OMN-13531, OMN-13532) are added and one existing contract (OMN-592) is amended, each specifying enforcement rules, evidence requirements, DOD items, and emergency bypass settings. Corresponding command.yaml DOD receipt files are added under drift/dod_receipts/ for each ticket, recording PASS attestations with provenance metadata.

Changes

OCC Contracts and DOD Receipts

Layer / File(s)	Summary
OMN-13530 contract and receipts contracts/OMN-13530.yaml, drift/dod_receipts/OMN-13530/dod-deploy-no-runtime-impact/command.yaml, drift/dod_receipts/OMN-13530/dod-omnibase-core-pr-1310/command.yaml, drift/dod_receipts/OMN-13530/dod-occ-pr-self/command.yaml	Adds the enforcement contract for the handler-routing operation_match gate (static/CI/pre-commit only, no deploy required) plus three DOD receipts: deploy-no-runtime-impact attestation, omnibase-core PR-1310 head-ref verification, and OCC self-binding receipt, all PASS.
OMN-13531 contract and receipts contracts/OMN-13531.yaml, drift/dod_receipts/OMN-13531/dod-deploy-no-runtime-impact/command.yaml, drift/dod_receipts/OMN-13531/dod-omnibase-infra-pr-2080/command.yaml, drift/dod_receipts/OMN-13531/dod-omniclaude-pr-1796/command.yaml, drift/dod_receipts/OMN-13531/dod-occ-pr-self/command.yaml	Adds the contract for skill-node catalog resolver / pr_review skill surface alignment (interface_change: public_api) plus four DOD receipts: deploy-no-runtime-impact, omnibase-infra PR-2080, omniclaude PR-1796, and OCC self-binding, all PASS.
OMN-13532 contract and receipts contracts/OMN-13532.yaml, drift/dod_receipts/OMN-13532/dod-deploy-no-runtime-impact/command.yaml, drift/dod_receipts/OMN-13532/dod-omnibase-infra-pr-2079/command.yaml, drift/dod_receipts/OMN-13532/dod-occ-pr-self/command.yaml	Adds the contract for the onex delegate --bus kafka live-bus path with bootstrap validation (interface_change, CLI scope only) plus three DOD receipts: deploy-no-runtime-impact, omnibase-infra PR-2079 head-ref, and OCC self-binding, all PASS.
OMN-592 contract amendment and receipts contracts/OMN-592.yaml, drift/dod_receipts/OMN-592/dod-deploy-no-runtime-impact/command.yaml, drift/dod_receipts/OMN-592/dod-occ-pr-self/command.yaml	Appends a dod-deploy-no-runtime-impact evidence entry to the existing OMN-592 contract and adds two matching receipts: deploy-no-runtime-impact attestation and OCC self-binding, both PASS, asserting no live deploy is required for the NodeReducer refactor.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Possibly related PRs

• OmniNode-ai/onex_change_control#2669: Adds OCC seam contracts and DOD evidence for enabling a Kafka live-bus target via CLI with bootstrap validation, directly parallel to the OMN-13532 contract added in this PR.

Poem

🐇 Hop, hop, through the contract gate I go,
Where YAML receipts line up in a row,
No live deploy, no broker to fret,
Just evidence stamps and a clean SHA set.
The DOD is sealed, the bypass denied—
This bunny's diff is certified! ✅

🚥 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 directly and specifically describes the main changes: adding OCC receipts for three specific tickets (OMN-13530, OMN-13531, OMN-13532), which aligns with the PR's primary objective.
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-13530-13532-occ-evidence

---

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

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@drift/dod_receipts/OMN-13531/dod-occ-pr-self/command.yaml`:
- Line 10: The commit_sha field in the command.yaml file for the dod-occ-pr-self
receipt type is currently set to an actual commit hash, but it should be set to
an all-zero placeholder value to maintain the self-binding provenance pattern.
Replace the current commit SHA value with the all-zero placeholder
(0000000000000000000000000000000000000000) as this is required for self-binding
OCC receipts prior to merge commit materialization.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 983b1805-dd82-4943-bdc8-c6bd90aa43d9

📥 Commits

Reviewing files that changed from the base of the PR and between 9178a13 and be1ce59.

📒 Files selected for processing (16)

• contracts/OMN-13530.yaml
• contracts/OMN-13531.yaml
• contracts/OMN-13532.yaml
• contracts/OMN-592.yaml
• drift/dod_receipts/OMN-13530/dod-deploy-no-runtime-impact/command.yaml
• drift/dod_receipts/OMN-13530/dod-occ-pr-self/command.yaml
• drift/dod_receipts/OMN-13530/dod-omnibase-core-pr-1310/command.yaml
• drift/dod_receipts/OMN-13531/dod-deploy-no-runtime-impact/command.yaml
• drift/dod_receipts/OMN-13531/dod-occ-pr-self/command.yaml
• drift/dod_receipts/OMN-13531/dod-omnibase-infra-pr-2080/command.yaml
• drift/dod_receipts/OMN-13531/dod-omniclaude-pr-1796/command.yaml
• drift/dod_receipts/OMN-13532/dod-deploy-no-runtime-impact/command.yaml
• drift/dod_receipts/OMN-13532/dod-occ-pr-self/command.yaml
• drift/dod_receipts/OMN-13532/dod-omnibase-infra-pr-2079/command.yaml
• drift/dod_receipts/OMN-592/dod-deploy-no-runtime-impact/command.yaml
• drift/dod_receipts/OMN-592/dod-occ-pr-self/command.yaml

✅ Files skipped from review due to trivial changes (10)

• drift/dod_receipts/OMN-13531/dod-deploy-no-runtime-impact/command.yaml
• drift/dod_receipts/OMN-13532/dod-deploy-no-runtime-impact/command.yaml
• drift/dod_receipts/OMN-13530/dod-deploy-no-runtime-impact/command.yaml
• drift/dod_receipts/OMN-13530/dod-occ-pr-self/command.yaml
• contracts/OMN-592.yaml
• drift/dod_receipts/OMN-592/dod-occ-pr-self/command.yaml
• contracts/OMN-13531.yaml
• drift/dod_receipts/OMN-13530/dod-omnibase-core-pr-1310/command.yaml
• drift/dod_receipts/OMN-592/dod-deploy-no-runtime-impact/command.yaml
• contracts/OMN-13532.yaml

🚧 Files skipped from review as they are similar to previous changes (4)

• contracts/OMN-13530.yaml
• drift/dod_receipts/OMN-13531/dod-omnibase-infra-pr-2080/command.yaml
• drift/dod_receipts/OMN-13532/dod-omnibase-infra-pr-2079/command.yaml
• drift/dod_receipts/OMN-13531/dod-omniclaude-pr-1796/command.yaml

[coderabbitai]

🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

Use the self-binding placeholder SHA for dod-occ-pr-self.

For this receipt type, commit_sha should be the all-zero placeholder until merge commit materialization; storing the current head SHA here breaks the established self-binding provenance pattern.

Suggested fix

-commit_sha: "4315b105ee4f60cdab1a6611efa6cf919aff3532"
+commit_sha: "0000000000000000000000000000000000000000"

Based on learnings: self-binding OCC receipts at drift/dod_receipts/<TICKET>/dod-occ-pr-self/command.yaml intentionally use an all-zero commit_sha placeholder pre-merge.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	commit_sha: "4315b105ee4f60cdab1a6611efa6cf919aff3532"
	commit_sha: "0000000000000000000000000000000000000000"

🤖 Prompt for AI Agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@drift/dod_receipts/OMN-13531/dod-occ-pr-self/command.yaml` at line 10, The
commit_sha field in the command.yaml file for the dod-occ-pr-self receipt type
is currently set to an actual commit hash, but it should be set to an all-zero
placeholder value to maintain the self-binding provenance pattern. Replace the
current commit SHA value with the all-zero placeholder
(0000000000000000000000000000000000000000) as this is required for self-binding
OCC receipts prior to merge commit materialization.

Source: Learnings