Enable strict concurrency (#14)

# Enable strict concurrency

## ♻️ Current situation & Problem
This PR enables strict concurrency for the project and fixes resulting
concurrency warnings.


## ⚙️ Release Notes 
* Comply with strict concurrency


## 📚 Documentation
--


## ✅ Testing
--

### Code of Conduct & Contributing Guidelines 

By submitting creating this pull request, you agree to follow our [Code
of
Conduct](https://github.com/StanfordBDHG/.github/blob/main/CODE_OF_CONDUCT.md)
and [Contributing
Guidelines](https://github.com/StanfordBDHG/.github/blob/main/CONTRIBUTING.md):
- [x] I agree to follow the [Code of
Conduct](https://github.com/StanfordBDHG/.github/blob/main/CODE_OF_CONDUCT.md)
and [Contributing
Guidelines](https://github.com/StanfordBDHG/.github/blob/main/CONTRIBUTING.md).

---------

Co-authored-by: Paul Schmiedmayer <[email protected]>

.github/workflows/build-and-test.yml

@@ -66,3 +66,5 @@ jobs:
     uses: StanfordBDHG/.github/.github/workflows/create-and-upload-coverage-report.yml@v2
     with:
       coveragereports: XCTRuntimeAssertions-iOS.xcresult XCTRuntimeAssertions-watchOS.xcresult XCTRuntimeAssertions-visionOS.xcresult XCTRuntimeAssertions-tvOS.xcresult XCTRuntimeAssertions-macOS.xcresult
+    secrets:
+      token: ${{ secrets.CODECOV_TOKEN }}

.swiftlint.yml

@@ -141,8 +141,6 @@ only_rules:
   - implicitly_unwrapped_optional
   # Identifiers should use inclusive language that avoids discrimination against groups of people based on race, gender, or socioeconomic status
   - inclusive_language
-  # If defer is at the end of its parent scope, it will be executed right where it is anyway.
-  - inert_defer
   # Prefer using Set.isDisjoint(with:) over Set.intersection(_:).isEmpty.
   - is_disjoint
   # Discouraged explicit usage of the default separator.
@@ -329,8 +327,6 @@ only_rules:
   - unowned_variable_capture
   # Catch statements should not declare error variables without type casting.
   - untyped_error_in_catch
-  # Unused reference in a capture list should be removed.
-  - unused_capture_list
   # Unused parameter in a closure should be replaced with _.
   - unused_closure_parameter
   # Unused control flow label should be removed.

Package.swift

@@ -8,8 +8,15 @@
 // SPDX-License-Identifier: MIT
 //
 
+import class Foundation.ProcessInfo
 import PackageDescription
 
+#if swift(<6)
+let swiftConcurrency: SwiftSetting = .enableExperimentalFeature("StrictConcurrency")
+#else
+let swiftConcurrency: SwiftSetting = .enableUpcomingFeature("StrictConcurrency")
+#endif
+
 
 let package = Package(
     name: "XCTRuntimeAssertions",
@@ -23,15 +30,41 @@ let package = Package(
     products: [
         .library(name: "XCTRuntimeAssertions", targets: ["XCTRuntimeAssertions"])
     ],
+    dependencies: swiftLintPackage(),
     targets: [
         .target(
-            name: "XCTRuntimeAssertions"
+            name: "XCTRuntimeAssertions",
+            swiftSettings: [
+                swiftConcurrency
+            ],
+            plugins: [] + swiftLintPlugin()
         ),
         .testTarget(
             name: "XCTRuntimeAssertionsTests",
             dependencies: [
                 .target(name: "XCTRuntimeAssertions")
-            ]
+            ],
+            swiftSettings: [
+                swiftConcurrency
+            ],
+            plugins: [] + swiftLintPlugin()
         )
     ]
 )
+
+func swiftLintPlugin() -> [Target.PluginUsage] {
+    // Fully quit Xcode and open again with `open --env SPEZI_DEVELOPMENT_SWIFTLINT /Applications/Xcode.app`
+    if ProcessInfo.processInfo.environment["SPEZI_DEVELOPMENT_SWIFTLINT"] != nil {
+        [.plugin(name: "SwiftLintBuildToolPlugin", package: "SwiftLint")]
+    } else {
+        []
+    }
+}
+
+func swiftLintPackage() -> [PackageDescription.Package.Dependency] {
+    if ProcessInfo.processInfo.environment["SPEZI_DEVELOPMENT_SWIFTLINT"] != nil {
+        [.package(url: "https://github.com/realm/SwiftLint.git", .upToNextMinor(from: "0.55.1"))]
+    } else {
+        []
+    }
+}

Sources/XCTRuntimeAssertions/XCTRuntimeAssertionInjector.swift

@@ -10,9 +10,39 @@
 import Foundation
 
 
+private final class RuntimeInjections: Sendable {
+    private nonisolated(unsafe) var injected: [XCTRuntimeAssertionInjector] = []
+    private let lock = NSLock()
+
+    @inlinable var isEmpty: Bool {
+        injected.isEmpty
+    }
+
+    var injections: [XCTRuntimeAssertionInjector] {
+        lock.withLock {
+            injected
+        }
+    }
+
+    init() {}
+
+    func append(_ element: XCTRuntimeAssertionInjector) {
+        lock.withLock {
+            injected.append(element)
+        }
+    }
+
+    func removeAll(for id: UUID) {
+        lock.withLock {
+            injected.removeAll(where: { $0.id == id })
+        }
+    }
+}
+
+
 class XCTRuntimeAssertionInjector {
-    private static var injected: [XCTRuntimeAssertionInjector] = []
-    
+    private static let injection = RuntimeInjections()
+
 
     let id: UUID
     private let _assert: (UUID, () -> Bool, () -> String, StaticString, UInt) -> Void
@@ -65,30 +95,32 @@ class XCTRuntimeAssertionInjector {
 
 
     static func inject(runtimeAssertionInjector: XCTRuntimeAssertionInjector) {
-        injected.append(runtimeAssertionInjector)
+        injection.append(runtimeAssertionInjector)
     }
 
     static func removeRuntimeAssertionInjector(withId id: UUID) {
-        injected.removeAll(where: { $0.id == id })
+        injection.removeAll(for: id)
     }
 
-
+
+    @inlinable
     static func assert(_ condition: () -> Bool, message: () -> String, file: StaticString, line: UInt) {
-        if injected.isEmpty {
+        if injection.isEmpty {
             Swift.assert(condition(), message(), file: file, line: line)
         }
 
-        for runtimeAssertionInjector in injected {
+        for runtimeAssertionInjector in injection.injections {
             runtimeAssertionInjector._assert(runtimeAssertionInjector.id, condition, message, file, line)
         }
     }
-
+
+    @inlinable
     static func precondition(_ condition: () -> Bool, message: () -> String, file: StaticString, line: UInt) {
-        if injected.isEmpty {
+        if injection.isEmpty {
             Swift.precondition(condition(), message(), file: file, line: line)
         }
 
-        for runtimeAssertionInjector in injected {
+        for runtimeAssertionInjector in injection.injections {
             runtimeAssertionInjector._precondition(runtimeAssertionInjector.id, condition, message, file, line)
         }
     }

Sources/XCTRuntimeAssertions/XCTRuntimePrecondition.swift

@@ -26,7 +26,7 @@ public func XCTRuntimePrecondition(
     _ message: @autoclosure () -> String = "",
     file: StaticString = #filePath,
     line: UInt = #line,
-    _ expression: @escaping () -> Void
+    _ expression: @escaping @Sendable () -> Void
 ) throws {
     let fulfillmentCount = Counter()
     let xctRuntimeAssertionId = setupXCTRuntimeAssertionInjector(
@@ -74,7 +74,7 @@ public func XCTRuntimePrecondition(
     _ message: @autoclosure () -> String = "",
     file: StaticString = #filePath,
     line: UInt = #line,
-    _ expression: @escaping () async -> Void
+    _ expression: @escaping @Sendable () async -> Void
 ) throws {
     let fulfillmentCount = Counter()
     let xctRuntimeAssertionId = setupXCTRuntimeAssertionInjector(