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

fix(discord): register slash command modules #95

Closed
Toutsu wants to merge 0 commits from codex/fix-issue-94-discord-slash-modules into main
pull from: codex/fix-issue-94-discord-slash-modules
merge into: Toutsu:main
Conversation 0 Commits - Files Changed -
Toutsu commented 2026-05-25 15:50:49 +03:00
Owner
Copy Link
Copy Source

Summary

  • Fixes #94 by registering NetCord application command modules with host.AddModules(typeof(Program).Assembly) after the Discord worker host is built.
  • Adds regression coverage for module registration and README Discord environment guidance.
  • Bumps release version to 3.0.2 across build, compose, deploy, nav, and version tests.

Root cause

/ping worked because it was registered directly through host.AddSlashCommand(...). The session commands were declared as NetCord modules, but the host never scanned the assembly with AddModules(...), so /newsession, /listsessions, and /reschedule were not published to Discord as slash commands.

Test plan

  • dotnet test tests/GmRelay.Bot.Tests/GmRelay.Bot.Tests.csproj --filter "FullyQualifiedName~DiscordStartupTests.Program_ShouldRegisterNetCordGatewayApplicationCommandsAndComponents"
  • dotnet test tests/GmRelay.Bot.Tests/GmRelay.Bot.Tests.csproj --filter "FullyQualifiedName~DiscordProjectStructureTests.Readme_ShouldNotAskForUnusedDiscordBotClientId"
  • dotnet build GM-Relay.slnx
  • dotnet test GM-Relay.slnx
  • Gitea Actions run #236 / run number 187 succeeded

Workflow

  • CI passes
  • Merged to main via fallback local merge commit 80e346d
  • Deployed by Gitea Actions run #237
  • Release v3.0.2 published
## Summary - Fixes #94 by registering NetCord application command modules with `host.AddModules(typeof(Program).Assembly)` after the Discord worker host is built. - Adds regression coverage for module registration and README Discord environment guidance. - Bumps release version to `3.0.2` across build, compose, deploy, nav, and version tests. ## Root cause `/ping` worked because it was registered directly through `host.AddSlashCommand(...)`. The session commands were declared as NetCord modules, but the host never scanned the assembly with `AddModules(...)`, so `/newsession`, `/listsessions`, and `/reschedule` were not published to Discord as slash commands. ## Test plan - [x] `dotnet test tests/GmRelay.Bot.Tests/GmRelay.Bot.Tests.csproj --filter "FullyQualifiedName~DiscordStartupTests.Program_ShouldRegisterNetCordGatewayApplicationCommandsAndComponents"` - [x] `dotnet test tests/GmRelay.Bot.Tests/GmRelay.Bot.Tests.csproj --filter "FullyQualifiedName~DiscordProjectStructureTests.Readme_ShouldNotAskForUnusedDiscordBotClientId"` - [x] `dotnet build GM-Relay.slnx` - [x] `dotnet test GM-Relay.slnx` - [x] Gitea Actions run #236 / run number 187 succeeded ## Workflow - [x] CI passes - [x] Merged to `main` via fallback local merge commit `80e346d` - [x] Deployed by Gitea Actions run #237 - [x] Release [v3.0.2](https://git.codeanddice.ru/Toutsu/GmRelayBot/releases/tag/v3.0.2) published
Toutsu added 1 commit 2026-05-25 15:50:49 +03:00
fix(discord): register slash command modules
PR Checks / test-and-build (pull_request) Successful in 8m27s
Details
eff0128d29 
Register NetCord application command modules after the host is built so module-based commands are published alongside the minimal /ping command.

Update README Discord env guidance to avoid the unused DISCORD_BOT_CLIENT_ID variable.

Bump version to 3.0.2.
Toutsu closed this pull request 2026-05-25 16:16:45 +03:00
Some checks are pending
PR Checks / test-and-build (pull_request) Successful in 8m27s
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
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#95
Delete Branch "codex/fix-issue-94-discord-slash-modules"

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?