Skip to content

fix(OMN-12245): register full onex dispatch topics - #1774

Merged
jonahgabriel merged 4 commits into
mainfrom
hotfix/omn-12245-event-topic-dispatch
May 27, 2026
Merged

jonahgabriel merged 4 commits into
mainfrom
hotfix/omn-12245-event-topic-dispatch

Conversation

@jonahgabriel

@jonahgabriel jonahgabriel commented May 27, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • register full onex.*.v1 subscribe topics as dispatch message_types alongside dot-path aliases
  • preserve existing class-name and alias routing while accepting canonical ModelEventEnvelope.event_type values
  • bump omnibase-infra to 0.37.2 for the follow-up runtime hotfix release

Verification

  • uv run pytest tests/unit/runtime/auto_wiring/test_handler_wiring_event_type_alias.py tests/integration/test_handler_event_type_alias_integration.py tests/unit/runtime/auto_wiring/test_wiring.py tests/integration/envelope_routing/test_projection_topic_extraction.py tests/integration/projectors/test_handler_wiring_projection_dispatch.py -q -> 58 passed
  • uv run ruff check src/omnibase_infra/runtime/auto_wiring/handler_wiring.py tests/unit/runtime/auto_wiring/test_handler_wiring_event_type_alias.py tests/integration/test_handler_event_type_alias_integration.py -> passed
  • git diff --check -> clean

Runtime Evidence

Stability .201 on 0.37.1 proved package identity and tier escalation code, then exposed the remaining dispatcher miss: canonical publish to onex.evt.omniclaude.task-delegated.v1 with ModelEventEnvelope.event_type set to the full topic went to DLQ because auto-wiring only registered omniclaude.task-delegated.

This PR adds the missing full-topic key and a regression for that exact task-delegated envelope case. Post-merge, the manual sweep will rebuild/redeploy .201 and rerun the projection DB proof.

Evidence-Ticket: OMN-12245
Evidence-Source: OCC#1785
hotfix-evidence: OCC-1785
backmerge: #1774

@coderabbitai

coderabbitai Bot commented May 27, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8354c9ef-1f4f-4e8b-b90e-c3b8649a43ec

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hotfix/omn-12245-event-topic-dispatch

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

@jonahgabriel
jonahgabriel merged commit eb1a402 into main May 27, 2026
123 of 126 checks passed
@jonahgabriel
jonahgabriel deleted the hotfix/omn-12245-event-topic-dispatch branch May 27, 2026 19:29
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