Toutsu/GmRelayBot
1
1
Fork 0
Code Issues 18 Pull Requests 1 Actions Packages Projects Releases 85 Wiki Activity

fix(bot): wizard creates DB session but does not publish Telegram topic or signup message #135

New Issue
Closed
opened 2026-06-09 16:07:30 +03:00 by Toutsu · 1 comment
Toutsu commented 2026-06-09 16:07:30 +03:00
Owner
Copy Link
Copy Source

Симптом

После релиза v3.9.8 wizard /newsession сообщает ✅ Создано: 1 сессия, и сессия видна через list sessions, но в Telegram не появляется forum topic и schedule/signup message.

Evidence

Production DB после пользовательского повтора показывала новую сессию:

id: f5be283c-919f-45fc-a205-5e38e9be04f0
title: Тест
created_at: 2026-06-09 12:59:43 UTC
thread_id: NULL
batch_message_id: NULL
topic_created_by_bot: false
platform: Telegram
external_group_id: -1003916537960

Production bot/db logs после deploy не показывали 42883, то есть DB create уже был успешен.

Подтверждённый root cause

src/GmRelay.Bot/Features/Sessions/CreateSession/CreateSessionHandler.cs::SubmitDraftAsync после _shared.HandleAsync(cmd, ct) только редактировал wizard draft message на ✅ Создано: ... и удалял draft.

Он не вызывал:

  • IPlatformMessenger.CreateThreadAsync(...)
  • IPlatformMessenger.SendScheduleAsync(...)
  • UPDATE sessions SET thread_id, batch_message_id, topic_created_by_bot

Поэтому DB rows создавались, но Telegram publication side-effects отсутствовали.

Решение (3.9.9)

  1. После успешного shared create создать topic при необходимости.
  2. Отправить schedule/signup message.
  3. Сохранить thread_id, batch_message_id, topic_created_by_bot для всего batch.
  4. Не повторять DB create при publication failure, чтобы не создавать дубликаты.
  5. Добавить Testcontainers regression test на wizard submit happy path.
## Симптом После релиза `v3.9.8` wizard `/newsession` сообщает `✅ Создано: 1 сессия`, и сессия видна через list sessions, но в Telegram не появляется forum topic и schedule/signup message. ## Evidence Production DB после пользовательского повтора показывала новую сессию: ```text id: f5be283c-919f-45fc-a205-5e38e9be04f0 title: Тест created_at: 2026-06-09 12:59:43 UTC thread_id: NULL batch_message_id: NULL topic_created_by_bot: false platform: Telegram external_group_id: -1003916537960 ``` Production bot/db logs после deploy не показывали `42883`, то есть DB create уже был успешен. ## Подтверждённый root cause `src/GmRelay.Bot/Features/Sessions/CreateSession/CreateSessionHandler.cs::SubmitDraftAsync` после `_shared.HandleAsync(cmd, ct)` только редактировал wizard draft message на `✅ Создано: ...` и удалял draft. Он не вызывал: - `IPlatformMessenger.CreateThreadAsync(...)` - `IPlatformMessenger.SendScheduleAsync(...)` - `UPDATE sessions SET thread_id, batch_message_id, topic_created_by_bot` Поэтому DB rows создавались, но Telegram publication side-effects отсутствовали. ## Решение (3.9.9) 1. После успешного shared create создать topic при необходимости. 2. Отправить schedule/signup message. 3. Сохранить `thread_id`, `batch_message_id`, `topic_created_by_bot` для всего batch. 4. Не повторять DB create при publication failure, чтобы не создавать дубликаты. 5. Добавить Testcontainers regression test на wizard submit happy path.
Toutsu commented 2026-06-09 16:55:25 +03:00
Author
Owner
Copy Link
Copy Source

Реализовано в PR #137 и выпущено в v3.9.9.

Root cause: SubmitDraftAsync создавал DB rows через shared handler, но не выполнял Telegram publication side-effects (CreateThreadAsync, SendScheduleAsync) и не сохранял thread_id/batch_message_id.

Реализовано в PR #137 и выпущено в v3.9.9. - PR: https://git.codeanddice.ru/Toutsu/GmRelayBot/pulls/137 - Release: https://git.codeanddice.ru/Toutsu/GmRelayBot/releases/tag/v3.9.9 - Merge commit: `bbd58142dbc3179137ca6b7864270b97f3054d98` - PR checks: run #315 — success - Deploy: run #316 — success Root cause: `SubmitDraftAsync` создавал DB rows через shared handler, но не выполнял Telegram publication side-effects (`CreateThreadAsync`, `SendScheduleAsync`) и не сохранял `thread_id`/`batch_message_id`.
Toutsu added reference 2026-06-09 16:55:38 +03:00
Toutsu added reference 2026-06-09 16:56:06 +03:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area:bot
Telegram bot and worker logic
 area:data
Database schema, migrations and persistence model
 area:discord
Discord bot service and Discord adapter
 area:infra
Infrastructure, runtime and deployment
 area:miniapp
Telegram Mini App dashboard
 area:platform
Platform-neutral abstractions and adapter boundaries
 area:shared
Shared domain and rendering contracts
 area:web
Blazor web dashboard
 in-progress
В работе (issue взят агентом)
 next-up
Recommended immediate backlog
 pending-approval
Ждёт согласования плана решения
 platform:discord
Discord-specific behavior or adapter work
 platform:multi
Cross-platform or platform-neutral work
 platform:telegram
Telegram-specific behavior or adapter work
 priority:p0
Critical priority
 priority:p0
Блокирующий приоритет: задача должна быть выполнена до признания решения production-ready
 priority:p1
High priority
 priority:p2
Medium priority
 priority:p3
Lower priority
 security
Security-sensitive work
 type:bug
Bug fix or corrective work
 type:chore
Maintenance and supporting work
 type:feature
New product functionality
 type:refactor
Structural change without direct feature growth
 type:test
Testing and coverage work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Toutsu
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Toutsu/GmRelayBot#135
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?