Comments

app/features/chat/ChatRoomResolver.server.ts

@@ -16,6 +16,8 @@ import {
 } from "~/utils/urls";
 import type { ChatRoomType } from "./chat-types";
 
+// xxx: direct DB calls here, should not be a thing, the logic also seems quite heavy, needed?
+
 export interface ResolvedRoom {
 	roomId: number;
 	type: ChatRoomType;

app/features/tournament-bracket/BracketRepository.server.ts

@@ -207,6 +207,7 @@ export function insertBracket(args: {
 
 		const statuses = matchStatuses(args.bracket);
 
+		// xxx: check if this is true
 		// only matches that can already be played get a chat room; the rest get
 		// theirs as they start (see syncStartedAt)
 		const chatRoomIdByMatchId = new Map<number, number>();