Replace string-based keyword references with Keyword enum in CodeGeneration

The CodeGeneration module was previously referring to keywords using string-based references like switch, which is not idiomatic Swift. To make the code more concise and in line with Swift's best practices, this commit introduces a Keyword enum in the KeywordSpec.swift file.
Now, instead of string-based references, the new Keyword enum contains all the possible keyword kinds. It also includes a computed property that returns the corresponding KeywordSpec for a given keyword, allowing for more concise and idiomatic references.
The code has been refactored to replace all string-based keyword references with members of the newly introduced Keyword enum. Associated tests have also been updated to reflect these changes.

Author: Matejkob

CodeGeneration/Sources/SyntaxSupport/AttributeNodes.swift

@@ -179,7 +179,7 @@ public let ATTRIBUTE_NODES: [Node] = [
       Child(
         name: "AvailabilityLabel",
         deprecatedName: "Label",
-        kind: .token(choices: [.keyword(text: "availability")]),
+        kind: .token(choices: [.keyword(.availability)]),
         nameForDiagnostics: "label",
         documentation: "The label of the argument"
       ),
@@ -241,7 +241,7 @@ public let ATTRIBUTE_NODES: [Node] = [
     children: [
       Child(
         name: "BeforeLabel",
-        kind: .token(choices: [.keyword(text: "before")]),
+        kind: .token(choices: [.keyword(.before)]),
         documentation: "The \"before\" label."
       ),
       Child(
@@ -277,7 +277,7 @@ public let ATTRIBUTE_NODES: [Node] = [
       ),
       Child(
         name: "CTypeLabel",
-        kind: .token(choices: [.keyword(text: "cType")]),
+        kind: .token(choices: [.keyword(.cType)]),
         isOptional: true
       ),
       Child(
@@ -302,7 +302,7 @@ public let ATTRIBUTE_NODES: [Node] = [
     children: [
       Child(
         name: "WitnessMethodLabel",
-        kind: .token(choices: [.keyword(text: "witness_method")])
+        kind: .token(choices: [.keyword(.witness_method)])
       ),
       Child(
         name: "Colon",
@@ -330,7 +330,7 @@ public let ATTRIBUTE_NODES: [Node] = [
     children: [
       Child(
         name: "OfLabel",
-        kind: .token(choices: [.keyword(text: "of")]),
+        kind: .token(choices: [.keyword(.of)]),
         documentation: "The \"of\" label."
       ),
       Child(
@@ -352,7 +352,7 @@ public let ATTRIBUTE_NODES: [Node] = [
       Child(
         name: "AccessorSpecifier",
         deprecatedName: "AccessorKind",
-        kind: .token(choices: [.keyword(text: "get"), .keyword(text: "set")]),
+        kind: .token(choices: [.keyword(.get), .keyword(.set)]),
         documentation: "The accessor name.",
         isOptional: true
       ),
@@ -392,7 +392,7 @@ public let ATTRIBUTE_NODES: [Node] = [
       Child(
         name: "Argument",
         deprecatedName: "Parameter",
-        kind: .token(choices: [.token(.identifier), .token(.integerLiteral), .keyword(text: "self")])
+        kind: .token(choices: [.token(.identifier), .token(.integerLiteral), .keyword(.self)])
       ),
       Child(
         name: "TrailingComma",
@@ -412,7 +412,7 @@ public let ATTRIBUTE_NODES: [Node] = [
     children: [
       Child(
         name: "WrtLabel",
-        kind: .token(choices: [.keyword(text: "wrt")]),
+        kind: .token(choices: [.keyword(.wrt)]),
         documentation: "The \"wrt\" label."
       ),
       Child(
@@ -478,7 +478,7 @@ public let ATTRIBUTE_NODES: [Node] = [
       Child(
         name: "KindSpecifier",
         deprecatedName: "DiffKind",
-        kind: .token(choices: [.keyword(text: "_forward"), .keyword(text: "reverse"), .keyword(text: "_linear")]),
+        kind: .token(choices: [.keyword(._forward), .keyword(.reverse), .keyword(._linear)]),
         isOptional: true
       ),
       Child(
@@ -521,7 +521,7 @@ public let ATTRIBUTE_NODES: [Node] = [
     children: [
       Child(
         name: "Label",
-        kind: .token(choices: [.keyword(text: "visibility"), .keyword(text: "metadata")]),
+        kind: .token(choices: [.keyword(.visibility), .keyword(.metadata)]),
         nameForDiagnostics: "label"
       ),
       Child(
@@ -535,11 +535,11 @@ public let ATTRIBUTE_NODES: [Node] = [
             name: "Token",
             kind: .token(choices: [
               .token(.identifier),
-              .keyword(text: "private"),
-              .keyword(text: "fileprivate"),
-              .keyword(text: "internal"),
-              .keyword(text: "public"),
-              .keyword(text: "open"),
+              .keyword(.private),
+              .keyword(.fileprivate),
+              .keyword(.internal),
+              .keyword(.public),
+              .keyword(.open),
             ])
           ),  // Keywords can be: public, internal, private, fileprivate, open
           Child(
@@ -573,7 +573,7 @@ public let ATTRIBUTE_NODES: [Node] = [
     children: [
       Child(
         name: "ForLabel",
-        kind: .token(choices: [.keyword(text: "for")], requiresTrailingSpace: false)
+        kind: .token(choices: [.keyword(.for)], requiresTrailingSpace: false)
       ),
       Child(
         name: "Colon",
@@ -662,13 +662,13 @@ public let ATTRIBUTE_NODES: [Node] = [
       Child(
         name: "Label",
         kind: .token(choices: [
-          .keyword(text: "target"),
-          .keyword(text: "availability"),
-          .keyword(text: "exported"),
-          .keyword(text: "kind"),
-          .keyword(text: "spi"),
-          .keyword(text: "spiModule"),
-          .keyword(text: "available"),
+          .keyword(.target),
+          .keyword(.availability),
+          .keyword(.exported),
+          .keyword(.kind),
+          .keyword(.spi),
+          .keyword(.spiModule),
+          .keyword(.available),
         ]),
         nameForDiagnostics: "label",
         documentation: "The label of the argument"
@@ -755,7 +755,7 @@ public let ATTRIBUTE_NODES: [Node] = [
     children: [
       Child(
         name: "ModuleLabel",
-        kind: .token(choices: [.keyword(text: "module")])
+        kind: .token(choices: [.keyword(.module)])
       ),
       Child(
         name: "Colon",
@@ -840,7 +840,7 @@ public let ATTRIBUTE_NODES: [Node] = [
       Child(
         name: "TargetLabel",
         deprecatedName: "Label",
-        kind: .token(choices: [.keyword(text: "target")]),
+        kind: .token(choices: [.keyword(.target)]),
         nameForDiagnostics: "label",
         documentation: "The label of the argument"
       ),
@@ -873,7 +873,7 @@ public let ATTRIBUTE_NODES: [Node] = [
     children: [
       Child(
         name: "MessageLabel",
-        kind: .token(choices: [.keyword(text: "message")])
+        kind: .token(choices: [.keyword(.message)])
       ),
       Child(
         name: "Colon",
@@ -894,7 +894,7 @@ public let ATTRIBUTE_NODES: [Node] = [
     children: [
       Child(
         name: "SourceFileLabel",
-        kind: .token(choices: [.keyword(text: "sourceFile")])
+        kind: .token(choices: [.keyword(.sourceFile)])
       ),
       Child(
         name: "Colon",

CodeGeneration/Sources/SyntaxSupport/AvailabilityNodes.swift

@@ -66,11 +66,11 @@ public let AVAILABILITY_NODES: [Node] = [
       Child(
         name: "Label",
         kind: .token(choices: [
-          .keyword(text: "message"),
-          .keyword(text: "renamed"),
-          .keyword(text: "introduced"),
-          .keyword(text: "obsoleted"),
-          .keyword(text: "deprecated"),
+          .keyword(.message),
+          .keyword(.renamed),
+          .keyword(.introduced),
+          .keyword(.obsoleted),
+          .keyword(.deprecated),
         ]),
         nameForDiagnostics: "label",
         documentation: "The label of the argument"

CodeGeneration/Sources/SyntaxSupport/Child.swift

@@ -15,7 +15,7 @@ import SwiftSyntax
 /// The kind of token a node can contain. Either a token of a specific kind or a
 /// keyword with the given text.
 public enum TokenChoice: Equatable {
-  case keyword(text: String)
+  case keyword(Keyword)
   case token(Token)
 
   public var isKeyword: Bool {

CodeGeneration/Sources/SyntaxSupport/CommonNodes.swift

@@ -91,12 +91,12 @@ public let COMMON_NODES: [Node] = [
     children: [
       Child(
         name: "AsyncSpecifier",
-        kind: .token(choices: [.keyword(text: "async")]),
+        kind: .token(choices: [.keyword(.async)]),
         isOptional: true
       ),
       Child(
         name: "ThrowsSpecifier",
-        kind: .token(choices: [.keyword(text: "throws")]),
+        kind: .token(choices: [.keyword(.throws)]),
         isOptional: true
       ),
     ]
@@ -113,12 +113,12 @@ public let COMMON_NODES: [Node] = [
     children: [
       Child(
         name: "AsyncSpecifier",
-        kind: .token(choices: [.keyword(text: "async"), .keyword(text: "reasync")]),
+        kind: .token(choices: [.keyword(.async), .keyword(.reasync)]),
         isOptional: true
       ),
       Child(
         name: "ThrowsSpecifier",
-        kind: .token(choices: [.keyword(text: "throws"), .keyword(text: "rethrows")]),
+        kind: .token(choices: [.keyword(.throws), .keyword(.rethrows)]),
         isOptional: true
       ),
     ]
@@ -133,7 +133,7 @@ public let COMMON_NODES: [Node] = [
     children: [
       Child(
         name: "AsyncSpecifier",
-        kind: .token(choices: [.keyword(text: "async")]),
+        kind: .token(choices: [.keyword(.async)]),
         isOptional: true
       )
     ]
@@ -316,12 +316,12 @@ public let COMMON_NODES: [Node] = [
     children: [
       Child(
         name: "AsyncSpecifier",
-        kind: .token(choices: [.keyword(text: "async")]),
+        kind: .token(choices: [.keyword(.async)]),
         isOptional: true
       ),
       Child(
         name: "ThrowsSpecifier",
-        kind: .token(choices: [.keyword(text: "throws")]),
+        kind: .token(choices: [.keyword(.throws)]),
         isOptional: true
       ),
     ]