fix: PerformanceAutoInstrumentation shouldn't load scripts unless bui…

…lding
getsentry · Nov 7, 2022 · 4468d5a · 4468d5a
1 parent 10b4c6a
commit 4468d5a
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/Sentry.Unity.Editor/AutoInstrumentation/PerformanceAutoInstrumentation.cs b/src/Sentry.Unity.Editor/AutoInstrumentation/PerformanceAutoInstrumentation.cs
@@ -20,11 +20,15 @@ public static class PerformanceAutoInstrumentation
         static PerformanceAutoInstrumentation()
         {
             var sentryUnityAssemblyPath = Path.GetFullPath(Path.Combine("Packages", SentryPackageInfo.GetName(), "Runtime", "Sentry.Unity.dll"));
-            var (options, cliOptions) = SentryScriptableObject.ConfiguredBuildtimeOptions();
-
             CompilationPipeline.assemblyCompilationFinished += (assemblyPath, _) =>
             {
-                if (!BuildPipeline.isBuildingPlayer || options == null)
+                if (!BuildPipeline.isBuildingPlayer)
+                {
+                    return;
+                }
+
+                var (options, cliOptions) = SentryScriptableObject.ConfiguredBuildtimeOptions();
+                if (options == null)
                 {
                     return;
                 }