Remove SyntaxKind conformance to CaseIterable

There’s no good reason to iterate over all cases in `SyntaxKind` and if `SyntaxKind` conforms to `CaseIterable`, we can’t mark members of it as deprecated (https://github.com/apple/swift-syntax/pull/2237/files#r1359917461)
art-divin · Oct 17, 2023 · a318593 · a318593
1 parent ccab07f
commit a318593
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 4 deletions.
diff --git a/CodeGeneration/Sources/generate-swift-syntax/templates/swiftsyntax/SyntaxKindFile.swift b/CodeGeneration/Sources/generate-swift-syntax/templates/swiftsyntax/SyntaxKindFile.swift
@@ -19,7 +19,7 @@ let syntaxKindFile = SourceFileSyntax(leadingTrivia: copyrightHeader) {
   try! EnumDeclSyntax(
     """
     /// Enumerates the known kinds of Syntax represented in the Syntax tree.
-    public enum SyntaxKind: CaseIterable
+    public enum SyntaxKind
     """
   ) {
     DeclSyntax("case token")

diff --git a/Release Notes/511.md b/Release Notes/511.md
@@ -16,6 +16,9 @@
 - Effect specifiers:
   - Description: The `unexpectedAfterThrowsSpecifier` node of the various effect specifiers has been removed.
   - Pull request: https://github.com/apple/swift-syntax/pull/2219
+- `SyntaxKind` removed conformance to `CaseIterable`
+  - Description: `SyntaxKind` no longer conforms to `CaseIterable` since there is no good use case to iterate over all syntax kinds. 
+  - Pull request: https://github.com/apple/swift-syntax/pull/2292
 
 
 ## Template

diff --git a/Sources/SwiftSyntax/generated/SyntaxKind.swift b/Sources/SwiftSyntax/generated/SyntaxKind.swift
@@ -13,7 +13,7 @@
 //===----------------------------------------------------------------------===//
 
 /// Enumerates the known kinds of Syntax represented in the Syntax tree.
-public enum SyntaxKind: CaseIterable {
+public enum SyntaxKind {
   case token
   case accessorBlock
   case accessorDeclList

diff --git a/SwiftParserCLI/Sources/swift-parser-cli/BasicFormat.swift b/SwiftParserCLI/Sources/swift-parser-cli/BasicFormat.swift
@@ -47,8 +47,15 @@ struct BasicFormat: ParsableCommand, ParseCommand {
   var nodeType: String = "SourceFileSyntax"
 
   func run() throws {
-    let parsableMetatypes = SyntaxKind.allCases.compactMap {
-      $0.syntaxNodeType as? SyntaxParseable.Type
+    guard case .choices(let choices) = Syntax.structure else {
+      fatalError("Expected `Syntax.structure` to be the `choices` case")
+    }
+
+    let parsableMetatypes = choices.compactMap { (choice) -> SyntaxParseable.Type? in
+      guard case .node(let nodeType) = choice else {
+        return nil
+      }
+      return nodeType as? SyntaxParseable.Type
     }
     let parsableMetatypesByName = Dictionary(
       parsableMetatypes.map {