feat(audience): rename event property keys to snake_case

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: nattb8

src/Packages/Audience/Runtime/Core/Session.cs

@@ -97,7 +97,7 @@ internal void Start()
 
             SafeTrack("session_start", new Dictionary<string, object>
             {
-                ["sessionId"] = sessionId
+                ["session_id"] = sessionId
             });
         }
 
@@ -173,8 +173,8 @@ internal void End()
             // wall-clock; dashboards should not assume parity.
             SafeTrack("session_end", new Dictionary<string, object>
             {
-                ["sessionId"] = sessionId,
-                ["durationSec"] = duration
+                ["session_id"] = sessionId,
+                ["duration_sec"] = duration
             });
         }
 
@@ -198,8 +198,8 @@ internal void EmitEndAndSeal()
 
             SafeTrack("session_end", new Dictionary<string, object>
             {
-                ["sessionId"] = sessionId,
-                ["durationSec"] = duration
+                ["session_id"] = sessionId,
+                ["duration_sec"] = duration
             });
         }
 
@@ -240,8 +240,8 @@ internal void OnHeartbeat()
             // Build outside _lock so track doesn't re-enter.
             var properties = new Dictionary<string, object>
             {
-                ["sessionId"] = sessionId,
-                ["durationSec"] = duration
+                ["session_id"] = sessionId,
+                ["duration_sec"] = duration
             };
 
             SafeTrack("session_heartbeat", properties);

src/Packages/Audience/Runtime/Events/TypedEvents.cs

@@ -95,7 +95,7 @@ public Dictionary<string, object> ToProperties()
             if (Level != null) props["level"] = Level;
             if (Stage != null) props["stage"] = Stage;
             if (Score.HasValue) props["score"] = Score.Value;
-            if (DurationSec.HasValue) props["durationSec"] = DurationSec.Value;
+            if (DurationSec.HasValue) props["duration_sec"] = DurationSec.Value;
 
             return props;
         }
@@ -182,8 +182,8 @@ public Dictionary<string, object> ToProperties()
                 ["amount"] = Amount.Value
             };
 
-            if (ItemType != null) props["itemType"] = ItemType;
-            if (ItemId != null) props["itemId"] = ItemId;
+            if (ItemType != null) props["item_type"] = ItemType;
+            if (ItemId != null) props["item_id"] = ItemId;
 
             return props;
         }
@@ -256,10 +256,10 @@ public Dictionary<string, object> ToProperties()
                 ["value"] = Value.Value
             };
 
-            if (ItemId != null) props["itemId"] = ItemId;
-            if (ItemName != null) props["itemName"] = ItemName;
+            if (ItemId != null) props["item_id"] = ItemId;
+            if (ItemName != null) props["item_name"] = ItemName;
             if (Quantity.HasValue) props["quantity"] = Quantity.Value;
-            if (TransactionId != null) props["transactionId"] = TransactionId;
+            if (TransactionId != null) props["transaction_id"] = TransactionId;
 
             return props;
         }

src/Packages/Audience/Runtime/ImmutableAudience.cs

@@ -1121,17 +1121,17 @@ private static void FireGameLaunch(
 
             // Config-supplied distributionPlatform overrides the provider value.
             if (config.DistributionPlatform != null)
-                properties["distributionPlatform"] = config.DistributionPlatform;
+                properties["distribution_platform"] = config.DistributionPlatform;
 
             // Emitted only on the first launch where SKAN registration fires.
             if (skanRegistered == true)
-                properties["skanRegistered"] = true;
+                properties["skan_registered"] = true;
 
             // iOS ATT/IDFA snapshot, merged after Unity context so attribution
             // keys are authoritative if both sources happen to set the same key.
             // idfa and gaid are cross-app device identifiers, same privacy class
-            // as userId; gate them at Full-only. State-class keys (attStatus,
-            // gaidLimitAdTracking) are non-identifying and ship at Anon+Full.
+            // as userId; gate them at Full-only. State-class keys (att_status,
+            // gaid_limit_ad_tracking) are non-identifying and ship at Anon+Full.
             if (attributionContext != null)
             {
                 var canIdentify = consentAtInit.CanIdentify();
@@ -1160,7 +1160,7 @@ private static void FireInstallReferrerReceivedOnce(AudienceConfig config, strin
 
             Track("install_referrer_received", new Dictionary<string, object>
             {
-                ["installReferrer"] = installReferrer,
+                ["install_referrer"] = installReferrer,
             });
 
             try
@@ -1235,8 +1235,8 @@ private static void CheckAndFireAttStatusChanged(
 
             var props = new Dictionary<string, object>
             {
-                ["previousStatus"] = AttStatusToString(previous.Value),
-                ["newStatus"] = AttStatusToString(currentStatus),
+                ["previous_status"] = AttStatusToString(previous.Value),
+                ["new_status"] = AttStatusToString(currentStatus),
             };
 
             if (currentStatus == 3 && consent.CanIdentify())

src/Packages/Audience/Runtime/Unity/DeviceCollector.cs

@@ -55,33 +55,33 @@ internal static Dictionary<string, object> CollectGameLaunchProperties(
             var props = new Dictionary<string, object>
             {
                 ["platform"] = PlatformName(platform),
-                ["isEditor"] = isEditor,
+                ["is_editor"] = isEditor,
                 ["version"] = Truncate(Application.version, 256),
-                ["buildGuid"] = Truncate(Application.buildGUID, 256),
-                ["unityVersion"] = Truncate(Application.unityVersion, 256),
-                ["osFamily"] = SystemInfo.operatingSystemFamily.ToString(),
-                ["deviceModel"] = Truncate(SystemInfo.deviceModel, 256),
+                ["build_guid"] = Truncate(Application.buildGUID, 256),
+                ["unity_version"] = Truncate(Application.unityVersion, 256),
+                ["os_family"] = SystemInfo.operatingSystemFamily.ToString(),
+                ["device_model"] = Truncate(SystemInfo.deviceModel, 256),
                 ["gpu"] = Truncate(SystemInfo.graphicsDeviceName, 256),
-                ["gpuVendor"] = Truncate(SystemInfo.graphicsDeviceVendor, 256),
+                ["gpu_vendor"] = Truncate(SystemInfo.graphicsDeviceVendor, 256),
                 ["cpu"] = Truncate(SystemInfo.processorType, 256),
-                ["cpuCores"] = SystemInfo.processorCount,
-                ["ramMb"] = SystemInfo.systemMemorySize,
+                ["cpu_cores"] = SystemInfo.processorCount,
+                ["ram_mb"] = SystemInfo.systemMemorySize,
             };
 
             // Screen.dpi can be 0 on some Linux WMs.
             var dpi = (int)Screen.dpi;
-            if (dpi > 0) props["screenDpi"] = dpi;
+            if (dpi > 0) props["screen_dpi"] = dpi;
 
             if (platform == RuntimePlatform.Android)
-                props["androidId"] = Truncate(SystemInfo.deviceUniqueIdentifier, 256);
+                props["android_id"] = Truncate(SystemInfo.deviceUniqueIdentifier, 256);
 
             if (platform == RuntimePlatform.IPhonePlayer)
             {
                 var idfv = IDFVBridge.GetIDFV();
                 if (idfv != null) props["idfv"] = Truncate(idfv, 256);
 
                 // iOS baseline is 163 DPI (1×); 326 → 2×, 401-460 → 3×.
-                if (dpi > 0) props["screenScale"] = (int)Math.Round(dpi / 163.0);
+                if (dpi > 0) props["screen_scale"] = (int)Math.Round(dpi / 163.0);
             }
 
             return props;

src/Packages/Audience/Runtime/Unity/Mobile/AttributionContext.cs

@@ -46,7 +46,7 @@ internal static IReadOnlyDictionary<string, object> Capture(string? persistentDa
             // never ships there. Native ATTBridge calls are themselves gated
             // by #if UNITY_IOS, so non-iOS editor targets get the safe stubs.
             var status = ATTBridge.GetStatus();
-            props["attStatus"] = AttStatusToString(status);
+            props["att_status"] = AttStatusToString(status);
 
             // Only ship IDFA when the user has authorized tracking. The native
             // bridge already returns null for the zero-UUID case, but gating
@@ -84,7 +84,7 @@ internal static void EmitGaidProps(GAIDInfo info, IDictionary<string, object> pr
         {
             if (!info.LimitAdTracking && !string.IsNullOrEmpty(info.Gaid))
                 props["gaid"] = info.Gaid;
-            props["gaidLimitAdTracking"] = info.LimitAdTracking;
+            props["gaid_limit_ad_tracking"] = info.LimitAdTracking;
         }
     }
 }

src/Packages/Audience/Tests/Runtime/Core/SessionTests.cs

@@ -35,8 +35,8 @@ public void Start_FiresSessionStart_WithSessionId()
 
             Assert.AreEqual(1, _events.Count);
             Assert.AreEqual("session_start", _events[0].name);
-            Assert.IsTrue(_events[0].props.ContainsKey("sessionId"));
-            Assert.IsNotEmpty((string)_events[0].props["sessionId"]);
+            Assert.IsTrue(_events[0].props.ContainsKey("session_id"));
+            Assert.IsNotEmpty((string)_events[0].props["session_id"]);
         }
 
         [Test]
@@ -64,9 +64,9 @@ public void End_FiresSessionEnd_WithDuration()
 
             var endEvent = _events.FirstOrDefault(e => e.name == "session_end");
             Assert.IsNotNull(endEvent.props);
-            Assert.IsTrue(endEvent.props.ContainsKey("sessionId"));
-            Assert.IsTrue(endEvent.props.ContainsKey("durationSec"));
-            Assert.AreEqual(2L, (long)endEvent.props["durationSec"]);
+            Assert.IsTrue(endEvent.props.ContainsKey("session_id"));
+            Assert.IsTrue(endEvent.props.ContainsKey("duration_sec"));
+            Assert.AreEqual(2L, (long)endEvent.props["duration_sec"]);
         }
 
         [Test]
@@ -121,8 +121,8 @@ void Track(string name, Dictionary<string, object> props)
             {
                 var beat = events.FirstOrDefault(e => e.name == "session_heartbeat");
                 Assert.IsNotNull(beat.props, "heartbeat event should carry a properties dictionary");
-                Assert.IsTrue(beat.props.ContainsKey("sessionId"));
-                Assert.IsTrue(beat.props.ContainsKey("durationSec"));
+                Assert.IsTrue(beat.props.ContainsKey("session_id"));
+                Assert.IsTrue(beat.props.ContainsKey("duration_sec"));
             }
         }
 
@@ -198,7 +198,7 @@ public void Pause_CalledTwice_SecondCallIsNoOp()
             session.End();
 
             var sessionEnd = _events.Last(e => e.name == "session_end");
-            var duration = (long)sessionEnd.props["durationSec"];
+            var duration = (long)sessionEnd.props["duration_sec"];
             // Wall-clock Start→End = 13s, paused from T=5 to T=10 = 5s, engaged = 8s.
             Assert.AreEqual(8L, duration,
                 "double Pause must preserve the first Pause timestamp so engagement arithmetic covers the full pause window");
@@ -246,7 +246,7 @@ public void Resume_NegativePauseDuration_ClampsAccumulatorToZero()
             session.End();
 
             var sessionEnd = _events.Last(e => e.name == "session_end");
-            var duration = (long)sessionEnd.props["durationSec"];
+            var duration = (long)sessionEnd.props["duration_sec"];
             // Wall-clock from Start to End is 10 + (-5) + 2 = 7 s. The
             // pause duration was clamped to 0, so engaged seconds = 7 - 0 = 7.
             // Without the clamp, _accumulatedPause would be -5, the
@@ -277,7 +277,7 @@ public void End_ClockRewindsSinceStart_ClampsDurationToZero()
             session.End();
 
             var sessionEnd = _events.Last(e => e.name == "session_end");
-            var duration = (long)sessionEnd.props["durationSec"];
+            var duration = (long)sessionEnd.props["duration_sec"];
             Assert.AreEqual(0L, duration,
                 "negative engaged time from a wall-clock rewind must clamp to zero");
         }
@@ -306,7 +306,7 @@ public void End_ClockRewindsWhilePaused_DoesNotInflateDuration()
             session.End();
 
             var sessionEnd = _events.Last(e => e.name == "session_end");
-            var duration = (long)sessionEnd.props["durationSec"];
+            var duration = (long)sessionEnd.props["duration_sec"];
             Assert.LessOrEqual(duration, 5L,
                 "clock rewind while paused must not over-credit engagement past the wall-clock window");
         }
@@ -331,7 +331,7 @@ public void End_AfterShortPause_ReportsDurationMinusPause()
             session.End();
 
             var sessionEnd = _events.Last(e => e.name == "session_end");
-            var duration = (long)sessionEnd.props["durationSec"];
+            var duration = (long)sessionEnd.props["duration_sec"];
             Assert.AreEqual(7L, duration,
                 "session_end duration should exclude the 3s paused interval");
         }
@@ -353,7 +353,7 @@ public void End_WhilePaused_ExcludesInFlightPauseFromDuration()
             session.End(); // ends while paused
 
             var sessionEnd = _events.Last(e => e.name == "session_end");
-            var duration = (long)sessionEnd.props["durationSec"];
+            var duration = (long)sessionEnd.props["duration_sec"];
             Assert.AreEqual(5L, duration,
                 "session_end fired while paused should count only pre-pause engaged time");
         }
@@ -381,7 +381,7 @@ public void End_AfterExtendedPauseRollover_ReportsPrePauseDuration()
             session.Resume();
 
             var sessionEnd = _events.First(e => e.name == "session_end");
-            var duration = (long)sessionEnd.props["durationSec"];
+            var duration = (long)sessionEnd.props["duration_sec"];
             Assert.AreEqual(10L, duration,
                 "session_end on extended-pause rollover should report pre-pause engaged time, not wall-clock");
         }
@@ -406,7 +406,7 @@ public void Heartbeat_AfterShortPause_ReportsPauseAdjustedDuration()
             session.OnHeartbeat();
 
             var heartbeat = _events.Last(e => e.name == "session_heartbeat");
-            var duration = (long)heartbeat.props["durationSec"];
+            var duration = (long)heartbeat.props["duration_sec"];
             Assert.AreEqual(6L, duration,
                 "heartbeat duration should exclude the 2s paused interval");
         }

src/Packages/Audience/Tests/Runtime/Events/TypedEventTests.cs

@@ -39,7 +39,7 @@ public void Progression_Complete_ProducesCorrectProperties()
             Assert.AreEqual("tutorial", props["world"]);
             Assert.AreEqual("1", props["level"]);
             Assert.AreEqual(1500, props["score"]);
-            Assert.AreEqual(120.5f, props["durationSec"]);
+            Assert.AreEqual(120.5f, props["duration_sec"]);
         }
 
         [Test]
@@ -52,7 +52,7 @@ public void Progression_OptionalFieldsOmitted_WhenNull()
             Assert.IsFalse(props.ContainsKey("level"));
             Assert.IsFalse(props.ContainsKey("stage"));
             Assert.IsFalse(props.ContainsKey("score"));
-            Assert.IsFalse(props.ContainsKey("durationSec"));
+            Assert.IsFalse(props.ContainsKey("duration_sec"));
         }
 
         [Test]
@@ -72,8 +72,8 @@ public void Resource_Source_ProducesCorrectProperties()
             Assert.AreEqual("source", props["flow"]);
             Assert.AreEqual("gold", props["currency"]);
             Assert.AreEqual(100m, props["amount"]);
-            Assert.AreEqual("quest_reward", props["itemType"]);
-            Assert.AreEqual("main_quest_01", props["itemId"]);
+            Assert.AreEqual("quest_reward", props["item_type"]);
+            Assert.AreEqual("main_quest_01", props["item_id"]);
         }
 
         [Test]
@@ -126,10 +126,10 @@ public void Purchase_ProducesCorrectProperties()
 
             Assert.AreEqual("USD", props["currency"]);
             Assert.AreEqual(9.99m, props["value"]);
-            Assert.AreEqual("gem_pack_01", props["itemId"]);
-            Assert.AreEqual("Starter Gem Pack", props["itemName"]);
+            Assert.AreEqual("gem_pack_01", props["item_id"]);
+            Assert.AreEqual("Starter Gem Pack", props["item_name"]);
             Assert.AreEqual(1, props["quantity"]);
-            Assert.AreEqual("txn_abc123", props["transactionId"]);
+            Assert.AreEqual("txn_abc123", props["transaction_id"]);
         }
 
         [Test]
@@ -139,10 +139,10 @@ public void Purchase_OptionalFieldsOmitted_WhenNull()
 
             Assert.IsTrue(props.ContainsKey("currency"));
             Assert.IsTrue(props.ContainsKey("value"));
-            Assert.IsFalse(props.ContainsKey("itemId"));
-            Assert.IsFalse(props.ContainsKey("itemName"));
+            Assert.IsFalse(props.ContainsKey("item_id"));
+            Assert.IsFalse(props.ContainsKey("item_name"));
             Assert.IsFalse(props.ContainsKey("quantity"));
-            Assert.IsFalse(props.ContainsKey("transactionId"));
+            Assert.IsFalse(props.ContainsKey("transaction_id"));
         }
 
         [Test]