Make IntroducingToSequentialParentScopeSyntax and SequentialScope internal. Format.

Author: MAJKFL

Sources/SwiftLexicalLookup/IntroducingToSequentialParentScopeSyntax.swift

@@ -12,7 +12,7 @@
 
 import SwiftSyntax
 
-@_spi(Experimental) public protocol IntroducingToSequentialParentScopeSyntax: ScopeSyntax {
+protocol IntroducingToSequentialParentScopeSyntax: ScopeSyntax {
   /// Returns names matching lookup that should be
   /// handled by it's parent sequential scope.
   func introducesToSequentialParent(

Sources/SwiftLexicalLookup/LookupName.swift

@@ -23,7 +23,7 @@ import SwiftSyntax
   case newValue(AccessorDeclSyntax)
   /// `oldValue` available by default inside `didSet`.
   case oldValue(AccessorDeclSyntax)
-  
+
   /// Syntax associated with this name.
   @_spi(Experimental) public var syntax: SyntaxProtocol {
     switch self {
@@ -39,7 +39,7 @@ import SwiftSyntax
       syntax
     }
   }
-  
+
   /// Used for name comparison.
   var name: String {
     switch self {

Sources/SwiftLexicalLookup/LookupState.swift

@@ -15,7 +15,7 @@ import Foundation
 @_spi(Experimental) public struct LookupState {
   /// Specifies scopes that introduce names to their parent and
   /// should be skipped during lookup in sequential scopes.
-  @_spi(Experimental) public var skipSequentialIntroductionFrom: IntroducingToSequentialParentScopeSyntax?
-  
+  var skipSequentialIntroductionFrom: IntroducingToSequentialParentScopeSyntax?
+
   @_spi(Experimental) public init() {}
 }

Sources/SwiftLexicalLookup/ScopeImplementations.swift

@@ -185,7 +185,7 @@ import SwiftSyntax
   }
 }
 
-@_spi(Experimental) extension ForStmtSyntax: ScopeSyntax {  
+@_spi(Experimental) extension ForStmtSyntax: ScopeSyntax {
   /// Names introduced in the `for` body.
   @_spi(Experimental) public var introducedNames: [LookupName] {
     LookupName.getNames(from: pattern)
@@ -367,6 +367,8 @@ import SwiftSyntax
 @_spi(Experimental) extension ExtensionDeclSyntax: TypeScopeSyntax {}
 
 @_spi(Experimental) extension AccessorDeclSyntax: ScopeSyntax {
+  /// Implicit and/or explicit names introduced
+  /// withing the accessor..
   @_spi(Experimental) public var introducedNames: [LookupName] {
     if let parameters {
       LookupName.getNames(from: parameters)

Sources/SwiftLexicalLookup/ScopeSyntax.swift

@@ -58,10 +58,12 @@ extension SyntaxProtocol {
   var introducedNames: [LookupName] { get }
   /// Finds all declarations `name` refers to. `syntax` specifies the node lookup was triggered with.
   /// If `name` set to `nil`, returns all available names at the given node.
-  func lookup(for name: String?, 
-              at syntax: SyntaxProtocol,
-              with config: LookupConfig,
-              state: LookupState) -> [LookupResult]
+  func lookup(
+    for name: String?,
+    at syntax: SyntaxProtocol,
+    with config: LookupConfig,
+    state: LookupState
+  ) -> [LookupResult]
 }
 
 @_spi(Experimental) extension ScopeSyntax {
@@ -99,7 +101,8 @@ extension SyntaxProtocol {
     if filteredNames.isEmpty {
       return lookupInParent(for: name, at: syntax, with: config, state: state)
     } else {
-      return [.fromScope(self, withNames: filteredNames)] + lookupInParent(for: name, at: syntax, with: config, state: state)
+      return [.fromScope(self, withNames: filteredNames)]
+        + lookupInParent(for: name, at: syntax, with: config, state: state)
     }
   }
 

Sources/SwiftLexicalLookup/SequentialScopeSyntax.swift

@@ -15,7 +15,7 @@ import SwiftSyntax
 /// Scope that, in addition to names introduced by itself,
 /// also handles names introduced by
 /// `IntroducingToSequentialParentScopeSyntax` children scopes.
-@_spi(Experimental) public protocol SequentialScopeSyntax: ScopeSyntax {
+protocol SequentialScopeSyntax: ScopeSyntax {
   /// Returns names introduced by `codeBlockItems`
   /// and included `IntroducingToSequentialParentScopeSyntax` children
   /// scopes that match the lookup.
@@ -29,8 +29,8 @@ import SwiftSyntax
   ) -> [LookupResult]
 }
 
-@_spi(Experimental) extension SequentialScopeSyntax {
-  @_spi(Experimental) public func sequentialLookup(
+extension SequentialScopeSyntax {
+  func sequentialLookup(
     in codeBlockItems: any Collection<CodeBlockItemSyntax>,
     for name: String?,
     at syntax: SyntaxProtocol,
@@ -47,7 +47,8 @@ import SwiftSyntax
       {
         // Check if the enocountered scope should be ignored.
         if let scopeToSkip = state.skipSequentialIntroductionFrom,
-           scopeToSkip.id == introducingToParentScope.id {
+          scopeToSkip.id == introducingToParentScope.id
+        {
           continue
         }
 
@@ -59,7 +60,12 @@ import SwiftSyntax
 
         // Add names introduced by the encountered scope.
         result.append(
-          contentsOf: introducingToParentScope.introducesToSequentialParent(for: name, at: syntax, with: config, state: state)
+          contentsOf: introducingToParentScope.introducesToSequentialParent(
+            for: name,
+            at: syntax,
+            with: config,
+            state: state
+          )
         )
       } else {
         // Extract new names from encountered node.

Sources/SwiftLexicalLookup/TypeScopeSyntax.swift

@@ -13,6 +13,8 @@
 import SwiftSyntax
 
 @_spi(Experimental) public protocol TypeScopeSyntax: ScopeSyntax, DeclSyntaxProtocol {
+  /// `self` and `Self` names referring to
+  /// this scope.
   var implicitInstanceAndTypeNames: [LookupName] { get }
 }
 

Tests/SwiftLexicalLookupTest/ExpectedName.swift

@@ -31,7 +31,7 @@ enum ImplicitNameExpectation {
   case error(String)
   case newValue(String)
   case oldValue(String)
-  
+
   func assertExpectation(marker: String, for name: LookupImplicitNameKind) {
     switch (name, self) {
     case (.self, .self): break
@@ -43,7 +43,7 @@ enum ImplicitNameExpectation {
       XCTFail("For marker \(marker), actual name kind \(name) doesn't match expected \(self)")
     }
   }
-  
+
   var marker: String {
     switch self {
     case .self(let marker),
@@ -66,7 +66,7 @@ enum NameExpectation: ExpectedName {
   var marker: String {
     switch self {
     case .identifier(let marker),
-        .declaration(let marker):
+      .declaration(let marker):
       marker
     case .implicit(let implicitName):
       implicitName.marker