Skip to content

evidence(OMN-13247): bind infra PR 2042 receipt (event_model wiring fix) - #2797

Merged
jonahgabriel merged 3 commits into
devfrom
jonahgabriel/omn-13247-event-model-wiring-occ
Jun 19, 2026
Merged

jonahgabriel merged 3 commits into
devfrom
jonahgabriel/omn-13247-event-model-wiring-occ

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented Jun 19, 2026 •

Copy link
Copy Markdown
Contributor

OMN-13247 — OCC evidence: bind omnibase_infra PR #2042 (event_model wiring fix)

Central change-control evidence for the e2e-discovered coding-agent bus wiring fix.

omnibase_infra PR #2042 renames the PER-HANDLER input_model: -> event_model: in the four coding-agent node contracts so the auto-wiring parser (_parse_handler_routing, which reads only the per-handler event_model key) populates event_model; without it the dispatch callback handed the orchestrator the materialized dispatch dict instead of a typed ModelEventEnvelope, and the handler crashed on envelope.event_type. The PR also adds a real-dispatch-path regression test (fails pre-fix, passes post-fix) and commits the coding-agent runtime env vars.

Evidence added

  • contracts/OMN-13247.yaml: new dod-infra-pr-2042 dod_evidence entry binding the infra PR.
  • drift/dod_receipts/OMN-13247/dod-infra-pr-2042/command.yaml: PASS binding receipt (pr_number 2042, commit_sha 35fcdfc2..., probe_command + probe_stdout, verifier=receipt-gate-local != runner).

The OCC self-binding receipt for this PR is added in a follow-up commit once the PR number is assigned.

Refs OMN-13247 (epic OMN-13245).


Evidence-Source: OCC#SELF
Evidence-Ticket: OMN-13247

Summary by CodeRabbit

  • Chores
    • Updated verification receipt records with new contract hash values across multiple deployment assessments.
    • Added new deployment verification evidence entries for infrastructure updates and operational verification tracking with execution confirmations.

Adds the OCC binding receipt + dod_evidence entry for omnibase_infra PR #2042,
which fixes the e2e-discovered coding-agent bus wiring bug (per-handler
input_model -> event_model rename so the auto-wiring parser populates event_model
and the dispatch callback re-hydrates a typed ModelEventEnvelope instead of
handing the orchestrator a raw dict). The OCC self-binding receipt is added in a
follow-up commit once this PR number is known.
@coderabbitai

coderabbitai Bot commented Jun 19, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0b2a725a-70e3-4037-837b-737296d218fd

📥 Commits

Reviewing files that changed from the base of the PR and between 501de7f and 750abd4.

📒 Files selected for processing (8)
  • contracts/OMN-13247.yaml
  • drift/dod_receipts/OMN-13247/dod-deploy-assessment/command.yaml
  • drift/dod_receipts/OMN-13247/dod-infra-pr-2031/command.yaml
  • drift/dod_receipts/OMN-13247/dod-infra-pr-2041/command.yaml
  • drift/dod_receipts/OMN-13247/dod-infra-pr-2042/command.yaml
  • drift/dod_receipts/OMN-13247/dod-occ-pr-2041-self/command.yaml
  • drift/dod_receipts/OMN-13247/dod-occ-pr-2042-self/command.yaml
  • drift/dod_receipts/OMN-13247/dod-occ-pr-self/command.yaml

📝 Walkthrough

Walkthrough

Two new dod_evidence entries (dod-infra-pr-2042 and dod-occ-pr-2042-self) are added to contracts/OMN-13247.yaml, with corresponding new command.yaml receipt files. Five existing receipt files have their contract_sha256 field updated to a new digest.

Changes

OMN-13247 DoD Receipt Update

Layer / File(s) Summary
New PR-2042 evidence bindings and receipt files
contracts/OMN-13247.yaml, drift/dod_receipts/OMN-13247/dod-infra-pr-2042/command.yaml, drift/dod_receipts/OMN-13247/dod-occ-pr-2042-self/command.yaml
Two new dod_evidence list items (dod-infra-pr-2042 and dod-occ-pr-2042-self) are appended to the contract, and their full command.yaml receipt files are introduced with grep-based PASS checks, contract hash, execution metadata, and actual_output descriptions.
contract_sha256 refresh in existing receipts
drift/dod_receipts/OMN-13247/dod-deploy-assessment/command.yaml, drift/dod_receipts/OMN-13247/dod-infra-pr-2031/command.yaml, drift/dod_receipts/OMN-13247/dod-infra-pr-2041/command.yaml, drift/dod_receipts/OMN-13247/dod-occ-pr-2041-self/command.yaml, drift/dod_receipts/OMN-13247/dod-occ-pr-self/command.yaml
The contract_sha256 field is updated to a new SHA-256 digest in five existing receipt files; all other fields remain unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐇 A new receipt hops into the trail,
PR-2042 now leaves its mark,
SHA hashes updated without fail,
Each PASS stamped fresh in the dark.
The contract grows, the evidence stacks —
This bunny checks the drifted tracks! ✅

🚥 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 change: adding an evidence binding for infra PR 2042 that fixes an event_model wiring bug, 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 jonahgabriel/omn-13247-event-model-wiring-occ

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

Adds dod-occ-pr-2042-self (binds this OCC evidence PR #2797) and the matching
dod_evidence entry; both receipts now carry the final contract sha256.
@jonahgabriel
jonahgabriel enabled auto-merge June 19, 2026 08:23
Adding the dod-infra-pr-2042 + dod-occ-pr-2042-self evidence entries changed the
OMN-13247 contract hash. The OCC merge-eligibility validator requires EVERY receipt
for the ticket to pin the current contract hash; the four pre-existing receipts
(2031, 2041, occ-pr-self, occ-pr-2041-self) and the deploy-assessment receipt still
pinned the prior hash. Repin all to sha256:969af9e4... so eligibility passes. The
underlying evidence is unchanged — this only refreshes the contract-hash binding.
@jonahgabriel
jonahgabriel added this pull request to the merge queue Jun 19, 2026
Merged via the queue into dev with commit 085eb78 Jun 19, 2026
43 checks passed
@jonahgabriel
jonahgabriel deleted the jonahgabriel/omn-13247-event-model-wiring-occ branch June 19, 2026 09:14
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