From d75c0493cf4d0c4d4bd02aff6062461abd47a94b Mon Sep 17 00:00:00 2001 From: Claude Date: Wed, 1 Apr 2026 20:20:45 +0000 Subject: [PATCH] Fix lint errors: replace .sort() with .toSorted() and remove non-null assertion - Replace Array#sort() with Array#toSorted() in two places (unicorn rule) - Remove non-null assertion on Map#get(); use early continue instead https://claude.ai/code/session_01QkFxCF4xmKjdVCNPysWDML --- app/models/__tests__/scoring-event-dashboard.test.ts | 2 +- app/models/scoring-event.ts | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/app/models/__tests__/scoring-event-dashboard.test.ts b/app/models/__tests__/scoring-event-dashboard.test.ts index 3de7659..b2b631d 100644 --- a/app/models/__tests__/scoring-event-dashboard.test.ts +++ b/app/models/__tests__/scoring-event-dashboard.test.ts @@ -298,7 +298,7 @@ describe("getEventsForDates", () => { const result = await getEventsForDates(["2026-03-20", "2026-03-21"]); expect(result).toHaveLength(2); - const displayDates = result.map((e) => e.displayDate).sort(); + const displayDates = result.map((e) => e.displayDate).toSorted(); expect(displayDates).toEqual(["2026-03-20", "2026-03-21"]); // Both entries share the same event id expect(result[0].id).toBe("e1"); diff --git a/app/models/scoring-event.ts b/app/models/scoring-event.ts index 211fea7..1c900b3 100644 --- a/app/models/scoring-event.ts +++ b/app/models/scoring-event.ts @@ -676,7 +676,8 @@ export async function getEventsForDates( if (!eventDatesMap.has(row.id)) { eventDatesMap.set(row.id, new Set()); } - const dateSet = eventDatesMap.get(row.id)!; + const dateSet = eventDatesMap.get(row.id); + if (!dateSet) continue; if (row.eventDate && dates.includes(row.eventDate)) { dateSet.add(row.eventDate); } @@ -712,7 +713,7 @@ export async function getEventsForDates( // Return one entry per (event, date) pair so that an event with games on // both today and tomorrow appears in both tabs of the dashboard widget. return events.flatMap((e) => { - const matchingDates = [...(eventDatesMap.get(e.id) ?? [])].sort(); + const matchingDates = [...(eventDatesMap.get(e.id) ?? [])].toSorted(); const base = { id: e.id, name: e.name,