Update precondition implementation

Author: Kyle-Ye

Package.swift

@@ -17,13 +17,21 @@ func envEnable(_ key: String, default defaultValue: Bool = false) -> Bool {
     }
 }
 
-let isXcodeEnv = Context.environment["__CFBundleIdentifier"] == "com.apple.dt.Xcode"
-let development = envEnable("OPENSWIFTUI_DEVELOPMENT", default: false)
-
-// Xcode use clang as linker which supports "-iframework" while SwiftPM use swiftc as linker which supports "-Fsystem"
-let systemFrameworkSearchFlag = isXcodeEnv ? "-iframework" : "-Fsystem"
+var sharedSwiftSettings: [SwiftSetting] = [
+    .enableUpcomingFeature("BareSlashRegexLiterals"),
+    .enableUpcomingFeature("InternalImportsByDefault"),
+    .define("OPENSWIFTUI_SUPPRESS_DEPRECATED_WARNINGS"),
+    .swiftLanguageMode(.v5),
+]
 
 let releaseVersion = Context.environment["OPENSWIFTUI_TARGET_RELEASE"].flatMap { Int($0) } ?? 2024
+sharedSwiftSettings.append(.define("OPENSWIFTUI_RELEASE_\(releaseVersion)"))
+if releaseVersion >= 2021 {
+    for year in 2021 ... releaseVersion {
+        sharedSwiftSettings.append(.define("OPENSWIFTUI_SUPPORT_\(year)_API"))
+    }
+}
+
 let platforms: [SupportedPlatform] = switch releaseVersion {
 case 2024: // iOS 18.0
     [
@@ -53,18 +61,15 @@ default:
     ]
 }
 
-var sharedSwiftSettings: [SwiftSetting] = [
-    .enableUpcomingFeature("BareSlashRegexLiterals"),
-    .enableUpcomingFeature("InternalImportsByDefault"),
-    .define("OPENSWIFTUI_SUPPRESS_DEPRECATED_WARNINGS"),
-    .define("OPENSWIFTUI_RELEASE_\(releaseVersion)"),
-    .swiftLanguageMode(.v5),
-]
+let isXcodeEnv = Context.environment["__CFBundleIdentifier"] == "com.apple.dt.Xcode"
 
-if releaseVersion >= 2021 {
-    for year in 2021 ... releaseVersion {
-        sharedSwiftSettings.append(.define("OPENSWIFTUI_SUPPORT_\(year)_API"))
-    }
+// Xcode use clang as linker which supports "-iframework" while SwiftPM use swiftc as linker which supports "-Fsystem"
+let systemFrameworkSearchFlag = isXcodeEnv ? "-iframework" : "-Fsystem"
+
+let development = envEnable("OPENSWIFTUI_DEVELOPMENT")
+
+if development {
+    sharedSwiftSettings.append(.define("OPENSWIFTUI_DEVELOPMENT"))
 }
 
 let warningsAsErrorsCondition = envEnable("OPENSWIFTUI_WERROR", default: isXcodeEnv && development)

Sources/OpenSwiftUI/Accessibility/Modifier/internal/AccessibilityConfigurationModifier.swift

@@ -15,5 +15,5 @@ protocol AccessibilityConfigurationModifier {
 }
 
 extension AccessibilityConfigurationModifier where Configuration == Never {
-    var configuration: Configuration { fatalError() }
+    var configuration: Configuration { preconditionFailure() }
 }

Sources/OpenSwiftUI/App/FinishLaunchTestAction.swift

@@ -17,7 +17,7 @@ struct FinishLaunchTestAction {
         #if os(iOS)
         UIApplication.shared.finishedTest(UIApplication.shared._launchTestName())
         #else
-        fatalError("Unimplemented for other platform")
+        preconditionFailure("Unimplemented for other platform")
         #endif
     }
 }

Sources/OpenSwiftUI/App/LegacyApp.swift

@@ -19,7 +19,7 @@ extension __App {
         var rootView: V
 
         init() {
-            fatalError("Not a standalone App.")
+            preconditionFailure("Not a standalone App.")
         }
 
         init(rootView: V) {

Sources/OpenSwiftUI/App/OpenSwiftUIApplication.swift

@@ -25,7 +25,7 @@ private final class OpenSwiftUIApplication: NSApplication {
     }
 
     required init?(coder: NSCoder) {
-        fatalError("init(coder:) has not been implemented")
+        preconditionFailure("init(coder:) has not been implemented")
     }
 }
 #else

Sources/OpenSwiftUI/Core/Modifier/SceneModifier/TODO/_SceneModifier.swift

@@ -22,6 +22,6 @@ extension _SceneModifier {
 extension _SceneModifier where Body == Never {
     @inline(__always)
     public func body(content _: SceneContent) -> Body {
-        fatalError("body() should not be called on \(Self.self)")
+        preconditionFailure("body() should not be called on \(Self.self)")
     }
 }

Sources/OpenSwiftUI/Core/Render/DisplayLink.swift

@@ -28,11 +28,11 @@ final class DisplayLink: NSObject {
 
     #if os(iOS)
     init(host: AnyUIHostingView, window: UIWindow) {
-        fatalError("TODO")
+        preconditionFailure("TODO")
     }
     #elseif os(macOS)
     init(host: AnyUIHostingView, window: NSWindow) {
-        fatalError("TODO")
+        preconditionFailure("TODO")
     }
     #endif
 

Sources/OpenSwiftUI/EventHandling/Focus/FocusStoreLocation.swift

@@ -11,16 +11,16 @@ class FocusStoreLocation<A: Hashable>: AnyLocation<A>, @unchecked Sendable {
     }
 
     override func get() -> A {
-        fatalError("TODO")
+        preconditionFailure("TODO")
     }
 
     override func set(_ value: A, transaction: Transaction) {
-        fatalError("TODO")
+        preconditionFailure("TODO")
     }
 
     typealias Value = A
 
-    override init() { fatalError() }
+    override init() { preconditionFailure() }
 
     var store: FocusStore
     weak var host: GraphHost?

Sources/OpenSwiftUI/Integration/Representable/UIKit/UIViewRepresentable.swift

@@ -185,11 +185,11 @@ extension UIViewRepresentable {
     static func _modifyBridgedViewInputs(_ inputs: inout _ViewInputs) {}
 
     public static func _makeView(view: _GraphValue<Self>, inputs: _ViewInputs) -> _ViewOutputs {
-        fatalError("TODO")
+        preconditionFailure("TODO")
     }
 
     public static func _makeViewList(view: _GraphValue<Self>, inputs: _ViewListInputs) -> _ViewListOutputs {
-        fatalError("TODO")
+        preconditionFailure("TODO")
     }
 }
 

Sources/OpenSwiftUI/Integration/UIKit/UIHostingController.swift

@@ -28,7 +28,7 @@ open class UIHostingController<Content> : UIViewController where Content : View
     }
 
     public required init?(coder: NSCoder) {
-        fatalError("init(coder:) must be implemented in a subclass and call super.init(coder:, rootView:)")
+        preconditionFailure("init(coder:) must be implemented in a subclass and call super.init(coder:, rootView:)")
     }
 
     func _commonInit() {