refactor(sdk)!: restructure query builder + command naming to match new API routes #199

Activity

1. EricAndrechek

   self-assigned this

   on Jun 2, 2026
2. EricAndrechek

   added

   enhancementNew feature or requestNew feature or request

   area/queryStructured query AST, SQL builderStructured query AST, SQL builder

   area/sdkTypeScript SDK (clients/ts/)TypeScript SDK (clients/ts/)

   breaking-changeBreaking change to public API, CLI, or configBreaking change to public API, CLI, or config

   on Jun 2, 2026
3. github-project-automation

   added this to WaveHouse Task Boardon Jun 2, 2026
4. github-project-automation

   moved this to Backlog in WaveHouse Task Boardon Jun 2, 2026
5. coderabbitai

   added

   area/apiHTTP handlers, routing, middlewareHTTP handlers, routing, middleware

   on Jun 2, 2026
6. github-actions

   added

   area/observabilityMetrics, logs, traces, health, profilingMetrics, logs, traces, health, profiling

   on Jun 2, 2026
7. EricAndrechek

   added a parent issue on Jun 2, 2026

   • epic(sdk): SDK feature rollout after #190 cleanup #194
8. EricAndrechek

   moved this from Backlog to Ready in WaveHouse Task Boardon Jun 2, 2026
9. EricAndrechek

   added a commit that references this issue on Jun 2, 2026

   refactor(sdk)!: remove WebSockets + consolidate tooling/coverage (#190)

   ...

   a4d72c1
10. EricAndrechek

    removed

    area/observabilityMetrics, logs, traces, health, profilingMetrics, logs, traces, health, profiling

    on Jun 3, 2026
11. EricAndrechek commented on Jun 4, 2026

    

    EricAndrechek

    on Jun 4, 2026

    MemberAuthor

    More actions

    SDK ergonomics input from nas-observability dogfooding (jfwoods, 2026-05-05) — two friction points worth folding into the query-builder restructure:

    1. liveQuery discoverability — it lives on QueryBuilder (.select().liveQuery(...)), not on TableRef, but live-streaming is the most common reason users reach for .from(table). Consider a TableRef.liveQuery(...) shorthand, or at least document the chain in the TableRef docstring.
    2. Database constraint friendliness — DB extends Record<string, Record<string, unknown>> rejects the natural interface MyDB { table: MyRow } (no index signature), forcing a manual [t: string]: Record<string, unknown>. Either relax to a structural shape explicit-key interfaces satisfy, or document the workaround prominently.
12. EricAndrechek

    added a commit that references this issue on Jun 5, 2026

    fix(sdk): stop hardcoding received_timestamp in default query order (#…

    ...

    0975f88
13. EricAndrechek

    moved this from Ready to Backlog in WaveHouse Task Boardon Jun 9, 2026
14. EricAndrechek commented on Aug 25, 2026

    

    EricAndrechek

    on Aug 25, 2026

    MemberAuthor

    More actions

    Triage note — one constraint to carry into this restructure: #483 is an open, present-tense contract defect in the current builder. selectAll() preserves previously selected columns and select() preserves the selectAll state, so select('page').selectAll() silently changes the projection and selectAll().count() reaches the server and 400s. The docs already claim these are mutually exclusive.

    #483 was considered for folding into this issue and kept discrete — it's live today, and this restructure has no timeline. But whichever lands first, the other has to agree: if this restructure ships before #483 is fixed, the new builder needs to enforce (or explicitly document) that exclusivity rather than inherit the current ambiguity.

    Related: #483.

    ---

    Triage note from the pm-triage routine, 2026-08-25.