fix(bot,discord): в /newsession нельзя задать сессию без лимита мест #123
Reference in New Issue
Block a user
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?
Проблема
В Telegram-визарде создания сессий (
/newsession) на шаге «Лимит мест» пользователь не мог задать сессию без лимита. Бот отвечал «Не заполнены поля: лимит мест» (см. скриншот), даже когда пользователь пытался нажать «Без waitlist», полагая, что это «без лимита».Скриншот
Скриншот из Telegram: при выборе «Без waitlist» на шаге Capacity бот отвечает «❌ Не заполнены поля: лимит мест» (10:30 на скриншоте пользователя).
Корневые причины
В
WizardStepViewBuilder.BuildCapacity(Telegram) иDiscordWizardStep.RenderCapacity/RenderPoolSlotCapacity(Discord) не было кнопки «Без лимита» — были только кнопки waitlist. Кнопкаwaitlist:offмолча перепрыгивала на следующий шаг (ApplyCapacityChoice), не устанавливаяMaxPlayers— то есть оставлялаnull.В
BuildCommand(Telegram:CreateSessionHandler.cs:173, Discord:DiscordWizardSubmitter.cs:137) стоял?? 0— если быnullMaxPlayers дошёл до команды (через гонку или старый draft), он превратился бы в0, что нарушает DB CHECKck_sessions_max_players(V006__add_session_capacity_waitlist.sql:max_players IS NULL OR max_players > 0).DB и контракт
CreateSessionCommand.MaxPlayers: int?уже корректно поддерживалиNULL(тестCreateSessionCommandContractTests.cs:16, рендерSessionBatchViewBuilderTests.cs:140, web-формаEditSession.razor:56сplaceholder="Без лимита"). Сломан был только wizard-флоу.Решение
Добавлена кнопка
♾ Без лимита(callback value: no_limit) в:WizardStepViewBuilder.BuildCapacity(Telegram) —src/GmRelay.Shared/Features/Sessions/CreateSession/Wizard/WizardStepViewBuilder.csDiscordWizardStep.RenderCapacity(Discord) —src/GmRelay.DiscordBot/Features/Sessions/Wizard/DiscordWizardStep.csDiscordWizardStep.RenderPoolSlotCapacity(Discord) —src/GmRelay.DiscordBot/Features/Sessions/Wizard/DiscordWizardStep.csОбработка
no_limit:GameCreationWizard.ApplyCapacityChoice(shared) —src/GmRelay.Shared/Features/Sessions/CreateSession/Wizard/GameCreationWizard.cs:301-307— ветка"no_limit" => (Visibility, SetMaxPlayers(p, null)).SetMaxPlayersпереведён сint vнаint? vдля поддержки null.BuildCommandв обоих submitter'ах принимаетint? maxPlayers,?? 0убран.UX-улучшение: в Discord-embed подтверждения (
BuildConfirmDescription) приMaxPlayers = nullтеперь явно пишется👥 Без лимита, waitlist вкл/выклвместо игнорирования поля.Тесты
Добавлено 9 регрессионных тестов:
Telegram (5):
WizardStepRenderTests.CapacityStep_HasWaitlistButtons— добавлена проверка наличия кнопки «Без лимита»GameCreationWizardStepTransitionsTests.NoLimitCapacityButton_AdvancesToVisibility_AndLeavesMaxPlayersNull— чтоno_limitведёт наVisibilityи оставляетMaxPlayers = nullв JSON draft'аGameCreationWizardStepTransitionsTests.ChoiceCallback_AdvancesToExpectedStep[Capacity/no_limit]— Theory-кейсCreateSessionHandlerBuildCommandTests.BuildCommand_WhenSingleMaxPlayersIsNull_PropagatesNull— null доходит до командыCreateSessionHandlerBuildCommandTests.BuildCommand_WhenSingleMaxPlayersIsSet_PropagatesValue— число доходитDiscord (3 + 1):
DiscordWizardStepCapacityRenderTests.RenderCapacity_ContainsNoLimitButtonDiscordWizardStepCapacityRenderTests.RenderPoolSlotCapacity_ContainsNoLimitButtonDiscordWizardStepCapacityRenderTests.RenderCapacity_NoLimitButton_HasChoiceCustomIdForNoLimitDiscordWizardSubmitterBuildCommandTests.BuildCommand_WhenSingleMaxPlayersIsNull_PropagatesNullDiscordWizardSubmitterBuildCommandTests.BuildCommand_WhenSingleMaxPlayersIsSet_PropagatesValueТакже добавлен
InternalsVisibleTo("GmRelay.Bot.Tests")вsrc/GmRelay.DiscordBot/Properties/AssemblyInfo.cs(для unit-тестов internalBuildCommand).Верификация
dotnet test— 601 пройдено, 2 пропущены (Testcontainers, как до фикса)dotnet format --verify-no-changes— чистdotnet build— без warning'овdotnet list package --vulnerable— чистpackages.lock.jsonне изменялись (новых пакетов нет)Аудит follow-up
Проверил, не страдает ли тем же багом Web-форма: нет.
EditSession.razor:56уже использует<InputNumber @bind-Value="model.MaxPlayers" min="1" placeholder="Без лимита" />сint?, аAuthorizedSessionService.cs:506отвергает только<= 0, null допустим. Контрактint? MaxPlayersуже полностью поддержан.Также проверил пул-флоу:
WizardSlotInput.MaxPlayers: int(не nullable) — слоты пула by design всегда с лимитом 1..50,MaxPlayersForPoolкорректно работает сint. Архитектурное изменение здесь не требуется.Реализовано в PR #124 и выпущено в v3.9.3.
Спасибо за подробный скриншот — он сразу указал на нужный шаг визарда.