Rename TargetDescription -> Target

Author: DougGregor

Sources/SwiftRefactor/CMakeLists.txt

@@ -42,7 +42,7 @@ add_swift_syntax_library(SwiftRefactor
   PackageManifest/SourceControlURL.swift
   PackageManifest/StringUtils.swift
   PackageManifest/SyntaxEditUtils.swift
-  PackageManifest/TargetDescription.swift
+  PackageManifest/Target.swift
 )
 
 target_link_swift_syntax_libraries(SwiftRefactor PUBLIC

Sources/SwiftRefactor/PackageManifest/AddPluginUsage.swift

@@ -19,9 +19,9 @@ import SwiftSyntaxBuilder
 public struct AddPluginUsage: ManifestEditRefactoringProvider {
   public struct Context {
     public let targetName: String
-    public let pluginUsage: TargetDescription.PluginUsage
+    public let pluginUsage: Target.PluginUsage
 
-    public init(targetName: String, pluginUsage: TargetDescription.PluginUsage) {
+    public init(targetName: String, pluginUsage: Target.PluginUsage) {
       self.targetName = targetName
       self.pluginUsage = pluginUsage
     }

Sources/SwiftRefactor/PackageManifest/AddTarget.swift

@@ -17,10 +17,10 @@ import SwiftSyntaxBuilder
 /// Add a target to a manifest's source code.
 public struct AddTarget: ManifestEditRefactoringProvider {
   public struct Context {
-    public let target: TargetDescription
+    public let target: Target
     public let configuration: Configuration
 
-    public init(target: TargetDescription, configuration: Configuration = .init()) {
+    public init(target: Target, configuration: Configuration = .init()) {
       self.target = target
       self.configuration = configuration
     }
@@ -187,7 +187,7 @@ public struct AddTarget: ManifestEditRefactoringProvider {
   /// source files.
   fileprivate static func addPrimarySourceFile(
     outerPath: RelativePath,
-    target: TargetDescription,
+    target: Target,
     configuration: Configuration,
     to auxiliaryFiles: inout AuxiliaryFiles
   ) {
@@ -300,7 +300,7 @@ public struct AddTarget: ManifestEditRefactoringProvider {
   /// for a macro target.
   fileprivate static func addProvidedMacrosSourceFile(
     outerPath: RelativePath,
-    target: TargetDescription,
+    target: Target,
     to auxiliaryFiles: inout AuxiliaryFiles
   ) {
     auxiliaryFiles.addSourceFile(
@@ -321,7 +321,7 @@ public struct AddTarget: ManifestEditRefactoringProvider {
   }
 }
 
-fileprivate extension TargetDescription.Dependency {
+fileprivate extension Target.Dependency {
   /// Retrieve the name of the dependency
   var name: String {
     switch self {
@@ -349,7 +349,7 @@ fileprivate extension AuxiliaryFiles {
 
 /// The set of dependencies we need to introduce to a newly-created macro
 /// target.
-fileprivate let macroTargetDependencies: [TargetDescription.Dependency] = [
+fileprivate let macroTargetDependencies: [Target.Dependency] = [
   .product(name: "SwiftCompilerPlugin", package: "swift-syntax"),
   .product(name: "SwiftSyntaxMacros", package: "swift-syntax"),
 ]
@@ -375,7 +375,7 @@ fileprivate extension PackageDependency {
   }
 }
 
-fileprivate extension TargetDescription {
+fileprivate extension Target {
   var sanitizedName: String {
     name
       .mangledToC99ExtendedIdentifier()

Sources/SwiftRefactor/PackageManifest/AddTargetDependency.swift

@@ -18,12 +18,12 @@ import SwiftSyntaxBuilder
 public struct AddTargetDependency: ManifestEditRefactoringProvider {
   public struct Context {
     /// The dependency to add.
-    public var dependency: TargetDescription.Dependency
+    public var dependency: Target.Dependency
 
     /// The name of the target to which the dependency will be added.
     public var targetName: String
 
-    public init(dependency: TargetDescription.Dependency, targetName: String) {
+    public init(dependency: Target.Dependency, targetName: String) {
       self.dependency = dependency
       self.targetName = targetName
     }
@@ -78,7 +78,7 @@ public struct AddTargetDependency: ManifestEditRefactoringProvider {
 
   /// Implementation of adding a target dependency to an existing call.
   static func addTargetDependencyLocal(
-    _ dependency: TargetDescription.Dependency,
+    _ dependency: Target.Dependency,
     to targetCall: FunctionCallExprSyntax
   ) throws -> FunctionCallExprSyntax {
     try targetCall.appendingToArrayArgument(

…/PackageManifest/TargetDescription.swift …iftRefactor/PackageManifest/Target.swiftSources/SwiftRefactor/PackageManifest/TargetDescription.swift renamed to Sources/SwiftRefactor/PackageManifest/Target.swift Sources/SwiftRefactor/PackageManifest/TargetDescription.swift renamed to Sources/SwiftRefactor/PackageManifest/Target.swift

@@ -14,7 +14,7 @@ import SwiftSyntax
 
 /// Syntactic wrapper type that describes a target for refactoring
 /// purposes but does not interpret its contents.
-public struct TargetDescription {
+public struct Target {
   public let name: String
 
   /// The type of target.
@@ -71,7 +71,7 @@ public struct TargetDescription {
   }
 }
 
-extension TargetDescription: ManifestSyntaxRepresentable {
+extension Target: ManifestSyntaxRepresentable {
   /// The function name in the package manifest.
   private var functionName: String {
     switch type {
@@ -111,7 +111,7 @@ extension TargetDescription: ManifestSyntaxRepresentable {
   }
 }
 
-extension TargetDescription.Dependency: ManifestSyntaxRepresentable {
+extension Target.Dependency: ManifestSyntaxRepresentable {
   func asSyntax() -> ExprSyntax {
     switch self {
     case .byName(name: let name):
@@ -129,7 +129,7 @@ extension TargetDescription.Dependency: ManifestSyntaxRepresentable {
   }
 }
 
-extension TargetDescription.PluginUsage: ManifestSyntaxRepresentable {
+extension Target.PluginUsage: ManifestSyntaxRepresentable {
   func asSyntax() -> ExprSyntax {
     switch self {
     case .plugin(name: let name, package: nil):

Tests/SwiftRefactorTest/ManifestEditTests.swift

@@ -345,7 +345,7 @@ final class ManifestEditTests: XCTestCase {
       ],
       provider: AddTarget.self,
       context: .init(
-        target: TargetDescription(name: "MyLib")
+        target: Target(name: "MyLib")
       )
     )
   }
@@ -384,7 +384,7 @@ final class ManifestEditTests: XCTestCase {
       ],
       provider: AddTarget.self,
       context: .init(
-        target: TargetDescription(
+        target: Target(
           name: "MyLib",
           dependencies: [
             .byName(name: "OtherLib"),
@@ -442,7 +442,7 @@ final class ManifestEditTests: XCTestCase {
       ],
       provider: AddTarget.self,
       context: .init(
-        target: TargetDescription(
+        target: Target(
           name: "MyProgram target-name",
           type: .executable,
           dependencies: [
@@ -513,7 +513,7 @@ final class ManifestEditTests: XCTestCase {
       ],
       provider: AddTarget.self,
       context: .init(
-        target: TargetDescription(
+        target: Target(
           name: "MyMacro target-name",
           type: .macro
         )
@@ -553,7 +553,7 @@ final class ManifestEditTests: XCTestCase {
       ],
       provider: AddTarget.self,
       context: .init(
-        target: TargetDescription(
+        target: Target(
           name: "MyTest target-name",
           type: .test
         ),