Hide CLI commands API with package access control (#7381)

Known clients of SwiftPM have not adopted these CLI APIs that were
previously marked as `public`. To prevent accidental adoption by clients
in the future, we should use `package` access control instead.

Also replaced remaining usages of `@_spi(SwiftPMInternal)` with
`package` across all of our codebase.

Author: MaxDesiatov

Sources/Basics/ProgressAnimation/NinjaProgressAnimation.swift

@@ -15,8 +15,7 @@ import protocol TSCBasic.WritableByteStream
 
 extension ProgressAnimation {
     /// A ninja-like progress animation that adapts to the provided output stream.
-    @_spi(SwiftPMInternal)
-    public static func ninja(
+    package static func ninja(
         stream: WritableByteStream,
         verbose: Bool
     ) -> any ProgressAnimationProtocol {

Sources/Basics/ProgressAnimation/PercentProgressAnimation.swift

@@ -15,8 +15,7 @@ import protocol TSCBasic.WritableByteStream
 
 extension ProgressAnimation {
     /// A percent-based progress animation that adapts to the provided output stream.
-    @_spi(SwiftPMInternal)
-    public static func percent(
+    package static func percent(
         stream: WritableByteStream,
         verbose: Bool,
         header: String

Sources/Basics/ProgressAnimation/ProgressAnimationProtocol.swift

@@ -15,12 +15,10 @@ import class TSCBasic.LocalFileOutputByteStream
 import protocol TSCBasic.WritableByteStream
 import protocol TSCUtility.ProgressAnimationProtocol
 
-@_spi(SwiftPMInternal)
-public typealias ProgressAnimationProtocol = TSCUtility.ProgressAnimationProtocol
+package typealias ProgressAnimationProtocol = TSCUtility.ProgressAnimationProtocol
 
 /// Namespace to nest public progress animations under.
-@_spi(SwiftPMInternal)
-public enum ProgressAnimation {
+package enum ProgressAnimation {
     /// Dynamically create a progress animation based on the current stream
     /// capabilities and desired verbosity.
     ///

Sources/Basics/ProgressAnimation/ThrottledProgressAnimation.swift

@@ -56,27 +56,23 @@ final class ThrottledProgressAnimation: ProgressAnimationProtocol {
     }
 }
 
-@_spi(SwiftPMInternal)
 extension ProgressAnimationProtocol {
-    @_spi(SwiftPMInternal)
-    public func throttled<C: Clock>(
+    package func throttled<C: Clock>(
         now: @escaping () -> C.Instant,
         interval: C.Duration,
         clock: C.Type = C.self
     ) -> some ProgressAnimationProtocol {
         ThrottledProgressAnimation(self, now: now, interval: interval, clock: clock)
     }
 
-    @_spi(SwiftPMInternal)
-    public func throttled<C: Clock>(
+    package func throttled<C: Clock>(
         clock: C,
         interval: C.Duration
     ) -> some ProgressAnimationProtocol {
         self.throttled(now: { clock.now }, interval: interval, clock: C.self)
     }
 
-    @_spi(SwiftPMInternal)
-    public func throttled(
+    package func throttled(
         interval: ContinuousClock.Duration
     )  -> some ProgressAnimationProtocol  {
         self.throttled(clock: ContinuousClock(), interval: interval)

Sources/Build/BuildDescription/ProductBuildDescription.swift

@@ -13,7 +13,6 @@
 import Basics
 import PackageGraph
 
-@_spi(SwiftPMInternal)
 import PackageModel
 
 import OrderedCollections

Sources/Build/BuildDescription/SwiftTargetBuildDescription.swift

@@ -17,7 +17,6 @@ import PackageGraph
 import PackageLoading
 import PackageModel
 
-@_spi(SwiftPMInternal)
 import SPMBuildCore
 
 #if USE_IMPL_ONLY_IMPORTS

Sources/Build/BuildOperation.swift

@@ -10,18 +10,15 @@
 //
 //===----------------------------------------------------------------------===//
 
-@_spi(SwiftPMInternal)
 import Basics
 
-@_spi(SwiftPMInternal)
 import Build
 
 import LLBuildManifest
 import PackageGraph
 import PackageLoading
 import PackageModel
 
-@_spi(SwiftPMInternal)
 import SPMBuildCore
 
 import SPMLLBuild
@@ -43,8 +40,7 @@ import DriverSupport
 import SwiftDriver
 #endif
 
-@_spi(SwiftPMInternal)
-public final class BuildOperation: PackageStructureDelegate, SPMBuildCore.BuildSystem, BuildErrorAdviceProvider {
+package final class BuildOperation: PackageStructureDelegate, SPMBuildCore.BuildSystem, BuildErrorAdviceProvider {
     /// The delegate used by the build system.
     public weak var delegate: SPMBuildCore.BuildSystemDelegate?
 

Sources/Build/BuildOperationBuildSystemDelegateHandler.swift

@@ -10,14 +10,12 @@
 //
 //===----------------------------------------------------------------------===//
 
-@_spi(SwiftPMInternal)
 import Basics
 import Dispatch
 import Foundation
 import LLBuildManifest
 import PackageModel
 
-@_spi(SwiftPMInternal)
 import SPMBuildCore
 
 import SPMLLBuild

Sources/Build/BuildPlan/BuildPlan+Product.swift

@@ -18,7 +18,6 @@ import struct PackageGraph.ResolvedTarget
 import class PackageModel.BinaryTarget
 import class PackageModel.ClangTarget
 
-@_spi(SwiftPMInternal)
 import class PackageModel.Target
 
 import class PackageModel.SwiftTarget

Sources/Build/BuildPlan/BuildPlan.swift

@@ -225,8 +225,7 @@ public class BuildPlan: SPMBuildCore.BuildPlan {
     /// source files as well as directories to which any changes should cause us to reevaluate the build plan.
     public let prebuildCommandResults: [ResolvedTarget.ID: [PrebuildCommandResult]]
 
-    @_spi(SwiftPMInternal)
-    public private(set) var derivedTestTargetsMap: [ResolvedProduct.ID: [ResolvedTarget]] = [:]
+    package private(set) var derivedTestTargetsMap: [ResolvedProduct.ID: [ResolvedTarget]] = [:]
 
     /// Cache for pkgConfig flags.
     private var pkgConfigCache = [SystemLibraryTarget: (cFlags: [String], libs: [String])]()