refactor: migrate from swift-log to BushelKit FelinePine logging

Replace custom BushelLogger (swift-log wrapper) with BushelKit's FelinePine
logging system. This consolidates logging infrastructure across the project.

Changes:
- Remove swift-log dependency, add BushelLogging from BushelKit
- Create ConsoleOutput utility for user-facing CLI messages
- Migrate all types to use Loggable protocol with Self.logger
- Map subsystems to categories: cloudKit→.data, dataSource→.hub, sync→.application
- Remove custom logging features (emoji, explain() method)
- Delete Logger.swift

All services, fetchers, and CLI commands now use standard FelinePine logging.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: leogdion

Package.swift

@@ -95,8 +95,7 @@ let package = Package(
         .package(url: "https://github.com/brightdigit/IPSWDownloads.git", from: "1.0.0"),
         .package(url: "https://github.com/brightdigit/BushelKit.git", from: "3.0.0-alpha.1"),
         .package(url: "https://github.com/scinfu/SwiftSoup.git", from: "2.6.0"),
-        .package(url: "https://github.com/apple/swift-argument-parser.git", from: "1.5.0"),
-        .package(url: "https://github.com/apple/swift-log.git", from: "1.0.0")
+        .package(url: "https://github.com/apple/swift-argument-parser.git", from: "1.5.0")
     ],
     targets: [
         .target(
@@ -105,7 +104,7 @@ let package = Package(
                 .product(name: "MistKit", package: "MistKit"),
                 .product(name: "IPSWDownloads", package: "IPSWDownloads"),
                 .product(name: "SwiftSoup", package: "SwiftSoup"),
-                .product(name: "Logging", package: "swift-log")
+                .product(name: "BushelLogging", package: "BushelKit")
             ],
             swiftSettings: swiftSettings
         ),

Sources/BushelCloudCLI/Commands/ClearCommand.swift

@@ -68,8 +68,8 @@ struct ClearCommand: AsyncParsableCommand {
   // MARK: - Execution
 
   mutating func run() async throws {
-    // Enable verbose logging if requested
-    BushelLogger.isVerbose = verbose
+    // Enable verbose console output if requested
+    ConsoleOutput.isVerbose = verbose
 
     // Get Server-to-Server credentials from environment if not provided
     let resolvedKeyID =

Sources/BushelCloudCLI/Commands/ExportCommand.swift

@@ -76,8 +76,8 @@ struct ExportCommand: AsyncParsableCommand {
   // MARK: - Execution
 
   mutating func run() async throws {
-    // Enable verbose logging if requested
-    BushelLogger.isVerbose = verbose
+    // Enable verbose console output if requested
+    ConsoleOutput.isVerbose = verbose
 
     // Get Server-to-Server credentials from environment if not provided
     let resolvedKeyID =

Sources/BushelCloudCLI/Commands/SyncCommand.swift

@@ -99,8 +99,8 @@ struct SyncCommand: AsyncParsableCommand {
   // MARK: - Execution
 
   mutating func run() async throws {
-    // Enable verbose logging if requested
-    BushelLogger.isVerbose = verbose
+    // Enable verbose console output if requested
+    ConsoleOutput.isVerbose = verbose
 
     // Get Server-to-Server credentials from environment if not provided
     let resolvedKeyID =

Sources/BushelCloudKit/CloudKit/BushelCloudKitService.swift

@@ -29,6 +29,7 @@
 
 public import Foundation
 public import MistKit
+public import BushelLogging
 
 /// CloudKit service wrapper for Bushel demo operations
 ///
@@ -112,26 +113,24 @@ public struct BushelCloudKitService: Sendable, RecordManaging, CloudKitRecordCol
     let batches = operations.chunked(into: batchSize)
 
     print("Syncing \(operations.count) \(recordType) record(s) in \(batches.count) batch(es)...")
-    BushelLogger.verbose(
-      "CloudKit batch limit: 200 operations/request. Using \(batches.count) batch(es) for \(operations.count) records.",
-      subsystem: BushelLogger.cloudKit
+    Self.logger.debug(
+      "CloudKit batch limit: 200 operations/request. Using \(batches.count) batch(es) for \(operations.count) records."
     )
 
     var totalSucceeded = 0
     var totalFailed = 0
 
     for (index, batch) in batches.enumerated() {
       print("  Batch \(index + 1)/\(batches.count): \(batch.count) records...")
-      BushelLogger.verbose(
-        "Calling MistKit service.modifyRecords() with \(batch.count) RecordOperation objects",
-        subsystem: BushelLogger.cloudKit
+      Self.logger.debug(
+        "Calling MistKit service.modifyRecords() with \(batch.count) RecordOperation objects"
       )
 
       let results = try await service.modifyRecords(batch)
 
-      BushelLogger.verbose(
-        "Received \(results.count) RecordInfo responses from CloudKit",
-        subsystem: BushelLogger.cloudKit)
+      Self.logger.debug(
+        "Received \(results.count) RecordInfo responses from CloudKit"
+      )
 
       // Filter out error responses using isError property
       let successfulRecords = results.filter { !$0.isError }
@@ -147,14 +146,14 @@ public struct BushelCloudKitService: Sendable, RecordManaging, CloudKitRecordCol
         // Log error details in verbose mode
         let errorRecords = results.filter { $0.isError }
         for errorRecord in errorRecords {
-          BushelLogger.verbose(
-            "Error: recordName=\(errorRecord.recordName), reason=\(errorRecord.recordType)",
-            subsystem: BushelLogger.cloudKit
+          Self.logger.debug(
+            "Error: recordName=\(errorRecord.recordName), reason=\(errorRecord.recordType)"
           )
         }
       } else {
-        BushelLogger.success(
-          "CloudKit confirmed \(successfulRecords.count) records", subsystem: BushelLogger.cloudKit)
+        Self.logger.info(
+          "CloudKit confirmed \(successfulRecords.count) records"
+        )
       }
     }
 
@@ -164,10 +163,14 @@ public struct BushelCloudKitService: Sendable, RecordManaging, CloudKitRecordCol
 
     if totalFailed > 0 {
       print("   ❌ Failed: \(totalFailed) operations")
-      BushelLogger.explain(
-        "Use --verbose flag to see CloudKit error details (serverErrorCode, reason, etc.)",
-        subsystem: BushelLogger.cloudKit
+      Self.logger.debug(
+        "Use --verbose flag to see CloudKit error details (serverErrorCode, reason, etc.)"
       )
     }
   }
 }
+
+// MARK: - Loggable Conformance
+extension BushelCloudKitService: Loggable {
+  public static let loggingCategory: BushelLogging.Category = .data
+}

Sources/BushelCloudKit/CloudKit/SyncEngine.swift

@@ -29,6 +29,7 @@
 
 public import Foundation
 public import MistKit
+public import BushelLogging
 
 /// Orchestrates the complete sync process from data sources to CloudKit
 ///
@@ -79,33 +80,32 @@ public struct SyncEngine: Sendable {
   /// Execute full sync from all data sources to CloudKit
   public func sync(options: SyncOptions = SyncOptions()) async throws -> SyncResult {
     print("\n" + String(repeating: "=", count: 60))
-    BushelLogger.info("🔄 Starting Bushel CloudKit Sync", subsystem: BushelLogger.sync)
+    ConsoleOutput.info("Starting Bushel CloudKit Sync")
     print(String(repeating: "=", count: 60))
+    Self.logger.info("Sync started")
 
     if options.dryRun {
-      BushelLogger.info(
-        "🧪 DRY RUN MODE - No changes will be made to CloudKit", subsystem: BushelLogger.sync)
+      ConsoleOutput.info("DRY RUN MODE - No changes will be made to CloudKit")
+      Self.logger.info("Sync running in dry-run mode")
     }
 
-    BushelLogger.explain(
-      "This sync demonstrates MistKit's Server-to-Server authentication and bulk record operations",
-      subsystem: BushelLogger.sync
+    Self.logger.debug(
+      "Using MistKit Server-to-Server authentication for bulk record operations"
     )
 
     // Step 1: Fetch from all data sources
     print("\n📥 Step 1: Fetching data from external sources...")
-    BushelLogger.verbose(
-      "Initializing data source pipeline to fetch from ipsw.me, TheAppleWiki, MESU, and other sources",
-      subsystem: BushelLogger.dataSource)
+    Self.logger.debug(
+      "Initializing data source pipeline to fetch from ipsw.me, TheAppleWiki, MESU, and other sources"
+    )
 
     let fetchResult = try await pipeline.fetch(options: options.pipelineOptions)
 
-    BushelLogger.verbose(
-      "Data fetch complete. Beginning deduplication and merge phase.",
-      subsystem: BushelLogger.dataSource)
-    BushelLogger.explain(
-      "Multiple data sources may have overlapping data. The pipeline deduplicates by version+build number.",
-      subsystem: BushelLogger.dataSource
+    Self.logger.debug(
+      "Data fetch complete. Beginning deduplication and merge phase."
+    )
+    Self.logger.debug(
+      "Multiple data sources may have overlapping data. The pipeline deduplicates by version+build number."
     )
 
     let stats = SyncResult(
@@ -124,18 +124,18 @@ public struct SyncEngine: Sendable {
     print("   ─────────────────────")
     print("   Total: \(totalRecords) records")
 
-    BushelLogger.verbose(
-      "Records ready for CloudKit upload: \(totalRecords) total", subsystem: BushelLogger.sync)
+    Self.logger.debug(
+      "Records ready for CloudKit upload: \(totalRecords) total"
+    )
 
     // Step 2: Sync to CloudKit (unless dry run)
     if !options.dryRun {
       print("\n☁️  Step 2: Syncing to CloudKit...")
-      BushelLogger.verbose(
-        "Using MistKit to batch upload records to CloudKit public database",
-        subsystem: BushelLogger.cloudKit)
-      BushelLogger.explain(
-        "MistKit handles authentication, batching (200 records/request), and error handling automatically",
-        subsystem: BushelLogger.cloudKit
+      Self.logger.debug(
+        "Using MistKit to batch upload records to CloudKit public database"
+      )
+      Self.logger.debug(
+        "MistKit handles authentication, batching (200 records/request), and error handling automatically"
       )
 
       // Sync in dependency order: SwiftVersion → RestoreImage → XcodeVersion
@@ -151,73 +151,79 @@ public struct SyncEngine: Sendable {
       print("   • \(stats.restoreImagesCount) restore images")
       print("   • \(stats.xcodeVersionsCount) Xcode versions")
       print("   • \(stats.swiftVersionsCount) Swift versions")
-      BushelLogger.verbose(
-        "Dry run mode: No CloudKit operations performed", subsystem: BushelLogger.sync)
+      Self.logger.debug(
+        "Dry run mode: No CloudKit operations performed"
+      )
     }
 
     print("\n" + String(repeating: "=", count: 60))
-    BushelLogger.success("Sync completed successfully!", subsystem: BushelLogger.sync)
+    ConsoleOutput.success("Sync completed successfully!")
     print(String(repeating: "=", count: 60))
+    Self.logger.info("Sync completed successfully")
 
     return stats
   }
 
   /// Delete all records from CloudKit
   public func clear() async throws {
     print("\n" + String(repeating: "=", count: 60))
-    BushelLogger.info("🗑️  Clearing all CloudKit data", subsystem: BushelLogger.cloudKit)
+    ConsoleOutput.info("Clearing all CloudKit data")
     print(String(repeating: "=", count: 60))
+    Self.logger.info("Clearing all CloudKit records")
 
     try await cloudKitService.deleteAllRecords()
 
     print("\n" + String(repeating: "=", count: 60))
-    BushelLogger.success("Clear completed successfully!", subsystem: BushelLogger.sync)
+    ConsoleOutput.success("Clear completed successfully!")
     print(String(repeating: "=", count: 60))
+    Self.logger.info("Clear completed successfully")
   }
 
   /// Export all records from CloudKit to a structured format
   public func export() async throws -> ExportResult {
     print("\n" + String(repeating: "=", count: 60))
-    BushelLogger.info("📤 Exporting data from CloudKit", subsystem: BushelLogger.cloudKit)
+    ConsoleOutput.info("Exporting data from CloudKit")
     print(String(repeating: "=", count: 60))
+    Self.logger.info("Exporting CloudKit data")
 
-    BushelLogger.explain(
-      "Using MistKit's queryRecords() to fetch all records of each type from the public database",
-      subsystem: BushelLogger.cloudKit
+    Self.logger.debug(
+      "Using MistKit queryRecords() to fetch all records of each type from the public database"
     )
 
     print("\n📥 Fetching RestoreImage records...")
-    BushelLogger.verbose(
-      "Querying CloudKit for recordType: 'RestoreImage' with limit: 1000",
-      subsystem: BushelLogger.cloudKit)
+    Self.logger.debug(
+      "Querying CloudKit for recordType: 'RestoreImage' with limit: 1000"
+    )
     let restoreImages = try await cloudKitService.queryRecords(recordType: "RestoreImage")
-    BushelLogger.verbose(
-      "Retrieved \(restoreImages.count) RestoreImage records", subsystem: BushelLogger.cloudKit)
+    Self.logger.debug(
+      "Retrieved \(restoreImages.count) RestoreImage records"
+    )
 
     print("📥 Fetching XcodeVersion records...")
-    BushelLogger.verbose(
-      "Querying CloudKit for recordType: 'XcodeVersion' with limit: 1000",
-      subsystem: BushelLogger.cloudKit)
+    Self.logger.debug(
+      "Querying CloudKit for recordType: 'XcodeVersion' with limit: 1000"
+    )
     let xcodeVersions = try await cloudKitService.queryRecords(recordType: "XcodeVersion")
-    BushelLogger.verbose(
-      "Retrieved \(xcodeVersions.count) XcodeVersion records", subsystem: BushelLogger.cloudKit)
+    Self.logger.debug(
+      "Retrieved \(xcodeVersions.count) XcodeVersion records"
+    )
 
     print("📥 Fetching SwiftVersion records...")
-    BushelLogger.verbose(
-      "Querying CloudKit for recordType: 'SwiftVersion' with limit: 1000",
-      subsystem: BushelLogger.cloudKit)
+    Self.logger.debug(
+      "Querying CloudKit for recordType: 'SwiftVersion' with limit: 1000"
+    )
     let swiftVersions = try await cloudKitService.queryRecords(recordType: "SwiftVersion")
-    BushelLogger.verbose(
-      "Retrieved \(swiftVersions.count) SwiftVersion records", subsystem: BushelLogger.cloudKit)
+    Self.logger.debug(
+      "Retrieved \(swiftVersions.count) SwiftVersion records"
+    )
 
     print("\n✅ Exported:")
     print("   • \(restoreImages.count) restore images")
     print("   • \(xcodeVersions.count) Xcode versions")
     print("   • \(swiftVersions.count) Swift versions")
 
-    BushelLogger.explain(
-      "MistKit returns RecordInfo structs with record metadata. Use .fields to access CloudKit field values.",
-      subsystem: BushelLogger.cloudKit
+    Self.logger.debug(
+      "MistKit returns RecordInfo structs with record metadata. Use .fields to access CloudKit field values."
     )
 
     return ExportResult(
@@ -255,3 +261,8 @@ public struct SyncEngine: Sendable {
     }
   }
 }
+
+// MARK: - Loggable Conformance
+extension SyncEngine: Loggable {
+  public static let loggingCategory: BushelLogging.Category = .application
+}