Add Periphery config and CI workflow; remove dead code

.github/workflows/periphery.yml

@@ -0,0 +1,14 @@
+name: Periphery
+on:
+  push:
+    branches: [main]
+  pull_request:
+jobs:
+  periphery:
+    runs-on: macos-latest
+    steps:
+      - uses: actions/checkout@v4
+      - name: Install Periphery
+        run: brew install peripheryapp/periphery/periphery
+      - name: Run Periphery
+        run: periphery scan

.periphery.yml

@@ -0,0 +1 @@
+retain_public: true

Sources/SwiftMETAR/Common/Parsers/DateParser.swift

@@ -220,22 +220,6 @@ class HourMinutePeriodParser {
 class HourMinuteParser {
   private let dateRef = Reference<(UInt8?, UInt8)>()
 
-  lazy var hourRequiredRx = Regex {
-    Capture(as: dateRef) {
-      Repeat(.digit, count: 4)
-    } transform: { string in
-      let hourRange = string.startIndex...string.index(string.startIndex, offsetBy: 1)
-      let minuteRange =
-        string.index(string.startIndex, offsetBy: 2)...string.index(string.startIndex, offsetBy: 3)
-
-      guard let hour = UInt8(string[hourRange]),
-        var minute = UInt8(string[minuteRange])
-      else { throw Error.invalidDate(String(string)) }
-
-      return (hour, minute)
-    }
-  }
-
   lazy var hourOptionalRx = Regex {
     Capture(as: dateRef) {
       Optionally { Repeat(.digit, count: 2) }

Sources/SwiftMETAR/Common/Parsers/VisibilityParser.swift

@@ -5,7 +5,6 @@ import NumberKit
 class VisibilityParser {
   private let integerParser = IntegerDistanceParser()
   private let fractionParser = FractionalDistanceParser()
-  private let visibilityRef = Reference<Substring>()
 
   private lazy var integerRx = Regex {
     Anchor.startOfSubject
@@ -29,20 +28,6 @@ class VisibilityParser {
     Anchor.endOfSubject
   }
 
-  lazy var rx = Regex {
-    Capture(as: visibilityRef) {
-      ChoiceOf {
-        ChoiceOf {
-          "10SM"
-          "9999"
-        }
-        notRecordedRx
-        fractionRx
-        integerRx
-      }
-    }
-  }
-
   func parse(_ parts: inout [String.SubSequence]) throws -> Visibility? {
     guard !parts.isEmpty else { return nil }
 
@@ -61,14 +46,6 @@ class VisibilityParser {
     return nil
   }
 
-  func parse<T>(_ match: Regex<T>.Match) throws -> Visibility? {
-    let visStr = match[visibilityRef]
-    guard let visibility = try parse(String(visStr)) else {
-      preconditionFailure("Visibility rx should have captured parseable substring")
-    }
-    return visibility
-  }
-
   private func parse(_ vizStr: String) throws -> Visibility? {
     if vizStr == "CAVOK" {
       return .greaterThan(.meters(9999))

Sources/SwiftMETAR/METAR/Parsers/METARXMLParser.swift

@@ -120,7 +120,6 @@ actor METARXMLParser {
     var entries = [Entry]()
 
     private var currentEntry: Entry?
-    private var currentElement: String?
     private var characterBuffer = ""
     private var inQualityControlFlags = false
 
@@ -131,8 +130,6 @@ actor METARXMLParser {
       qualifiedName _: String?,
       attributes attributeDict: [String: String] = [:]
     ) {
-      currentElement = elementName
-
       if elementName == "METAR" {
         currentEntry = Entry()
       } else if elementName == "quality_control_flags" {
@@ -205,8 +202,6 @@ actor METARXMLParser {
           }
         }
       }
-
-      currentElement = nil
     }
 
     func parser(_: XMLParser, foundCharacters string: String) {

Sources/SwiftMETAR/Remarks/Parsers/Types/NavalForecasterParser.swift

@@ -1,10 +1,6 @@
 import Foundation
 @preconcurrency import RegexBuilder
 
-private let nocapWeatherCenterRx = Remark.NavalWeatherCenter.allCases.map(\.rawValue).joined(
-  separator: "|"
-)
-
 final class NavalForecasterParser: RemarkParser {
   var urgency = Remark.Urgency.routine
 

Sources/SwiftMETAR/Remarks/Parsers/Types/WindChangeParser.swift

@@ -1,8 +1,6 @@
 import Foundation
 @preconcurrency import RegexBuilder
 
-private let nocapExtremesRegex = Remark.Extreme.allCases.map(\.rawValue).joined(separator: "|")
-
 final class WindChangeParser: RemarkParser {
   var urgency = Remark.Urgency.routine
 

Sources/SwiftMETAR/Support/Date.swift

@@ -36,19 +36,6 @@ extension Date {
   func next(_ components: DateComponents) -> Date? {
     return zuluCal.nextDate(after: self, matching: components, matchingPolicy: .nextTime)
   }
-
-  func next(
-    year: Int? = nil,
-    month: Int? = nil,
-    day: Int? = nil,
-    hour: Int? = nil,
-    minute: Int? = nil,
-    second: Int? = nil
-  ) -> Date? {
-    return next(
-      .init(year: year, month: month, day: day, hour: hour, minute: minute, second: second)
-    )
-  }
 }
 
 func applyComponents(

Sources/SwiftMETAR/Support/Ratio.swift

@@ -4,12 +4,6 @@ import NumberKit
 /// A ratio of two integers.
 public typealias Ratio = Rational<Int>
 
-extension Float {
-  init(_ ratio: Ratio) {
-    self = ratio.floatValue
-  }
-}
-
 extension Ratio {
   init(_ whole: Int, numerator: Int, denominator: Int) {
     self.init(denominator * whole + numerator, denominator)

Sources/SwiftMETAR/Support/String+SwiftMETAR.swift

@@ -1,8 +1,6 @@
 import Foundation
 
 extension String {
-  var nsRange: NSRange { return NSRange(location: 0, length: count) }
-
   func partitionSlices(by length: Int) -> [Substring] {
     var startIndex = self.startIndex
     var results = [Substring]()
@@ -46,13 +44,4 @@ extension String {
     substrings.append(self[rangeStart...])
     return substrings
   }
-
-  func substring(with nsRange: NSRange) -> SubSequence {
-    let range =
-      index(
-        startIndex,
-        offsetBy: nsRange.location
-      )..<index(startIndex, offsetBy: nsRange.location + nsRange.length)
-    return self[range]
-  }
 }

Sources/SwiftMETAR/Support/XMLParsing.swift

@@ -19,37 +19,6 @@ enum XMLParsing {
     return zuluCal.dateComponents(in: zulu, from: date)
   }
 
-  static func parseRatio(_ string: String) -> Ratio? {
-    if let intVal = Int(string) {
-      return Ratio(intVal)
-    }
-
-    // Handle decimal like "1.75"
-    if string.contains(".") {
-      let parts = string.split(separator: ".")
-      guard parts.count == 2,
-        let whole = Int(parts[0]),
-        let fracStr = parts.last
-      else { return nil }
-
-      let denominator = Int(pow(10.0, Double(fracStr.count)))
-      guard let numerator = Int(fracStr) else { return nil }
-      return Ratio(whole * denominator + numerator, denominator)
-    }
-
-    // Handle fraction like "3/4"
-    if string.contains("/") {
-      let parts = string.split(separator: "/")
-      guard parts.count == 2,
-        let num = Int(parts[0]),
-        let den = Int(parts[1])
-      else { return nil }
-      return Ratio(num, den)
-    }
-
-    return nil
-  }
-
   static func buildWind(dirDegrees: String?, speedKt: String?, gustKt: String?) throws -> Wind? {
     guard let speedStr = speedKt else { return nil }
     guard let speed = UInt16(speedStr) else {

Sources/SwiftMETAR/TAF/Parsers/TAFXMLParser.swift

@@ -281,7 +281,6 @@ actor TAFXMLParser {
 
     private var currentEntry: Entry?
     private var currentForecast: ForecastEntry?
-    private var currentElement: String?
     private var characterBuffer = ""
     private var inForecast = false
 
@@ -292,8 +291,6 @@ actor TAFXMLParser {
       qualifiedName _: String?,
       attributes attributeDict: [String: String] = [:]
     ) {
-      currentElement = elementName
-
       if elementName == "TAF" {
         currentEntry = Entry()
       } else if elementName == "forecast" {
@@ -393,8 +390,6 @@ actor TAFXMLParser {
           }
         }
       }
-
-      currentElement = nil
     }
 
     func parser(_: XMLParser, foundCharacters string: String) {