Skip to content

feat: Add the FDv2 connection modes to the client - #619

Draft
beekld wants to merge 6 commits into
bklimt/SDK-3033/client-fdv2-cache-initializerfrom
bklimt/SDK-3034/client-fdv2-connection-modes
Draft

beekld wants to merge 6 commits into
bklimt/SDK-3033/client-fdv2-cache-initializerfrom
bklimt/SDK-3034/client-fdv2-connection-modes

Conversation

@beekld

@beekld beekld commented Sep 25, 2026

Copy link
Copy Markdown
Contributor

Summary

Defines the client's FDv2 connection modes and assembles the data sources each one uses.

  • Adds a ConnectionMode enum and a client FDv2Config describing each mode's data sources.
  • Adds the desktop defaults: three modes, starting in streaming, with automatic mode switching off.
  • Adds BuildModeSources, which turns a mode's config into the initializer and synchronizer factories it needs.
  • Seeds the polling synchronizer's first-poll time from the cache's per-context freshness, so relaunching does not produce a burst of requests.

@beekld
beekld added this pull request to stack #608 September 25, 2026 17:01
@beekld
beekld force-pushed the bklimt/SDK-3034/client-fdv2-connection-modes branch from 282dc25 to f77e2c9 Compare September 26, 2026 05:54
@beekld
beekld force-pushed the bklimt/SDK-3033/client-fdv2-cache-initializer branch from fc7d735 to 60b04ea Compare September 26, 2026 05:54
@beekld
beekld force-pushed the bklimt/SDK-3034/client-fdv2-connection-modes branch from f77e2c9 to 58353ce Compare September 26, 2026 07:43
@beekld
beekld force-pushed the bklimt/SDK-3033/client-fdv2-cache-initializer branch from 60b04ea to a1f1bec Compare October 2, 2026 23:36
@beekld
beekld force-pushed the bklimt/SDK-3034/client-fdv2-connection-modes branch from 58353ce to 6d6cf49 Compare October 2, 2026 23:36
@beekld
beekld force-pushed the bklimt/SDK-3034/client-fdv2-connection-modes branch from 6d6cf49 to e58619d Compare October 3, 2026 01:21
@beekld
beekld removed this pull request from stack #608 October 3, 2026 01:24

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant