Skip to content

Trim notification batch envelope planner#1457

Merged
shuxueshuxue merged 1 commit into
devfrom
trim-notification-envelope-list-planner
May 12, 2026
Merged

Trim notification batch envelope planner#1457
shuxueshuxue merged 1 commit into
devfrom
trim-notification-envelope-list-planner

Conversation

@shuxueshuxue
Copy link
Copy Markdown
Collaborator

Summary

  • remove the last batch envelope planner helper from runtime action adapters
  • dispatch notification actions by planning one envelope at a time and lazily borrowing the runtime gateway only when an envelope exists
  • add a boundary test preventing plan_runtime_*_envelopes helpers from becoming public adapter surface again

Verification

  • RED: uv run pytest tests/Unit/backend/web/services/test_agent_runtime_gateway_protocol_boundary.py -q
  • uv run pytest tests/Unit/backend/web/services/test_workflow_event_notification.py::test_workflow_event_notification_fn_skips_gateway_when_no_runtime_recipients tests/Unit/backend/web/services/test_agent_runtime_gateway_protocol_boundary.py tests/Unit/backend/web/services/test_runtime_notification_action.py -q
  • uv run ruff check backend/threads/chat_adapters/runtime_notification_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_workflow_event_notification.py
  • uv run ruff format --check backend/threads/chat_adapters/runtime_notification_action.py tests/Unit/backend/web/services/test_agent_runtime_gateway_protocol_boundary.py
  • uv run pyright backend/threads/chat_adapters/runtime_notification_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_workflow_event_notification.py
  • uv run pytest tests/Unit -q

@shuxueshuxue shuxueshuxue merged commit 16464d8 into dev May 12, 2026
6 checks passed
@shuxueshuxue shuxueshuxue deleted the trim-notification-envelope-list-planner branch May 12, 2026 23:39
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