fix/playtest-feedback #19

Merged

ayoub merged 3 commits from fix/playtest-feedback into dev 2026-06-12 10:30:50 +00:00

Conversation 0 Commits 3 Files changed 23 +296 -60

ayoub commented 2026-06-12 10:30:43 +00:00

Owner

Copy link

No description provided.

ayoub self-assigned this 2026-06-12 10:30:43 +00:00

ayoub added 3 commits 2026-06-12 10:30:44 +00:00

fix(server): connect via PG* vars (robust to special chars in password) ... cfc5339a25

postgres-js failed to parse DATABASE_URL when the password contained URL-special
characters (@ : / # ...). New createSqlClient(): uses DATABASE_URL if set, else
falls back to PG* env vars (PGHOST/PGUSER/PGPASSWORD/...) which postgres-js reads
directly — password passed verbatim, no URL encoding. docker-compose.prod.yml now
sets PG* for the server instead of an inline DSN.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

fix(web): blindtest sound, hidden vote status, authoritative tracks, quiz input ... 6362b97508

From friends playtest feedback:
- blindtest "who added it?" (and mixed): hide the per-player vote status — the
  contributor never validates, which gave them away
- blindtest sound: set YT player `origin`/`enablejsapi` (fixes the postMessage
  origin warning) and add a per-game "Activer le son" gesture for non-DJ
  (browsers block audible autoplay without a user gesture)
- track list: render each player's own tracks from a server-sent, authoritative
  blindtest:myTracks event (emitted on submit/remove/reconnect) instead of local
  state that desynced on remount — fixes the "3/3 but only 2 shown" case
- quiz: reset the free-answer input between questions (was pre-filled with the
  previous answer)

Verified e2e: myTracks emitted on submit + reconnect.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

feat: DJ can be contributor, volume, theme switch, clock sync, 90s blindtest ... 7a41bbf8c1

Playtest feedback round 2:
- blindtest DJ can now be anyone incl. the track's contributor (pickDj only
  excludes bots) — no more "immunity" for whoever isn't picked
- blindtest rounds last 90s (longer listen + typing)
- per-player volume slider (persisted), YT player setVolume + origin/enablejsapi
- theme switcher (light/dark/system) on home + room header
- clock sync: clients estimate the server-clock offset (time:sync) so the
  countdown is identical across devices (was off by each device's clock skew);
  also used for blindtest media sync

Note on duplicates: anti-repeat is per room code (quiz_played) — it works when
you replay in the SAME room, but a new room starts fresh. Verified e2e: two
chained games in one room share 0 questions.

Verified: typecheck 4/4, lint 4/4, 24 server tests, web build; e2e dedup (0
overlap) and clock/DJ behavior.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

ayoub merged commit 4497d697f7 into dev 2026-06-12 10:30:50 +00:00

ayoub deleted branch fix/playtest-feedback 2026-06-12 10:30:50 +00:00

ayoub referenced this pull request from a commit 2026-06-12 10:30:52 +00:00

Merge pull request 'fix/playtest-feedback' (#19) from fix/playtest-feedback into dev

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

ayoub

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: ayoub/nerdware#19

No description provided.