Skip to content

fix: simplify launch game logic#557

Draft
garrappachc wants to merge 25 commits intomasterfrom
refactor/simplify-launch-game
Draft

fix: simplify launch game logic#557
garrappachc wants to merge 25 commits intomasterfrom
refactor/simplify-launch-game

Conversation

@garrappachc
Copy link
Member

@garrappachc garrappachc commented Mar 19, 2026

Simplify launch game logic and add a few fixes:

  • remove game-servers module
  • the launchGame() function is simplified to avoid excessive event-based scattered logic
  • selected game server is passed via JSON instead of relying on string stiching

@garrappachc garrappachc self-assigned this Mar 19, 2026
@garrappachc garrappachc force-pushed the refactor/simplify-launch-game branch 3 times, most recently from 5daa8aa to 9d79611 Compare March 19, 2026 13:54
@github-actions
Copy link
Contributor

github-actions bot commented Mar 19, 2026

Playwright test results

passed  197 passed

Details

stats  197 tests across 29 suites
duration  7 minutes, 59 seconds
commit  f8e1c24

@garrappachc garrappachc force-pushed the refactor/simplify-launch-game branch 2 times, most recently from c2582ea to eaac1a1 Compare March 19, 2026 16:46
garrappachc and others added 22 commits March 20, 2026 10:20
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…e export

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…test

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ooking

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…onfigure

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ions

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Without an overload, calling findOne() with no pluck argument caused
Keys to default to never, making PickDeep<GameModel, never> return an
empty type and all downstream property accesses unsafe (triggering
@typescript-eslint/no-unsafe-assignment in the API v1 slots route).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@garrappachc garrappachc force-pushed the refactor/simplify-launch-game branch from 9a784a3 to 5164329 Compare March 20, 2026 09:20
@garrappachc garrappachc marked this pull request as draft March 20, 2026 12:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant