Improve startup plugin collection. (#45)

Co-authored-by: Brandon Sneed <brandon.sneed@segment.com>

Author: bsneed

Sources/Segment/Plugins/Platforms/Vendors/AppleUtils.swift

@@ -76,6 +76,10 @@ internal class iOSVendorSystem: VendorSystem {
         return connectionStatus()
     }
 
+    override var requiredPlugins: [PlatformPlugin] {
+        return [iOSLifecycleMonitor(), DeviceToken()]
+    }
+    
     private func deviceModel() -> String {
         var name: [Int32] = [CTL_HW, HW_MACHINE]
         var size: Int = 2
@@ -157,6 +161,10 @@ internal class watchOSVendorSystem: VendorSystem {
         return ConnectionStatus.unknown
     }
 
+    override var requiredPlugins: [PlatformPlugin] {
+        return [watchOSLifecycleMonitor()]
+    }
+    
     private func deviceModel() -> String {
         var name: [Int32] = [CTL_HW, HW_MACHINE]
         var size: Int = 2
@@ -227,6 +235,10 @@ internal class MacOSVendorSystem: VendorSystem {
         return connectionStatus()
     }
 
+    override var requiredPlugins: [PlatformPlugin] {
+        return [macOSLifecycleMonitor(), DeviceToken()]
+    }
+    
     private func deviceModel() -> String {
         var systemInfo = utsname()
         uname(&systemInfo)

Sources/Segment/Plugins/Platforms/Vendors/LinuxUtils.swift

@@ -49,6 +49,10 @@ class LinuxVendorSystem: VendorSystem {
     override var connection: ConnectionStatus {
         return ConnectionStatus.unknown
     }
+    
+    override var requiredPlugins: [PlatformPlugin] {
+        return []
+    }
 }
 
 #endif

Sources/Segment/Plugins/Platforms/Vendors/VendorSystem.swift

@@ -65,7 +65,11 @@ internal class VendorSystem {
         return ConnectionStatus.unknown
     }
 
-    static var current: VendorSystem {
+    var requiredPlugins: [PlatformPlugin] {
+        return []
+    }
+    
+    static var current: VendorSystem = {
         #if os(iOS) || os(tvOS) || targetEnvironment(macCatalyst)
         return iOSVendorSystem()
         #elseif os(macOS)
@@ -77,5 +81,5 @@ internal class VendorSystem {
         #else
         return VendorSystem()
         #endif
-    }
+    }()
 }

Sources/Segment/Startup.swift

@@ -40,20 +40,24 @@ extension Analytics: Subscriber {
         // add context plugin as well as it's platform specific internally.
         // this must come first.
         plugins.append(Context())
+        
+        plugins += VendorSystem.current.requiredPlugins
 
         // setup lifecycle if desired
         if configuration.values.trackApplicationLifecycleEvents {
             #if os(iOS) || os(tvOS)
-            plugins += [iOSLifecycleMonitor(), iOSLifecycleEvents(), DeviceToken()]
+            plugins.append(iOSLifecycleEvents())
             #endif
             #if os(watchOS)
-            plugins += [watchOSLifecycleMonitor(), watchOSLifecycleEvents()]
+            plugins.append(watchOSLifecycleEvents())
             #endif
             #if os(macOS)
-            plugins += [macOSLifecycleMonitor(), DeviceToken()]
+            // placeholder - need to build this
+            //plugins.append(macOSLifecycleEvents())
             #endif
             #if os(Linux)
-            plugins.append(LinuxLifecycleMonitor())
+            // placeholder - not sure what this is yet
+            //plugins.append(LinuxLifecycleMonitor())
             #endif
         }