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

feat(discord): implement /newsession and /listsessions (issue #28) #85

Closed
Toutsu wants to merge 0 commits from feature/issue-28-discord-newsession-listsessions into main
pull from: feature/issue-28-discord-newsession-listsessions
merge into: Toutsu:main
Conversation 0 Commits - Files Changed -
Toutsu commented 2026-05-19 12:46:56 +03:00
Owner
Copy Link
Copy Source

Discord /newsession ? /listsessions ? Issue #28

??? ???????????

  • Slash-??????? /newsession ??? ???????? ??????? ?????? ????? ?? Discord.
  • Slash-??????? /listsessions ??? ????????? ??????????? ??? ? ???????.
  • DiscordPermissionChecker ? ???????? ???? (owner / admin / manager).
  • DiscordPlatformMessenger ? ?????????? IPlatformMessenger ??? Discord (NetCord REST).
  • ?????? ?????????? ? DI (Program.cs).

???????????

  • Vertical slice: ?????? ??????? ? ????????? ???? (Command + Handler).
  • Platform-agnostic SQL: ???????????? ??????? platform, external_group_id, external_user_id.
  • ????????? ?????????????? ???????????? DiscordSessionBatchRenderer.

TDD

  • 212 ??????, ??? ???????.
  • Source-level ????? ????????? ????????: Dapper, Npgsql, ??????????, CancellationToken, ????????????? ?????????????.

??????

  • Minor bump: 2.3.0 ? 2.4.0
  • ????????????????: Directory.Build.props, compose.yaml, deploy.yml, NavMenu.razor.

Closes #28

# Discord /newsession ? /listsessions ? Issue #28 ## ??? ??????????? - Slash-??????? /newsession ??? ???????? ??????? ?????? ????? ?? Discord. - Slash-??????? /listsessions ??? ????????? ??????????? ??? ? ???????. - DiscordPermissionChecker ? ???????? ???? (owner / admin / manager). - DiscordPlatformMessenger ? ?????????? IPlatformMessenger ??? Discord (NetCord REST). - ?????? ?????????? ? DI (Program.cs). ## ??????????? - Vertical slice: ?????? ??????? ? ????????? ???? (Command + Handler). - Platform-agnostic SQL: ???????????? ??????? platform, external_group_id, external_user_id. - ????????? ?????????????? ???????????? DiscordSessionBatchRenderer. ## TDD - 212 ??????, ??? ???????. - Source-level ????? ????????? ????????: Dapper, Npgsql, ??????????, CancellationToken, ????????????? ?????????????. ## ?????? - Minor bump: 2.3.0 ? 2.4.0 - ????????????????: Directory.Build.props, compose.yaml, deploy.yml, NavMenu.razor. Closes #28
Toutsu added 8 commits 2026-05-19 12:46:57 +03:00
feat(discord): add DiscordPermissionChecker for session management rights ac8f03ecc9 
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
feat(discord): add /listsessions slash command and handler 76456cc28a 
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
feat(discord): add /newsession slash command and handler 95aad3a2f6 
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
feat(discord): add DiscordPlatformMessenger IPlatformMessenger implementation d373ff49ba 
Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
feat: register Discord session handlers and permission checker in DI 8666b8984e 
Task 5: DI wiring for DiscordNewSessionHandler, DiscordListSessionsHandler,
DiscordPermissionChecker, and DiscordPlatformMessenger.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
chore: bump version to 2.4.0 474e7f62f7 
Synchronized across Directory.Build.props, compose.yaml, deploy.yml,
NavMenu.razor, and project structure tests.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
fix(discord): resolve permission checking for /newsession command daa59335cc 
- DiscordPermissionChecker: removed dead-code userRoles overload;
  now only uses resolvedPermissions bitflag (Administrator = 0x8).
- DiscordNewSessionCommand: computes resolved permissions from guild
  user roles via Context.Guild.Users[Id].RoleIds + guild.Roles.
- DiscordNewSessionHandler: updated signature to accept ulong
  resolvedPermissions instead of unused userRoles.
- Added ILogger to command for diagnostics on unexpected errors.
- Added test: regular user with ManageServer (but not Admin) is rejected.

Refs issue #28

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
feat(discord): improve UX and add source-level tests for /newsession
PR Checks / test-and-build (pull_request) Successful in 5m59s
Details
d55003a2a9 
- DiscordNewSessionCommand: on success, renders session details via
  DiscordSessionBatchRenderer.Render() with embeds and action rows.
- DiscordNewSessionCommand: uses Discord emoji shortcodes for error
  and success messages (✅, ⛔, 💥).
- DiscordNewSessionHandlerTests: added 7 source-level structural tests
  verifying Dapper usage, NpgsqlDataSource, permission checks,
  platform neutrality, transaction safety, CancellationToken usage,
  and embed rendering in the command.

Refs issue #28

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Toutsu closed this pull request 2026-05-19 14:21:38 +03:00
Some checks are pending
PR Checks / test-and-build (pull_request) Successful in 5m59s
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
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#85
Delete Branch "feature/issue-28-discord-newsession-listsessions"

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?