@tanstack/db 0.1.0 → 0.1.3

package/src/query/builder/types.ts

@@ -1,6 +1,7 @@
 import type { CollectionImpl } from "../../collection.js"
-import type { Aggregate, BasicExpression } from "../ir.js"
+import type { Aggregate, BasicExpression, OrderByDirection } from "../ir.js"
 import type { QueryBuilder } from "./index.js"
+import type { ResolveType } from "../../types.js"
 
 export interface Context {
   // The collections available in the base schema
@@ -27,13 +28,16 @@ export type Source = {
 }
 
 // Helper type to infer collection type from CollectionImpl
+// This uses ResolveType directly to ensure consistency with collection creation logic
 export type InferCollectionType<T> =
-  T extends CollectionImpl<infer U> ? U : never
+  T extends CollectionImpl<infer U, any, any, infer TSchema, any>
+    ? ResolveType<U, TSchema, U>
+    : never
 
 // Helper type to create schema from source
 export type SchemaFromSource<T extends Source> = Prettify<{
-  [K in keyof T]: T[K] extends CollectionImpl<infer U>
-    ? U
+  [K in keyof T]: T[K] extends CollectionImpl<any, any, any, any, any>
+    ? InferCollectionType<T[K]>
     : T[K] extends QueryBuilder<infer TContext>
       ? GetResult<TContext>
       : never
@@ -58,16 +62,18 @@ export type SelectObject<
 // Helper type to get the result type from a select object
 export type ResultTypeFromSelect<TSelectObject> = {
   [K in keyof TSelectObject]: TSelectObject[K] extends RefProxy<infer T>
-    ? // For RefProxy, preserve the type as-is (including optionality from joins)
-      T
+    ? T
     : TSelectObject[K] extends BasicExpression<infer T>
       ? T
       : TSelectObject[K] extends Aggregate<infer T>
         ? T
         : TSelectObject[K] extends RefProxyFor<infer T>
-          ? // For RefProxyFor, preserve the type as-is (including optionality from joins)
-            T
-          : never
+          ? T
+          : TSelectObject[K] extends undefined
+            ? undefined
+            : TSelectObject[K] extends { __type: infer U }
+              ? U
+              : never
 }
 
 // Callback type for orderBy clauses
@@ -75,6 +81,29 @@ export type OrderByCallback<TContext extends Context> = (
   refs: RefProxyForContext<TContext>
 ) => any
 
+export type OrderByOptions = {
+  direction?: OrderByDirection
+  nulls?: `first` | `last`
+} & StringSortOpts
+
+export type StringSortOpts =
+  | {
+      stringSort?: `lexical`
+    }
+  | {
+      stringSort?: `locale`
+      locale?: string
+      localeOptions?: object
+    }
+
+export type CompareOptions = {
+  direction: OrderByDirection
+  nulls: `first` | `last`
+  stringSort: `lexical` | `locale`
+  locale?: string
+  localeOptions?: object
+}
+
 // Callback type for groupBy clauses
 export type GroupByCallback<TContext extends Context> = (
   refs: RefProxyForContext<TContext>
@@ -119,12 +148,11 @@ export type RefProxyFor<T> = OmitRefProxy<
       ? // T is optional (T | undefined) but not exactly undefined
         NonUndefined<T> extends Record<string, any>
         ? {
-            // Properties are accessible and their types become optional
-            [K in keyof NonUndefined<T>]: NonUndefined<T>[K] extends Record<
+            [K in keyof NonUndefined<T>]-?: NonUndefined<T>[K] extends Record<
               string,
               any
             >
-              ? RefProxyFor<NonUndefined<T>[K] | undefined> &
+              ? RefProxyFor<NonUndefined<T>[K]> &
                   RefProxy<NonUndefined<T>[K] | undefined>
               : RefProxy<NonUndefined<T>[K] | undefined>
           } & RefProxy<T>
@@ -132,9 +160,14 @@ export type RefProxyFor<T> = OmitRefProxy<
       : // T is not optional
         T extends Record<string, any>
         ? {
-            [K in keyof T]: T[K] extends Record<string, any>
-              ? RefProxyFor<T[K]> & RefProxy<T[K]>
-              : RefProxy<T[K]>
+            // Make all properties required, but for optional ones, include undefined in the RefProxy type
+            [K in keyof T]-?: undefined extends T[K]
+              ? T[K] extends Record<string, any>
+                ? RefProxyFor<T[K]> & RefProxy<T[K]>
+                : RefProxy<T[K]>
+              : T[K] extends Record<string, any>
+                ? RefProxyFor<T[K]> & RefProxy<T[K]>
+                : RefProxy<T[K]>
           } & RefProxy<T>
         : RefProxy<T>
 >

package/src/query/compiler/order-by.ts

@@ -1,5 +1,5 @@
 import { orderByWithFractionalIndex } from "@tanstack/db-ivm"
-import { ascComparator, descComparator } from "../../utils/comparison.js"
+import { defaultComparator, makeComparator } from "../../utils/comparison.js"
 import { compileExpression } from "./evaluators.js"
 import type { OrderByClause } from "../ir.js"
 import type { NamespacedAndKeyedStream, NamespacedRow } from "../../types.js"
@@ -19,7 +19,7 @@ export function processOrderBy(
   // Pre-compile all order by expressions
   const compiledOrderBy = orderByClause.map((clause) => ({
     compiledExpression: compileExpression(clause.expression),
-    direction: clause.direction,
+    compareOptions: clause.compareOptions,
   }))
 
   // Create a value extractor function for the orderBy operator
@@ -53,35 +53,31 @@ export function processOrderBy(
   }
 
   // Create a multi-property comparator that respects the order and direction of each property
-  const makeComparator = () => {
-    return (a: unknown, b: unknown) => {
-      // If we're comparing arrays (multiple properties), compare each property in order
-      if (orderByClause.length > 1) {
-        const arrayA = a as Array<unknown>
-        const arrayB = b as Array<unknown>
-        for (let i = 0; i < orderByClause.length; i++) {
-          const direction = orderByClause[i]!.direction
-          const compareFn =
-            direction === `desc` ? descComparator : ascComparator
-          const result = compareFn(arrayA[i], arrayB[i])
-          if (result !== 0) {
-            return result
-          }
+  const comparator = (a: unknown, b: unknown) => {
+    // If we're comparing arrays (multiple properties), compare each property in order
+    if (orderByClause.length > 1) {
+      const arrayA = a as Array<unknown>
+      const arrayB = b as Array<unknown>
+      for (let i = 0; i < orderByClause.length; i++) {
+        const clause = orderByClause[i]!
+        const compareFn = makeComparator(clause.compareOptions)
+        const result = compareFn(arrayA[i], arrayB[i])
+        if (result !== 0) {
+          return result
         }
-        return arrayA.length - arrayB.length
-      }
-
-      // Single property comparison
-      if (orderByClause.length === 1) {
-        const direction = orderByClause[0]!.direction
-        return direction === `desc` ? descComparator(a, b) : ascComparator(a, b)
       }
+      return arrayA.length - arrayB.length
+    }
 
-      return ascComparator(a, b)
+    // Single property comparison
+    if (orderByClause.length === 1) {
+      const clause = orderByClause[0]!
+      const compareFn = makeComparator(clause.compareOptions)
+      return compareFn(a, b)
     }
-  }
 
-  const comparator = makeComparator()
+    return defaultComparator(a, b)
+  }
 
   // Use fractional indexing and return the tuple [value, index]
   return pipeline.pipe(

package/src/query/ir.ts

@@ -2,6 +2,7 @@
 This is the intermediate representation of the query.
 */
 
+import type { CompareOptions } from "./builder/types"
 import type { CollectionImpl } from "../collection"
 import type { NamespacedRow } from "../types"
 
@@ -48,7 +49,7 @@ export type OrderBy = Array<OrderByClause>
 
 export type OrderByClause = {
   expression: BasicExpression
-  direction: OrderByDirection
+  compareOptions: CompareOptions
 }
 
 export type OrderByDirection = `asc` | `desc`

package/src/transactions.ts

@@ -355,6 +355,7 @@ class Transaction<T extends object = Record<string, unknown>> {
 
     if (this.mutations.length === 0) {
       this.setState(`completed`)
+      this.isPersisted.resolve(this)
 
       return this
     }

package/src/utils/comparison.ts

@@ -1,3 +1,5 @@
+import type { CompareOptions } from "../query/builder/types"
+
 // WeakMap to store stable IDs for objects
 const objectIds = new WeakMap<object, number>()
 let nextObjectId = 1
@@ -19,21 +21,26 @@ function getObjectId(obj: object): number {
  * Handles null/undefined, strings, arrays, dates, objects, and primitives
  * Always sorts null/undefined values first
  */
-export const ascComparator = (a: any, b: any): number => {
+export const ascComparator = (a: any, b: any, opts: CompareOptions): number => {
+  const { nulls } = opts
+
   // Handle null/undefined
   if (a == null && b == null) return 0
-  if (a == null) return -1
-  if (b == null) return 1
+  if (a == null) return nulls === `first` ? -1 : 1
+  if (b == null) return nulls === `first` ? 1 : -1
 
   // if a and b are both strings, compare them based on locale
   if (typeof a === `string` && typeof b === `string`) {
-    return a.localeCompare(b)
+    if (opts.stringSort === `locale`) {
+      return a.localeCompare(b, opts.locale, opts.localeOptions)
+    }
+    // For lexical sort we rely on direct comparison for primitive values
   }
 
   // if a and b are both arrays, compare them element by element
   if (Array.isArray(a) && Array.isArray(b)) {
     for (let i = 0; i < Math.min(a.length, b.length); i++) {
-      const result = ascComparator(a[i], b[i])
+      const result = ascComparator(a[i], b[i], opts)
       if (result !== 0) {
         return result
       }
@@ -74,6 +81,32 @@ export const ascComparator = (a: any, b: any): number => {
  * Descending comparator function for ordering values
  * Handles null/undefined as largest values (opposite of ascending)
  */
-export const descComparator = (a: unknown, b: unknown): number => {
-  return ascComparator(b, a)
+export const descComparator = (
+  a: unknown,
+  b: unknown,
+  opts: CompareOptions
+): number => {
+  return ascComparator(b, a, {
+    ...opts,
+    nulls: opts.nulls === `first` ? `last` : `first`,
+  })
+}
+
+export function makeComparator(
+  opts: CompareOptions
+): (a: any, b: any) => number {
+  return (a, b) => {
+    if (opts.direction === `asc`) {
+      return ascComparator(a, b, opts)
+    } else {
+      return descComparator(a, b, opts)
+    }
+  }
 }
+
+/** Default comparator orders values in ascending order with nulls first and locale string comparison. */
+export const defaultComparator = makeComparator({
+  direction: `asc`,
+  nulls: `first`,
+  stringSort: `locale`,
+})