fixes

Author: polatolu

.github/pull_request_template.md

@@ -24,6 +24,6 @@
 - [ ] 📝 Documentation
 - [ ] 🗑️ Chore
 
-## Estimated time to fix the ticket(s) or epic(s) refernced by the PR in days
+## Estimated time to fix the ticket(s) or epic(s) referenced by the PR in days
 
 <!--- Add estimate to complete the work -->

FlagsmithClient/Classes/Flagsmith.swift

@@ -25,35 +25,30 @@ public final class Flagsmith: @unchecked Sendable {
     /// User-Agent header value for HTTP requests
     /// Format: flagsmith-swift-ios-sdk/<version>
     /// Falls back to "unknown" if version is not discoverable at runtime
-    public static var userAgent: String {
+    public static let userAgent: String {
         let version = getSDKVersion()
         return "flagsmith-swift-ios-sdk/\(version)"
     }
 
     /// Get the SDK version from the bundle at runtime
     /// Falls back to hardcoded constant or "unknown" if version is not discoverable
     private static func getSDKVersion() -> String {
-        // Try to get version from the FlagsmithClient bundle first (CocoaPods)
+        // Try CocoaPods bundle first
         if let bundle = Bundle(identifier: "org.cocoapods.FlagsmithClient"),
-           let version = bundle.infoDictionary?["CFBundleShortVersionString"] as? String, 
-           !version.isEmpty && version != "1.0" {
+        let version = bundle.infoDictionary?["CFBundleShortVersionString"] as? String, 
+        !version.isEmpty,
+        version.range(of: #"^\d+\.\d+\.\d+"#, options: .regularExpression) != nil {
             return version
         }
 
-        // Try to get version from the current bundle (for SPM or direct integration)
-        if let version = Bundle(for: Flagsmith.self).infoDictionary?["CFBundleShortVersionString"] as? String, 
-           !version.isEmpty && version != "1.0" {
+        // Try SPM bundle
+        if let version = Bundle(for: Flagsmith.self).infoDictionary?["CFBundleShortVersionString"] as? String,
+        !version.isEmpty,
+        version.range(of: #"^\d+\.\d+\.\d+"#, options: .regularExpression) != nil {
             return version
         }
 
-        // Try to get version from the main bundle as last resort (but avoid app versions)
-        if let version = Bundle.main.infoDictionary?["CFBundleShortVersionString"] as? String, 
-           !version.isEmpty && version != "1.0" && version.range(of: #"^\d+\.\d+\.\d+$"#, options: NSString.CompareOptions.regularExpression) != nil {
-            return version
-        }
-        
-        // Fallback to hardcoded SDK version constant
-        return sdkVersionConstant
+        return "unknown"
     }
     private let apiManager: APIManager
     private let sseManager: SSEManager

FlagsmithClient/Classes/Internal/Router.swift

@@ -100,9 +100,9 @@ enum Router: Sendable {
         if let body = try body(using: encoder) {
             request.httpBody = body
         }
-        request.addValue(apiKey, forHTTPHeaderField: "X-Environment-Key")
-        request.addValue("application/json", forHTTPHeaderField: "Content-Type")
-        request.addValue(Flagsmith.userAgent, forHTTPHeaderField: "User-Agent")
+        request.setValue(apiKey, forHTTPHeaderField: "X-Environment-Key")
+        request.setValue("application/json", forHTTPHeaderField: "Content-Type")
+        request.setValue(Flagsmith.userAgent, forHTTPHeaderField: "User-Agent")
 
         return request
     }

FlagsmithClient/Tests/RouterTests.swift

@@ -39,8 +39,9 @@ final class RouterTests: FlagsmithClientTestCase {
         let expectedPattern = "^flagsmith-swift-ios-sdk/[0-9]+\\.[0-9]+\\.[0-9]+$"
         let regex = try NSRegularExpression(pattern: expectedPattern)
         let range = NSRange(location: 0, length: userAgent?.count ?? 0)
-        XCTAssertTrue(regex.firstMatch(in: userAgent ?? "", options: [], range: range) != nil, 
-                     "User-Agent should match pattern 'flagsmith-swift-ios-sdk/<version>', got: \(userAgent ?? "nil")")
+        let match = regex.firstMatch(in: userAgent ?? "", options: [], range: range)
+        let message = "User-Agent should match pattern 'flagsmith-swift-ios-sdk/<version>', got: \(userAgent ?? "nil")"
+        XCTAssertTrue(match != nil, message)
     }
 
     func testUserAgentHeaderFormat() {