Finish adding SPIs and lowering access levels.

Author: allevato

Sources/SwiftFormat/Core/Context.swift

@@ -20,6 +20,7 @@ import SwiftParser
 ///
 /// Specifically, it is the container for the shared configuration, diagnostic consumer, and URL of
 /// the current file.
+@_spi(Rules)
 public final class Context {
 
   /// Tracks whether `XCTest` has been imported so that certain logic can be modified for files that

Sources/SwiftFormat/Core/DocumentationComment.swift

@@ -19,6 +19,7 @@ import SwiftSyntax
 /// also the nested information that can be provided on a parameter. For example, when a parameter
 /// is a function type, it can provide not only a brief summary but also its own parameter and
 /// return value descriptions.
+@_spi(Testing)
 public struct DocumentationComment {
   /// A description of a parameter in a documentation comment.
   public struct Parameter {

Sources/SwiftFormat/Core/DocumentationCommentText.swift

@@ -17,6 +17,7 @@ import SwiftSyntax
 /// This type should be used when only the text of the comment is important, not the Markdown
 /// structural organization. It automatically handles trimming leading indentation from comments as
 /// well as "ASCII art" in block comments (i.e., leading asterisks on each line).
+@_spi(Testing)
 public struct DocumentationCommentText {
   /// Denotes the kind of punctuation used to introduce the comment.
   public enum Introducer {

Sources/SwiftFormat/Core/ImportsXCTestVisitor.swift

@@ -52,6 +52,7 @@ private class ImportsXCTestVisitor: SyntaxVisitor {
 /// - Parameters:
 ///   - context: The context information of the target source file.
 ///   - sourceFile: The file to be visited.
+@_spi(Testing)
 public func setImportsXCTest(context: Context, sourceFile: SourceFileSyntax) {
   guard context.importsXCTest == .notDetermined else { return }
   let visitor = ImportsXCTestVisitor(context: context)

Sources/SwiftFormat/Core/LegacyTriviaBehavior.swift

@@ -5,6 +5,7 @@ import SwiftSyntax
 ///
 /// Eventually we should get rid of this and update the core formatting code to adjust to the new
 /// behavior, but this workaround lets us keep the current implementation without larger changes.
+@_spi(Testing)
 public func restoringLegacyTriviaBehavior(_ sourceFile: SourceFileSyntax) -> SourceFileSyntax {
   return LegacyTriviaBehaviorRewriter().visit(sourceFile)
 }

Sources/SwiftFormat/Core/Rule.swift

@@ -14,6 +14,7 @@ import Foundation
 import SwiftSyntax
 
 /// A Rule is a linting or formatting pass that executes in a given context.
+@_spi(Rules)
 public protocol Rule {
   /// The context in which the rule is executed.
   var context: Context { get }

Sources/SwiftFormat/Core/RuleMask.swift

@@ -37,6 +37,7 @@ import SwiftSyntax
 ///
 /// The rules themselves reference RuleMask to see if it is disabled for the line it is currently
 /// examining.
+@_spi(Testing)
 public class RuleMask {
   /// Stores the source ranges in which all rules are ignored.
   private var allRulesIgnoredRanges: [SourceRange] = []

Sources/SwiftFormat/Core/RuleNameCache+Generated.swift

@@ -13,6 +13,7 @@
 // This file is automatically generated with generate-pipeline. Do Not Edit!
 
 /// By default, the `Rule.ruleName` should be the name of the implementing rule type.
+@_spi(Testing)
 public let ruleNameCache: [ObjectIdentifier: String] = [
   ObjectIdentifier(AllPublicDeclarationsHaveDocumentation.self): "AllPublicDeclarationsHaveDocumentation",
   ObjectIdentifier(AlwaysUseLowerCamelCase.self): "AlwaysUseLowerCamelCase",

Sources/SwiftFormat/Core/RuleState.swift

@@ -12,6 +12,7 @@
 
 /// The enablement of a lint/format rule based on the presence or absence of comment directives in
 /// the source file.
+@_spi(Testing)
 public enum RuleState {
 
   /// There is no explicit information in the source file about whether the rule should be enabled

Sources/SwiftFormat/Core/SyntaxFormatRule.swift

@@ -13,10 +13,11 @@
 import SwiftSyntax
 
 /// A rule that both formats and lints a given file.
-open class SyntaxFormatRule: SyntaxRewriter, Rule {
+@_spi(Rules)
+public class SyntaxFormatRule: SyntaxRewriter, Rule {
   /// Whether this rule is opt-in, meaning it's disabled by default. Rules are opt-out unless they
   /// override this property.
-  open class var isOptIn: Bool {
+  public class var isOptIn: Bool {
     return false
   }
 
@@ -28,7 +29,7 @@ open class SyntaxFormatRule: SyntaxRewriter, Rule {
     self.context = context
   }
 
-  open override func visitAny(_ node: Syntax) -> Syntax? {
+  public override func visitAny(_ node: Syntax) -> Syntax? {
     // If the rule is not enabled, then return the node unmodified; otherwise, returning nil tells
     // SwiftSyntax to continue with the standard dispatch.
     guard context.isRuleEnabled(type(of: self), node: node) else { return node }