Skip to content

contract(OMN-10278): add dod receipts for omnibase_compat PR #85 and omniclaude PR #1561 - #917

Merged
jonahgabriel merged 3 commits into
mainfrom
jonah/omn-10278-occ-receipts-compat-85-claude-1561
May 10, 2026
Merged

jonahgabriel merged 3 commits into
mainfrom
jonah/omn-10278-occ-receipts-compat-85-claude-1561

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented May 10, 2026 •

Copy link
Copy Markdown
Contributor

OMN-10278

Adds OCC evidence receipts so omnibase_compat#85 and omniclaude#1561 can pass the verify gate:

Also extends the contract with dod_evidence entries for these three items.

Evidence-Ticket: OMN-10278
Evidence-Source: OCC#917

Summary by CodeRabbit

  • Chores
    • Updated internal verification and audit records to validate pull request compatibility and requirements across related components.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 10, 2026 •

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 6128be86-4088-400b-9b72-627d2a916f52

📥 Commits

Reviewing files that changed from the base of the PR and between aab9547 and 223ed8e.

📒 Files selected for processing (4)
  • contracts/OMN-10278.yaml
  • drift/dod_receipts/OMN-10278/dod-occ-pr-917/command.yaml
  • drift/dod_receipts/OMN-10278/dod-omnibase-compat-pr-85/command.yaml
  • drift/dod_receipts/OMN-10278/dod-omniclaude-pr-1561/command.yaml

📝 Walkthrough

Walkthrough

This PR adds evidence records to close design-of-documentation ticket OMN-10278. The contract is updated to declare three evidence requirements. Three new receipt files are added, each containing a GitHub CLI command probe result that verifies PR state (PASS status) for PR #917 (self), PR #85 (omnibase_compat), and PR #1561 (omniclaude).

Changes

DoD Evidence Collection

Layer / File(s) Summary
Evidence Contract Declaration
contracts/OMN-10278.yaml
Contract updated with three dod_evidence entries for PR #85, PR #1561, and PR #917, each specifying a command check that greps for PASS status and corresponding PR identifier in receipt files.
PR #917 Self-Verification Receipt
drift/dod_receipts/OMN-10278/dod-occ-pr-917/command.yaml
Evidence record for current PR adds GitHub CLI probe command, PASS status, run metadata (timestamp, runner, verifier), captured stdout JSON showing PR state and refs, and branch identifier.
Dependency PR Evidence Records
drift/dod_receipts/OMN-10278/dod-omnibase-compat-pr-85/command.yaml, drift/dod_receipts/OMN-10278/dod-omniclaude-pr-1561/command.yaml
Evidence records for PR #85 and PR #1561 each capture GitHub CLI probe results with PASS status, execution metadata, captured PR state output, and commit/branch identifiers.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Poem

🐰 Evidence gathered, tickets grow tight,
Three PRs proved in YAML light,
Receipts are tucked in /drift so deep,
Where GitHub commands their vigil keep.
OMN-10278 sealed with a hop—hop—hooray! ✨

🚥 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 summarizes the main change: adding DOD receipts for two specific PRs (omnibase_compat PR #85 and omniclaude PR #1561) to satisfy contract OMN-10278 verification requirements.
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-10278-occ-receipts-compat-85-claude-1561

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

@jonahgabriel
jonahgabriel merged commit cb2aacd into main May 10, 2026
29 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-10278-occ-receipts-compat-85-claude-1561 branch May 10, 2026 19:40
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