Provide Sendable conformance in all other places

Author: Jeehut

Package.swift

@@ -1,4 +1,4 @@
-// swift-tools-version:5.9
+// swift-tools-version: 5.9
 import PackageDescription
 
 let package = Package(

Sources/TelemetryClient/LogHandler.swift

@@ -1,7 +1,7 @@
 import Foundation
 
-public struct LogHandler {
-    public enum LogLevel: Int, CustomStringConvertible {
+public struct LogHandler: Sendable {
+    public enum LogLevel: Int, CustomStringConvertible, Sendable {
         case debug = 0
         case info = 1
         case error = 2
@@ -19,9 +19,9 @@ public struct LogHandler {
     }
 
     let logLevel: LogLevel
-    let handler: (LogLevel, String) -> Void
+    let handler: @Sendable (LogLevel, String) -> Void
 
-    public init(logLevel: LogHandler.LogLevel, handler: @escaping (LogHandler.LogLevel, String) -> Void) {
+    public init(logLevel: LogHandler.LogLevel, handler: @escaping @Sendable (LogHandler.LogLevel, String) -> Void) {
         self.logLevel = logLevel
         self.handler = handler
     }
@@ -32,7 +32,7 @@ public struct LogHandler {
         }
     }
 
-    public static var stdout = { logLevel in
+    public static func stdout(_ logLevel: LogLevel) -> LogHandler {
         LogHandler(logLevel: logLevel) { level, message in
             print("[TelemetryDeck: \(level.description)] \(message)")
         }

Sources/TelemetryClient/SignalCache.swift

@@ -7,7 +7,7 @@ import Foundation
 /// correctly.
 ///
 /// Currently the cache is only in-memory. This will probably change in the near future.
-internal class SignalCache<T> where T: Codable {
+internal class SignalCache<T>: @unchecked Sendable where T: Codable {
     internal var logHandler: LogHandler?
 
     private var cachedSignals: [T] = []

Sources/TelemetryClient/SignalEnricher.swift

@@ -1,6 +1,6 @@
 import Foundation
 
-public protocol SignalEnricher {
+public protocol SignalEnricher: Sendable {
     func enrich(
         signalType: String,
         for clientUser: String?,

Sources/TelemetryClient/SignalManager.swift

@@ -104,6 +104,7 @@ internal final class SignalManager: SignalManageable, @unchecked Sendable {
     /// Sends one batch of signals from the cache if not empty.
     /// If signals fail to send, we put them back into the cache to try again later.
     @objc
+    @Sendable
     internal func attemptToSendNextBatchOfCachedSignals() {
         configuration.logHandler?.log(.debug, message: "Current signal cache count: \(signalCache.count())")
 

Sources/TelemetryClient/TelemetryClient.swift

@@ -182,7 +182,7 @@ public struct TelemetryManagerConfiguration: Sendable {
 /// Accepts signals that signify events in your app's life cycle, collects and caches them, and pushes them to the Telemetry API.
 ///
 /// Use an instance of `TelemetryManagerConfiguration` to configure this at initialization and during its lifetime.
-public class TelemetryManager {
+public final class TelemetryManager: @unchecked Sendable {
     /// Returns `true` when the TelemetryManager already has been initialized correctly, `false` otherwise.
     public static var isInitialized: Bool {
         initializedTelemetryManager != nil
@@ -337,6 +337,7 @@ public class TelemetryManager {
         self.signalManager = signalManager
     }
 
+    nonisolated(unsafe)
     private static var initializedTelemetryManager: TelemetryManager?
 
     private var configuration: TelemetryManagerConfiguration