Fix wrong fix-it in modifier for variable decl

Author: kimdv

Sources/SwiftParserDiagnostics/ParseDiagnosticsGenerator.swift

@@ -1525,6 +1525,40 @@ public class ParseDiagnosticsGenerator: SyntaxAnyVisitor {
     if shouldSkip(node) {
       return .skipChildren
     }
+
+    if let modifiers = node.modifiers, modifiers.hasError {
+      for modifier in modifiers {
+        if let detail = modifier.detail, detail.detail != .keyword(.set) {
+          let unexpectedBetweenLeftParenAndDetail = detail.unexpectedBetweenLeftParenAndDetail?.tokens(viewMode: .all)
+          let unexpectedBetweenDetailAndRightParen = detail.unexpectedBetweenDetailAndRightParen?.tokens(viewMode: .all)
+          let unexpectedTokens = (unexpectedBetweenLeftParenAndDetail?.map { $0 } ?? []) + (unexpectedBetweenDetailAndRightParen?.map { $0 } ?? [])
+
+          // If there is no unexpected tokens it means we miss a paren or set keyword.
+          // So we just skip the handling here
+          guard !unexpectedTokens.isEmpty else {
+            return .visitChildren
+          }
+
+          addDiagnostic(
+            unexpectedTokens.first!,
+            MissingNodesError(missingNodes: [Syntax(detail.detail)]),
+            fixIts: [
+              FixIt(
+                message: ReplaceTokensFixIt(
+                  replaceTokens: unexpectedTokens.map { $0 },
+                  replacements: [detail.detail]
+                ),
+                changes: [
+                  FixIt.MultiNodeChange.makePresent(detail.detail)
+                ] + unexpectedTokens.map { FixIt.MultiNodeChange.makeMissing($0) }
+              )
+            ],
+            handledNodes: [detail.id] + unexpectedTokens.map(\.id)
+          )
+        }
+      }
+    }
+
     let missingTries = node.bindings.compactMap({
       return $0.initializer?.value.as(TryExprSyntax.self)?.tryKeyword
     })

Tests/SwiftParserTest/DeclarationTests.swift

@@ -343,11 +343,10 @@ final class DeclarationTests: XCTestCase {
       private(1️⃣get) var a = 0
       """,
       diagnostics: [
-        DiagnosticSpec(message: "expected 'set' in modifier", fixIts: ["insert 'set'"]),
-        DiagnosticSpec(message: "unexpected code 'get' in modifier"),
+        DiagnosticSpec(message: "expected 'set' in modifier", fixIts: ["replace 'get' with 'set'"])
       ],
       fixedSource: """
-        private(setget) var a = 0
+        private(set) var a = 0
         """
     )
 
@@ -374,17 +373,29 @@ final class DeclarationTests: XCTestCase {
       private(1️⃣get, set) var a = 0
       """,
       diagnostics: [
-        DiagnosticSpec(message: "unexpected code 'get,' in modifier")
-      ]
+        DiagnosticSpec(
+          message: "expected 'set' in modifier",
+          fixIts: ["replace 'get,' with 'set'"]
+        )
+      ],
+      fixedSource: """
+        private(set) var a = 0
+        """
     )
 
     assertParse(
       """
       private(1️⃣get: set) var a = 0
       """,
       diagnostics: [
-        DiagnosticSpec(message: "unexpected code 'get:' in modifier")
-      ]
+        DiagnosticSpec(
+          message: "expected 'set' in modifier",
+          fixIts: ["replace 'get:' with 'set'"]
+        )
+      ],
+      fixedSource: """
+        private(set) var a = 0
+        """
     )
 
     assertParse(
@@ -410,12 +421,17 @@ final class DeclarationTests: XCTestCase {
 
     assertParse(
       """
-      private(1️⃣get, set2️⃣, didSet) var a = 0
+      private(1️⃣get, set, didSet) var a = 0
       """,
       diagnostics: [
-        DiagnosticSpec(locationMarker: "1️⃣", message: "unexpected code 'get,' in modifier"),
-        DiagnosticSpec(locationMarker: "2️⃣", message: "unexpected code ', didSet' in modifier"),
-      ]
+        DiagnosticSpec(
+          message: "expected 'set' in modifier",
+          fixIts: ["replace 'get, , didSet' with 'set'"]
+        )
+      ],
+      fixedSource: """
+        private(set) var a = 0
+        """
     )
 
     assertParse(