Fix SwiftLint violations

.swiftlint.yml

@@ -0,0 +1,6 @@
+disabled_rules:
+  - multiple_closures_with_trailing_closure
+  - non_optional_string_data_conversion
+  - todo
+line_length:
+  ignores_comments: true

Maccy/FloatingPanel.swift

@@ -83,7 +83,7 @@ class FloatingPanel<Content: View>: NSPanel, NSWindowDelegate {
     newSize.height = min(newHeight, newSize.height)
 
     var newOrigin = frame.origin
-    newOrigin.y = newOrigin.y + (frame.height - newSize.height)
+    newOrigin.y += (frame.height - newSize.height)
 
     NSAnimationContext.runAnimationGroup { (context) in
       context.duration = 0.2

Maccy/Intents/AppIntentError.swift

@@ -5,7 +5,7 @@ enum AppIntentError: Swift.Error, CustomLocalizedStringResourceConvertible {
 
   var localizedStringResource: LocalizedStringResource {
     switch self {
-      case .notFound: return "Clipboard item not found"
+    case .notFound: return "Clipboard item not found"
     }
   }
 }

Maccy/Intents/Get.swift

@@ -5,7 +5,10 @@ struct Get: AppIntent, CustomIntentMigratedAppIntent {
   static let intentClassName = "GetIntent"
 
   static var title: LocalizedStringResource = "Get Item from Clipboard History"
-  static var description = IntentDescription("Gets an item from Maccy clipboard history. The returned item can be used to access its plain/rich/HTML text, image contents or file location.")
+  static var description = IntentDescription("""
+  Gets an item from Maccy clipboard history.
+  The returned item can be used to access its plain/rich/HTML text, image contents or file location.
+  """)
 
   @Parameter(title: "Selected", default: true)
   var selected: Bool

Maccy/Intents/Select.swift

@@ -4,7 +4,10 @@ struct Select: AppIntent, CustomIntentMigratedAppIntent {
   static let intentClassName = "SelectIntent"
 
   static var title: LocalizedStringResource = "Select Item in Clipboard History"
-  static var description = IntentDescription("Selects an item in Maccy clipboard history. Depending on Maccy settings, it might trigger pasting of the selected item.")
+  static var description = IntentDescription("""
+  Selects an item in Maccy clipboard history.
+  Depending on Maccy settings, it might trigger pasting of the selected item.
+  """)
 
   static var parameterSummary: some ParameterSummary {
     Summary("Select \(\.$number) Item in Clipboard History")

Maccy/KeyChord.swift

@@ -49,7 +49,7 @@ enum KeyChord: CaseIterable {
     )
   }
 
-  init(_ key: Key, _ modifierFlags: NSEvent.ModifierFlags) {
+  init(_ key: Key, _ modifierFlags: NSEvent.ModifierFlags) { // swiftlint:disable:this cyclomatic_complexity
     switch (key, modifierFlags) {
     case (.delete, [.command, .option]):
       self = .clearHistory
@@ -91,5 +91,4 @@ enum KeyChord: CaseIterable {
       self = .unknown
     }
   }
-  // swiftlint:enable cyclomatic_complexity
 }

Maccy/KeyShortcut.swift

@@ -4,10 +4,11 @@ import Sauce
 
 struct KeyShortcut: Identifiable {
   static func create(character: String) -> [KeyShortcut] {
+    let key = Key(character: character, virtualKeyCode: nil)
     return [
-      KeyShortcut(key: Key(character: character, virtualKeyCode: nil)),
-      KeyShortcut(key: Key(character: character, virtualKeyCode: nil), modifierFlags: [.option]),
-      KeyShortcut(key: Key(character: character, virtualKeyCode: nil), modifierFlags: [Defaults[.pasteByDefault] ? .command : .option, .shift])
+      KeyShortcut(key: key),
+      KeyShortcut(key: key, modifierFlags: [.option]),
+      KeyShortcut(key: key, modifierFlags: [Defaults[.pasteByDefault] ? .command : .option, .shift])
     ]
   }
 

Maccy/Models/HistoryItem.swift

@@ -31,8 +31,8 @@ class HistoryItem {
     let descriptor = FetchDescriptor<HistoryItem>(
       predicate: #Predicate { $0.pin != nil }
     )
-    let pins = try! Storage.shared.context.fetch(descriptor).compactMap({ $0.pin })
-    let assignedPins = Set(pins)
+    let pins = try? Storage.shared.context.fetch(descriptor).compactMap({ $0.pin })
+    let assignedPins = Set(pins ?? [])
     return Array(supportedPins.subtracting(assignedPins))
   }
 

Maccy/Observables/AppState.swift

@@ -37,7 +37,8 @@ class AppState: Sendable {
   }
 
   var menuIconText: String {
-    var title = history.unpinnedItems.first?.text.shortened(to: 100).trimmingCharacters(in: .whitespacesAndNewlines) ?? ""
+    var title = history.unpinnedItems.first?.text.shortened(to: 100)
+      .trimmingCharacters(in: .whitespacesAndNewlines) ?? ""
     title.unicodeScalars.removeAll(where: CharacterSet.newlines.contains)
     return title.shortened(to: 20)
   }
@@ -131,7 +132,7 @@ class AppState: Sendable {
   }
 
   @MainActor
-  func openPreferences() {
+  func openPreferences() { // swiftlint:disable:this function_body_length
     if settingsWindowController == nil {
       settingsWindowController = SettingsWindowController(
         panes: [

Maccy/Observables/Footer.swift

@@ -39,7 +39,7 @@ class Footer {
         confirmation: .init(
           message: "clear_alert_message",
           comment: "clear_alert_comment",
-          ok: "clear_alert_confirm",
+          confirm: "clear_alert_confirm",
           cancel: "clear_alert_cancel"
         ),
         suppressConfirmation: suppressClearAlert
@@ -53,7 +53,7 @@ class Footer {
         confirmation: .init(
           message: "clear_alert_message",
           comment: "clear_alert_comment",
-          ok: "clear_alert_confirm",
+          confirm: "clear_alert_confirm",
           cancel: "clear_alert_cancel"
         ),
         suppressConfirmation: suppressClearAlert

Maccy/Observables/FooterItem.swift

@@ -5,7 +5,7 @@ class FooterItem: Equatable, Identifiable {
   struct Confirmation {
     var message: LocalizedStringKey
     var comment: LocalizedStringKey
-    var ok: LocalizedStringKey
+    var confirm: LocalizedStringKey
     var cancel: LocalizedStringKey
   }
 

Maccy/Observables/History.swift

@@ -7,7 +7,7 @@ import Settings
 import SwiftData
 
 @Observable
-class History {
+class History { // swiftlint:disable:this type_body_length
   static let shared = History()
 
   var items: [HistoryItemDecorator] = []
@@ -132,7 +132,7 @@ class History {
         item.contents = existingHistoryItem.contents
       }
       item.firstCopiedAt = existingHistoryItem.firstCopiedAt
-      item.numberOfCopies = item.numberOfCopies + existingHistoryItem.numberOfCopies
+      item.numberOfCopies += existingHistoryItem.numberOfCopies
       item.pin = existingHistoryItem.pin
       item.title = existingHistoryItem.title
       if !item.fromMaccy {

Maccy/PopupPosition.swift

@@ -56,8 +56,7 @@ enum PopupPosition: String, CaseIterable, Identifiable, CustomStringConvertible,
     }
 
     var point = NSEvent.mouseLocation
-    point.y = point.y - size.height
+    point.y -= size.height
     return point
   }
-
 }

Maccy/Sorter.swift

@@ -2,6 +2,7 @@ import AppKit
 import Defaults
 
 // swiftlint:disable identifier_name
+// swiftlint:disable type_name
 class Sorter {
   enum By: String, CaseIterable, Identifiable, CustomStringConvertible, Defaults.Serializable {
     case lastCopiedAt
@@ -48,3 +49,4 @@ class Sorter {
   }
 }
 // swiftlint:enable identifier_name
+// swiftlint:enable type_name

Maccy/Storage.swift

@@ -10,6 +10,10 @@ class Storage {
 
   init() {
     let config = ModelConfiguration(url: URL.applicationSupportDirectory.appending(path: "Maccy/Storage.sqlite"))
-    container = try! ModelContainer(for: HistoryItem.self, configurations: config)
+    do {
+      container = try ModelContainer(for: HistoryItem.self, configurations: config)
+    } catch let error {
+      fatalError("Cannot load database: \(error.localizedDescription).")
+    }
   }
 }

Maccy/Views/ConfirmationView.swift

@@ -16,7 +16,7 @@ struct ConfirmationView<Content: View>: View {
         }
         .confirmationDialog(confirmation.message, isPresented: $item.showConfirmation) {
           Text(confirmation.comment)
-          Button(confirmation.ok, role: .destructive) {
+          Button(confirmation.confirm, role: .destructive) {
             item.action()
           }
           Button(confirmation.cancel, role: .cancel) {}

MaccyTests/HistoryDecoratorTests.swift

@@ -122,7 +122,10 @@ class HistoryItemDecoratorTests: XCTestCase {
 
   func testHighlight() {
     let itemDecorator = historyItemDecorator("foo bar baz")
-    itemDecorator.highlight("random", [range(from: 1, to: 2, in: itemDecorator), range(from: 8, to: 10, in: itemDecorator)])
+    itemDecorator.highlight("random", [
+      range(from: 1, to: 2, in: itemDecorator),
+      range(from: 8, to: 10, in: itemDecorator)
+    ])
     var expectedTitle = AttributedString("foo bar baz")
     expectedTitle[expectedTitle.range(of: "oo")!].font = .bold(.body)()
     expectedTitle[expectedTitle.range(of: "baz")!].font = .bold(.body)()
@@ -131,7 +134,10 @@ class HistoryItemDecoratorTests: XCTestCase {
     XCTAssertEqual(itemDecorator.attributedTitle, nil)
   }
 
-  private func historyItemDecorator(_ value: String?, application: String? = "com.apple.finder") -> HistoryItemDecorator {
+  private func historyItemDecorator(
+    _ value: String?,
+    application: String? = "com.apple.finder"
+  ) -> HistoryItemDecorator {
     let contents = [
       HistoryItemContent(
         type: NSPasteboard.PasteboardType.string.rawValue,
@@ -149,7 +155,10 @@ class HistoryItemDecoratorTests: XCTestCase {
     return HistoryItemDecorator(item)
   }
 
-  private func historyItemDecorator(_ value: Data?, _ type: NSPasteboard.PasteboardType) -> HistoryItemDecorator {
+  private func historyItemDecorator(
+    _ value: Data?,
+    _ type: NSPasteboard.PasteboardType
+  ) -> HistoryItemDecorator {
     let contents = [
       HistoryItemContent(
         type: type.rawValue,
@@ -210,6 +219,7 @@ class HistoryItemDecoratorTests: XCTestCase {
     return HistoryItemDecorator(item)
   }
 
+  // swiftlint:disable:next identifier_name
   private func range(from: Int, to: Int, in item: HistoryItemDecorator) -> Range<String.Index> {
     let startIndex = item.title.startIndex
     let lowerBound = item.title.index(startIndex, offsetBy: from)