Fix strict lint voilations

Mintfile

@@ -1,3 +1,3 @@
 nicklockwood/SwiftFormat@0.47.0
 realm/SwiftLint@0.41.0
-peripheryapp/periphery@2.12.3
+peripheryapp/periphery@2.18.0

Sources/SyndiKit/Decoding/DecodingError.swift

@@ -1,15 +1,15 @@
 import Foundation
 
 extension DecodingError {
-  private struct Dictionary: Error {
+  internal struct Dictionary: Error {
     internal init?(errors: [String: DecodingError]) {
       guard errors.count > 1 else {
         return nil
       }
       self.errors = errors
     }
 
-    private let errors: [String: DecodingError]
+    internal let errors: [String: DecodingError]
   }
 
   internal static func failedAttempts(_ errors: [String: DecodingError]) -> Self {

Sources/SyndiKit/Decoding/SynDecoder.swift

@@ -58,7 +58,7 @@ public class SynDecoder {
 
       return (source.source, type.anyDecoding(using: decoder))
     }
-    return Dictionary(grouping: decodings, by: { $0.0 })
+    return Dictionary(grouping: decodings) { $0.0 }
       .mapValues { $0
         .map { $0.1 }
         .map { (type(of: $0).label, $0) }

Sources/SyndiKit/Formats/Blogs/SiteCategory.swift

@@ -7,7 +7,7 @@ public struct SiteCategory {
       return nil
     }
     self.type = type
-    descriptors = Dictionary(grouping: languages, by: { $0.language })
+    descriptors = Dictionary(grouping: languages) { $0.language }
       .compactMapValues { $0.first?.descriptor }
   }
 }

Sources/SyndiKit/Formats/Blogs/SiteDirectory.swift

@@ -9,14 +9,14 @@ public struct SiteCollectionDirectory: SiteDirectory {
   public typealias CategorySequence =
     Dictionary<SiteCategoryType, SiteCategory>.Values
 
-  private struct Instance {
-    fileprivate let allSites: [Site]
-    fileprivate let languageDictionary: [SiteLanguageType: SiteLanguage]
-    fileprivate let categoryDictionary: [SiteCategoryType: SiteCategory]
-    fileprivate let languageIndicies: [SiteLanguageType: Set<Int>]
-    fileprivate let categoryIndicies: [SiteCategoryType: Set<Int>]
-
-    fileprivate func sites(
+  internal struct Instance {
+    internal let allSites: [Site]
+    internal let languageDictionary: [SiteLanguageType: SiteLanguage]
+    internal let categoryDictionary: [SiteCategoryType: SiteCategory]
+    internal let languageIndicies: [SiteLanguageType: Set<Int>]
+    internal let categoryIndicies: [SiteCategoryType: Set<Int>]
+
+    internal func sites(
       withLanguage language: SiteLanguageType?,
       withCategory category: SiteCategoryType?
     ) -> [Site] {
@@ -56,7 +56,7 @@ public struct SiteCollectionDirectory: SiteDirectory {
     }
 
     // swiftlint:disable function_body_length
-    fileprivate init(blogs: SiteCollection) {
+    internal init(blogs: SiteCollection) {
       var categories = [CategoryLanguage]()
       var languages = [SiteLanguage]()
       var sites = [Site]()
@@ -90,10 +90,8 @@ public struct SiteCollectionDirectory: SiteDirectory {
         languages.append(language)
       }
 
-      categoryDictionary = Dictionary(
-        grouping: categories,
-        by: { $0.type }
-      ).compactMapValues(SiteCategory.init)
+      categoryDictionary = Dictionary(grouping: categories) { $0.type }
+        .compactMapValues(SiteCategory.init)
       languageDictionary = Dictionary(
         uniqueKeysWithValues: languages.map { ($0.type, $0) }
       )

Sources/SyndiKit/Formats/Blogs/SiteLanguageCategory+Site.swift

@@ -1,4 +1,6 @@
 import Foundation
+
+// swiftlint:disable nesting
 extension SiteLanguageCategory {
   public struct Site: Codable {
     public let title: String

Sources/SyndiKit/Formats/Media/Podcast/PodcastChapters+MimeType.swift

@@ -10,8 +10,11 @@ extension PodcastChapters {
 
     init?(mimeType: MimeType) {
       switch mimeType {
-      case .json: self = .json
-      case .unknown: return nil
+      case .json:
+        self = .json
+
+      case .unknown:
+        return nil
       }
     }
   }
@@ -47,7 +50,8 @@ extension PodcastChapters {
 
     private init(knownMimeType: KnownMimeType) {
       switch knownMimeType {
-      case .json: self = .json
+      case .json:
+        self = .json
       }
     }
   }

Sources/SyndiKit/Formats/Media/Podcast/PodcastPerson+Role.swift

@@ -16,14 +16,29 @@ extension PodcastPerson {
 
     init?(role: Role) {
       switch role {
-      case .guest: self = .guest
-      case .host: self = .host
-      case .editor: self = .editor
-      case .writer: self = .writer
-      case .designer: self = .designer
-      case .composer: self = .composer
-      case .producer: self = .producer
-      case .unknown: return nil
+      case .guest:
+        self = .guest
+
+      case .host:
+        self = .host
+
+      case .editor:
+        self = .editor
+
+      case .writer:
+        self = .writer
+
+      case .designer:
+        self = .designer
+
+      case .composer:
+        self = .composer
+
+      case .producer:
+        self = .producer
+
+      case .unknown:
+        return nil
       }
     }
   }
@@ -65,13 +80,26 @@ extension PodcastPerson {
 
     private init(knownRole: KnownRole) {
       switch knownRole {
-      case .guest: self = .guest
-      case .host: self = .host
-      case .editor: self = .editor
-      case .writer: self = .writer
-      case .designer: self = .designer
-      case .composer: self = .composer
-      case .producer: self = .producer
+      case .guest:
+        self = .guest
+
+      case .host:
+        self = .host
+
+      case .editor:
+        self = .editor
+
+      case .writer:
+        self = .writer
+
+      case .designer:
+        self = .designer
+
+      case .composer:
+        self = .composer
+
+      case .producer:
+        self = .producer
       }
     }
   }

Sources/SyndiKit/Formats/Media/Podcast/PodcastTranscript+MimeType.swift

@@ -15,13 +15,26 @@ extension PodcastTranscript {
 
     init?(mimeType: MimeType) {
       switch mimeType {
-      case .plain: self = .plain
-      case .html: self = .html
-      case .srt: self = .srt
-      case .vtt: self = .vtt
-      case .json: self = .json
-      case .subrip: self = .subrip
-      case .unknown: return nil
+      case .plain:
+        self = .plain
+
+      case .html:
+        self = .html
+
+      case .srt:
+        self = .srt
+
+      case .vtt:
+        self = .vtt
+
+      case .json:
+        self = .json
+
+      case .subrip:
+        self = .subrip
+
+      case .unknown:
+        return nil
       }
     }
   }
@@ -62,12 +75,23 @@ extension PodcastTranscript {
 
     private init(knownMimeType: KnownMimeType) {
       switch knownMimeType {
-      case .plain: self = .plain
-      case .html: self = .html
-      case .srt: self = .srt
-      case .vtt: self = .vtt
-      case .json: self = .json
-      case .subrip: self = .subrip
+      case .plain:
+        self = .plain
+
+      case .html:
+        self = .html
+
+      case .srt:
+        self = .srt
+
+      case .vtt:
+        self = .vtt
+
+      case .json:
+        self = .json
+
+      case .subrip:
+        self = .subrip
       }
     }
   }

Sources/SyndiKit/Formats/Media/Wordpress/WordPressPost.swift

@@ -6,16 +6,6 @@ public enum WordPressError: Error, Equatable {
   case missingField(WordPressPost.Field)
 }
 
-extension Entryable {
-  public var wpPost: WordPressPost? {
-    guard let rssItem = self as? RSSItem else {
-      return nil
-    }
-
-    return try? WordPressPost(item: rssItem)
-  }
-}
-
 public struct WordPressPost {
   public typealias PostType = String
   public typealias CommentStatus = String
@@ -184,11 +174,8 @@ extension WordPressPost {
     self.body = body.value
     tags = categoryDictionary["post_tag", default: []].map { $0.value }
     categories = categoryDictionary["category", default: []].map { $0.value }
-    self.meta = Dictionary(grouping: meta, by: {
-      $0.key.value
-    }).compactMapValues {
-      $0.last?.value.value
-    }
+    self.meta = Dictionary(grouping: meta) { $0.key.value }
+      .compactMapValues { $0.last?.value.value }
     self.status = status.value
     self.commentStatus = commentStatus.value
     self.pingStatus = pingStatus.value
@@ -212,3 +199,13 @@ extension WordPressPost: Hashable {
     hasher.combine(id)
   }
 }
+
+extension Entryable {
+  public var wpPost: WordPressPost? {
+    guard let rssItem = self as? RSSItem else {
+      return nil
+    }
+
+    return try? WordPressPost(item: rssItem)
+  }
+}

Tests/SyndiKitTests/WordpressTests.swift

@@ -143,7 +143,7 @@ final class WordpressTests: XCTestCase {
     XCTAssertEqual(post.parentID, wpPostParent)
     XCTAssertEqual(post.menuOrder, wpMenuOrder)
     XCTAssertEqual(post.isSticky, wpIsSticky != 0)
-    XCTAssertEqual(post.ID, wpPostID)
+    XCTAssertEqual(post.id, wpPostID)
     XCTAssertEqual(post.postDate, wpPostDate)
     XCTAssertEqual(post.modifiedDate, wpModifiedDate)
     XCTAssertEqual(post.name, wpPostName)
@@ -229,7 +229,7 @@ final class WordpressTests: XCTestCase {
     XCTAssertEqual(post.parentID, wpPostParent)
     XCTAssertEqual(post.menuOrder, wpMenuOrder)
     XCTAssertEqual(post.isSticky, wpIsSticky != 0)
-    XCTAssertEqual(post.ID, wpPostID)
+    XCTAssertEqual(post.id, wpPostID)
     XCTAssertEqual(post.postDate, wpPostDate)
     XCTAssertEqual(post.modifiedDate, wpModifiedDate)
     XCTAssertEqual(post.name, wpPostName)
@@ -312,7 +312,7 @@ final class WordpressTests: XCTestCase {
     XCTAssertEqual(post.parentID, wpPostParent)
     XCTAssertEqual(post.menuOrder, wpMenuOrder)
     XCTAssertEqual(post.isSticky, wpIsSticky != 0)
-    XCTAssertEqual(post.ID, wpPostID)
+    XCTAssertEqual(post.id, wpPostID)
     XCTAssertEqual(post.postDate, wpPostDate)
     XCTAssertEqual(post.modifiedDate, wpModifiedDate)
     XCTAssertEqual(post.name, wpPostName)