Skip to content

chore(OMN-10081): add runtime local ingress receipts - #472

Merged
jonahgabriel merged 1 commit into
mainfrom
jonah/omn-10081-receipts
Apr 28, 2026
Merged

jonahgabriel merged 1 commit into
mainfrom
jonah/omn-10081-receipts

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented Apr 28, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • add PASS DoD receipts for OMN-10081 runtime local ingress transport
  • cover socket lifecycle, host-process dispatch, and route discovery acceptance checks

Verification

  • uv run pytest tests/unit/runtime/test_runtime_local_ingress.py::test_runtime_local_ingress_server_round_trip tests/unit/runtime/test_runtime_local_ingress.py::test_runtime_local_ingress_server_refuses_non_socket_path -q
  • uv run pytest tests/integration/runtime/test_runtime_local_ingress.py -q
  • uv run pytest tests/unit/runtime/test_runtime_local_ingress.py::test_runtime_host_process_dispatch_local_ingress_uses_handler_semaphore tests/unit/runtime/test_runtime_local_ingress.py::test_discover_runtime_local_ingress_routes_skips_malformed_contract -q
  • PYTHONPATH=/Users/jonah/Code/omni_home/omnibase_core/src uv run python -m omnibase_core.validation.receipt_gate_cli --pr-body "$body" --pr-title "$title" --contracts-dir contracts --receipts-dir drift/dod_receipts

Summary by CodeRabbit

  • Tests
    • Added verification evidence records documenting integration test execution results for ingress server behavior and socket handling validation.

@github-actions
github-actions Bot enabled auto-merge April 28, 2026 01:45
@coderabbitai

coderabbitai Bot commented Apr 28, 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: 8516d9b6-d8e0-49d4-b057-0e2289f9a573

📥 Commits

Reviewing files that changed from the base of the PR and between e172148 and 6666ccc.

📒 Files selected for processing (3)
  • drift/dod_receipts/OMN-10081/dod-001/command.yaml
  • drift/dod_receipts/OMN-10081/dod-002/command.yaml
  • drift/dod_receipts/OMN-10081/dod-003/command.yaml

📝 Walkthrough

Walkthrough

This change adds three new test evidence receipt YAML files (dod-001, dod-002, dod-003) for ticket OMN-10081, each documenting pytest execution results with PASS status and exit code 0, capturing test output and metadata including commit SHA, runner details, and timestamp information.

Changes

Cohort / File(s) Summary
Test Evidence Receipts
drift/dod_receipts/OMN-10081/dod-001/command.yaml, drift/dod_receipts/OMN-10081/dod-002/command.yaml, drift/dod_receipts/OMN-10081/dod-003/command.yaml
Adds three new YAML evidence receipt files documenting pytest command execution results. dod-001 records 2 passed tests for ingress server behavior; dod-002 records 1 passed test for socket integration flow; dod-003 records 2 passed tests for semaphore and contract handling. Each includes schema version, metadata (commit SHA, timestamp, runner, verifier, PR number), command details, stdout output, and pass/fail status.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Three receipts hop into place,
Test results saved with elegant grace,
Exit codes zero, all passing with cheer,
Evidence gathered, the proof crystal clear!
—A rabbit's delight in verification's embrace 🐇✨

🚥 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 PR title accurately and specifically describes the main change: adding DoD receipts for the OMN-10081 runtime local ingress feature.
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-10081-receipts

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

@jonahgabriel
jonahgabriel added this pull request to the merge queue Apr 28, 2026
Merged via the queue into main with commit 45bdac2 Apr 28, 2026
27 checks passed
@jonahgabriel
jonahgabriel deleted the jonah/omn-10081-receipts branch April 28, 2026 01:50
jonahgabriel added a commit that referenced this pull request May 26, 2026
…1652)

* contracts(OMN-12154): add OCC contract for Valkey rate limiter fix

Documents the ValkeyRateLimiter implementation that replaces
InMemoryRateLimiter in omninode_infra. DoD evidence: PR #472, 11 unit
tests, 2 integration tests, 171 existing tests with no regressions.

* contracts(OMN-12154): add hash-bound receipts
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