Merge pull request #27 from futuredapp/feature/architecture-cleaning

Housekeep: Remove legacy files and polish the Coordinator API

Sources/FuturedArchitecture/Architecture/Coordinator.swift

@@ -11,6 +11,7 @@ public protocol Coordinator: ObservableObject {
     @ViewBuilder
     static func rootView(with instance: Self) -> RootView
 
+    @MainActor
     var modalCover: ModalCoverModel<Destination>? { get set }
 
     @ViewBuilder
@@ -19,7 +20,7 @@ public protocol Coordinator: ObservableObject {
 }
 
 public extension Coordinator {
-    func present(modal destination: Destination, type: ModalCoverModel<Destination>.Style) {
+    func present(modal destination: Destination, type: ModalCoverModelStyle) {
         switch type {
         case .sheet:
             Task { @MainActor in
@@ -46,10 +47,12 @@ public extension Coordinator {
 public protocol TabCoordinator: Coordinator {
     associatedtype Tab: Hashable
 
+    @MainActor
     var selectedTab: Tab { get set }
 }
 
 public protocol NavigationStackCoordinator: Coordinator {
+    @MainActor
     var path: [Destination] { get set }
 }
 

Sources/FuturedArchitecture/Architecture/ModalCoverModel.swift

@@ -7,17 +7,17 @@
 
 import Foundation
 
-public struct ModalCoverModel<Destination: Hashable & Identifiable>: Identifiable {
-    let destination: Destination
-    let style: Style
-
-    public enum Style {
-        case sheet
-        #if !os(macOS)
-        case fullscreenCover
-        #endif
-    }
+public enum ModalCoverModelStyle {
+    case sheet
+    #if !os(macOS)
+    case fullscreenCover
+    #endif
+}
 
+public struct ModalCoverModel<Destination: Hashable & Identifiable>: Identifiable {
+    public let destination: Destination
+    public let style: ModalCoverModelStyle
+
     public var id: Destination.ID {
         destination.id
     }

Sources/FuturedHelpers/Helpers/CoordinatorSceneFlowProvider.swift

@@ -60,7 +60,7 @@ public protocol CoordinatorSceneFlowProvider {
     var navigateTo: (Destination) -> Void { get }
     var pop: () -> Void { get }
 
-    var present: ((Destination, ModalCoverModel<Destination>.Style) -> Void)? { get }
+    var present: ((Destination, ModalCoverModelStyle) -> Void)? { get }
     var dismissModal: (() -> Void)? { get }
     var onModalDismiss: (() -> Void)? { get }
     var popTo: ((Destination?) -> Void)? { get }

Tests/FuturedKitTests/PlaceholderTest.swift

@@ -0,0 +1,7 @@
+import XCTest
+
+// We might want to write future test cases in SwiftTesting.
+final class PlaceholderTests: XCTestCase {
+    func testNothing() {
+    }
+}