Skip to content

fix(OMN-11811): baseline merged swarm guard debt - #1492

Merged
jonahgabriel merged 5 commits into
mainfrom
jonah/omn-11811-swarm-guard-allowlist
May 24, 2026
Merged

jonahgabriel merged 5 commits into
mainfrom
jonah/omn-11811-swarm-guard-allowlist

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented May 24, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Validation

  • uv run check-imperative-contracts --repo-root /Users/jonah/Code/omni_home/omni_worktrees/OMN-11777/omnimarket-pr778 --allowlists-dir allowlists
  • uv run check-imperative-contracts --repo-root /Users/jonah/Code/omni_home/omni_worktrees/OMN-11811/omnimarket --allowlists-dir allowlists
  • uv run check-imperative-contracts --repo-root /Users/jonah/Code/omni_home/omni_worktrees/OMN-11813/omnimarket --allowlists-dir allowlists

Evidence-Ticket: OMN-11811

Summary by CodeRabbit

Release Notes

  • Chores
    • Updated allowlist configurations with two new handler entries associated with respective tickets.
    • Added receipt documentation capturing command check validation and metadata for compliance tracking.

Review Change Stack

@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 currently allows 1 review/hour. Refill in 53 minutes and 12 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: 1af1b957-3ccb-40e0-8f10-5d796b87f503

📥 Commits

Reviewing files that changed from the base of the PR and between 9366abc and 75a0211.

📒 Files selected for processing (5)
  • contracts/OMN-11811.yaml
  • drift/dod_receipts/OMN-11811/dod-deploy/command.yaml
  • drift/dod_receipts/OMN-11811/dod-occ-pr-1462/command.yaml
  • drift/dod_receipts/OMN-11811/dod-occ-pr-1492/command.yaml
  • drift/dod_receipts/OMN-11811/dod-omnimarket-pr-783/command.yaml
📝 Walkthrough

Walkthrough

This PR adds two new handler allowlist entries to the omnimarket allowlist for swarm-related components, and records a command receipt documenting evidence collection and validation status for the PR.

Changes

Handler Allowlist and Evidence Record

Layer / File(s) Summary
Handler allowlist entries
allowlists/omnimarket.yaml
Adds two new allowlisted_handlers entries: swarm decomposer handler with hybrid violation and ticket OMN-11809, and endpoint health handler with imperative violation and ticket OMN-11810.
Evidence receipt
drift/dod_receipts/OMN-11811/dod-occ-pr-1492/command.yaml
Records command check receipt with schema metadata, probe command execution details, PASS status, timestamp and commit information, and contract hash verification.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • OmniNode-ai/onex_change_control#1300: The new command receipt includes contract_sha256 verification field, consistent with the PR #1300 addition of contract hash evidence to OMN-11424 command receipts.

Poem

🐰 Two handlers now safely dwell,
In allowlists where rabbits tell,
Of swarms that decompose with care,
While evidence floats through the air!
Receipt recorded, validation done—
A tidy PR, completely won! ✨

🚥 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 'fix(OMN-11811): baseline merged swarm guard debt' is directly related to the changeset, which adds allowlist entries for swarm handlers and a receipt to resolve the OMN-11811 ticket regarding swarm guard debt.
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-11811-swarm-guard-allowlist

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

@jonahgabriel
jonahgabriel merged commit 326b41f into main May 24, 2026
33 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-11811-swarm-guard-allowlist branch May 24, 2026 00:50
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