linting

Sources/UserScript/UserScriptMessageEncryption.swift

@@ -72,7 +72,7 @@ extension UserScriptMessageEncryption {
         return params
     }
 
-    public func encrypt(reply: String, with params: SecureMessagingParams, into webView: WKWebView?)  {
+    public func encrypt(reply: String, with params: SecureMessagingParams, into webView: WKWebView?) {
 
         guard let encryption = try? encrypter.encryptReply(reply, key: params.key, iv: params.iv) else { return }
 

Sources/UserScript/UserScriptMessaging.swift

@@ -60,12 +60,12 @@ public protocol Subfeature {
 
     /// Subfeatures may sometime need access to the message broker - for example to push messages into the page
     var broker: UserScriptMessageBroker? { get set }
-    func with(broker: UserScriptMessageBroker) -> Void;
+    func with(broker: UserScriptMessageBroker)
 }
 
 extension Subfeature {
     /// providing a blank implementation since not all features will need this
-    public func with(broker: UserScriptMessageBroker) -> Void {
+    public func with(broker: UserScriptMessageBroker) {
         // nothing
     }
 }
@@ -76,7 +76,7 @@ public protocol UserScriptMessaging: UserScript {
 
 extension UserScriptMessaging {
     public func registerSubFeature(delegate: Subfeature) {
-        delegate.with(broker: broker);
+        delegate.with(broker: broker)
         broker.registerSubFeature(delegate: delegate)
     }
 }
@@ -123,7 +123,7 @@ public final class UserScriptMessageBroker: NSObject, UserScriptMessageEncryptio
         if #available(macOS 11.0, iOS 14.0, *) {
             config.hasModernWebkitAPI = true
         }
-        return config;
+        return config
     }
 
     public func push(method: String, params: Encodable?, for delegate: Subfeature, into webView: WKWebView) {
@@ -134,7 +134,7 @@ public final class UserScriptMessageBroker: NSObject, UserScriptMessageEncryptio
                 params: params ?? [:] as [String: String]
         )
         else {
-            return;
+            return
         }
         if #available(macOS 11.0, iOS 14.0, *) {
             if !self.requiresRunInPageContentWorld {
@@ -180,7 +180,6 @@ public final class UserScriptMessageBroker: NSObject, UserScriptMessageEncryptio
             return .error(.invalidParams)
         }
 
-
         /// Now try to match the message to a registered delegate
         guard let delegate = callbacks[featureName] else {
             return .error(.notFoundFeature(featureName))
@@ -235,7 +234,7 @@ public final class UserScriptMessageBroker: NSObject, UserScriptMessageEncryptio
             /// As far as the client is concerned, a `notification` is fire-and-forget
         case .notify(let handler, let notification):
             do {
-                try await handler(notification.params, original);
+                try await handler(notification.params, original)
             } catch {
                 os_log("UserScriptMessaging: unhandled exception %s", type: .error, String(describing: error.localizedDescription))
             }

Sources/UserScript/UserScriptMessagingSchema.swift

@@ -157,8 +157,8 @@ public struct SubscriptionEvent {
 }
 
 public struct WebkitMessagingConfig: Encodable {
-    let webkitMessageHandlerNames: [String];
-    let secret: String;
+    let webkitMessageHandlerNames: [String]
+    let secret: String
     var hasModernWebkitAPI: Bool
 }
 

Tests/UserScriptTests/UserScriptMessagingTests.swift

@@ -143,7 +143,7 @@ class UserScriptMessagingTests: XCTestCase {
         ])
 
         do {
-            let _ = try await testee.execute(action: action, original: original);
+            _ = try await testee.execute(action: action, original: original)
         } catch let error {
             XCTAssertEqual(error.localizedDescription, "feature named `this_feature_doesnt_exist` was not found")
         }
@@ -162,7 +162,7 @@ class UserScriptMessagingTests: XCTestCase {
         ])
 
         do {
-            let _ = try await testee.execute(action: action, original: original)
+            _ = try await testee.execute(action: action, original: original)
         } catch let error {
             XCTAssertEqual(error.localizedDescription, "the incoming message is ignored because the feature `fooBarFeature` couldn't provide a handler for method `an_unknown_method_name_but_no_id`")
         }
@@ -181,7 +181,7 @@ class UserScriptMessagingTests: XCTestCase {
         ])
 
         do {
-            let _ = try await testee.execute(action: action, original: original)
+            _ = try await testee.execute(action: action, original: original)
         } catch {
             XCTAssertEqual(error.localizedDescription, "The incoming message was not valid - one or more of 'featureName', 'method'  or 'context' was missing")
         }
@@ -240,7 +240,7 @@ struct TestDelegate: Subfeature {
 
     /// An example that represents handling a [NotificationMessage](https://duckduckgo.github.io/content-scope-scripts/classes/Messaging_Schema.NotificationMessage.html)
     func notifyExample(params: Any, original: WKScriptMessage) async throws -> Encodable? {
-        print("not replying...");
+        print("not replying...")
         return nil
     }