feat: Add AsyncConfig for the async SDK client - #471
Open
jsonbailey wants to merge 4 commits into
Open
Conversation
Contributor
Author
|
@cursor review |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 23ca0b6. Configure here.
…interface Mirror how the sync Config types event_processor_class against the EventProcessor interface. Add an AsyncEventProcessor ABC to interfaces and reference it directly instead of the concrete implementation, which also removes the aiohttp-avoiding TYPE_CHECKING import.
Extract the default base/events/stream LaunchDarkly URIs into named constants in ldclient.config and reference them from both Config and AsyncConfig, so the two clients can't silently drift to different default endpoints. Also type update_processor_class against the AsyncUpdateProcessor interface rather than the sync UpdateProcessor.
Match sync Config, which always defaults to an in-memory store when none is provided, so callers and the async client can rely on a non-null feature store.
jsonbailey
marked this pull request as ready for review
July 28, 2026 16:15
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds the full
AsyncConfigclass toldclient/async_config.py, building on the previously-mergedAsyncBigSegmentsConfigseed (#462). This is a stacked-PR slice of the async Python SDK epic (SDK-60), unblocked now that #463 mergedAsyncHook/AsyncPlugin.AsyncConfigmirrors the syncConfigsurface for the async client: base/events/stream URIs, streaming vs. polling, event buffering, private attributes, big segments, hooks, plugins, HTTP config, application metadata, and the experimental data system config. The concreteAsyncEventProcessoris imported only underTYPE_CHECKINGto keep aiohttp out of the runtime import graph.This is self-contained and reviewable on its own. It unblocks the upcoming async FDv1 polling (PR 7) and async event processor (PR 8), both of which import the full
AsyncConfig.The public
AsyncConfig(andAsyncBigSegmentsConfig) carry an experimental.. caution::note — this API may change without notice and is not yet production-ready.Tracked internally: SDK-2768
Note
Medium Risk
Large new configuration surface and async lifecycle contracts affect how the async client will connect, store flags, and send events; behavior is experimental but touches core SDK integration points.
Overview
Introduces
AsyncConfigforAsyncLDClient, paralleling syncConfig: URIs, streaming/polling, events, privacy, big segments, hooks/plugins, HTTP, payload filters, and experimental data-system settings. Defaults toAsyncInMemoryFeatureStore, filtersAsyncHook/AsyncPlugin, and is marked experimental in docs.config.pyadds sharedDEFAULT_BASE_URI,DEFAULT_EVENTS_URI, andDEFAULT_STREAM_URI; syncConfignow uses those constants instead of inline URLs.interfaces.pyaddsAsyncUpdateProcessorandAsyncEventProcessor(asyncstop) as the extension points wired fromAsyncConfigfor upcoming async polling and event processing.Reviewed by Cursor Bugbot for commit 6a5ca9c. Bugbot is set up for automated code reviews on this repo. Configure here.