fix(db): remove redundant queryOnce undefined fallback

samwillis · samwillis · commit 491900bc3b88 · 2026-03-10T11:41:54.000Z
Drop the unnecessary nullish-coalescing in queryOnce so the single-result path returns the iterator value directly while preserving the existing undefined-on-empty behavior.

Made-with: Cursor
diff --git a/packages/db/src/query/query-once.ts b/packages/db/src/query/query-once.ts
@@ -105,7 +105,7 @@ export async function queryOnce<TContext extends Context>(
       const first = collection.values().next().value as
         | InferResultType<TContext>
         | undefined
-      return (first ?? undefined) as InferResultType<TContext>
+      return first as InferResultType<TContext>
     }
     return collection.toArray as InferResultType<TContext>
   } finally {

Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ export async function queryOnce<TContext extends Context>(`
`105`	`105`	`const first = collection.values().next().value as`
`106`	`106`	`\| InferResultType<TContext>`
`107`	`107`	`\| undefined`
`108`		`- return (first ?? undefined) as InferResultType<TContext>`
	`108`	`+ return first as InferResultType<TContext>`
`109`	`109`	`}`
`110`	`110`	`return collection.toArray as InferResultType<TContext>`
`111`	`111`	`} finally {`