Skip to content

evidence(OMN-13405): OCC contract + receipts for workspace core dev-HEAD vendoring - #2875

Merged
jonahgabriel merged 2 commits into
devfrom
jonah/omn-13405-occ-evidence
Jun 21, 2026
Merged

jonahgabriel merged 2 commits into
devfrom
jonah/omn-13405-occ-evidence

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented Jun 21, 2026 •

Copy link
Copy Markdown
Contributor

Carries contracts/OMN-13405.yaml (central Evidence-Source for omnibase_infra PR for OMN-13405's Receipt Gate) + PASS dod_receipts for:

  • dod-001 — workspace build installs omnibase_core from a file:// local path (defeating [tool.uv.sources]); build-provenance proofs pass.
  • dod-002 — cold dev-lane bring-up HEALTHY: omnimarket-projection-api restarts=0 /health 200 (v4 ModuleNotFoundError gone), runtime/effects healthy, broker EventBusKafka, migrations 0017/0018/0019 applied.
  • dod-occ-pr — self-bind.

Pairs with the omnibase_infra fix PR (OMN-13405). Evidence: .onex_state/runtime-e2e-2026-06-21/02-dev-deploy/a6/CAPTURE.md.

Summary by CodeRabbit

  • Chores
    • Added build contract and verification documentation for ticket OMN-13405, resolving dependency handling for the core package in workspace mode.
    • Included test verification receipts confirming system health and configuration validity across multiple checks.

…EAD vendoring

Carries contracts/OMN-13405.yaml (central Evidence-Source for omnibase_infra
PR #2058 Receipt Gate) + PASS dod_receipts for dod-001 (workspace build installs
core from file:// local path; build provenance green), dod-002 (cold dev bring-up
HEALTHY, projection-api restarts=0 /health 200, enum imports, broker EventBusKafka,
migrations 0017/0018/0019), and dod-occ-pr (self-bind).
@coderabbitai

coderabbitai Bot commented Jun 21, 2026 •

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@jonahgabriel, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 44 minutes and 11 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based credits.

🚦 How do rate limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan refill rate.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, the refill rate gradually slows as usage increases. The highest same-day bursts are limited more strictly.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e6b2dddd-26b4-40d3-b024-8369884146c6

📥 Commits

Reviewing files that changed from the base of the PR and between 84849e6 and fee5cde.

📒 Files selected for processing (3)
  • drift/dod_receipts/OMN-13405/dod-001/command.yaml
  • drift/dod_receipts/OMN-13405/dod-002/command.yaml
  • drift/dod_receipts/OMN-13405/dod-occ-pr/command.yaml
📝 Walkthrough

Walkthrough

Adds contracts/OMN-13405.yaml declaring that omnibase_core is installed from a dev-HEAD workspace clone in dev mode (not the lock-pinned wheel), while release mode remains unchanged. Three DoD evidence receipt files are added under drift/dod_receipts/OMN-13405/, each recording a PASS outcome for pytest, docker health, and OCC PR grep checks.

Changes

OMN-13405 Contract and DoD Receipts

Layer / File(s) Summary
Workspace-mode build contract
contracts/OMN-13405.yaml
Defines schema/ticket metadata, workspace-install intent for omnibase_core from dev-HEAD, release-mode unchanged declaration, is_seam_ticket: false, empty interfaces_touched, and dod_evidence entries with grep-based verification commands.
DoD evidence receipts
drift/dod_receipts/OMN-13405/dod-001/command.yaml, drift/dod_receipts/OMN-13405/dod-002/command.yaml, drift/dod_receipts/OMN-13405/dod-occ-pr/command.yaml
Three command.yaml receipts recording PASS outcomes: dod-001 captures pytest run (15 passed in 1.06s), dod-002 captures docker container health probe (healthy 0), and dod-occ-pr captures grep-based verification of the contract file presence in the OCC PR tree. Each includes timestamps, commit SHA, contract hash, runner/verifier IDs, and PR number.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐇 A contract is signed, the receipts are in place,
omnibase_core installed at workspace pace.
No crash-loops at import, no lock-wheel to dread,
The kernel stays healthy, dev-HEAD straight ahead.
Three DOD checks: all green, all PASS—
This bunny hops forward, fast as the grass! 🌿

🚥 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 clearly and specifically summarizes the main change: adding an OCC contract and receipts for workspace core dev-HEAD vendoring related to ticket OMN-13405, which matches the content of the PR.
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-13405-occ-evidence

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

@jonahgabriel
jonahgabriel enabled auto-merge June 21, 2026 09:51
…bility

OCC eligibility (pr_ticket_mismatch) requires each PASS receipt's commit_sha to be
a commit on the OCC PR; the prior receipts referenced the infra feature SHA. Point
commit_sha at the OCC PR's first commit 84849e6 (in PR #2875's commit set).
@jonahgabriel
jonahgabriel added this pull request to the merge queue Jun 21, 2026
Merged via the queue into dev with commit be26de8 Jun 21, 2026
38 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-13405-occ-evidence branch June 21, 2026 10:03
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