Reduce visibility of some things to internal.

These changes are only the ones that don't cause compilation errors.

Author: allevato

Sources/SwiftFormat/Core/Context.swift

@@ -37,28 +37,28 @@ public final class Context {
   }
 
   /// The configuration for this run of the pipeline, provided by a configuration JSON file.
-  public let configuration: Configuration
+  let configuration: Configuration
 
   /// Defines the operators and their precedence relationships that were used during parsing.
-  public let operatorTable: OperatorTable
+  let operatorTable: OperatorTable
 
   /// Emits findings to the finding consumer.
-  public let findingEmitter: FindingEmitter
+  let findingEmitter: FindingEmitter
 
   /// The URL of the file being linted or formatted.
-  public let fileURL: URL
+  let fileURL: URL
 
   /// Indicates whether the file is known to import XCTest.
   public var importsXCTest: XCTestImportState
 
   /// An object that converts `AbsolutePosition` values to `SourceLocation` values.
-  public let sourceLocationConverter: SourceLocationConverter
+  let sourceLocationConverter: SourceLocationConverter
 
   /// Contains the rules have been disabled by comments for certain line numbers.
-  public let ruleMask: RuleMask
+  let ruleMask: RuleMask
 
   /// Contains all the available rules' names associated to their types' object identifiers.
-  public let ruleNameCache: [ObjectIdentifier: String]
+  let ruleNameCache: [ObjectIdentifier: String]
 
   /// Creates a new Context with the provided configuration, diagnostic engine, and file URL.
   public init(
@@ -87,7 +87,7 @@ public final class Context {
 
   /// Given a rule's name and the node it is examining, determine if the rule is disabled at this
   /// location or not.
-  public func isRuleEnabled<R: Rule>(_ rule: R.Type, node: Syntax) -> Bool {
+  func isRuleEnabled<R: Rule>(_ rule: R.Type, node: Syntax) -> Bool {
     let loc = node.startLocation(converter: self.sourceLocationConverter)
 
     assert(

Sources/SwiftFormat/Core/Finding+Convenience.swift

@@ -14,7 +14,7 @@ import SwiftSyntax
 
 extension Finding.Location {
   /// Creates a new `Finding.Location` by converting the given `SourceLocation` from `SwiftSyntax`.
-  public init(_ sourceLocation: SourceLocation) {
+  init(_ sourceLocation: SourceLocation) {
     self.init(file: sourceLocation.file, line: sourceLocation.line, column: sourceLocation.column)
   }
 }

Sources/SwiftFormat/Core/FindingEmitter.swift

@@ -19,7 +19,7 @@
 /// If the consumer function is nil, then the `emit` function is a no-op. This allows callers, such
 /// as lint/format rules and the pretty-printer, to emit findings unconditionally, without wrapping
 /// each call in a check about whether the client is interested in receiving those findings or not.
-public final class FindingEmitter {
+final class FindingEmitter {
   /// An optional function that will be called and passed a finding each time one is emitted.
   private let consumer: ((Finding) -> Void)?
 

Sources/SwiftFormat/Core/SyntaxProtocol+Convenience.swift

@@ -24,7 +24,7 @@ extension SyntaxProtocol {
   /// - Parameter index: The index of the trivia piece in the leading trivia whose position should
   ///   be returned.
   /// - Returns: The absolute position of the trivia piece.
-  public func position(ofLeadingTriviaAt index: Trivia.Index) -> AbsolutePosition {
+  func position(ofLeadingTriviaAt index: Trivia.Index) -> AbsolutePosition {
     guard leadingTrivia.indices.contains(index) else {
       preconditionFailure("Index was out of bounds in the node's leading trivia.")
     }
@@ -50,7 +50,7 @@ extension SyntaxProtocol {
   ///   - converter: The `SourceLocationConverter` that was previously initialized using the root
   ///     tree of this node.
   /// - Returns: The source location of the trivia piece.
-  public func startLocation(
+  func startLocation(
     ofLeadingTriviaAt index: Trivia.Index,
     converter: SourceLocationConverter
   ) -> SourceLocation {
@@ -60,7 +60,7 @@ extension SyntaxProtocol {
 
 extension SyntaxCollection {
   /// The first element in the syntax collection if it is the *only* element, or nil otherwise.
-  public var firstAndOnly: Element? {
+  var firstAndOnly: Element? {
     var iterator = makeIterator()
     guard let first = iterator.next() else { return nil }
     guard iterator.next() == nil else { return nil }

Sources/SwiftFormat/Core/Trivia+Convenience.swift

@@ -13,7 +13,7 @@
 import SwiftSyntax
 
 extension Trivia {
-  public var numberOfComments: Int {
+  var numberOfComments: Int {
     var count = 0
     for piece in self {
       switch piece {
@@ -27,15 +27,15 @@ extension Trivia {
   }
 
   /// Returns whether the trivia contains at least 1 `lineComment`.
-  public var hasLineComment: Bool {
+  var hasLineComment: Bool {
     return self.contains {
       if case .lineComment = $0 { return true }
       return false
     }
   }
 
   /// Returns this set of trivia, without any whitespace characters.
-  public func withoutSpaces() -> Trivia {
+  func withoutSpaces() -> Trivia {
     return Trivia(
       pieces: filter {
         if case .spaces = $0 { return false }
@@ -45,7 +45,7 @@ extension Trivia {
   }
 
   /// Returns this set of trivia, without any leading spaces.
-  public func withoutLeadingSpaces() -> Trivia {
+  func withoutLeadingSpaces() -> Trivia {
     return Trivia(
       pieces: Array(drop {
         if case .spaces = $0 { return false }
@@ -55,7 +55,7 @@ extension Trivia {
   }
 
   /// Returns this set of trivia, without any newlines.
-  public func withoutNewlines() -> Trivia {
+  func withoutNewlines() -> Trivia {
     return Trivia(
       pieces: filter {
         if case .newlines = $0 { return false }
@@ -66,7 +66,7 @@ extension Trivia {
   /// Returns this trivia, excluding the last newline and anything following it.
   ///
   /// If there is no newline in the trivia, it is returned unmodified.
-  public func withoutLastLine() -> Trivia {
+  func withoutLastLine() -> Trivia {
     var maybeLastNewlineOffset: Int? = nil
     for (offset, piece) in self.enumerated() {
       switch piece {
@@ -82,30 +82,30 @@ extension Trivia {
 
   /// Returns this set of trivia, with all spaces removed except for one at the
   /// end.
-  public func withOneTrailingSpace() -> Trivia {
+  func withOneTrailingSpace() -> Trivia {
     return withoutSpaces() + .spaces(1)
   }
 
   /// Returns this set of trivia, with all spaces removed except for one at the
   /// beginning.
-  public func withOneLeadingSpace() -> Trivia {
+  func withOneLeadingSpace() -> Trivia {
     return .spaces(1) + withoutSpaces()
   }
 
   /// Returns this set of trivia, with all newlines removed except for one.
-  public func withOneLeadingNewline() -> Trivia {
+  func withOneLeadingNewline() -> Trivia {
     return .newlines(1) + withoutNewlines()
   }
 
   /// Returns this set of trivia, with all newlines removed except for one.
-  public func withOneTrailingNewline() -> Trivia {
+  func withOneTrailingNewline() -> Trivia {
     return withoutNewlines() + .newlines(1)
   }
 
   /// Walks through trivia looking for multiple separate trivia entities with
   /// the same base kind, and condenses them.
   /// `[.spaces(1), .spaces(2)]` becomes `[.spaces(3)]`.
-  public func condensed() -> Trivia {
+  func condensed() -> Trivia {
     guard var prev = first else { return self }
     var pieces = [TriviaPiece]()
     for piece in dropFirst() {
@@ -136,7 +136,7 @@ extension Trivia {
   }
 
   /// Returns `true` if this trivia contains any newlines.
-  public var containsNewlines: Bool {
+  var containsNewlines: Bool {
     return contains(
       where: {
         if case .newlines = $0 { return true }
@@ -145,7 +145,7 @@ extension Trivia {
   }
 
   /// Returns `true` if this trivia contains any spaces.
-  public var containsSpaces: Bool {
+  var containsSpaces: Bool {
     return contains(
       where: {
         if case .spaces = $0 { return true }
@@ -156,7 +156,7 @@ extension Trivia {
 
   /// Returns `true` if this trivia contains any backslahes (used for multiline string newline
   /// suppression).
-  public var containsBackslashes: Bool {
+  var containsBackslashes: Bool {
     return contains(
       where: {
         if case .backslashes = $0 { return true }