fix: address review suggestions on legacy replay parsing

- Rename regression test to cover all defaulted legacy fields
- Constrain matchJoinTimeoutInMs to non-negative durations
- Drop void prefix on test() to match repo style

Author: alfaoz

packages/backend/src/persistence/gameHistoryRepository.test.ts

@@ -3,7 +3,7 @@ import test from 'node:test';
 
 import { zDatabaseGame, zFinishedGameRecord } from '@ih3t/shared';
 
-void test(`parses legacy tournament replay records without live timeout fields`, () => {
+test(`parses legacy tournament replay records with defaulted timeout, pending extension, and profile fields`, () => {
     const legacyTournamentGame = {
         id: `game-1`,
         version: 3,

packages/shared/src/tournaments.ts

@@ -570,6 +570,7 @@ export const zSessionTournamentInfo = z.object({
         .nonnegative(),
     pendingExtension: z.boolean().default(false),
     matchJoinTimeoutInMs: z.number().int()
+        .nonnegative()
         .nullable()
         .default(null),
     leftDisplayName: z.string().nullable(),