Skip to content

evidence(OMN-14397): OCC companion for omnibase_infra PR 2265 - #3973

Merged
jonahgabriel merged 2 commits into
devfrom
jonah/omn-14397-occ-pr2265
Jul 12, 2026
Merged

jonahgabriel merged 2 commits into
devfrom
jonah/omn-14397-occ-pr2265

Conversation

@jonahgabriel

Copy link
Copy Markdown
Contributor

Summary

  • OCC companion for omnibase_infra PR chore(OMN-12766): promote receipt integrity corrections to OCC main #2265 (OMN-14397): onex delegate gets a fresh correlation_id per invocation and a SIGALRM-based hard timeout backstop.
  • This companion carries independent adversarial verification across both rounds: round 1 found a real DEFECT (the SIGALRM-raised DelegateTimeoutExceededError(RuntimeError) was silently swallowed by run_receipt_mode's own broad except Exception around the exact call that hangs); round 2 remediation (BaseException base class, commit 3e5e86d8) was independently reproduced RED→GREEN, with an explicit audit confirming no remaining BaseException escape path and finally-block integrity under exception unwind.
  • runner: fix-14397 / verifier: verify-14397 — distinct roles, satisfying enforce_adversarial_invariants().

Evidence

  • contracts/OMN-14397.yaml
  • drift/dod_receipts/OMN-14397/dod-omnibase-infra-pr-2265-head-3e5e86d8/command.yaml
  • drift/dod_receipts/OMN-14397/dod-deploy-assessment-head-3e5e86d8/command.yaml
  • drift/dod_receipts/OMN-14397/dod-occ-self-commit/command.yaml

Closes OMN-14397

Test plan

  • uv run validate-yaml contracts/OMN-14397.yaml — OK
  • uv run pre-commit run --files <all 4 new files> — all hooks passed
  • All three receipt check_value grep probes verified passing locally
  • contract_sha256 computed from the post-yamlfmt formatted contract file, matches across all three receipts

Independent adversarial verification (round 2, post-remediation) of
omnibase_infra PR #2265 head 3e5e86d890cc624860906f0fd32fdf5e23589084.
Round 1 found a real DEFECT (DelegateTimeoutExceededError swallowed by
run_receipt_mode's broad except Exception); round 2 remediation
(BaseException base class) was independently reproduced RED->GREEN,
along with a full audit of the call path for any remaining
BaseException escape hazard. See contract dod_evidence for full detail.

Refs OMN-14397
@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your recent review volume is higher than typical usage, so adaptive limits are currently applied.

Next review available in: 37 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

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 reviews.

How do review limits work?

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

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, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8cfc80c2-92be-4427-ac74-9ee9e19e1463

📥 Commits

Reviewing files that changed from the base of the PR and between 9466ee7 and 6f016b0.

📒 Files selected for processing (4)
  • contracts/OMN-14397.yaml
  • drift/dod_receipts/OMN-14397/dod-deploy-assessment-head-3e5e86d8/command.yaml
  • drift/dod_receipts/OMN-14397/dod-occ-self-commit/command.yaml
  • drift/dod_receipts/OMN-14397/dod-omnibase-infra-pr-2265-head-3e5e86d8/command.yaml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jonah/omn-14397-occ-pr2265

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

@jonahgabriel
jonahgabriel merged commit f47e8ff into dev Jul 12, 2026
45 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-14397-occ-pr2265 branch July 12, 2026 01:37
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