Skip to content

receipt(OMN-10943): bind omniclaude delegate shim PR - #997

Merged
jonahgabriel merged 1 commit into
mainfrom
jonah/omn-10943-omniclaude-1578-receipt
May 13, 2026
Merged

jonahgabriel merged 1 commit into
mainfrom
jonah/omn-10943-omniclaude-1578-receipt

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented May 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Verification

  • uv run pytest tests/test_dod_evidence_schema.py tests/unit/scripts/test_check_dod_compliance.py -q
  • PYTHONPATH=/Users/jonah/Code/omni_home/omnibase_core/src uv run python -m omnibase_core.validation.validator_occ_merge_eligibility --repo OmniNode-ai/omniclaude --pr-number 1578 --pr-title "fix(OMN-10943): prefer runtime HTTP delegation dispatch" --pr-body-file /tmp/omniclaude-1578-body.md --pr-branch jonah/omn-10943-delegate-shim-runtime-http --pr-commit-sha b0ec43d7e45d4d85ad3a639c6a1a1309684269aa --pr-commit-text "fix(OMN-10943): prefer runtime HTTP delegation dispatch" --occ-commit-sha 0000000000000000000000000000000000000000 --contracts-dir contracts --receipts-dir drift/dod_receipts

Summary by CodeRabbit

  • Chores
    • Added additional manual validation evidence and a new automated command-based check to the validation framework.
    • Updated an existing command-based receipt to reflect a recent verification run and its results, improving audit trail fidelity and compliance reporting.

Review Change Stack

@coderabbitai

coderabbitai Bot commented May 13, 2026 •

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 749b8349-cab4-4a18-a1e5-69fd46f54363

📥 Commits

Reviewing files that changed from the base of the PR and between 8993823 and f242849.

📒 Files selected for processing (3)
  • contracts/OMN-10943.yaml
  • drift/dod_receipts/OMN-10943/dod-occ-pr/command.yaml
  • drift/dod_receipts/OMN-10943/dod-omniclaude-pr-1578/command.yaml
✅ Files skipped from review due to trivial changes (2)
  • drift/dod_receipts/OMN-10943/dod-occ-pr/command.yaml
  • drift/dod_receipts/OMN-10943/dod-omniclaude-pr-1578/command.yaml
🚧 Files skipped from review as they are similar to previous changes (1)
  • contracts/OMN-10943.yaml

📝 Walkthrough

Walkthrough

Adds a manual evidence requirement for omniclaude PR #1578 to OMN-10943 and records its PASS in a new drift receipt; also updates the OCC PR command receipt metadata to reference PR #997.

Changes

OMN-10943 Evidence Requirement and Receipt

Layer / File(s) Summary
Evidence requirement and receipt for omniclaude PR #1578
contracts/OMN-10943.yaml, drift/dod_receipts/OMN-10943/dod-omniclaude-pr-1578/command.yaml
Adds a manual gh pr diff ... --name-only evidence requirement for omniclaude PR #1578 and records a passing command receipt with probe_command, probe_stdout (affected paths), status PASS, exit_code 0, and provenance (run_timestamp, commit_sha, runner, verifier, pr_number).
Update OCC PR command receipt
drift/dod_receipts/OMN-10943/dod-occ-pr/command.yaml
Replaces recorded execution metadata and probe results to target PR #997 (updated run_timestamp, commit_sha, runner, verifier, probe_command, probe_stdout, actual_output, and pr_number).

Estimated Code Review Effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly Related PRs

Poem

🐰 I found a receipt, I gave it a hop,
A gh pr diff check—then a joyful stop.
PR one-five-seven-eight waved PASS today,
Contracts and drift files now sing hooray.
🥕

🚥 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 directly references OMN-10943 and describes binding the omniclaude delegate shim PR, which aligns with the main objective of adding an evidence item for omniclaude PR #1578 to OMN-10943.
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-10943-omniclaude-1578-receipt

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

@jonahgabriel
jonahgabriel enabled auto-merge (squash) May 13, 2026 19:02
@jonahgabriel
jonahgabriel force-pushed the jonah/omn-10943-omniclaude-1578-receipt branch from 8993823 to f242849 Compare May 13, 2026 19:03
@jonahgabriel
jonahgabriel merged commit 7a975c8 into main May 13, 2026
29 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-10943-omniclaude-1578-receipt branch May 13, 2026 19:07
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