feat(chat): лимит внутриигрового чата — 1 сообщение за ход #60

Merged

developer merged 2 commits from feature/chat-one-message-per-turn into development 2026-06-14 18:33:01 +00:00

Conversation 1 Commits 2 Files Changed 18

+217 -16

developer commented 2026-06-14 18:19:31 +00:00

Owner

Copy Link

Copy Source

Что сделано

Внутриигровой чат ограничен одним сообщением за ход на обоих концах.

• Бэкенд: PostMessage отклоняет второе сообщение за тот же ход (ErrChatAlreadySentThisTurn → 409 chat_already_sent). Граница хода — move-driven turn_started_at (новый game.TurnStartedAt, добавлен в social.GameReader); переиспользован существующий lastMessageAt.
• UI: «уже писал в этом ходу» вычисляется из списка сообщений против GameView.lastActivityUnix (lib/chatlimit.ts) — без счётчика, переживает переоткрытие чата, сбрасывается при смене хода. После лимита поле сменяется короткой подписью (не кнопкой «подтолкнуть»). Экран чата теперь дочитывает состояние на событиях смены хода (your_turn/opponent_moved/game_over/opponent_joined). Enter защищён !busy от двойной отправки в окне ожидания ответа.

Решения (согласовано в интервью)

• Лимит и в UI, и жёстко на бэкенде (защита от модифицированного клиента).
• После лимита — небольшая подпись, область ввода не «схлопывается».

Тесты

• Бэкенд: новый TestChatOnePerTurn + строка в dto_test; полный интеграционный слой зелёный; build/vet/gofmt чисто.
• UI: новый chatlimit.test.ts; расширен чат e2e; pnpm check / test:unit / build / test:e2e (Chromium + WebKit) — всё зелёное.
• Без правок схемы БД и wire/fbs — контур не требует вайпа.

Доки обновлены: FUNCTIONAL (+ru), ARCHITECTURE, UI_DESIGN.

## Что сделано Внутриигровой чат ограничен **одним сообщением за ход** на обоих концах. - **Бэкенд:** `PostMessage` отклоняет второе сообщение за тот же ход (`ErrChatAlreadySentThisTurn` → 409 `chat_already_sent`). Граница хода — move-driven `turn_started_at` (новый `game.TurnStartedAt`, добавлен в `social.GameReader`); переиспользован существующий `lastMessageAt`. - **UI:** «уже писал в этом ходу» вычисляется из списка сообщений против `GameView.lastActivityUnix` (`lib/chatlimit.ts`) — без счётчика, переживает переоткрытие чата, сбрасывается при смене хода. После лимита поле сменяется короткой подписью (не кнопкой «подтолкнуть»). Экран чата теперь дочитывает состояние на событиях смены хода (`your_turn`/`opponent_moved`/`game_over`/`opponent_joined`). Enter защищён `!busy` от двойной отправки в окне ожидания ответа. ## Решения (согласовано в интервью) - Лимит и в UI, и жёстко на бэкенде (защита от модифицированного клиента). - После лимита — небольшая подпись, область ввода не «схлопывается». ## Тесты - Бэкенд: новый `TestChatOnePerTurn` + строка в `dto_test`; полный интеграционный слой зелёный; build/vet/gofmt чисто. - UI: новый `chatlimit.test.ts`; расширен чат e2e; `pnpm check` / `test:unit` / `build` / `test:e2e` (Chromium + WebKit) — всё зелёное. - Без правок схемы БД и wire/fbs — контур не требует вайпа. Доки обновлены: FUNCTIONAL (+ru), ARCHITECTURE, UI_DESIGN.

developer added 1 commit 2026-06-14 18:19:31 +00:00

feat(chat): limit in-game chat to one message per turn ... 02681ae9e0

Enforce one chat message per turn on both ends. The backend rejects a
second message in the same turn (ErrChatAlreadySentThisTurn -> 409
chat_already_sent), keyed on the move-driven turn start (turn_started_at).
The UI derives "already wrote this turn" from the message list against
GameView.lastActivityUnix (no counter, survives reopening, resets on turn
change), hides the field behind a short caption once the limit is reached,
and now reloads the game state on turn/game-state events so the toggle and
the limit follow the live game. Enter is gated on !busy to avoid a
double-send in the in-flight window.

Backend: new game.TurnStartedAt; social GameReader gains it; PostMessage
enforces the limit reusing lastMessageAt. UI: new lib/chatlimit.ts pure
logic + unit tests; Chat/ChatScreen wiring; chat.sentThisTurn and
error.chat_already_sent i18n (en/ru); extended chat e2e. Docs: FUNCTIONAL
(+ru), ARCHITECTURE, UI_DESIGN.

developer added 1 commit 2026-06-14 18:29:48 +00:00

fix(chat): no chat field or nudge on a finished game ... 5f53ec81b9

A finished game is read-only: gate the nudge behind canNudge (active game,
opponent's turn) so it no longer shows on a finished (or otherwise
non-active) game — where the backend rejects it anyway. The message field
is already hidden off your own turn. Extend the finished-game e2e to assert
neither Send nor Nudge is offered.

owner approved these changes 2026-06-14 18:32:29 +00:00

developer merged commit 9d85090075 into development 2026-06-14 18:33:01 +00:00

developer deleted branch feature/chat-one-message-per-turn 2026-06-14 18:33:01 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

owner

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

owner (Ilia) developer

No Assignees

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: developer/scrabble-game#60