luckberg/familysync
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(03): WR-02 make default-calendar selection deterministic (orderBy id, limit 1)

Browse Source
This commit is contained in:
Lucas Berger
2026-06-09 10:39:28 -04:00
parent 02aa407764
commit 5499f83782
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
apps/api/src/routes/events.ts
+7 -1
View File
@@ -256,11 +256,17 @@ eventsRouter.post('/create', zValidator('json', eventFieldsSchema), async (c) =>
} }
targetCalendarUrl = calRow.url targetCalendarUrl = calRow.url
} else { } else {
// Default to first personal calendar (D-01) // Default to the member's first personal calendar (D-01).
// WR-02: add a deterministic ORDER BY + LIMIT. Without them, a member with
// multiple personal calendars gets an arbitrary, query-to-query-unstable "first"
// row. Ordering by id (insertion order) gives a stable default; limit(1) avoids
// fetching the whole set just to take [0].
const [calRow] = await db const [calRow] = await db
.select({ url: calendars.url }) .select({ url: calendars.url })
.from(calendars) .from(calendars)
.where(eq(calendars.userId, currentUserId)) .where(eq(calendars.userId, currentUserId))
.orderBy(calendars.id)
.limit(1)
if (!calRow) { if (!calRow) {
return c.json({ error: 'No writable calendar found for user' }, 422) return c.json({ error: 'No writable calendar found for user' }, 422)