Skip to content

fix(telegram): report cron topic fallback#28527

Merged
teknium1 merged 2 commits into
mainfrom
hermes/hermes-6063e704
May 19, 2026
Merged

fix(telegram): report cron topic fallback#28527
teknium1 merged 2 commits into
mainfrom
hermes/hermes-6063e704

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Salvage of #25368 (@el-analista). Retry Telegram message_thread_id thread-not-found errors once before falling back to base chat. Mark cron delivery as degraded when a configured Telegram topic is not found and delivery falls back to the base chat (so operators see the misconfiguration in cron output).

Conflict resolution: combined with #28505's future is None check from #28484 area.

Authorship preserved via cherry-pick. 168/168 cron + thread-fallback tests passing.

@teknium1 teknium1 merged commit 4f6fef1 into main May 19, 2026
@teknium1 teknium1 deleted the hermes/hermes-6063e704 branch May 19, 2026 05:45
@github-actions

Copy link
Copy Markdown
Contributor

🔎 Lint report: hermes/hermes-6063e704 vs origin/main

ruff

Total: 0 on HEAD, 0 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 0 pre-existing issues carried over.

ty (type checker)

Total: 8933 on HEAD, 8933 on base (➖ 0)

🆕 New issues: none

✅ Fixed issues: none

Unchanged: 4680 pre-existing issues carried over.

Diagnostics are surfaced as warnings — this check never fails the build.

@alt-glitch alt-glitch added type/bug Something isn't working platform/telegram Telegram bot adapter comp/gateway Gateway runner, session dispatch, delivery comp/cron Cron scheduler and job management P2 Medium — degraded but workaround exists labels May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cron Cron scheduler and job management comp/gateway Gateway runner, session dispatch, delivery P2 Medium — degraded but workaround exists platform/telegram Telegram bot adapter type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants