Extract SFHFKeychainUtils to dedicated Swift package

Modules/Package.swift

@@ -11,6 +11,7 @@ let package = Package(
         .library(name: "AsyncImageKit", targets: ["AsyncImageKit"]),
         .library(name: "DesignSystem", targets: ["DesignSystem"]),
         .library(name: "JetpackStatsWidgetsCore", targets: ["JetpackStatsWidgetsCore"]),
+        .library(name: "SFHFKeychainUtils", targets: ["SFHFKeychainUtils"]),
         .library(name: "WordPressData", targets: ["WordPressData"]),
         .library(name: "WordPressFlux", targets: ["WordPressFlux"]),
         .library(name: "WordPressShared", targets: ["WordPressShared"]),
@@ -60,6 +61,12 @@ let package = Package(
         ]),
         .target(name: "DesignSystem", swiftSettings: [.swiftLanguageMode(.v5)]),
         .target(name: "JetpackStatsWidgetsCore", swiftSettings: [.swiftLanguageMode(.v5)]),
+        // SFHFKeychainUtils is an old Objective-C keychain wrapper.
+        // The implementatoin predates ARC, hence the dedicated target with ARC disabled, for the time being.
+        .target(
+            name: "SFHFKeychainUtils",
+            cSettings: [.unsafeFlags(["-fno-objc-arc"])]
+        ),
         .target(name: "UITestsFoundation", dependencies: [
             .product(name: "ScreenObject", package: "ScreenObject"),
             .product(name: "XCUITestHelpers", package: "ScreenObject"),
@@ -146,6 +153,7 @@ enum XcodeSupport {
         ]
 
         let shareAndDraftExtensionsDependencies: [Target.Dependency] = [
+            "SFHFKeychainUtils",
             "WordPressShared",
             "WordPressUI",
             .product(name: "CocoaLumberjackSwift", package: "CocoaLumberjack"),
@@ -168,6 +176,7 @@ enum XcodeSupport {
             .xcodeTarget("XcodeTarget_App", dependencies: [
                 "DesignSystem",
                 "JetpackStatsWidgetsCore",
+                "SFHFKeychainUtils",
                 "WordPressData",
                 "WordPressFlux",
                 "WordPressShared",
@@ -206,6 +215,7 @@ enum XcodeSupport {
                 .product(name: "WordPressEditor", package: "AztecEditor-iOS"),
             ]),
             .xcodeTarget("XcodeTarget_WordPressTests", dependencies: testDependencies + [
+                "SFHFKeychainUtils",
                 "WordPressShared",
                 .product(name: "Gravatar", package: "Gravatar-SDK-iOS"),
                 .product(name: "Nimble", package: "Nimble"),
@@ -216,10 +226,12 @@ enum XcodeSupport {
             .xcodeTarget("XcodeTarget_ShareExtension", dependencies: shareAndDraftExtensionsDependencies),
             .xcodeTarget("XcodeTarget_DraftActionExtension", dependencies: shareAndDraftExtensionsDependencies),
             .xcodeTarget("XcodeTarget_NotificationServiceExtension", dependencies: [
+                "SFHFKeychainUtils",
                 "WordPressShared",
             ]),
             .xcodeTarget("XcodeTarget_StatsWidget", dependencies: [
                 "JetpackStatsWidgetsCore",
+                "SFHFKeychainUtils",
                 "WordPressShared",
                 "WordPressUI",
                 .product(name: "CocoaLumberjackSwift", package: "CocoaLumberjack"),

WordPress/Classes/Utility/SFHFKeychainUtils.h → Modules/Sources/SFHFKeychainUtils/include/SFHFKeychainUtils.h

@@ -27,7 +27,7 @@
 //  OTHER DEALINGS IN THE SOFTWARE.
 //
 
-#import <UIKit/UIKit.h>
+#import <Foundation/Foundation.h>
 
 
 @interface SFHFKeychainUtils : NSObject {
@@ -68,4 +68,4 @@
 + (NSArray<NSDictionary<NSString *, NSString *> *> *)getAllPasswordsForAccessGroup:(NSString *)accessGroup
                                                                              error:(NSError **)error;
 
-@end
+@end

WordPress/Classes/Models/Blog/Blog.m

@@ -5,6 +5,7 @@
 #import "Constants.h"
 #import "WPUserAgent.h"
 #import "WordPress-Swift.h"
+@import SFHFKeychainUtils;
 
 @import NSObject_SafeExpectations;
 @import NSURL_IDN;

WordPress/Classes/Models/WPAccount.m

@@ -1,3 +1,4 @@
+@import SFHFKeychainUtils;
 #import "WPAccount.h"
 #import "WordPress-Swift.h"
 

WordPress/Classes/Services/CredentialsService.swift

@@ -1,3 +1,5 @@
+import SFHFKeychainUtils
+
 protocol CredentialsProvider {
     func getPassword(username: String, service: String) -> String?
 }

WordPress/Classes/Services/NotificationSupportService.swift

@@ -1,4 +1,5 @@
 import Foundation
+import SFHFKeychainUtils
 
 @objc
 open class NotificationSupportService: NSObject {

WordPress/Classes/Services/ShareExtensionService.swift

@@ -1,4 +1,5 @@
 import Foundation
+import SFHFKeychainUtils
 
 @objc
 open class ShareExtensionService: NSObject {

WordPress/Classes/Stores/StatsWidgetsStore.swift

@@ -1,4 +1,5 @@
 import JetpackStatsWidgetsCore
+import SFHFKeychainUtils
 import WidgetKit
 
 class StatsWidgetsStore {

WordPress/Classes/System/WordPress-Bridging-Header.h

@@ -1,5 +1,3 @@
-#import "SFHFKeychainUtils.h"
-
 // WordPress Module
 
 #import "ActivityLogViewController.h"
@@ -55,7 +53,6 @@
 #import "SettingTableViewCell.h"
 #import "SettingsTextViewController.h"
 #import "SharingViewController.h"
-#import "SFHFKeychainUtils.h"
 #import "SiteSettingsViewController.h"
 #import "SourcePostAttribution.h"
 #import "StatsViewController.h"

WordPress/Classes/System/WordPressAppDelegate.swift

@@ -1,3 +1,4 @@
+import SFHFKeychainUtils
 import UIKit
 import CocoaLumberjackSwift
 import Reachability

WordPress/Classes/Utility/FormattableContent/Actions/FormattableContentActionCommand.swift

@@ -1,3 +1,5 @@
+import UIKit
+
 /// Abstracts the logic behind contextual actions that can be applied to FormattableContent.
 ///
 protocol FormattableContentActionCommand: CustomStringConvertible {

WordPress/Classes/Utility/FormattableContent/FormattableContentStyles.swift

@@ -1,3 +1,4 @@
+import UIKit
 
 /// Styles definition to be applied to a single FormattableContent entity.
 public protocol FormattableContentStyles {

WordPress/Classes/Utility/KeychainUtils.swift

@@ -1,3 +1,5 @@
+import SFHFKeychainUtils
+
 @objcMembers
 class KeychainUtils: NSObject {
 

WordPress/Classes/Utility/Migrations/10-11/BlogToAccount.m

@@ -5,6 +5,7 @@
 
 @import NSObject_SafeExpectations;
 @import NSURL_IDN;
+@import SFHFKeychainUtils;
 
 @implementation BlogToAccount {
     NSString *_defaultWpcomUsername;

WordPress/Classes/Utility/Migrations/10-11/BlogToJetpackAccount.m

@@ -3,6 +3,7 @@
 #import "WordPress-Swift.h"
 
 @import NSObject_SafeExpectations;
+@import SFHFKeychainUtils;
 
 static NSString * const BlogJetpackKeychainPrefix = @"jetpackblog-";
 

WordPress/Classes/Utility/Migrations/32-33/BlogToBlog32to33.swift

@@ -1,3 +1,4 @@
+import SFHFKeychainUtils
 import UIKit
 
 class BlogToBlog32to33: NSEntityMigrationPolicy {

WordPress/Classes/ViewRelated/Aztec/Extensions/FormatBarItemProviders.swift

@@ -1,6 +1,7 @@
 import Foundation
 import Gridicons
 import Aztec
+import UIKit
 
 protocol FormatBarItemProvider {
     var iconImage: UIImage { get }

WordPress/Classes/ViewRelated/Aztec/Extensions/TextList+WordPress.swift

@@ -1,5 +1,6 @@
 import Foundation
 import Aztec
+import UIKit
 
 // MARK: - TextList.Style
 //

WordPress/Classes/ViewRelated/NUX/Helpers/WordPressAuthenticationManager.swift

@@ -1,4 +1,5 @@
 import Foundation
+import SFHFKeychainUtils
 import WordPressAuthenticator
 import Gridicons
 import UIKit

WordPress/Classes/ViewRelated/Notifications/FormattableContent/NotificationTextContent.swift

@@ -1,5 +1,5 @@
-
 import Foundation
+import UIKit
 
 extension FormattableContentKind {
     static let image = FormattableContentKind("image")

WordPress/JetpackStatsWidgets/Remote service/StatsWidgetsService.swift

@@ -1,3 +1,4 @@
+import SFHFKeychainUtils
 import WordPressKit
 import JetpackStatsWidgetsCore
 import WordPressShared

WordPress/JetpackStatsWidgets/Supporting Files/JetpackStatsWidgets-Bridging-Header.h

@@ -1,2 +1 @@
-#import "SFHFKeychainUtils.h"
 #import "Constants.h"

WordPress/Shared/SharePost.swift

@@ -1,4 +1,5 @@
 import Foundation
+import UIKit
 import MobileCoreServices
 
 /// A simple shared model to represent a Site

WordPress/WordPress.xcodeproj/project.pbxproj

@@ -3297,9 +3297,6 @@
 /* Begin PBXFileSystemSynchronizedBuildFileExceptionSet section */
 		24CE68AE2CD3375300C7B37D /* Exceptions for "Classes" folder in "WordPress" target */ = {
 			isa = PBXFileSystemSynchronizedBuildFileExceptionSet;
-			additionalCompilerFlagsByRelativePath = {
-				Utility/SFHFKeychainUtils.m = "-fno-objc-arc";
-			};
 			membershipExceptions = (
 				"Utility/Migrations/20-21/SafeReaderTopicToReaderTopic.m",
 				"ViewRelated/Me/All Domains/Coordinators/AllDomainsAddDomainCoordinator.swift",
@@ -3314,9 +3311,6 @@
 		};
 		24CE68AF2CD3375300C7B37D /* Exceptions for "Classes" folder in "WordPressShareExtension" target */ = {
 			isa = PBXFileSystemSynchronizedBuildFileExceptionSet;
-			additionalCompilerFlagsByRelativePath = {
-				Utility/SFHFKeychainUtils.m = "-fno-objc-arc";
-			};
 			membershipExceptions = (
 				"Extensions/Colors and Styles/WPStyleGuide+ApplicationStyles.swift",
 				"Extensions/Colors and Styles/WPStyleGuide+Gridicon.swift",
@@ -3337,7 +3331,6 @@
 				Utility/BuildInformation/FeatureFlagRolloutStore.swift,
 				Utility/KeychainUtils.swift,
 				Utility/Logging/CocoaLumberjack.swift,
-				Utility/SFHFKeychainUtils.m,
 				ViewRelated/Aztec/Extensions/FormatBarItemProviders.swift,
 				"ViewRelated/Aztec/Extensions/Header+WordPress.swift",
 				"ViewRelated/Aztec/Extensions/TextList+WordPress.swift",
@@ -3351,9 +3344,6 @@
 		};
 		24CE68B02CD3375300C7B37D /* Exceptions for "Classes" folder in "WordPressDraftActionExtension" target */ = {
 			isa = PBXFileSystemSynchronizedBuildFileExceptionSet;
-			additionalCompilerFlagsByRelativePath = {
-				Utility/SFHFKeychainUtils.m = "-fno-objc-arc";
-			};
 			membershipExceptions = (
 				"Extensions/Colors and Styles/WPStyleGuide+ApplicationStyles.swift",
 				"Extensions/Colors and Styles/WPStyleGuide+Gridicon.swift",
@@ -3374,7 +3364,6 @@
 				Utility/BuildInformation/FeatureFlagRolloutStore.swift,
 				Utility/KeychainUtils.swift,
 				Utility/Logging/CocoaLumberjack.swift,
-				Utility/SFHFKeychainUtils.m,
 				ViewRelated/Aztec/Extensions/FormatBarItemProviders.swift,
 				"ViewRelated/Aztec/Extensions/Header+WordPress.swift",
 				"ViewRelated/Aztec/Extensions/TextList+WordPress.swift",
@@ -3388,9 +3377,6 @@
 		};
 		24CE68B22CD3375300C7B37D /* Exceptions for "Classes" folder in "WordPressNotificationServiceExtension" target */ = {
 			isa = PBXFileSystemSynchronizedBuildFileExceptionSet;
-			additionalCompilerFlagsByRelativePath = {
-				Utility/SFHFKeychainUtils.m = "-fno-objc-arc";
-			};
 			membershipExceptions = (
 				"Extensions/String+Helpers.swift",
 				"Extensions/UIImage+Extensions.swift",
@@ -3416,7 +3402,6 @@
 				Utility/FormattableContent/FormattableRangesFactory.swift,
 				Utility/FormattableContent/FormattableTextContent.swift,
 				Utility/KeychainUtils.swift,
-				Utility/SFHFKeychainUtils.m,
 				ViewRelated/Notifications/FormattableContent/FormattableCommentContent.swift,
 				ViewRelated/Notifications/FormattableContent/FormattableUserContent.swift,
 				ViewRelated/Notifications/FormattableContent/Notifiable.swift,
@@ -3446,9 +3431,6 @@
 		};
 		24CE68B52CD3375300C7B37D /* Exceptions for "Classes" folder in "Jetpack" target */ = {
 			isa = PBXFileSystemSynchronizedBuildFileExceptionSet;
-			additionalCompilerFlagsByRelativePath = {
-				Utility/SFHFKeychainUtils.m = "-fno-objc-arc";
-			};
 			membershipExceptions = (
 				"Utility/App Configuration/AppConfiguration.swift",
 				"Utility/App Configuration/AppConstants.swift",
@@ -3460,9 +3442,6 @@
 		};
 		24CE68B62CD3375300C7B37D /* Exceptions for "Classes" folder in "JetpackShareExtension" target */ = {
 			isa = PBXFileSystemSynchronizedBuildFileExceptionSet;
-			additionalCompilerFlagsByRelativePath = {
-				Utility/SFHFKeychainUtils.m = "-fno-objc-arc";
-			};
 			membershipExceptions = (
 				"Extensions/Colors and Styles/WPStyleGuide+ApplicationStyles.swift",
 				"Extensions/Colors and Styles/WPStyleGuide+Gridicon.swift",
@@ -3481,7 +3460,6 @@
 				Utility/BuildInformation/FeatureFlagRolloutStore.swift,
 				Utility/KeychainUtils.swift,
 				Utility/Logging/CocoaLumberjack.swift,
-				Utility/SFHFKeychainUtils.m,
 				ViewRelated/Aztec/Extensions/FormatBarItemProviders.swift,
 				"ViewRelated/Aztec/Extensions/Header+WordPress.swift",
 				"ViewRelated/Aztec/Extensions/TextList+WordPress.swift",
@@ -3495,9 +3473,6 @@
 		};
 		24CE68B72CD3375300C7B37D /* Exceptions for "Classes" folder in "JetpackDraftActionExtension" target */ = {
 			isa = PBXFileSystemSynchronizedBuildFileExceptionSet;
-			additionalCompilerFlagsByRelativePath = {
-				Utility/SFHFKeychainUtils.m = "-fno-objc-arc";
-			};
 			membershipExceptions = (
 				"Extensions/Colors and Styles/WPStyleGuide+ApplicationStyles.swift",
 				"Extensions/Colors and Styles/WPStyleGuide+Gridicon.swift",
@@ -3516,7 +3491,6 @@
 				Utility/BuildInformation/FeatureFlagRolloutStore.swift,
 				Utility/KeychainUtils.swift,
 				Utility/Logging/CocoaLumberjack.swift,
-				Utility/SFHFKeychainUtils.m,
 				ViewRelated/Aztec/Extensions/FormatBarItemProviders.swift,
 				"ViewRelated/Aztec/Extensions/Header+WordPress.swift",
 				"ViewRelated/Aztec/Extensions/TextList+WordPress.swift",
@@ -3530,9 +3504,6 @@
 		};
 		24CE68B82CD3375300C7B37D /* Exceptions for "Classes" folder in "JetpackNotificationServiceExtension" target */ = {
 			isa = PBXFileSystemSynchronizedBuildFileExceptionSet;
-			additionalCompilerFlagsByRelativePath = {
-				Utility/SFHFKeychainUtils.m = "-fno-objc-arc";
-			};
 			membershipExceptions = (
 				"Extensions/String+Helpers.swift",
 				"Extensions/UIImage+Extensions.swift",
@@ -3556,7 +3527,6 @@
 				Utility/FormattableContent/FormattableRangesFactory.swift,
 				Utility/FormattableContent/FormattableTextContent.swift,
 				Utility/KeychainUtils.swift,
-				Utility/SFHFKeychainUtils.m,
 				ViewRelated/Notifications/FormattableContent/FormattableCommentContent.swift,
 				ViewRelated/Notifications/FormattableContent/FormattableUserContent.swift,
 				ViewRelated/Notifications/FormattableContent/Notifiable.swift,
@@ -3571,9 +3541,6 @@
 		};
 		24CE68B92CD3375300C7B37D /* Exceptions for "Classes" folder in "JetpackStatsWidgets" target */ = {
 			isa = PBXFileSystemSynchronizedBuildFileExceptionSet;
-			additionalCompilerFlagsByRelativePath = {
-				Utility/SFHFKeychainUtils.m = "-fno-objc-arc";
-			};
 			membershipExceptions = (
 				Services/KeyValueDatabase.swift,
 				System/Constants/Constants.m,
@@ -3586,21 +3553,16 @@
 				Utility/BuildInformation/FeatureFlagRolloutStore.swift,
 				Utility/KeychainUtils.swift,
 				Utility/Logging/CocoaLumberjack.swift,
-				Utility/SFHFKeychainUtils.m,
 				"ViewRelated/Stats/Extensions/Double+Stats.swift",
 				"ViewRelated/Stats/Extensions/NumberFormatter+Stats.swift",
 			);
 			target = 0107E0B128F97D5000DE87DB /* JetpackStatsWidgets */;
 		};
 		24CE68BA2CD3375300C7B37D /* Exceptions for "Classes" folder in "JetpackIntents" target */ = {
 			isa = PBXFileSystemSynchronizedBuildFileExceptionSet;
-			additionalCompilerFlagsByRelativePath = {
-				Utility/SFHFKeychainUtils.m = "-fno-objc-arc";
-			};
 			membershipExceptions = (
 				System/Constants/Constants.m,
 				Utility/Logging/CocoaLumberjack.swift,
-				Utility/SFHFKeychainUtils.m,
 			);
 			target = 0107E13828FE9DB200DE87DB /* JetpackIntents */;
 		};

WordPress/WordPressDraftActionExtension/WordPressDraftActionExtension-Bridging-Header.h

@@ -2,5 +2,4 @@
 #import "Constants.h"
 
 // Shared Helpers
-#import "SFHFKeychainUtils.h"
 #import "TextBundleWrapper.h"

WordPress/WordPressNotificationServiceExtension/Sources/NotificationService.swift

@@ -1,3 +1,4 @@
+import SFHFKeychainUtils
 import UserNotifications
 
 import WordPressKit

WordPress/WordPressNotificationServiceExtension/Sources/WordPressNotificationServiceExtension-Bridging-Header.h

@@ -1,5 +1,2 @@
 // Main App Helpers
 #import "Constants.h"
-
-// Shared Helpers
-#import "SFHFKeychainUtils.h"

WordPress/WordPressShareExtension/UINavigationController+Extensions.swift

@@ -1,4 +1,5 @@
 import Foundation
+import UIKit
 
 /// Encapsulates UINavigationController Helper Methods.
 ///

WordPress/WordPressShareExtension/WordPressShare-Bridging-Header.h

@@ -2,5 +2,4 @@
 #import "Constants.h"
 
 // Shared Helpers
-#import "SFHFKeychainUtils.h"
 #import "TextBundleWrapper.h"