Add support for systemLibrary, binaryTarget, and plugin target types

Author: tonyarnold

Sources/SwiftDependencyAuditLib/DependencyAnalyzer.swift

@@ -6,6 +6,17 @@ public actor DependencyAnalyzer {
     public init() {}
 
     public func analyzeTarget(_ target: Target, in packageInfo: PackageInfo, customWhitelist: Set<String> = []) async throws -> AnalysisResult {
+        // Skip source scanning for target types that don't have Swift source files
+        if target.type == .systemLibrary || target.type == .binaryTarget {
+            return AnalysisResult(
+                target: target,
+                missingDependencies: [],
+                unusedDependencies: [],
+                correctDependencies: [],
+                sourceFiles: []
+            )
+        }
+        
         // Scan source files for imports
         let sourceFiles = try await importScanner.scanDirectory(at: packageInfo.path, targetName: target.name, customWhitelist: customWhitelist)
 
@@ -66,8 +77,9 @@ public actor DependencyAnalyzer {
 
     private func getInternalModules(from packageInfo: PackageInfo, excluding currentTarget: Target) -> Set<String> {
         // Get names of other targets in the same package that can be imported
+        // Exclude test targets, system libraries, and binary targets from being considered as importable modules
         return Set(packageInfo.targets
-            .filter { $0.name != currentTarget.name && $0.type != .test }
+            .filter { $0.name != currentTarget.name && $0.type != .test && $0.type != .systemLibrary && $0.type != .binaryTarget }
             .map { $0.name })
     }
 
@@ -112,6 +124,9 @@ public actor DependencyAnalyzer {
         case .executable: "🔧"
         case .library: "📚"
         case .test: "🧪"
+        case .systemLibrary: "🏛️"
+        case .binaryTarget: "📦"
+        case .plugin: "🔌"
         }
 
         if !result.hasIssues && quiet {

Sources/SwiftDependencyAuditLib/Models.swift

@@ -17,6 +17,9 @@ public struct Target: Sendable {
         case executable
         case library
         case test
+        case systemLibrary
+        case binaryTarget
+        case plugin
     }
 }
 

Sources/SwiftDependencyAuditLib/PackageParser.swift

@@ -109,6 +109,9 @@ public actor PackageParser {
         let libraryRegex = /\.target\s*\(\s*name:\s*"([^"]+)"[^.]*?dependencies:\s*\[([^\]]*(?:\[[^\]]*\][^\]]*)*)\]/
         let testRegex = /\.testTarget\s*\(\s*name:\s*"([^"]+)"[^.]*?dependencies:\s*\[([^\]]*(?:\[[^\]]*\][^\]]*)*)\]/
         let macroRegex = /\.macro\s*\(\s*name:\s*"([^"]+)"[^.]*?dependencies:\s*\[([^\]]*(?:\[[^\]]*\][^\]]*)*)\]/
+        let systemLibraryRegex = /\.systemLibrary\s*\(\s*name:\s*"([^"]+)"[^)]*\)/
+        let binaryTargetRegex = /\.binaryTarget\s*\(\s*name:\s*"([^"]+)"[^)]*\)/
+        let pluginRegex = /\.plugin\s*\(\s*name:\s*"([^"]+)"[^.]*?dependencies:\s*\[([^\]]*(?:\[[^\]]*\][^\]]*)*)\]/
 
         // Parse executable targets
         for match in targetsSection.matches(of: executableRegex) {
@@ -166,9 +169,47 @@ public actor PackageParser {
             ))
         }
 
+        // Parse system library targets (no dependencies to parse)
+        for match in targetsSection.matches(of: systemLibraryRegex) {
+            let name = String(match.1)
+            
+            targets.append(Target(
+                name: name,
+                type: .systemLibrary,
+                dependencies: [], // System libraries don't have Swift dependencies
+                path: nil
+            ))
+        }
+        
+        // Parse binary targets (no dependencies to parse)
+        for match in targetsSection.matches(of: binaryTargetRegex) {
+            let name = String(match.1)
+            
+            targets.append(Target(
+                name: name,
+                type: .binaryTarget,
+                dependencies: [], // Binary targets don't have Swift dependencies
+                path: nil
+            ))
+        }
+        
+        // Parse plugin targets
+        for match in targetsSection.matches(of: pluginRegex) {
+            let name = String(match.1)
+            let dependenciesStr = String(match.2)
+            let dependencies = parseDependencyList(dependenciesStr)
+            
+            targets.append(Target(
+                name: name,
+                type: .plugin,
+                dependencies: dependencies,
+                path: nil
+            ))
+        }
+        
         // Parse targets without dependencies by looking for all target declarations
         // and seeing which ones don't have dependency arrays
-        let allTargetRegex = /\.(target|executableTarget|testTarget|macro)\s*\(\s*name:\s*"([^"]+)"([^)]*)\)/
+        let allTargetRegex = /\.(target|executableTarget|testTarget|macro|plugin)\s*\(\s*name:\s*"([^"]+)"([^)]*)\)/
         var targetNamesWithDeps = Set<String>()
         for target in targets {
             targetNamesWithDeps.insert(target.name)
@@ -194,6 +235,8 @@ public actor PackageParser {
                     targetType = .test
                 case "macro":
                     targetType = .library // Treat macros as library targets for dependency analysis
+                case "plugin":
+                    targetType = .plugin
                 default: // "target"
                     targetType = .library
                 }