From 063bf35df41dce9830016656b41c72a06a1cc9cf Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Fri, 12 Jan 2024 16:15:55 -0900
Subject: [PATCH 01/34] test: integration test for app launch profiling

---
 .../xcshareddata/xcschemes/iOS-Swift.xcscheme |  14 +-
 .../xcschemes/iOS-SwiftUITests.xcscheme       |   2 +-
 Samples/iOS-Swift/iOS-Swift/AppDelegate.swift |  26 +-
 .../iOS-Swift/Base.lproj/Main.storyboard      |  61 +++-
 .../Profiling/ProfilingViewController.swift   |  97 ++++--
 .../TransactionsViewController.swift          |  37 +--
 .../iOS-SwiftUITests/BaseUITest.swift         |  32 +-
 .../iOS-SwiftUITests/ProfilingUITests.swift   | 287 ++++++++++++++++--
 Sentry.xcodeproj/project.pbxproj              |   4 +-
 Sources/Sentry/SentryProfiler.mm              |  67 +++-
 Sources/Sentry/SentryTracer.m                 |   5 +
 ...tProperties.h => SentryFileManager+Test.h} |   1 +
 .../SentryTests/SentryTests-Bridging-Header.h |   2 +-
 13 files changed, 530 insertions(+), 105 deletions(-)
 rename Tests/SentryTests/Helper/{SentryFileManager+TestProperties.h => SentryFileManager+Test.h} (86%)

diff --git a/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme b/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
index fa07f21b62..72e340ada0 100644
--- a/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
+++ b/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
@@ -23,7 +23,7 @@
       </BuildActionEntries>
    </BuildAction>
    <TestAction
-      buildConfiguration = "Debug"
+      buildConfiguration = "Test"
       selectedDebuggerIdentifier = "Xcode.DebuggerFoundation.Debugger.LLDB"
       selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB"
       shouldUseLaunchSchemeArgsEnv = "YES">
@@ -73,6 +73,18 @@
             argument = "--disable-file-io-tracing"
             isEnabled = "NO">
          </CommandLineArgument>
+         <CommandLineArgument
+            argument = "--disable-watchdog-tracking"
+            isEnabled = "NO">
+         </CommandLineArgument>
+         <CommandLineArgument
+            argument = "--disable-tracing"
+            isEnabled = "NO">
+         </CommandLineArgument>
+         <CommandLineArgument
+            argument = "--io.sentry.test.profilesAppLaunches"
+            isEnabled = "NO">
+         </CommandLineArgument>
          <CommandLineArgument
             argument = "--disable-crash-handler"
             isEnabled = "NO">
diff --git a/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-SwiftUITests.xcscheme b/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-SwiftUITests.xcscheme
index 55e1974867..37a4ca1e81 100644
--- a/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-SwiftUITests.xcscheme
+++ b/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-SwiftUITests.xcscheme
@@ -25,7 +25,7 @@
       </Testables>
    </TestAction>
    <LaunchAction
-      buildConfiguration = "Debug"
+      buildConfiguration = "Test"
       selectedDebuggerIdentifier = "Xcode.DebuggerFoundation.Debugger.LLDB"
       selectedLauncherIdentifier = "Xcode.DebuggerFoundation.Launcher.LLDB"
       launchStyle = "0"
diff --git a/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift b/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift
index 9379c6e4f8..3b6c30fe4a 100644
--- a/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift
+++ b/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift
@@ -34,7 +34,7 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
             }
             options.tracesSampleRate = tracesSampleRate
             
-            if let tracesSamplerValue = env["--io.sentry.tracersSamplerValue"] {
+            if let tracesSamplerValue = env["--io.sentry.tracesSamplerValue"] {
                 options.tracesSampler = { _ in
                     return NSNumber(value: (tracesSamplerValue as NSString).integerValue)
                 }
@@ -51,7 +51,9 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
                     return NSNumber(value: (profilesSamplerValue as NSString).integerValue)
                 }
             }
-                        
+
+            options.profileAppLaunches = args.contains("--profile-app-launches")
+
             options.sessionTrackingIntervalMillis = 5_000
             options.attachScreenshot = true
             options.attachViewHierarchy = true
@@ -81,9 +83,11 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
             options.enableNetworkBreadcrumbs = !args.contains("--disable-network-breadcrumbs")
             options.enableSwizzling = !args.contains("--disable-swizzling")
             options.enableCrashHandler = !args.contains("--disable-crash-handler")
+            options.enableTracing = !args.contains("--disable-tracing")
 
             // because we run CPU for 15 seconds at full throttle, we trigger ANR issues being sent. disable such during benchmarks.
             options.enableAppHangTracking = !isBenchmarking && !args.contains("--disable-anr-tracking")
+            options.enableWatchdogTerminationTracking = !isBenchmarking && !args.contains("--disable-watchdog-tracking")
             options.appHangTimeoutInterval = 2
             options.enableCaptureFailedRequests = true
             let httpStatusCodeRange = HttpStatusCodeRange(min: 400, max: 599)
@@ -96,7 +100,7 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
             }
             
             options.initialScope = { scope in
-                let processInfoEnvironment = ProcessInfo.processInfo.environment["io.sentry.sdk-environment"]
+                let processInfoEnvironment = env["io.sentry.sdk-environment"]
                 
                 if processInfoEnvironment != nil {
                     scope.setEnvironment(processInfoEnvironment)
@@ -129,7 +133,6 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
         print("[iOS-Swift] launch arguments: \(ProcessInfo.processInfo.arguments)")
         print("[iOS-Swift] environment: \(ProcessInfo.processInfo.environment)")
         
-        maybeWipeData()
         AppDelegate.startSentry()
         
         if #available(iOS 15.0, *) {
@@ -159,18 +162,3 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
         // swiftlint:enable force_cast
     }
 }
-
-private extension AppDelegate {
-    func maybeWipeData() {
-        if ProcessInfo.processInfo.arguments.contains("--io.sentry.wipe-data") {
-            print("[iOS-Swift] removing app data")
-            let appSupport = NSSearchPathForDirectoriesInDomains(.applicationSupportDirectory, .userDomainMask, true).first!
-            let cache = NSSearchPathForDirectoriesInDomains(.cachesDirectory, .userDomainMask, true).first!
-            for path in [appSupport, cache] {
-                for item in FileManager.default.enumerator(atPath: path)! {
-                    try! FileManager.default.removeItem(atPath: (path as NSString).appendingPathComponent((item as! String)))
-                }
-            }
-        }
-    }
-}
diff --git a/Samples/iOS-Swift/iOS-Swift/Base.lproj/Main.storyboard b/Samples/iOS-Swift/iOS-Swift/Base.lproj/Main.storyboard
index 094218d85b..749845d15d 100644
--- a/Samples/iOS-Swift/iOS-Swift/Base.lproj/Main.storyboard
+++ b/Samples/iOS-Swift/iOS-Swift/Base.lproj/Main.storyboard
@@ -1,9 +1,9 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<document type="com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB" version="3.0" toolsVersion="22154" targetRuntime="iOS.CocoaTouch" propertyAccessControl="none" useAutolayout="YES" useTraitCollections="YES" useSafeAreas="YES" colorMatched="YES" initialViewController="5CD-RQ-aBU">
+<document type="com.apple.InterfaceBuilder3.CocoaTouch.Storyboard.XIB" version="3.0" toolsVersion="22505" targetRuntime="iOS.CocoaTouch" propertyAccessControl="none" useAutolayout="YES" useTraitCollections="YES" useSafeAreas="YES" colorMatched="YES" initialViewController="5CD-RQ-aBU">
     <device id="retina4_0" orientation="portrait" appearance="light"/>
     <dependencies>
         <deployment identifier="iOS"/>
-        <plugIn identifier="com.apple.InterfaceBuilder.IBCocoaTouchPlugin" version="22130"/>
+        <plugIn identifier="com.apple.InterfaceBuilder.IBCocoaTouchPlugin" version="22504"/>
         <capability name="Image references" minToolsVersion="12.0"/>
         <capability name="Safe area layout guides" minToolsVersion="9.0"/>
         <capability name="System colors in document resources" minToolsVersion="11.0"/>
@@ -172,7 +172,7 @@
                         <autoresizingMask key="autoresizingMask" widthSizable="YES" heightSizable="YES"/>
                         <subviews>
                             <stackView opaque="NO" contentMode="scaleToFill" axis="vertical" alignment="center" translatesAutoresizingMaskIntoConstraints="NO" id="jya-5z-xKN">
-                                <rect key="frame" x="0.0" y="64" width="320" height="56"/>
+                                <rect key="frame" x="0.0" y="64" width="320" height="190"/>
                                 <subviews>
                                     <stackView opaque="NO" contentMode="scaleToFill" alignment="center" translatesAutoresizingMaskIntoConstraints="NO" id="DOu-47-e6X">
                                         <rect key="frame" x="93" y="0.0" width="134" height="28"/>
@@ -223,8 +223,57 @@
                                             </button>
                                         </subviews>
                                     </stackView>
+                                    <button opaque="NO" contentMode="scaleToFill" contentHorizontalAlignment="center" contentVerticalAlignment="center" buttonType="system" lineBreakMode="middleTruncation" translatesAutoresizingMaskIntoConstraints="NO" id="Pcb-L7-DFD">
+                                        <rect key="frame" x="102.5" y="56" width="115" height="28"/>
+                                        <fontDescription key="fontDescription" type="system" pointSize="13"/>
+                                        <inset key="imageEdgeInsets" minX="0.0" minY="0.0" maxX="2.2250738585072014e-308" maxY="0.0"/>
+                                        <state key="normal" title="View launch config"/>
+                                        <connections>
+                                            <action selector="viewLaunchConfig:" destination="NZr-bH-g9o" eventType="touchUpInside" id="dq1-Ng-UNd"/>
+                                        </connections>
+                                    </button>
+                                    <button opaque="NO" contentMode="scaleToFill" contentHorizontalAlignment="center" contentVerticalAlignment="center" buttonType="system" lineBreakMode="middleTruncation" translatesAutoresizingMaskIntoConstraints="NO" id="vwa-d9-1BM">
+                                        <rect key="frame" x="111.5" y="84" width="97" height="28"/>
+                                        <fontDescription key="fontDescription" type="system" pointSize="13"/>
+                                        <inset key="imageEdgeInsets" minX="0.0" minY="0.0" maxX="2.2250738585072014e-308" maxY="0.0"/>
+                                        <state key="normal" title="View last profile"/>
+                                        <connections>
+                                            <action selector="viewLastProfile:" destination="NZr-bH-g9o" eventType="touchUpInside" id="7Xd-19-rsP"/>
+                                        </connections>
+                                    </button>
+                                    <button opaque="NO" contentMode="scaleToFill" contentHorizontalAlignment="center" contentVerticalAlignment="center" buttonType="system" lineBreakMode="middleTruncation" translatesAutoresizingMaskIntoConstraints="NO" id="F1h-qu-BzX">
+                                        <rect key="frame" x="102.5" y="112" width="115" height="28"/>
+                                        <fontDescription key="fontDescription" type="system" pointSize="13"/>
+                                        <inset key="imageEdgeInsets" minX="0.0" minY="0.0" maxX="2.2250738585072014e-308" maxY="0.0"/>
+                                        <state key="normal" title="View launch profile"/>
+                                        <connections>
+                                            <action selector="viewLaunchProfile:" destination="NZr-bH-g9o" eventType="touchUpInside" id="DPn-8C-b7l"/>
+                                        </connections>
+                                    </button>
+                                    <stackView opaque="NO" contentMode="scaleToFill" alignment="center" spacing="8" translatesAutoresizingMaskIntoConstraints="NO" id="LIr-jM-FSX">
+                                        <rect key="frame" x="19" y="140" width="282" height="50"/>
+                                        <subviews>
+                                            <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="UI test data marshalling:" textAlignment="natural" lineBreakMode="tailTruncation" baselineAdjustment="alignBaselines" adjustsFontSizeToFit="NO" translatesAutoresizingMaskIntoConstraints="NO" id="Uhg-B3-Rao" userLabel="UI test data marshaling:">
+                                                <rect key="frame" x="0.0" y="15" width="184" height="20.5"/>
+                                                <fontDescription key="fontDescription" type="system" pointSize="17"/>
+                                                <nil key="textColor"/>
+                                                <nil key="highlightedColor"/>
+                                            </label>
+                                            <textField opaque="NO" contentMode="scaleToFill" horizontalHuggingPriority="248" contentHorizontalAlignment="left" contentVerticalAlignment="center" borderStyle="roundedRect" textAlignment="natural" minimumFontSize="17" translatesAutoresizingMaskIntoConstraints="NO" id="5zI-Ix-rMJ">
+                                                <rect key="frame" x="192" y="8" width="32" height="34"/>
+                                                <fontDescription key="fontDescription" type="system" pointSize="14"/>
+                                                <textInputTraits key="textInputTraits"/>
+                                            </textField>
+                                            <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="" textAlignment="natural" lineBreakMode="tailTruncation" baselineAdjustment="alignBaselines" adjustsFontSizeToFit="NO" translatesAutoresizingMaskIntoConstraints="NO" id="2LR-W8-mNE" userLabel="Profiling UI Test Data Marshaling status">
+                                                <rect key="frame" x="232" y="0.0" width="50" height="50"/>
+                                                <fontDescription key="fontDescription" type="system" pointSize="17"/>
+                                                <nil key="textColor"/>
+                                                <nil key="highlightedColor"/>
+                                            </label>
+                                        </subviews>
+                                    </stackView>
                                     <textField hidden="YES" opaque="NO" contentMode="scaleToFill" horizontalHuggingPriority="248" contentHorizontalAlignment="left" contentVerticalAlignment="center" borderStyle="roundedRect" textAlignment="natural" minimumFontSize="17" translatesAutoresizingMaskIntoConstraints="NO" id="uTM-T7-Gv6">
-                                        <rect key="frame" x="0.0" y="56" width="320" height="0.0"/>
+                                        <rect key="frame" x="0.0" y="190" width="320" height="0.0"/>
                                         <accessibility key="accessibilityConfiguration" identifier="io.sentry.benchmark.value-marshaling-text-field"/>
                                         <fontDescription key="fontDescription" type="system" pointSize="14"/>
                                         <textInputTraits key="textInputTraits"/>
@@ -232,7 +281,7 @@
                                 </subviews>
                             </stackView>
                             <stackView opaque="NO" contentMode="scaleToFill" axis="vertical" alignment="center" translatesAutoresizingMaskIntoConstraints="NO" id="hFG-bX-qLp">
-                                <rect key="frame" x="10" y="174" width="300" height="159.5"/>
+                                <rect key="frame" x="10" y="308" width="300" height="159.5"/>
                                 <subviews>
                                     <label opaque="NO" userInteractionEnabled="NO" contentMode="left" horizontalHuggingPriority="251" verticalHuggingPriority="251" text="Work threads:" textAlignment="natural" lineBreakMode="tailTruncation" baselineAdjustment="alignBaselines" adjustsFontSizeToFit="NO" translatesAutoresizingMaskIntoConstraints="NO" id="t2C-jL-6qO">
                                         <rect key="frame" x="97" y="0.0" width="106.5" height="20.5"/>
@@ -346,6 +395,8 @@
                         <outlet property="maxWorkIntensityTextField" destination="rJh-QS-fAl" id="u9q-Sj-6SY"/>
                         <outlet property="minThreadsTextField" destination="sCM-tI-skI" id="D0Z-yT-LbU"/>
                         <outlet property="minWorkIntensityTextField" destination="CuN-wx-fvQ" id="Eik-Z2-7M7"/>
+                        <outlet property="profilingUITestDataMarshalingStatus" destination="2LR-W8-mNE" id="S4k-Yg-EsB"/>
+                        <outlet property="profilingUITestDataMarshalingTextField" destination="5zI-Ix-rMJ" id="2o6-Cc-NMl"/>
                         <outlet property="valueTextField" destination="uTM-T7-Gv6" id="M59-lx-juE"/>
                         <outlet property="workIntensityFactorLabel" destination="cAO-gr-3HC" id="BvB-NZ-TXI"/>
                         <outlet property="workIntervalSlider" destination="N8m-ny-upJ" id="3MU-ZK-5qD"/>
diff --git a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
index bf8e2745a5..7d462cd5a8 100644
--- a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
+++ b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
@@ -1,20 +1,23 @@
 import UIKit
 
-@available(iOS 13.0, *)
+@available(iOS 16.0, *)
 class ProfilingViewController: UIViewController, UITextFieldDelegate {
-
+    
     @IBOutlet weak var workThreadLabel: UILabel!
     @IBOutlet weak var workIntensityFactorLabel: UILabel!
-
+    
     @IBOutlet weak var workThreadSlider: UISlider!
     @IBOutlet weak var workIntervalSlider: UISlider!
-
+    
     @IBOutlet weak var maxThreadsTextField: UITextField!
     @IBOutlet weak var minThreadsTextField: UITextField!
     @IBOutlet weak var minWorkIntensityTextField: UITextField!
     @IBOutlet weak var maxWorkIntensityTextField: UITextField!
     @IBOutlet weak var valueTextField: UITextField!
-
+    
+    @IBOutlet weak var profilingUITestDataMarshalingTextField: UITextField!
+    @IBOutlet weak var profilingUITestDataMarshalingStatus: UILabel!
+    
     override func viewDidLoad() {
         super.viewDidLoad()
         minWorkIntensityTextField.text = String(defaultLongestIntervalMicros)
@@ -24,73 +27,125 @@ class ProfilingViewController: UIViewController, UITextFieldDelegate {
         [maxThreadsTextField, minThreadsTextField, minWorkIntensityTextField, maxWorkIntensityTextField].forEach {
             $0?.delegate = self
         }
+        profilingUITestDataMarshalingTextField.accessibilityLabel = "io.sentry.ui-tests.profile-marshaling-text-field"
     }
     
     override func viewDidAppear(_ animated: Bool) {
         super.viewDidAppear(animated)
         SentrySDK.reportFullyDisplayed()
     }
-
+    
     @IBAction func startBenchmark(_ sender: UIButton) {
         highlightButton(sender)
         SentryBenchmarking.startBenchmark()
     }
-
+    
     @IBAction func stopBenchmark(_ sender: UIButton) {
         highlightButton(sender)
         let value = SentryBenchmarking.stopBenchmark()!
         valueTextField.isHidden = false
         valueTextField.text = value
-        print("[iOS-Swift] benchmarking results:\n\(value)")
+        print("[iOS-Swift] [Profiling] benchmarking results:\n\(value)")
     }
-
+    
     @IBAction func startCPUWork(_ sender: UIButton) {
         highlightButton(sender)
         _adjustWorkThreadsToCurrentRequirement()
     }
-
+    
     @IBAction func minWorkThreadCountChanged(_ sender: Any) {
         _adjustWorkThreadsToCurrentRequirement()
     }
-
+    
     @IBAction func workThreadSliderChanged(_ sender: Any) {
         _adjustWorkThreadsToCurrentRequirement()
     }
-
+    
     @IBAction func maxWorkThreadCountChanged(_ sender: Any) {
         _adjustWorkThreadsToCurrentRequirement()
     }
-
+    
     @IBAction func endCPUWork(_ sender: UIButton) {
         highlightButton(sender)
         cpuWorkthreads.forEach { $0.cancel() }
         cpuWorkthreads.removeAll()
     }
-
+    
     @IBAction func minWorkIntervalChanged(_ sender: Any) {
         _adjustWorkIntervalToCurrentRequirements()
     }
-
+    
     @IBAction func workIntensityChanged(_ sender: UISlider) {
         _adjustWorkIntervalToCurrentRequirements()
     }
-
+    
     @IBAction func maxWorkIntervalChanged(_ sender: Any) {
         _adjustWorkIntervalToCurrentRequirements()
     }
-
+    
     @IBAction func bgBrightnessChanged(_ sender: UISlider) {
         view.backgroundColor = .init(white: CGFloat(sender.value), alpha: 1)
     }
-
+    
+    @IBAction func viewLaunchConfig(_ sender: Any) {
+        let launchConfigPath = ((NSSearchPathForDirectoriesInDomains(.applicationSupportDirectory, .userDomainMask, true).first! as NSString).appendingPathComponent("io.sentry") as NSString).appendingPathComponent("launchConfig")
+        let url = URL(fileURLWithPath: launchConfigPath)
+        handleContents(file: url)
+    }
+    
+    @IBAction func viewLastProfile(_ sender: Any) {
+        profilingUITestDataMarshalingTextField.text = "<fetching...>"
+        withProfile(fileName: "profile") { file in
+            handleContents(file: file)
+        }
+    }
+    
+    @IBAction func viewLaunchProfile(_ sender: Any) {
+        profilingUITestDataMarshalingTextField.text = "<fetching...>"
+        withProfile(fileName: "launchProfile") { file in
+            handleContents(file: file)
+        }
+    }
+        
     // MARK: UITextFieldDelegate
-
+    
     func textFieldShouldReturn(_ textField: UITextField) -> Bool {
         textField.resignFirstResponder()
     }
+}
 
-    // MARK: Private
-
+// MARK: Private
+@available(iOS 16.0, *)
+extension ProfilingViewController {
+    func withProfile(fileName: String, block: (URL?) -> Void) {
+        let appSupportDirectory = NSSearchPathForDirectoriesInDomains(.applicationSupportDirectory, .userDomainMask, true).first!
+        for file in FileManager.default.enumerator(at: URL(filePath: appSupportDirectory), includingPropertiesForKeys: [URLResourceKey.nameKey])! {
+            let url = file as! URL
+            if url.absoluteString.contains(fileName) {
+                block(url)
+                return
+            }
+        }
+        block(nil)
+    }
+    
+    func handleContents(file: URL?) {
+        guard let file = file else {
+            profilingUITestDataMarshalingTextField.text = "<missing>"
+            profilingUITestDataMarshalingStatus.text = "❌"
+            return
+        }
+        guard let data = try? Data(contentsOf: file) else {
+            profilingUITestDataMarshalingTextField.text = "<empty>"
+            profilingUITestDataMarshalingStatus.text = "❌"
+            return
+        }
+        let contents = data.base64EncodedString()
+        print("[iOS-Swift] [Profiling] contents of file at \(file): \(String(data: data, encoding: .utf8))")
+        profilingUITestDataMarshalingTextField.text = contents
+        profilingUITestDataMarshalingStatus.text = "✅"
+    }
+    
     func _adjustWorkThreadsToCurrentRequirement() {
         let maxThreads = (maxThreadsTextField.text! as NSString).integerValue
         let minThreads = (minThreadsTextField.text! as NSString).integerValue
diff --git a/Samples/iOS-Swift/iOS-Swift/TransactionsViewController.swift b/Samples/iOS-Swift/iOS-Swift/TransactionsViewController.swift
index 65d9404966..9a2cb61fb6 100644
--- a/Samples/iOS-Swift/iOS-Swift/TransactionsViewController.swift
+++ b/Samples/iOS-Swift/iOS-Swift/TransactionsViewController.swift
@@ -61,31 +61,12 @@ class TransactionsViewController: UIViewController {
     }
 
     var spans = [Span]()
-    let profilerNotification = NSNotification.Name("SentryProfileCompleteNotification")
 
     @IBAction func startTransaction(_ sender: UIButton) {
         highlightButton(sender)
         startNewTransaction()
     }
 
-    fileprivate func startNewTransaction() {
-        spans.append(SentrySDK.startTransaction(name: "Manual Transaction", operation: "Manual Operation"))
-
-        NotificationCenter.default.addObserver(forName: profilerNotification, object: nil, queue: nil) { note in
-            DispatchQueue.main.async {
-                let alert = UIAlertController(title: "Profile completed", message: nil, preferredStyle: .alert)
-                alert.addTextField {
-                    //swiftlint:disable force_unwrapping
-                    $0.text = try! JSONSerialization.data(withJSONObject: note.userInfo!).base64EncodedString()
-                    //swiftlint:enable force_unwrapping
-                    $0.accessibilityLabel = "io.sentry.ui-tests.profile-marshaling-text-field"
-                }
-                alert.addAction(UIAlertAction(title: "OK", style: .default))
-                self.present(alert, animated: false)
-            }
-        }
-    }
-
     @IBAction func startTransactionFromOtherThread(_ sender: UIButton) {
         highlightButton(sender)
 
@@ -97,12 +78,6 @@ class TransactionsViewController: UIViewController {
     @IBAction func stopTransaction(_ sender: UIButton) {
         highlightButton(sender)
 
-        defer {
-            if spans.isEmpty {
-                NotificationCenter.default.removeObserver(self, name: profilerNotification, object: nil)
-            }
-        }
-
         func showConfirmation(span: Span) {
             DispatchQueue.main.async {
                 let confirmation = UIAlertController(title: "Finished span \(span.spanId.sentrySpanIdString)", message: nil, preferredStyle: .alert)
@@ -118,6 +93,12 @@ class TransactionsViewController: UIViewController {
             })!)
             showConfirmation(span: span)
         }
+        
+        if spans.isEmpty {
+            if let launchSpan = SentrySDK.span {
+                spans.append(launchSpan)
+            }
+        }
 
         if spans.count == 1 {
             finishSpan(span: spans[0])
@@ -197,3 +178,9 @@ class TransactionsViewController: UIViewController {
         navigationController?.pushViewController(controller, animated: false)
     }
 }
+
+fileprivate extension TransactionsViewController {
+    func startNewTransaction() {
+        spans.append(SentrySDK.startTransaction(name: "Manual Transaction", operation: "Manual Operation"))
+    }
+}
diff --git a/Samples/iOS-Swift/iOS-SwiftUITests/BaseUITest.swift b/Samples/iOS-Swift/iOS-SwiftUITests/BaseUITest.swift
index 94914d37c7..f80cd4373c 100644
--- a/Samples/iOS-Swift/iOS-SwiftUITests/BaseUITest.swift
+++ b/Samples/iOS-Swift/iOS-SwiftUITests/BaseUITest.swift
@@ -1,26 +1,44 @@
 import XCTest
 
 class BaseUITest: XCTestCase {
-    internal let app: XCUIApplication = XCUIApplication()
+    internal lazy var app: XCUIApplication = newAppSession()
+    
+    //swiftlint:disable implicit_getter
+    var automaticallyManageAppSession: Bool { get { true } }
+    //swiftlint:enable implicit_getter
     
     override func setUp() {
         super.setUp()
         continueAfterFailure = false
         XCUIDevice.shared.orientation = .portrait
-        app.launchEnvironment["io.sentry.ui-test.test-name"] = name
         app.launchEnvironment["io.sentry.sdk-environment"] = "ui-tests"
-        app.launch()
-        
-        waitForExistenceOfMainScreen()
+        if automaticallyManageAppSession {
+            launchApp()
+        }
     }
     
     override func tearDown() {
-        app.terminate()
+        if automaticallyManageAppSession {
+            app.terminate()
+        }
         super.tearDown()
     }
+}
+
+extension BaseUITest {
+    func newAppSession() -> XCUIApplication {
+        let app = XCUIApplication()
+        app.launchEnvironment["io.sentry.ui-test.test-name"] = name
+        return app
+    }
+    
+    func launchApp() {
+        app.launch()
+        waitForExistenceOfMainScreen()
+    }
     
     func waitForExistenceOfMainScreen() {
-        app.waitForExistence( "Home Screen doesn't exist.")
+        app.waitForExistence("Home Screen doesn't exist.")
     }
     
 }
diff --git a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
index 3089f4ab7f..418e3111a6 100644
--- a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
+++ b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
@@ -1,39 +1,286 @@
 import XCTest
 
-final class ProfilingUITests: BaseUITest {
+//swiftlint:disable function_body_length todo
 
+class ProfilingUITests: BaseUITest {    
+    override var automaticallyManageAppSession: Bool { false }
+    
+    override func setUp() async throws {
+        try await super.setUp()
+        
+        guard shouldRunProfilingUITest() else {
+           throw XCTSkip("iOS version too old for profiling test.")
+        }
+        
+        app.launchArguments.append(contentsOf: [
+            "--io.sentry.wipe-data" // wipe all the profiles stored to disk, and any launch profiling config files, before running each test
+        ]
+        )
+    }
+    
+    func testProfiledAppLaunches() throws {
+        launchApp()
+        
+        // First launch enables in-app profiling by setting traces/profiles sample rates to 1 (which is the default configuration in the sample app), but not launch profiling; assert that we did not write a config to allow the next launch to be profiled
+        try performAssertions(shouldProfileThisLaunch: false, shouldProfileNextLaunch: false)
+        
+        // no profiling should be done on this launch; set the option to allow launch profiling for the next launch, keeping the default numerical sampling rates of 1 for traces and profiles
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true)
+        
+        // this launch should run the profiler, then set the option to allow launch profiling to true, but set the numerical sample rates to 0 so that the next launch should not profile
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, profilesSampleRate: 0, tracesSampleRate: 0)
+        
+        // this launch should not run the profiler; configure sampler functions returning 1 and numerical rates set to 0, which should result in a profile being taken as samplers override numerical rates
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, profilesSampleRate: 0, tracesSampleRate: 0, profilesSamplerValue: 1, tracesSamplerValue: 1)
+        
+        // this launch has the configuration to run the profiler, but because swizzling is disabled, it will not run due to the ui.load transaction not being allowed to be created but configure it not to run the next launch due to disabling swizzling, which would override the option to enable launch profiling
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, shouldDisableSwizzling: true)
+        
+        // this launch should not run the profiler and configure it not to run the next launch due to disabling automatic performance tracking, which would override the option to enable launch profiling
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, shouldDisableAutoPerformanceTracking: true)
+        
+        // this launch should not run the profiler and configure it not to run the next launch launch due to disabling UIViewController tracing, which would override the option to enable launch profiling
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, shouldDisableUIViewControllerTracing: true)
+        
+        // this launch should not run the profiler and configure it not to run the next launch launch due to disabling tracing, which would override the option to enable launch profiling
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, shouldDisableTracing: true)
+        
+        // make sure the profiler respects the last configuration not to run
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true)
+    }
+    
     /**
      * We had a bug where we forgot to install the frames tracker into the profiler, so weren't sending any GPU frame information with profiles. Since it's not possible to enforce such installation via the compiler, we test for the results we expect here, by starting a transaction, triggering an ANR which will cause degraded frame rendering, stop the transaction, and inspect the profile payload.
      */
     func testProfilingGPUInfo() throws {
-        // We don't need to test this on multiple OSes right now, and older versions seem to have issues; older devices or VM images running simulators might just be slower. Latest OS is enough coverage for our needs for now.
-        guard #available(iOS 15.0, *) else {
-           throw XCTSkip("Only run on iOS 15 and above.")
-        }
-
-        app.tabBars["Tab Bar"].buttons["Extra"].tap()
-        app.tabBars["Tab Bar"].buttons["Transactions"].tap()
-        app.buttons["Start transaction (main thread)"].afterWaitingForExistence("Couldn't find button to start transaction").tap()
-
+        app.launchArguments.append("--disable-swizzling") // we're only interested in the manual transaction, the automatic stuff messes up how we try to retrieve the target profile info
+        launchApp()
+        
+        goToTransactions()
+        startTransaction()
+        
         app.buttons["ANR filling run loop"].afterWaitingForExistence("Couldn't find button to ANR").tap()
-        app.buttons["Stop transaction"].afterWaitingForExistence("Couldn't find button to end transaction").tap()
-
-        let textField = app.textFields["io.sentry.ui-tests.profile-marshaling-text-field"]
-        textField.waitForExistence("Couldn't find profile marshaling text field.")
-
-        let profileBase64DataString = try XCTUnwrap(textField.value as? NSString)
-        let profileData = try XCTUnwrap(Data(base64Encoded: profileBase64DataString as String))
-        let profileDict = try XCTUnwrap(try JSONSerialization.jsonObject(with: profileData) as? [String: Any])
-
+        stopTransaction()
+    
+        goToProfiling()
+        retrieveLastProfileData()
+        let profileDict = try marshalJSONDictionaryFromApp()
+        
         let metrics = try XCTUnwrap(profileDict["measurements"] as? [String: Any])
         // We can only be sure about frozen frames when triggering an ANR.
         // It could be that there is no slow frame for the captured transaction.
         let frozenFrames = try XCTUnwrap(metrics["frozen_frame_renders"] as? [String: Any])
         let frozenFrameValues = try XCTUnwrap(frozenFrames["values"] as? [[String: Any]])
         XCTAssertFalse(frozenFrameValues.isEmpty, "The test triggered an ANR while the transaction is running. There must be at least one frozen frame, but there was none.")
-
+        
         let frameRates = try XCTUnwrap(metrics["screen_frame_rates"] as? [String: Any])
         let frameRateValues = try XCTUnwrap(frameRates["values"] as? [[String: Any]])
         XCTAssertFalse(frameRateValues.isEmpty)
     }
 }
+
+extension ProfilingUITests {
+    func shouldRunProfilingUITest() -> Bool {
+        if #available(iOS 16.0, *) {
+            return true
+        } else {
+            return false
+        }
+    }
+    
+    enum Error: Swift.Error {
+        case missingFile
+        case emptyFile
+    }
+    
+    func marshalJSONDictionaryFromApp() throws -> [String: Any] {
+        let string = try XCTUnwrap(app.textFields["io.sentry.ui-tests.profile-marshaling-text-field"].afterWaitingForExistence("Couldn't find data marshaling text field.").value as? NSString)
+        if string == "<missing>" {
+            throw Error.missingFile
+        }
+        if string == "<empty>" {
+            throw Error.emptyFile
+        }
+        let data = try XCTUnwrap(Data(base64Encoded: string as String))
+        return try XCTUnwrap(try JSONSerialization.jsonObject(with: data) as? [String: Any])
+    }
+    
+    func goToTransactions() {
+        app.tabBars["Tab Bar"].buttons["Transactions"].tap()
+    }
+    
+    func startTransaction() {
+        app.buttons["Start transaction (main thread)"].afterWaitingForExistence("Couldn't find button to start transaction").tap()
+    }
+    
+    func stopTransaction() {
+        app.buttons["Stop transaction"].afterWaitingForExistence("Couldn't find button to end transaction").tap()
+    }
+    
+    func goToProfiling() {
+        app.tabBars["Tab Bar"].buttons["Profiling"].afterWaitingForExistence("Couldn't find profiling tab bar button").tap()
+    }
+    
+    func retrieveLaunchProfilingConfig() {
+        app.buttons["View launch config"].afterWaitingForExistence("Couldn't find button to view launch config").tap()
+    }
+    
+    func retrieveLastProfileData() {
+        app.buttons["View last profile"].afterWaitingForExistence("Couldn't find button to view last profile").tap()
+    }
+    
+    func retrieveLaunchProfileData() {
+        app.buttons["View launch profile"].afterWaitingForExistence("Couldn't find button to view launch profile").tap()
+    }
+
+    func assertLaunchProfile(ran: Bool) throws {
+        retrieveLaunchProfileData()
+        
+        var lastProfile: [String: Any]
+        do {
+            lastProfile = try marshalJSONDictionaryFromApp()
+        } catch {
+            guard ran else {
+                return
+            }
+            throw error
+        }
+        
+        let sampledProfile = try XCTUnwrap(lastProfile["profile"] as? [String: Any])
+        let stacks = try XCTUnwrap(sampledProfile["stacks"] as? [[Int]])
+        let frames = try XCTUnwrap(sampledProfile["frames"] as? [[String: Any]])
+        let functions = stacks.map({ stack in
+            stack.map { stackFrame in
+                frames[stackFrame]["function"]
+            }
+        })
+        let stack = try XCTUnwrap(functions.first { nextStack in
+            let result = nextStack.contains { frame in
+                let result = (frame as! String).contains("+[SentryProfiler(SlowLoad) load]")
+                return result
+            }
+            return result
+        }).map { any in
+            try XCTUnwrap(any as? String)
+        }
+        
+        XCTAssertFalse(stack.contains("main"))
+        XCTAssertFalse(stack.contains("UIApplicationMain"))
+        XCTAssertFalse(stack.contains("-[UIApplication _run]"))
+        
+        // ???: can we assert that this stack was on the main thread?
+        // TODO: yes! need to correlate the samples.[].stack_id with samples.[].thread_id
+    }
+    
+    func assertLaunchProfileConfig(expectedFile: Bool, expectedProfilesSampleRate: Int? = nil, expectedTracesSampleRate: Int? = nil) throws {
+        retrieveLaunchProfilingConfig()
+        var launchConfig: [String: Any]
+        
+        do {
+            launchConfig = try marshalJSONDictionaryFromApp()
+        } catch {
+            guard expectedFile else {
+                return
+            }
+            throw error
+        }
+        
+        if let expectedProfilesSampleRate = expectedProfilesSampleRate {
+            XCTAssertEqual(try XCTUnwrap(launchConfig["profilesSampleRate"] as? Int), expectedProfilesSampleRate)
+        }
+        if let expectedTracesSampleRate = expectedTracesSampleRate {
+            XCTAssertEqual(try XCTUnwrap(launchConfig["tracesSampleRate"] as? Int), expectedTracesSampleRate)
+        }
+    }
+     
+    /**
+     * Performs the various operations for the launch profiler test case:
+     * - terminates an existing app session
+     * - creates a new one
+     * - sets launch args and env vars to set the appropriate `SentryOption` values for the desired behavior
+     * - launches the new configured app session
+     * - asserts the expected outcomes of the config file and launch profiler
+     */
+    func relaunchAndConfigureSubsequentLaunches(
+        shouldProfileThisLaunch: Bool,
+        shouldEnableLaunchProfilingOptionForNextLaunch: Bool,
+        profilesSampleRate: Int? = nil,
+        tracesSampleRate: Int? = nil,
+        profilesSamplerValue: Int? = nil,
+        tracesSamplerValue: Int? = nil,
+        shouldDisableAutoPerformanceTracking: Bool = false,
+        shouldDisableUIViewControllerTracing: Bool = false,
+        shouldDisableSwizzling: Bool = false,
+        shouldDisableTracing: Bool = false
+    ) throws {
+        app.terminate()
+        app = newAppSession()
+                
+        if shouldProfileThisLaunch {
+            app.launchArguments.append("--io.sentry.slow-load-method")
+        }
+        if shouldEnableLaunchProfilingOptionForNextLaunch {
+            app.launchArguments.append("--profile-app-launches")
+        }
+        
+        var resolvedProfilesSampleRate: Int?
+        if let profilesSampleRate = profilesSampleRate {
+            app.launchEnvironment["--io.sentry.profilesSampleRate"] = String(profilesSampleRate)
+            resolvedProfilesSampleRate = profilesSampleRate
+        }
+        if let profilesSamplerValue = profilesSamplerValue {
+            app.launchEnvironment["--io.sentry.profilesSamplerValue"] = String(profilesSamplerValue)
+            resolvedProfilesSampleRate = profilesSamplerValue
+        }
+        
+        var resolvedTracesSampleRate: Int?
+        if let tracesSampleRate = tracesSampleRate {
+            app.launchEnvironment["--io.sentry.tracesSampleRate"] = String(tracesSampleRate)
+            resolvedTracesSampleRate = tracesSampleRate
+        }
+        if let tracesSamplerValue = tracesSamplerValue {
+            app.launchEnvironment["--io.sentry.tracesSamplerValue"] = String(tracesSamplerValue)
+            resolvedTracesSampleRate = tracesSamplerValue
+        }
+        
+        if shouldDisableTracing {
+            app.launchArguments.append("--disable-tracing")
+        }
+        if shouldDisableSwizzling {
+            app.launchArguments.append("--disable-swizzling")
+        }
+        if shouldDisableAutoPerformanceTracking {
+            app.launchArguments.append("--disable-auto-performance-tracing")
+        }
+        if shouldDisableUIViewControllerTracing {
+            app.launchArguments.append("--disable-uiviewcontroller-tracing")
+        }
+        
+        launchApp()
+        
+        try performAssertions(
+            shouldProfileThisLaunch: shouldProfileThisLaunch,
+            shouldProfileNextLaunch: shouldEnableLaunchProfilingOptionForNextLaunch && !(shouldDisableTracing || shouldDisableSwizzling || shouldDisableAutoPerformanceTracking || shouldDisableUIViewControllerTracing),
+            expectedProfilesSampleRate: resolvedProfilesSampleRate,
+            expectedTracesSampleRate: resolvedTracesSampleRate
+        )
+    }
+    
+    func performAssertions(shouldProfileThisLaunch: Bool,
+                           shouldProfileNextLaunch: Bool,
+                           expectedProfilesSampleRate: Int? = nil,
+                           expectedTracesSampleRate: Int? = nil
+    ) throws {
+        goToProfiling()
+        
+        if shouldProfileNextLaunch {
+            try assertLaunchProfileConfig(expectedFile: shouldProfileNextLaunch, expectedProfilesSampleRate: expectedProfilesSampleRate, expectedTracesSampleRate: expectedTracesSampleRate)
+        }
+        
+        if shouldProfileThisLaunch {
+            try assertLaunchProfile(ran: shouldProfileThisLaunch)
+        }
+    }
+}
+
+//swiftlint:enable function_body_length todo
diff --git a/Sentry.xcodeproj/project.pbxproj b/Sentry.xcodeproj/project.pbxproj
index 0072858825..51d068d24d 100644
--- a/Sentry.xcodeproj/project.pbxproj
+++ b/Sentry.xcodeproj/project.pbxproj
@@ -1277,7 +1277,7 @@
 		7B42C47F27E08F33009B58C2 /* SentryDependencyContainer.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = SentryDependencyContainer.h; path = include/HybridPublic/SentryDependencyContainer.h; sourceTree = "<group>"; };
 		7B42C48127E08F4B009B58C2 /* SentryDependencyContainer.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentryDependencyContainer.m; sourceTree = "<group>"; };
 		7B451483263ABAB7008AFFEB /* SentryDebugImageProvider+TestInit.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = "SentryDebugImageProvider+TestInit.h"; sourceTree = "<group>"; };
-		7B4C817124D1BC2B0076ACE4 /* SentryFileManager+TestProperties.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = "SentryFileManager+TestProperties.h"; sourceTree = "<group>"; };
+		7B4C817124D1BC2B0076ACE4 /* SentryFileManager+Test.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = "SentryFileManager+Test.h"; sourceTree = "<group>"; };
 		7B4D308926FC616B00C94DE9 /* SentryHttpTransportTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = SentryHttpTransportTests.swift; sourceTree = "<group>"; };
 		7B4E23AE2519E13800060D68 /* SentryCrashIntegration+TestInit.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = "SentryCrashIntegration+TestInit.h"; sourceTree = "<group>"; };
 		7B4E23B5251A07BD00060D68 /* SentryDispatchQueueWrapperTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = SentryDispatchQueueWrapperTests.swift; sourceTree = "<group>"; };
@@ -2855,7 +2855,7 @@
 				7BE3C7742445C82300A38442 /* SentryCurrentDateTests.swift */,
 				7BD729992463EA4A00EA3610 /* SentryDateUtilTests.swift */,
 				7B85BD8D24C5C3A6000A4225 /* SentryFileManagerTestExtension.swift */,
-				7B4C817124D1BC2B0076ACE4 /* SentryFileManager+TestProperties.h */,
+				7B4C817124D1BC2B0076ACE4 /* SentryFileManager+Test.h */,
 				7B98D7EB25FB7C4900C5A389 /* SentryAppStateTests.swift */,
 				0A94158128F6C4C2006A5DD1 /* SentryAppStateManagerTests.swift */,
 				69BEE6F62620729E006DF9DF /* UrlSessionDelegateSpy.swift */,
diff --git a/Sources/Sentry/SentryProfiler.mm b/Sources/Sentry/SentryProfiler.mm
index c27554a563..1c1fc90b4c 100644
--- a/Sources/Sentry/SentryProfiler.mm
+++ b/Sources/Sentry/SentryProfiler.mm
@@ -50,6 +50,29 @@
 #        import <UIKit/UIKit.h>
 #    endif // SENTRY_HAS_UIKIT
 
+#    if defined(TEST) || defined(TESTCI)
+#        import "SentryFileManager+Test.h"
+
+@interface
+SentryProfiler (SlowLoad)
+@end
+
+@implementation
+SentryProfiler (SlowLoad)
++ (void)load
+{
+    if ([NSProcessInfo.processInfo.arguments containsObject:@"--io.sentry.slow-load-method"]) {
+        NSMutableString *a = [NSMutableString string];
+        // 1,000,000 iterations takes about 225 milliseconds in the iPhone 15 simulator on an
+        // M2 macbook pro; we might have to adapt this for CI
+        for (NSUInteger i = 0; i < 1000000; i++) {
+            [a appendFormat:@"%d", arc4random() % 12345];
+        }
+    }
+}
+@end
+#    endif // defined(TEST) || defined(TESTCI)
+
 const int kSentryProfilerFrequencyHz = 101;
 NSTimeInterval kSentryProfilerTimeoutInterval = 30;
 
@@ -402,6 +425,45 @@ + (nullable SentryEnvelopeItem *)createEnvelopeItemForProfilePayload:
     return [[SentryEnvelopeItem alloc] initWithHeader:header data:JSONData];
 }
 
+#    if defined(TEST) || defined(TESTCI)
+void
+writeProfileFile(NSDictionary<NSString *, id> *payload)
+{
+    NSData *data = [SentrySerialization dataWithJSONObject:payload];
+    NSFileManager *fm = [NSFileManager defaultManager];
+    NSString *appSupportDirPath = SentryFileManager.sentryApplicationSupportPath;
+
+    if (![fm fileExistsAtPath:appSupportDirPath]) {
+        SENTRY_LOG_DEBUG(@"Creating app support directory.");
+        NSError *error;
+        if (!SENTRY_CASSERT_RETURN([fm createDirectoryAtPath:appSupportDirPath
+                                       withIntermediateDirectories:NO
+                                                        attributes:nil
+                                                             error:&error],
+                @"Failed to create sentry app support directory")) {
+            return;
+        }
+    } else {
+        SENTRY_LOG_DEBUG(@"App support directory already exists.");
+    }
+
+    NSString *pathToWrite;
+    if (isTracingAppLaunch) {
+        pathToWrite = [appSupportDirPath stringByAppendingPathComponent:@"launchProfile"];
+        if ([fm fileExistsAtPath:pathToWrite]) {
+            SENTRY_LOG_DEBUG(@"Already a launch profile file present.");
+            return;
+        }
+    } else {
+        pathToWrite = [appSupportDirPath stringByAppendingPathComponent:@"profile"];
+    }
+
+    SENTRY_LOG_DEBUG(@"Writing app launch profile to file.");
+    SENTRY_CASSERT(
+        [data writeToFile:pathToWrite atomically:YES], @"Failed to write profile to test file");
+}
+#    endif // defined(TEST) || defined(TESTCI)
+
 + (nullable NSMutableDictionary<NSString *, id> *)collectProfileBetween:(uint64_t)startSystemTime
                                                                     and:(uint64_t)endSystemTime
                                                                forTrace:(SentryId *)traceId
@@ -415,10 +477,9 @@ + (nullable SentryEnvelopeItem *)createEnvelopeItemForProfilePayload:
     const auto payload = [profiler serializeBetween:startSystemTime and:endSystemTime onHub:hub];
 
 #    if defined(TEST) || defined(TESTCI)
-    [NSNotificationCenter.defaultCenter postNotificationName:@"SentryProfileCompleteNotification"
-                                                      object:nil
-                                                    userInfo:payload];
+    writeProfileFile(payload);
 #    endif // defined(TEST) || defined(TESTCI)
+
     return payload;
 }
 
diff --git a/Sources/Sentry/SentryTracer.m b/Sources/Sentry/SentryTracer.m
index ef2913eb87..259478185c 100644
--- a/Sources/Sentry/SentryTracer.m
+++ b/Sources/Sentry/SentryTracer.m
@@ -48,6 +48,11 @@
 #    import <SentryScreenFrames.h>
 #endif // SENTRY_HAS_UIKIT
 
+#if defined(TEST) || defined(TESTCI)
+#    import "SentryFileManager+Test.h"
+#    import "SentryInternalDefines.h"
+#endif // defined(TEST) || defined(TESTCI)
+
 NS_ASSUME_NONNULL_BEGIN
 
 static const void *spanTimestampObserver = &spanTimestampObserver;
diff --git a/Tests/SentryTests/Helper/SentryFileManager+TestProperties.h b/Tests/SentryTests/Helper/SentryFileManager+Test.h
similarity index 86%
rename from Tests/SentryTests/Helper/SentryFileManager+TestProperties.h
rename to Tests/SentryTests/Helper/SentryFileManager+Test.h
index e85d818f13..b575d3464d 100644
--- a/Tests/SentryTests/Helper/SentryFileManager+TestProperties.h
+++ b/Tests/SentryTests/Helper/SentryFileManager+Test.h
@@ -9,6 +9,7 @@ SentryFileManager ()
 @property (nonatomic, copy) NSString *envelopesPath;
 @property (nonatomic, copy) NSString *timezoneOffsetFilePath;
 
++ (NSString *)sentryApplicationSupportPath;
 @end
 
 NS_ASSUME_NONNULL_END
diff --git a/Tests/SentryTests/SentryTests-Bridging-Header.h b/Tests/SentryTests/SentryTests-Bridging-Header.h
index acbe3676ea..c2308050e3 100644
--- a/Tests/SentryTests/SentryTests-Bridging-Header.h
+++ b/Tests/SentryTests/SentryTests-Bridging-Header.h
@@ -107,7 +107,7 @@
 #import "SentryEvent+Private.h"
 #import "SentryFileContents.h"
 #import "SentryFileIOTrackingIntegration.h"
-#import "SentryFileManager+TestProperties.h"
+#import "SentryFileManager+Test.h"
 #import "SentryFileManager.h"
 #import "SentryFormatter.h"
 #import "SentryFrame.h"

From 46341fd62221b6e115d53c6c05f044cd35f81dad Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Fri, 19 Jan 2024 15:35:16 -0900
Subject: [PATCH 02/34] fix integration tests after moving to a launch
 profiling marker file

---
 Samples/iOS-Swift/iOS-Swift/AppDelegate.swift |  2 +-
 .../iOS-Swift/Base.lproj/Main.storyboard      | 17 ++--
 .../Profiling/ProfilingViewController.swift   | 13 ++-
 .../iOS-SwiftUITests/ProfilingUITests.swift   | 90 ++++++-------------
 4 files changed, 44 insertions(+), 78 deletions(-)

diff --git a/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift b/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift
index 3b6c30fe4a..cc75237b20 100644
--- a/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift
+++ b/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift
@@ -52,7 +52,7 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
                 }
             }
 
-            options.profileAppLaunches = args.contains("--profile-app-launches")
+            options.enableAppLaunchProfiling = args.contains("--profile-app-launches")
 
             options.sessionTrackingIntervalMillis = 5_000
             options.attachScreenshot = true
diff --git a/Samples/iOS-Swift/iOS-Swift/Base.lproj/Main.storyboard b/Samples/iOS-Swift/iOS-Swift/Base.lproj/Main.storyboard
index 749845d15d..67696810cb 100644
--- a/Samples/iOS-Swift/iOS-Swift/Base.lproj/Main.storyboard
+++ b/Samples/iOS-Swift/iOS-Swift/Base.lproj/Main.storyboard
@@ -223,22 +223,22 @@
                                             </button>
                                         </subviews>
                                     </stackView>
-                                    <button opaque="NO" contentMode="scaleToFill" contentHorizontalAlignment="center" contentVerticalAlignment="center" buttonType="system" lineBreakMode="middleTruncation" translatesAutoresizingMaskIntoConstraints="NO" id="Pcb-L7-DFD">
-                                        <rect key="frame" x="102.5" y="56" width="115" height="28"/>
+                                    <button opaque="NO" contentMode="scaleToFill" contentHorizontalAlignment="center" contentVerticalAlignment="center" buttonType="system" lineBreakMode="middleTruncation" translatesAutoresizingMaskIntoConstraints="NO" id="vwa-d9-1BM">
+                                        <rect key="frame" x="111.5" y="56" width="97" height="28"/>
                                         <fontDescription key="fontDescription" type="system" pointSize="13"/>
                                         <inset key="imageEdgeInsets" minX="0.0" minY="0.0" maxX="2.2250738585072014e-308" maxY="0.0"/>
-                                        <state key="normal" title="View launch config"/>
+                                        <state key="normal" title="View last profile"/>
                                         <connections>
-                                            <action selector="viewLaunchConfig:" destination="NZr-bH-g9o" eventType="touchUpInside" id="dq1-Ng-UNd"/>
+                                            <action selector="viewLastProfile:" destination="NZr-bH-g9o" eventType="touchUpInside" id="7Xd-19-rsP"/>
                                         </connections>
                                     </button>
-                                    <button opaque="NO" contentMode="scaleToFill" contentHorizontalAlignment="center" contentVerticalAlignment="center" buttonType="system" lineBreakMode="middleTruncation" translatesAutoresizingMaskIntoConstraints="NO" id="vwa-d9-1BM">
-                                        <rect key="frame" x="111.5" y="84" width="97" height="28"/>
+                                    <button opaque="NO" contentMode="scaleToFill" contentHorizontalAlignment="center" contentVerticalAlignment="center" buttonType="system" lineBreakMode="middleTruncation" translatesAutoresizingMaskIntoConstraints="NO" id="keL-m5-7hv">
+                                        <rect key="frame" x="33.5" y="84" width="253" height="28"/>
                                         <fontDescription key="fontDescription" type="system" pointSize="13"/>
                                         <inset key="imageEdgeInsets" minX="0.0" minY="0.0" maxX="2.2250738585072014e-308" maxY="0.0"/>
-                                        <state key="normal" title="View last profile"/>
+                                        <state key="normal" title="Check launch profile marker file existence"/>
                                         <connections>
-                                            <action selector="viewLastProfile:" destination="NZr-bH-g9o" eventType="touchUpInside" id="7Xd-19-rsP"/>
+                                            <action selector="checkLaunchProfilingMarkerFile:" destination="NZr-bH-g9o" eventType="touchUpInside" id="IcT-D3-Oya"/>
                                         </connections>
                                     </button>
                                     <button opaque="NO" contentMode="scaleToFill" contentHorizontalAlignment="center" contentVerticalAlignment="center" buttonType="system" lineBreakMode="middleTruncation" translatesAutoresizingMaskIntoConstraints="NO" id="F1h-qu-BzX">
@@ -391,6 +391,7 @@
                     </view>
                     <tabBarItem key="tabBarItem" title="Profiling" image="flame.fill" catalog="system" selectedImage="flame.fill" id="Wfx-IJ-h9f"/>
                     <connections>
+                        <outlet property="launchProfilingMarkerFileCheckButton" destination="keL-m5-7hv" id="s2U-XE-Flv"/>
                         <outlet property="maxThreadsTextField" destination="COd-X5-alN" id="6Pn-Be-aUe"/>
                         <outlet property="maxWorkIntensityTextField" destination="rJh-QS-fAl" id="u9q-Sj-6SY"/>
                         <outlet property="minThreadsTextField" destination="sCM-tI-skI" id="D0Z-yT-LbU"/>
diff --git a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
index 7d462cd5a8..7c33832ce0 100644
--- a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
+++ b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
@@ -15,6 +15,7 @@ class ProfilingViewController: UIViewController, UITextFieldDelegate {
     @IBOutlet weak var maxWorkIntensityTextField: UITextField!
     @IBOutlet weak var valueTextField: UITextField!
     
+    @IBOutlet weak var launchProfilingMarkerFileCheckButton: UIButton!
     @IBOutlet weak var profilingUITestDataMarshalingTextField: UITextField!
     @IBOutlet weak var profilingUITestDataMarshalingStatus: UILabel!
     
@@ -28,6 +29,7 @@ class ProfilingViewController: UIViewController, UITextFieldDelegate {
             $0?.delegate = self
         }
         profilingUITestDataMarshalingTextField.accessibilityLabel = "io.sentry.ui-tests.profile-marshaling-text-field"
+        launchProfilingMarkerFileCheckButton.accessibilityLabel = "io.sentry.ui-tests.app-launch-profile-marker-file-button"
     }
     
     override func viewDidAppear(_ animated: Bool) {
@@ -87,10 +89,13 @@ class ProfilingViewController: UIViewController, UITextFieldDelegate {
         view.backgroundColor = .init(white: CGFloat(sender.value), alpha: 1)
     }
     
-    @IBAction func viewLaunchConfig(_ sender: Any) {
-        let launchConfigPath = ((NSSearchPathForDirectoriesInDomains(.applicationSupportDirectory, .userDomainMask, true).first! as NSString).appendingPathComponent("io.sentry") as NSString).appendingPathComponent("launchConfig")
-        let url = URL(fileURLWithPath: launchConfigPath)
-        handleContents(file: url)
+    @IBAction func checkLaunchProfilingMarkerFile(_ sender: Any) {
+        let launchProfileMarkerPath = ((NSSearchPathForDirectoriesInDomains(.applicationSupportDirectory, .userDomainMask, true).first! as NSString).appendingPathComponent("io.sentry") as NSString).appendingPathComponent("profileLaunch")
+        if FileManager.default.fileExists(atPath: launchProfileMarkerPath) {
+            profilingUITestDataMarshalingTextField.text = "<exists>"
+        } else {
+            profilingUITestDataMarshalingTextField.text = "<missing>"
+        }
     }
     
     @IBAction func viewLastProfile(_ sender: Any) {
diff --git a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
index 418e3111a6..dcac565e14 100644
--- a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
+++ b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
@@ -5,17 +5,10 @@ import XCTest
 class ProfilingUITests: BaseUITest {    
     override var automaticallyManageAppSession: Bool { false }
     
+    // this will run before the non-async BaseUITest.setUp, so we can bail out before running any of the logic in there
     override func setUp() async throws {
         try await super.setUp()
-        
-        guard shouldRunProfilingUITest() else {
-           throw XCTSkip("iOS version too old for profiling test.")
-        }
-        
-        app.launchArguments.append(contentsOf: [
-            "--io.sentry.wipe-data" // wipe all the profiles stored to disk, and any launch profiling config files, before running each test
-        ]
-        )
+        try checkOSVersionForProfilingTest()
     }
     
     func testProfiledAppLaunches() throws {
@@ -80,11 +73,10 @@ class ProfilingUITests: BaseUITest {
 }
 
 extension ProfilingUITests {
-    func shouldRunProfilingUITest() -> Bool {
-        if #available(iOS 16.0, *) {
-            return true
-        } else {
-            return false
+    // We don't need to test these on multiple OSes right now, and older versions seem to have issues; older devices or VM images running simulators might just be slower. Latest OS is enough coverage for our needs for now.
+    func checkOSVersionForProfilingTest() throws {
+        guard #available(iOS 16.0, *) else {
+            throw XCTSkip("iOS version too old for profiling test.")
         }
     }
     
@@ -105,6 +97,12 @@ extension ProfilingUITests {
         return try XCTUnwrap(try JSONSerialization.jsonObject(with: data) as? [String: Any])
     }
     
+    func checkLaunchProfileMarkerFileExistence() throws -> Bool {
+        app.buttons["io.sentry.ui-tests.app-launch-profile-marker-file-button"].afterWaitingForExistence("Couldn't find app launch profile marker file check button").tap()
+        let string = try XCTUnwrap(app.textFields["io.sentry.ui-tests.profile-marshaling-text-field"].afterWaitingForExistence("Couldn't find data marshaling text field.").value as? NSString)
+        return string == "<exists>"
+    }
+    
     func goToTransactions() {
         app.tabBars["Tab Bar"].buttons["Transactions"].tap()
     }
@@ -121,10 +119,6 @@ extension ProfilingUITests {
         app.tabBars["Tab Bar"].buttons["Profiling"].afterWaitingForExistence("Couldn't find profiling tab bar button").tap()
     }
     
-    func retrieveLaunchProfilingConfig() {
-        app.buttons["View launch config"].afterWaitingForExistence("Couldn't find button to view launch config").tap()
-    }
-    
     func retrieveLastProfileData() {
         app.buttons["View last profile"].afterWaitingForExistence("Couldn't find button to view last profile").tap()
     }
@@ -133,19 +127,10 @@ extension ProfilingUITests {
         app.buttons["View launch profile"].afterWaitingForExistence("Couldn't find button to view launch profile").tap()
     }
 
-    func assertLaunchProfile(ran: Bool) throws {
+    func assertLaunchProfile() throws {
         retrieveLaunchProfileData()
         
-        var lastProfile: [String: Any]
-        do {
-            lastProfile = try marshalJSONDictionaryFromApp()
-        } catch {
-            guard ran else {
-                return
-            }
-            throw error
-        }
-        
+        var lastProfile = try marshalJSONDictionaryFromApp()
         let sampledProfile = try XCTUnwrap(lastProfile["profile"] as? [String: Any])
         let stacks = try XCTUnwrap(sampledProfile["stacks"] as? [[Int]])
         let frames = try XCTUnwrap(sampledProfile["frames"] as? [[String: Any]])
@@ -171,27 +156,6 @@ extension ProfilingUITests {
         // ???: can we assert that this stack was on the main thread?
         // TODO: yes! need to correlate the samples.[].stack_id with samples.[].thread_id
     }
-    
-    func assertLaunchProfileConfig(expectedFile: Bool, expectedProfilesSampleRate: Int? = nil, expectedTracesSampleRate: Int? = nil) throws {
-        retrieveLaunchProfilingConfig()
-        var launchConfig: [String: Any]
-        
-        do {
-            launchConfig = try marshalJSONDictionaryFromApp()
-        } catch {
-            guard expectedFile else {
-                return
-            }
-            throw error
-        }
-        
-        if let expectedProfilesSampleRate = expectedProfilesSampleRate {
-            XCTAssertEqual(try XCTUnwrap(launchConfig["profilesSampleRate"] as? Int), expectedProfilesSampleRate)
-        }
-        if let expectedTracesSampleRate = expectedTracesSampleRate {
-            XCTAssertEqual(try XCTUnwrap(launchConfig["tracesSampleRate"] as? Int), expectedTracesSampleRate)
-        }
-    }
      
     /**
      * Performs the various operations for the launch profiler test case:
@@ -258,27 +222,23 @@ extension ProfilingUITests {
         
         launchApp()
         
-        try performAssertions(
-            shouldProfileThisLaunch: shouldProfileThisLaunch,
-            shouldProfileNextLaunch: shouldEnableLaunchProfilingOptionForNextLaunch && !(shouldDisableTracing || shouldDisableSwizzling || shouldDisableAutoPerformanceTracking || shouldDisableUIViewControllerTracing),
-            expectedProfilesSampleRate: resolvedProfilesSampleRate,
-            expectedTracesSampleRate: resolvedTracesSampleRate
-        )
+        let sdkOptionsConfigurationAllowsLaunchProfiling = !(shouldDisableTracing || shouldDisableSwizzling || shouldDisableAutoPerformanceTracking || shouldDisableUIViewControllerTracing)
+        
+        // these tests only set sample rates to 0 or 1, or don't provide an override (and the sample app sets them to 1 by default)
+        let sampleRatesAllowLaunchProfiling = (resolvedTracesSampleRate == nil || resolvedTracesSampleRate! == 1) && (resolvedProfilesSampleRate == nil || resolvedProfilesSampleRate == 1)
+        
+        let shouldProfileNextLaunch = shouldEnableLaunchProfilingOptionForNextLaunch && sdkOptionsConfigurationAllowsLaunchProfiling && sampleRatesAllowLaunchProfiling
+        
+        try performAssertions(shouldProfileThisLaunch: shouldProfileThisLaunch, shouldProfileNextLaunch: shouldProfileNextLaunch)
     }
     
-    func performAssertions(shouldProfileThisLaunch: Bool,
-                           shouldProfileNextLaunch: Bool,
-                           expectedProfilesSampleRate: Int? = nil,
-                           expectedTracesSampleRate: Int? = nil
-    ) throws {
+    func performAssertions(shouldProfileThisLaunch: Bool, shouldProfileNextLaunch: Bool ) throws {
         goToProfiling()
         
-        if shouldProfileNextLaunch {
-            try assertLaunchProfileConfig(expectedFile: shouldProfileNextLaunch, expectedProfilesSampleRate: expectedProfilesSampleRate, expectedTracesSampleRate: expectedTracesSampleRate)
-        }
+        XCTAssertEqual(shouldProfileNextLaunch, try checkLaunchProfileMarkerFileExistence())
         
         if shouldProfileThisLaunch {
-            try assertLaunchProfile(ran: shouldProfileThisLaunch)
+            try assertLaunchProfile()
         }
     }
 }

From 3893093a8fb1245eb76e235239b02a108db8b97e Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 18 Jan 2024 17:05:05 -0900
Subject: [PATCH 03/34] pr feedback: test new option's default value

---
 Tests/SentryTests/SentryOptionsTest.m | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/Tests/SentryTests/SentryOptionsTest.m b/Tests/SentryTests/SentryOptionsTest.m
index b723224f8c..d81c44493b 100644
--- a/Tests/SentryTests/SentryOptionsTest.m
+++ b/Tests/SentryTests/SentryOptionsTest.m
@@ -1192,6 +1192,14 @@ - (void)testInitialScope
     XCTAssertIdentical(initialScope, options.initialScope);
 }
 
+#if SENTRY_TARGET_PROFILING_SUPPORTED
+- (void)testEnableAppLaunchProfilingDefaultValue
+{
+    SentryOptions *options = [self getValidOptions:@{}];
+    XCTAssertFalse(options.enableAppLaunchProfiling);
+}
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED
+
 - (SentryOptions *)getValidOptions:(NSDictionary<NSString *, id> *)dict
 {
     NSError *error = nil;

From ceddcd9161f9debe6dbbc07d1f60b2034b77f2da Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Mon, 22 Jan 2024 12:25:49 -0900
Subject: [PATCH 04/34] comment conditional complier closures

---
 .../SentryProfilerSwiftTests.swift            | 45 +++++++++----------
 1 file changed, 22 insertions(+), 23 deletions(-)

diff --git a/Tests/SentryProfilerTests/SentryProfilerSwiftTests.swift b/Tests/SentryProfilerTests/SentryProfilerSwiftTests.swift
index 90364f90cd..d79f2d66ee 100644
--- a/Tests/SentryProfilerTests/SentryProfilerSwiftTests.swift
+++ b/Tests/SentryProfilerTests/SentryProfilerSwiftTests.swift
@@ -37,7 +37,7 @@ class SentryProfilerSwiftTests: XCTestCase {
 #if !os(macOS)
         lazy var displayLinkWrapper = TestDisplayLinkWrapper(dateProvider: currentDateProvider)
         lazy var framesTracker = SentryFramesTracker(displayLinkWrapper: displayLinkWrapper, dateProvider: currentDateProvider, keepDelayedFramesDuration: 0)
-#endif
+#endif // !os(macOS)
 
         init() {
             SentryDependencyContainer.sharedInstance().dateProvider = currentDateProvider
@@ -61,7 +61,7 @@ class SentryProfilerSwiftTests: XCTestCase {
             SentryDependencyContainer.sharedInstance().framesTracker = framesTracker
             framesTracker.start()
             displayLinkWrapper.call()
-#endif
+#endif // !os(macOS)
         }
 
         /// Advance the mock date provider, start a new transaction and return its handle.
@@ -95,9 +95,7 @@ class SentryProfilerSwiftTests: XCTestCase {
 #if !os(macOS)
         // SentryFramesTracker starts assuming a frame rate of 60 Hz and will only log an update if it changes, so the first value here needs to be different for it to register.
         let mockFrameRateChangesPerBatch: [FrameRate] = [.high, .low, .high, .low]
-#endif
 
-#if !os(macOS)
         // Absolute timestamps must be adjusted per span when asserting
         var expectedSlowFrames = [[String: Any]]()
         var expectedFrozenFrames = [[String: Any]]()
@@ -108,7 +106,7 @@ class SentryProfilerSwiftTests: XCTestCase {
             expectedFrozenFrames = [[String: Any]]()
             expectedFrameRateChanges = [[String: Any]]()
         }
-#endif
+#endif // !os(macOS)
 
         func gatherMockedMetrics(span: Span) throws {
             // clear out any errors that might've been set in previous calls
@@ -124,14 +122,14 @@ class SentryProfilerSwiftTests: XCTestCase {
                 systemWrapper.overrides.cpuEnergyUsage = NSNumber(value: systemWrapper.overrides.cpuEnergyUsage!.intValue + mockEnergyUsage.intValue)
             }
 
-    #if !os(macOS)
+#if !os(macOS)
             var shouldRecordFrameRateExpectation = true
-
+            
             func changeFrameRate(_ new: FrameRate) {
                 displayLinkWrapper.changeFrameRate(new)
                 shouldRecordFrameRateExpectation = true
             }
-
+            
             func renderGPUFrame(_ type: GPUFrame) {
                 switch type {
                 case .normal:
@@ -165,7 +163,7 @@ class SentryProfilerSwiftTests: XCTestCase {
                     ])
                 }
             }
-
+            
             /*
              * Mock a series of GPU frame renders of varying quality (normal/slow/frozen) and
              * refresh rate changes. The refresh rate changes ("|") happen at the same time as
@@ -198,7 +196,7 @@ class SentryProfilerSwiftTests: XCTestCase {
             renderGPUFrame(.slow)
             renderGPUFrame(.normal)
             renderGPUFrame(.frozen)
-    #endif
+#endif // !os(macOS)
 
             // mock errors gathering cpu usage and memory footprint and fire a callback for them to ensure they don't add more information to the payload
             systemWrapper.overrides.cpuUsageError = NSError(domain: "test-error", code: 0)
@@ -218,7 +216,7 @@ class SentryProfilerSwiftTests: XCTestCase {
         var appStartDuration = 0.5
         lazy var appStartEnd = appStart.addingTimeInterval(appStartDuration)
 
-        #if os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
+#if !os(macOS)
         func getAppStartMeasurement(type: SentryAppStartType, preWarmed: Bool = false) -> SentryAppStartMeasurement {
             let runtimeInitDuration = 0.05
             let runtimeInit = appStart.addingTimeInterval(runtimeInitDuration)
@@ -231,7 +229,7 @@ class SentryProfilerSwiftTests: XCTestCase {
             return SentryAppStartMeasurement(type: type, isPreWarmed: preWarmed, appStartTimestamp: appStart, duration: appStartDuration, runtimeInitTimestamp: runtimeInit, moduleInitializationTimestamp: main,
                                              sdkStartTimestamp: appStart, didFinishLaunchingTimestamp: didFinishLaunching)
         }
-        #endif // os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
+#endif // !os(macOS)
     }
 
     private var fixture: Fixture!
@@ -323,7 +321,7 @@ class SentryProfilerSwiftTests: XCTestCase {
         spans.removeAll()
 #if !os(macOS)
         fixture.resetGPUExpectations()
-#endif
+#endif // !os(macOS)
 
         try createConcurrentSpansWithMetrics()
     }
@@ -380,7 +378,7 @@ class SentryProfilerSwiftTests: XCTestCase {
         try performTest(transactionEnvironment: expectedEnvironment)
     }
 
-    #if os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
+#if !os(macOS)
     func testProfileWithTransactionContainingStartupSpansForColdStart() throws {
         try performTest(uikitParameters: UIKitParameters(launchType: .cold, prewarmed: false))
     }
@@ -392,7 +390,7 @@ class SentryProfilerSwiftTests: XCTestCase {
     func testProfileWithTransactionContainingStartupSpansForPrewarmedStart() throws {
         try performTest(uikitParameters: UIKitParameters(launchType: .cold, prewarmed: true))
     }
-#endif // os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
+#endif // !os(macOS)
 
     func testProfilingDataContainsEnvironmentSetFromConfigureScope() throws {
         let expectedEnvironment = "test-environment"
@@ -534,7 +532,7 @@ class SentryProfilerSwiftTests: XCTestCase {
         XCTAssertEqual(self.fixture.client?.captureEventWithScopeInvocations.count, 0)
     }
 
-#if os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
+#if !os(macOS)
     /// based on ``SentryTracerTests.testFinish_WaitForAllChildren_StartTimeModified_NoTransactionCaptured``
     func testProfilerCleanedUpAfterTransactionDiscarded_WaitForAllChildren_StartTimeModified() throws {
         XCTAssertEqual(SentryProfiler.currentProfiledTracers(), UInt(0))
@@ -551,7 +549,8 @@ class SentryProfilerSwiftTests: XCTestCase {
         XCTAssertEqual(SentryProfiler.currentProfiledTracers(), UInt(0))
         XCTAssertEqual(self.fixture.client?.captureEventWithScopeInvocations.count, 0)
     }
-#endif // os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
+#endif // !os(macOS)
+    
 }
 
 private extension SentryProfilerSwiftTests {
@@ -581,16 +580,16 @@ private extension SentryProfilerSwiftTests {
     }
 
     struct UIKitParameters {
-        #if os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
+#if os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
         var launchType: SentryAppStartType
         var prewarmed: Bool
-        #endif // os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
+#endif // os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
     }
 
     func performTest(transactionEnvironment: String = kSentryDefaultEnvironment, shouldTimeOut: Bool = false, uikitParameters: UIKitParameters? = nil) throws {
         var testingAppLaunchSpans = false
-
-            #if os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
+        
+#if os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
         if let uikitParameters = uikitParameters {
             testingAppLaunchSpans = true
             let appStartMeasurement = fixture.getAppStartMeasurement(type: uikitParameters.launchType, preWarmed: uikitParameters.prewarmed)
@@ -637,7 +636,7 @@ private extension SentryProfilerSwiftTests {
         try assertMetricEntries(measurements: measurements, key: kSentryProfilerSerializationKeySlowFrameRenders, expectedEntries: fixture.expectedSlowFrames, transaction: transaction)
         try assertMetricEntries(measurements: measurements, key: kSentryProfilerSerializationKeyFrozenFrameRenders, expectedEntries: fixture.expectedFrozenFrames, transaction: transaction)
         try assertMetricEntries(measurements: measurements, key: kSentryProfilerSerializationKeyFrameRates, expectedEntries: fixture.expectedFrameRateChanges, transaction: transaction)
-#endif
+#endif // !os(macOS)
     }
 
     func sortedByTimestamps(_ entries: [[String: Any]]) -> [[String: Any]] {
@@ -724,7 +723,7 @@ private extension SentryProfilerSwiftTests {
         XCTAssertTrue(try XCTUnwrap(device["is_emulator"] as? Bool))
 #else
         XCTAssertFalse(try XCTUnwrap(device["is_emulator"] as? Bool))
-#endif
+#endif // targetEnvironment(simulator)
 
         let os = try XCTUnwrap(profile["os"] as? [String: Any?])
         XCTAssertNotNil(os)

From 7d8b5f79db53664e0336e8fc53eacbf2a4be594a Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Mon, 22 Jan 2024 16:06:21 -0900
Subject: [PATCH 05/34] add unit tests for setting the option and option
 configurations

---
 Sentry.xcodeproj/project.pbxproj              |   4 +
 Sources/Sentry/SentryProfiler.mm              |   1 +
 .../SentryAppLaunchProfilingTests.m           | 157 ++++++++++++++++++
 .../SentryProfilerTests.mm                    |   2 +-
 Tests/SentryTests/SentrySDK+Tests.h           |  10 +-
 5 files changed, 172 insertions(+), 2 deletions(-)
 create mode 100644 Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m

diff --git a/Sentry.xcodeproj/project.pbxproj b/Sentry.xcodeproj/project.pbxproj
index 51d068d24d..8e23835c3a 100644
--- a/Sentry.xcodeproj/project.pbxproj
+++ b/Sentry.xcodeproj/project.pbxproj
@@ -607,6 +607,7 @@
 		7DC8310A2398283C0043DD9A /* SentryCrashIntegration.h in Headers */ = {isa = PBXBuildFile; fileRef = 7DC831082398283C0043DD9A /* SentryCrashIntegration.h */; };
 		7DC8310C2398283C0043DD9A /* SentryCrashIntegration.m in Sources */ = {isa = PBXBuildFile; fileRef = 7DC831092398283C0043DD9A /* SentryCrashIntegration.m */; };
 		840A11122B61E27500650D02 /* SentrySamplerDecision.m in Sources */ = {isa = PBXBuildFile; fileRef = 840A11102B61E27500650D02 /* SentrySamplerDecision.m */; };
+		840A110A2B5F47F700650D02 /* SentryAppLaunchProfilingTests.m in Sources */ = {isa = PBXBuildFile; fileRef = 840A11092B5F47F700650D02 /* SentryAppLaunchProfilingTests.m */; };
 		84281C432A578E5600EE88F2 /* SentryProfilerState.mm in Sources */ = {isa = PBXBuildFile; fileRef = 84281C422A578E5600EE88F2 /* SentryProfilerState.mm */; };
 		84281C462A57905700EE88F2 /* SentrySample.h in Headers */ = {isa = PBXBuildFile; fileRef = 84281C442A57905700EE88F2 /* SentrySample.h */; };
 		84281C472A57905700EE88F2 /* SentrySample.m in Sources */ = {isa = PBXBuildFile; fileRef = 84281C452A57905700EE88F2 /* SentrySample.m */; };
@@ -1574,6 +1575,7 @@
 		7DC831082398283C0043DD9A /* SentryCrashIntegration.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = SentryCrashIntegration.h; path = include/SentryCrashIntegration.h; sourceTree = "<group>"; };
 		7DC831092398283C0043DD9A /* SentryCrashIntegration.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentryCrashIntegration.m; sourceTree = "<group>"; };
 		840A11102B61E27500650D02 /* SentrySamplerDecision.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentrySamplerDecision.m; sourceTree = "<group>"; };
+		840A11092B5F47F700650D02 /* SentryAppLaunchProfilingTests.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentryAppLaunchProfilingTests.m; sourceTree = "<group>"; };
 		8419C0C328C1889D001C8259 /* SentryProfilerSwiftTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = SentryProfilerSwiftTests.swift; sourceTree = "<group>"; };
 		84281C422A578E5600EE88F2 /* SentryProfilerState.mm */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.objcpp; path = SentryProfilerState.mm; sourceTree = "<group>"; };
 		84281C442A57905700EE88F2 /* SentrySample.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = SentrySample.h; path = ../include/SentrySample.h; sourceTree = "<group>"; };
@@ -3239,6 +3241,7 @@
 				035E73CB27D575B3005EEB11 /* SentrySamplingProfilerTests.mm */,
 				035E73CD27D5790A005EEB11 /* SentryThreadMetadataCacheTests.mm */,
 				03F9D37B2819A65C00602916 /* SentryProfilerTests.mm */,
+				840A11092B5F47F700650D02 /* SentryAppLaunchProfilingTests.m */,
 				8419C0C328C1889D001C8259 /* SentryProfilerSwiftTests.swift */,
 			);
 			path = SentryProfilerTests;
@@ -4494,6 +4497,7 @@
 				0A9415BA28F96CAC006A5DD1 /* TestSentryReachability.swift in Sources */,
 				D880E3A728573E87008A90DB /* SentryBaggageTests.swift in Sources */,
 				7B16FD022654F86B008177D3 /* SentrySysctlTests.swift in Sources */,
+				840A110A2B5F47F700650D02 /* SentryAppLaunchProfilingTests.m in Sources */,
 				7BAF3DB5243C743E008A5414 /* SentryClientTests.swift in Sources */,
 				8EAE8E5E2681768000D6958B /* URLSessionTaskMock.m in Sources */,
 				D8CE69BC277E39C700C6EC5C /* SentryFileIOTrackingIntegrationObjCTests.m in Sources */,
diff --git a/Sources/Sentry/SentryProfiler.mm b/Sources/Sentry/SentryProfiler.mm
index 1c1fc90b4c..a6e13a9090 100644
--- a/Sources/Sentry/SentryProfiler.mm
+++ b/Sources/Sentry/SentryProfiler.mm
@@ -52,6 +52,7 @@
 
 #    if defined(TEST) || defined(TESTCI)
 #        import "SentryFileManager+Test.h"
+#        import "SentryLaunchProfiling.h"
 
 @interface
 SentryProfiler (SlowLoad)
diff --git a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
new file mode 100644
index 0000000000..3522bd00e2
--- /dev/null
+++ b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
@@ -0,0 +1,157 @@
+#import "SentryOptions+HybridSDKs.h"
+#import "SentryOptions+Private.h"
+#import "SentryProfilingConditionals.h"
+#import "SentrySDK+Tests.h"
+#import <XCTest/XCTest.h>
+
+#if SENTRY_TARGET_PROFILING_SUPPORTED
+
+@interface SentryAppLaunchProfilingTests : XCTestCase
+@end
+
+@implementation SentryAppLaunchProfilingTests
+
+#    define SENTRY_OPTION(name, value)                                                             \
+        NSStringFromSelector(@selector(name))                                                      \
+            : value
+
+- (void)testDefaultOptionsDoNotEnableLaunchProfiling
+{
+    XCTAssertFalse(shouldProfileNextLaunch([self defaultOptionsWithOverrides:nil]),
+        @"Default options should not enable launch profiling");
+}
+
+- (void)testAppLaunchProfilingNotSufficientToEnableLaunchProfiling
+{
+    XCTAssertFalse(
+        shouldProfileNextLaunch(
+            [self defaultOptionsWithOverrides:@{ SENTRY_OPTION(enableAppLaunchProfiling, @YES) }]),
+        @"Default options with only launch profiling option set is not sufficient to enable launch "
+        @"profiling");
+}
+
+- (void)testAppLaunchProfilingAndTracingOptionsNotSufficientToEnableAppLaunchProfiling
+{
+    XCTAssertFalse(shouldProfileNextLaunch([self
+                       defaultOptionsWithOverrides:@{ SENTRY_OPTION(enableAppLaunchProfiling, @YES),
+                                                       SENTRY_OPTION(enableTracing, @YES) }]),
+        @"Default options with app launch profiling and tracing enabled are not sufficient to "
+        @"enable launch profiling");
+}
+
+- (void)
+    testAppLaunchProfilingAndTracingAndTracesSampleRateOptionsNotSufficientToEnableAppLaunchProfiling
+{
+    XCTAssertFalse(shouldProfileNextLaunch([self
+                       defaultOptionsWithOverrides:@{ SENTRY_OPTION(enableAppLaunchProfiling, @YES),
+                                                       SENTRY_OPTION(enableTracing, @YES),
+                                                       SENTRY_OPTION(tracesSampleRate, @1) }]),
+        @"Default options with app launch profiling and tracing enabled with traces sample rate of "
+        @"1 are not sufficient to enable launch profiling");
+}
+
+- (void)testMinimumOptionsRequiredToEnableAppLaunchProfiling
+{
+    XCTAssert(shouldProfileNextLaunch([self defaultLaunchProfilingOptionsWithOverrides:nil]),
+        @"Default options with app launch profiling and tracing enabled and traces and profiles "
+        @"sample rates of 1 should enable launch profiling");
+}
+
+- (void)testDisablingLaunchProfilingOptionDisablesAppLaunchProfiling
+{
+    XCTAssertFalse(
+        shouldProfileNextLaunch(
+            [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
+                                                                 enableAppLaunchProfiling, @NO) }]),
+        @"Default options with tracing enabled, traces and profiles sample rates of 1, but app "
+        @"launch profiling disabled should not enable launch profiling");
+}
+
+- (void)testDisablingTracingOptionDisablesAppLaunchProfiling
+{
+    XCTAssertFalse(
+        shouldProfileNextLaunch([self
+            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(enableTracing, @NO) }]),
+        @"Default options with app launch profiling enabled, traces and profiles sample rates of "
+        @"1, but tracing disabled should not enable launch profiling");
+}
+
+- (void)testSettingTracesSampleRateTo0DisablesAppLaunchProfiling
+{
+    XCTAssertFalse(
+        shouldProfileNextLaunch([self
+            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(tracesSampleRate, @0) }]),
+        @"Default options with app launch profiling and tracing enabled, profiles sample rate of "
+        @"1, but traces sample rate of 0 should not enable launch profiling");
+}
+
+- (void)testSettingProfilesSampleRateTo0DisablesAppLaunchProfiling
+{
+    XCTAssertFalse(
+        shouldProfileNextLaunch([self
+            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(profilesSampleRate, @0) }]),
+        @"Default options with app launch profiling and tracing enabled, traces sample rate of 1, "
+        @"but profiles sample rate of 0 should not enable launch profiling");
+}
+
+- (void)testDisablingSwizzlingOptionDisablesAppLaunchProfiling
+{
+    XCTAssertFalse(
+        shouldProfileNextLaunch([self
+            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(enableSwizzling, @NO) }]),
+        @"Default options with app launch profiling and tracing enabled, traces and profiles "
+        @"sample rates of 1, but swizzling disabled should not enable launch profiling");
+}
+
+- (void)testDisablingAutoPerformanceTracingOptionDisablesAppLaunchProfiling
+{
+    XCTAssertFalse(
+        shouldProfileNextLaunch([self
+            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
+                                                           enableAutoPerformanceTracing, @NO) }]),
+        @"Default options with app launch profiling and tracing enabled, traces and profiles "
+        @"sample rates of 1, but automatic performance tracing disabled should not enable launch "
+        @"profiling");
+}
+
+- (void)testDisablingUIViewControllerTracingOptionDisablesAppLaunchProfiling
+{
+    XCTAssertFalse(
+        shouldProfileNextLaunch([self
+            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
+                                                           enableUIViewControllerTracing, @NO) }]),
+        @"Default options with app launch profiling and tracing enabled, traces and profiles "
+        @"sample rates of 1, but UIViewController tracing disabled should not enable launch "
+        @"profiling");
+}
+
+- (SentryOptions *)defaultLaunchProfilingOptionsWithOverrides:
+    (NSDictionary<NSString *, id> *)overrides
+{
+    NSMutableDictionary<NSString *, id> *options = [NSMutableDictionary<NSString *, id>
+        dictionaryWithDictionary:@{ SENTRY_OPTION(enableAppLaunchProfiling, @YES),
+                                     SENTRY_OPTION(enableTracing, @YES),
+                                     SENTRY_OPTION(tracesSampleRate, @1),
+                                     SENTRY_OPTION(profilesSampleRate, @1) }];
+    [options addEntriesFromDictionary:overrides];
+    return [self defaultOptionsWithOverrides:options];
+}
+
+- (SentryOptions *)defaultOptionsWithOverrides:(nullable NSDictionary<NSString *, id> *)overrides
+{
+    NSMutableDictionary<NSString *, id> *options = [NSMutableDictionary<NSString *, id>
+        dictionaryWithObject:@"https://username:password@sentry.io/1"
+                      forKey:@"dsn"];
+    [options addEntriesFromDictionary:overrides];
+    NSError *error;
+    SentryOptions *sentryOptions = [[SentryOptions alloc] initWithDict:options
+                                                      didFailWithError:&error];
+    XCTAssertNil(error);
+    return sentryOptions;
+}
+
+#    undef SENTRY_OPTION
+
+@end
+
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED
diff --git a/Tests/SentryProfilerTests/SentryProfilerTests.mm b/Tests/SentryProfilerTests/SentryProfilerTests.mm
index 30de6e3c2f..ddeed62c42 100644
--- a/Tests/SentryProfilerTests/SentryProfilerTests.mm
+++ b/Tests/SentryProfilerTests/SentryProfilerTests.mm
@@ -262,4 +262,4 @@ - (void)testProfilerPayload
 
 @end
 
-#endif
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED
diff --git a/Tests/SentryTests/SentrySDK+Tests.h b/Tests/SentryTests/SentrySDK+Tests.h
index 159ab4e784..fcdddabdd7 100644
--- a/Tests/SentryTests/SentrySDK+Tests.h
+++ b/Tests/SentryTests/SentrySDK+Tests.h
@@ -1,7 +1,15 @@
+#import "SentryDefines.h"
+#import "SentrySDK.h"
+
+@class SentryEnvelope;
+@class SentryHub;
+
 NS_ASSUME_NONNULL_BEGIN
 
+SENTRY_EXTERN BOOL shouldProfileNextLaunch(SentryOptions *options);
+
 @interface
-SentrySDK (Tests)
+SentrySDK ()
 
 + (void)setCurrentHub:(nullable SentryHub *)hub;
 

From 98155c53a1d2d15766e410cee09b5cd66001eeac Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Wed, 24 Jan 2024 21:28:30 -0900
Subject: [PATCH 06/34] fix tests after reimplementing writing the sample rates
 to the file

---
 Sentry.xcodeproj/project.pbxproj              |  8 ++-
 .../Sentry/Profiling/SentryLaunchProfiling.m  |  8 +--
 Sources/Sentry/SentryProfiler.mm              |  2 +-
 .../SentryAppLaunchProfilingTests.m           | 71 ++++++++++++-------
 .../Helper/SentryFileManager+Test.h           |  2 +
 .../SentryTests/SentryLaunchProfiling+Tests.h | 17 +++++
 Tests/SentryTests/SentrySDK+Tests.h           |  2 -
 7 files changed, 71 insertions(+), 39 deletions(-)
 create mode 100644 Tests/SentryTests/SentryLaunchProfiling+Tests.h

diff --git a/Sentry.xcodeproj/project.pbxproj b/Sentry.xcodeproj/project.pbxproj
index 8e23835c3a..ff9fee69cc 100644
--- a/Sentry.xcodeproj/project.pbxproj
+++ b/Sentry.xcodeproj/project.pbxproj
@@ -607,7 +607,7 @@
 		7DC8310A2398283C0043DD9A /* SentryCrashIntegration.h in Headers */ = {isa = PBXBuildFile; fileRef = 7DC831082398283C0043DD9A /* SentryCrashIntegration.h */; };
 		7DC8310C2398283C0043DD9A /* SentryCrashIntegration.m in Sources */ = {isa = PBXBuildFile; fileRef = 7DC831092398283C0043DD9A /* SentryCrashIntegration.m */; };
 		840A11122B61E27500650D02 /* SentrySamplerDecision.m in Sources */ = {isa = PBXBuildFile; fileRef = 840A11102B61E27500650D02 /* SentrySamplerDecision.m */; };
-		840A110A2B5F47F700650D02 /* SentryAppLaunchProfilingTests.m in Sources */ = {isa = PBXBuildFile; fileRef = 840A11092B5F47F700650D02 /* SentryAppLaunchProfilingTests.m */; };
+		840A11132B61FE5800650D02 /* SentryAppLaunchProfilingTests.m in Sources */ = {isa = PBXBuildFile; fileRef = 840A11092B5F47F700650D02 /* SentryAppLaunchProfilingTests.m */; };
 		84281C432A578E5600EE88F2 /* SentryProfilerState.mm in Sources */ = {isa = PBXBuildFile; fileRef = 84281C422A578E5600EE88F2 /* SentryProfilerState.mm */; };
 		84281C462A57905700EE88F2 /* SentrySample.h in Headers */ = {isa = PBXBuildFile; fileRef = 84281C442A57905700EE88F2 /* SentrySample.h */; };
 		84281C472A57905700EE88F2 /* SentrySample.m in Sources */ = {isa = PBXBuildFile; fileRef = 84281C452A57905700EE88F2 /* SentrySample.m */; };
@@ -1574,8 +1574,9 @@
 		7DC830FF239826280043DD9A /* SentryIntegrationProtocol.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = SentryIntegrationProtocol.h; path = Public/SentryIntegrationProtocol.h; sourceTree = "<group>"; };
 		7DC831082398283C0043DD9A /* SentryCrashIntegration.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = SentryCrashIntegration.h; path = include/SentryCrashIntegration.h; sourceTree = "<group>"; };
 		7DC831092398283C0043DD9A /* SentryCrashIntegration.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentryCrashIntegration.m; sourceTree = "<group>"; };
-		840A11102B61E27500650D02 /* SentrySamplerDecision.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentrySamplerDecision.m; sourceTree = "<group>"; };
 		840A11092B5F47F700650D02 /* SentryAppLaunchProfilingTests.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentryAppLaunchProfilingTests.m; sourceTree = "<group>"; };
+		840A11102B61E27500650D02 /* SentrySamplerDecision.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentrySamplerDecision.m; sourceTree = "<group>"; };
+		840A11142B62041600650D02 /* SentryLaunchProfiling+Tests.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = "SentryLaunchProfiling+Tests.h"; sourceTree = "<group>"; };
 		8419C0C328C1889D001C8259 /* SentryProfilerSwiftTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = SentryProfilerSwiftTests.swift; sourceTree = "<group>"; };
 		84281C422A578E5600EE88F2 /* SentryProfilerState.mm */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.objcpp; path = SentryProfilerState.mm; sourceTree = "<group>"; };
 		84281C442A57905700EE88F2 /* SentrySample.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = SentrySample.h; path = ../include/SentrySample.h; sourceTree = "<group>"; };
@@ -2300,6 +2301,7 @@
 				7B569DFE2590EEF600B653FC /* SentryScope+Equality.h */,
 				7B569E052590F04700B653FC /* SentryScope+Properties.h */,
 				7B9421C4260CA393001F9349 /* SentrySDK+Tests.h */,
+				840A11142B62041600650D02 /* SentryLaunchProfiling+Tests.h */,
 				639889D21EDF06C100EA7442 /* SentryTests-Bridging-Header.h */,
 				63B819131EC352A7002FDF4C /* SentryInterfacesTests.m */,
 				630C01931EC3402C00C52CEF /* SentryKSCrashReportConverterTests.m */,
@@ -4497,7 +4499,6 @@
 				0A9415BA28F96CAC006A5DD1 /* TestSentryReachability.swift in Sources */,
 				D880E3A728573E87008A90DB /* SentryBaggageTests.swift in Sources */,
 				7B16FD022654F86B008177D3 /* SentrySysctlTests.swift in Sources */,
-				840A110A2B5F47F700650D02 /* SentryAppLaunchProfilingTests.m in Sources */,
 				7BAF3DB5243C743E008A5414 /* SentryClientTests.swift in Sources */,
 				8EAE8E5E2681768000D6958B /* URLSessionTaskMock.m in Sources */,
 				D8CE69BC277E39C700C6EC5C /* SentryFileIOTrackingIntegrationObjCTests.m in Sources */,
@@ -4614,6 +4615,7 @@
 				8431EFE129B27B5300D8DC56 /* SentryThreadMetadataCacheTests.mm in Sources */,
 				8431EFE029B27B5300D8DC56 /* SentryBacktraceTests.mm in Sources */,
 				8431EFDF29B27B5300D8DC56 /* SentryThreadHandleTests.mm in Sources */,
+				840A11132B61FE5800650D02 /* SentryAppLaunchProfilingTests.m in Sources */,
 				8431EFE829B27BAD00D8DC56 /* SentrySystemWrapperTests.swift in Sources */,
 				8431EFE529B27BAD00D8DC56 /* SentryNSProcessInfoWrapperTests.swift in Sources */,
 				8431EFDE29B27B5300D8DC56 /* SentryProfilerSwiftTests.swift in Sources */,
diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index 0ba138c638..c121003bc9 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -1,4 +1,4 @@
-#import "SentryLaunchProfiling.h"
+#import "SentryLaunchProfiling+Tests.h"
 
 #if SENTRY_TARGET_PROFILING_SUPPORTED
 
@@ -25,12 +25,6 @@
 static NSString *const kSentryLaunchProfileConfigKeyTracesSampleRate = @"traces";
 static NSString *const kSentryLaunchProfileConfigKeyProfilesSampleRate = @"profiles";
 
-typedef struct {
-    BOOL shouldProfile;
-    SentrySamplerDecision *tracesDecision;
-    SentrySamplerDecision *profilesDecision;
-} SentryLaunchProfileConfig;
-
 #    pragma mark - Private
 
 SentryLaunchProfileConfig
diff --git a/Sources/Sentry/SentryProfiler.mm b/Sources/Sentry/SentryProfiler.mm
index a6e13a9090..2d960f7ae0 100644
--- a/Sources/Sentry/SentryProfiler.mm
+++ b/Sources/Sentry/SentryProfiler.mm
@@ -432,7 +432,7 @@ + (nullable SentryEnvelopeItem *)createEnvelopeItemForProfilePayload:
 {
     NSData *data = [SentrySerialization dataWithJSONObject:payload];
     NSFileManager *fm = [NSFileManager defaultManager];
-    NSString *appSupportDirPath = SentryFileManager.sentryApplicationSupportPath;
+    NSString *appSupportDirPath = sentryApplicationSupportPath();
 
     if (![fm fileExistsAtPath:appSupportDirPath]) {
         SENTRY_LOG_DEBUG(@"Creating app support directory.");
diff --git a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
index 3522bd00e2..4bce7f8016 100644
--- a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
+++ b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
@@ -1,3 +1,4 @@
+#import "SentryLaunchProfiling+Tests.h"
 #import "SentryOptions+HybridSDKs.h"
 #import "SentryOptions+Private.h"
 #import "SentryProfilingConditionals.h"
@@ -17,7 +18,7 @@ @implementation SentryAppLaunchProfilingTests
 
 - (void)testDefaultOptionsDoNotEnableLaunchProfiling
 {
-    XCTAssertFalse(shouldProfileNextLaunch([self defaultOptionsWithOverrides:nil]),
+    XCTAssertFalse(shouldProfileNextLaunch([self defaultOptionsWithOverrides:nil]).shouldProfile,
         @"Default options should not enable launch profiling");
 }
 
@@ -25,16 +26,19 @@ - (void)testAppLaunchProfilingNotSufficientToEnableLaunchProfiling
 {
     XCTAssertFalse(
         shouldProfileNextLaunch(
-            [self defaultOptionsWithOverrides:@{ SENTRY_OPTION(enableAppLaunchProfiling, @YES) }]),
+            [self defaultOptionsWithOverrides:@{ SENTRY_OPTION(enableAppLaunchProfiling, @YES) }])
+            .shouldProfile,
         @"Default options with only launch profiling option set is not sufficient to enable launch "
         @"profiling");
 }
 
 - (void)testAppLaunchProfilingAndTracingOptionsNotSufficientToEnableAppLaunchProfiling
 {
-    XCTAssertFalse(shouldProfileNextLaunch([self
-                       defaultOptionsWithOverrides:@{ SENTRY_OPTION(enableAppLaunchProfiling, @YES),
-                                                       SENTRY_OPTION(enableTracing, @YES) }]),
+    XCTAssertFalse(
+        shouldProfileNextLaunch(
+            [self defaultOptionsWithOverrides:@{ SENTRY_OPTION(enableAppLaunchProfiling, @YES),
+                                                  SENTRY_OPTION(enableTracing, @YES) }])
+            .shouldProfile,
         @"Default options with app launch profiling and tracing enabled are not sufficient to "
         @"enable launch profiling");
 }
@@ -42,17 +46,20 @@ - (void)testAppLaunchProfilingAndTracingOptionsNotSufficientToEnableAppLaunchPro
 - (void)
     testAppLaunchProfilingAndTracingAndTracesSampleRateOptionsNotSufficientToEnableAppLaunchProfiling
 {
-    XCTAssertFalse(shouldProfileNextLaunch([self
-                       defaultOptionsWithOverrides:@{ SENTRY_OPTION(enableAppLaunchProfiling, @YES),
-                                                       SENTRY_OPTION(enableTracing, @YES),
-                                                       SENTRY_OPTION(tracesSampleRate, @1) }]),
+    XCTAssertFalse(
+        shouldProfileNextLaunch(
+            [self defaultOptionsWithOverrides:@{ SENTRY_OPTION(enableAppLaunchProfiling, @YES),
+                                                  SENTRY_OPTION(enableTracing, @YES),
+                                                  SENTRY_OPTION(tracesSampleRate, @1) }])
+            .shouldProfile,
         @"Default options with app launch profiling and tracing enabled with traces sample rate of "
         @"1 are not sufficient to enable launch profiling");
 }
 
 - (void)testMinimumOptionsRequiredToEnableAppLaunchProfiling
 {
-    XCTAssert(shouldProfileNextLaunch([self defaultLaunchProfilingOptionsWithOverrides:nil]),
+    XCTAssert(shouldProfileNextLaunch([self defaultLaunchProfilingOptionsWithOverrides:nil])
+                  .shouldProfile,
         @"Default options with app launch profiling and tracing enabled and traces and profiles "
         @"sample rates of 1 should enable launch profiling");
 }
@@ -62,16 +69,18 @@ - (void)testDisablingLaunchProfilingOptionDisablesAppLaunchProfiling
     XCTAssertFalse(
         shouldProfileNextLaunch(
             [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
-                                                                 enableAppLaunchProfiling, @NO) }]),
+                                                                 enableAppLaunchProfiling, @NO) }])
+            .shouldProfile,
         @"Default options with tracing enabled, traces and profiles sample rates of 1, but app "
         @"launch profiling disabled should not enable launch profiling");
 }
 
 - (void)testDisablingTracingOptionDisablesAppLaunchProfiling
 {
-    XCTAssertFalse(
-        shouldProfileNextLaunch([self
-            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(enableTracing, @NO) }]),
+    XCTAssertFalse(shouldProfileNextLaunch(
+                       [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
+                                                                            enableTracing, @NO) }])
+                       .shouldProfile,
         @"Default options with app launch profiling enabled, traces and profiles sample rates of "
         @"1, but tracing disabled should not enable launch profiling");
 }
@@ -79,8 +88,10 @@ - (void)testDisablingTracingOptionDisablesAppLaunchProfiling
 - (void)testSettingTracesSampleRateTo0DisablesAppLaunchProfiling
 {
     XCTAssertFalse(
-        shouldProfileNextLaunch([self
-            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(tracesSampleRate, @0) }]),
+        shouldProfileNextLaunch(
+            [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
+                                                                 tracesSampleRate, @0) }])
+            .shouldProfile,
         @"Default options with app launch profiling and tracing enabled, profiles sample rate of "
         @"1, but traces sample rate of 0 should not enable launch profiling");
 }
@@ -88,8 +99,10 @@ - (void)testSettingTracesSampleRateTo0DisablesAppLaunchProfiling
 - (void)testSettingProfilesSampleRateTo0DisablesAppLaunchProfiling
 {
     XCTAssertFalse(
-        shouldProfileNextLaunch([self
-            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(profilesSampleRate, @0) }]),
+        shouldProfileNextLaunch(
+            [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
+                                                                 profilesSampleRate, @0) }])
+            .shouldProfile,
         @"Default options with app launch profiling and tracing enabled, traces sample rate of 1, "
         @"but profiles sample rate of 0 should not enable launch profiling");
 }
@@ -97,8 +110,10 @@ - (void)testSettingProfilesSampleRateTo0DisablesAppLaunchProfiling
 - (void)testDisablingSwizzlingOptionDisablesAppLaunchProfiling
 {
     XCTAssertFalse(
-        shouldProfileNextLaunch([self
-            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(enableSwizzling, @NO) }]),
+        shouldProfileNextLaunch(
+            [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
+                                                                 enableSwizzling, @NO) }])
+            .shouldProfile,
         @"Default options with app launch profiling and tracing enabled, traces and profiles "
         @"sample rates of 1, but swizzling disabled should not enable launch profiling");
 }
@@ -106,9 +121,11 @@ - (void)testDisablingSwizzlingOptionDisablesAppLaunchProfiling
 - (void)testDisablingAutoPerformanceTracingOptionDisablesAppLaunchProfiling
 {
     XCTAssertFalse(
-        shouldProfileNextLaunch([self
-            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
-                                                           enableAutoPerformanceTracing, @NO) }]),
+        shouldProfileNextLaunch(
+            [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
+                                                                 enableAutoPerformanceTracing,
+                                                                 @NO) }])
+            .shouldProfile,
         @"Default options with app launch profiling and tracing enabled, traces and profiles "
         @"sample rates of 1, but automatic performance tracing disabled should not enable launch "
         @"profiling");
@@ -117,9 +134,11 @@ - (void)testDisablingAutoPerformanceTracingOptionDisablesAppLaunchProfiling
 - (void)testDisablingUIViewControllerTracingOptionDisablesAppLaunchProfiling
 {
     XCTAssertFalse(
-        shouldProfileNextLaunch([self
-            defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
-                                                           enableUIViewControllerTracing, @NO) }]),
+        shouldProfileNextLaunch(
+            [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
+                                                                 enableUIViewControllerTracing,
+                                                                 @NO) }])
+            .shouldProfile,
         @"Default options with app launch profiling and tracing enabled, traces and profiles "
         @"sample rates of 1, but UIViewController tracing disabled should not enable launch "
         @"profiling");
diff --git a/Tests/SentryTests/Helper/SentryFileManager+Test.h b/Tests/SentryTests/Helper/SentryFileManager+Test.h
index b575d3464d..515c76b872 100644
--- a/Tests/SentryTests/Helper/SentryFileManager+Test.h
+++ b/Tests/SentryTests/Helper/SentryFileManager+Test.h
@@ -2,6 +2,8 @@
 
 NS_ASSUME_NONNULL_BEGIN
 
+SENTRY_EXTERN NSString *sentryApplicationSupportPath(void);
+
 @interface
 SentryFileManager ()
 
diff --git a/Tests/SentryTests/SentryLaunchProfiling+Tests.h b/Tests/SentryTests/SentryLaunchProfiling+Tests.h
new file mode 100644
index 0000000000..91997dc7ba
--- /dev/null
+++ b/Tests/SentryTests/SentryLaunchProfiling+Tests.h
@@ -0,0 +1,17 @@
+#import "SentryInternalDefines.h"
+#import "SentryLaunchProfiling.h"
+
+@class SentrySamplerDecision;
+@class SentryOptions;
+
+NS_ASSUME_NONNULL_BEGIN
+
+typedef struct {
+    BOOL shouldProfile;
+    SentrySamplerDecision *_Nullable tracesDecision;
+    SentrySamplerDecision *_Nullable profilesDecision;
+} SentryLaunchProfileConfig;
+
+SENTRY_EXTERN SentryLaunchProfileConfig shouldProfileNextLaunch(SentryOptions *options);
+
+NS_ASSUME_NONNULL_END
diff --git a/Tests/SentryTests/SentrySDK+Tests.h b/Tests/SentryTests/SentrySDK+Tests.h
index fcdddabdd7..2dab5e5c38 100644
--- a/Tests/SentryTests/SentrySDK+Tests.h
+++ b/Tests/SentryTests/SentrySDK+Tests.h
@@ -6,8 +6,6 @@
 
 NS_ASSUME_NONNULL_BEGIN
 
-SENTRY_EXTERN BOOL shouldProfileNextLaunch(SentryOptions *options);
-
 @interface
 SentrySDK ()
 

From 39efa27b742b5b25a3c605b737981274b1fc7048 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Tue, 30 Jan 2024 13:56:56 -0900
Subject: [PATCH 07/34] pr feedback: rename new var

---
 Samples/iOS-Swift/iOS-SwiftUITests/BaseUITest.swift       | 6 +++---
 Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/Samples/iOS-Swift/iOS-SwiftUITests/BaseUITest.swift b/Samples/iOS-Swift/iOS-SwiftUITests/BaseUITest.swift
index f80cd4373c..dfdedee18d 100644
--- a/Samples/iOS-Swift/iOS-SwiftUITests/BaseUITest.swift
+++ b/Samples/iOS-Swift/iOS-SwiftUITests/BaseUITest.swift
@@ -4,7 +4,7 @@ class BaseUITest: XCTestCase {
     internal lazy var app: XCUIApplication = newAppSession()
     
     //swiftlint:disable implicit_getter
-    var automaticallyManageAppSession: Bool { get { true } }
+    var automaticallyLaunchAndTerminateApp: Bool { get { true } }
     //swiftlint:enable implicit_getter
     
     override func setUp() {
@@ -12,13 +12,13 @@ class BaseUITest: XCTestCase {
         continueAfterFailure = false
         XCUIDevice.shared.orientation = .portrait
         app.launchEnvironment["io.sentry.sdk-environment"] = "ui-tests"
-        if automaticallyManageAppSession {
+        if automaticallyLaunchAndTerminateApp {
             launchApp()
         }
     }
     
     override func tearDown() {
-        if automaticallyManageAppSession {
+        if automaticallyLaunchAndTerminateApp {
             app.terminate()
         }
         super.tearDown()
diff --git a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
index dcac565e14..e526f56c0a 100644
--- a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
+++ b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
@@ -3,7 +3,7 @@ import XCTest
 //swiftlint:disable function_body_length todo
 
 class ProfilingUITests: BaseUITest {    
-    override var automaticallyManageAppSession: Bool { false }
+    override var automaticallyLaunchAndTerminateApp: Bool { false }
     
     // this will run before the non-async BaseUITest.setUp, so we can bail out before running any of the logic in there
     override func setUp() async throws {

From ca7127cd211f51df8d0b49f3c21e90381f6e349c Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Tue, 30 Jan 2024 13:58:39 -0900
Subject: [PATCH 08/34] pr feedback: iOS availability

---
 .../iOS-Swift/Profiling/ProfilingViewController.swift     | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
index 7c33832ce0..7909a0ca0d 100644
--- a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
+++ b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
@@ -1,6 +1,5 @@
 import UIKit
 
-@available(iOS 16.0, *)
 class ProfilingViewController: UIViewController, UITextFieldDelegate {
     
     @IBOutlet weak var workThreadLabel: UILabel!
@@ -120,11 +119,14 @@ class ProfilingViewController: UIViewController, UITextFieldDelegate {
 }
 
 // MARK: Private
-@available(iOS 16.0, *)
 extension ProfilingViewController {
     func withProfile(fileName: String, block: (URL?) -> Void) {
         let appSupportDirectory = NSSearchPathForDirectoriesInDomains(.applicationSupportDirectory, .userDomainMask, true).first!
-        for file in FileManager.default.enumerator(at: URL(filePath: appSupportDirectory), includingPropertiesForKeys: [URLResourceKey.nameKey])! {
+        guard let url = URL(string: appSupportDirectory) else {
+            block(nil)
+            return
+        }
+        for file in FileManager.default.enumerator(at: url, includingPropertiesForKeys: [URLResourceKey.nameKey])! {
             let url = file as! URL
             if url.absoluteString.contains(fileName) {
                 block(url)

From 435a9eb4cc433b9c2a1ccd53cb96a9ea6ca82fe2 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Tue, 30 Jan 2024 14:13:07 -0900
Subject: [PATCH 09/34] pr feedback: don't include the test interface extension

---
 Sources/Sentry/Profiling/SentryLaunchProfiling.m | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index c121003bc9..0f7fb74282 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -1,4 +1,4 @@
-#import "SentryLaunchProfiling+Tests.h"
+#import "SentryLaunchProfiling.h"
 
 #if SENTRY_TARGET_PROFILING_SUPPORTED
 
@@ -27,6 +27,12 @@
 
 #    pragma mark - Private
 
+typedef struct {
+    BOOL shouldProfile;
+    SentrySamplerDecision *_Nullable tracesDecision;
+    SentrySamplerDecision *_Nullable profilesDecision;
+} SentryLaunchProfileConfig;
+
 SentryLaunchProfileConfig
 shouldProfileNextLaunch(SentryOptions *options)
 {

From 0422c94045151a95ab609201910a94551a888f10 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Tue, 30 Jan 2024 14:13:35 -0900
Subject: [PATCH 10/34] fix macos build

---
 Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
index 4bce7f8016..2cd47f9d7b 100644
--- a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
+++ b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
@@ -131,6 +131,7 @@ - (void)testDisablingAutoPerformanceTracingOptionDisablesAppLaunchProfiling
         @"profiling");
 }
 
+#    if SENTRY_HAS_UIKIT
 - (void)testDisablingUIViewControllerTracingOptionDisablesAppLaunchProfiling
 {
     XCTAssertFalse(
@@ -143,6 +144,7 @@ - (void)testDisablingUIViewControllerTracingOptionDisablesAppLaunchProfiling
         @"sample rates of 1, but UIViewController tracing disabled should not enable launch "
         @"profiling");
 }
+#    endif // SENTRY_HAS_UIKIT
 
 - (SentryOptions *)defaultLaunchProfilingOptionsWithOverrides:
     (NSDictionary<NSString *, id> *)overrides

From 231d81647997fc1d7bd57ae753bfade9034ec79c Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Tue, 30 Jan 2024 14:16:52 -0900
Subject: [PATCH 11/34] pr feedback: move declaration from SentryFileManager
 test ext

---
 Sources/Sentry/SentryProfiler.mm                  | 2 +-
 Sources/Sentry/include/SentryFileManager.h        | 4 ++++
 Tests/SentryTests/Helper/SentryFileManager+Test.h | 3 ---
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/Sources/Sentry/SentryProfiler.mm b/Sources/Sentry/SentryProfiler.mm
index 2d960f7ae0..a743697ab2 100644
--- a/Sources/Sentry/SentryProfiler.mm
+++ b/Sources/Sentry/SentryProfiler.mm
@@ -51,7 +51,7 @@
 #    endif // SENTRY_HAS_UIKIT
 
 #    if defined(TEST) || defined(TESTCI)
-#        import "SentryFileManager+Test.h"
+#        import "SentryFileManager.h"
 #        import "SentryLaunchProfiling.h"
 
 @interface
diff --git a/Sources/Sentry/include/SentryFileManager.h b/Sources/Sentry/include/SentryFileManager.h
index be5b602d68..c2bd9ed3a4 100644
--- a/Sources/Sentry/include/SentryFileManager.h
+++ b/Sources/Sentry/include/SentryFileManager.h
@@ -14,6 +14,10 @@ NS_ASSUME_NONNULL_BEGIN
 @class SentryOptions;
 @class SentrySession;
 
+#if SENTRY_TARGET_PROFILING_SUPPORTED
+SENTRY_EXTERN NSString *sentryApplicationSupportPath(void);
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED
+
 NS_SWIFT_NAME(SentryFileManager)
 @interface SentryFileManager : NSObject
 SENTRY_NO_INIT
diff --git a/Tests/SentryTests/Helper/SentryFileManager+Test.h b/Tests/SentryTests/Helper/SentryFileManager+Test.h
index 515c76b872..e85d818f13 100644
--- a/Tests/SentryTests/Helper/SentryFileManager+Test.h
+++ b/Tests/SentryTests/Helper/SentryFileManager+Test.h
@@ -2,8 +2,6 @@
 
 NS_ASSUME_NONNULL_BEGIN
 
-SENTRY_EXTERN NSString *sentryApplicationSupportPath(void);
-
 @interface
 SentryFileManager ()
 
@@ -11,7 +9,6 @@ SentryFileManager ()
 @property (nonatomic, copy) NSString *envelopesPath;
 @property (nonatomic, copy) NSString *timezoneOffsetFilePath;
 
-+ (NSString *)sentryApplicationSupportPath;
 @end
 
 NS_ASSUME_NONNULL_END

From 17d006ec6e8062539db41cea7a575b38d0b2501e Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Tue, 30 Jan 2024 23:55:36 -0900
Subject: [PATCH 12/34] don't set isTracingAppLaunch to NO until after doing
 the work to package up the profile

this is primarily to fix the integration test, but if there is any
    logic in the profiler that relies on the value of that variable,
    it needs to be correct
---
 Sources/Sentry/SentryTracer.m | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Sources/Sentry/SentryTracer.m b/Sources/Sentry/SentryTracer.m
index 259478185c..7f51d77856 100644
--- a/Sources/Sentry/SentryTracer.m
+++ b/Sources/Sentry/SentryTracer.m
@@ -602,6 +602,8 @@ - (void)finishInternal
 
 #if SENTRY_TARGET_PROFILING_SUPPORTED
     if (self.isProfiling) {
+        [self captureTransactionWithProfile:transaction];
+
         // as long as this isn't used for any conditional branching logic, and is just being set to
         // NO, we don't need to synchronize the read here
         if (isTracingAppLaunch) {
@@ -609,7 +611,7 @@ - (void)finishInternal
                 isTracingAppLaunch = NO;
             }
         }
-        [self captureTransactionWithProfile:transaction];
+
         return;
     }
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED

From 4ef847be1a46771e8424e733bc8d0f5eb7ee3bbc Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Tue, 30 Jan 2024 23:56:50 -0900
Subject: [PATCH 13/34] only check for enableSwizzling in UIKit builds

    since the only thing we need that is swizzled is the automatic UIViewController tracking
---
 Sources/Sentry/Profiling/SentryLaunchProfiling.m | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index 0f7fb74282..23aa75a99d 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -37,11 +37,10 @@
 shouldProfileNextLaunch(SentryOptions *options)
 {
     BOOL shouldProfileNextLaunch = options.enableAppLaunchProfiling
-        && options.enableAutoPerformanceTracing
 #    if SENTRY_UIKIT_AVAILABLE
-        && options.enableUIViewControllerTracing
+        && options.enableUIViewControllerTracing && options.enableSwizzling
 #    endif // SENTRY_UIKIT_AVAILABLE
-        && options.enableSwizzling && options.enableTracing;
+        && options.enableAutoPerformanceTracing && options.enableTracing;
     if (!shouldProfileNextLaunch) {
 #    if SENTRY_UIKIT_AVAILABLE
         SENTRY_LOG_DEBUG(

From f715546c7354443cf0511e0aecb538c30ff58c3a Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 1 Feb 2024 16:38:11 -0900
Subject: [PATCH 14/34] test: manage files in sample app from integration test

---
 .../iOS-Swift.xcodeproj/project.pbxproj       |  6 +++++
 Samples/iOS-Swift/iOS-Swift/AppStartup.h      | 17 ++++++++++++
 Samples/iOS-Swift/iOS-Swift/AppStartup.m      | 26 +++++++++++++++++++
 .../Profiling/ProfilingViewController.swift   |  1 +
 4 files changed, 50 insertions(+)
 create mode 100644 Samples/iOS-Swift/iOS-Swift/AppStartup.h
 create mode 100644 Samples/iOS-Swift/iOS-Swift/AppStartup.m

diff --git a/Samples/iOS-Swift/iOS-Swift.xcodeproj/project.pbxproj b/Samples/iOS-Swift/iOS-Swift.xcodeproj/project.pbxproj
index 4f50f3ca2a..5afbfb896f 100644
--- a/Samples/iOS-Swift/iOS-Swift.xcodeproj/project.pbxproj
+++ b/Samples/iOS-Swift/iOS-Swift.xcodeproj/project.pbxproj
@@ -38,6 +38,7 @@
 		84B527BD28DD25E400475E8D /* SentryDevice.mm in Sources */ = {isa = PBXBuildFile; fileRef = 84B527BC28DD25E400475E8D /* SentryDevice.mm */; };
 		84BE546F287503F100ACC735 /* SentrySDKPerformanceBenchmarkTests.m in Sources */ = {isa = PBXBuildFile; fileRef = 84BE546E287503F100ACC735 /* SentrySDKPerformanceBenchmarkTests.m */; };
 		84BE547E287645B900ACC735 /* SentryProcessInfo.m in Sources */ = {isa = PBXBuildFile; fileRef = 84BE54792876451D00ACC735 /* SentryProcessInfo.m */; };
+		84DEE88E2B6A4D1200A7BC17 /* AppStartup.m in Sources */ = {isa = PBXBuildFile; fileRef = 84DEE88D2B6A4D1200A7BC17 /* AppStartup.m */; };
 		84FB812A284001B800F3A94A /* SentryBenchmarking.mm in Sources */ = {isa = PBXBuildFile; fileRef = 84FB8129284001B800F3A94A /* SentryBenchmarking.mm */; };
 		84FB812B284001B800F3A94A /* SentryBenchmarking.mm in Sources */ = {isa = PBXBuildFile; fileRef = 84FB8129284001B800F3A94A /* SentryBenchmarking.mm */; };
 		8E8C57AF25EF16E6001CEEFA /* TraceTestViewController.swift in Sources */ = {isa = PBXBuildFile; fileRef = 8E8C57AE25EF16E6001CEEFA /* TraceTestViewController.swift */; };
@@ -314,6 +315,8 @@
 		84BE546E287503F100ACC735 /* SentrySDKPerformanceBenchmarkTests.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentrySDKPerformanceBenchmarkTests.m; sourceTree = "<group>"; };
 		84BE54782876451D00ACC735 /* SentryProcessInfo.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = SentryProcessInfo.h; sourceTree = "<group>"; };
 		84BE54792876451D00ACC735 /* SentryProcessInfo.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentryProcessInfo.m; sourceTree = "<group>"; };
+		84DEE88C2B6A4D1200A7BC17 /* AppStartup.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = AppStartup.h; sourceTree = "<group>"; };
+		84DEE88D2B6A4D1200A7BC17 /* AppStartup.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = AppStartup.m; sourceTree = "<group>"; };
 		84FB8125284001B800F3A94A /* SentryBenchmarking.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = SentryBenchmarking.h; sourceTree = "<group>"; };
 		84FB8129284001B800F3A94A /* SentryBenchmarking.mm */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.objcpp; path = SentryBenchmarking.mm; sourceTree = "<group>"; };
 		84FB812C2840021B00F3A94A /* iOS-Swift-Bridging-Header.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = "iOS-Swift-Bridging-Header.h"; sourceTree = "<group>"; };
@@ -475,6 +478,8 @@
 				D8DBDA73274D4DF900007380 /* ViewControllers */,
 				63F93AA9245AC91600A500DB /* iOS-Swift.entitlements */,
 				637AFDA9243B02760034958B /* AppDelegate.swift */,
+				84DEE88C2B6A4D1200A7BC17 /* AppStartup.h */,
+				84DEE88D2B6A4D1200A7BC17 /* AppStartup.m */,
 				637AFDAD243B02760034958B /* TransactionsViewController.swift */,
 				84AB90782A50031B0054C99A /* Profiling */,
 				D80D021229EE93630084393D /* ErrorsViewController.swift */,
@@ -953,6 +958,7 @@
 				7B79000429028C7300A7F467 /* MetricKitManager.swift in Sources */,
 				D8D7BB4A2750067900044146 /* UIAssert.swift in Sources */,
 				D8F3D057274E574200B56F8C /* LoremIpsumViewController.swift in Sources */,
+				84DEE88E2B6A4D1200A7BC17 /* AppStartup.m in Sources */,
 				629EC8AD2B0B537400858855 /* ANRs.swift in Sources */,
 				D8DBDA78274D5FC400007380 /* SplitViewController.swift in Sources */,
 				84ACC43C2A73CB5900932A18 /* ProfilingNetworkScanner.swift in Sources */,
diff --git a/Samples/iOS-Swift/iOS-Swift/AppStartup.h b/Samples/iOS-Swift/iOS-Swift/AppStartup.h
new file mode 100644
index 0000000000..1b7bb41899
--- /dev/null
+++ b/Samples/iOS-Swift/iOS-Swift/AppStartup.h
@@ -0,0 +1,17 @@
+//
+//  AppStartup.h
+//  iOS-Swift
+//
+//  Created by Andrew McKnight on 1/31/24.
+//  Copyright © 2024 Sentry. All rights reserved.
+//
+
+#import <Foundation/Foundation.h>
+
+NS_ASSUME_NONNULL_BEGIN
+
+@interface AppStartup : NSObject
+
+@end
+
+NS_ASSUME_NONNULL_END
diff --git a/Samples/iOS-Swift/iOS-Swift/AppStartup.m b/Samples/iOS-Swift/iOS-Swift/AppStartup.m
new file mode 100644
index 0000000000..f3d7488e68
--- /dev/null
+++ b/Samples/iOS-Swift/iOS-Swift/AppStartup.m
@@ -0,0 +1,26 @@
+#import "AppStartup.h"
+
+@implementation AppStartup
+
+// we must do this in objective c, because it's not permitted to be overridden in Swift
++ (void)load
+{
+    if ([NSProcessInfo.processInfo.arguments containsObject:@"--io.sentry.wipe-data"]) {
+        NSLog(@"[iOS-Swift] removing app data");
+        NSString *appSupport = [NSSearchPathForDirectoriesInDomains(
+            NSApplicationSupportDirectory, NSUserDomainMask, true) firstObject];
+        NSString *cache = [NSSearchPathForDirectoriesInDomains(
+            NSCachesDirectory, NSUserDomainMask, true) firstObject];
+        NSFileManager *fm = NSFileManager.defaultManager;
+        for (NSString *dir in @[ appSupport, cache ]) {
+            for (NSString *file in [fm enumeratorAtPath:dir]) {
+                NSError *error;
+                if (![fm removeItemAtPath:[dir stringByAppendingPathComponent:file] error:&error]) {
+                    NSLog(@"[iOS-Swift] failed to remove data at app startup.");
+                }
+            }
+        }
+    }
+}
+
+@end
diff --git a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
index 7909a0ca0d..9d042d5937 100644
--- a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
+++ b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
@@ -130,6 +130,7 @@ extension ProfilingViewController {
             let url = file as! URL
             if url.absoluteString.contains(fileName) {
                 block(url)
+                try! FileManager.default.removeItem(at: url)
                 return
             }
         }

From 9c143a79672e54735bc072e1a7a9d017f09d2721 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 1 Feb 2024 16:42:18 -0900
Subject: [PATCH 15/34] test: integration test fixes and additions

---
 .../iOS-SwiftUITests/ProfilingUITests.swift   | 34 ++++++++++++++-----
 1 file changed, 25 insertions(+), 9 deletions(-)

diff --git a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
index e526f56c0a..dd78ba683e 100644
--- a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
+++ b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
@@ -12,6 +12,7 @@ class ProfilingUITests: BaseUITest {
     }
     
     func testProfiledAppLaunches() throws {
+        app.launchArguments.append("--io.sentry.wipe-data")
         launchApp()
         
         // First launch enables in-app profiling by setting traces/profiles sample rates to 1 (which is the default configuration in the sample app), but not launch profiling; assert that we did not write a config to allow the next launch to be profiled
@@ -21,12 +22,12 @@ class ProfilingUITests: BaseUITest {
         try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true)
         
         // this launch should run the profiler, then set the option to allow launch profiling to true, but set the numerical sample rates to 0 so that the next launch should not profile
-        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, profilesSampleRate: 0, tracesSampleRate: 0)
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: true, shouldEnableLaunchProfilingOptionForNextLaunch: true, profilesSampleRate: 0, tracesSampleRate: 0)
         
         // this launch should not run the profiler; configure sampler functions returning 1 and numerical rates set to 0, which should result in a profile being taken as samplers override numerical rates
         try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, profilesSampleRate: 0, tracesSampleRate: 0, profilesSamplerValue: 1, tracesSamplerValue: 1)
         
-        // this launch has the configuration to run the profiler, but because swizzling is disabled, it will not run due to the ui.load transaction not being allowed to be created but configure it not to run the next launch due to disabling swizzling, which would override the option to enable launch profiling
+        // this launch has the configuration to run the profiler, but because swizzling is disabled, it will not be saved due to the ui.load transaction not being allowed to be created. it will also configure it to not run a profile for the next launch due to disabling swizzling, which would override the option to enable launch profiling. this specific scenario, where a previous launch configures a profile, but then something prevents the associated tx from running, is not automatically avoidable. in the future we will create a dummy transaction to attach the profile to.
         try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, shouldDisableSwizzling: true)
         
         // this launch should not run the profiler and configure it not to run the next launch due to disabling automatic performance tracking, which would override the option to enable launch profiling
@@ -130,31 +131,46 @@ extension ProfilingUITests {
     func assertLaunchProfile() throws {
         retrieveLaunchProfileData()
         
-        var lastProfile = try marshalJSONDictionaryFromApp()
+        let lastProfile = try marshalJSONDictionaryFromApp()
         let sampledProfile = try XCTUnwrap(lastProfile["profile"] as? [String: Any])
         let stacks = try XCTUnwrap(sampledProfile["stacks"] as? [[Int]])
         let frames = try XCTUnwrap(sampledProfile["frames"] as? [[String: Any]])
-        let functions = stacks.map({ stack in
+        let stackFunctions = stacks.map({ stack in
             stack.map { stackFrame in
                 frames[stackFrame]["function"]
             }
         })
-        let stack = try XCTUnwrap(functions.first { nextStack in
-            let result = nextStack.contains { frame in
+        
+        // grab the first stack that contained frames from the fixture code that simulates a slow +[load] method
+        var stackID: Int?
+        let stack = try XCTUnwrap(stackFunctions.enumerated().first { nextStack in
+            let result = nextStack.element.contains { frame in
                 let result = (frame as! String).contains("+[SentryProfiler(SlowLoad) load]")
+                if result {
+                    stackID = nextStack.offset
+                }
                 return result
             }
             return result
-        }).map { any in
+        }).element.map { any in
             try XCTUnwrap(any as? String)
         }
+        guard stackID != nil else {
+            XCTFail("Didn't find the ID of the stack containing the target function")
+            return
+        }
         
+        // ensure that the stack doesn't contain any calls to main functions; this ensures we actually captured pre-main stacks
         XCTAssertFalse(stack.contains("main"))
         XCTAssertFalse(stack.contains("UIApplicationMain"))
         XCTAssertFalse(stack.contains("-[UIApplication _run]"))
         
-        // ???: can we assert that this stack was on the main thread?
-        // TODO: yes! need to correlate the samples.[].stack_id with samples.[].thread_id
+        // ensure that the stack happened on the main thread; this is a cross-check to make sure we didn't accidentally grab a stack from a different thread that wouldn't have had a call to main() anyways, thereby possibly missing the real stack that may have contained main() calls (but shouldn't for this test)
+        let samples = try XCTUnwrap(sampledProfile["samples"] as? [[String: Any]])
+        let sample = try XCTUnwrap(samples.first { nextSample in
+            try XCTUnwrap(nextSample["stack_id"] as? NSNumber).intValue == stackID
+        })
+        XCTAssert(try XCTUnwrap(sample["thread_id"] as? String) == "259") // the main thread is always ID 259
     }
      
     /**

From 7bc6c281d3195822b0d402b7c5447eccfcc6186f Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 1 Feb 2024 17:10:50 -0900
Subject: [PATCH 16/34] logging

---
 .../iOS-Swift/Profiling/ProfilingViewController.swift        | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
index 9d042d5937..ece70bdbbd 100644
--- a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
+++ b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
@@ -46,7 +46,7 @@ class ProfilingViewController: UIViewController, UITextFieldDelegate {
         let value = SentryBenchmarking.stopBenchmark()!
         valueTextField.isHidden = false
         valueTextField.text = value
-        print("[iOS-Swift] [Profiling] benchmarking results:\n\(value)")
+        print("[iOS-Swift] [ProfilingViewController] benchmarking results:\n\(value)")
     }
     
     @IBAction func startCPUWork(_ sender: UIButton) {
@@ -130,6 +130,7 @@ extension ProfilingViewController {
             let url = file as! URL
             if url.absoluteString.contains(fileName) {
                 block(url)
+                print("[iOS-Swift] [ProfilingViewController] removing file at \(url)")
                 try! FileManager.default.removeItem(at: url)
                 return
             }
@@ -149,7 +150,7 @@ extension ProfilingViewController {
             return
         }
         let contents = data.base64EncodedString()
-        print("[iOS-Swift] [Profiling] contents of file at \(file): \(String(data: data, encoding: .utf8))")
+        print("[iOS-Swift] [ProfilingViewController] contents of file at \(file): \(String(describing: String(data: data, encoding: .utf8)))")
         profilingUITestDataMarshalingTextField.text = contents
         profilingUITestDataMarshalingStatus.text = "✅"
     }

From 8b854eccab56e58a4174a75c8669e4f3b3d4db3e Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 1 Feb 2024 18:37:55 -0900
Subject: [PATCH 17/34] feat(app launch profiling): save sample rates config to
 backup

---
 .../Sentry/Profiling/SentryLaunchProfiling.m  |  9 ++-
 Sources/Sentry/SentryFileManager.m            | 64 +++++++++++++++++--
 Sources/Sentry/include/SentryFileManager.h    | 15 +++++
 3 files changed, 83 insertions(+), 5 deletions(-)

diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index 23aa75a99d..b349ee86b3 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -101,7 +101,11 @@
                 = config.profilesDecision.sampleRate;
             writeAppLaunchProfilingConfigFile(configDict);
         } else {
-            removeAppLaunchProfilingConfigFile();
+            if (isTracingAppLaunch) {
+                saveAppLaunchProfilingConfigFile();
+            } else {
+                removeAppLaunchProfilingConfigFile();
+            }
         }
     }];
 }
@@ -142,6 +146,9 @@
     SentryTransactionContext *transactionContext, SentryTracerConfiguration *configuration)
 {
     NSDictionary<NSString *, NSNumber *> *rates = appLaunchProfileConfiguration();
+    if (isTracingAppLaunch) {
+        removeAppLaunchProfilingConfigBackupFile();
+    }
     NSNumber *profilesRate = rates[kSentryLaunchProfileConfigKeyProfilesSampleRate];
     NSNumber *tracesRate = rates[kSentryLaunchProfileConfigKeyTracesSampleRate];
     if (profilesRate == nil || tracesRate == nil) {
diff --git a/Sources/Sentry/SentryFileManager.m b/Sources/Sentry/SentryFileManager.m
index cf189eeb24..6ee03a8deb 100644
--- a/Sources/Sentry/SentryFileManager.m
+++ b/Sources/Sentry/SentryFileManager.m
@@ -17,6 +17,10 @@
 #import "SentryOptions.h"
 #import "SentrySerialization.h"
 
+#if SENTRY_TARGET_PROFILING_SUPPORTED
+#    import "SentryLaunchProfiling.h"
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED
+
 NS_ASSUME_NONNULL_BEGIN
 
 NSString *const EnvelopesPathComponent = @"envelopes";
@@ -724,12 +728,32 @@ - (void)createPathsWithOptions:(SentryOptions *_Nonnull)options
     return sentryLaunchConfigFileURL;
 }
 
+NSURL *
+launchProfileConfigBackupFileURL(void)
+{
+    static NSURL *sentryLaunchConfigFileURL;
+    static dispatch_once_t onceToken;
+    dispatch_once(&onceToken, ^{
+        sentryLaunchConfigFileURL =
+            [[NSURL fileURLWithPath:[sentryApplicationSupportPath()
+                                        stringByAppendingPathComponent:@"profileLaunch"]]
+                URLByAppendingPathExtension:@"bak"];
+    });
+    return sentryLaunchConfigFileURL;
+}
+
 NSDictionary<NSString *, NSNumber *> *_Nullable appLaunchProfileConfiguration(void)
 {
     NSError *error;
-    NSDictionary<NSString *, NSNumber *> *config = [NSDictionary<NSString *, NSNumber *>
-        dictionaryWithContentsOfURL:launchProfileConfigFileURL()
-                              error:&error];
+    NSURL *configFileURL;
+    if (isTracingAppLaunch) {
+        configFileURL = launchProfileConfigBackupFileURL();
+    } else {
+        configFileURL = launchProfileConfigFileURL();
+    }
+    NSDictionary<NSString *, NSNumber *> *config =
+        [NSDictionary<NSString *, NSNumber *> dictionaryWithContentsOfURL:configFileURL
+                                                                    error:&error];
     SENTRY_CASSERT(
         error == nil, @"Encountered error trying to retrieve app launch profile config: %@", error);
     return config;
@@ -760,7 +784,39 @@ - (void)createPathsWithOptions:(SentryOptions *_Nonnull)options
 
     NSError *error;
     SENTRY_CASSERT([fm removeItemAtPath:path error:&error],
-        @"Failed to remove launch profile marker file: %@", error);
+        @"Failed to remove launch profile config file: %@", error);
+}
+
+void
+removeAppLaunchProfilingConfigBackupFile(void)
+{
+    NSFileManager *fm = [NSFileManager defaultManager];
+    NSString *path = launchProfileConfigBackupFileURL().path;
+    if (![fm fileExistsAtPath:path]) {
+        return;
+    }
+
+    NSError *error;
+    SENTRY_CASSERT([fm removeItemAtPath:path error:&error],
+        @"Failed to remove launch profile config file: %@", error);
+}
+
+void
+saveAppLaunchProfilingConfigFile(void)
+{
+    NSFileManager *fm = [NSFileManager defaultManager];
+    NSString *path = launchProfileConfigFileURL().path;
+    if (!SENTRY_CASSERT_RETURN([fm fileExistsAtPath:path],
+            @"Expect to have a current launch profile config to use for subsequent transaction.")) {
+        return;
+    }
+
+    NSError *error;
+    SENTRY_CASSERT([fm moveItemAtPath:path
+                               toPath:launchProfileConfigBackupFileURL().path
+                                error:&error],
+        @"Failed to backup launch profile config file for use to set up associated transaction: %@",
+        error);
 }
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
 
diff --git a/Sources/Sentry/include/SentryFileManager.h b/Sources/Sentry/include/SentryFileManager.h
index c2bd9ed3a4..74df53db09 100644
--- a/Sources/Sentry/include/SentryFileManager.h
+++ b/Sources/Sentry/include/SentryFileManager.h
@@ -119,6 +119,21 @@ SENTRY_EXTERN void writeAppLaunchProfilingConfigFile(
  * start the profiler.
  */
 SENTRY_EXTERN void removeAppLaunchProfilingConfigFile(void);
+
+/**
+ * Save a current launch profile config file for use when the associated transaction needs to be
+ * started. This is when checking @c SentryOptions for whether to write a config for the next
+ * launch; if there's no current app launch profile going, then we'd simply remove whatever config
+ * file may exist, but if there is a launch profile going, we need to save the config that it used.
+ */
+SENTRY_EXTERN void saveAppLaunchProfilingConfigFile(void);
+
+/*
+ * Clean up the backup file saved off in the case that an app launch is in-flight when the SDK
+ * configures subsequent launches not to profile, requiring the current config to be saved for the
+ * associated transaction. This can be cleaned up when that transaction is started.
+ */
+SENTRY_EXTERN void removeAppLaunchProfilingConfigBackupFile(void);
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
 
 @end

From 357449478f751f2664dd01495dcfddc07a31a939 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Mon, 5 Feb 2024 18:30:39 -0900
Subject: [PATCH 18/34] pr feedback: tests, edge cases and refactor/reuse file
 removal logic

---
 Sentry.xcodeproj/project.pbxproj              |   2 -
 .../Sentry/Profiling/SentryLaunchProfiling.m  |  10 +-
 Sources/Sentry/SentryFileManager.m            |  89 ++---
 Sources/Sentry/include/SentryFileManager.h    |   4 +-
 .../Sentry/include/SentryLaunchProfiling.h    |   2 +-
 .../Helper/SentryFileManager+Test.h           |   3 +
 .../Helper/SentryFileManagerTests.swift       | 359 ++++++++++++------
 .../SentryTests/SentryLaunchProfiling+Tests.h |   5 +-
 .../SentryTests/SentryTests-Bridging-Header.h |   1 +
 9 files changed, 308 insertions(+), 167 deletions(-)

diff --git a/Sentry.xcodeproj/project.pbxproj b/Sentry.xcodeproj/project.pbxproj
index 64db5e26c6..0365bc7dbd 100644
--- a/Sentry.xcodeproj/project.pbxproj
+++ b/Sentry.xcodeproj/project.pbxproj
@@ -626,7 +626,6 @@
 		8431EFE229B27BAD00D8DC56 /* SentryNSTimerFactoryTest.swift in Sources */ = {isa = PBXBuildFile; fileRef = 849472842971C41A002603DE /* SentryNSTimerFactoryTest.swift */; };
 		8431EFE529B27BAD00D8DC56 /* SentryNSProcessInfoWrapperTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 849472822971C2CD002603DE /* SentryNSProcessInfoWrapperTests.swift */; };
 		8431EFE829B27BAD00D8DC56 /* SentrySystemWrapperTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 849472802971C107002603DE /* SentrySystemWrapperTests.swift */; };
-		8431F00529B2849A00D8DC56 /* (null) in Sources */ = {isa = PBXBuildFile; };
 		8431F01529B2851500D8DC56 /* TestSentryNSTimerFactory.swift in Sources */ = {isa = PBXBuildFile; fileRef = 844EDCE72947DCD700C86F34 /* TestSentryNSTimerFactory.swift */; };
 		8431F01629B2851500D8DC56 /* TestSentryNSProcessInfoWrapper.swift in Sources */ = {isa = PBXBuildFile; fileRef = 844EDC712941442200C86F34 /* TestSentryNSProcessInfoWrapper.swift */; };
 		8431F01729B2851500D8DC56 /* TestSentrySystemWrapper.swift in Sources */ = {isa = PBXBuildFile; fileRef = 844EDC7829415AB300C86F34 /* TestSentrySystemWrapper.swift */; };
@@ -4519,7 +4518,6 @@
 				0A2D7BBA29152CBF008727AF /* SentryWatchdogTerminationsScopeObserverTests.swift in Sources */,
 				7BD4BD4B27EB2DC20071F4FF /* SentryDiscardedEventTests.swift in Sources */,
 				63FE721A20DA66EC00CDBAE8 /* SentryCrashSysCtl_Tests.m in Sources */,
-				8431F00529B2849A00D8DC56 /* (null) in Sources */,
 				7B88F30424BC8E6500ADF90A /* SentrySerializationTests.swift in Sources */,
 				7B34721728086A9D0041F047 /* SentrySwizzleWrapperTests.swift in Sources */,
 				8EC4CF5025C3A0070093DEE9 /* SentrySpanContextTests.swift in Sources */,
diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index b349ee86b3..6fd50c1bb7 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -22,8 +22,8 @@
 SentryId *_Nullable appLaunchTraceId;
 NSObject *appLaunchTraceLock;
 uint64_t appLaunchSystemTime;
-static NSString *const kSentryLaunchProfileConfigKeyTracesSampleRate = @"traces";
-static NSString *const kSentryLaunchProfileConfigKeyProfilesSampleRate = @"profiles";
+NSString *const kSentryLaunchProfileConfigKeyTracesSampleRate = @"traces";
+NSString *const kSentryLaunchProfileConfigKeyProfilesSampleRate = @"profiles";
 
 #    pragma mark - Private
 
@@ -102,7 +102,7 @@
             writeAppLaunchProfilingConfigFile(configDict);
         } else {
             if (isTracingAppLaunch) {
-                saveAppLaunchProfilingConfigFile();
+                backupAppLaunchProfilingConfigFile();
             } else {
                 removeAppLaunchProfilingConfigFile();
             }
@@ -146,9 +146,7 @@
     SentryTransactionContext *transactionContext, SentryTracerConfiguration *configuration)
 {
     NSDictionary<NSString *, NSNumber *> *rates = appLaunchProfileConfiguration();
-    if (isTracingAppLaunch) {
-        removeAppLaunchProfilingConfigBackupFile();
-    }
+    removeAppLaunchProfilingConfigBackupFile();
     NSNumber *profilesRate = rates[kSentryLaunchProfileConfigKeyProfilesSampleRate];
     NSNumber *tracesRate = rates[kSentryLaunchProfileConfigKeyTracesSampleRate];
     if (profilesRate == nil || tracesRate == nil) {
diff --git a/Sources/Sentry/SentryFileManager.m b/Sources/Sentry/SentryFileManager.m
index 4b5f8793c6..d85297663e 100644
--- a/Sources/Sentry/SentryFileManager.m
+++ b/Sources/Sentry/SentryFileManager.m
@@ -17,10 +17,6 @@
 #import "SentryOptions.h"
 #import "SentrySerialization.h"
 
-#if SENTRY_TARGET_PROFILING_SUPPORTED
-#    import "SentryLaunchProfiling.h"
-#endif // SENTRY_TARGET_PROFILING_SUPPORTED
-
 NS_ASSUME_NONNULL_BEGIN
 
 NSString *const EnvelopesPathComponent = @"envelopes";
@@ -40,6 +36,28 @@
     return YES;
 }
 
+/**
+ * @warning This is called from a `@synchronized` context in instance methods, but doesn't require
+ * that when calling from other static functions. Make sure you pay attention to where this is used
+ * from.
+ */
+void
+_unsafe_removeFileAtPath(NSString *path)
+{
+    NSError *error = nil;
+    NSFileManager *fileManager = [NSFileManager defaultManager];
+    if (![fileManager removeItemAtPath:path error:&error]) {
+        if (error.code == NSFileNoSuchFileError) {
+            SENTRY_LOG_DEBUG(@"No file to delete at %@", path);
+        } else {
+            SENTRY_LOG_ERROR(
+                @"Error occurred while deleting file at %@ because of %@", path, error);
+        }
+    } else {
+        SENTRY_LOG_DEBUG(@"Successfully deleted file at %@", path);
+    }
+}
+
 @interface
 SentryFileManager ()
 
@@ -288,19 +306,8 @@ - (void)deleteAllEnvelopes
 
 - (void)removeFileAtPath:(NSString *)path
 {
-    NSFileManager *fileManager = [NSFileManager defaultManager];
-    NSError *error = nil;
     @synchronized(self) {
-        if (![fileManager removeItemAtPath:path error:&error]) {
-            if (error.code == NSFileNoSuchFileError) {
-                SENTRY_LOG_DEBUG(@"No file to delete at %@", path);
-            } else {
-                SENTRY_LOG_ERROR(
-                    @"Error occurred while deleting file at %@ because of %@", path, error);
-            }
-        } else {
-            SENTRY_LOG_DEBUG(@"Successfully deleted file at %@", path);
-        }
+        _unsafe_removeFileAtPath(path);
     }
 }
 
@@ -771,16 +778,15 @@ - (void)createPathsWithOptions:(SentryOptions *)options
 
 NSDictionary<NSString *, NSNumber *> *_Nullable appLaunchProfileConfiguration(void)
 {
-    NSError *error;
-    NSURL *configFileURL;
-    if (isTracingAppLaunch) {
-        configFileURL = launchProfileConfigBackupFileURL();
-    } else {
-        configFileURL = launchProfileConfigFileURL();
+    NSURL *url = launchProfileConfigBackupFileURL();
+    if (![[NSFileManager defaultManager] fileExistsAtPath:url.path]) {
+        return nil;
     }
-    NSDictionary<NSString *, NSNumber *> *config =
-        [NSDictionary<NSString *, NSNumber *> dictionaryWithContentsOfURL:configFileURL
-                                                                    error:&error];
+
+    NSError *error;
+    NSDictionary<NSString *, NSNumber *> *config = [NSDictionary<NSString *, NSNumber *>
+        dictionaryWithContentsOfURL:launchProfileConfigBackupFileURL()
+                              error:&error];
     SENTRY_CASSERT(
         error == nil, @"Encountered error trying to retrieve app launch profile config: %@", error);
     return config;
@@ -803,45 +809,30 @@ - (void)createPathsWithOptions:(SentryOptions *)options
 void
 removeAppLaunchProfilingConfigFile(void)
 {
-    NSFileManager *fm = [NSFileManager defaultManager];
-    NSString *path = launchProfileConfigFileURL().path;
-    if (![fm fileExistsAtPath:path]) {
-        return;
-    }
-
-    NSError *error;
-    SENTRY_CASSERT([fm removeItemAtPath:path error:&error],
-        @"Failed to remove launch profile config file: %@", error);
+    _unsafe_removeFileAtPath(launchProfileConfigFileURL().path);
 }
 
 void
 removeAppLaunchProfilingConfigBackupFile(void)
 {
-    NSFileManager *fm = [NSFileManager defaultManager];
-    NSString *path = launchProfileConfigBackupFileURL().path;
-    if (![fm fileExistsAtPath:path]) {
-        return;
-    }
-
-    NSError *error;
-    SENTRY_CASSERT([fm removeItemAtPath:path error:&error],
-        @"Failed to remove launch profile config file: %@", error);
+    _unsafe_removeFileAtPath(launchProfileConfigBackupFileURL().path);
 }
 
 void
-saveAppLaunchProfilingConfigFile(void)
+backupAppLaunchProfilingConfigFile(void)
 {
     NSFileManager *fm = [NSFileManager defaultManager];
-    NSString *path = launchProfileConfigFileURL().path;
-    if (!SENTRY_CASSERT_RETURN([fm fileExistsAtPath:path],
+    NSString *fromPath = launchProfileConfigFileURL().path;
+    if (!SENTRY_CASSERT_RETURN([fm fileExistsAtPath:fromPath],
             @"Expect to have a current launch profile config to use for subsequent transaction.")) {
         return;
     }
 
+    NSString *toPath = launchProfileConfigBackupFileURL().path;
+    _unsafe_removeFileAtPath(toPath);
+
     NSError *error;
-    SENTRY_CASSERT([fm moveItemAtPath:path
-                               toPath:launchProfileConfigBackupFileURL().path
-                                error:&error],
+    SENTRY_CASSERT([fm moveItemAtPath:fromPath toPath:toPath error:&error],
         @"Failed to backup launch profile config file for use to set up associated transaction: %@",
         error);
 }
diff --git a/Sources/Sentry/include/SentryFileManager.h b/Sources/Sentry/include/SentryFileManager.h
index f4ca09bf5b..8d3322bf01 100644
--- a/Sources/Sentry/include/SentryFileManager.h
+++ b/Sources/Sentry/include/SentryFileManager.h
@@ -130,9 +130,9 @@ SENTRY_EXTERN void removeAppLaunchProfilingConfigFile(void);
  * launch; if there's no current app launch profile going, then we'd simply remove whatever config
  * file may exist, but if there is a launch profile going, we need to save the config that it used.
  */
-SENTRY_EXTERN void saveAppLaunchProfilingConfigFile(void);
+SENTRY_EXTERN void backupAppLaunchProfilingConfigFile(void);
 
-/*
+/**
  * Clean up the backup file saved off in the case that an app launch is in-flight when the SDK
  * configures subsequent launches not to profile, requiring the current config to be saved for the
  * associated transaction. This can be cleaned up when that transaction is started.
diff --git a/Sources/Sentry/include/SentryLaunchProfiling.h b/Sources/Sentry/include/SentryLaunchProfiling.h
index 1146a3482d..a67ea4d497 100644
--- a/Sources/Sentry/include/SentryLaunchProfiling.h
+++ b/Sources/Sentry/include/SentryLaunchProfiling.h
@@ -18,7 +18,7 @@ SENTRY_EXTERN NSObject *appLaunchTraceLock;
 
 void startLaunchProfile(void);
 
-/*
+/**
  * Write a file to disk containing sample rates for profiles and traces. The presence of this file
  * will let the profiler know to start on the app launch, and the sample rates contained will help
  * thread sampling decisions through to SentryHub later when it needs to start a transaction for the
diff --git a/Tests/SentryTests/Helper/SentryFileManager+Test.h b/Tests/SentryTests/Helper/SentryFileManager+Test.h
index e85d818f13..1285179f52 100644
--- a/Tests/SentryTests/Helper/SentryFileManager+Test.h
+++ b/Tests/SentryTests/Helper/SentryFileManager+Test.h
@@ -2,6 +2,9 @@
 
 NS_ASSUME_NONNULL_BEGIN
 
+SENTRY_EXTERN NSURL *launchProfileConfigFileURL(void);
+SENTRY_EXTERN NSURL *launchProfileConfigBackupFileURL(void);
+
 @interface
 SentryFileManager ()
 
diff --git a/Tests/SentryTests/Helper/SentryFileManagerTests.swift b/Tests/SentryTests/Helper/SentryFileManagerTests.swift
index 39fb13664a..4ead32d4e5 100644
--- a/Tests/SentryTests/Helper/SentryFileManagerTests.swift
+++ b/Tests/SentryTests/Helper/SentryFileManagerTests.swift
@@ -14,13 +14,13 @@ class SentryFileManagerTests: XCTestCase {
         let dispatchQueueWrapper: TestSentryDispatchQueueWrapper!
         
         let options: Options
-
+        
         let session = SentrySession(releaseName: "1.0.0", distinctId: "some-id")
         let sessionEnvelope: SentryEnvelope
-
+        
         let sessionUpdate: SentrySession
         let sessionUpdateEnvelope: SentryEnvelope
-
+        
         let expectedSessionUpdate: SentrySession
         
         // swiftlint:disable weak_delegate
@@ -32,23 +32,23 @@ class SentryFileManagerTests: XCTestCase {
         init() {
             currentDateProvider = TestCurrentDateProvider()
             dispatchQueueWrapper = TestSentryDispatchQueueWrapper()
-
+            
             eventIds = (0...(maxCacheItems + 10)).map { _ in SentryId() }
             
             options = Options()
             options.dsn = TestConstants.dsnAsString(username: "SentryFileManagerTests")
             
             sessionEnvelope = SentryEnvelope(session: session)
-
+            
             let sessionCopy = session.copy() as! SentrySession
             sessionCopy.incrementErrors()
             // We need to serialize in order to set the timestamp and the duration
             sessionUpdate = SentrySession(jsonObject: sessionCopy.serialize())!
-
+            
             let event = Event()
             let items = [SentryEnvelopeItem(session: sessionUpdate), SentryEnvelopeItem(event: event)]
             sessionUpdateEnvelope = SentryEnvelope(id: event.eventId, items: items)
-
+            
             let sessionUpdateCopy = sessionUpdate.copy() as! SentrySession
             // We need to serialize in order to set the timestamp and the duration
             expectedSessionUpdate = SentrySession(jsonObject: sessionUpdateCopy.serialize())!
@@ -70,19 +70,19 @@ class SentryFileManagerTests: XCTestCase {
             sut.setDelegate(delegate)
             return sut
         }
-
+        
     }
-
+    
     private var fixture: Fixture!
     private var sut: SentryFileManager!
-
+    
     override func setUp() {
         super.setUp()
         fixture = Fixture()
         SentryDependencyContainer.sharedInstance().dateProvider = fixture.currentDateProvider
-
+        
         sut = fixture.getSut()
-
+        
         sut.deleteAllEnvelopes()
         sut.deleteTimestampLastInForeground()
     }
@@ -102,10 +102,10 @@ class SentryFileManagerTests: XCTestCase {
         sut.store(TestConstants.envelope)
         sut.storeCurrentSession(SentrySession(releaseName: "1.0.0", distinctId: "some-id"))
         sut.storeTimestampLast(inForeground: Date())
-
+        
         _ = try! SentryFileManager(options: fixture.options, dispatchQueueWrapper: TestSentryDispatchQueueWrapper())
         let fileManager = try! SentryFileManager(options: fixture.options, dispatchQueueWrapper: TestSentryDispatchQueueWrapper())
-
+        
         XCTAssertEqual(1, fileManager.getAllEnvelopes().count)
         XCTAssertNotNil(fileManager.readCurrentSession())
         XCTAssertNotNil(fileManager.readTimestampLastInForeground())
@@ -122,7 +122,7 @@ class SentryFileManagerTests: XCTestCase {
     func testInitDoesntCreateEventsFolder() {
         assertEventFolderDoesntExist()
     }
-
+    
     func testStoreEnvelope() throws {
         let envelope = TestConstants.envelope
         sut.store(envelope)
@@ -135,13 +135,13 @@ class SentryFileManagerTests: XCTestCase {
         let actualData = envelopes[0].contents
         XCTAssertEqual(expectedData, actualData as Data)
     }
-
+    
     func testDeleteOldEnvelopes() throws {
         try givenOldEnvelopes()
-
+        
         sut = fixture.getSut()
         sut.deleteOldEnvelopeItems()
-
+        
         XCTAssertEqual(sut.getAllEnvelopes().count, 0)
     }
     
@@ -214,39 +214,39 @@ class SentryFileManagerTests: XCTestCase {
         sut.deleteOldEnvelopeItems()
         
         try givenOldEnvelopes()
-
+        
         sut.deleteOldEnvelopeItems()
-
+        
         XCTAssertEqual(sut.getAllEnvelopes().count, 0)
     }
-
+    
     func testDontDeleteYoungEnvelopes() throws {
         let envelope = TestConstants.envelope
         let path = sut.store(envelope)
-
+        
         let timeIntervalSince1970 = fixture.currentDateProvider.date().timeIntervalSince1970 - (90 * 24 * 60 * 60)
         let date = Date(timeIntervalSince1970: timeIntervalSince1970)
         try FileManager.default.setAttributes([FileAttributeKey.creationDate: date], ofItemAtPath: path)
-
+        
         XCTAssertEqual(sut.getAllEnvelopes().count, 1)
-
+        
         sut = fixture.getSut()
-
+        
         XCTAssertEqual(sut.getAllEnvelopes().count, 1)
     }
-
+    
     func testDontDeleteYoungEnvelopesFromOldEnvelopesFolder() throws {
         let envelope = TestConstants.envelope
         sut.store(envelope)
-
+        
         let timeIntervalSince1970 = fixture.currentDateProvider.date().timeIntervalSince1970 - (90 * 24 * 60 * 60)
         let date = Date(timeIntervalSince1970: timeIntervalSince1970)
         try FileManager.default.setAttributes([FileAttributeKey.creationDate: date], ofItemAtPath: sut.envelopesPath)
-
+        
         XCTAssertEqual(sut.getAllEnvelopes().count, 1)
-
+        
         sut = fixture.getSut()
-
+        
         XCTAssertEqual(sut.getAllEnvelopes().count, 1)
     }
     
@@ -254,7 +254,7 @@ class SentryFileManagerTests: XCTestCase {
         try givenOldEnvelopes()
         
         fixture.dispatchQueueWrapper.dispatchAsyncExecutesBlock = false
-
+        
         // Initialize sut in extra function so ARC deallocates it
         func getSut() {
             _ = fixture.getSut()
@@ -265,7 +265,7 @@ class SentryFileManagerTests: XCTestCase {
         
         XCTAssertEqual(sut.getAllEnvelopes().count, 1)
     }
-
+    
     func testCreateDirDoesNotThrow() throws {
         try SentryFileManager.createDirectory(atPath: "a")
     }
@@ -276,12 +276,12 @@ class SentryFileManagerTests: XCTestCase {
         sut.removeFile(atPath: "x")
         XCTAssertFalse(logOutput.loggedMessages.contains(where: { $0.contains("[error]") }))
     }
-
+    
     func testDefaultMaxEnvelopes() {
         for _ in 0...(fixture.maxCacheItems + 1) {
             sut.store(TestConstants.envelope)
         }
-
+        
         let events = sut.getAllEnvelopes()
         XCTAssertEqual(fixture.maxCacheItems, events.count)
     }
@@ -302,7 +302,7 @@ class SentryFileManagerTests: XCTestCase {
         let expected: [SentryDataCategory] = [.error, .attachment, .session, .error]
         XCTAssertEqual(expected, fixture.delegate.envelopeItemsDeleted.invocations)
     }
-
+    
     func testDefaultMaxEnvelopesConcurrent() {
         let maxCacheItems = 1
         let sut = fixture.getSut(maxCacheItems: UInt(maxCacheItems))
@@ -323,7 +323,7 @@ class SentryFileManagerTests: XCTestCase {
         queue.activate()
         
         wait(for: [envelopeStoredExpectation], timeout: 10)
-
+        
         let events = sut.getAllEnvelopes()
         XCTAssertEqual(maxCacheItems, events.count)
     }
@@ -337,7 +337,7 @@ class SentryFileManagerTests: XCTestCase {
         let events = sut.getAllEnvelopes()
         XCTAssertEqual(maxCacheItems, UInt(events.count))
     }
-
+    
     func testMigrateSessionInit_SessionUpdateIsLast() {
         sut.store(fixture.sessionEnvelope)
         // just some other session
@@ -346,22 +346,22 @@ class SentryFileManagerTests: XCTestCase {
             sut.store(TestConstants.envelope)
         }
         sut.store(fixture.sessionUpdateEnvelope)
-
+        
         assertSessionInitMoved(sut.getAllEnvelopes().last!)
         assertSessionEnvelopesStored(count: 2)
     }
-
+    
     func testMigrateSessionInit_SessionUpdateIsSecond() {
         sut.store(fixture.sessionEnvelope)
         sut.store(fixture.sessionUpdateEnvelope)
         for _ in 0...(fixture.maxCacheItems - 2) {
             sut.store(TestConstants.envelope)
         }
-
+        
         assertSessionInitMoved(sut.getAllEnvelopes().first!)
         assertSessionEnvelopesStored(count: 1)
     }
-
+    
     func testMigrateSessionInit_IsInMiddle() {
         sut.store(fixture.sessionEnvelope)
         for _ in 0...10 {
@@ -371,7 +371,7 @@ class SentryFileManagerTests: XCTestCase {
         for _ in 0...18 {
             sut.store(TestConstants.envelope)
         }
-
+        
         assertSessionInitMoved(sut.getAllEnvelopes()[10])
         assertSessionEnvelopesStored(count: 1)
     }
@@ -387,20 +387,20 @@ class SentryFileManagerTests: XCTestCase {
         for _ in 0...16 {
             sut.store(TestConstants.envelope)
         }
-
+        
         assertSessionInitMoved(sut.getAllEnvelopes()[10])
         assertSessionInitNotMoved(sut.getAllEnvelopes()[11])
         assertSessionInitNotMoved(sut.getAllEnvelopes()[12])
         assertSessionEnvelopesStored(count: 3)
     }
-
+    
     func testMigrateSessionInit_NoOtherSessionUpdate() {
         sut.store(fixture.sessionEnvelope)
         sut.store(fixture.sessionUpdateEnvelope)
         for _ in 0...(fixture.maxCacheItems - 1) {
             sut.store(TestConstants.envelope)
         }
-
+        
         assertSessionEnvelopesStored(count: 0)
     }
     
@@ -418,7 +418,7 @@ class SentryFileManagerTests: XCTestCase {
         try! fileManager.createDirectory(atPath: envelopePath, withIntermediateDirectories: false, attributes: nil)
         
         sut.store(fixture.sessionUpdateEnvelope)
-
+        
         assertSessionInitMoved(sut.getAllEnvelopes().last!)
     }
     
@@ -428,18 +428,18 @@ class SentryFileManagerTests: XCTestCase {
         for _ in 0...(fixture.maxCacheItems - 2) {
             sut.store(TestConstants.envelope)
         }
-
+        
         XCTAssertEqual(0, fixture.delegate.envelopeItemsDeleted.count)
     }
-
+    
     func testGetAllEnvelopesAreSortedByDateAscending() {
         givenMaximumEnvelopes()
-
+        
         let envelopes = sut.getAllEnvelopes()
-
+        
         // Envelopes are sorted ascending by date and only the latest amount of maxCacheItems are kept
         let expectedEventIds = Array(fixture.eventIds[11..<fixture.eventIds.count])
-
+        
         XCTAssertEqual(fixture.maxCacheItems, envelopes.count)
         for i in 0..<fixture.maxCacheItems {
             let envelope = SentrySerialization.envelope(with: envelopes[i].contents)
@@ -447,26 +447,26 @@ class SentryFileManagerTests: XCTestCase {
             XCTAssertEqual(expectedEventIds[i], actualEventId)
         }
     }
-
+    
     func testGetOldestEnvelope() {
         givenMaximumEnvelopes()
-
+        
         let actualEnvelope = SentrySerialization.envelope(with: sut.getOldestEnvelope()?.contents ?? Data())
-
+        
         XCTAssertEqual(fixture.eventIds[11], actualEnvelope?.header.eventId)
     }
-
+    
     func testGetOldestEnvelope_WhenNoEnvelopes() {
         XCTAssertNil(sut.getOldestEnvelope())
     }
-
+    
     func testGetOldestEnvelope_WithGarbageInEnvelopesFolder() {
         givenGarbageInEnvelopesFolder()
-
+        
         let actualEnvelope = SentrySerialization.envelope(with: sut.getOldestEnvelope()?.contents ?? Data())
         XCTAssertNil(actualEnvelope)
     }
-
+    
     func testStoreAndReadCurrentSession() {
         let expectedSession = SentrySession(releaseName: "1.0.0", distinctId: "some-id")
         sut.storeCurrentSession(expectedSession)
@@ -481,7 +481,7 @@ class SentryFileManagerTests: XCTestCase {
         let actualSession = sut.readCrashedSession()
         XCTAssertTrue(expectedSession.distinctId == actualSession?.distinctId)
     }
-
+    
     func testStoreDeleteCurrentSession() {
         sut.storeCurrentSession(SentrySession(releaseName: "1.0.0", distinctId: "some-id"))
         sut.deleteCurrentSession()
@@ -495,14 +495,14 @@ class SentryFileManagerTests: XCTestCase {
         let actualSession = sut.readCrashedSession()
         XCTAssertNil(actualSession)
     }
-
+    
     func testStoreAndReadTimestampLastInForeground() {
         let expectedTimestamp = TestCurrentDateProvider().date()
         sut.storeTimestampLast(inForeground: expectedTimestamp)
         let actualTimestamp = sut.readTimestampLastInForeground()
         XCTAssertEqual(actualTimestamp, expectedTimestamp)
     }
-
+    
     func testStoreDeleteTimestampLastInForeground() {
         sut.storeTimestampLast(inForeground: Date())
         sut.deleteTimestampLastInForeground()
@@ -513,7 +513,7 @@ class SentryFileManagerTests: XCTestCase {
     func testGetAllStoredEventsAndEnvelopes() {
         sut.store(TestConstants.envelope)
         sut.store(TestConstants.envelope)
-
+        
         XCTAssertEqual(2, sut.getAllEnvelopes().count)
     }
     
@@ -523,7 +523,7 @@ class SentryFileManagerTests: XCTestCase {
         sut.storeCurrentSession(SentrySession(releaseName: "1.0.1", distinctId: "some-id"))
         
         sut.deleteAllFolders()
-
+        
         XCTAssertEqual(0, sut.getAllEnvelopes().count)
         XCTAssertNil(sut.readCurrentSession())
         assertEventFolderDoesntExist()
@@ -533,7 +533,7 @@ class SentryFileManagerTests: XCTestCase {
         sut.store(TestConstants.envelope)
         
         sut.deleteAllEnvelopes()
-
+        
         XCTAssertEqual(0, sut.getAllEnvelopes().count)
     }
     
@@ -561,12 +561,12 @@ class SentryFileManagerTests: XCTestCase {
         sut.deleteAppState()
         XCTAssertNil(sut.readAppState())
     }
-
+    
     func testDeletePreviousAppState() {
         sut.store(TestData.appState)
         sut.moveAppStateToPreviousAppState()
         sut.deleteAppState()
-
+        
         XCTAssertNil(sut.readAppState())
         XCTAssertNil(sut.readPreviousAppState())
     }
@@ -601,58 +601,58 @@ class SentryFileManagerTests: XCTestCase {
         sut.deleteAppState()
         XCTAssertNotNil(sut.readAppState())
     }
-
+    
     func testMoveAppStateAndReadPreviousAppState() {
         sut.store(TestData.appState)
         sut.moveAppStateToPreviousAppState()
-
+        
         let actual = sut.readPreviousAppState()
         XCTAssertEqual(TestData.appState, actual)
     }
-
+    
     func testMoveAppStateWhenPreviousAppStateAlreadyExists() {
         sut.store(TestData.appState)
         sut.moveAppStateToPreviousAppState()
-
+        
         let newAppState = SentryAppState(releaseName: "2.0.0", osVersion: "14.4.1", vendorId: "12345678-1234-1234-1234-12344567890AB", isDebugging: false, systemBootTimestamp: Date(timeIntervalSince1970: 10))
         sut.store(newAppState)
         sut.moveAppStateToPreviousAppState()
-
+        
         let actual = sut.readPreviousAppState()
         XCTAssertEqual(newAppState, actual)
     }
-
+    
     func testStoreAndReadTimezoneOffset() {
         sut.storeTimezoneOffset(7_200)
         XCTAssertEqual(sut.readTimezoneOffset(), 7_200)
     }
-
+    
     func testtestStoreAndReadNegativeTimezoneOffset() {
         sut.storeTimezoneOffset(-1_000)
         XCTAssertEqual(sut.readTimezoneOffset(), -1_000)
     }
-
+    
     func testStoreDeleteTimezoneOffset() {
         sut.storeTimezoneOffset(7_200)
         sut.deleteTimezoneOffset()
         XCTAssertNil(sut.readTimezoneOffset())
     }
-
+    
     func testStore_WhenFileImmutable_TimezoneOffsetIsNotOverwritten() {
         sut.storeTimezoneOffset(7_200)
-
+        
         setImmutableForTimezoneOffset(immutable: true)
-
+        
         sut.storeTimezoneOffset(9_600)
-
+        
         XCTAssertEqual(sut.readTimezoneOffset(), 7_200)
     }
-
+    
     func testStoreDeleteTimezoneOffset_WhenFileLocked_DontCrash() throws {
         sut.storeTimezoneOffset(7_200)
-
+        
         setImmutableForTimezoneOffset(immutable: true)
-
+        
         sut.deleteTimezoneOffset()
         XCTAssertNotNil(sut.readTimezoneOffset())
     }
@@ -663,20 +663,20 @@ class SentryFileManagerTests: XCTestCase {
         
         XCTAssertTrue(sut.sentryPath.hasPrefix("/var/tmp/io.sentry"))
     }
-
-    #if os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
-
+    
+#if os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
+    
     func testReadPreviousBreadcrumbs() {
         let observer = SentryWatchdogTerminationScopeObserver(maxBreadcrumbs: 2, fileManager: sut)
-
+        
         for count in 0..<3 {
             let crumb = TestData.crumb
             crumb.message = "\(count)"
             let serializedBreadcrumb = crumb.serialize()
-
+            
             observer.addSerializedBreadcrumb(serializedBreadcrumb)
         }
-
+        
         sut.moveBreadcrumbsToPreviousBreadcrumbs()
         let result = sut.readPreviousBreadcrumbs()
         XCTAssertEqual(result.count, 3)
@@ -684,18 +684,18 @@ class SentryFileManagerTests: XCTestCase {
         XCTAssertEqual((result[1] as! NSDictionary)["message"] as! String, "1")
         XCTAssertEqual((result[2] as! NSDictionary)["message"] as! String, "2")
     }
-
+    
     func testReadPreviousBreadcrumbsCorrectOrderWhenFileTwoHasMoreCrumbs() {
         let observer = SentryWatchdogTerminationScopeObserver(maxBreadcrumbs: 2, fileManager: sut)
-
+        
         for count in 0..<5 {
             let crumb = TestData.crumb
             crumb.message = "\(count)"
             let serializedBreadcrumb = crumb.serialize()
-
+            
             observer.addSerializedBreadcrumb(serializedBreadcrumb)
         }
-
+        
         sut.moveBreadcrumbsToPreviousBreadcrumbs()
         let result = sut.readPreviousBreadcrumbs()
         XCTAssertEqual(result.count, 3)
@@ -703,15 +703,162 @@ class SentryFileManagerTests: XCTestCase {
         XCTAssertEqual((result[1] as! NSDictionary)["message"] as! String, "3")
         XCTAssertEqual((result[2] as! NSDictionary)["message"] as! String, "4")
     }
-
+    
 #endif // os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
-
+    
     func testReadGarbageTimezoneOffset() throws {
         try "garbage".write(to: URL(fileURLWithPath: sut.timezoneOffsetFilePath), atomically: true, encoding: .utf8)
         XCTAssertNil(sut.readTimezoneOffset())
     }
+}
+
+#if os(iOS) || os(macOS) || targetEnvironment(macCatalyst)
+// MARK: App Launch profiling tests
+extension SentryFileManagerTests {
+    // if app launch profiling was configured to take place
+    func testAppLaunchProfileConfigFileExists_fileExists() throws {
+        try ensureAppLaunchProfileConfig()
+        expect(appLaunchProfileConfigFileExists()) == true
+    }
+    
+    // if app launch profiling was not configured to take place
+    func testAppLaunchProfileConfigFileExists_fileDoesNotExist() throws {
+        try ensureAppLaunchProfileConfig(exists: false)
+        expect(appLaunchProfileConfigFileExists()) == false
+    }
+    
+    func testAppLaunchProfileConfiguration() throws {
+        let expectedTracesSampleRate = 0.12
+        let expectedProfilesSampleRate = 0.34
+        try ensureAppLaunchProfileConfig(tracesSampleRate: expectedTracesSampleRate, profilesSampleRate: expectedProfilesSampleRate, backup: true)
+        let config = appLaunchProfileConfiguration()
+        let actualTracesSampleRate = try XCTUnwrap(config?[kSentryLaunchProfileConfigKeyTracesSampleRate]).doubleValue
+        let actualProfilesSampleRate = try XCTUnwrap(config?[kSentryLaunchProfileConfigKeyProfilesSampleRate]).doubleValue
+        expect(actualTracesSampleRate) == expectedTracesSampleRate
+        expect(actualProfilesSampleRate) == expectedProfilesSampleRate
+    }
+    
+    // if a file isn't present when we expect it to be, like if there was an issue when we went to write it to disk
+    func testAppLaunchProfileConfiguration_noConfigurationExists() throws {
+        try ensureAppLaunchProfileConfig(exists: false, backup: true)
+        expect(appLaunchProfileConfiguration()) == nil
+    }
+    
+    func testWriteAppLaunchProfilingConfigFile_noCurrentFileExists() throws {
+        try ensureAppLaunchProfileConfig(exists: false)
+        
+        let expectedTracesSampleRate = 0.12
+        let expectedProfilesSampleRate = 0.34
+        writeAppLaunchProfilingConfigFile([
+            kSentryLaunchProfileConfigKeyTracesSampleRate: expectedTracesSampleRate,
+            kSentryLaunchProfileConfigKeyProfilesSampleRate: expectedProfilesSampleRate
+        ])
+        
+        let config = NSDictionary(contentsOf: launchProfileConfigFileURL())
+        
+        let actualTracesSampleRate = try XCTUnwrap(config?[kSentryLaunchProfileConfigKeyTracesSampleRate] as? NSNumber).doubleValue
+        let actualProfilesSampleRate = try XCTUnwrap(config?[kSentryLaunchProfileConfigKeyProfilesSampleRate] as? NSNumber).doubleValue
+        expect(actualTracesSampleRate) == expectedTracesSampleRate
+        expect(actualProfilesSampleRate) == expectedProfilesSampleRate
+    }
+    
+    // if a file is still present in the primary location, like if a crash occurred before it could be removed, or an error occurred when trying to remove it or move it to the backup location, make sure we overwrite it
+    func testWriteAppLaunchProfilingConfigFile_fileAlreadyExists() throws {
+        try ensureAppLaunchProfileConfig(exists: true, tracesSampleRate: 0.75, profilesSampleRate: 0.75)
+        
+        let expectedTracesSampleRate = 0.12
+        let expectedProfilesSampleRate = 0.34
+        writeAppLaunchProfilingConfigFile([
+            kSentryLaunchProfileConfigKeyTracesSampleRate: expectedTracesSampleRate,
+            kSentryLaunchProfileConfigKeyProfilesSampleRate: expectedProfilesSampleRate
+        ])
+        
+        let config = NSDictionary(contentsOf: launchProfileConfigFileURL())
+        
+        let actualTracesSampleRate = try XCTUnwrap(config?[kSentryLaunchProfileConfigKeyTracesSampleRate] as? NSNumber).doubleValue
+        let actualProfilesSampleRate = try XCTUnwrap(config?[kSentryLaunchProfileConfigKeyProfilesSampleRate] as? NSNumber).doubleValue
+        expect(actualTracesSampleRate) == expectedTracesSampleRate
+        expect(actualProfilesSampleRate) == expectedProfilesSampleRate
+    }
+    
+    func testRemoveAppLaunchProfilingConfigFile() throws {
+        try ensureAppLaunchProfileConfig(exists: true)
+        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) != nil
+        removeAppLaunchProfilingConfigFile()
+        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
+    }
+    
+    // if there's not a file when we expect one, just make sure we don't crash
+    func testRemoveAppLaunchProfilingConfigFile_noFileExists() throws {
+        try ensureAppLaunchProfileConfig(exists: false)
+        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
+        removeAppLaunchProfilingConfigFile()
+        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
+    }
+    
+    func testBackupAppLaunchProfilingConfigFile() throws {
+        try ensureAppLaunchProfileConfig(exists: true)
+        try ensureAppLaunchProfileConfig(exists: false, backup: true)
+        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) != nil
+        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
+        backupAppLaunchProfilingConfigFile()
+        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
+        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
+    }
+    
+    // if a file is still present in the backup location, like if a crash occurred before it could be removed, or an error occurred when trying to remove it, make sure we overwrite it
+    func testBackupAppLaunchProfilingConfigFile_anotherBackupFilePresent() throws {
+        try ensureAppLaunchProfileConfig(exists: true, tracesSampleRate: 0.1, profilesSampleRate: 0.2)
+        try ensureAppLaunchProfileConfig(exists: true, tracesSampleRate: 0.3, profilesSampleRate: 0.4, backup: true)
+        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) != nil
+        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
+        backupAppLaunchProfilingConfigFile()
+        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
+        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
+    }
+    
+    func testRemoveAppLaunchProfilingConfigBackupFile() throws {
+        try ensureAppLaunchProfileConfig(exists: true, backup: true)
+        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
+        removeAppLaunchProfilingConfigBackupFile()
+        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
+    }
+    
+    // if there's not a file when we expect one, just make sure we don't crash
+    func testRemoveAppLaunchProfilingConfigBackupFile_noFileExists() throws {
+        try ensureAppLaunchProfileConfig(exists: false, backup: true)
+        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
+        removeAppLaunchProfilingConfigBackupFile()
+        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
+    }
+}
+
+// MARK: Private profiling tests
+private extension SentryFileManagerTests {
+    func ensureAppLaunchProfileConfig(exists: Bool = true, tracesSampleRate: Double = 1, profilesSampleRate: Double = 1, backup: Bool = false) throws {
+        let url: URL
+        if backup {
+            url = launchProfileConfigBackupFileURL()
+        } else {
+            url = launchProfileConfigFileURL()
+        }
+        
+        if exists {
+            let dict = [kSentryLaunchProfileConfigKeyTracesSampleRate: tracesSampleRate, kSentryLaunchProfileConfigKeyProfilesSampleRate: profilesSampleRate]
+            try (dict as NSDictionary).write(to: url)
+        } else {
+            let fm = FileManager.default
+            if fm.fileExists(atPath: url.path) {
+                try fm.removeItem(at: url)
+            }
+        }
+    }
+}
+#endif // os(iOS) || os(macOS) || targetEnvironment(macCatalyst)
 
-    private func givenMaximumEnvelopes() {
+// MARK: Private
+private extension SentryFileManagerTests {
+    func givenMaximumEnvelopes() {
         fixture.eventIds.forEach { id in
             let envelope = SentryEnvelope(id: id, singleItem: SentryEnvelopeItem(event: Event()))
 
@@ -720,7 +867,7 @@ class SentryFileManagerTests: XCTestCase {
         }
     }
 
-    private func givenGarbageInEnvelopesFolder() {
+    func givenGarbageInEnvelopesFolder() {
         do {
             try "garbage".write(to: URL(fileURLWithPath: "\(sut.envelopesPath)/garbage.json"), atomically: true, encoding: .utf8)
         } catch {
@@ -728,7 +875,7 @@ class SentryFileManagerTests: XCTestCase {
         }
     }
     
-    private func givenOldEnvelopes() throws {
+    func givenOldEnvelopes() throws {
         let envelope = TestConstants.envelope
         let path = sut.store(envelope)
 
@@ -739,7 +886,7 @@ class SentryFileManagerTests: XCTestCase {
         XCTAssertEqual(sut.getAllEnvelopes().count, 1)
     }
 
-    private func storeEvent() {
+    func storeEvent() {
         do {
             // Store a fake event to the events folder
             try FileManager.default.createDirectory(atPath: sut.eventsPath, withIntermediateDirectories: true, attributes: nil)
@@ -749,7 +896,7 @@ class SentryFileManagerTests: XCTestCase {
         }
     }
     
-    private func setImmutableForAppState(immutable: Bool) {
+    func setImmutableForAppState(immutable: Bool) {
         let appStateFilePath = Dynamic(sut).appStateFilePath.asString ?? ""
         let fileManager = FileManager.default
         
@@ -764,7 +911,7 @@ class SentryFileManagerTests: XCTestCase {
         }
     }
 
-    private func setImmutableForTimezoneOffset(immutable: Bool) {
+    func setImmutableForTimezoneOffset(immutable: Bool) {
         let timezoneOffsetFilePath = Dynamic(sut).timezoneOffsetFilePath.asString ?? ""
         let fileManager = FileManager.default
 
@@ -779,12 +926,12 @@ class SentryFileManagerTests: XCTestCase {
         }
     }
 
-    private func assertEventFolderDoesntExist() {
+    func assertEventFolderDoesntExist() {
         XCTAssertFalse(FileManager.default.fileExists(atPath: sut.eventsPath),
                         "Folder for events should be deleted on init: \(sut.eventsPath)")
     }
 
-    private func assertSessionInitMoved(_ actualSessionFileContents: SentryFileContents) {
+    func assertSessionInitMoved(_ actualSessionFileContents: SentryFileContents) {
         let actualSessionEnvelope = SentrySerialization.envelope(with: actualSessionFileContents.contents)
         XCTAssertEqual(2, actualSessionEnvelope?.items.count)
 
@@ -794,7 +941,7 @@ class SentryFileManagerTests: XCTestCase {
         XCTAssertEqual(fixture.expectedSessionUpdate, actualSession)
     }
     
-    private func assertSessionInitNotMoved(_ actualSessionFileContents: SentryFileContents) {
+    func assertSessionInitNotMoved(_ actualSessionFileContents: SentryFileContents) {
         let actualSessionEnvelope = SentrySerialization.envelope(with: actualSessionFileContents.contents)
         XCTAssertEqual(2, actualSessionEnvelope?.items.count)
 
@@ -804,7 +951,7 @@ class SentryFileManagerTests: XCTestCase {
         XCTAssertEqual(fixture.sessionUpdate, actualSession)
     }
 
-    private func assertSessionEnvelopesStored(count: Int) {
+    func assertSessionEnvelopesStored(count: Int) {
         let fileContentsWithSession = sut.getAllEnvelopes().filter { envelopeFileContents in
             let envelope = SentrySerialization.envelope(with: envelopeFileContents.contents)
             return !(envelope?.items.filter { item in item.header.type == SentryEnvelopeItemTypeSession }.isEmpty ?? false)
@@ -813,16 +960,16 @@ class SentryFileManagerTests: XCTestCase {
         XCTAssertEqual(count, fileContentsWithSession.count)
     }
     
-    private func assertValidAppStateStored() {
+    func assertValidAppStateStored() {
         let actual = sut.readAppState()
         XCTAssertEqual(TestData.appState, actual)
     }
 
-    private func advanceTime(bySeconds: TimeInterval) {
+    func advanceTime(bySeconds: TimeInterval) {
         fixture.currentDateProvider.setDate(date: fixture.currentDateProvider.date().addingTimeInterval(bySeconds))
     }
     
-    private class AppStateWithFaultySerialization: SentryAppState {
+    class AppStateWithFaultySerialization: SentryAppState {
         override func serialize() -> [String: Any] {
             return ["app": self]
         }
diff --git a/Tests/SentryTests/SentryLaunchProfiling+Tests.h b/Tests/SentryTests/SentryLaunchProfiling+Tests.h
index 91997dc7ba..92cbff552e 100644
--- a/Tests/SentryTests/SentryLaunchProfiling+Tests.h
+++ b/Tests/SentryTests/SentryLaunchProfiling+Tests.h
@@ -1,4 +1,4 @@
-#import "SentryInternalDefines.h"
+#import "SentryDefines.h"
 #import "SentryLaunchProfiling.h"
 
 @class SentrySamplerDecision;
@@ -14,4 +14,7 @@ typedef struct {
 
 SENTRY_EXTERN SentryLaunchProfileConfig shouldProfileNextLaunch(SentryOptions *options);
 
+SENTRY_EXTERN NSString *const kSentryLaunchProfileConfigKeyTracesSampleRate;
+SENTRY_EXTERN NSString *const kSentryLaunchProfileConfigKeyProfilesSampleRate;
+
 NS_ASSUME_NONNULL_END
diff --git a/Tests/SentryTests/SentryTests-Bridging-Header.h b/Tests/SentryTests/SentryTests-Bridging-Header.h
index c2308050e3..26bdb95b5c 100644
--- a/Tests/SentryTests/SentryTests-Bridging-Header.h
+++ b/Tests/SentryTests/SentryTests-Bridging-Header.h
@@ -125,6 +125,7 @@
 #import "SentryInstallation+Test.h"
 #import "SentryInstallation.h"
 #import "SentryInternalNotificationNames.h"
+#import "SentryLaunchProfiling+Tests.h"
 #import "SentryLevelMapper.h"
 #import "SentryLog+TestInit.h"
 #import "SentryLog.h"

From 15f4b7b06f2cc45f8958f1e3f2ad68cac42ad0f6 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Mon, 5 Feb 2024 14:32:00 -0900
Subject: [PATCH 19/34] wip: need to thread the stored sample rate to
 isTracingEnabled or where its used for the uivc tracker installation

---
 Sources/Sentry/SentryOptions.m | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/Sources/Sentry/SentryOptions.m b/Sources/Sentry/SentryOptions.m
index 22c0115055..f7e8ad2885 100644
--- a/Sources/Sentry/SentryOptions.m
+++ b/Sources/Sentry/SentryOptions.m
@@ -562,6 +562,9 @@ - (void)setTracesSampler:(SentryTracesSamplerCallback)tracesSampler
 
 - (BOOL)isTracingEnabled
 {
+    // TODO: we need to also consider the stored app launch trace sample rate here, because this is
+    // called to see if SentryUIViewControllerPerformanceTracker should even be installed; if it is
+    // not installed, we don't get the automatic ui.load transaction to attach the profile to
     return _enableTracing
         && ((_tracesSampleRate != nil && [_tracesSampleRate doubleValue] > 0)
             || _tracesSampler != nil);

From 2fd98c081c09244874f5ec4bc6e64a37987c63b2 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Tue, 6 Feb 2024 11:42:47 -0900
Subject: [PATCH 20/34] pr feedback: rename

---
 Sources/Sentry/SentryFileManager.m | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/Sources/Sentry/SentryFileManager.m b/Sources/Sentry/SentryFileManager.m
index d85297663e..ba382eb2cb 100644
--- a/Sources/Sentry/SentryFileManager.m
+++ b/Sources/Sentry/SentryFileManager.m
@@ -42,7 +42,7 @@
  * from.
  */
 void
-_unsafe_removeFileAtPath(NSString *path)
+_non_thread_safe_removeFileAtPath(NSString *path)
 {
     NSError *error = nil;
     NSFileManager *fileManager = [NSFileManager defaultManager];
@@ -307,7 +307,7 @@ - (void)deleteAllEnvelopes
 - (void)removeFileAtPath:(NSString *)path
 {
     @synchronized(self) {
-        _unsafe_removeFileAtPath(path);
+        _non_thread_safe_removeFileAtPath(path);
     }
 }
 
@@ -809,13 +809,13 @@ - (void)createPathsWithOptions:(SentryOptions *)options
 void
 removeAppLaunchProfilingConfigFile(void)
 {
-    _unsafe_removeFileAtPath(launchProfileConfigFileURL().path);
+    _non_thread_safe_removeFileAtPath(launchProfileConfigFileURL().path);
 }
 
 void
 removeAppLaunchProfilingConfigBackupFile(void)
 {
-    _unsafe_removeFileAtPath(launchProfileConfigBackupFileURL().path);
+    _non_thread_safe_removeFileAtPath(launchProfileConfigBackupFileURL().path);
 }
 
 void
@@ -829,7 +829,7 @@ - (void)createPathsWithOptions:(SentryOptions *)options
     }
 
     NSString *toPath = launchProfileConfigBackupFileURL().path;
-    _unsafe_removeFileAtPath(toPath);
+    _non_thread_safe_removeFileAtPath(toPath);
 
     NSError *error;
     SENTRY_CASSERT([fm moveItemAtPath:fromPath toPath:toPath error:&error],

From 3af711baf9e8f712379729e055227bf11af611d4 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Tue, 6 Feb 2024 17:03:03 -0900
Subject: [PATCH 21/34] fix test on macos (isnt applicable)

---
 .../SentryAppLaunchProfilingTests.m            | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
index 2cd47f9d7b..f622f9a0c9 100644
--- a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
+++ b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
@@ -107,31 +107,31 @@ - (void)testSettingProfilesSampleRateTo0DisablesAppLaunchProfiling
         @"but profiles sample rate of 0 should not enable launch profiling");
 }
 
-- (void)testDisablingSwizzlingOptionDisablesAppLaunchProfiling
+- (void)testDisablingAutoPerformanceTracingOptionDisablesAppLaunchProfiling
 {
     XCTAssertFalse(
         shouldProfileNextLaunch(
             [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
-                                                                 enableSwizzling, @NO) }])
+                                                                 enableAutoPerformanceTracing,
+                                                                 @NO) }])
             .shouldProfile,
         @"Default options with app launch profiling and tracing enabled, traces and profiles "
-        @"sample rates of 1, but swizzling disabled should not enable launch profiling");
+        @"sample rates of 1, but automatic performance tracing disabled should not enable launch "
+        @"profiling");
 }
 
-- (void)testDisablingAutoPerformanceTracingOptionDisablesAppLaunchProfiling
+#    if SENTRY_HAS_UIKIT
+- (void)testDisablingSwizzlingOptionDisablesAppLaunchProfiling
 {
     XCTAssertFalse(
         shouldProfileNextLaunch(
             [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
-                                                                 enableAutoPerformanceTracing,
-                                                                 @NO) }])
+                                                                 enableSwizzling, @NO) }])
             .shouldProfile,
         @"Default options with app launch profiling and tracing enabled, traces and profiles "
-        @"sample rates of 1, but automatic performance tracing disabled should not enable launch "
-        @"profiling");
+        @"sample rates of 1, but swizzling disabled should not enable launch profiling");
 }
 
-#    if SENTRY_HAS_UIKIT
 - (void)testDisablingUIViewControllerTracingOptionDisablesAppLaunchProfiling
 {
     XCTAssertFalse(

From b799c57418174382aefa9f2e841036c06ca2f64d Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Wed, 7 Feb 2024 11:42:54 -0900
Subject: [PATCH 22/34] add availability declaration to fix some ui tests that
 are failing to build due to the async setUp override

---
 Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
index e526f56c0a..e754ea4c36 100644
--- a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
+++ b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
@@ -2,7 +2,8 @@ import XCTest
 
 //swiftlint:disable function_body_length todo
 
-class ProfilingUITests: BaseUITest {    
+@available(iOS 16, *)
+class ProfilingUITests: BaseUITest {
     override var automaticallyLaunchAndTerminateApp: Bool { false }
     
     // this will run before the non-async BaseUITest.setUp, so we can bail out before running any of the logic in there
@@ -72,6 +73,7 @@ class ProfilingUITests: BaseUITest {
     }
 }
 
+@available(iOS 16, *)
 extension ProfilingUITests {
     // We don't need to test these on multiple OSes right now, and older versions seem to have issues; older devices or VM images running simulators might just be slower. Latest OS is enough coverage for our needs for now.
     func checkOSVersionForProfilingTest() throws {

From b79b3036b9d410ae6d05dda4e6bdf430e952d1e2 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Wed, 7 Feb 2024 13:52:03 -0900
Subject: [PATCH 23/34] wip

---
 Sources/Sentry/SentryOptions.m | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/Sources/Sentry/SentryOptions.m b/Sources/Sentry/SentryOptions.m
index f7e8ad2885..d3f24d8ee6 100644
--- a/Sources/Sentry/SentryOptions.m
+++ b/Sources/Sentry/SentryOptions.m
@@ -32,7 +32,6 @@
 #if SENTRY_HAS_METRIC_KIT
 #    import "SentryMetricKitIntegration.h"
 #endif // SENTRY_HAS_METRIC_KIT
-
 NSString *const kSentryDefaultEnvironment = @"production";
 
 @implementation SentryOptions {
@@ -562,9 +561,6 @@ - (void)setTracesSampler:(SentryTracesSamplerCallback)tracesSampler
 
 - (BOOL)isTracingEnabled
 {
-    // TODO: we need to also consider the stored app launch trace sample rate here, because this is
-    // called to see if SentryUIViewControllerPerformanceTracker should even be installed; if it is
-    // not installed, we don't get the automatic ui.load transaction to attach the profile to
     return _enableTracing
         && ((_tracesSampleRate != nil && [_tracesSampleRate doubleValue] > 0)
             || _tracesSampler != nil);

From 854d0633a0d4e5842a0724c8d458eb541344589a Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Wed, 7 Feb 2024 16:33:05 -0900
Subject: [PATCH 24/34] wip moving to starting a trace for the launch profile
 instead storing the sample rates to inject into a ui.load trace

---
 .../Sentry/Profiling/SentryLaunchProfiling.m  | 17 ++++--
 Sources/Sentry/SentryHub.m                    | 52 ++++++++-----------
 .../Sentry/include/SentryLaunchProfiling.h    |  2 +
 3 files changed, 37 insertions(+), 34 deletions(-)

diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index 6fd50c1bb7..1a7b3bdad0 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -15,6 +15,7 @@
 #    import "SentrySamplerDecision.h"
 #    import "SentrySampling.h"
 #    import "SentrySamplingContext.h"
+#    import "SentryTracer.h"
 #    import "SentryTracerConfiguration.h"
 #    import "SentryTransactionContext.h"
 
@@ -24,6 +25,7 @@
 uint64_t appLaunchSystemTime;
 NSString *const kSentryLaunchProfileConfigKeyTracesSampleRate = @"traces";
 NSString *const kSentryLaunchProfileConfigKeyProfilesSampleRate = @"profiles";
+SentryTracer *launchTracer;
 
 #    pragma mark - Private
 
@@ -118,7 +120,8 @@
     // directly to customers, and we'll want to ensure it only runs once. dispatch_once is an
     // efficient operation so it's fine to leave this in the launch path in any case.
     dispatch_once(&onceToken, ^{
-        if (!appLaunchProfileConfigFileExists()) {
+        isTracingAppLaunch = appLaunchProfileConfigFileExists();
+        if (!isTracingAppLaunch) {
             return;
         }
 
@@ -135,9 +138,13 @@
 
         SENTRY_LOG_INFO(@"Starting app launch profile at %llu", appLaunchSystemTime);
 
-        // don't worry about synchronizing the write here, as there should be no other tracing
-        // activity going on this early in the process. this codepath is also behind a dispatch_once
-        isTracingAppLaunch = [SentryProfiler startWithTracer:appLaunchTraceId];
+        SentryTransactionContext *context =
+            [[SentryTransactionContext alloc] initWithName:@"launch" operation:@"app.lifecycle"];
+        SentryTracerConfiguration *config = [SentryTracerConfiguration defaultConfiguration];
+        injectLaunchSamplerDecisions(context, config);
+        launchTracer = [[SentryTracer alloc] initWithTransactionContext:context
+                                                                    hub:nil
+                                                          configuration:config];
     });
 }
 
@@ -155,7 +162,7 @@
     configuration.profilesSamplerDecision =
         [[SentrySamplerDecision alloc] initWithDecision:kSentrySampleDecisionYes
                                           forSampleRate:profilesRate];
-    transactionContext.sampleRate = rates[kSentryLaunchProfileConfigKeyTracesSampleRate];
+    transactionContext.sampleRate = tracesRate;
     return YES;
 }
 
diff --git a/Sources/Sentry/SentryHub.m b/Sources/Sentry/SentryHub.m
index 315589e867..7172a17a6c 100644
--- a/Sources/Sentry/SentryHub.m
+++ b/Sources/Sentry/SentryHub.m
@@ -16,14 +16,13 @@
 #import "SentryPerformanceTracker.h"
 #import "SentryProfilingConditionals.h"
 #import "SentrySDK+Private.h"
-#import "SentrySamplerDecision.h"
-#import "SentrySampling.h"
 #import "SentrySamplingContext.h"
 #import "SentryScope+Private.h"
 #import "SentrySerialization.h"
 #import "SentrySession+Private.h"
 #import "SentryTraceOrigins.h"
 #import "SentryTracer.h"
+#import "SentryTracesSampler.h"
 #import "SentryTransaction.h"
 #import "SentryTransactionContext+Private.h"
 
@@ -32,7 +31,7 @@
 #endif // SENTRY_HAS_UIKIT
 
 #if SENTRY_TARGET_PROFILING_SUPPORTED
-#    import "SentryLaunchProfiling.h"
+#    import "SentryProfilesSampler.h"
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED"
 
 NS_ASSUME_NONNULL_BEGIN
@@ -43,6 +42,10 @@
 @property (nullable, nonatomic, strong) SentryClient *client;
 @property (nullable, nonatomic, strong) SentryScope *scope;
 @property (nonatomic, strong) SentryCrashWrapper *crashWrapper;
+@property (nonatomic, strong) SentryTracesSampler *tracesSampler;
+#if SENTRY_TARGET_PROFILING_SUPPORTED
+@property (nonatomic, strong) SentryProfilesSampler *profilesSampler;
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED
 @property (nonatomic, strong) NSMutableArray<id<SentryIntegrationProtocol>> *installedIntegrations;
 @property (nonatomic, strong) NSMutableSet<NSString *> *installedIntegrationNames;
 @property (nonatomic) NSUInteger errorsBeforeSession;
@@ -65,7 +68,13 @@ - (instancetype)initWithClient:(nullable SentryClient *)client
         _installedIntegrations = [[NSMutableArray alloc] init];
         _installedIntegrationNames = [[NSMutableSet alloc] init];
         _crashWrapper = [SentryCrashWrapper sharedInstance];
+        _tracesSampler = [[SentryTracesSampler alloc] initWithOptions:client.options];
         _errorsBeforeSession = 0;
+#if SENTRY_TARGET_PROFILING_SUPPORTED
+        if (client.options.isProfilingEnabled) {
+            _profilesSampler = [[SentryProfilesSampler alloc] initWithOptions:client.options];
+        }
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED
     }
     return self;
 }
@@ -373,36 +382,21 @@ - (SentryTracer *)startTransactionWithContext:(SentryTransactionContext *)transa
                         customSamplingContext:(NSDictionary<NSString *, id> *)customSamplingContext
                                 configuration:(SentryTracerConfiguration *)configuration
 {
+    SentrySamplingContext *samplingContext =
+        [[SentrySamplingContext alloc] initWithTransactionContext:transactionContext
+                                            customSamplingContext:customSamplingContext];
 
-    BOOL applySampling = YES;
+    SentryTracesSamplerDecision *samplerDecision = [_tracesSampler sample:samplingContext];
+    transactionContext = [self transactionContext:transactionContext
+                                      withSampled:samplerDecision.decision];
+    transactionContext.sampleRate = samplerDecision.sampleRate;
 
 #if SENTRY_TARGET_PROFILING_SUPPORTED
-    // in order to sample launch profiles, we compute the trace/profile sampling decisions in
-    // SentrySDK.startWithOptions and if the decisions evaluate to yes, we write a config file with
-    // the actual sampling rates used, and the presence of this file indicates to the profiler on
-    // the next launch that it should take the profile. then, we need a transaction to attach it to,
-    // so we also don't want to reevaluate the sampling logic and possibly sample out the
-    // transaction it would be attached to, thus rendering the profile as wasted overhead for the
-    // launch. here we can assume a sample decision of yes, and we retrieve the sample rates to
-    // inject into the context/configuration
-    applySampling
-        = !isTracingAppLaunch || injectLaunchSamplerDecisions(transactionContext, configuration);
-#endif // SENTRY_TARGET_PROFILING_SUPPORTED
+    SentryProfilesSamplerDecision *profilesSamplerDecision =
+        [_profilesSampler sample:samplingContext tracesSamplerDecision:samplerDecision];
 
-    if (applySampling) {
-        SentrySamplingContext *samplingContext =
-            [[SentrySamplingContext alloc] initWithTransactionContext:transactionContext
-                                                customSamplingContext:customSamplingContext];
-        SentrySamplerDecision *tracesSamplerDecision
-            = sampleTrace(samplingContext, _client.options);
-        transactionContext = [self transactionContext:transactionContext
-                                          withSampled:tracesSamplerDecision.decision];
-        transactionContext.sampleRate = tracesSamplerDecision.sampleRate;
-#if SENTRY_TARGET_PROFILING_SUPPORTED
-        configuration.profilesSamplerDecision
-            = sampleProfile(samplingContext, tracesSamplerDecision, _client.options);
-#endif // SENTRY_TARGET_PROFILING_SUPPORTED
-    }
+    configuration.profilesSamplerDecision = profilesSamplerDecision;
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED"
 
     SentryTracer *tracer = [[SentryTracer alloc] initWithTransactionContext:transactionContext
                                                                         hub:self
diff --git a/Sources/Sentry/include/SentryLaunchProfiling.h b/Sources/Sentry/include/SentryLaunchProfiling.h
index a67ea4d497..dbd53d102f 100644
--- a/Sources/Sentry/include/SentryLaunchProfiling.h
+++ b/Sources/Sentry/include/SentryLaunchProfiling.h
@@ -8,9 +8,11 @@
 @class SentryOptions;
 @class SentryTracerConfiguration;
 @class SentryTransactionContext;
+@class SentryTracer;
 
 NS_ASSUME_NONNULL_BEGIN
 
+SENTRY_EXTERN SentryTracer *launchTracer;
 SENTRY_EXTERN BOOL isTracingAppLaunch;
 SENTRY_EXTERN SentryId *_Nullable appLaunchTraceId;
 SENTRY_EXTERN uint64_t appLaunchSystemTime;

From f249844e17d4a4b32b9cd9c2ce79d3c33a26acf1 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Wed, 7 Feb 2024 18:04:37 -0900
Subject: [PATCH 25/34] put back hub the way it was

---
 Sources/Sentry/SentryHub.m              | 28 +++++++------------------
 Sources/Sentry/SentrySampling.m         |  2 +-
 Sources/Sentry/include/SentrySampling.h |  2 +-
 3 files changed, 10 insertions(+), 22 deletions(-)

diff --git a/Sources/Sentry/SentryHub.m b/Sources/Sentry/SentryHub.m
index 7172a17a6c..ef1727fb72 100644
--- a/Sources/Sentry/SentryHub.m
+++ b/Sources/Sentry/SentryHub.m
@@ -16,13 +16,14 @@
 #import "SentryPerformanceTracker.h"
 #import "SentryProfilingConditionals.h"
 #import "SentrySDK+Private.h"
+#import "SentrySamplerDecision.h"
+#import "SentrySampling.h"
 #import "SentrySamplingContext.h"
 #import "SentryScope+Private.h"
 #import "SentrySerialization.h"
 #import "SentrySession+Private.h"
 #import "SentryTraceOrigins.h"
 #import "SentryTracer.h"
-#import "SentryTracesSampler.h"
 #import "SentryTransaction.h"
 #import "SentryTransactionContext+Private.h"
 
@@ -30,10 +31,6 @@
 #    import "SentryUIViewControllerPerformanceTracker.h"
 #endif // SENTRY_HAS_UIKIT
 
-#if SENTRY_TARGET_PROFILING_SUPPORTED
-#    import "SentryProfilesSampler.h"
-#endif // SENTRY_TARGET_PROFILING_SUPPORTED"
-
 NS_ASSUME_NONNULL_BEGIN
 
 @interface
@@ -42,10 +39,6 @@
 @property (nullable, nonatomic, strong) SentryClient *client;
 @property (nullable, nonatomic, strong) SentryScope *scope;
 @property (nonatomic, strong) SentryCrashWrapper *crashWrapper;
-@property (nonatomic, strong) SentryTracesSampler *tracesSampler;
-#if SENTRY_TARGET_PROFILING_SUPPORTED
-@property (nonatomic, strong) SentryProfilesSampler *profilesSampler;
-#endif // SENTRY_TARGET_PROFILING_SUPPORTED
 @property (nonatomic, strong) NSMutableArray<id<SentryIntegrationProtocol>> *installedIntegrations;
 @property (nonatomic, strong) NSMutableSet<NSString *> *installedIntegrationNames;
 @property (nonatomic) NSUInteger errorsBeforeSession;
@@ -68,13 +61,7 @@ - (instancetype)initWithClient:(nullable SentryClient *)client
         _installedIntegrations = [[NSMutableArray alloc] init];
         _installedIntegrationNames = [[NSMutableSet alloc] init];
         _crashWrapper = [SentryCrashWrapper sharedInstance];
-        _tracesSampler = [[SentryTracesSampler alloc] initWithOptions:client.options];
         _errorsBeforeSession = 0;
-#if SENTRY_TARGET_PROFILING_SUPPORTED
-        if (client.options.isProfilingEnabled) {
-            _profilesSampler = [[SentryProfilesSampler alloc] initWithOptions:client.options];
-        }
-#endif // SENTRY_TARGET_PROFILING_SUPPORTED
     }
     return self;
 }
@@ -386,14 +373,15 @@ - (SentryTracer *)startTransactionWithContext:(SentryTransactionContext *)transa
         [[SentrySamplingContext alloc] initWithTransactionContext:transactionContext
                                             customSamplingContext:customSamplingContext];
 
-    SentryTracesSamplerDecision *samplerDecision = [_tracesSampler sample:samplingContext];
+    SentrySamplerDecision *tracesSamplerDecision
+        = sampleTrace(samplingContext, self.client.options);
     transactionContext = [self transactionContext:transactionContext
-                                      withSampled:samplerDecision.decision];
-    transactionContext.sampleRate = samplerDecision.sampleRate;
+                                      withSampled:tracesSamplerDecision.decision];
+    transactionContext.sampleRate = tracesSamplerDecision.sampleRate;
 
 #if SENTRY_TARGET_PROFILING_SUPPORTED
-    SentryProfilesSamplerDecision *profilesSamplerDecision =
-        [_profilesSampler sample:samplingContext tracesSamplerDecision:samplerDecision];
+    SentrySamplerDecision *profilesSamplerDecision
+        = sampleProfile(samplingContext, tracesSamplerDecision, self.client.options);
 
     configuration.profilesSamplerDecision = profilesSamplerDecision;
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED"
diff --git a/Sources/Sentry/SentrySampling.m b/Sources/Sentry/SentrySampling.m
index a0eda6170b..3edac3fcb7 100644
--- a/Sources/Sentry/SentrySampling.m
+++ b/Sources/Sentry/SentrySampling.m
@@ -84,7 +84,7 @@
 
 SentrySamplerDecision *
 sampleProfile(SentrySamplingContext *context, SentrySamplerDecision *tracesSamplerDecision,
-    SentryOptions *options)
+    SentryOptions *_Nullable options)
 {
     // Profiles are always undersampled with respect to traces. If the trace is not sampled,
     // the profile will not be either. If the trace is sampled, we can proceed to checking
diff --git a/Sources/Sentry/include/SentrySampling.h b/Sources/Sentry/include/SentrySampling.h
index 5c404b9113..c9a2b6a877 100644
--- a/Sources/Sentry/include/SentrySampling.h
+++ b/Sources/Sentry/include/SentrySampling.h
@@ -18,7 +18,7 @@ SentrySamplerDecision *sampleTrace(SentrySamplingContext *context, SentryOptions
  * whether the trace corresponding to the profile was sampled.
  */
 SentrySamplerDecision *sampleProfile(SentrySamplingContext *context,
-    SentrySamplerDecision *tracesSamplerDecision, SentryOptions *options);
+    SentrySamplerDecision *tracesSamplerDecision, SentryOptions *_Nullable options);
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
 
 NS_ASSUME_NONNULL_END

From 6cd924adeb51e612c1a1c2c4c775d408dd7c76d5 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Wed, 7 Feb 2024 18:15:01 -0900
Subject: [PATCH 26/34] remove the need for a backup config file

since we're no longer threading the sample rates to the first transaction created after SentrySDK.start where we can wind up with mismatched sample rates
---
 .../Sentry/Profiling/SentryLaunchProfiling.m  | 53 +++++++------------
 Sources/Sentry/SentryFileManager.m            | 46 ++--------------
 Sources/Sentry/include/SentryFileManager.h    | 15 ------
 .../Sentry/include/SentryLaunchProfiling.h    |  8 ---
 .../Helper/SentryFileManager+Test.h           |  1 -
 .../Helper/SentryFileManagerTests.swift       | 49 ++---------------
 6 files changed, 27 insertions(+), 145 deletions(-)

diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index 1a7b3bdad0..a040ed5dbb 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -94,21 +94,18 @@
 {
     [SentryDependencyContainer.sharedInstance.dispatchQueueWrapper dispatchAsyncWithBlock:^{
         SentryLaunchProfileConfig config = shouldProfileNextLaunch(options);
-        if (config.shouldProfile) {
-            NSMutableDictionary<NSString *, NSNumber *> *configDict =
-                [NSMutableDictionary<NSString *, NSNumber *> dictionary];
-            configDict[kSentryLaunchProfileConfigKeyTracesSampleRate]
-                = config.tracesDecision.sampleRate;
-            configDict[kSentryLaunchProfileConfigKeyProfilesSampleRate]
-                = config.profilesDecision.sampleRate;
-            writeAppLaunchProfilingConfigFile(configDict);
-        } else {
-            if (isTracingAppLaunch) {
-                backupAppLaunchProfilingConfigFile();
-            } else {
-                removeAppLaunchProfilingConfigFile();
-            }
+        if (!config.shouldProfile) {
+            removeAppLaunchProfilingConfigFile();
+            return;
         }
+
+        NSMutableDictionary<NSString *, NSNumber *> *configDict =
+            [NSMutableDictionary<NSString *, NSNumber *> dictionary];
+        configDict[kSentryLaunchProfileConfigKeyTracesSampleRate]
+            = config.tracesDecision.sampleRate;
+        configDict[kSentryLaunchProfileConfigKeyProfilesSampleRate]
+            = config.profilesDecision.sampleRate;
+        writeAppLaunchProfilingConfigFile(configDict);
     }];
 }
 
@@ -141,29 +138,19 @@
         SentryTransactionContext *context =
             [[SentryTransactionContext alloc] initWithName:@"launch" operation:@"app.lifecycle"];
         SentryTracerConfiguration *config = [SentryTracerConfiguration defaultConfiguration];
-        injectLaunchSamplerDecisions(context, config);
+        NSDictionary<NSString *, NSNumber *> *rates = appLaunchProfileConfiguration();
+        NSNumber *profilesRate = rates[kSentryLaunchProfileConfigKeyProfilesSampleRate];
+        NSNumber *tracesRate = rates[kSentryLaunchProfileConfigKeyTracesSampleRate];
+        if (profilesRate != nil && tracesRate != nil) {
+            config.profilesSamplerDecision =
+                [[SentrySamplerDecision alloc] initWithDecision:kSentrySampleDecisionYes
+                                                  forSampleRate:profilesRate];
+            context.sampleRate = tracesRate;
+        }
         launchTracer = [[SentryTracer alloc] initWithTransactionContext:context
                                                                     hub:nil
                                                           configuration:config];
     });
 }
 
-BOOL
-injectLaunchSamplerDecisions(
-    SentryTransactionContext *transactionContext, SentryTracerConfiguration *configuration)
-{
-    NSDictionary<NSString *, NSNumber *> *rates = appLaunchProfileConfiguration();
-    removeAppLaunchProfilingConfigBackupFile();
-    NSNumber *profilesRate = rates[kSentryLaunchProfileConfigKeyProfilesSampleRate];
-    NSNumber *tracesRate = rates[kSentryLaunchProfileConfigKeyTracesSampleRate];
-    if (profilesRate == nil || tracesRate == nil) {
-        return NO;
-    }
-    configuration.profilesSamplerDecision =
-        [[SentrySamplerDecision alloc] initWithDecision:kSentrySampleDecisionYes
-                                          forSampleRate:profilesRate];
-    transactionContext.sampleRate = tracesRate;
-    return YES;
-}
-
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
diff --git a/Sources/Sentry/SentryFileManager.m b/Sources/Sentry/SentryFileManager.m
index d85297663e..2890ba083d 100644
--- a/Sources/Sentry/SentryFileManager.m
+++ b/Sources/Sentry/SentryFileManager.m
@@ -762,31 +762,16 @@ - (void)createPathsWithOptions:(SentryOptions *)options
     return sentryLaunchConfigFileURL;
 }
 
-NSURL *
-launchProfileConfigBackupFileURL(void)
-{
-    static NSURL *sentryLaunchConfigFileURL;
-    static dispatch_once_t onceToken;
-    dispatch_once(&onceToken, ^{
-        sentryLaunchConfigFileURL =
-            [[NSURL fileURLWithPath:[sentryApplicationSupportPath()
-                                        stringByAppendingPathComponent:@"profileLaunch"]]
-                URLByAppendingPathExtension:@"bak"];
-    });
-    return sentryLaunchConfigFileURL;
-}
-
 NSDictionary<NSString *, NSNumber *> *_Nullable appLaunchProfileConfiguration(void)
 {
-    NSURL *url = launchProfileConfigBackupFileURL();
+    NSURL *url = launchProfileConfigFileURL();
     if (![[NSFileManager defaultManager] fileExistsAtPath:url.path]) {
         return nil;
     }
 
     NSError *error;
-    NSDictionary<NSString *, NSNumber *> *config = [NSDictionary<NSString *, NSNumber *>
-        dictionaryWithContentsOfURL:launchProfileConfigBackupFileURL()
-                              error:&error];
+    NSDictionary<NSString *, NSNumber *> *config =
+        [NSDictionary<NSString *, NSNumber *> dictionaryWithContentsOfURL:url error:&error];
     SENTRY_CASSERT(
         error == nil, @"Encountered error trying to retrieve app launch profile config: %@", error);
     return config;
@@ -811,31 +796,6 @@ - (void)createPathsWithOptions:(SentryOptions *)options
 {
     _unsafe_removeFileAtPath(launchProfileConfigFileURL().path);
 }
-
-void
-removeAppLaunchProfilingConfigBackupFile(void)
-{
-    _unsafe_removeFileAtPath(launchProfileConfigBackupFileURL().path);
-}
-
-void
-backupAppLaunchProfilingConfigFile(void)
-{
-    NSFileManager *fm = [NSFileManager defaultManager];
-    NSString *fromPath = launchProfileConfigFileURL().path;
-    if (!SENTRY_CASSERT_RETURN([fm fileExistsAtPath:fromPath],
-            @"Expect to have a current launch profile config to use for subsequent transaction.")) {
-        return;
-    }
-
-    NSString *toPath = launchProfileConfigBackupFileURL().path;
-    _unsafe_removeFileAtPath(toPath);
-
-    NSError *error;
-    SENTRY_CASSERT([fm moveItemAtPath:fromPath toPath:toPath error:&error],
-        @"Failed to backup launch profile config file for use to set up associated transaction: %@",
-        error);
-}
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
 
 @end
diff --git a/Sources/Sentry/include/SentryFileManager.h b/Sources/Sentry/include/SentryFileManager.h
index 8d3322bf01..7694698278 100644
--- a/Sources/Sentry/include/SentryFileManager.h
+++ b/Sources/Sentry/include/SentryFileManager.h
@@ -123,21 +123,6 @@ SENTRY_EXTERN void writeAppLaunchProfilingConfigFile(
  * start the profiler.
  */
 SENTRY_EXTERN void removeAppLaunchProfilingConfigFile(void);
-
-/**
- * Save a current launch profile config file for use when the associated transaction needs to be
- * started. This is when checking @c SentryOptions for whether to write a config for the next
- * launch; if there's no current app launch profile going, then we'd simply remove whatever config
- * file may exist, but if there is a launch profile going, we need to save the config that it used.
- */
-SENTRY_EXTERN void backupAppLaunchProfilingConfigFile(void);
-
-/**
- * Clean up the backup file saved off in the case that an app launch is in-flight when the SDK
- * configures subsequent launches not to profile, requiring the current config to be saved for the
- * associated transaction. This can be cleaned up when that transaction is started.
- */
-SENTRY_EXTERN void removeAppLaunchProfilingConfigBackupFile(void);
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
 
 @end
diff --git a/Sources/Sentry/include/SentryLaunchProfiling.h b/Sources/Sentry/include/SentryLaunchProfiling.h
index dbd53d102f..68c1094cac 100644
--- a/Sources/Sentry/include/SentryLaunchProfiling.h
+++ b/Sources/Sentry/include/SentryLaunchProfiling.h
@@ -28,14 +28,6 @@ void startLaunchProfile(void);
  */
 void configureLaunchProfiling(SentryOptions *options);
 
-/**
- * If there were previously persisted sampling rates sed when decidign the launch profile/trace,
- * inject them into the context and configuration.
- * @return @c YES if persisted rates were found and injected, @c NO otherwise.
- */
-BOOL injectLaunchSamplerDecisions(
-    SentryTransactionContext *transactionContext, SentryTracerConfiguration *configuration);
-
 NS_ASSUME_NONNULL_END
 
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
diff --git a/Tests/SentryTests/Helper/SentryFileManager+Test.h b/Tests/SentryTests/Helper/SentryFileManager+Test.h
index 1285179f52..aa102040fb 100644
--- a/Tests/SentryTests/Helper/SentryFileManager+Test.h
+++ b/Tests/SentryTests/Helper/SentryFileManager+Test.h
@@ -3,7 +3,6 @@
 NS_ASSUME_NONNULL_BEGIN
 
 SENTRY_EXTERN NSURL *launchProfileConfigFileURL(void);
-SENTRY_EXTERN NSURL *launchProfileConfigBackupFileURL(void);
 
 @interface
 SentryFileManager ()
diff --git a/Tests/SentryTests/Helper/SentryFileManagerTests.swift b/Tests/SentryTests/Helper/SentryFileManagerTests.swift
index 4ead32d4e5..809329bd30 100644
--- a/Tests/SentryTests/Helper/SentryFileManagerTests.swift
+++ b/Tests/SentryTests/Helper/SentryFileManagerTests.swift
@@ -730,7 +730,7 @@ extension SentryFileManagerTests {
     func testAppLaunchProfileConfiguration() throws {
         let expectedTracesSampleRate = 0.12
         let expectedProfilesSampleRate = 0.34
-        try ensureAppLaunchProfileConfig(tracesSampleRate: expectedTracesSampleRate, profilesSampleRate: expectedProfilesSampleRate, backup: true)
+        try ensureAppLaunchProfileConfig(tracesSampleRate: expectedTracesSampleRate, profilesSampleRate: expectedProfilesSampleRate)
         let config = appLaunchProfileConfiguration()
         let actualTracesSampleRate = try XCTUnwrap(config?[kSentryLaunchProfileConfigKeyTracesSampleRate]).doubleValue
         let actualProfilesSampleRate = try XCTUnwrap(config?[kSentryLaunchProfileConfigKeyProfilesSampleRate]).doubleValue
@@ -740,7 +740,7 @@ extension SentryFileManagerTests {
     
     // if a file isn't present when we expect it to be, like if there was an issue when we went to write it to disk
     func testAppLaunchProfileConfiguration_noConfigurationExists() throws {
-        try ensureAppLaunchProfileConfig(exists: false, backup: true)
+        try ensureAppLaunchProfileConfig(exists: false)
         expect(appLaunchProfileConfiguration()) == nil
     }
     
@@ -795,53 +795,12 @@ extension SentryFileManagerTests {
         removeAppLaunchProfilingConfigFile()
         expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
     }
-    
-    func testBackupAppLaunchProfilingConfigFile() throws {
-        try ensureAppLaunchProfileConfig(exists: true)
-        try ensureAppLaunchProfileConfig(exists: false, backup: true)
-        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) != nil
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
-        backupAppLaunchProfilingConfigFile()
-        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
-    }
-    
-    // if a file is still present in the backup location, like if a crash occurred before it could be removed, or an error occurred when trying to remove it, make sure we overwrite it
-    func testBackupAppLaunchProfilingConfigFile_anotherBackupFilePresent() throws {
-        try ensureAppLaunchProfileConfig(exists: true, tracesSampleRate: 0.1, profilesSampleRate: 0.2)
-        try ensureAppLaunchProfileConfig(exists: true, tracesSampleRate: 0.3, profilesSampleRate: 0.4, backup: true)
-        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) != nil
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
-        backupAppLaunchProfilingConfigFile()
-        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
-    }
-    
-    func testRemoveAppLaunchProfilingConfigBackupFile() throws {
-        try ensureAppLaunchProfileConfig(exists: true, backup: true)
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
-        removeAppLaunchProfilingConfigBackupFile()
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
-    }
-    
-    // if there's not a file when we expect one, just make sure we don't crash
-    func testRemoveAppLaunchProfilingConfigBackupFile_noFileExists() throws {
-        try ensureAppLaunchProfileConfig(exists: false, backup: true)
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
-        removeAppLaunchProfilingConfigBackupFile()
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
-    }
 }
 
 // MARK: Private profiling tests
 private extension SentryFileManagerTests {
-    func ensureAppLaunchProfileConfig(exists: Bool = true, tracesSampleRate: Double = 1, profilesSampleRate: Double = 1, backup: Bool = false) throws {
-        let url: URL
-        if backup {
-            url = launchProfileConfigBackupFileURL()
-        } else {
-            url = launchProfileConfigFileURL()
-        }
+    func ensureAppLaunchProfileConfig(exists: Bool = true, tracesSampleRate: Double = 1, profilesSampleRate: Double = 1) throws {
+        let url = launchProfileConfigFileURL()
         
         if exists {
             let dict = [kSentryLaunchProfileConfigKeyTracesSampleRate: tracesSampleRate, kSentryLaunchProfileConfigKeyProfilesSampleRate: profilesSampleRate]

From 266d060ba4704f30a7373d38b40a97cedf1ab197 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Wed, 7 Feb 2024 19:44:51 -0900
Subject: [PATCH 27/34] remainder of conversion to creating dedicated launch
 traces

put back tracer init logic to previous state

stop profiled traces in calls to SentrySDK.startWithOptions
---
 .../Sentry/Profiling/SentryLaunchProfiling.m  | 13 ++++-
 Sources/Sentry/SentryProfiler.mm              |  2 +
 Sources/Sentry/SentrySDK.m                    |  4 ++
 Sources/Sentry/SentryTracer.m                 | 49 +++++++------------
 .../Sentry/include/SentryLaunchProfiling.h    |  6 ++-
 5 files changed, 41 insertions(+), 33 deletions(-)

diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index a040ed5dbb..134cf132c8 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -25,7 +25,7 @@
 uint64_t appLaunchSystemTime;
 NSString *const kSentryLaunchProfileConfigKeyTracesSampleRate = @"traces";
 NSString *const kSentryLaunchProfileConfigKeyProfilesSampleRate = @"profiles";
-SentryTracer *launchTracer;
+SentryTracer *_Nullable launchTracer;
 
 #    pragma mark - Private
 
@@ -153,4 +153,15 @@
     });
 }
 
+void
+stopLaunchProfile(void)
+{
+    if (launchTracer == nil) {
+        SENTRY_LOG_DEBUG(@"No launch tracer present to stop.");
+    }
+
+    SENTRY_LOG_DEBUG(@"Finishing launch tracer.");
+    [launchTracer finish];
+}
+
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
diff --git a/Sources/Sentry/SentryProfiler.mm b/Sources/Sentry/SentryProfiler.mm
index a743697ab2..329f71947b 100644
--- a/Sources/Sentry/SentryProfiler.mm
+++ b/Sources/Sentry/SentryProfiler.mm
@@ -400,11 +400,13 @@ + (void)recordMetrics
 + (nullable SentryEnvelopeItem *)createProfilingEnvelopeItemForTransaction:
     (SentryTransaction *)transaction
 {
+    SENTRY_LOG_DEBUG(@"Creating profiling envelope item");
     const auto payload = [self collectProfileBetween:transaction.startSystemTime
                                                  and:transaction.endSystemTime
                                             forTrace:transaction.trace.internalID
                                                onHub:transaction.trace.hub];
     if (payload == nil) {
+        SENTRY_LOG_DEBUG(@"Payload was empty, will not create a profiling envelope item.");
         return nil;
     }
 
diff --git a/Sources/Sentry/SentrySDK.m b/Sources/Sentry/SentrySDK.m
index c5769f5ca7..f07aae9ccf 100644
--- a/Sources/Sentry/SentrySDK.m
+++ b/Sources/Sentry/SentrySDK.m
@@ -164,6 +164,10 @@ + (void)setStartTimestamp:(NSDate *)value
 
 + (void)startWithOptions:(SentryOptions *)options
 {
+#if SENTRY_TARGET_PROFILING_SUPPORTED
+    stopLaunchProfile();
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED
+
     [SentryLog configure:options.debug diagnosticLevel:options.diagnosticLevel];
 
     // We accept the tradeoff that the SDK might not be fully initialized directly after
diff --git a/Sources/Sentry/SentryTracer.m b/Sources/Sentry/SentryTracer.m
index 7f51d77856..01e8687773 100644
--- a/Sources/Sentry/SentryTracer.m
+++ b/Sources/Sentry/SentryTracer.m
@@ -190,30 +190,17 @@ - (instancetype)initWithTransactionContext:(SentryTransactionContext *)transacti
 #endif // SENTRY_HAS_UIKIT
 
 #if SENTRY_TARGET_PROFILING_SUPPORTED
-    BOOL isTracingAppLaunchValue;
-    @synchronized(appLaunchTraceLock) {
-        isTracingAppLaunchValue = isTracingAppLaunch;
-    }
-    if (isTracingAppLaunchValue) {
-        SENTRY_ASSERT(appLaunchTraceId != nil, @"Expected an app launch trace ID.");
-        _internalID = appLaunchTraceId;
-        SENTRY_LOG_DEBUG(
-            @"App launch profile in progress, will attach to trace %@", _internalID.sentryIdString);
-        appLaunchTraceId = nil;
-        _isProfiling = isTracingAppLaunchValue;
-        _startSystemTime = appLaunchSystemTime;
-    } else {
-        if (_configuration.profilesSamplerDecision.decision == kSentrySampleDecisionYes) {
-            _internalID = [[SentryId alloc] init];
-            if ((_isProfiling = [SentryProfiler startWithTracer:_internalID])) {
-                SENTRY_LOG_DEBUG(@"Started profiler for trace %@", _internalID.sentryIdString);
-            }
-            _startSystemTime = SentryDependencyContainer.sharedInstance.dateProvider.systemTime;
+    if (_configuration.profilesSamplerDecision.decision == kSentrySampleDecisionYes) {
+        _internalID = [[SentryId alloc] init];
+        if ((_isProfiling = [SentryProfiler startWithTracer:_internalID])) {
+            SENTRY_LOG_DEBUG(@"Started profiler for trace %@ with internal id %@",
+                transactionContext.traceId.sentryIdString, _internalID.sentryIdString);
         }
+        _startSystemTime = SentryDependencyContainer.sharedInstance.dateProvider.systemTime;
     }
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
 
-    SENTRY_LOG_DEBUG(@"Started tracer with id: %@", transactionContext.traceId);
+    SENTRY_LOG_DEBUG(@"Started tracer with id: %@", transactionContext.traceId.sentryIdString);
 
     return self;
 }
@@ -522,10 +509,13 @@ - (void)finishInternal
 {
     [self cancelDeadlineTimer];
     if (self.isFinished) {
+        SENTRY_LOG_DEBUG(@"Tracer %@ was already finished.", _traceContext.traceId.sentryIdString);
         return;
     }
     @synchronized(self) {
         if (self.isFinished) {
+            SENTRY_LOG_DEBUG(@"Tracer %@ was already finished after synchronizing.",
+                _traceContext.traceId.sentryIdString);
             return;
         }
         // Keep existing status of auto generated transactions if set by the user.
@@ -566,7 +556,15 @@ - (void)finishInternal
         return;
     }
 
-    if (_hub == nil) {
+    BOOL shouldBailWithNilHub = _hub == nil;
+#if SENTRY_TARGET_PROFILING_SUPPORTED
+    if (isTracingAppLaunch) {
+        shouldBailWithNilHub = NO;
+    }
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED
+    if (shouldBailWithNilHub) {
+        SENTRY_LOG_DEBUG(
+            @"Hub was nil for tracer %@, nothing to do.", _traceContext.traceId.sentryIdString);
         return;
     }
 
@@ -603,15 +601,6 @@ - (void)finishInternal
 #if SENTRY_TARGET_PROFILING_SUPPORTED
     if (self.isProfiling) {
         [self captureTransactionWithProfile:transaction];
-
-        // as long as this isn't used for any conditional branching logic, and is just being set to
-        // NO, we don't need to synchronize the read here
-        if (isTracingAppLaunch) {
-            @synchronized(appLaunchTraceLock) {
-                isTracingAppLaunch = NO;
-            }
-        }
-
         return;
     }
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
diff --git a/Sources/Sentry/include/SentryLaunchProfiling.h b/Sources/Sentry/include/SentryLaunchProfiling.h
index 68c1094cac..cebed488b9 100644
--- a/Sources/Sentry/include/SentryLaunchProfiling.h
+++ b/Sources/Sentry/include/SentryLaunchProfiling.h
@@ -8,18 +8,20 @@
 @class SentryOptions;
 @class SentryTracerConfiguration;
 @class SentryTransactionContext;
-@class SentryTracer;
 
 NS_ASSUME_NONNULL_BEGIN
 
-SENTRY_EXTERN SentryTracer *launchTracer;
 SENTRY_EXTERN BOOL isTracingAppLaunch;
 SENTRY_EXTERN SentryId *_Nullable appLaunchTraceId;
 SENTRY_EXTERN uint64_t appLaunchSystemTime;
 SENTRY_EXTERN NSObject *appLaunchTraceLock;
 
+/** Try to start a profiled trace for this app launch, if the configuration allows. */
 void startLaunchProfile(void);
 
+/** Stop any profiled trace that may be in flight from the start of the app launch. */
+void stopLaunchProfile(void);
+
 /**
  * Write a file to disk containing sample rates for profiles and traces. The presence of this file
  * will let the profiler know to start on the app launch, and the sample rates contained will help

From a69b25e42ad9da1ef02cf27cb13272c25b5f4404 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 8 Feb 2024 14:00:44 -0900
Subject: [PATCH 28/34] wip: debug why SlowLoad frames don't appear in
 flamegraphs

---
 .../xcshareddata/xcschemes/iOS-Swift.xcscheme      | 12 ++++++++----
 Sources/Sentry/Profiling/SentryProfilerState.mm    | 11 ++++++++++-
 Sources/Sentry/SentryProfileTimeseries.mm          |  2 ++
 Sources/Sentry/SentryProfiler.mm                   | 14 +++++++++++---
 4 files changed, 31 insertions(+), 8 deletions(-)

diff --git a/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme b/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
index 72e340ada0..316624e952 100644
--- a/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
+++ b/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
@@ -74,15 +74,19 @@
             isEnabled = "NO">
          </CommandLineArgument>
          <CommandLineArgument
-            argument = "--disable-watchdog-tracking"
-            isEnabled = "NO">
+            argument = "--io.sentry.slow-load-method"
+            isEnabled = "YES">
          </CommandLineArgument>
          <CommandLineArgument
-            argument = "--disable-tracing"
+            argument = "--profile-app-launches"
+            isEnabled = "YES">
+         </CommandLineArgument>
+         <CommandLineArgument
+            argument = "--disable-watchdog-tracking"
             isEnabled = "NO">
          </CommandLineArgument>
          <CommandLineArgument
-            argument = "--io.sentry.test.profilesAppLaunches"
+            argument = "--disable-tracing"
             isEnabled = "NO">
          </CommandLineArgument>
          <CommandLineArgument
diff --git a/Sources/Sentry/Profiling/SentryProfilerState.mm b/Sources/Sentry/Profiling/SentryProfilerState.mm
index 59032cc9cd..77c14bb74b 100644
--- a/Sources/Sentry/Profiling/SentryProfilerState.mm
+++ b/Sources/Sentry/Profiling/SentryProfilerState.mm
@@ -115,6 +115,10 @@ - (void)appendBacktrace:(const Backtrace &)backtrace
 #    endif
 
         const auto stack = [NSMutableArray<NSNumber *> array];
+#    if defined(DEBUG)
+        printf("[Sentry] [debug] [SentryProfilerState:%d] stack frames: %lu; stack thread: %s\n",
+            __LINE__, backtrace.addresses.size(), threadID.UTF8String);
+#    endif // defined(DEBUG)
         for (std::vector<uintptr_t>::size_type backtraceAddressIdx = 0;
              backtraceAddressIdx < backtrace.addresses.size(); backtraceAddressIdx++) {
             const auto instructionAddress
@@ -125,8 +129,12 @@ - (void)appendBacktrace:(const Backtrace &)backtrace
                 const auto frame = [NSMutableDictionary<NSString *, id> dictionary];
                 frame[@"instruction_addr"] = instructionAddress;
 #    if defined(DEBUG)
-                frame[@"function"]
+                const auto functionName
                     = parseBacktraceSymbolsFunctionName(symbols[backtraceAddressIdx]);
+                printf("[Sentry] [debug] [SentryProfilerState:%d] function name: %s; instruction "
+                       "address: %s\n",
+                    __LINE__, functionName.UTF8String, instructionAddress.UTF8String);
+                frame[@"function"] = functionName;
 #    endif
                 const auto newFrameIndex = @(state.frames.count);
                 [stack addObject:newFrameIndex];
@@ -137,6 +145,7 @@ - (void)appendBacktrace:(const Backtrace &)backtrace
             }
         }
 #    if defined(DEBUG)
+        printf("-----finished backtrace-----\n");
         free(symbols);
 #    endif
 
diff --git a/Sources/Sentry/SentryProfileTimeseries.mm b/Sources/Sentry/SentryProfileTimeseries.mm
index 0a9ec459e1..4e9344d423 100644
--- a/Sources/Sentry/SentryProfileTimeseries.mm
+++ b/Sources/Sentry/SentryProfileTimeseries.mm
@@ -46,6 +46,8 @@
         return nil;
     }
 
+    SENTRY_LOG_DEBUG(@"Finding relevant samples from %lu total.", (unsigned long)samples.count);
+
     const auto firstIndex =
         [samples indexOfObjectWithOptions:NSEnumerationConcurrent
                               passingTest:^BOOL(SentrySample *_Nonnull sample, NSUInteger idx,
diff --git a/Sources/Sentry/SentryProfiler.mm b/Sources/Sentry/SentryProfiler.mm
index 329f71947b..2eb5edcc36 100644
--- a/Sources/Sentry/SentryProfiler.mm
+++ b/Sources/Sentry/SentryProfiler.mm
@@ -50,10 +50,16 @@
 #        import <UIKit/UIKit.h>
 #    endif // SENTRY_HAS_UIKIT
 
-#    if defined(TEST) || defined(TESTCI)
+#    if defined(TEST) || defined(TESTCI) || defined(DEBUG)
 #        import "SentryFileManager.h"
 #        import "SentryLaunchProfiling.h"
 
+/**
+ * A category that overrides its `+[load]` method to deliberately take a long time to run, so we can
+ * see it show up in profile stack traces. Categories' `+[load]` methods are guaranteed to be called
+ * after all of a module's normal class' overrides, so we can be confident the ordering will always
+ * have started the launch profiler by the time this runs.
+ */
 @interface
 SentryProfiler (SlowLoad)
 @end
@@ -62,17 +68,19 @@
 SentryProfiler (SlowLoad)
 + (void)load
 {
+    SENTRY_LOG_DEBUG(@"Starting slow load method");
     if ([NSProcessInfo.processInfo.arguments containsObject:@"--io.sentry.slow-load-method"]) {
         NSMutableString *a = [NSMutableString string];
         // 1,000,000 iterations takes about 225 milliseconds in the iPhone 15 simulator on an
         // M2 macbook pro; we might have to adapt this for CI
-        for (NSUInteger i = 0; i < 1000000; i++) {
+        for (NSUInteger i = 0; i < 4000000; i++) {
             [a appendFormat:@"%d", arc4random() % 12345];
         }
     }
+    SENTRY_LOG_DEBUG(@"Finishing slow load method");
 }
 @end
-#    endif // defined(TEST) || defined(TESTCI)
+#    endif // defined(TEST) || defined(TESTCI) || defined(DEBUG)
 
 const int kSentryProfilerFrequencyHz = 101;
 NSTimeInterval kSentryProfilerTimeoutInterval = 30;

From 715eaf7c5f69de05e298b207cd0f3423c60006dc Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Mon, 12 Feb 2024 14:26:22 -0800
Subject: [PATCH 29/34] fix merge conflict resolutions

---
 .../xcshareddata/xcschemes/iOS-Swift.xcscheme        | 12 ++++--------
 Sources/Sentry/Profiling/SentryLaunchProfiling.m     |  4 ----
 Sources/Sentry/SentryFileManager.m                   |  4 +---
 3 files changed, 5 insertions(+), 15 deletions(-)

diff --git a/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme b/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
index 316624e952..72e340ada0 100644
--- a/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
+++ b/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
@@ -73,14 +73,6 @@
             argument = "--disable-file-io-tracing"
             isEnabled = "NO">
          </CommandLineArgument>
-         <CommandLineArgument
-            argument = "--io.sentry.slow-load-method"
-            isEnabled = "YES">
-         </CommandLineArgument>
-         <CommandLineArgument
-            argument = "--profile-app-launches"
-            isEnabled = "YES">
-         </CommandLineArgument>
          <CommandLineArgument
             argument = "--disable-watchdog-tracking"
             isEnabled = "NO">
@@ -89,6 +81,10 @@
             argument = "--disable-tracing"
             isEnabled = "NO">
          </CommandLineArgument>
+         <CommandLineArgument
+            argument = "--io.sentry.test.profilesAppLaunches"
+            isEnabled = "NO">
+         </CommandLineArgument>
          <CommandLineArgument
             argument = "--disable-crash-handler"
             isEnabled = "NO">
diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index acf89bc91c..134cf132c8 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -156,10 +156,6 @@
 void
 stopLaunchProfile(void)
 {
-    NSDictionary<NSString *, NSNumber *> *rates = appLaunchProfileConfiguration();
-    removeAppLaunchProfilingConfigBackupFile();
-    NSNumber *profilesRate = rates[kSentryLaunchProfileConfigKeyProfilesSampleRate];
-    NSNumber *tracesRate = rates[kSentryLaunchProfileConfigKeyTracesSampleRate];
     if (launchTracer == nil) {
         SENTRY_LOG_DEBUG(@"No launch tracer present to stop.");
     }
diff --git a/Sources/Sentry/SentryFileManager.m b/Sources/Sentry/SentryFileManager.m
index f3ee9ccd19..b89b98a071 100644
--- a/Sources/Sentry/SentryFileManager.m
+++ b/Sources/Sentry/SentryFileManager.m
@@ -785,9 +785,7 @@ - (void)createPathsWithOptions:(SentryOptions *)options
 
     NSError *error;
     NSDictionary<NSString *, NSNumber *> *config =
-        dictionaryWithContentsOfURL:launchProfileConfigBackupFileURL()
-                              error:&error];
-    [NSDictionary<NSString *, NSNumber *> dictionaryWithContentsOfURL:url error:&error];
+        [NSDictionary<NSString *, NSNumber *> dictionaryWithContentsOfURL:url error:&error];
     SENTRY_CASSERT(
         error == nil, @"Encountered error trying to retrieve app launch profile config: %@", error);
     return config;

From 9ff69f0e12ac20c120ee6496ca4e92f07f9085ef Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 15 Feb 2024 12:27:44 +0800
Subject: [PATCH 30/34] remove requirements on auto perf, swizzling and uivc
 tracking

---
 .../iOS-SwiftUITests/ProfilingUITests.swift   | 21 +++++++--------
 .../Sentry/Profiling/SentryLaunchProfiling.m  | 26 +++----------------
 2 files changed, 13 insertions(+), 34 deletions(-)

diff --git a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
index 8e0297de1d..a675c6c7c0 100644
--- a/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
+++ b/Samples/iOS-Swift/iOS-SwiftUITests/ProfilingUITests.swift
@@ -16,7 +16,7 @@ class ProfilingUITests: BaseUITest {
         app.launchArguments.append("--io.sentry.wipe-data")
         launchApp()
         
-        // First launch enables in-app profiling by setting traces/profiles sample rates to 1 (which is the default configuration in the sample app), but not launch profiling; assert that we did not write a config to allow the next launch to be profiled
+        // First launch enables in-app profiling by setting traces/profiles sample rates to 1 (which is the default configuration in the sample app), but not launch profiling; assert that we did not write a config to allow the next launch to be profiled.
         try performAssertions(shouldProfileThisLaunch: false, shouldProfileNextLaunch: false)
         
         // no profiling should be done on this launch; set the option to allow launch profiling for the next launch, keeping the default numerical sampling rates of 1 for traces and profiles
@@ -28,20 +28,17 @@ class ProfilingUITests: BaseUITest {
         // this launch should not run the profiler; configure sampler functions returning 1 and numerical rates set to 0, which should result in a profile being taken as samplers override numerical rates
         try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, profilesSampleRate: 0, tracesSampleRate: 0, profilesSamplerValue: 1, tracesSamplerValue: 1)
         
-        // this launch has the configuration to run the profiler, but because swizzling is disabled, it will not be saved due to the ui.load transaction not being allowed to be created. it will also configure it to not run a profile for the next launch due to disabling swizzling, which would override the option to enable launch profiling. this specific scenario, where a previous launch configures a profile, but then something prevents the associated tx from running, is not automatically avoidable. in the future we will create a dummy transaction to attach the profile to.
-        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, shouldDisableSwizzling: true)
+        // this launch should run the profiler; configure sampler functions returning 0 and numerical rates set to 0, which should result in no profile being taken next launch
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: true, shouldEnableLaunchProfilingOptionForNextLaunch: true, profilesSamplerValue: 0, tracesSamplerValue: 0)
         
-        // this launch should not run the profiler and configure it not to run the next launch due to disabling automatic performance tracking, which would override the option to enable launch profiling
-        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, shouldDisableAutoPerformanceTracking: true)
-        
-        // this launch should not run the profiler and configure it not to run the next launch launch due to disabling UIViewController tracing, which would override the option to enable launch profiling
-        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, shouldDisableUIViewControllerTracing: true)
+        // this launch should not run the profiler, but configure it to run the next launch
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true)
         
-        // this launch should not run the profiler and configure it not to run the next launch launch due to disabling tracing, which would override the option to enable launch profiling
-        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true, shouldDisableTracing: true)
+        // this launch should run the profiler, and configure it not to run the next launch due to disabling tracing, which would override the option to enable launch profiling
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: true, shouldEnableLaunchProfilingOptionForNextLaunch: true, shouldDisableTracing: true)
         
-        // make sure the profiler respects the last configuration not to run
-        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: true)
+        // make sure the profiler respects the last configuration not to run; don't let another config file get written
+        try relaunchAndConfigureSubsequentLaunches(shouldProfileThisLaunch: false, shouldEnableLaunchProfilingOptionForNextLaunch: false)
     }
     
     /**
diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index 134cf132c8..ae805b120e 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -38,29 +38,11 @@
 SentryLaunchProfileConfig
 shouldProfileNextLaunch(SentryOptions *options)
 {
-    BOOL shouldProfileNextLaunch = options.enableAppLaunchProfiling
-#    if SENTRY_UIKIT_AVAILABLE
-        && options.enableUIViewControllerTracing && options.enableSwizzling
-#    endif // SENTRY_UIKIT_AVAILABLE
-        && options.enableAutoPerformanceTracing && options.enableTracing;
+    BOOL shouldProfileNextLaunch = options.enableAppLaunchProfiling && options.enableTracing;
     if (!shouldProfileNextLaunch) {
-#    if SENTRY_UIKIT_AVAILABLE
-        SENTRY_LOG_DEBUG(
-            @"Won't profile next launch due to specified options configuration: "
-            @"options.enableAppLaunchProfiling: %d; options.enableAutoPerformanceTracing: %d; "
-            @"options.enableUIViewControllerTracing: %d; options.enableSwizzling: %d; "
-            @"options.enableTracing: %d",
-            options.enableAppLaunchProfiling, options.enableAutoPerformanceTracing,
-            options.enableUIViewControllerTracing, options.enableSwizzling, options.enableTracing);
-#    else
-        SENTRY_LOG_DEBUG(
-            @"Won't profile next launch due to specified options configuration: "
-            @"options.enableAppLaunchProfiling: %d; options.enableAutoPerformanceTracing: %d; "
-            @"options.enableSwizzling: %d; options.enableTracing: %d",
-            options.enableAppLaunchProfiling, options.enableAutoPerformanceTracing,
-            options.enableSwizzling, options.enableTracing);
-#    endif // SENTRY_UIKIT_AVAILABLE
-
+        SENTRY_LOG_DEBUG(@"Won't profile next launch due to specified options configuration: "
+                         @"options.enableAppLaunchProfiling: %d; options.enableTracing: %d",
+            options.enableAppLaunchProfiling, options.enableTracing);
         return (SentryLaunchProfileConfig) { NO, nil, nil };
     }
 

From e28778c24d5256d820855d6c5e9a666c2a96df59 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 15 Feb 2024 12:45:23 +0800
Subject: [PATCH 31/34] update unit tests

---
 .../SentryAppLaunchProfilingTests.m           | 39 -------------------
 .../Helper/SentryFileManagerTests.swift       | 36 -----------------
 2 files changed, 75 deletions(-)

diff --git a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
index f622f9a0c9..6cd2b02563 100644
--- a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
+++ b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
@@ -107,45 +107,6 @@ - (void)testSettingProfilesSampleRateTo0DisablesAppLaunchProfiling
         @"but profiles sample rate of 0 should not enable launch profiling");
 }
 
-- (void)testDisablingAutoPerformanceTracingOptionDisablesAppLaunchProfiling
-{
-    XCTAssertFalse(
-        shouldProfileNextLaunch(
-            [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
-                                                                 enableAutoPerformanceTracing,
-                                                                 @NO) }])
-            .shouldProfile,
-        @"Default options with app launch profiling and tracing enabled, traces and profiles "
-        @"sample rates of 1, but automatic performance tracing disabled should not enable launch "
-        @"profiling");
-}
-
-#    if SENTRY_HAS_UIKIT
-- (void)testDisablingSwizzlingOptionDisablesAppLaunchProfiling
-{
-    XCTAssertFalse(
-        shouldProfileNextLaunch(
-            [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
-                                                                 enableSwizzling, @NO) }])
-            .shouldProfile,
-        @"Default options with app launch profiling and tracing enabled, traces and profiles "
-        @"sample rates of 1, but swizzling disabled should not enable launch profiling");
-}
-
-- (void)testDisablingUIViewControllerTracingOptionDisablesAppLaunchProfiling
-{
-    XCTAssertFalse(
-        shouldProfileNextLaunch(
-            [self defaultLaunchProfilingOptionsWithOverrides:@{ SENTRY_OPTION(
-                                                                 enableUIViewControllerTracing,
-                                                                 @NO) }])
-            .shouldProfile,
-        @"Default options with app launch profiling and tracing enabled, traces and profiles "
-        @"sample rates of 1, but UIViewController tracing disabled should not enable launch "
-        @"profiling");
-}
-#    endif // SENTRY_HAS_UIKIT
-
 - (SentryOptions *)defaultLaunchProfilingOptionsWithOverrides:
     (NSDictionary<NSString *, id> *)overrides
 {
diff --git a/Tests/SentryTests/Helper/SentryFileManagerTests.swift b/Tests/SentryTests/Helper/SentryFileManagerTests.swift
index 51e1b33f13..809329bd30 100644
--- a/Tests/SentryTests/Helper/SentryFileManagerTests.swift
+++ b/Tests/SentryTests/Helper/SentryFileManagerTests.swift
@@ -795,42 +795,6 @@ extension SentryFileManagerTests {
         removeAppLaunchProfilingConfigFile()
         expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
     }
-    
-    func testBackupAppLaunchProfilingConfigFile() throws {
-        try ensureAppLaunchProfileConfig(exists: true)
-        try ensureAppLaunchProfileConfig(exists: false, backup: true)
-        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) != nil
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
-        backupAppLaunchProfilingConfigFile()
-        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
-    }
-    
-    // if a file is still present in the backup location, like if a crash occurred before it could be removed, or an error occurred when trying to remove it, make sure we overwrite it
-    func testBackupAppLaunchProfilingConfigFile_anotherBackupFilePresent() throws {
-        try ensureAppLaunchProfileConfig(exists: true, tracesSampleRate: 0.1, profilesSampleRate: 0.2)
-        try ensureAppLaunchProfileConfig(exists: true, tracesSampleRate: 0.3, profilesSampleRate: 0.4, backup: true)
-        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) != nil
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
-        backupAppLaunchProfilingConfigFile()
-        expect(NSDictionary(contentsOf: launchProfileConfigFileURL())) == nil
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
-    }
-    
-    func testRemoveAppLaunchProfilingConfigBackupFile() throws {
-        try ensureAppLaunchProfileConfig(exists: true, backup: true)
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) != nil
-        removeAppLaunchProfilingConfigBackupFile()
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
-    }
-    
-    // if there's not a file when we expect one, just make sure we don't crash
-    func testRemoveAppLaunchProfilingConfigBackupFile_noFileExists() throws {
-        try ensureAppLaunchProfileConfig(exists: false, backup: true)
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
-        removeAppLaunchProfilingConfigBackupFile()
-        expect(NSDictionary(contentsOf: launchProfileConfigBackupFileURL())) == nil
-    }
 }
 
 // MARK: Private profiling tests

From e642a917f34c4aecb372c1d602b8cea0ed913de2 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 15 Feb 2024 14:12:15 +0800
Subject: [PATCH 32/34] remove nullable annotation

---
 Sources/Sentry/SentrySampling.m         | 2 +-
 Sources/Sentry/include/SentrySampling.h | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/Sources/Sentry/SentrySampling.m b/Sources/Sentry/SentrySampling.m
index 3edac3fcb7..a0eda6170b 100644
--- a/Sources/Sentry/SentrySampling.m
+++ b/Sources/Sentry/SentrySampling.m
@@ -84,7 +84,7 @@
 
 SentrySamplerDecision *
 sampleProfile(SentrySamplingContext *context, SentrySamplerDecision *tracesSamplerDecision,
-    SentryOptions *_Nullable options)
+    SentryOptions *options)
 {
     // Profiles are always undersampled with respect to traces. If the trace is not sampled,
     // the profile will not be either. If the trace is sampled, we can proceed to checking
diff --git a/Sources/Sentry/include/SentrySampling.h b/Sources/Sentry/include/SentrySampling.h
index c9a2b6a877..5c404b9113 100644
--- a/Sources/Sentry/include/SentrySampling.h
+++ b/Sources/Sentry/include/SentrySampling.h
@@ -18,7 +18,7 @@ SentrySamplerDecision *sampleTrace(SentrySamplingContext *context, SentryOptions
  * whether the trace corresponding to the profile was sampled.
  */
 SentrySamplerDecision *sampleProfile(SentrySamplingContext *context,
-    SentrySamplerDecision *tracesSamplerDecision, SentryOptions *_Nullable options);
+    SentrySamplerDecision *tracesSamplerDecision, SentryOptions *options);
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
 
 NS_ASSUME_NONNULL_END

From f45a7f74d0e48c87d621b26b6b5a8c02435379be Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 15 Feb 2024 16:33:53 +0800
Subject: [PATCH 33/34] fix(launch profiling): fix data wipe between
 integration test runs (#3633)

Co-authored-by: Philipp Hofmann <philipp.hofmann@sentry.io>
---
 .../iOS-Swift.xcodeproj/project.pbxproj       |  6 --
 .../xcshareddata/xcschemes/iOS-Swift.xcscheme | 12 +++-
 Samples/iOS-Swift/iOS-Swift/AppDelegate.swift | 21 +++++-
 Samples/iOS-Swift/iOS-Swift/AppStartup.h      | 17 -----
 Samples/iOS-Swift/iOS-Swift/AppStartup.m      | 26 --------
 .../Profiling/ProfilingViewController.swift   |  6 +-
 .../Sentry/Profiling/SentryLaunchProfiling.m  | 66 ++++++++++++-------
 Sources/Sentry/SentryProfiler.mm              | 24 ++++---
 Sources/Sentry/SentrySDK.m                    | 14 ++--
 Sources/Sentry/SentryTracer.m                 |  9 ---
 Sources/Sentry/include/SentryFileManager.h    | 14 ----
 .../Sentry/include/SentryLaunchProfiling.h    |  8 +--
 Sources/Sentry/include/SentryTraceOrigins.h   |  1 +
 .../SentryAppLaunchProfilingTests.m           | 12 ++++
 .../SentryTests/SentryLaunchProfiling+Tests.h |  7 ++
 develop-docs/README.md                        | 16 +++++
 16 files changed, 136 insertions(+), 123 deletions(-)
 delete mode 100644 Samples/iOS-Swift/iOS-Swift/AppStartup.h
 delete mode 100644 Samples/iOS-Swift/iOS-Swift/AppStartup.m

diff --git a/Samples/iOS-Swift/iOS-Swift.xcodeproj/project.pbxproj b/Samples/iOS-Swift/iOS-Swift.xcodeproj/project.pbxproj
index 3ed5c9a5fb..b722db6895 100644
--- a/Samples/iOS-Swift/iOS-Swift.xcodeproj/project.pbxproj
+++ b/Samples/iOS-Swift/iOS-Swift.xcodeproj/project.pbxproj
@@ -38,7 +38,6 @@
 		84B527BD28DD25E400475E8D /* SentryDevice.mm in Sources */ = {isa = PBXBuildFile; fileRef = 84B527BC28DD25E400475E8D /* SentryDevice.mm */; };
 		84BE546F287503F100ACC735 /* SentrySDKPerformanceBenchmarkTests.m in Sources */ = {isa = PBXBuildFile; fileRef = 84BE546E287503F100ACC735 /* SentrySDKPerformanceBenchmarkTests.m */; };
 		84BE547E287645B900ACC735 /* SentryProcessInfo.m in Sources */ = {isa = PBXBuildFile; fileRef = 84BE54792876451D00ACC735 /* SentryProcessInfo.m */; };
-		84DEE88E2B6A4D1200A7BC17 /* AppStartup.m in Sources */ = {isa = PBXBuildFile; fileRef = 84DEE88D2B6A4D1200A7BC17 /* AppStartup.m */; };
 		84FB812A284001B800F3A94A /* SentryBenchmarking.mm in Sources */ = {isa = PBXBuildFile; fileRef = 84FB8129284001B800F3A94A /* SentryBenchmarking.mm */; };
 		84FB812B284001B800F3A94A /* SentryBenchmarking.mm in Sources */ = {isa = PBXBuildFile; fileRef = 84FB8129284001B800F3A94A /* SentryBenchmarking.mm */; };
 		8E8C57AF25EF16E6001CEEFA /* TraceTestViewController.swift in Sources */ = {isa = PBXBuildFile; fileRef = 8E8C57AE25EF16E6001CEEFA /* TraceTestViewController.swift */; };
@@ -315,8 +314,6 @@
 		84BE546E287503F100ACC735 /* SentrySDKPerformanceBenchmarkTests.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentrySDKPerformanceBenchmarkTests.m; sourceTree = "<group>"; };
 		84BE54782876451D00ACC735 /* SentryProcessInfo.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = SentryProcessInfo.h; sourceTree = "<group>"; };
 		84BE54792876451D00ACC735 /* SentryProcessInfo.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentryProcessInfo.m; sourceTree = "<group>"; };
-		84DEE88C2B6A4D1200A7BC17 /* AppStartup.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = AppStartup.h; sourceTree = "<group>"; };
-		84DEE88D2B6A4D1200A7BC17 /* AppStartup.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = AppStartup.m; sourceTree = "<group>"; };
 		84FB8125284001B800F3A94A /* SentryBenchmarking.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = SentryBenchmarking.h; sourceTree = "<group>"; };
 		84FB8129284001B800F3A94A /* SentryBenchmarking.mm */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.cpp.objcpp; path = SentryBenchmarking.mm; sourceTree = "<group>"; };
 		84FB812C2840021B00F3A94A /* iOS-Swift-Bridging-Header.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; path = "iOS-Swift-Bridging-Header.h"; sourceTree = "<group>"; };
@@ -478,8 +475,6 @@
 				D8DBDA73274D4DF900007380 /* ViewControllers */,
 				63F93AA9245AC91600A500DB /* iOS-Swift.entitlements */,
 				637AFDA9243B02760034958B /* AppDelegate.swift */,
-				84DEE88C2B6A4D1200A7BC17 /* AppStartup.h */,
-				84DEE88D2B6A4D1200A7BC17 /* AppStartup.m */,
 				637AFDAD243B02760034958B /* TransactionsViewController.swift */,
 				84AB90782A50031B0054C99A /* Profiling */,
 				D80D021229EE93630084393D /* ErrorsViewController.swift */,
@@ -958,7 +953,6 @@
 				7B79000429028C7300A7F467 /* MetricKitManager.swift in Sources */,
 				D8D7BB4A2750067900044146 /* UIAssert.swift in Sources */,
 				D8F3D057274E574200B56F8C /* LoremIpsumViewController.swift in Sources */,
-				84DEE88E2B6A4D1200A7BC17 /* AppStartup.m in Sources */,
 				629EC8AD2B0B537400858855 /* ANRs.swift in Sources */,
 				D8DBDA78274D5FC400007380 /* SplitViewController.swift in Sources */,
 				84ACC43C2A73CB5900932A18 /* ProfilingNetworkScanner.swift in Sources */,
diff --git a/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme b/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
index 72e340ada0..dc3a836c25 100644
--- a/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
+++ b/Samples/iOS-Swift/iOS-Swift.xcodeproj/xcshareddata/xcschemes/iOS-Swift.xcscheme
@@ -73,6 +73,14 @@
             argument = "--disable-file-io-tracing"
             isEnabled = "NO">
          </CommandLineArgument>
+         <CommandLineArgument
+            argument = "--io.sentry.wipe-data"
+            isEnabled = "YES">
+         </CommandLineArgument>
+         <CommandLineArgument
+            argument = "--io.sentry.slow-load-method"
+            isEnabled = "YES">
+         </CommandLineArgument>
          <CommandLineArgument
             argument = "--disable-watchdog-tracking"
             isEnabled = "NO">
@@ -82,8 +90,8 @@
             isEnabled = "NO">
          </CommandLineArgument>
          <CommandLineArgument
-            argument = "--io.sentry.test.profilesAppLaunches"
-            isEnabled = "NO">
+            argument = "--profile-app-launches"
+            isEnabled = "YES">
          </CommandLineArgument>
          <CommandLineArgument
             argument = "--disable-crash-handler"
diff --git a/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift b/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift
index cc75237b20..e4eaf21e21 100644
--- a/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift
+++ b/Samples/iOS-Swift/iOS-Swift/AppDelegate.swift
@@ -130,9 +130,10 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
 
     func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplication.LaunchOptionsKey: Any]?) -> Bool {
         
-        print("[iOS-Swift] launch arguments: \(ProcessInfo.processInfo.arguments)")
-        print("[iOS-Swift] environment: \(ProcessInfo.processInfo.environment)")
+        print("[iOS-Swift] [debug] launch arguments: \(ProcessInfo.processInfo.arguments)")
+        print("[iOS-Swift] [debug] environment: \(ProcessInfo.processInfo.environment)")
         
+        maybeWipeData()
         AppDelegate.startSentry()
         
         if #available(iOS 15.0, *) {
@@ -162,3 +163,19 @@ class AppDelegate: UIResponder, UIApplicationDelegate {
         // swiftlint:enable force_cast
     }
 }
+
+private extension AppDelegate {
+    // previously tried putting this in an AppDelegate.load override in ObjC, but it wouldn't run until after a launch profiler would have an opportunity to run, since SentryProfiler.load would always run first due to being dynamically linked in a framework module. it is sufficient to do it before calling SentrySDK.startWithOptions to clear state for testProfiledAppLaunches because we don't make any assertions on a launch profile the first launch of the app in that test 
+    func maybeWipeData() {
+        if ProcessInfo.processInfo.arguments.contains("--io.sentry.wipe-data") {
+            print("[iOS-Swift] [debug] removing app data")
+            let appSupport = NSSearchPathForDirectoriesInDomains(.applicationSupportDirectory, .userDomainMask, true).first!
+            let cache = NSSearchPathForDirectoriesInDomains(.cachesDirectory, .userDomainMask, true).first!
+            for path in [appSupport, cache] {
+                for item in FileManager.default.enumerator(atPath: path)! {
+                    try! FileManager.default.removeItem(atPath: (path as NSString).appendingPathComponent((item as! String)))
+                }
+            }
+        }
+    }
+}
diff --git a/Samples/iOS-Swift/iOS-Swift/AppStartup.h b/Samples/iOS-Swift/iOS-Swift/AppStartup.h
deleted file mode 100644
index 1b7bb41899..0000000000
--- a/Samples/iOS-Swift/iOS-Swift/AppStartup.h
+++ /dev/null
@@ -1,17 +0,0 @@
-//
-//  AppStartup.h
-//  iOS-Swift
-//
-//  Created by Andrew McKnight on 1/31/24.
-//  Copyright © 2024 Sentry. All rights reserved.
-//
-
-#import <Foundation/Foundation.h>
-
-NS_ASSUME_NONNULL_BEGIN
-
-@interface AppStartup : NSObject
-
-@end
-
-NS_ASSUME_NONNULL_END
diff --git a/Samples/iOS-Swift/iOS-Swift/AppStartup.m b/Samples/iOS-Swift/iOS-Swift/AppStartup.m
deleted file mode 100644
index f3d7488e68..0000000000
--- a/Samples/iOS-Swift/iOS-Swift/AppStartup.m
+++ /dev/null
@@ -1,26 +0,0 @@
-#import "AppStartup.h"
-
-@implementation AppStartup
-
-// we must do this in objective c, because it's not permitted to be overridden in Swift
-+ (void)load
-{
-    if ([NSProcessInfo.processInfo.arguments containsObject:@"--io.sentry.wipe-data"]) {
-        NSLog(@"[iOS-Swift] removing app data");
-        NSString *appSupport = [NSSearchPathForDirectoriesInDomains(
-            NSApplicationSupportDirectory, NSUserDomainMask, true) firstObject];
-        NSString *cache = [NSSearchPathForDirectoriesInDomains(
-            NSCachesDirectory, NSUserDomainMask, true) firstObject];
-        NSFileManager *fm = NSFileManager.defaultManager;
-        for (NSString *dir in @[ appSupport, cache ]) {
-            for (NSString *file in [fm enumeratorAtPath:dir]) {
-                NSError *error;
-                if (![fm removeItemAtPath:[dir stringByAppendingPathComponent:file] error:&error]) {
-                    NSLog(@"[iOS-Swift] failed to remove data at app startup.");
-                }
-            }
-        }
-    }
-}
-
-@end
diff --git a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
index ece70bdbbd..5634db0680 100644
--- a/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
+++ b/Samples/iOS-Swift/iOS-Swift/Profiling/ProfilingViewController.swift
@@ -46,7 +46,7 @@ class ProfilingViewController: UIViewController, UITextFieldDelegate {
         let value = SentryBenchmarking.stopBenchmark()!
         valueTextField.isHidden = false
         valueTextField.text = value
-        print("[iOS-Swift] [ProfilingViewController] benchmarking results:\n\(value)")
+        print("[iOS-Swift] [debug] [ProfilingViewController] benchmarking results:\n\(value)")
     }
     
     @IBAction func startCPUWork(_ sender: UIButton) {
@@ -130,7 +130,7 @@ extension ProfilingViewController {
             let url = file as! URL
             if url.absoluteString.contains(fileName) {
                 block(url)
-                print("[iOS-Swift] [ProfilingViewController] removing file at \(url)")
+                print("[iOS-Swift] [debug] [ProfilingViewController] removing file at \(url)")
                 try! FileManager.default.removeItem(at: url)
                 return
             }
@@ -150,7 +150,7 @@ extension ProfilingViewController {
             return
         }
         let contents = data.base64EncodedString()
-        print("[iOS-Swift] [ProfilingViewController] contents of file at \(file): \(String(describing: String(data: data, encoding: .utf8)))")
+        print("[iOS-Swift] [debug] [ProfilingViewController] contents of file at \(file): \(String(describing: String(data: data, encoding: .utf8)))")
         profilingUITestDataMarshalingTextField.text = contents
         profilingUITestDataMarshalingStatus.text = "✅"
     }
diff --git a/Sources/Sentry/Profiling/SentryLaunchProfiling.m b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
index ae805b120e..b89fc60bd4 100644
--- a/Sources/Sentry/Profiling/SentryLaunchProfiling.m
+++ b/Sources/Sentry/Profiling/SentryLaunchProfiling.m
@@ -15,17 +15,17 @@
 #    import "SentrySamplerDecision.h"
 #    import "SentrySampling.h"
 #    import "SentrySamplingContext.h"
-#    import "SentryTracer.h"
+#    import "SentryTraceOrigins.h"
+#    import "SentryTracer+Private.h"
 #    import "SentryTracerConfiguration.h"
-#    import "SentryTransactionContext.h"
+#    import "SentryTransactionContext+Private.h"
+
+NS_ASSUME_NONNULL_BEGIN
 
 BOOL isTracingAppLaunch;
-SentryId *_Nullable appLaunchTraceId;
-NSObject *appLaunchTraceLock;
-uint64_t appLaunchSystemTime;
 NSString *const kSentryLaunchProfileConfigKeyTracesSampleRate = @"traces";
 NSString *const kSentryLaunchProfileConfigKeyProfilesSampleRate = @"profiles";
-SentryTracer *_Nullable launchTracer;
+static SentryTracer *_Nullable launchTracer;
 
 #    pragma mark - Private
 
@@ -91,6 +91,29 @@
     }];
 }
 
+SentryTransactionContext *
+context(NSNumber *tracesRate)
+{
+    SentryTransactionContext *context =
+        [[SentryTransactionContext alloc] initWithName:@"launch"
+                                            nameSource:kSentryTransactionNameSourceCustom
+                                             operation:@"app.lifecycle"
+                                                origin:SentryTraceOriginAutoAppStartProfile
+                                               sampled:kSentrySampleDecisionYes];
+    context.sampleRate = tracesRate;
+    return context;
+}
+
+SentryTracerConfiguration *
+config(NSNumber *profilesRate)
+{
+    SentryTracerConfiguration *config = [SentryTracerConfiguration defaultConfiguration];
+    config.profilesSamplerDecision =
+        [[SentrySamplerDecision alloc] initWithDecision:kSentrySampleDecisionYes
+                                          forSampleRate:profilesRate];
+    return config;
+}
+
 void
 startLaunchProfile(void)
 {
@@ -111,39 +134,36 @@
         [SentryLog configure:YES diagnosticLevel:kSentryLevelDebug];
 #    endif // defined(DEBUG)
 
-        appLaunchSystemTime = SentryDependencyContainer.sharedInstance.dateProvider.systemTime;
-        appLaunchTraceLock = [[NSObject alloc] init];
-        appLaunchTraceId = [[SentryId alloc] init];
-
-        SENTRY_LOG_INFO(@"Starting app launch profile at %llu", appLaunchSystemTime);
-
-        SentryTransactionContext *context =
-            [[SentryTransactionContext alloc] initWithName:@"launch" operation:@"app.lifecycle"];
-        SentryTracerConfiguration *config = [SentryTracerConfiguration defaultConfiguration];
         NSDictionary<NSString *, NSNumber *> *rates = appLaunchProfileConfiguration();
         NSNumber *profilesRate = rates[kSentryLaunchProfileConfigKeyProfilesSampleRate];
         NSNumber *tracesRate = rates[kSentryLaunchProfileConfigKeyTracesSampleRate];
-        if (profilesRate != nil && tracesRate != nil) {
-            config.profilesSamplerDecision =
-                [[SentrySamplerDecision alloc] initWithDecision:kSentrySampleDecisionYes
-                                                  forSampleRate:profilesRate];
-            context.sampleRate = tracesRate;
+        if (profilesRate == nil || tracesRate == nil) {
+            SENTRY_LOG_DEBUG(
+                @"Received a nil configured launch sample rate, will not trace or profile.");
+            return;
         }
-        launchTracer = [[SentryTracer alloc] initWithTransactionContext:context
+
+        SENTRY_LOG_INFO(@"Starting app launch profile.");
+        launchTracer = [[SentryTracer alloc] initWithTransactionContext:context(tracesRate)
                                                                     hub:nil
-                                                          configuration:config];
+                                                          configuration:config(profilesRate)];
     });
 }
 
 void
-stopLaunchProfile(void)
+stopLaunchProfile(SentryHub *hub)
 {
     if (launchTracer == nil) {
         SENTRY_LOG_DEBUG(@"No launch tracer present to stop.");
+        return;
     }
 
     SENTRY_LOG_DEBUG(@"Finishing launch tracer.");
+
+    launchTracer.hub = hub;
     [launchTracer finish];
 }
 
+NS_ASSUME_NONNULL_END
+
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
diff --git a/Sources/Sentry/SentryProfiler.mm b/Sources/Sentry/SentryProfiler.mm
index 2eb5edcc36..b98753280e 100644
--- a/Sources/Sentry/SentryProfiler.mm
+++ b/Sources/Sentry/SentryProfiler.mm
@@ -52,6 +52,7 @@
 
 #    if defined(TEST) || defined(TESTCI) || defined(DEBUG)
 #        import "SentryFileManager.h"
+#        import "SentryInternalDefines.h"
 #        import "SentryLaunchProfiling.h"
 
 /**
@@ -436,7 +437,7 @@ + (nullable SentryEnvelopeItem *)createEnvelopeItemForProfilePayload:
     return [[SentryEnvelopeItem alloc] initWithHeader:header data:JSONData];
 }
 
-#    if defined(TEST) || defined(TESTCI)
+#    if defined(TEST) || defined(TESTCI) || defined(DEBUG)
 void
 writeProfileFile(NSDictionary<NSString *, id> *payload)
 {
@@ -460,20 +461,25 @@ + (nullable SentryEnvelopeItem *)createEnvelopeItemForProfilePayload:
 
     NSString *pathToWrite;
     if (isTracingAppLaunch) {
+        SENTRY_LOG_DEBUG(@"Writing app launch profile.");
         pathToWrite = [appSupportDirPath stringByAppendingPathComponent:@"launchProfile"];
-        if ([fm fileExistsAtPath:pathToWrite]) {
-            SENTRY_LOG_DEBUG(@"Already a launch profile file present.");
-            return;
-        }
     } else {
+        SENTRY_LOG_DEBUG(@"Overwriting last non-launch profile.");
         pathToWrite = [appSupportDirPath stringByAppendingPathComponent:@"profile"];
     }
 
-    SENTRY_LOG_DEBUG(@"Writing app launch profile to file.");
+    if ([fm fileExistsAtPath:pathToWrite]) {
+        SENTRY_LOG_DEBUG(@"Already a%@ profile file present; make sure to remove them right after "
+                         @"using them, and that tests clean state in between so there isn't "
+                         @"leftover config producing one when it isn't expected.",
+            isTracingAppLaunch ? @" launch" : @"");
+    }
+
+    SENTRY_LOG_DEBUG(@"Writing%@ profile to file.", isTracingAppLaunch ? @" launch" : @"");
     SENTRY_CASSERT(
         [data writeToFile:pathToWrite atomically:YES], @"Failed to write profile to test file");
 }
-#    endif // defined(TEST) || defined(TESTCI)
+#    endif // defined(TEST) || defined(TESTCI) || defined(DEBUG)
 
 + (nullable NSMutableDictionary<NSString *, id> *)collectProfileBetween:(uint64_t)startSystemTime
                                                                     and:(uint64_t)endSystemTime
@@ -487,9 +493,9 @@ + (nullable SentryEnvelopeItem *)createEnvelopeItemForProfilePayload:
 
     const auto payload = [profiler serializeBetween:startSystemTime and:endSystemTime onHub:hub];
 
-#    if defined(TEST) || defined(TESTCI)
+#    if defined(TEST) || defined(TESTCI) || defined(DEBUG)
     writeProfileFile(payload);
-#    endif // defined(TEST) || defined(TESTCI)
+#    endif // defined(TEST) || defined(TESTCI) || defined(DEBUG)
 
     return payload;
 }
diff --git a/Sources/Sentry/SentrySDK.m b/Sources/Sentry/SentrySDK.m
index 34e083a982..4493999fc0 100644
--- a/Sources/Sentry/SentrySDK.m
+++ b/Sources/Sentry/SentrySDK.m
@@ -164,10 +164,6 @@ + (void)setStartTimestamp:(NSDate *)value
 
 + (void)startWithOptions:(SentryOptions *)options
 {
-#if SENTRY_TARGET_PROFILING_SUPPORTED
-    stopLaunchProfile();
-#endif // SENTRY_TARGET_PROFILING_SUPPORTED
-
     [SentryLog configure:options.debug diagnosticLevel:options.diagnosticLevel];
 
     // We accept the tradeoff that the SDK might not be fully initialized directly after
@@ -190,7 +186,8 @@ + (void)startWithOptions:(SentryOptions *)options
         = options.initialScope([[SentryScope alloc] initWithMaxBreadcrumbs:options.maxBreadcrumbs]);
     // The Hub needs to be initialized with a client so that closing a session
     // can happen.
-    [SentrySDK setCurrentHub:[[SentryHub alloc] initWithClient:newClient andScope:scope]];
+    SentryHub *hub = [[SentryHub alloc] initWithClient:newClient andScope:scope];
+    [SentrySDK setCurrentHub:hub];
     SENTRY_LOG_DEBUG(@"SDK initialized! Version: %@", SentryMeta.versionString);
 
     SENTRY_LOG_DEBUG(@"Dispatching init work required to run on main thread.");
@@ -204,11 +201,14 @@ + (void)startWithOptions:(SentryOptions *)options
 #if TARGET_OS_IOS && SENTRY_HAS_UIKIT
         [SentryDependencyContainer.sharedInstance.uiDeviceWrapper start];
 #endif // TARGET_OS_IOS && SENTRY_HAS_UIKIT
-    }];
 
 #if SENTRY_TARGET_PROFILING_SUPPORTED
-    configureLaunchProfiling(options);
+        [SentryDependencyContainer.sharedInstance.dispatchQueueWrapper dispatchAsyncWithBlock:^{
+            stopLaunchProfile(hub);
+            configureLaunchProfiling(options);
+        }];
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
+    }];
 }
 
 + (void)startWithConfigureOptions:(void (^)(SentryOptions *options))configureOptions
diff --git a/Sources/Sentry/SentryTracer.m b/Sources/Sentry/SentryTracer.m
index a0188c06a8..883efc3490 100644
--- a/Sources/Sentry/SentryTracer.m
+++ b/Sources/Sentry/SentryTracer.m
@@ -609,15 +609,6 @@ - (void)finishInternal
 #if SENTRY_TARGET_PROFILING_SUPPORTED
     if (self.isProfiling) {
         [self captureTransactionWithProfile:transaction];
-
-        // as long as this isn't used for any conditional branching logic, and is just being set to
-        // NO, we don't need to synchronize the read here
-        if (isTracingAppLaunch) {
-            @synchronized(appLaunchTraceLock) {
-                isTracingAppLaunch = NO;
-            }
-        }
-
         return;
     }
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
diff --git a/Sources/Sentry/include/SentryFileManager.h b/Sources/Sentry/include/SentryFileManager.h
index 8d3322bf01..cb774591c7 100644
--- a/Sources/Sentry/include/SentryFileManager.h
+++ b/Sources/Sentry/include/SentryFileManager.h
@@ -124,20 +124,6 @@ SENTRY_EXTERN void writeAppLaunchProfilingConfigFile(
  */
 SENTRY_EXTERN void removeAppLaunchProfilingConfigFile(void);
 
-/**
- * Save a current launch profile config file for use when the associated transaction needs to be
- * started. This is when checking @c SentryOptions for whether to write a config for the next
- * launch; if there's no current app launch profile going, then we'd simply remove whatever config
- * file may exist, but if there is a launch profile going, we need to save the config that it used.
- */
-SENTRY_EXTERN void backupAppLaunchProfilingConfigFile(void);
-
-/**
- * Clean up the backup file saved off in the case that an app launch is in-flight when the SDK
- * configures subsequent launches not to profile, requiring the current config to be saved for the
- * associated transaction. This can be cleaned up when that transaction is started.
- */
-SENTRY_EXTERN void removeAppLaunchProfilingConfigBackupFile(void);
 #endif // SENTRY_TARGET_PROFILING_SUPPORTED
 
 @end
diff --git a/Sources/Sentry/include/SentryLaunchProfiling.h b/Sources/Sentry/include/SentryLaunchProfiling.h
index cebed488b9..d7d9863439 100644
--- a/Sources/Sentry/include/SentryLaunchProfiling.h
+++ b/Sources/Sentry/include/SentryLaunchProfiling.h
@@ -4,6 +4,7 @@
 
 #if SENTRY_TARGET_PROFILING_SUPPORTED
 
+@class SentryHub;
 @class SentryId;
 @class SentryOptions;
 @class SentryTracerConfiguration;
@@ -12,15 +13,12 @@
 NS_ASSUME_NONNULL_BEGIN
 
 SENTRY_EXTERN BOOL isTracingAppLaunch;
-SENTRY_EXTERN SentryId *_Nullable appLaunchTraceId;
-SENTRY_EXTERN uint64_t appLaunchSystemTime;
-SENTRY_EXTERN NSObject *appLaunchTraceLock;
 
 /** Try to start a profiled trace for this app launch, if the configuration allows. */
-void startLaunchProfile(void);
+SENTRY_EXTERN void startLaunchProfile(void);
 
 /** Stop any profiled trace that may be in flight from the start of the app launch. */
-void stopLaunchProfile(void);
+void stopLaunchProfile(SentryHub *hub);
 
 /**
  * Write a file to disk containing sample rates for profiles and traces. The presence of this file
diff --git a/Sources/Sentry/include/SentryTraceOrigins.h b/Sources/Sentry/include/SentryTraceOrigins.h
index 9be87b76a7..7922cdb9a7 100644
--- a/Sources/Sentry/include/SentryTraceOrigins.h
+++ b/Sources/Sentry/include/SentryTraceOrigins.h
@@ -4,6 +4,7 @@ static NSString *const SentryTraceOriginManual = @"manual";
 static NSString *const SentryTraceOriginUIEventTracker = @"auto.ui.event_tracker";
 
 static NSString *const SentryTraceOriginAutoAppStart = @"auto.app.start";
+static NSString *const SentryTraceOriginAutoAppStartProfile = @"auto.app.start.profile";
 static NSString *const SentryTraceOriginAutoNSData = @"auto.file.ns_data";
 static NSString *const SentryTraceOriginAutoDBCoreData = @"auto.db.core_data";
 static NSString *const SentryTraceOriginAutoHttpNSURLSession = @"auto.http.ns_url_session";
diff --git a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
index 6cd2b02563..45270a23cb 100644
--- a/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
+++ b/Tests/SentryProfilerTests/SentryAppLaunchProfilingTests.m
@@ -3,6 +3,8 @@
 #import "SentryOptions+Private.h"
 #import "SentryProfilingConditionals.h"
 #import "SentrySDK+Tests.h"
+#import "SentryTraceOrigins.h"
+#import "SentryTransactionContext.h"
 #import <XCTest/XCTest.h>
 
 #if SENTRY_TARGET_PROFILING_SUPPORTED
@@ -12,6 +14,14 @@ @interface SentryAppLaunchProfilingTests : XCTestCase
 
 @implementation SentryAppLaunchProfilingTests
 
+- (void)testLaunchProfileTransactionContext
+{
+    SentryTransactionContext *actualContext = context(@1);
+    XCTAssertEqual(actualContext.nameSource, kSentryTransactionNameSourceCustom);
+    XCTAssert([actualContext.origin isEqualToString:SentryTraceOriginAutoAppStartProfile]);
+    XCTAssert(actualContext.sampled);
+}
+
 #    define SENTRY_OPTION(name, value)                                                             \
         NSStringFromSelector(@selector(name))                                                      \
             : value
@@ -107,6 +117,8 @@ - (void)testSettingProfilesSampleRateTo0DisablesAppLaunchProfiling
         @"but profiles sample rate of 0 should not enable launch profiling");
 }
 
+#    pragma mark - Private
+
 - (SentryOptions *)defaultLaunchProfilingOptionsWithOverrides:
     (NSDictionary<NSString *, id> *)overrides
 {
diff --git a/Tests/SentryTests/SentryLaunchProfiling+Tests.h b/Tests/SentryTests/SentryLaunchProfiling+Tests.h
index 92cbff552e..c4f1ba31d6 100644
--- a/Tests/SentryTests/SentryLaunchProfiling+Tests.h
+++ b/Tests/SentryTests/SentryLaunchProfiling+Tests.h
@@ -1,6 +1,8 @@
 #import "SentryDefines.h"
 #import "SentryLaunchProfiling.h"
 
+#if SENTRY_TARGET_PROFILING_SUPPORTED
+
 @class SentrySamplerDecision;
 @class SentryOptions;
 
@@ -17,4 +19,9 @@ SENTRY_EXTERN SentryLaunchProfileConfig shouldProfileNextLaunch(SentryOptions *o
 SENTRY_EXTERN NSString *const kSentryLaunchProfileConfigKeyTracesSampleRate;
 SENTRY_EXTERN NSString *const kSentryLaunchProfileConfigKeyProfilesSampleRate;
 
+SENTRY_EXTERN SentryTransactionContext *context(NSNumber *tracesRate);
+SENTRY_EXTERN SentryTracerConfiguration *config(NSNumber *profilesRate);
+
 NS_ASSUME_NONNULL_END
+
+#endif // SENTRY_TARGET_PROFILING_SUPPORTED
diff --git a/develop-docs/README.md b/develop-docs/README.md
index b0548c4241..43967e649f 100644
--- a/develop-docs/README.md
+++ b/develop-docs/README.md
@@ -111,3 +111,19 @@ This feature is experimental and is currently not compatible with SPM.
 ## Logging
 
 We have a set of macros for debugging at various levels defined in SentryLog.h. These are not async-safe; to log from special places like crash handlers, see SentryCrashLogger.h; see the headerdocs in that header for how to work with those logging macros. There are also separate macros in SentryProfilingLogging.hpp specifically for the profiler; these are completely compiled out of release builds due to https://github.com/getsentry/sentry-cocoa/issues/3336.
+
+## Profiling
+
+The profiler runs on a dedicated thread, and on a predefined interval will enumerate all other threads and gather the backtrace on each non-idle thread. 
+
+The information is stored in deduplicated frame and stack indexed lookups for memory and transmission efficiency. These are maintained in `SentryProfilerState`.
+
+If enabled and sampled in (controlled by `SentryOptions.profilesSampleRate` or `SentryOptions.profilesSampler`), the profiler will start along with a trace, and the profile information is sliced to the start and end of each transaction and sent with them an envelope attachments. 
+
+The profiler will automatically time out if it is not stopped within 30 seconds, and also stops automatically if the app is sent to the background.
+
+There's only ever one profiler instance running at a time, but instances that have timed out will be kept in memory until all traces that ran concurrently with it have finished and serialized to envelopes. The associations between profiler instances and traces are maintained in `SentryProfiledTracerConcurrency`. 
+
+App launches can be automatically profiled if configured with `SentryOptions.enableAppLaunchProfiling`. If enabled, when `SentrySDK.startWithOptions` is called, `SentryLaunchProfiling.configureLaunchProfiling` will get a sample rate for traces and profiles with their respective options, and store those rates in a file to be read on the next launch. On each launch, `SentryLaunchProfiling.startLaunchProfile` checks for the presence of that file is used to decide whether to start an app launch profiled trace, and afterwards retrieves those rates to initialize a `SentryTransactionContext` and `SentryTracerConfiguration`, and provides them to a new `SentryTracer` instance, which is what actually starts the profiler. There is no hub at this time; also in the call to `SentrySDK.startWithOptions`, any current profiled launch trace is attempted to be finished, and the hub that exists by that time is provided to the `SentryTracer` instance via `SentryLaunchProfiling.stopLaunchProfile` so that when it needs to transmit the transaction envelope, the infrastructure is in place to do so.
+
+In testing and debug environments, when a profile payload is serialized for transmission, the dictionary will also be written to a file in application support that can be retrieved by a sample app. This helps with UI tests that want to verify the contents of a profile after some app interaction. See `iOS-Swift.ProfilingViewController.viewLastProfile`  and `iOS-SwiftUITests.ProfilingUITests`.

From 807ac4789d085ecb7e93db581caac7d9bcfa5ec1 Mon Sep 17 00:00:00 2001
From: Andrew McKnight <andrew.mcknight@sentry.io>
Date: Thu, 15 Feb 2024 16:44:40 +0800
Subject: [PATCH 34/34] fix changelog

---
 CHANGELOG.md | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 0de68b055a..1bb8aa2067 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## Unreleased
 
+### Features
+
+- Automatically profile app launches (#3529)
+
 ### Improvements
 
 - Cache installationID async to avoid file IO on the main thread when starting the SDK (#3601)
@@ -9,7 +13,7 @@
 ### Fixes
 
 - Finish TTID span when transaction finishes (#3610)
-- Dont take screenshot and view hierarchy for app hanging (#3620)
+- Don't take screenshot and view hierarchy for app hanging (#3620)
 
 ## 8.20.0
 
@@ -17,7 +21,6 @@
 
 - Add visionOS as device family (#3548)
 - Add VisionOS Support for Carthage (#3565)
-- Automatically profile app launches (#3529)
 
 ### Fixes