Skip to content

Add initial support for feature flags - #244

Merged
tdumitrescu merged 16 commits into
masterfrom
efahk_flags
Nov 12, 2025
Merged

tdumitrescu merged 16 commits into
masterfrom
efahk_flags

Conversation

@efahk

@efahk efahk commented Nov 6, 2025 •

Copy link
Copy Markdown
Contributor

Implements a first pass at server side support for feature flagging

supports local evaluation - Flag settings are polled in the background and user assignment to specific variants is implemented on the SDK itself

remote evaluation - Each invocation to get a variant value makes a network request to the flags endpoint

@efahk
efahk requested a review from tdumitrescu November 6, 2025 04:09
@efahk efahk assigned msiebert and unassigned msiebert Nov 6, 2025
Comment thread lib/flags/local_flags.js
Comment thread lib/flags/local_flags.js Outdated
Comment thread lib/flags/local_flags.js Outdated
Comment thread lib/flags/remote_flags.d.ts Outdated
Comment thread lib/flags/utils.js Outdated
@efahk
efahk requested a review from msiebert November 7, 2025 01:31
Comment thread lib/flags/local_flags.js
Comment thread lib/flags/utils.js
Comment thread lib/flags/local_flags.js Outdated
Comment thread lib/flags/local_flags.js Outdated
Comment thread lib/flags/local_flags.js Outdated
Comment thread lib/flags/local_flags.js Outdated
Comment thread lib/flags/remote_flags.js
Comment thread lib/flags/remote_flags.js Outdated
Comment thread lib/flags/utils.js Outdated
Comment thread lib/flags/utils.js
Comment thread lib/flags/utils.js Outdated
Comment thread test/flags/utils.js
@efahk
efahk requested a review from tdumitrescu November 11, 2025 01:06

@tdumitrescu tdumitrescu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the updates, it's just about ready now

Comment thread lib/flags/local_flags.js Outdated
Comment thread test/flags/utils.js
});
});

describe('should handle special characters', function() {

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice. Thanks for extending test coverage

Comment thread test/flags/utils.js Outdated
Comment thread history.md Outdated
Comment thread package.json Outdated
Comment thread test/flags/utils.js Outdated
Comment thread package-lock.json
@@ -1,7 +1,7 @@
{
"name": "mixpanel",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tdumitrescu, lock file generated on install dropped a bunch of metadata. Shall i leave all of the changes made out completely?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for pointing out. @hans-lizihan merged some dep updates to master just now too, so I got this branch up to date and updated the lockfile

Comment thread lib/flags/local_flags.js Outdated
@tdumitrescu
tdumitrescu merged commit 27942b9 into master Nov 12, 2025
6 checks passed
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.

4 participants