Skip to content

feat(OMN-10821): add omnimarket handler gate evidence - #1224

Merged
jonahgabriel merged 3 commits into
mainfrom
jonah/omn-10821-omnimarket-697-evidence
May 20, 2026
Merged

jonahgabriel merged 3 commits into
mainfrom
jonah/omn-10821-omnimarket-697-evidence

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented May 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Adds central OMN-10821 contract evidence for omnimarket PR docs(OMN-10530): add omniintelligence cascade evidence #697
  • Records focused local handler-gate validation at omnimarket head 7a9cd3720616e6fd0a9fdfa71fb2098be687f71d
  • Binds the omnimarket PR to the OCC receipt source required by Receipt Gate

Validation

  • uv run python -m omnibase_core.validation.receipt_gate_cli --pr-body "Closes OMN-10821" --pr-title "feat(OMN-10821): add unimported-handler CI check for dead Handler* classes" --contracts-dir contracts --receipts-dir drift/dod_receipts --evidence-ticket OMN-10821 --branch-name jonah/omn-10821-add-unimported-handler-ci-check

Evidence-Ticket: OMN-10821

Summary by CodeRabbit

Release Notes

  • Chores
    • Implemented new CI validation framework including automated check configuration definitions, validation evidence recording mechanisms, and compliance verification audit trails to strengthen quality assurance and traceability across the development process.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 20, 2026 •

Copy link
Copy Markdown

Warning

Rate limit exceeded

@jonahgabriel has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 2 minutes and 30 seconds before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b7d3da42-f63e-49fc-b684-003aa17eb2bb

📥 Commits

Reviewing files that changed from the base of the PR and between 48651fc and 95aa379.

📒 Files selected for processing (1)
  • drift/dod_receipts/OMN-10821/dod-occ-pr-1224/command.yaml
📝 Walkthrough

Walkthrough

This PR adds a new ticket contract and supporting evidence records for OMN-10821. The contract file declares what evidence is required, while three receipt files provide timestamped proof that local validation, PR #1224, and an external omnimarket PR #697 all successfully completed their checks with PASS status.

Changes

OMN-10821 Ticket Evidence

Layer / File(s) Summary
OMN-10821 Contract Definition
contracts/OMN-10821.yaml
Contract defines ticket metadata for "Add unimported-handler CI check," specifies evidence requirements (local validation with pytest/ruff, CI check for omnimarket PR #697), lists DOD receipt references, and disables emergency bypass.
Evidence Receipt Records
drift/dod_receipts/OMN-10821/dod-local-validation/command.yaml, drift/dod_receipts/OMN-10821/dod-occ-pr-1224/command.yaml, drift/dod_receipts/OMN-10821/dod-omnimarket-pr-697/command.yaml
Three receipts document proof of checks: local validation captures pytest/ruff/diff results; PR #1224 receipt records GitHub PR state and metadata; omnimarket PR #697 receipt captures changed files and PR details. All receipts record PASS status, execution metadata, commit SHAs, and captured probe output.

🎯 2 (Simple) | ⏱️ ~8 minutes

A rabbit hops in with proof so neat,
OMN-10821 now complete!
Contracts and receipts, all aligned,
Validation checks all PASS assigned. 🐰✓

🚥 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 summarizes the main change: adding evidence for the omnimarket handler gate to OMN-10821, which is directly reflected in the four added files documenting contract evidence and command receipts.
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-10821-omnimarket-697-evidence

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

@jonahgabriel
jonahgabriel force-pushed the jonah/omn-10821-omnimarket-697-evidence branch from 7be9c35 to 39f516f Compare May 20, 2026 11:10
@jonahgabriel
jonahgabriel force-pushed the jonah/omn-10821-omnimarket-697-evidence branch from 39f516f to 4f5e76d Compare May 20, 2026 11:13

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@drift/dod_receipts/OMN-10821/dod-occ-pr-1224/command.yaml`:
- Around line 6-14: The check declares extra JSON fields but the probe_command
and probe_stdout don't include them: update the probe_command to match
check_value by adding the `files` field (i.e., make probe_command use `--json
number,state,headRefName,headRefOid,files`), then re-run that GH PR view command
and replace probe_stdout with the resulting JSON output that includes `files`;
ensure the commit_sha, number, headRefName, headRefOid and files entries in
probe_stdout reflect the actual command output.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 73a63014-7328-4ec0-a36a-87d81943e6a3

📥 Commits

Reviewing files that changed from the base of the PR and between c38413e and 48651fc.

📒 Files selected for processing (4)
  • contracts/OMN-10821.yaml
  • drift/dod_receipts/OMN-10821/dod-local-validation/command.yaml
  • drift/dod_receipts/OMN-10821/dod-occ-pr-1224/command.yaml
  • drift/dod_receipts/OMN-10821/dod-omnimarket-pr-697/command.yaml

Comment thread drift/dod_receipts/OMN-10821/dod-occ-pr-1224/command.yaml Outdated
@jonahgabriel
jonahgabriel merged commit 85c3791 into main May 20, 2026
32 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-10821-omnimarket-697-evidence branch May 20, 2026 11:27
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