[Commands] Switch AddSetting/Migrate to use refactoring action from swift-syntax

Author: xedin

Sources/Commands/PackageCommands/AddSetting.swift

@@ -16,8 +16,8 @@ import CoreCommands
 import Foundation
 import PackageGraph
 import PackageModel
-import PackageModelSyntax
 import SwiftParser
+@_spi(PackageRefactor) import SwiftRefactor
 import TSCBasic
 import TSCUtility
 import Workspace
@@ -124,7 +124,7 @@ extension SwiftPackageCommand {
                     }
                 }
 
-                let editResult: PackageEditResult
+                let editResult: PackageEdit
 
                 switch setting {
                 case .experimentalFeature:
@@ -146,7 +146,7 @@ extension SwiftPackageCommand {
 
                     editResult = try AddSwiftSetting.languageMode(
                         to: target,
-                        mode: mode,
+                        mode: mode.rawValue,
                         manifest: manifestSyntax
                     )
                 case .strictMemorySafety:

Sources/Commands/PackageCommands/Migrate.swift

@@ -23,7 +23,9 @@ import OrderedCollections
 
 import PackageGraph
 import PackageModel
-import enum PackageModelSyntax.ManifestEditError
+
+@_spi(PackageRefactor)
+import enum SwiftRefactor.ManifestEditError
 
 import SPMBuildCore
 import SwiftFixIt
@@ -283,16 +285,14 @@ extension SwiftPackageCommand {
                     switch error {
                     case .cannotFindPackage,
                          .cannotAddSettingsToPluginTarget,
-                         .existingDependency:
+                         .existingDependency,
+                         .malformedManifest:
                         break
                     case .cannotFindArrayLiteralArgument,
                          // This means the target could not be found
                          // syntactically, not that it does not exist.
                          .cannotFindTargets,
-                         .cannotFindTarget,
-                         // This means the swift-tools-version is lower than
-                         // the version where one of the setting was introduced.
-                         .oldManifest:
+                         .cannotFindTarget:
                         let settings = try features.map {
                             try $0.swiftSettingDescription
                         }.joined(separator: ", ")