Fix possible concurrent mutation crash

Author: ileitch

.github/workflows/test.yml

@@ -9,7 +9,7 @@ env:
   swift_package_resolve: swift package resolve
   swift_build: swift build
   swift_test: swift test
-  periphery_scan: ./.build/debug/periphery scan --quiet --clean-build --strict
+  periphery_scan: ./.build/debug/periphery scan --quiet --clean-build --strict --format csv
   cache_version: 1
 jobs:
   lint:

CHANGELOG.md

@@ -25,6 +25,7 @@
 - Fix inconsistent unused parameter results when the function is declared in a file that is a member of multiple targets.
 - Fix inconsistent results caused by identical extensions declared in different modules.
 - Static `@_dynamicReplacement` members are now retained.
+- Fix possible concurrent mutation crash when accessing filename matching configuration options.
 
 ## 3.1.0 (2025-04-05)
 

Package.swift

@@ -120,6 +120,7 @@ var targets: [PackageDescription.Target] = [
         dependencies: [
             .target(name: "PeripheryKit"),
             .target(name: "ProjectDrivers"),
+            .target(name: "Configuration"),
         ],
         path: "Tests/Shared"
     ),

Sources/Configuration/Configuration.swift

@@ -172,10 +172,6 @@ public final class Configuration {
         }
     }
 
-    public func reset() {
-        settings.forEach { $0.reset() }
-    }
-
     // MARK: - Helpers
 
     public func apply<T: Equatable>(_ path: KeyPath<Configuration, Setting<T>>, _ value: T) {
@@ -186,35 +182,9 @@ public final class Configuration {
         }
     }
 
-    private var _indexExcludeMatchers: [FilenameMatcher]?
-    public var indexExcludeMatchers: [FilenameMatcher] {
-        if let _indexExcludeMatchers {
-            return _indexExcludeMatchers
-        }
-
-        let matchers = buildFilenameMatchers(with: indexExclude)
-        _indexExcludeMatchers = matchers
-        return matchers
-    }
-
-    private var _retainFilesMatchers: [FilenameMatcher]?
-    public var retainFilesMatchers: [FilenameMatcher] {
-        if let _retainFilesMatchers {
-            return _retainFilesMatchers
-        }
-
-        let matchers = buildFilenameMatchers(with: retainFiles)
-        _retainFilesMatchers = matchers
-        return matchers
-    }
-
-    public func resetMatchers() {
-        _indexExcludeMatchers = nil
-        _retainFilesMatchers = nil
-    }
-
+    public lazy var indexExcludeMatchers: [FilenameMatcher] = buildFilenameMatchers(with: indexExclude)
+    public lazy var retainFilesMatchers: [FilenameMatcher] = buildFilenameMatchers(with: retainFiles)
     public lazy var reportExcludeMatchers: [FilenameMatcher] = buildFilenameMatchers(with: reportExclude)
-
     public lazy var reportIncludeMatchers: [FilenameMatcher] = buildFilenameMatchers(with: reportInclude)
 
     // MARK: - Private
@@ -255,7 +225,6 @@ protocol AbstractSetting {
     var hasNonDefaultValue: Bool { get }
     var wrappedValue: Value { get }
 
-    func reset()
     func assign(_ value: Any)
 }
 
@@ -293,10 +262,6 @@ protocol AbstractSetting {
     public func assign(_ newValue: Any) {
         value = setter(newValue) ?? defaultValue
     }
-
-    func reset() {
-        wrappedValue = defaultValue
-    }
 }
 
 private let filePathSetter: (Any) -> FilePath? = { value in