comment

Author: raju-opti

lib/core/decision_service/index.ts

@@ -72,9 +72,7 @@ import {
 import { OptimizelyError } from '../../error/optimizly_error';
 import { CmabService } from './cmab/cmab_service';
 import { Maybe, OpType, OpValue } from '../../utils/type';
-import { opValue, Value } from '../../utils/promise/operation_value';
-import { error } from 'console';
-import { get } from 'http';
+import { Value } from '../../utils/promise/operation_value';
 
 export const EXPERIMENT_NOT_RUNNING = 'Experiment %s is not running.';
 export const RETURNING_STORED_VARIATION =

lib/utils/promise/operation_value.ts

@@ -11,6 +11,11 @@ export const opValue = <OP extends OpType, V>(op: OP, val: V | Promise<V>): OpVa
   return Value.of(op, val).get();
 }
 
+/**
+ * A class to wraps a value that can be either a synchronous value or a promise and provides 
+ * a promise like API. This class is used to handle both synchronous and asynchronous values 
+ * in a uniform way.
+ */
 export class Value<OP extends OpType, V> {
   constructor(public op: OP, public val: OpValue<OP, V>) {}