feat: Add app_memory to Context.App (#2398)

Author: bitsandfoxes

CHANGELOG.md

@@ -2,7 +2,11 @@
 
 ## Unreleased
 
-### Fixes
+### Features
+
+- The SDK now also reports the currently allocated memory when reporting an event or transaction. ([#2398](https://github.com/getsentry/sentry-unity/pull/2398))
+
+### Fixes 
 
 - The SDK now correctly sets the currently active scene's name on the event ([#2400](https://github.com/getsentry/sentry-unity/pull/2400))
 - Fixed an issue where screenshot capture triggered on a burst job would crash the game. The SDK can now also capture screenshots on events that occur outside of the main thread ([#2392](https://github.com/getsentry/sentry-unity/pull/2392))

src/Sentry.Unity/UnityEventProcessor.cs

@@ -43,6 +43,7 @@ private void SetEventContext(IEventLike sentryEvent)
     {
         try
         {
+            PopulateApp(sentryEvent.Contexts.App);
             PopulateDevice(sentryEvent.Contexts.Device);
 
             // The Unity context should get set in the UnityScopeIntegration automatically sets it when it gets registered
@@ -67,6 +68,21 @@ private void SetEventContext(IEventLike sentryEvent)
         }
     }
 
+    private void PopulateApp(App app)
+    {
+        if (!MainThreadData.IsMainThread())
+        {
+            return;
+        }
+
+        // The Profiler returns '0' if it is not available
+        var totalAllocatedMemory = UnityEngine.Profiling.Profiler.GetTotalAllocatedMemoryLong();
+        if (totalAllocatedMemory > 0)
+        {
+            app.Memory = totalAllocatedMemory;
+        }
+    }
+
     private void PopulateDevice(Device device)
     {
         if (!MainThreadData.IsMainThread())

test/Sentry.Unity.Tests/UnityEventScopeTests.cs

@@ -156,6 +156,10 @@ public void SentrySdkCaptureEvent(bool captureOnUiThread)
         Assert.AreEqual(systemInfo.DeviceModel!.Value, @event.Contexts.Device.Model);
         Assert.AreEqual(systemInfo.DeviceUniqueIdentifier!.Value, @event.Contexts.Device.DeviceUniqueIdentifier);
         Assert.AreEqual(systemInfo.IsDebugBuild!.Value ? "debug" : "release", @event.Contexts.App.BuildType);
+        if (captureOnUiThread)
+        {
+            Assert.IsNotNull(@event.Contexts.App.Memory);
+        }
 
         @event.Contexts.TryGetValue(Unity.Protocol.Unity.Type, out var unityProtocolObject);
         var unityContext = unityProtocolObject as Unity.Protocol.Unity;
@@ -276,6 +280,34 @@ public void AppProtocol_Assigned()
         Assert.IsNotNull(scope.Contexts.App.BuildType);
     }
 
+    [Test]
+    public void AppMemory_SetByEventProcessor()
+    {
+        // arrange
+        var unityEventProcessor = new UnityEventProcessor(_sentryOptions);
+        var sentryEvent = new SentryEvent();
+
+        // act
+        unityEventProcessor.Process(sentryEvent);
+
+        // assert
+        Assert.IsNotNull(sentryEvent.Contexts.App.Memory);
+    }
+
+    [Test]
+    public void AppMemory_SetByEventProcessorForTransactions()
+    {
+        // arrange
+        var unityEventProcessor = new UnityEventProcessor(_sentryOptions);
+        var transaction = new SentryTransaction("test-transaction", "test-operation");
+
+        // act
+        unityEventProcessor.Process(transaction);
+
+        // assert
+        Assert.IsNotNull(transaction.Contexts.App.Memory);
+    }
+
     [Test]
     public void AppProtocol_AppNameIsApplicationName()
     {