1
0
Fork 0
mirror of synced 2024-09-20 11:27:56 +12:00

Expose a nicer API for getting/setting feature flags on the current context.

This commit is contained in:
Sam Rose 2024-08-14 15:56:12 +01:00
parent 08a56ef480
commit 011859397d
No known key found for this signature in database
2 changed files with 21 additions and 9 deletions

View file

@ -375,3 +375,22 @@ export function getCurrentContext(): ContextMap | undefined {
return undefined return undefined
} }
} }
export function getFeatureFlags<T extends Record<string, any>>(
key: string
): T | undefined {
const context = getCurrentContext()
if (!context) {
return undefined
}
return context.featureFlagCache?.[key] as T
}
export function setFeatureFlags(key: string, value: Record<string, any>) {
const context = getCurrentContext()
if (!context) {
return
}
context.featureFlagCache ??= {}
context.featureFlagCache[key] = value
}

View file

@ -130,10 +130,7 @@ export class FlagSet<V extends Flag<any>, T extends { [key: string]: V }> {
async fetch(ctx?: UserCtx): Promise<FlagValues<T>> { async fetch(ctx?: UserCtx): Promise<FlagValues<T>> {
return await tracer.trace("features.fetch", async span => { return await tracer.trace("features.fetch", async span => {
const requestContext = context.getCurrentContext() const cachedFlags = context.getFeatureFlags<FlagValues<T>>(this.setId)
const cachedFlags = requestContext?.featureFlagCache?.[this.setId] as
| FlagValues<T>
| undefined
if (cachedFlags) { if (cachedFlags) {
span?.addTags({ fromCache: true }) span?.addTags({ fromCache: true })
return cachedFlags return cachedFlags
@ -252,11 +249,7 @@ export class FlagSet<V extends Flag<any>, T extends { [key: string]: V }> {
} }
} }
if (requestContext) { context.setFeatureFlags(this.setId, flagValues)
requestContext.featureFlagCache ??= {}
requestContext.featureFlagCache[this.setId] = flagValues
}
for (const [key, value] of Object.entries(flagValues)) { for (const [key, value] of Object.entries(flagValues)) {
tags[`flags.${key}.value`] = value tags[`flags.${key}.value`] = value
} }