Merge pull request #2951 from swiftlang/elsh/at-documentation-with-package-arg

Fix parsing `package` in @_documentation(visibility: package)

Author: nkcsgexi

Sources/SwiftParser/Attributes.swift

@@ -1170,6 +1170,7 @@ extension Parser {
           case `private`
           case `fileprivate`
           case `internal`
+          case `package`
           case `public`
           case `open`
 
@@ -1178,6 +1179,7 @@ extension Parser {
             case .private: return .keyword(.private)
             case .fileprivate: return .keyword(.fileprivate)
             case .internal: return .keyword(.internal)
+            case .package: return .keyword(.package)
             case .public: return .keyword(.public)
             case .open: return .keyword(.open)
             }
@@ -1188,6 +1190,7 @@ extension Parser {
             case TokenSpec(.private): self = .private
             case TokenSpec(.fileprivate): self = .fileprivate
             case TokenSpec(.internal): self = .internal
+            case TokenSpec(.package): self = .package
             case TokenSpec(.public): self = .public
             case TokenSpec(.open): self = .open
             default: return nil

Tests/SwiftParserTest/AttributeTests.swift

@@ -671,6 +671,7 @@ final class AttributeTests: ParserTestCase {
 
   func testDocumentationAttribute() {
     assertParse("@_documentation(visibility: internal) @_exported import A")
+    assertParse("@_documentation(visibility: package) @objc final public class Klass {}")
     assertParse("@_documentation(metadata: cool_stuff) public class SomeClass {}")
     assertParse(#"@_documentation(metadata: "this is a longer string") public class OtherClass {}"#)
     assertParse(