feat: demonstrate per-hook evaluation exposure deduplication - #73
feat: demonstrate per-hook evaluation exposure deduplication#73abelonogov-ld wants to merge 5 commits into
Conversation
Register two counting hooks with different windows so the sample app shows how opt-in Hook.evaluationExposureDeduper works on iOS. Co-authored-by: Cursor <cursoragent@cursor.com>
Match the observability TestApp pattern: gitignored Secrets.xcconfig feeds Info.plist via Debug/Release wrappers that also include CocoaPods. Co-authored-by: Cursor <cursoragent@cursor.com>
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 6437dd1. Configure here.
This reverts commit 6437dd1.
The SDK no longer takes one, since tracking a result per flag bounds what a hook remembers on its own. Co-authored-by: Cursor <cursoragent@cursor.com>
Follows the SDK, where deduplication moved from a property a hook declares to a decorator that wraps it. Co-authored-by: Cursor <cursoragent@cursor.com>
|
The hello apps are meant to demonstrate default configuration applications initializing and do basic flag evaluation. Since we have not agreed that deduping should be a default mobile behavior, I don't think this is mergable. The hello apps are not meant to be a demonstration of various configuration options / feature demos. That would be something for the docs pages. SDK Spec HELLO-hello-apps has |

Summary
ExposureCountingHook+ExposureDedupeDemo) that registers two hooks with 5s and 10sevaluationExposureDeduperwindows — the same per-hook, opt-in API as ios-client-sdk#516.andrey/flag-exposure-dedupeSDK branch until that work is released; watchOS still usesLDClientConfiguratorwithout hooks.Test plan
pod installand openhello-ios.xcworkspaceExposureDedupeDemo.mobileKeyand a booleanfeatureFlagKeyLDClientConfiguratorMade with Cursor