fix: correct actor name field mapping in session info

Changed identifyActorName to use the dedicated userName field from the identify response instead of extracting it from nested userData.name property

Author: naji247

src/modules/session.ts

@@ -66,7 +66,7 @@ export function getSessionInfo(
     clientName: clientInfo?.name,
     clientVersion: clientInfo?.version,
     identifyActorGivenId: actorInfo?.userId,
-    identifyActorName: actorInfo?.userData?.name,
+    identifyActorName: actorInfo?.userName,
     identifyActorData: actorInfo?.userData || {},
   };
 

src/tests/identify.test.ts

@@ -457,6 +457,7 @@ describe("Identify Feature", () => {
   describe("Identity Data in Session Info", () => {
     it("should populate actorGivenId, actorName, and actorData in session info", async () => {
       const testUserId = `session-user-${randomUUID()}`;
+      const testUserName = `Session User ${randomUUID()}`;
       const testUserData = {
         name: `Session Test User ${randomUUID()}`,
         role: "Developer",
@@ -468,6 +469,7 @@ describe("Identify Feature", () => {
         enableTracing: true,
         identify: async () => ({
           userId: testUserId,
+          userName: testUserName,
           userData: testUserData,
         }),
       });
@@ -490,7 +492,7 @@ describe("Identify Feature", () => {
 
       expect(sessionInfo).toBeDefined();
       expect(sessionInfo?.identifyActorGivenId).toBe(testUserId);
-      expect(sessionInfo?.identifyActorName).toBe(testUserData.name);
+      expect(sessionInfo?.identifyActorName).toBe(testUserName);
       expect(sessionInfo?.identifyActorData).toEqual(testUserData);
     });
 
@@ -499,6 +501,7 @@ describe("Identify Feature", () => {
       await eventCapture.start();
 
       const testUserId = `event-user-${randomUUID()}`;
+      const testUserName = `Event User ${randomUUID()}`;
       const testUserData = {
         name: `Event Test User ${randomUUID()}`,
         subscription: "premium",
@@ -509,6 +512,7 @@ describe("Identify Feature", () => {
         enableTracing: true,
         identify: async () => ({
           userId: testUserId,
+          userName: testUserName,
           userData: testUserData,
         }),
       });
@@ -539,7 +543,7 @@ describe("Identify Feature", () => {
 
       const data = getServerTrackingData(server.server);
       expect(data?.sessionInfo.identifyActorGivenId).toBe(testUserId);
-      expect(data?.sessionInfo.identifyActorName).toBe(testUserData.name);
+      expect(data?.sessionInfo.identifyActorName).toBe(testUserName);
       expect(data?.sessionInfo.identifyActorData).toEqual(testUserData);
 
       await eventCapture.stop();

src/tests/redaction.test.ts

@@ -437,10 +437,10 @@ describe("redactEvent integration tests", () => {
       redactSensitiveInformation: redactSensitiveData,
       identify: async () => ({
         userId: "test-user-123",
+        userName: "John Doe",
         userData: {
           email: "user@example.com",
           apiKey: "secret-api-key-123",
-          name: "John Doe",
         },
       }),
     });
@@ -618,8 +618,8 @@ describe("redactEvent integration tests", () => {
       redactSensitiveInformation: aggressiveRedact,
       identify: async () => ({
         userId: "user-with-id-123",
+        userName: "David Smith",
         userData: {
-          name: "David Smith",
           internalId: "internal-id-456",
         },
       }),