Skip to content

contract(OMN-10698): add dod-pr-592-head receipt for ground truth manifest PR - #911

Merged
jonahgabriel merged 3 commits into
mainfrom
jonah/omn-10698-pr-592-receipt
May 10, 2026
Merged

jonahgabriel merged 3 commits into
mainfrom
jonah/omn-10698-pr-592-receipt

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented May 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

OMN-10698

Evidence-Source: c91fc3a

Summary by CodeRabbit

  • Documentation

    • Updated contract documentation to reference follow-up work and curated ground-truth manifest entries.
  • Chores

    • Added verification records with manual validation checks for configuration status tracking.

Review Change Stack

Evidence-Ticket: OMN-10698

…ifest PR

PR #592 populates the ADR canary ground truth manifest with 3 real
hand-curated entries and adds onex-allow-model-id annotations to fix
the leaked literals gate. Binds the evidence to OCC.
@coderabbitai

coderabbitai Bot commented May 10, 2026 •

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Contract OMN-10698 is updated with summary text mentioning PR #592's ground-truth manifest population and onex-allow-model-id annotations. A new evidence entry references this PR, backed by a command verification record that documents the gh pr view check output and PASS status.

Changes

PR #592 Evidence Documentation

Layer / File(s) Summary
Contract Reference and Evidence Entry
contracts/OMN-10698.yaml
Contract summary extended to reference PR #592's ADR ground-truth manifest and annotation work; new dod_evidence entry (dod-pr-592-head) added with manual command check asserting PASS status.
Evidence Verification Record
drift/dod_receipts/OMN-10698/dod-pr-592-head/command.yaml
New verification record captures gh pr view command execution, timestamp, commit SHA, runner metadata, and recorded PASS status with branch context for PR #592.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A contract grows with evidence so clear,
PR #592 marked for the year,
The manifest curated, annotations in place,
Each proof recorded, sealed with grace. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding a dod-pr-592-head receipt to the OMN-10698 contract for the ground truth manifest PR, which aligns with the actual changeset modifications.
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-10698-pr-592-receipt

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

@jonahgabriel
jonahgabriel merged commit 0c2ed3c into main May 10, 2026
29 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-10698-pr-592-receipt branch May 10, 2026 15:39
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