chore(audience-sdk): remove em dashes from comments and trim verbose phrasing

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

Author: nattb8

src/Packages/Audience/Editor/AudienceMobileBuildSettings.cs

@@ -27,7 +27,7 @@ public sealed class AudienceMobileBuildSettings : ScriptableObject
 
         [SerializeField]
         [Tooltip("Copy shown in the iOS App Tracking Transparency prompt. " +
-                 "Apple rejects empty or generic strings — describe what is " +
+                 "Apple rejects empty or generic strings. Describe what is " +
                  "collected and why.")]
         private string trackingUsageDescription = DefaultTrackingUsageDescription;
 
@@ -64,7 +64,7 @@ private static void CreateAsset()
             if (guids.Length > 1)
             {
                 Debug.LogWarning(
-                    $"[ImmutableAudience] Multiple AudienceMobileBuildSettings assets found — " +
+                    $"[ImmutableAudience] Multiple AudienceMobileBuildSettings assets found - " +
                     $"using '{path}'. Remove the duplicates to avoid unexpected build behaviour.");
             }
             return AssetDatabase.LoadAssetAtPath<AudienceMobileBuildSettings>(path);

src/Packages/Audience/Editor/iOSFrameworkPostProcessor.cs

@@ -19,15 +19,15 @@ namespace Immutable.Audience.Editor
     /// <c>AdSupport.framework</c> hosts <c>ASIdentifierManager</c> (the IDFA
     /// accessor). Without it linked, the runtime <c>NSClassFromString</c>
     /// lookup returns nil and the IDFA is silently dropped from
-    /// <c>game_launch</c> — the Info.plist key alone does not load this
+    /// <c>game_launch</c>. The Info.plist key alone does not load this
     /// framework. Linked as a required dep; the framework has shipped on
     /// every iOS release since 6.0.
     /// </para>
     /// <para>
     /// <c>AppTrackingTransparency.framework</c> hosts <c>ATTrackingManager</c>.
     /// Unity often auto-links this when <c>NSUserTrackingUsageDescription</c>
     /// is present, but the auto-link heuristic is undocumented and
-    /// version-sensitive — an explicit weak link is immune to that drift and
+    /// version-sensitive. An explicit weak link is immune to that drift and
     /// keeps the binary loadable on iOS &lt; 14 (the runtime code already
     /// guards via <c>NSClassFromString</c>).
     /// </para>
@@ -40,7 +40,7 @@ namespace Immutable.Audience.Editor
     internal static class iOSFrameworkPostProcessor
     {
         // Runs just after the Info.plist post-processor (9050). Order is
-        // independent in practice — both edit different files — but keeping
+        // independent in practice (both edit different files), but keeping
         // them adjacent makes the build log obvious.
         internal const int CallbackOrder = 9051;
 
@@ -76,7 +76,7 @@ internal static void OnPostProcessBuild(BuildTarget target, string pathToBuiltPr
 #endif
         }
 
-        // Reads the iOS-target define list specifically — the post-processor
+        // Reads the iOS-target define list specifically. The post-processor
         // mutates iOS build output regardless of which target the editor is
         // currently focused on.
         private static bool AttributionDefineEnabled()

src/Packages/Audience/Editor/iOSInfoPlistPostProcessor.cs

@@ -18,7 +18,7 @@ namespace Immutable.Audience.Editor
     /// <remarks>
     /// Both keys are gated by the <c>AUDIENCE_MOBILE_ATTRIBUTION</c>
     /// scripting define so a studio that hasn't opted into attribution
-    /// ships a clean <c>Info.plist</c> — Apple flags apps that include
+    /// ships a clean <c>Info.plist</c>. Apple flags apps that include
     /// either key without the corresponding code paths.
     ///
     /// Values come from the <see cref="AudienceMobileBuildSettings"/>
@@ -88,11 +88,11 @@ private static void ValidateBuildSettings()
                 $"  NSUserTrackingUsageDescription: {description}\n" +
                 $"  SKAdNetworkItems: {ids.Length} id(s)\n" +
                 (ids.Length == 0
-                    ? "  (no SKAdNetwork ids configured — set them on the AudienceMobileBuildSettings asset)\n"
+                    ? "  (no SKAdNetwork ids configured - set them on the AudienceMobileBuildSettings asset)\n"
                     : string.Concat(System.Array.ConvertAll(ids, id => $"    - {id}\n"))));
         }
 
-        // Reads the iOS-target define list specifically — the post-processor
+        // Reads the iOS-target define list specifically. The post-processor
         // mutates iOS build output regardless of which target the editor is
         // currently focused on.
         private static bool AttributionDefineEnabled()
@@ -114,7 +114,7 @@ internal static void ApplyTrackingUsageDescription(
                 ? settings.TrackingUsageDescription
                 : AudienceMobileBuildSettings.DefaultTrackingUsageDescription;
 
-            // Always overwrite — the settings asset is the source of truth,
+            // Always overwrite. The settings asset is the source of truth,
             // beating any placeholder a lower-order post-processor wrote.
             root.SetString("NSUserTrackingUsageDescription", description);
         }

src/Packages/Audience/Editor/iOSPrivacyManifestPostProcessor.cs

@@ -62,7 +62,7 @@ internal static void OnPostProcessBuild(BuildTarget target, string pathToBuiltPr
         /// <summary>
         /// Adds the attribution-specific privacy declarations to an existing
         /// (already Unity-merged) <c>PrivacyInfo.xcprivacy</c> plist root.
-        /// Idempotent — safe to call on a manifest that already has these entries.
+        /// Idempotent. Safe to call on a manifest that already has these entries.
         /// </summary>
         internal static void ApplyAttributionPrivacyEntries(PlistElementDict root)
         {

src/Packages/Audience/Runtime/AudienceConfig.cs

@@ -68,7 +68,7 @@ public class AudienceConfig
         ///    collected at runtime. Without the define, this setter is a
         ///    no-op.
         ///
-        /// Studios who set neither ship a clean binary — no AD_ID permission,
+        /// Studios who set neither ship a clean binary: no AD_ID permission,
         /// no native attribution code, <c>NSPrivacyTracking = false</c>.
         /// </remarks>
         public bool EnableMobileAttribution { get; set; } = false;

src/Packages/Audience/Runtime/ImmutableAudience.cs

@@ -742,7 +742,7 @@ private static void SyncConsentToBackend(AudienceConfig config, ConsentLevel lev
         /// </summary>
         /// <remarks>
         /// <para>
-        /// Studios decide when to show the prompt — Apple's HIG requires it
+        /// Studios decide when to show the prompt. Apple's HIG requires it
         /// to fire at a moment that makes the value to the player obvious,
         /// not at SDK Init. The IDFA, when authorized, is collected
         /// automatically on the next <see cref="ImmutableAudience.Init"/> via
@@ -751,7 +751,7 @@ private static void SyncConsentToBackend(AudienceConfig config, ConsentLevel lev
         /// </para>
         /// <para>
         /// Resolves to <see cref="TrackingAuthorizationStatus.NotDetermined"/>
-        /// in three distinct cases — callers who use this signal to decide
+        /// in three distinct cases. Callers who use this signal to decide
         /// whether to retry should consult the SDK log to disambiguate:
         /// </para>
         /// <list type="bullet">
@@ -1148,7 +1148,7 @@ private static void FireGameLaunch(
             if (skanRegistered == true)
                 properties["skanRegistered"] = true;
 
-            // iOS ATT/IDFA snapshot — merged after Unity context so attribution
+            // 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,
@@ -1170,7 +1170,7 @@ private static void FireGameLaunch(
         }
 
         // Fires install_referrer_received exactly once per install. Cache
-        // file presence alone isn't enough — on first launch the bridge may
+        // file presence alone isn't enough. On first launch the bridge may
         // write the cache after Init has already run, so the event must be
         // dispatched at the next Init that observes a cache hit. The on-disk
         // "sent" marker provides idempotency across that boundary.
@@ -1193,7 +1193,7 @@ private static void FireInstallReferrerReceivedOnce(AudienceConfig config, strin
             }
             catch (Exception ex) when (ex is IOException || ex is UnauthorizedAccessException)
             {
-                // Marker write failed — the event will re-fire on the next
+                // Marker write failed. The event will re-fire on the next
                 // launch. Pipeline-side dedup or the cost of one duplicate is
                 // less bad than never sending the event at all.
                 Log.Warn(AudienceLogs.InstallReferrerSentMarkerWriteFailed(ex));
@@ -1220,7 +1220,7 @@ private static string AttStatusToString(int status)
         // RequestTrackingAuthorizationAsync resolves).
         //
         // First observation (no file): persists the baseline and returns without
-        // firing — game_launch already captures the initial state on that Init.
+        // firing. game_launch already captures the initial state on that Init.
         private static void CheckAndFireAttStatusChanged(
             AudienceConfig config,
             ConsentLevel consent,

src/Packages/Audience/Runtime/Plugins/Android/proguard-user.txt

@@ -13,7 +13,7 @@
 # Keep Google Play Services AdvertisingIdClient symbols (GAID).
 #
 # AdvertisingIdClient and its inner Info class are reflected via JNI from
-# GAIDBridge — they have no managed-side reference for R8 to follow, so
+# GAIDBridge - they have no managed-side reference for R8 to follow, so
 # fullMode in studio projects can drop them. Defensive rules ensure the
 # getAdvertisingIdInfo / getId / isLimitAdTrackingEnabled surface survives.
 -keep class com.google.android.gms.ads.identifier.** { *; }

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

@@ -47,7 +47,7 @@ internal static class ATTBridge
         // Hold a single delegate instance for the lifetime of the process. If
         // we passed a fresh delegate to each P/Invoke call, IL2CPP could GC
         // the wrapper before Apple's completion handler fires (which can be
-        // seconds or minutes later — the user may swipe the prompt away or
+        // seconds or minutes later. The user may swipe the prompt away or
         // background the app).
         private static readonly NativeStatusCallback _staticCallback = OnATTStatus;
 
@@ -60,7 +60,7 @@ internal static class ATTBridge
         // - MonoPInvokeCallback so IL2CPP emits the trampoline that lets
         //   native code call back into managed code.
         // - Preserve so managed-code stripping at High doesn't drop this
-        //   method (it has no managed callers — only the function pointer).
+        //   method (no managed callers; only the function pointer).
         // Without both, the await in RequestAsync never completes on a
         // shipping build with stripping enabled.
         [MonoPInvokeCallback(typeof(NativeStatusCallback))]

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

@@ -14,7 +14,7 @@ namespace Immutable.Audience.Unity.Mobile
     // Android: gaid + gaidLimitAdTracking are read from the GAIDBridge disk
     // cache populated by the previous launch's background fetch (Google's
     // AdvertisingIdClient is sync + must run off main thread, so first launch
-    // ships nothing — gaidLimitAdTracking shows up on launch #2 onwards).
+    // ships nothing; gaidLimitAdTracking shows up on launch #2 onwards).
     internal static class AttributionContext
     {
         // Maps Apple's ATTrackingManagerAuthorizationStatus to the wire
@@ -33,7 +33,7 @@ internal static string AttStatusToString(int status)
         }
 
         // persistentDataPath is required for Android (GAID disk cache); iOS
-        // ignores it. Returns a possibly-empty dict — never null — so callers
+        // ignores it. Returns a possibly-empty dict (never null) so callers
         // can merge unconditionally.
         internal static IReadOnlyDictionary<string, object> Capture(string? persistentDataPath = null)
         {

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

@@ -14,7 +14,7 @@ namespace Immutable.Audience.Unity.Mobile
     /// Reads gaid + limitAdTracking via AdvertisingIdClient. Google requires
     /// the call run off the main thread, so we dispatch on a dedicated worker
     /// and cache the result to disk for the next launch. GAID can change
-    /// (user reset), so we refresh every launch — first launch ships nothing,
+    /// (user reset), so we refresh every launch. First launch ships nothing;
     /// launch #2+ ships the previously-cached value.
     /// </summary>
     internal static class GAIDBridge
@@ -101,7 +101,7 @@ internal static void WriteCacheEntry(string persistentDataPath, string gaidOrEmp
         private static void StartFetchNative(string persistentDataPath)
         {
             // Dedicated Thread (not ThreadPool) so Attach/Detach pair on the
-            // same one-shot worker — ThreadPool reuse strands JVM state.
+            // same one-shot worker. ThreadPool reuse strands JVM state.
             var thread = new Thread(() => FetchOnWorkerThread(persistentDataPath))
             {
                 IsBackground = true,