Skip to content

Trim runtime envelope dispatch helper surface#1455

Merged
shuxueshuxue merged 1 commit into
devfrom
trim-runtime-envelope-dispatch-helpers
May 12, 2026
Merged

Trim runtime envelope dispatch helper surface#1455
shuxueshuxue merged 1 commit into
devfrom
trim-runtime-envelope-dispatch-helpers

Conversation

@shuxueshuxue
Copy link
Copy Markdown
Collaborator

Summary

  • remove unused envelope-level dispatch helper exports from runtime action adapters
  • keep gateway dispatch inside each adapter action dispatcher
  • add a boundary test to keep envelope dispatch helpers internal

Verification

  • RED: uv run pytest tests/Unit/backend/web/services/test_agent_runtime_gateway_protocol_boundary.py -q failed before implementation because dispatch_runtime_chat_delivery_envelopes was still exported
  • uv run pytest tests/Unit/backend/web/services/test_agent_runtime_gateway_protocol_boundary.py tests/Unit/backend/web/services/test_runtime_notification_action.py tests/Unit/backend/web/services/test_chat_delivery_hook.py tests/Unit/integration_contracts/test_threads_router.py -q -> 65 passed
  • uv run pyright backend/threads/chat_adapters/runtime_chat_delivery_action.py backend/threads/chat_adapters/runtime_notification_action.py backend/threads/chat_adapters/runtime_thread_input_action.py tests/Unit/backend/web/services/test_agent_runtime_gateway_protocol_boundary.py tests/Unit/integration_contracts/test_threads_router.py -> 0 errors
  • uv run ruff check backend/threads/chat_adapters/runtime_chat_delivery_action.py backend/threads/chat_adapters/runtime_notification_action.py backend/threads/chat_adapters/runtime_thread_input_action.py tests/Unit/backend/web/services/test_agent_runtime_gateway_protocol_boundary.py tests/Unit/backend/web/services/test_runtime_notification_action.py tests/Unit/backend/web/services/test_chat_delivery_hook.py tests/Unit/integration_contracts/test_threads_router.py -> All checks passed
  • uv run ruff format --check backend/threads/chat_adapters/runtime_chat_delivery_action.py backend/threads/chat_adapters/runtime_notification_action.py backend/threads/chat_adapters/runtime_thread_input_action.py tests/Unit/backend/web/services/test_agent_runtime_gateway_protocol_boundary.py tests/Unit/integration_contracts/test_threads_router.py -> 5 files already formatted
  • uv run pytest tests/Unit -q -> 2239 passed, 3 skipped, 15 warnings

@shuxueshuxue shuxueshuxue merged commit db89ea9 into dev May 12, 2026
7 checks passed
@shuxueshuxue shuxueshuxue deleted the trim-runtime-envelope-dispatch-helpers branch May 12, 2026 23: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