Skip to content

feat: add FDv2 configuration builder#545

Draft
beekld wants to merge 2 commits into
beeklimt/SDK-2379-4from
beeklimt/SDK-2100
Draft

feat: add FDv2 configuration builder#545
beekld wants to merge 2 commits into
beeklimt/SDK-2379-4from
beeklimt/SDK-2100

Conversation

@beekld
Copy link
Copy Markdown
Contributor

@beekld beekld commented Jun 2, 2026

Stacked on #540. Adds a public FDv2Builder and wires the FDv2 data system into MakeDataSystem, so end users can opt into the FDv2 protocol via the existing config-builder pattern.

// Default FDv2 stack (streaming primary, polling fallback,
// FDv1 streaming last-resort)
auto config = ConfigBuilder("sdk-key-123")
    .DataSystem().Method(DataSystemBuilder::FDv2())
    /* ... */;

// Customised: payload filter + shorter fallback timeout, no FDv1 fallback
auto config = ConfigBuilder("sdk-key-123")
    .DataSystem().Method(
        DataSystemBuilder::FDv2()
            .Streaming(FDv2Builder::StreamingSource().Filter("mobile-flags"))
            .FallbackTimeout(std::chrono::seconds{30})
            .DisableFDv1Fallback())
    /* ... */;

Test plan

  • 5 new FDv2 builder tests in config_builder_test.cpp
  • Full server suite green (503 tests)

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