Fix: Reorder overloads to fix type inference

The poison pill overloads were matching BEFORE the specific overloads, causing TypeScript to infer DisabledQueryError for valid queries. TypeScript checks overloads top-to-bottom and uses the first match. Since QueryBuilder is assignable to QueryBuilder | undefined | null, the poison pill overloads were matching first. Solution: Move poison pill overloads to the END, just before the implementation. This ensures: 1. Specific overloads (without undefined) match first 2. Poison pill overloads (with undefined) only match when needed All tests now pass with no type errors.

Author: claude

packages/react-db/src/useLiveSuspenseQuery.ts

@@ -113,32 +113,6 @@ type DisabledQueryError = {
  * )
  * ```
  */
-// "Poison pill" overloads - catch disabled queries and show custom compile-time error
-export function useLiveSuspenseQuery<TContext extends Context>(
- queryFn: (
- q: InitialQueryBuilder
- ) => QueryBuilder<TContext> | undefined | null,
- deps?: Array<unknown>
-): DisabledQueryError
-
-export function useLiveSuspenseQuery<TContext extends Context>(
- queryFn: (
- q: InitialQueryBuilder
- ) => LiveQueryCollectionConfig<TContext> | undefined | null,
- deps?: Array<unknown>
-): DisabledQueryError
-
-export function useLiveSuspenseQuery<
- TResult extends object,
- TKey extends string | number,
- TUtils extends Record<string, any>,
->(
- queryFn: (
- q: InitialQueryBuilder
- ) => Collection<TResult, TKey, TUtils> | undefined | null,
- deps?: Array<unknown>
-): DisabledQueryError
-
 // Overload 1: Accept query function that always returns QueryBuilder
 export function useLiveSuspenseQuery<TContext extends Context>(
  queryFn: (q: InitialQueryBuilder) => QueryBuilder<TContext>,
@@ -185,6 +159,33 @@ export function useLiveSuspenseQuery<
  collection: Collection<TResult, TKey, TUtils> & SingleResult
 }
 
+// "Poison pill" overloads - catch disabled queries and show custom compile-time error
+// These MUST come AFTER the specific overloads so TypeScript tries them last
+export function useLiveSuspenseQuery<TContext extends Context>(
+ queryFn: (
+ q: InitialQueryBuilder
+ ) => QueryBuilder<TContext> | undefined | null,
+ deps?: Array<unknown>
+): DisabledQueryError
+
+export function useLiveSuspenseQuery<TContext extends Context>(
+ queryFn: (
+ q: InitialQueryBuilder
+ ) => LiveQueryCollectionConfig<TContext> | undefined | null,
+ deps?: Array<unknown>
+): DisabledQueryError
+
+export function useLiveSuspenseQuery<
+ TResult extends object,
+ TKey extends string | number,
+ TUtils extends Record<string, any>,
+>(
+ queryFn: (
+ q: InitialQueryBuilder
+ ) => Collection<TResult, TKey, TUtils> | undefined | null,
+ deps?: Array<unknown>
+): DisabledQueryError
+
 // Implementation - uses useLiveQuery internally and adds Suspense logic
 export function useLiveSuspenseQuery(
  configOrQueryOrCollection: any,