Add support for skipInitialPoll parameter (FF-2084) (#64)

* bump to commons 3.0.6; add LocalStorageBackedAsyncStore (FF-1980)

* Add support for skipInitialPoll parameter (FF-2084)

* rename to skipInitialRequest

Author: leoromanovsky

src/index.spec.ts

@@ -506,6 +506,28 @@ describe('initialization options', () => {
     expect(client.getStringAssignment(flagKey, 'subject', {}, 'default-value')).toBe('control');
   });
 
+  it('skips initial request', async () => {
+    let callCount = 0;
+
+    global.fetch = jest.fn(() => {
+      callCount += 1;
+
+      return Promise.resolve({
+        ok: true,
+        status: 200,
+        json: () => Promise.resolve(mockConfigResponse),
+      });
+    }) as jest.Mock;
+
+    await init({
+      apiKey,
+      baseUrl,
+      assignmentLogger: mockLogger,
+      skipInitialRequest: true,
+    });
+    expect(callCount).toBe(0);
+  });
+
   describe('With reloaded index module', () => {
     // eslint-disable-next-line @typescript-eslint/ban-types
     let init: Function;

src/index.ts

@@ -64,6 +64,11 @@ export interface IClientConfig {
    * backoff. (Default: 7)
    */
   numPollRequestRetries?: number;
+
+  /**
+   * Skip the request for new configurations during initialization. (default: false)
+   */
+  skipInitialRequest?: boolean;
 }
 
 export { IAssignmentLogger, IAssignmentEvent, IEppoClient } from '@eppo/js-client-sdk-common';
@@ -160,6 +165,7 @@ export async function init(config: IClientConfig): Promise<IEppoClient> {
       pollAfterSuccessfulInitialization: config.pollAfterSuccessfulInitialization ?? false,
       pollAfterFailedInitialization: config.pollAfterFailedInitialization ?? false,
       throwOnFailedInitialization: true, // always use true here as underlying instance fetch is surrounded by try/catch
+      skipInitialPoll: config.skipInitialRequest ?? false,
     };
 
     EppoJSClient.instance.setLogger(config.assignmentLogger);