Remove comment

Author: ncooke3

FirebaseSessions/Sources/Development/DevEventConsoleLogger.swift

@@ -19,7 +19,7 @@ import Foundation
   import FirebaseSessionsObjC
 #endif // SWIFT_PACKAGE
 
-class DevEventConsoleLogger: EventGDTLoggerProtocol {
+final class DevEventConsoleLogger: EventGDTLoggerProtocol {
   private let commandLineArgument = "-FIRSessionsDebugEvents"
 
   func logEvent(event: SessionStartEvent, completion: @escaping (Result<Void, Error>) -> Void) {

FirebaseSessions/Sources/EventGDTLogger.swift

@@ -17,7 +17,7 @@ import Foundation
 
 internal import GoogleDataTransport
 
-protocol EventGDTLoggerProtocol {
+protocol EventGDTLoggerProtocol: Sendable {
   func logEvent(event: SessionStartEvent, completion: @escaping (Result<Void, Error>) -> Void)
 }
 
@@ -26,7 +26,7 @@ protocol EventGDTLoggerProtocol {
 ///   1) Creating GDT Events and logging them to the GoogleDataTransport SDK
 ///   2) Handling debugging situations (eg. running in Simulator or printing the event to console)
 ///
-class EventGDTLogger: EventGDTLoggerProtocol {
+final class EventGDTLogger: EventGDTLoggerProtocol {
   let googleDataTransport: GoogleDataTransportProtocol
   let devEventConsoleLogger: EventGDTLoggerProtocol
 

FirebaseSessions/Sources/FirebaseSessions.swift

@@ -62,13 +62,13 @@ private enum GoogleDataTransportConfig {
 
   // Initializes the SDK and top-level classes
   required convenience init(appID: String, installations: InstallationsProtocol) {
-    let googleDataTransport = GDTCORTransport(
+    let googleDataTransport = GoogleDataTransporter(
       mappingID: GoogleDataTransportConfig.sessionsLogSource,
       transformers: nil,
       target: GoogleDataTransportConfig.sessionsTarget
     )
 
-    let fireLogger = EventGDTLogger(googleDataTransport: googleDataTransport!)
+    let fireLogger = EventGDTLogger(googleDataTransport: googleDataTransport)
 
     let appInfo = ApplicationInfo(appID: appID)
     let settings = SessionsSettings(

FirebaseSessions/Sources/GoogleDataTransport+GoogleDataTransportProtocol.swift

@@ -15,20 +15,31 @@
 
 import Foundation
 
-internal import GoogleDataTransport
+@preconcurrency internal import GoogleDataTransport
 
 enum GoogleDataTransportProtocolErrors: Error {
   case writeFailure
 }
 
-protocol GoogleDataTransportProtocol {
+protocol GoogleDataTransportProtocol: Sendable {
   func logGDTEvent(event: GDTCOREvent, completion: @escaping (Result<Void, Error>) -> Void)
   func eventForTransport() -> GDTCOREvent
 }
 
-extension GDTCORTransport: GoogleDataTransportProtocol {
-  func logGDTEvent(event: GDTCOREvent, completion: @escaping (Result<Void, Error>) -> Void) {
-    sendDataEvent(event) { wasWritten, error in
+/// Workaround in combo with preconcurrency import of GDT. When GDT's
+/// `GDTCORTransport`type conforms to Sendable within the GDT module,
+/// this can be removed.
+final class GoogleDataTransporter: GoogleDataTransportProtocol {
+  private let transporter: GDTCORTransport
+
+  init(mappingID: String,
+       transformers: [any GDTCOREventTransformer]?,
+       target: GDTCORTarget) {
+    transporter = GDTCORTransport(mappingID: mappingID, transformers: transformers, target: target)!
+  }
+
+  func logGDTEvent(event: GDTCOREvent, completion: @escaping (Result<Void, any Error>) -> Void) {
+    transporter.sendDataEvent(event) { wasWritten, error in
       if let error {
         completion(.failure(error))
       } else if !wasWritten {
@@ -38,4 +49,8 @@ extension GDTCORTransport: GoogleDataTransportProtocol {
       }
     }
   }
+
+  func eventForTransport() -> GDTCOREvent {
+    transporter.eventForTransport()
+  }
 }

FirebaseSessions/Sources/SessionCoordinator.swift

@@ -26,8 +26,7 @@ protocol SessionCoordinatorProtocol: Sendable {
 final class SessionCoordinator: SessionCoordinatorProtocol {
   let installations: InstallationsProtocol
 
-  // TODO: Make this type sendable
-  nonisolated(unsafe) let fireLogger: EventGDTLoggerProtocol
+  let fireLogger: EventGDTLoggerProtocol
 
   init(installations: InstallationsProtocol,
        fireLogger: EventGDTLoggerProtocol) {

FirebaseSessions/Tests/Unit/Mocks/MockGDTLogger.swift

@@ -17,7 +17,8 @@ import Foundation
 
 @testable import FirebaseSessions
 
-class MockGDTLogger: EventGDTLoggerProtocol {
+// TODO(Swift 6): Add checked Sendable support.
+final class MockGDTLogger: EventGDTLoggerProtocol, @unchecked Sendable {
   var loggedEvent: SessionStartEvent?
   var result: Result<Void, Error> = .success(())