NFC: Resolve some Sendable-related warnings (#7822)

Assorted fixes for sendability warnings.

Author: MaxDesiatov

Sources/Basics/Netrc.swift

@@ -13,7 +13,7 @@
 import Foundation
 
 /// Representation of Netrc configuration
-public struct Netrc {
+public struct Netrc: Sendable {
     /// Representation of `machine` connection settings & `default` connection settings.
     /// If `default` connection settings present, they will be last element.
     public let machines: [Machine]
@@ -37,7 +37,7 @@ public struct Netrc {
     }
 
     /// Representation of connection settings
-    public struct Machine: Equatable {
+    public struct Machine: Equatable, Sendable {
         public let name: String
         public let login: String
         public let password: String
@@ -147,7 +147,6 @@ public enum NetrcError: Error, Equatable {
 }
 
 private enum RegexUtil {
-    @frozen
     fileprivate enum Token: String, CaseIterable {
         case machine, login, password, account, macdef, `default`
 

Sources/_InternalTestSupport/MockArchiver.swift

@@ -13,19 +13,23 @@
 import Basics
 
 package final class MockArchiver: Archiver {
-    package typealias ExtractionHandler = (
+    package typealias ExtractionHandler = @Sendable (
         MockArchiver,
         AbsolutePath,
         AbsolutePath,
         (Result<Void, Error>) -> Void
     ) throws -> Void
-    package typealias CompressionHandler = (
+    package typealias CompressionHandler = @Sendable (
         MockArchiver,
         AbsolutePath,
         AbsolutePath,
         (Result<Void, Error>) -> Void
     ) throws -> Void
-    package typealias ValidationHandler = (MockArchiver, AbsolutePath, (Result<Bool, Error>) -> Void) throws -> Void
+    package typealias ValidationHandler = @Sendable (
+        MockArchiver,
+        AbsolutePath,
+        (Result<Bool, Error>) -> Void
+    ) throws -> Void
 
     package struct Extraction: Equatable {
         public let archivePath: AbsolutePath

Tests/BasicsTests/CancellatorTests.swift

@@ -348,6 +348,7 @@ final class CancellatorTests: XCTestCase {
         struct Worker {
             func work()  {}
 
+            @Sendable
             func cancel() {
                 Thread.sleep(forTimeInterval: 5)
             }
@@ -391,6 +392,7 @@ fileprivate struct Worker {
         return self.semaphore.wait(timeout: deadline)
     }
 
+    @Sendable
     func cancel() {
         print("\(self.name) cancel")
         self.semaphore.signal()

Tests/CommandsTests/PackageRegistryCommandTests.swift

@@ -716,6 +716,7 @@ final class PackageRegistryCommandTests: CommandsTestCase {
             )
         }
 
+        @Sendable
         func validateManifest(
             manifestFile: String,
             in archivePath: AbsolutePath,

Tests/CommandsTests/TestCommandTests.swift

@@ -191,7 +191,7 @@ final class TestCommandTests: CommandsTestCase {
         }
 
         try await fixture(name: "Miscellaneous/SkipTests") { fixturePath in
-            let (stdout, stderr) = try await SwiftPM.Test.execute(["--skip", "Tests"], packagePath: fixturePath)
+            let (stdout, _) = try await SwiftPM.Test.execute(["--skip", "Tests"], packagePath: fixturePath)
             // in "swift test" test output goes to stdout
             XCTAssertNoMatch(stdout, .contains("testExample1"))
             XCTAssertNoMatch(stdout, .contains("testExample2"))