Skip to content

Back game logic

AGUECH OMAR requested to merge back-game-logic into develop

Description

GameTable.ts et game.gateway.ts : version develop à l’identique (paris cachés, ready, orchestrateur, avatars, CI, etc.) — l’autre branche proposait un GameTable trop différent et une gateway corrompue par des marqueurs de conflit. CashGameController.ts : base develop, avec les ajouts suivants : type CashBalanceSnapshot onHandComplete() renvoie les stacks finales avant teardown, puis applique la même logique qu’avant seat.avatarUrl = null aussi pour les sièges libérés via pendingQuitUserIds game.api.routes.ts : develop pour applyPokerAction + authMiddleware sur POST /:gameId/action, plus : POST /sync-balance (jetons DB pour l’utilisateur connecté) POST /record-result avec won + delta?, stats (gains/pertes en jetons) et User.chips: { increment: chipsDelta } si besoin completeCashHandAndBroadcast dans la gateway : après buildHiddenBetResolutionPayload, onHandComplete() puis prisma.$transaction pour aligner User.chips sur les stacks des joueurs de la main (comme sur la branche collègue, mais après la résolution des hidden bets qui dépend encore de l’état pré-teardown côté snapshot — l’ordre develop est conservé : snapshot hidden bets → onHandComplete → persistance DB → resolveHiddenBetsForHand → suite). Vérifications npm run build et npm test (380 tests) : OK. Merge commit sur back-game-logic : 4897e2a1. Tu peux pousser back-game-logic ou ouvrir la PR vers develop selon votre flux.

Lien vers la tâche

Type de changement

  • Feature — nouvelle fonctionnalité
  • Fix — correction de bug
  • Refactor — modification du code sans changement fonctionnel
  • Docs — documentation uniquement
  • Chore — maintenance (deps, config, etc.)
  • Hotfix — correction urgente en production

Modifications principales

Comment tester

  1. ...
  2. ...
  3. ...

Checklist

  • Code relu et conforme aux conventions du projet
  • Tests ajoutés ou mis à jour si nécessaire
  • Documentation mise à jour (si applicable)
  • Pipeline CI vert
  • Pas de régression détectée

Impact potentiel

  • Breaking change — migration nécessaire
  • Performance — impacts à surveiller
  • Sécurité — audit recommandé

Reviewers

  • Backend (@azra, @soheil)
  • Frontend (@mohamed, @yigit)
  • Sécurité (@linda, @massi)
  • BDD (@elhadj)
  • Revue générale (@aguech)
  • Validation chef de projet (@omar)
Edited by AGUECH OMAR

Merge request reports