From 309381e43be0959bf220fe58b9e81e5733768018 Mon Sep 17 00:00:00 2001 From: Claude Date: Wed, 15 Apr 2026 18:22:41 +0000 Subject: [PATCH] Fix no-non-null-assertion lint errors in isScoring Map lookups Replace Map.has(key) ? Map.get(key)! : fallback pattern with Map.get(key) ?? fallback to satisfy oxlint no-non-null-assertion rule. https://claude.ai/code/session_01D3NbnVQdaH82Fk7Jm8y3sB --- ...min.sports-seasons.$id.events.$eventId.bracket.server.ts | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/app/routes/admin.sports-seasons.$id.events.$eventId.bracket.server.ts b/app/routes/admin.sports-seasons.$id.events.$eventId.bracket.server.ts index a3d82d0..72242e0 100644 --- a/app/routes/admin.sports-seasons.$id.events.$eventId.bracket.server.ts +++ b/app/routes/admin.sports-seasons.$id.events.$eventId.bracket.server.ts @@ -418,7 +418,7 @@ export async function action({ request, params }: Route.ActionArgs) { round: match.round, winnerId, loserId, - isScoring: roundIsScoring.has(match.round) ? roundIsScoring.get(match.round)! : (match.isScoring ?? true), + isScoring: roundIsScoring.get(match.round) ?? (match.isScoring ?? true), sportsSeasonId: event.sportsSeasonId, bracketTemplateId: event.bracketTemplateId, eventName: event.name ?? undefined, @@ -619,9 +619,7 @@ export async function action({ request, params }: Route.ActionArgs) { // defaults to true, so legacy play-in rows that predate the column are // incorrectly marked as scoring and would assign finalPosition=0 to losers // who still have a second game (e.g. NBA 7v8 play-in loser → Round 2). - const isScoring = templateRoundIsScoring.has(match.round) - ? templateRoundIsScoring.get(match.round)! - : (match.isScoring ?? true); + const isScoring = templateRoundIsScoring.get(match.round) ?? (match.isScoring ?? true); await processMatchResult( { round: match.round,