Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 8 additions & 1 deletion cron/scheduler.py
Original file line number Diff line number Diff line change
Expand Up @@ -434,8 +434,15 @@ def _resolve_single_delivery_target(job: dict, deliver_value: str) -> Optional[d

if deliver_value == "origin":
if origin:
platform = origin["platform"]
# WebUI surfaces cron runs by reading the local session DB —
# no external gateway delivery is needed. Returning None
# treats it like "local", avoiding "unknown platform 'webui'"
# errors (#45360).
if platform.lower() == "webui":
return None
return {
"platform": origin["platform"],
"platform": platform,
"chat_id": str(origin["chat_id"]),
"thread_id": origin.get("thread_id"),
}
Expand Down
22 changes: 22 additions & 0 deletions tests/cron/test_scheduler.py
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,28 @@ def test_origin_delivery_preserves_thread_id(self):
"thread_id": "17585",
}

def test_webui_origin_platform_returns_none(self):
"""WebUI reads from local session DB — no gateway delivery needed (#45360)."""
job = {
"deliver": "origin",
"origin": {
"platform": "webui",
"chat_id": "session-abc",
},
}
assert _resolve_delivery_target(job) is None

def test_webui_origin_platform_case_insensitive(self):
"""Platform name comparison should be case-insensitive."""
job = {
"deliver": "origin",
"origin": {
"platform": "WebUI",
"chat_id": "session-abc",
},
}
assert _resolve_delivery_target(job) is None

@pytest.mark.parametrize(
("platform", "env_var", "chat_id"),
[
Expand Down
Loading