optimizely / javascript-sdk

78%

LAST BUILD ON BRANCH master

branch: SELECT

CHANGE BRANCH

x

• No branch selected
• 2.0-consolidation
• 4.x.x
• 5.x.x
• ai/esrakartalOpt/FSSDK-12735-exclude-targeted-delivery
• ai/farhananjum-opti/FSSDK-12337-feature-rollouts
• ai/jaeopt/FSSDK-12546
• ai/jaeopt/FSSDK-12670-odp-event
• ai/jaeopt/FSSDK-12721-clean-build-test
• ai/jaeopt/FSSDK-12721-clean-build-test-2
• ai/jaeopt/FSSDK-12760-local-datafile
• ai/jaeopt/FSSDK-12813-holdout-event
• ai/mat001/FSSDK-12248
• ai/mat001/FSSDK-12248-retry-interval
• ai/mat001/FSSDK-12368-legacy-holdout-cleanup
• ai/mat001/FSSDK-12369-local-holdouts
• ai/raju-opti/FSSDK-12275-experiment-type
• ai/raju-opti/FSSDK-12750-test-1
• client-rum
• dependabot/npm_and_yarn/happy-dom-16.6.0
• dependabot/npm_and_yarn/multi-9739d0f518
• dependabot/npm_and_yarn/multi-cc6a460733
• dependabot/npm_and_yarn/rollup-2.79.2
• esra/FSSDK-11100_audience_evaluator_test
• esra/FSSDK-12168_fix_dependabot_alerts
• esra/dependabot/update_happy-dom
• esra/vitest/FSSDK-11095_rewrite_condition_tree_test
• fahad/enforce-linting
• farhan-anjum/FSSDK-11073-GHA-publish-to-GPR
• farhan-anjum/FSSDK-11073-update-gha-for-gpr-publish-on-5.x.x
• farhan-anjum/FSSDK-11446-add-exp/var-id-to-notif-listener
• farhan/FSSDK-12891-trusted-publishing
• js-test-branch-1
• js-test-branch-2
• junaed/fssdk-10655-fsc-decide-reason-fix
• junaed/fssdk-10935-refactor-log-export
• junaed/fssdk-11003-make-services-disposable-for-ssr
• junaed/fssdk-11034-tests-js-to-ts
• junaed/fssdk-11098-issue-with-akamai-edge-worker
• junaed/fssdk-1119-test-js-to-ts
• junaed/fssdk-11197-event-tag-type-fix
• junaed/fssdk-11197-event-tag-type-fix-5.x.x
• junaed/fssdk-11238-convert-notification-center-utils-test
• junaed/fssdk-11403-readme-update
• junaed/fssdk-11473-remove-crossbrowser-and-umd-from-pipeline
• junaed/fssdk-11483-browser-content-length-removal
• junaed/fssdk-11483-react-native-peer-range-update
• junaed/fssdk-11502-async-decides-to-opti-user-context
• junaed/fssdk-11509-netinfo-require-error
• junaed/fssdk-11515-changelog-update
• junaed/fssdk-11528-add-holdout-support
• junaed/fssdk-11529-update-impression-event
• junaed/fssdk-11938-custom-headers-support
• junaed/fssdk-11981-bug-bash-addressed
• junaed/fssdk-12009-expose-prediction-endpoint-cmab
• junaed/fssdk-12114-peer-deps-update
• junaed/fssdk-12115-notification-center-type
• junaed/fssdk-12865-url-parse-alt
• junaed/fssdk-12881-ghr-workflow-adjustmet
• junaed/odp-manager-fix
• junaed/prepare-5.3.5
• junaed/prepare-5.4.1
• junaed/release-5.4.0
• master
• mpirnovar-testing-fsc-js
• raju/browser_check
• raju/bucket
• raju/build_refacator
• raju/cache_methods
• raju/child_logger
• raju/cleanup-log
• raju/cmab_client
• raju/cmab_concurrent
• raju/cmab_conf
• raju/cmab_config
• raju/cmab_decision
• raju/cmab_retry
• raju/cmab_traffic
• raju/cmabservice
• raju/config-cleanup
• raju/config-refactor
• raju/config_cache
• raju/content-length-fix
• raju/context
• raju/createInstance-perf
• raju/custom-header
• raju/datafile_validation
• raju/date
• raju/decision_notification
• raju/decision_test
• raju/default-opt
• raju/dep
• raju/dependabot
• raju/entrypoint_types
• raju/errmsg
• raju/es6
• raju/event_cleanup
• raju/event_created_at
• raju/event_default_flush
• raju/event_experiment
• raju/event_flush_default
• raju/event_limit_fix
• raju/event_retry
• raju/event_store_limit
• raju/event_store_ttl
• raju/exception-msg
• raju/exclude_cmab_ups
• raju/export-entrypoint
• raju/export_map
• raju/factory_throw
• raju/flushImmediate
• raju/fsc-test
• raju/high_alert
• raju/holdout-noti
• raju/holdout-toggle
• raju/holdout_config
• raju/http_support
• raju/karma
• raju/log-alias
• raju/log-fix
• raju/logger-factory
• raju/logging-error-handling
• raju/loglevel
• raju/maxRetires
• raju/message-gen
• raju/more-type
• raju/msg
• raju/msgen-refactor
• raju/no_config
• raju/node18
• raju/node24
• raju/notification-center
• raju/odp-cleanup
• raju/odp-log
• raju/odp-refactor
• raju/odp_event_api
• raju/odp_factory
• raju/opaque_factory
• raju/opti_service
• raju/optimizely_client_refactor
• raju/platform_validator
• raju/prep-6.3.0
• raju/prepare-6.0.0
• raju/public_error_message
• raju/readme
• raju/region
• raju/release-5.3.6
• raju/release-6.1.0
• raju/release-6.2.0
• raju/release-6.3.1
• raju/release-6.5.0
• raju/remove-plugins-dir
• raju/remove_xb
• raju/retry_config
• raju/sync_update_config
• raju/typo
• raju/typo_fix
• raju/ua-parser
• raju/umd_test
• raju/universal_entrypoint
• raju/unstable_condition_evaluator
• raju/update_version
• raju/ups_async
• raju/user_attr
• raju/usercontext
• raju/util-cleanup
• raju/uuid-13-commonjs-refactor
• raju/validate
• raju/validation
• raju/version-test
• raju/vitest
• raju/vitest-public
• raju/workflow
• refs/heads/master
• remove-sourceclear-vulnerability
• sohail/removetokens

Build # 30569315153

Build Type

push

github

Committed by web-flow

Commit Message

[AI-FSSDK] [FSSDK-12735] Add holdout exclusion logic for Targeted Delivery rules (#1171)

Implements holdout exclusion logic for Targeted Delivery rules. When a holdout's `excludeTargetedDeliveries` flag is true, users bucketed into that holdout will still be served Targeted Delivery (rollout) rules normally, while A/B Test and Multi-Armed Bandit experiment rules continue to be blocked by the holdout.

## Changes

- Added `excludeTargetedDeliveries` field to the Holdout interface in shared types
- Updated `resolveVariationForFlag` to evaluate delivery rules when holdout has exclusion enabled, falling back to holdout decision if no delivery rule matches
- Added decision reason logging for when Targeted Delivery rules are excluded from holdout logic

---------

Co-authored-by: Raju Ahmed <raju.ahmed@optimizely.com>

Coverage Stats

1549 of 3027 branches covered (51.17%)

Branch coverage included in aggregate %.

6 of 15 new or added lines in 2 files covered. (40.0%)

1 existing line in 1 file now uncovered.

8990 of 10549 relevant lines covered (85.22%)

261.48 hits per line