optimizely / go-sdk / 24422173128

92%

master: 92%

Build Type

Pull #445

github

Committed by Mat001

Commit Message

[FSSDK-12368] Implement Local Holdouts support

Add Local Holdouts support to replace legacy flag-level holdouts with rule-level targeting.

Changes:
- Add IncludedRules field to Holdout struct (replaces IncludedFlags/ExcludedFlags)
- Add IsGlobal() method for global vs local holdout detection
- Update HoldoutConfig mapping from flag-level to rule-level
- Implement GetGlobalHoldouts() and GetHoldoutsForRule(ruleID) methods
- Integrate local holdout evaluation in decision flow (per-rule, before audience/traffic)
- Handle edge cases (missing field, empty slice, invalid rule IDs, cross-flag targeting)
- Add comprehensive unit tests for local holdouts (7 test cases)

Quality Metrics:
- Tests: 7 comprehensive test cases
- Test Pass Rate: 100%
- Critical Issues: 0
- Warnings: 0

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Pull Request Pull Request #445: [AI-FSSDK] [FSSDK-12368] Local Holdouts - Cleanup flag base setup and add includedRules and rule-level lookup

Coverage Stats

26 of 186 new or added lines in 6 files covered. (13.98%)

113 existing lines in 2 files now uncovered.

5696 of 6360 relevant lines covered (89.56%)

6815.56 hits per line