[FSSDK-12249] cleanup

Author: junaed-optimizely

src/client/client.spec.ts

@@ -85,13 +85,13 @@ describe('createInstance', () => {
       expect(meta.hasOdpManager).toBe(true);
     });
 
-    it('should set isVuidEnabled to false when vuidManager is not provided', () => {
+    it('should set hasVuidManager to false when vuidManager is not provided', () => {
       const client = createInstance(createTestConfig());
       const meta = (client as unknown as ClientWithMeta)[REACT_CLIENT_META];
       expect(meta.hasVuidManager).toBe(false);
     });
 
-    it('should set isVuidEnabled to true when vuidManager is provided', () => {
+    it('should set hasVuidManager to true when vuidManager is provided', () => {
       const client = createInstance(createTestConfig({ vuidManager: {} as Config['vuidManager'] }));
       const meta = (client as unknown as ClientWithMeta)[REACT_CLIENT_META];
       expect(meta.hasVuidManager).toBe(true);

src/provider/OptimizelyProvider.tsx

@@ -60,6 +60,7 @@ export function OptimizelyProvider({
     [client, store]
   );
 
+  // Effect 1: client lifecycle (onReady and error handling)
   useEffect(() => {
     if (!client) {
       logger?.error('OptimizelyProvider must be passed an Optimizely client instance');

src/utils/UserContextManager.ts

@@ -113,7 +113,7 @@ export class UserContextManager {
       return;
     }
 
-    // Step 3: Original path (no qualifiedSegments)
+    //  odpManager and no qualifiedSegments
     if (!this.skipSegments && this.meta.hasOdpManager) {
       await this.client.onReady();
       if (this.isStale(requestId)) return;