Fix team having many tournament level check-ins

Concrete bug it caused: wrong team count for tournament card on the front page

app/features/tournament/TournamentRepository.server.ts

@@ -490,7 +490,9 @@ export function forShowcase() {
 						eb("CalendarEventDate.startTime", ">", databaseTimestampNow()),
 					]),
 				)
-				.select(({ fn }) => [fn.countAll<number>().as("teamsCount")])
+				.select(({ fn }) => [
+					fn.count<number>("TournamentTeam.id").distinct().as("teamsCount"),
+				])
 				.as("teamsCount"),
 			tournamentLogoWithDefault(eb).as("logoUrl"),
 			jsonObjectFrom(

app/features/tournament/TournamentTeamRepository.server.ts

@@ -423,11 +423,17 @@ export function checkIn(
 	return db.transaction().execute(async (trx) => {
 		let query = trx
 			.deleteFrom("TournamentTeamCheckIn")
-			.where("TournamentTeamCheckIn.tournamentTeamId", "=", tournamentTeamId)
-			.where("TournamentTeamCheckIn.isCheckOut", "=", 1);
+			.where("TournamentTeamCheckIn.tournamentTeamId", "=", tournamentTeamId);
 
 		if (typeof bracketIdx === "number") {
 			query = query.where("TournamentTeamCheckIn.bracketIdx", "=", bracketIdx);
+		} else {
+			query = query.where((eb) =>
+				eb.or([
+					eb("TournamentTeamCheckIn.isCheckOut", "=", 1),
+					eb("TournamentTeamCheckIn.bracketIdx", "is", null),
+				]),
+			);
 		}
 
 		await query.execute();

migrations/135-tournament-team-check-in-unique.js

@@ -0,0 +1,19 @@
+export function up(db) {
+	db.transaction(() => {
+		db.prepare(
+			/* sql */ `
+			delete from "TournamentTeamCheckIn"
+			where "rowid" not in (
+				select max("rowid")
+				from "TournamentTeamCheckIn"
+				group by "tournamentTeamId", coalesce("bracketIdx", -1)
+			)`,
+		).run();
+
+		db.prepare(
+			/* sql */ `
+			create unique index "tournament_team_check_in_team_bracket_unique"
+			on "TournamentTeamCheckIn"("tournamentTeamId", coalesce("bracketIdx", -1))`,
+		).run();
+	})();
+}