[FSSDK-11395] add specific type for decision notification

Author: raju-opti

lib/common_exports.ts

@@ -30,7 +30,6 @@ export { createErrorNotifier } from './error/error_notifier_factory';
 
 export { 
   DECISION_SOURCES,
-  DECISION_NOTIFICATION_TYPES,
   NOTIFICATION_TYPES,
 } from './utils/enums';
 

lib/core/decision_service/index.ts

@@ -19,6 +19,7 @@ import {
   AUDIENCE_EVALUATION_TYPES,
   CONTROL_ATTRIBUTES,
   DECISION_SOURCES,
+  DecisionSource,
 } from '../../utils/enums';
 import {
   getAudiencesById,
@@ -114,7 +115,7 @@ export const CMAB_FETCHED_VARIATION_INVALID = 'Fetched variation %s for cmab exp
 export interface DecisionObj {
   experiment: Experiment | null;
   variation: Variation | null;
-  decisionSource: string;
+  decisionSource: DecisionSource;
   cmabUuid?: string;
 }
 

lib/entrypoint.test-d.ts

@@ -48,7 +48,6 @@ import {
 
 import {
   DECISION_SOURCES,
-  DECISION_NOTIFICATION_TYPES,
   NOTIFICATION_TYPES,
 } from './utils/enums';
 
@@ -89,7 +88,6 @@ export type Entrypoint = {
 
   // enums
   DECISION_SOURCES: typeof DECISION_SOURCES;
-  DECISION_NOTIFICATION_TYPES: typeof DECISION_NOTIFICATION_TYPES;
   NOTIFICATION_TYPES: typeof NOTIFICATION_TYPES;
 
   // decide options

lib/entrypoint.universal.test-d.ts

@@ -41,7 +41,6 @@ import { RequestHandler } from './utils/http_request_handler/http';
 import { UniversalBatchEventProcessorOptions } from './event_processor/event_processor_factory.universal';
 import {
   DECISION_SOURCES,
-  DECISION_NOTIFICATION_TYPES,
   NOTIFICATION_TYPES,
 } from './utils/enums';
 
@@ -82,7 +81,6 @@ export type UniversalEntrypoint = {
 
   // enums
   DECISION_SOURCES: typeof DECISION_SOURCES;
-  DECISION_NOTIFICATION_TYPES: typeof DECISION_NOTIFICATION_TYPES;
   NOTIFICATION_TYPES: typeof NOTIFICATION_TYPES;
 
   // decide options

lib/notification_center/type.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright 2024, Optimizely
+ * Copyright 2024-2025, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -15,7 +15,9 @@
  */
 
 import { LogEvent } from '../event_processor/event_dispatcher/event_dispatcher';
-import { EventTags, Experiment, UserAttributes, Variation } from '../shared_types';
+import { EventTags, Experiment, FeatureVariableValue, UserAttributes, VariableType, Variation } from '../shared_types';
+import { DecisionSource } from '../utils/enums';
+import { Nullable } from '../utils/type';
 
 export type UserEventListenerPayload = {
   userId: string;
@@ -43,16 +45,75 @@ export const DECISION_NOTIFICATION_TYPES = {
   FLAG: 'flag',
 } as const;
 
+
 export type DecisionNotificationType = typeof DECISION_NOTIFICATION_TYPES[keyof typeof DECISION_NOTIFICATION_TYPES];
 
-// TODO: Add more specific types for decision info
-export type OptimizelyDecisionInfo = Record<string, any>;
+export type ExperimentAndVariationInfo = {
+  experimentKey: string;
+  variationKey: string;
+}
+
+export type DecisionSourceInfo = Partial<ExperimentAndVariationInfo>;
 
-export type DecisionListenerPayload = UserEventListenerPayload & {
-  type: DecisionNotificationType;
-  decisionInfo: OptimizelyDecisionInfo;
+export type AbTestDecisonInfo = Nullable<ExperimentAndVariationInfo, 'variationKey'>;
+
+type FeatureDecisionInfo = {
+  featureKey: string,
+  featureEnabled: boolean,
+  source: DecisionSource,
+  sourceInfo: Partial<ExperimentAndVariationInfo>,
 }
 
+export type FeatureTestDecisionInfo = Nullable<ExperimentAndVariationInfo, 'variationKey'>;
+
+export type FeatureVariableDecisionInfo = {
+  featureKey: string,
+  featureEnabled: boolean,
+  source: DecisionSource,
+  variableKey: string,
+  variableValue: FeatureVariableValue,
+  variableType: VariableType,
+  sourceInfo: DecisionSourceInfo,
+};
+
+export type VariablesMap = { [variableKey: string]: unknown }
+
+export type AllFeatureVariablesDecisionInfo = {
+  featureKey: string,
+  featureEnabled: boolean,
+  source: DecisionSource,
+  variableValues: VariablesMap,
+  sourceInfo: DecisionSourceInfo,
+};
+
+export type FlagDecisionInfo = {
+  flagKey: string,
+  enabled: boolean,
+  variationKey: string | null,
+  ruleKey: string | null,
+  variables: VariablesMap,
+  reasons: string[],
+  decisionEventDispatched: boolean,
+};
+
+export type DecisionInfo = {
+  [DECISION_NOTIFICATION_TYPES.AB_TEST]: AbTestDecisonInfo;
+  [DECISION_NOTIFICATION_TYPES.FEATURE]: FeatureDecisionInfo;
+  [DECISION_NOTIFICATION_TYPES.FEATURE_TEST]: FeatureTestDecisionInfo;
+  [DECISION_NOTIFICATION_TYPES.FEATURE_VARIABLE]: FeatureVariableDecisionInfo;
+  [DECISION_NOTIFICATION_TYPES.ALL_FEATURE_VARIABLES]: AllFeatureVariablesDecisionInfo;
+  [DECISION_NOTIFICATION_TYPES.FLAG]: FlagDecisionInfo;
+}
+
+export type DecisionListenerPayloadForType<T extends DecisionNotificationType> = UserEventListenerPayload & {
+  type: T;
+  decisionInfo: DecisionInfo[T];
+}
+
+export type DecisionListenerPayload = {
+  [T in DecisionNotificationType]: DecisionListenerPayloadForType<T>;
+}[DecisionNotificationType];
+
 export type LogEventListenerPayload = LogEvent;
 
 export type OptimizelyConfigUpdateListenerPayload = undefined;

lib/tests/test_data.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright 2016-2021, 2024 Optimizely
+ * Copyright 2016-2021, 2024-2025 Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -3555,7 +3555,7 @@ export var getMutexFeatureTestsConfig = function() {
 export var rolloutDecisionObj = {
   experiment: null,
   variation: null,
-  decisionSource: 'rollout',
+  decisionSource: 'rollout' as const,
 };
 
 export var featureTestDecisionObj = {
@@ -3611,7 +3611,7 @@ export var featureTestDecisionObj = {
     variables: [],
     variablesMap: {}
   },
-  decisionSource: 'feature-test',
+  decisionSource: 'feature-test' as const,
 };
 
 var similarRuleKeyConfig = {

lib/utils/enums/index.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright 2016-2024, Optimizely
+ * Copyright 2016-2025, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -44,15 +44,6 @@ export const NODE_CLIENT_ENGINE = 'node-sdk';
 export const REACT_NATIVE_JS_CLIENT_ENGINE = 'react-native-js-sdk';
 export const CLIENT_VERSION = '5.3.4';
 
-export const DECISION_NOTIFICATION_TYPES = {
-  AB_TEST: 'ab-test',
-  FEATURE: 'feature',
-  FEATURE_TEST: 'feature-test',
-  FEATURE_VARIABLE: 'feature-variable',
-  ALL_FEATURE_VARIABLES: 'all-feature-variables',
-  FLAG: 'flag',
-};
-
 /*
  * Represents the source of a decision for feature management. When a feature
  * is accessed through isFeatureEnabled or getVariableValue APIs, the decision
@@ -63,7 +54,9 @@ export const DECISION_SOURCES = {
   FEATURE_TEST: 'feature-test',
   ROLLOUT: 'rollout',
   EXPERIMENT: 'experiment',
-};
+} as const;
+
+export type DecisionSource = typeof DECISION_SOURCES[keyof typeof DECISION_SOURCES];
 
 export const AUDIENCE_EVALUATION_TYPES = {
   RULE: 'rule',

lib/utils/type.ts

@@ -1,5 +1,5 @@
 /**
- * Copyright 2024, Optimizely
+ * Copyright 2024-2025, Optimizely
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -31,3 +31,9 @@ export type Either<A, B> = { type: 'left', value: A } | { type: 'right', value:
 
 export type OpType = 'sync' | 'async';
 export type OpValue<O extends OpType, V> = O extends 'sync' ? V : Promise<V>;
+
+export type OrNull<T> = T | null;
+
+export type Nullable<T, K extends keyof T> = {
+  [P in keyof T]: P extends K ? OrNull<T[P]> : T[P];
+}