fix: adding wrapper name for react client (#1199)

<!-- CURSOR_SUMMARY -->
> [!NOTE]
> **Low Risk**
> Low risk: this only adjusts how `createClient` builds options for the
underlying JS client and adds focused unit tests; runtime behavior
changes are limited to default `wrapperName`/`wrapperVersion` values
sent to the base SDK.
> 
> **Overview**
> `createClient` in the React SDK now injects default `wrapperName`
(`react-client-sdk`) and `wrapperVersion` into the options passed to
`@launchdarkly/js-client-sdk`, while still merging user-provided options
and allowing explicit overrides.
> 
> Adds unit tests covering wrapper option behavior, and updates
`release-please-config.json` so `LDReactClient.tsx` is included in
version bump automation.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
8efb1e5. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

<!-- devin-review-badge-begin -->

---

<a href="https://app.devin.ai/review/launchdarkly/js-core/pull/1199"
target="_blank">
  <picture>
<source media="(prefers-color-scheme: dark)"
srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1">
<img
src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1"
alt="Open with Devin">
  </picture>
</a>
<!-- devin-review-badge-end -->

Author: joker23

packages/sdk/react/__tests__/client/LDReactClient.test.ts

@@ -321,3 +321,52 @@ it('noop client onInitializationStatusChange returns a no-op unsubscribe', () =>
   // @ts-ignore
   global.window = originalWindow;
 });
+
+it('passes wrapperName to the base client', () => {
+  const { mock } = makeMockBaseClient();
+  (createBaseClient as jest.Mock).mockReturnValue(mock);
+
+  createClient('test-id', { kind: 'user', key: 'u1' });
+
+  expect(createBaseClient).toHaveBeenCalledWith(
+    'test-id',
+    { kind: 'user', key: 'u1' },
+    expect.objectContaining({ wrapperName: 'react-client-sdk' }),
+  );
+});
+
+it('preserves user-provided options alongside wrapper defaults', () => {
+  const { mock } = makeMockBaseClient();
+  (createBaseClient as jest.Mock).mockReturnValue(mock);
+
+  createClient('test-id', { kind: 'user', key: 'u1' }, { streaming: false });
+
+  expect(createBaseClient).toHaveBeenCalledWith(
+    'test-id',
+    { kind: 'user', key: 'u1' },
+    expect.objectContaining({
+      wrapperName: 'react-client-sdk',
+      streaming: false,
+    }),
+  );
+});
+
+it('user-provided wrapperName/wrapperVersion overrides defaults', () => {
+  const { mock } = makeMockBaseClient();
+  (createBaseClient as jest.Mock).mockReturnValue(mock);
+
+  createClient(
+    'test-id',
+    { kind: 'user', key: 'u1' },
+    {
+      wrapperName: 'custom',
+      wrapperVersion: '9.9.9',
+    },
+  );
+
+  expect(createBaseClient).toHaveBeenCalledWith(
+    'test-id',
+    { kind: 'user', key: 'u1' },
+    expect.objectContaining({ wrapperName: 'custom', wrapperVersion: '9.9.9' }),
+  );
+});

packages/sdk/react/src/client/LDReactClient.tsx

@@ -5,6 +5,9 @@ import {
   type LDEvaluationDetailTyped,
   LDEvaluationReason,
   type LDFlagValue,
+  LDIdentifyOptions,
+  LDIdentifyResult,
+  LDOptions,
   LDStartOptions,
   LDWaitForInitializationResult,
 } from '@launchdarkly/js-client-sdk';
@@ -107,14 +110,21 @@ function createNoopReactClient(): LDReactClient {
 export function createClient(
   clientSideID: string,
   context: LDContext,
-  options?: LDReactClientOptions,
+  options: LDReactClientOptions = {},
 ): LDReactClient {
   if (isServerSide()) {
     return createNoopReactClient();
   }
-  const shouldUseCamelCaseFlagKeys = options?.useCamelCaseFlagKeys ?? true;
 
-  const baseClient = createBaseClient(clientSideID, context, options);
+  const { useCamelCaseFlagKeys: shouldUseCamelCaseFlagKeys = true, ...ldOptions } = options;
+
+  const baseClientOptions: LDOptions = {
+    ...ldOptions,
+    wrapperName: ldOptions?.wrapperName ?? 'react-client-sdk',
+    wrapperVersion: ldOptions?.wrapperVersion ?? '0.0.0', // x-release-please-version
+  };
+
+  const baseClient = createBaseClient(clientSideID, context, baseClientOptions);
   let initializationState: InitializedState = 'unknown';
   const subscribers = new Set<(context: LDContextStrict) => void>();
   const initStatusSubscribers = new Set<(result: LDWaitForInitializationResult) => void>();
@@ -129,15 +139,15 @@ export function createClient(
         return baseClient.start(startOptions);
       }
       initializationState = 'initializing';
-      return baseClient.start(startOptions).then((result) => {
+      return baseClient.start(startOptions).then((result: LDWaitForInitializationResult) => {
         initializationState = result.status;
         lastInitResult = result;
         initStatusSubscribers.forEach((cb) => cb(result));
         return result;
       });
     },
-    identify: (...args) =>
-      baseClient.identify(...args).then((result) => {
+    identify: (ldContext: LDContext, identifyOptions?: LDIdentifyOptions) =>
+      baseClient.identify(ldContext, identifyOptions).then((result: LDIdentifyResult) => {
         if (result.status === 'completed') {
           const newContext = baseClient.getContext();
           if (newContext) {
@@ -149,7 +159,7 @@ export function createClient(
     getInitializationState: () => initializationState,
     getInitializationError: () =>
       lastInitResult?.status === 'failed' ? lastInitResult.error : undefined,
-    onContextChange: (callback: (context: LDContextStrict) => void) => {
+    onContextChange: (callback: (ldContext: LDContextStrict) => void) => {
       subscribers.add(callback);
       return () => {
         subscribers.delete(callback);

release-please-config.json

@@ -104,7 +104,10 @@
       "bump-minor-pre-major": true
     },
     "packages/sdk/react": {
-      "bump-minor-pre-major": true
+      "bump-minor-pre-major": true,
+      "extra-files": [
+        "src/client/LDReactClient.tsx"
+      ]
     }
   },
   "plugins": [