fix(types): LDClient.waitForInitialization resolves this

[SevenOutman]

Requirements

• I have added test coverage for new or changed functionality
• I have followed the repository's pull request submission guidelines
• I have validated my changes against all supported platform versions

Related issues

#324

Describe the solution you've provided

Type the return type with this keyword

Describe alternatives you've considered

Additional context

In my use case, the problem was that the return type of waitForInitialization is not compatible with ld.LDClient

import * as ld from '@launchdarkly/node-server-sdk';

export function initLdClient(): Promise<ld.LDClient> {
  return ld.init(process.env.LAUNCH_DARKLY_SDK_KEY as string)
           .waitForInitialization(); // Type error
}

[chadxz]

In the docs for the waitForInitialization method, it says:

It is not recommended to use the returned client object. It will be removed in a future version.

FYI.

[kinyoklion]

Hello @SevenOutman,

As @chadxz mentioned we have updated the docs to indicate the return value here will be removed in the future.

Some previous discussion: #324

I would recommend something like this instead.

export async function initLdClient(): Promise<ld.LDClient> {
  const client =  ld.init(process.env.LAUNCH_DARKLY_SDK_KEY as string);
  await client.waitForInitialization();
  return client;
}

(The fix is nice though.)

Thank you,
Ryan

[SevenOutman]

Understood. Thanks both! I'm closing the PR.