Skip to content

feat: the game system picker is a grouped, searchable dropdown that a… - #110

Merged
over2take merged 2 commits into
feature/system-builderfrom
sb/2e-system-picker
Sep 30, 2026
Merged

over2take merged 2 commits into
feature/system-builderfrom
sb/2e-system-picker

Conversation

@over2take

Copy link
Copy Markdown
Owner

…sks before switching

The GAME tab's row of system buttons held four systems; custom ones make that any number. Now one dropdown: BUILT-IN in a fixed order, then YOUR SYSTEMS A to Z with their published version, a search box and arrow keys, portalled into the theme. Picking a different system opens a confirmation, since it changes the game for everyone online; a refused switch says nothing was changed. The runtime's system list now carries each custom system's version.

Summary

Test plan

  • Tested locally
  • Tests pass

Pre-merge checklist

Code quality:

  • Code follows project style
  • No breaking changes (or clearly documented)
  • No console errors or warnings

Version & Release:

  • Version bumped? If releasing to users, update:
    • frontend/package.json version
    • docker-compose.yml APP_VERSION
    • CHANGELOG.md with release notes
  • GitHub Actions will auto-tag Docker images with the new version

Before merging to main:

  • All tests passing
  • PR reviewed and approved
  • Branch is up to date with main
  • No merge conflicts

Related issues

Developer added 2 commits September 30, 2026 08:45
…sks before switching

The GAME tab's row of system buttons held four systems; custom ones make that any number.
Now one dropdown: BUILT-IN in a fixed order, then YOUR SYSTEMS A to Z with their published
version, a search box and arrow keys, portalled into the theme. Picking a different system
opens a confirmation, since it changes the game for everyone online; a refused switch says
nothing was changed. The runtime's system list now carries each custom system's version.
…he first connection

db.js seeds the admin login behind a bcrypt hash that finishes about 40ms after the token
move. The test closed the first connection on the token move alone, so the seed's INSERT
landed on a closed database; locally the process exited first, on the slower CI runner the
INSERT won and killed it with SQLITE_MISUSE. Reproduced by letting the second half run
500ms longer: fails without the wait, passes with it.
@over2take
over2take merged commit ba12347 into feature/system-builder Sep 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant