Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

Ordered, limited live query over a string key commits `loading` first (and `orderBy().findOne()` since 0.11.1)

Abierto
  • ¿Algo no está bien?
#2,152 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

@KyleAMathews ya está trabajando en esto.

Desde el 14/10/2026.

  • #2159 de @KyleAMathews — abierto

Evaluación

Dificultad
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
18/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
react, typescript
Área
data, frontend

Línea de trabajo

The fallback is in OrderedSourceLoader.loadBoundary, and observe() decides sync versus async around isInitialFullSource and canRetryRepair in the same loader file. Run the repro test in packages/react-db/tests to see the loading-then-ready commit. Done when a string-ordered limit or findOne is ready in the first commit and the db test suite stays green, but PR #2159 already targets this and the fix needs maintainer-level understanding of the sync cut.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

  • I've validated the bug against the latest version of DB packages (@tanstack/db 0.14.0, @tanstack/react-db 0.5.9)

Describe the bug

An ordered, limited live query whose first orderBy term is a string commits loading with no rows on its first render, even when the source already holds every row and answers synchronously. It becomes ready one task later. The same query ordered by a number is ready in the first commit, which is what #1896 set out to guarantee.

Since 0.11.1 this also hits orderBy(...).findOne(), because #1968 made findOne() imply limit(1) (compiler/order-by.ts). On 0.11.0, orderBy(stringField).findOne() was ready in the first commit; from 0.11.1 it commits loading first. Ordering by an ISO date string and taking the latest row is a common shape, so screens that branch on "no row yet" render their empty state for a frame.

Cause

With locale string collation, canExpressCursorOrder cannot express the boundary as a cursor, so OrderedSourceLoader.loadBoundary falls back to loadFullSource(). observe() treats every full-source request as an authoritative repair. Only a full-source request made before any request has settled (isInitialFullSource) shares the initial synchronous cut, so this fallback, which is still part of the initial load, settles asynchronously.

Logged observe() decisions for the same query:

number: ordered (sync) -> boundary (sync)          => ready in first commit
string: ordered (sync) -> full-source (async)      => loading, then ready

To Reproduce

packages/react-db/tests (jsdom, @testing-library/react):

import { renderHook } from '@testing-library/react'
import { useLayoutEffect } from 'react'
import { createCollection, localOnlyCollectionOptions } from '@tanstack/db'
import { expect, it } from 'vitest'
import { useLiveQuery } from '../src/useLiveQuery'

it(`publishes a synchronous ordered limit over a string key in the first layout commit`, () => {
  const source = createCollection(
    localOnlyCollectionOptions<{ id: string; at: string }, string>({
      getKey: (row) => row.id,
      initialData: [
        { id: `1`, at: `2026-10-01` },
        { id: `2`, at: `2026-10-02` },
      ],
    }),
  )
  const commits: Array<string> = []
  renderHook(() => {
    const result = useLiveQuery((q) =>
      q.from({ row: source }).orderBy(({ row }) => row.at, `desc`).findOne(),
    )
    useLayoutEffect(() => {
      commits.push(`${result.status}:${result.data?.id}`)
    })
  })
  expect(commits[0]).toBe(`ready:2`) // receives `loading:undefined`
})

The same happens with .limit(n) instead of .findOne(), with or without an index on the field (autoIndex: 'eager', BasicIndex), and over eager sync collections as well as local-only ones. Changing at to a number makes it pass.

Expected behavior

A full-source fallback made before the live query is ready is part of the initial load, so it should share the initial synchronous cut like the first full-source request of a requiresFullSource plan.

Possible fix

canRetryRepair (the loader's "live query is ready and no window operation is active" predicate) already distinguishes the initial load:

@@ -624,9 +624,13 @@ export class OrderedSourceLoader {
     const orderingInvalidationGeneration = this.orderingInvalidationGeneration
     const settlesAsync = result instanceof Promise
     // A requiresFullSource plan's first full-source request is its initial
-    // load, not a repair, so it shares the initial synchronous cut.
+    // load, not a repair, so it shares the initial synchronous cut. So does a
+    // full-source fallback made before the query is ready, such as a boundary
+    // that a locale string order cannot express as a cursor.
     const isInitialFullSource =
-      isFullSource && !this.hasSettledSourceRequest && !this.needsOrderingRepair
+      isFullSource &&
+      !this.needsOrderingRepair &&
+      (!this.hasSettledSourceRequest || !this.canRetryRepair())
     const canSettleSynchronously =
       !settlesAsync &&
       windowOperationGeneration === undefined &&

With this change at 0.14.0, the repro passes, all 383 react-db tests pass, and 12,905 of 12,906 db tests pass. The one failure, local-storage-peer-oracle.test.ts > publishes through the default same-tab Storage object, fails identically without the change.

Additional context

Related: #1894 / #1896 (same symptom for numeric keys). We are running this as a local patch on 0.14.0 in a React Native app (Expo, new architecture).

Lenguaje dominante
TypeScript
Estrellas
3.9k
Forks
272
Merge medio
1 d 48 min
PR fusionados (30 d)
238

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de TanStack/db

Todos los issues de TanStack/db

Issues similares

Más issues de TypeScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.