[PackageEditing] Fix dependency insertion when the existing deps list doesn't have a trailing comma

Author: owenv

Sources/SPMPackageEditor/ManifestRewriter.swift

@@ -344,7 +344,12 @@ final class PackageDependencyWriter: SyntaxRewriter {
         let rightBrace = SyntaxFactory.makeRightSquareBracketToken(
             leadingTrivia: [.newlines(1), .spaces(4)])
 
-        return ExprSyntax(node.addElement(newDependencyElement)
+        let newElements = SyntaxFactory.makeArrayElementList(
+            node.elements.dropLast() +
+                [node.elements.last?.withTrailingComma(SyntaxFactory.makeCommaToken()),
+                 newDependencyElement].compactMap {$0})
+
+        return ExprSyntax(node.withElements(newElements)
                             .withRightSquare(rightBrace))
     }
 }

Sources/SPMPackageEditor/PackageEditor.swift

@@ -73,7 +73,7 @@ public final class PackageEditor {
             requirement = .localPackage
         } else {
             // Otherwise, first lookup the dependency.
-            let spec = RepositorySpecifier(url: options.url)
+            let spec = RepositorySpecifier(url: url)
             let handle = try tsc_await{ context.repositoryManager.lookup(repository: spec, completion: $0) }
 
             let repo = try handle.open()

Tests/SPMPackageEditorTests/AddPackageDependencyTests.swift

@@ -233,4 +233,38 @@ final class AddPackageDependencyTests: XCTestCase {
             )
             """)
     }
+
+    func testAddPackageDependency7() throws {
+        let manifest = """
+            let package = Package(
+                name: "exec",
+                dependencies: [
+                    .package(url: "https://github.com/foo/bar", from: "1.0.3")
+                ],
+                targets: [
+                    .target(name: "exec")
+                ]
+            )
+            """
+
+
+        let editor = try ManifestRewriter(manifest)
+        try editor.addPackageDependency(
+            url: "https://github.com/foo/goo",
+            requirement: .upToNextMajor("1.0.1")
+        )
+
+        XCTAssertEqual(editor.editedManifest, """
+            let package = Package(
+                name: "exec",
+                dependencies: [
+                    .package(url: "https://github.com/foo/bar", from: "1.0.3"),
+                    .package(url: "https://github.com/foo/goo", from: "1.0.1"),
+                ],
+                targets: [
+                    .target(name: "exec")
+                ]
+            )
+            """)
+    }
 }