Skip to content

contracts(OMN-11913): add deploy evidence for omnimarket 824 - #1552

Merged
jonahgabriel merged 4 commits into
mainfrom
jonah/omn-11913-occ-deploy-evidence
May 24, 2026
Merged

jonahgabriel merged 4 commits into
mainfrom
jonah/omn-11913-occ-deploy-evidence

Conversation

@jonahgabriel

Copy link
Copy Markdown
Contributor

Summary

Adds follow-up central OCC evidence for OMN-11913 / omnimarket PR #824 after the product head moved to ca0ab5b3.

Verification

  • env -u LC_ALL -u LC_CTYPE LANG=C pre-commit run --files contracts/OMN-11913.yaml drift/dod_receipts/OMN-11913/dod-antipattern-index-local-validation/command.yaml drift/dod_receipts/OMN-11913/dod-occ-pr/command.yaml drift/dod_receipts/OMN-11913/dod-omnimarket-pr-824/command.yaml drift/dod_receipts/OMN-11913/dod-deploy-readiness/command.yaml

Ticket

OMN-11913

@coderabbitai

coderabbitai Bot commented May 24, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

@jonahgabriel, we couldn't start this review because you've used your available PR reviews for now.

Your plan includes 5 reviews of capacity. Refill in 26 minutes and 3 seconds.

Your organization has run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After more review capacity refills, 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 trial, open-source, and free plans. In all cases, review capacity refills continuously over time.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 67a51dd5-610c-4f81-9cb1-00c4e7696369

📥 Commits

Reviewing files that changed from the base of the PR and between e4d012f and 94673fc.

📒 Files selected for processing (5)
  • contracts/OMN-11913.yaml
  • drift/dod_receipts/OMN-11913/dod-antipattern-index-local-validation/command.yaml
  • drift/dod_receipts/OMN-11913/dod-deploy-readiness/command.yaml
  • drift/dod_receipts/OMN-11913/dod-occ-pr/command.yaml
  • drift/dod_receipts/OMN-11913/dod-omnimarket-pr-824/command.yaml
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch jonah/omn-11913-occ-deploy-evidence

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

@jonahgabriel
jonahgabriel merged commit 5f6f8b7 into main May 24, 2026
33 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-11913-occ-deploy-evidence branch May 24, 2026 20:25
andywu42 pushed a commit to andywu42/onex_change_control that referenced this pull request Jun 10, 2026
OmniNode-ai#868)

* feat(OMN-10734): add OCC contract and receipts for omniclaude fallback sweep

Binds omniclaude PR OmniNode-ai#1552 (jonah/omn-10734-fallback-sweep-claude) to central
OCC evidence. The PR removes localhost defaults from context_config.py and wires
validate_no_env_fallbacks.py as pre-commit hook + arch-no-env-fallbacks CI gate.

* fix(OMN-10734): add pr_number and correct commit_sha to receipts

Receipts must include pr_number binding to OCC PR OmniNode-ai#868 so the eligibility
validator can confirm they are bound to this PR. Also fixed placeholder
commit_sha to actual OCC branch HEAD SHA and aligned check_value fields
with the contract's grep-based check pattern.
andywu42 pushed a commit to andywu42/onex_change_control that referenced this pull request Jun 10, 2026
…OCC SHA (OmniNode-ai#874)

* fix(OMN-10734): correct receipts to bind to omniclaude PR OmniNode-ai#1552

Sets pr_number=1552 (omniclaude PR) and commit_sha to the omniclaude
commit SHA so the eligibility validator can bind the receipts to the
omniclaude PR being reviewed. Previous OCC PR OmniNode-ai#868 incorrectly set
pr_number=868 (the OCC PR itself).

* fix(OMN-10734): set commit_sha to OCC branch SHA for self-binding gate

All three receipts now carry commit_sha=d61beb07 (this OCC PR's commit),
enabling validator_occ_merge_eligibility to bind via SHA match when
receipt.pr_number (1552) != OCC PR number.

* fix(OMN-10734): update dod-occ-pr receipt to reference OCC PR OmniNode-ai#874
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