Assure trait-guarded dependencies are not included in resolution; precompute enabled traits before resolution

Sources/Commands/PackageCommands/APIDiff.swift

@@ -72,9 +72,6 @@ struct APIDiff: AsyncSwiftCommand {
             help: "One or more targets to include in the API comparison. If present, only the specified targets (and any products specified using `--products`) will be compared.")
     var targets: [String] = []
 
-    @OptionGroup(visibility: .hidden)
-    package var traits: TraitOptions
-
     @Option(name: .customLong("baseline-dir"),
             help: "The path to a directory used to store API baseline files. If unspecified, a temporary directory will be used.")
     var overrideBaselineDir: Basics.AbsolutePath?
@@ -106,7 +103,6 @@ struct APIDiff: AsyncSwiftCommand {
             )
         } else {
             let buildSystem = try await swiftCommandState.createBuildSystem(
-                traitConfiguration: .init(traitOptions: self.traits),
                 cacheBuildManifest: false,
             )
             try await runWithSwiftPMCoordinatedDiffing(
@@ -207,7 +203,6 @@ struct APIDiff: AsyncSwiftCommand {
         )
         let delegate = DiagnosticsCapturingBuildSystemDelegate()
         let buildSystem = try await swiftCommandState.createBuildSystem(
-            traitConfiguration: .init(traitOptions: self.traits),
             cacheBuildManifest: false,
             productsBuildParameters: productsBuildParameters,
             delegate: delegate
@@ -316,7 +311,6 @@ struct APIDiff: AsyncSwiftCommand {
         // Build the baseline module.
         // FIXME: We need to implement the build tool invocation closure here so that build tool plugins work with the APIDigester. rdar://86112934
         let buildSystem = try await swiftCommandState.createBuildSystem(
-            traitConfiguration: .init(),
             cacheBuildManifest: false,
             productsBuildParameters: productsBuildParameters,
             packageGraphLoader: { graph }

Sources/Commands/PackageCommands/DumpCommands.swift

@@ -54,7 +54,7 @@ struct DumpSymbolGraph: AsyncSwiftCommand {
         let buildSystem = try await swiftCommandState.createBuildSystem(
             explicitBuildSystem: .native,
             // We are enabling all traits for dumping the symbol graph.
-            traitConfiguration: .init(enableAllTraits: true),
+            enableAllTraits: true,
             cacheBuildManifest: false
         )
         try await buildSystem.build()

Sources/Commands/PackageCommands/Install.swift

@@ -88,7 +88,7 @@ extension SwiftPackageCommand {
                 commandState.preferredBuildConfiguration = .release
             }
 
-            try await commandState.createBuildSystem(explicitProduct: productToInstall.name, traitConfiguration: .init())
+            try await commandState.createBuildSystem(explicitProduct: productToInstall.name)
                 .build(subset: .product(productToInstall.name))
 
             let binPath = try commandState.productsBuildParameters.buildPath.appending(component: productToInstall.name)

Sources/Commands/PackageCommands/Migrate.swift

@@ -215,7 +215,6 @@ extension SwiftPackageCommand {
             }
 
             return try await swiftCommandState.createBuildSystem(
-                traitConfiguration: .init(),
                 // Don't attempt to cache manifests with temporary
                 // feature flags added just for migration purposes.
                 cacheBuildManifest: false,

Sources/Commands/PackageCommands/PluginCommand.swift

@@ -338,7 +338,6 @@ struct PluginCommand: AsyncSwiftCommand {
         // Build or bring up-to-date any executable host-side tools on which this plugin depends. Add them and any binary dependencies to the tool-names-to-path map.
         let buildSystem = try await swiftCommandState.createBuildSystem(
             explicitBuildSystem: buildSystemKind,
-            traitConfiguration: .init(),
             cacheBuildManifest: false,
             productsBuildParameters: swiftCommandState.productsBuildParameters,
             toolsBuildParameters: buildParameters,

Sources/Commands/PackageCommands/Resolve.swift

@@ -31,10 +31,6 @@ extension SwiftPackageCommand {
 
         @Argument(help: "The name of the package to resolve.")
         var packageName: String?
-
-        /// Specifies the traits to build.
-        @OptionGroup(visibility: .hidden)
-        package var traits: TraitOptions
     }
 
     struct Resolve: AsyncSwiftCommand {
@@ -50,10 +46,10 @@ extension SwiftPackageCommand {
         func run(_ swiftCommandState: SwiftCommandState) async throws {
             // If a package is provided, use that to resolve the dependencies.
             if let packageName = resolveOptions.packageName {
-                let workspace = try swiftCommandState.getActiveWorkspace(traitConfiguration: .init(traitOptions: resolveOptions.traits))
+                let workspace = try swiftCommandState.getActiveWorkspace()
                 try await workspace.resolve(
                     packageName: packageName,
-                    root: swiftCommandState.getWorkspaceRoot(traitConfiguration: .init(traitOptions: resolveOptions.traits)),
+                    root: swiftCommandState.getWorkspaceRoot(),
                     version: resolveOptions.version,
                     branch: resolveOptions.branch,
                     revision: resolveOptions.revision,
@@ -64,7 +60,7 @@ extension SwiftPackageCommand {
                 }
             } else {
                 // Otherwise, run a normal resolve.
-                try await swiftCommandState.resolve(.init(traitOptions: resolveOptions.traits))
+                try await swiftCommandState.resolve()
             }
         }
     }

Sources/Commands/Snippets/Cards/SnippetCard.swift

@@ -112,7 +112,7 @@ struct SnippetCard: Card {
 
     func runExample() async throws {
         print("Building '\(snippet.path)'\n")
-        let buildSystem = try await swiftCommandState.createBuildSystem(explicitProduct: snippet.name, traitConfiguration: .init())
+        let buildSystem = try await swiftCommandState.createBuildSystem(explicitProduct: snippet.name)
         try await buildSystem.build(subset: .product(snippet.name))
         let executablePath = try swiftCommandState.productsBuildParameters.buildPath.appending(component: snippet.name)
         if let exampleTarget = try await buildSystem.getPackageGraph().module(for: snippet.name) {

Sources/Commands/SwiftBuildCommand.swift

@@ -110,10 +110,6 @@ struct BuildCommandOptions: ParsableArguments {
     @OptionGroup(visibility: .private)
     var testLibraryOptions: TestLibraryOptions
 
-    /// Specifies the traits to build.
-    @OptionGroup(visibility: .hidden)
-    package var traits: TraitOptions
-
     /// If should link the Swift stdlib statically.
     @Flag(name: .customLong("static-swift-stdlib"), inversion: .prefixedNo, help: "Link Swift stdlib statically.")
     public var shouldLinkStaticSwiftStdlib: Bool = false
@@ -144,7 +140,6 @@ public struct SwiftBuildCommand: AsyncSwiftCommand {
             // FIXME: Doesn't seem ideal that we need an explicit build operation, but this concretely uses the `LLBuildManifest`.
             guard let buildOperation = try await swiftCommandState.createBuildSystem(
                 explicitBuildSystem: .native,
-                traitConfiguration: .init(traitOptions: self.options.traits)
             ) as? BuildOperation else {
                 throw StringError("asked for native build system but did not get it")
             }
@@ -194,7 +189,6 @@ public struct SwiftBuildCommand: AsyncSwiftCommand {
     ) async throws {
         let buildSystem = try await swiftCommandState.createBuildSystem(
             explicitProduct: options.product,
-            traitConfiguration: .init(traitOptions: self.options.traits),
             shouldLinkStaticSwiftStdlib: options.shouldLinkStaticSwiftStdlib,
             productsBuildParameters: productsBuildParameters,
             toolsBuildParameters: toolsBuildParameters,

Sources/Commands/SwiftRunCommand.swift

@@ -88,10 +88,6 @@ struct RunCommandOptions: ParsableArguments {
     @Argument(help: "The executable to run.", completion: .shellCommand("swift package completion-tool list-executables"))
     var executable: String?
 
-    /// Specifies the traits to build the product with.
-    @OptionGroup(visibility: .hidden)
-    package var traits: TraitOptions
-
     /// The arguments to pass to the executable.
     @Argument(parsing: .captureForPassthrough,
               help: "The arguments to pass to the executable.")
@@ -139,7 +135,6 @@ public struct SwiftRunCommand: AsyncSwiftCommand {
             // FIXME: We need to implement the build tool invocation closure here so that build tool plugins work with the REPL. rdar://86112934
             let buildSystem = try await swiftCommandState.createBuildSystem(
                 explicitBuildSystem: .native,
-                traitConfiguration: .init(traitOptions: self.options.traits),
                 cacheBuildManifest: false,
                 packageGraphLoader: asyncUnsafeGraphLoader
             )
@@ -161,7 +156,6 @@ public struct SwiftRunCommand: AsyncSwiftCommand {
             do {
                 let buildSystem = try await swiftCommandState.createBuildSystem(
                     explicitProduct: options.executable,
-                    traitConfiguration: .init(traitOptions: self.options.traits)
                 )
                 let productName = try await findProductName(in: buildSystem.getPackageGraph())
                 if options.shouldBuildTests {
@@ -217,9 +211,9 @@ public struct SwiftRunCommand: AsyncSwiftCommand {
             do {
                 let buildSystem = try await swiftCommandState.createBuildSystem(
                     explicitProduct: options.executable,
-                    traitConfiguration: .init(traitOptions: self.options.traits)
                 )
-                let productName = try await findProductName(in: buildSystem.getPackageGraph())
+                let modulesGraph = try await buildSystem.getPackageGraph()
+                let productName = try findProductName(in: modulesGraph)
                 if options.shouldBuildTests {
                     try await buildSystem.build(subset: .allIncludingTests)
                 } else if options.shouldBuild {

Sources/Commands/SwiftTestCommand.swift

@@ -220,9 +220,6 @@ struct TestCommandOptions: ParsableArguments {
     var enableExperimentalTestOutput: Bool {
         return testOutput == .experimentalSummary
     }
-
-    @OptionGroup(visibility: .hidden)
-    package var traits: TraitOptions
 }
 
 /// Tests filtering specifier, which is used to filter tests to run.
@@ -659,7 +656,7 @@ public struct SwiftTestCommand: AsyncSwiftCommand {
             productsBuildParameters: productsBuildParameters,
             toolsBuildParameters: toolsBuildParameters,
             testProduct: self.options.sharedOptions.testProduct,
-            traitConfiguration: .init(traitOptions: self.options.traits)
+            traitConfiguration: .init(traitOptions: self.globalOptions.traits)
         )
     }
 
@@ -741,9 +738,6 @@ extension SwiftTestCommand {
         @OptionGroup()
         var testEventStreamOptions: TestEventStreamOptions
 
-        @OptionGroup(visibility: .hidden)
-        package var traits: TraitOptions
-
         // for deprecated passthrough from SwiftTestTool (parse will fail otherwise)
         @Flag(name: [.customLong("list-tests"), .customShort("l")], help: .hidden)
         var _deprecated_passthrough: Bool = false
@@ -850,7 +844,7 @@ extension SwiftTestCommand {
                 productsBuildParameters: productsBuildParameters,
                 toolsBuildParameters: toolsBuildParameters,
                 testProduct: self.sharedOptions.testProduct,
-                traitConfiguration: .init(traitOptions: self.traits)
+                traitConfiguration: .init(traitOptions: self.globalOptions.traits)
             )
         }
     }
@@ -1561,7 +1555,6 @@ private func buildTestsIfNeeded(
     traitConfiguration: TraitConfiguration
 ) async throws -> [BuiltTestProduct] {
     let buildSystem = try await swiftCommandState.createBuildSystem(
-        traitConfiguration: traitConfiguration,
         productsBuildParameters: productsBuildParameters,
         toolsBuildParameters: toolsBuildParameters
     )

Sources/Commands/Utilities/APIDigester.swift

@@ -140,7 +140,6 @@ struct APIDigesterBaselineDumper {
         // FIXME: We need to implement the build tool invocation closure here so that build tool plugins work with the APIDigester. rdar://86112934
         let buildSystem = try await swiftCommandState.createBuildSystem(
             explicitBuildSystem: .native,
-            traitConfiguration: .init(),
             cacheBuildManifest: false,
             productsBuildParameters: productsBuildParameters,
             toolsBuildParameters: toolsBuildParameters,

Sources/Commands/Utilities/PluginDelegate.swift

@@ -171,7 +171,6 @@ final class PluginDelegate: PluginInvocationDelegate {
         let buildSystem = try await swiftCommandState.createBuildSystem(
             explicitBuildSystem: buildSystem,
             explicitProduct: explicitProduct,
-            traitConfiguration: .init(),
             cacheBuildManifest: false,
             productsBuildParameters: buildParameters,
             outputStream: outputStream,
@@ -181,6 +180,7 @@ final class PluginDelegate: PluginInvocationDelegate {
         // Run the build. This doesn't return until the build is complete.
         let success = await buildSystem.buildIgnoringError(subset: buildSubset)
 
+        swiftCommandState.observabilityScope.emit(warning: "plugin delegate getting package graph")
         let packageGraph = try await buildSystem.getPackageGraph()
 
         var builtArtifacts: [PluginInvocationBuildResult.BuiltArtifact] = []
@@ -246,7 +246,6 @@ final class PluginDelegate: PluginInvocationDelegate {
         toolsBuildParameters.testingParameters.explicitlyEnabledTestability = true
         toolsBuildParameters.testingParameters.enableCodeCoverage = parameters.enableCodeCoverage
         let buildSystem = try await swiftCommandState.createBuildSystem(
-            traitConfiguration: .init(),
             toolsBuildParameters: toolsBuildParameters
         )
         try await buildSystem.build(subset: .allIncludingTests)
@@ -410,7 +409,7 @@ final class PluginDelegate: PluginInvocationDelegate {
         // Create a build system for building the target., skipping the the cache because we need the build plan.
         let buildSystem = try await swiftCommandState.createBuildSystem(
             explicitBuildSystem: buildSystem,
-            traitConfiguration: TraitConfiguration(enableAllTraits: true),
+            enableAllTraits: true,
             cacheBuildManifest: false
         )
 

Sources/CoreCommands/BuildSystemSupport.swift

@@ -29,7 +29,7 @@ private struct NativeBuildSystemFactory: BuildSystemFactory {
 
     func makeBuildSystem(
         explicitProduct: String?,
-        traitConfiguration: TraitConfiguration,
+        enableAllTraits: Bool,
         cacheBuildManifest: Bool,
         productsBuildParameters: BuildParameters?,
         toolsBuildParameters: BuildParameters?,
@@ -39,7 +39,7 @@ private struct NativeBuildSystemFactory: BuildSystemFactory {
         observabilityScope: ObservabilityScope?,
         delegate: BuildSystemDelegate?
     ) async throws -> any BuildSystem {
-        _ = try await swiftCommandState.getRootPackageInformation(traitConfiguration: traitConfiguration)
+        _ = try await swiftCommandState.getRootPackageInformation(enableAllTraits)
         let testEntryPointPath = productsBuildParameters?.testProductStyle.explicitlySpecifiedEntryPointPath
         let cacheBuildManifest = if cacheBuildManifest {
             try await self.swiftCommandState.canUseCachedBuildManifest()
@@ -53,7 +53,7 @@ private struct NativeBuildSystemFactory: BuildSystemFactory {
             packageGraphLoader: packageGraphLoader ?? {
                 try await self.swiftCommandState.loadPackageGraph(
                     explicitProduct: explicitProduct,
-                    traitConfiguration: traitConfiguration,
+                    enableAllTraits: enableAllTraits,
                     testEntryPointPath: testEntryPointPath
                 )
             },
@@ -63,7 +63,7 @@ private struct NativeBuildSystemFactory: BuildSystemFactory {
                 disableSandbox: self.swiftCommandState.shouldDisableSandbox
             ),
             scratchDirectory: self.swiftCommandState.scratchDirectory,
-            traitConfiguration: traitConfiguration,
+            traitConfiguration: enableAllTraits ? .enableAllTraits : self.swiftCommandState.traitConfiguration,
             additionalFileRules: FileRuleDescription.swiftpmFileTypes,
             pkgConfigDirectories: self.swiftCommandState.options.locations.pkgConfigDirectories,
             outputStream: outputStream ?? self.swiftCommandState.outputStream,
@@ -79,7 +79,7 @@ private struct XcodeBuildSystemFactory: BuildSystemFactory {
 
     func makeBuildSystem(
         explicitProduct: String?,
-        traitConfiguration: TraitConfiguration,
+        enableAllTraits: Bool,
         cacheBuildManifest: Bool,
         productsBuildParameters: BuildParameters?,
         toolsBuildParameters: BuildParameters?,
@@ -94,7 +94,7 @@ private struct XcodeBuildSystemFactory: BuildSystemFactory {
             packageGraphLoader: packageGraphLoader ?? {
                 try await self.swiftCommandState.loadPackageGraph(
                     explicitProduct: explicitProduct,
-                    traitConfiguration: traitConfiguration
+                    enableAllTraits: enableAllTraits
                 )
             },
             outputStream: outputStream ?? self.swiftCommandState.outputStream,
@@ -111,7 +111,7 @@ private struct SwiftBuildSystemFactory: BuildSystemFactory {
 
     func makeBuildSystem(
         explicitProduct: String?,
-        traitConfiguration: TraitConfiguration,
+        enableAllTraits: Bool,
         cacheBuildManifest: Bool,
         productsBuildParameters: BuildParameters?,
         toolsBuildParameters: BuildParameters?,
@@ -126,7 +126,7 @@ private struct SwiftBuildSystemFactory: BuildSystemFactory {
             packageGraphLoader: packageGraphLoader ?? {
                 try await self.swiftCommandState.loadPackageGraph(
                     explicitProduct: explicitProduct,
-                    traitConfiguration: traitConfiguration,
+                    enableAllTraits: enableAllTraits,
                 )
             },
             packageManagerResourcesDirectory: swiftCommandState.packageManagerResourcesDirectory,

Sources/CoreCommands/Options.swift

@@ -63,6 +63,9 @@ public struct GlobalOptions: ParsableArguments {
 
     @OptionGroup(title: "Build Options")
     public var linker: LinkerOptions
+
+    @OptionGroup(title: "Trait Options")
+    public var traits: TraitOptions
 }
 
 public struct LocationOptions: ParsableArguments {
@@ -683,8 +686,8 @@ public struct TestLibraryOptions: ParsableArguments {
     }
 }
 
-package struct TraitOptions: ParsableArguments {
-    package init() {}
+public struct TraitOptions: ParsableArguments {
+    public init() {}
 
     /// The traits to enable for the package.
     @Option(
@@ -694,7 +697,7 @@ package struct TraitOptions: ParsableArguments {
     package var _enabledTraits: String?
 
     /// The set of enabled traits for the package.
-    package var enabledTraits: Set<String>? {
+    public var enabledTraits: Set<String>? {
         self._enabledTraits.flatMap { Set($0.components(separatedBy: ",")) }
     }
 
@@ -703,7 +706,7 @@ package struct TraitOptions: ParsableArguments {
         name: .customLong("enable-all-traits"),
         help: "Enables all traits of the package."
     )
-    package var enableAllTraits: Bool = false
+    public var enableAllTraits: Bool = false
 
     /// Disables all default traits of the package.
     @Flag(
@@ -714,7 +717,7 @@ package struct TraitOptions: ParsableArguments {
 }
 
 extension TraitConfiguration {
-    package init(traitOptions: TraitOptions) {
+    public init(traitOptions: TraitOptions) {
         var enabledTraits = traitOptions.enabledTraits
         if traitOptions.disableDefaultTraits {
             // If there are no enabled traits specified we can disable the