chore(coverage): restore SDK statements coverage threshold to ≥50

@EricAndrechek

Summary

The SDK statements coverage threshold in .testcoverage.yml was temporarily lowered from 50 → 40 in #174 ("refactor(table names): handle unsafe table names") to keep the PR scope manageable while new SDK endpoint/auth behavior was being shipped.

Goal

Restore the sdk statements threshold to 50 or higher once targeted SDK tests covering the new query-parameter endpoints and SSE auth changes are added.

Context

Lowered in: refactor(table names): handle unsafe table names #174 (comment: refactor(table names): handle unsafe table names #174 (comment))
Affected config key: suites.sdk in .testcoverage.yml
Files with new/changed SDK behavior that need test coverage:
- clients/ts/src/table.ts — ingest/schema endpoints changed to ?table= query params
- clients/ts/src/query-builder.ts — query endpoint changed to /v1/query?table=
- clients/ts/src/stream/sse.ts — SSEOptions gained optional auth callback; reconnect/error handling expanded
- clients/ts/src/client.ts — _createStream now passes auth to SSETransport

Requested by

@EricAndrechek

🔗 Related PRs

#123 - fix(api): drop CORS credentials + skip same-origin decoration [merged]

📝 Issue Planner

_{Check the box below or use the @coderabbitai plan command to generate an implementation plan and prompts that you can use with your favorite coding assistant.}

Create Plan

🧪 Issue enrichment is currently in open beta.

You can configure auto-planning by selecting labels in the issue_enrichment configuration.

To disable automatic issue enrichment, add the following to your .coderabbit.yaml:

issue_enrichment:
  auto_enrich:
    enabled: false

💬 Have feedback or questions? Drop into our discord!

@EricAndrechek

_createStream transport-selection test coverage

Flagged in PR #174 (comment: #174 (comment)) — the _createStream transport-selection test suite was removed because the async SSE auth path broke the sync-vs-async detection hack used to distinguish SSE from WS.

Specific cases that need to be covered once the testing approach is updated:

Auto mode, no auth → SSE transport is selected (EventSource constructor is called)
Auto mode, auth present → WS transport is selected (WebSocket constructor is called)
Forced SSE with auth → SSE transport is selected even when auth is set; auth token is injected as token query parameter in the SSE URL
Disconnect lifecycle → disconnect() closes the connection, decrements the active connection counter (clamped to 0), and emits a closed status

Requested by @EricAndrechek.

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

Navigation Menu

Repository navigation

chore(coverage): restore SDK statements coverage threshold to ≥50 #176

Summary

Goal

Context

Requested by

_createStream transport-selection test coverage

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

chore(coverage): restore SDK statements coverage threshold to ≥50 #176

Description

Summary

Goal

Context

Requested by

Activity

coderabbitai commented on May 23, 2026

coderabbitai commented on May 23, 2026

_createStream transport-selection test coverage

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions