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

refactor: перевести оставшиеся Telegram handlers на platform-neutral contracts #37

New Issue
Closed
opened 2026-04-28 09:36:34 +03:00 by Toutsu · 1 comment
Toutsu commented 2026-04-28 09:36:34 +03:00
Owner
Copy Link
Copy Source

Контекст

После MVP не нужно переписывать все handler-ы сразу, но в v3 стоит довести platform-neutral архитектуру до конца.

Scope

  • Найти handler-ы, которые все еще напрямую зависят от Telegram API types.
  • Перевести cancel/reschedule/notifications/calendar-related flows на neutral commands/contracts.
  • Оставить Telegram-specific код только в router/adapter/rendering слоях.

Acceptance criteria

  • Core handlers тестируются без Telegram SDK.
  • Telegram и Discord routers используют одинаковые command contracts.
  • Новая платформа может добавляться adapter-ом, а не переписыванием core flow.
## Контекст После MVP не нужно переписывать все handler-ы сразу, но в v3 стоит довести platform-neutral архитектуру до конца. ## Scope - Найти handler-ы, которые все еще напрямую зависят от Telegram API types. - Перевести cancel/reschedule/notifications/calendar-related flows на neutral commands/contracts. - Оставить Telegram-specific код только в router/adapter/rendering слоях. ## Acceptance criteria - Core handlers тестируются без Telegram SDK. - Telegram и Discord routers используют одинаковые command contracts. - Новая платформа может добавляться adapter-ом, а не переписыванием core flow.
Toutsu added this to the Версия 3.0 — Platform & Club Expansion milestone 2026-04-28 09:36:34 +03:00
Toutsu commented 2026-05-27 16:11:58 +03:00
Author
Owner
Copy Link
Copy Source

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

Также добавлен hotfix коммит 7a2965b — в PR #106 были забыты DI-регистрации для shared DeleteSessionHandler и ListSessionsHandler, что вызвало crash бота при старте на Native AOT-сборке. Исправлено и задеплоено через run #271.

Реализовано в PR #106 и выпущено в [v3.2.0](https://git.codeanddice.ru/Toutsu/GmRelayBot/releases/tag/v3.2.0). Также добавлен hotfix коммит `7a2965b` — в PR #106 были забыты DI-регистрации для shared `DeleteSessionHandler` и `ListSessionsHandler`, что вызвало crash бота при старте на Native AOT-сборке. Исправлено и задеплоено через run #271.
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 area:bot area:platform area:shared platform:multi priority:p2 type:refactor
Milestone
No items
No Milestone Версия 3.0 — Platform & Club Expansion
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#37
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?