Skip to content

contract(OMN-10278): add dod receipts for omnibase_infra PRs #1562 and #1563 - #916

Merged
jonahgabriel merged 1 commit into
mainfrom
jonah/omn-10278-infra-pr-receipts
May 10, 2026
Merged

jonahgabriel merged 1 commit into
mainfrom
jonah/omn-10278-infra-pr-receipts

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented May 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds PASS receipts binding OMN-10278 to omnibase_infra PRs #1562 and #1563, satisfying the receipt gate for both PRs.

  • drift/dod_receipts/OMN-10278/dod-infra-pr-1562/command.yaml -- handler = None removal PR
  • drift/dod_receipts/OMN-10278/dod-infra-pr-1563/command.yaml -- terminal_event contract fix PR
  • contracts/OMN-10278.yaml -- dod_evidence items added for both receipts

Ticket: OMN-10278
Evidence-Source: OMN-10278
Evidence-Ticket: OMN-10278

@coderabbitai

coderabbitai Bot commented May 10, 2026 •

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

This PR adds three evidence record files to document GitHub PR validations for ticket OMN-10278. The contract file declares three new dod_evidence entries, and three corresponding command.yaml files capture the verification probes, statuses, and metadata for two infra PRs and one self-binding receipt.

Changes

OMN-10278 Evidence Records

Layer / File(s) Summary
Contract Declaration
contracts/OMN-10278.yaml
Adds three dod_evidence entries to OMN-10278 contract: dod-infra-pr-1562, dod-infra-pr-1563, and dod-occ-pr-916, each linking to command-based checks.
PR 1562 Evidence
drift/dod_receipts/OMN-10278/dod-infra-pr-1562/command.yaml
Records command-based validation of GitHub PR #1562 in omnibase_infra with probe stdout (PR state OPEN, mergeable MERGEABLE), status PASS, and execution metadata (timestamp, commit SHA, runner, verifier).
PR 1563 Evidence
drift/dod_receipts/OMN-10278/dod-infra-pr-1563/command.yaml
Records command-based validation of GitHub PR #1563 in omnibase_infra with probe results (PR state, mergeable status, head OID), status PASS, and execution metadata (timestamp, runner, commit SHA, exit code).
Self-Binding Receipt PR 916
drift/dod_receipts/OMN-10278/dod-occ-pr-916/command.yaml
Records a validation receipt confirming status PASS, ticket_id OMN-10278, and pr_number 916 fields via grep-based command check, with probe results and execution context.

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Three evidence files hop into place,
PR checks and receipts in the trace,
OMN-10278 is now complete,
With validations so neat!
The drift records their feat. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: adding dod receipts for two omnibase_infra pull requests to the OMN-10278 contract.
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.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jonah/omn-10278-infra-pr-receipts

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

…#1563

Adds receipt files and dod_evidence entries binding OMN-10278 to:
- omnibase_infra PR #1562 (remove = None defaults from injectable handler params)
- omnibase_infra PR #1563 (declare terminal_event in node_delegation_orchestrator contract)
- OCC PR #916 self-binding receipt

Rebased cleanly on main to resolve CONFLICTING merge state.
@jonahgabriel
jonahgabriel force-pushed the jonah/omn-10278-infra-pr-receipts branch from 669e922 to b180bd6 Compare May 10, 2026 23:43
@jonahgabriel
jonahgabriel merged commit 1134eb0 into main May 10, 2026
30 of 31 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-10278-infra-pr-receipts branch May 10, 2026 23:50
jonahgabriel added a commit that referenced this pull request May 26, 2026
… batch (#1691)

* contracts(OMN-12208..12231): add OCC evidence for Wave 2-6 stub nodes batch

Add OCC contracts and DoD receipts for 10 omnimarket PRs (Wave 2-6 stub nodes):
- OMN-12208 (#903): node_self_healing_dispatch_orchestrator — 20 tests
- OMN-12211 (#908): node_pipeline_audit_orchestrator — 24 tests
- OMN-12216 (#909): node_recall_compute — 18 tests
- OMN-12222 (#910): Wave 5 contract_drift + verification_sweep — 42 tests
- OMN-12219 (#911): node_decision_store_orchestrator — 10 tests
- OMN-12218 (#912): node_runner_orchestrator — 13 tests
- OMN-12217 (#913): node_observability_sink_effect — 14 tests
- OMN-12228 (#914): Wave 6 design_to_plan Phase3 + generate_node — 21 tests
- OMN-12231 (#915): Wave 6 6 COMPUTE stubs — 36 tests
- OMN-12225 (#916): Wave 6 5 batch nodes — 50 tests

Also adds hardcoded_topic to the adapter_occ_contract.py allowlist to baseline
the pre-existing topic violation introduced in OMN-10278 (PR #885).

* chore: update receipt commit_sha to post-trigger-CI PR heads
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