chore(swift): fix SwiftFormat violations in CI

SwiftFormat fixes:
- Wrap single-line property bodies onto multiple lines (wrapPropertyBodies)
- Remove redundant property definitions (redundantProperty)
- Convert regular comments to doc comments for API declarations (docComments)

All violations from CI build resolved. 28/73 files formatted.

Author: lookbusy1344

swift-gui/ARMEmulator/ARMEmulatorApp.swift

@@ -1,15 +1,7 @@
 import SwiftUI
 
-// Environment key for accessing AppDelegate
-private struct AppDelegateKey: EnvironmentKey {
-    static let defaultValue: AppDelegate? = nil
-}
-
 extension EnvironmentValues {
-    var appDelegate: AppDelegate? {
-        get { self[AppDelegateKey.self] }
-        set { self[AppDelegateKey.self] = newValue }
-    }
+    @Entry var appDelegate: AppDelegate?
 }
 
 @MainActor
@@ -19,7 +11,7 @@ class AppDelegate: NSObject, NSApplicationDelegate {
     let settings = AppSettings.shared
     weak var viewModel: EmulatorViewModel?
 
-    // Startup file path from command-line arguments (parsed once on first access)
+    /// Startup file path from command-line arguments (parsed once on first access)
     private(set) lazy var startupFilePath: String? = {
         let args = CommandLine.arguments
 

swift-gui/ARMEmulator/Models/ProgramState.swift

@@ -42,10 +42,14 @@ struct DisassemblyInstruction: Codable, Identifiable, Hashable {
     var disassembly: String
     var symbol: String?
 
-    var id: UInt32 { address }
+    var id: UInt32 {
+        address
+    }
 
-    // Alias for compatibility with code expecting 'mnemonic'
-    var mnemonic: String { disassembly }
+    /// Alias for compatibility with code expecting 'mnemonic'
+    var mnemonic: String {
+        disassembly
+    }
 
     func hash(into hasher: inout Hasher) {
         hasher.combine(address)

swift-gui/ARMEmulator/Services/APIClient.swift

@@ -209,8 +209,7 @@ class APIClient: APIClientProtocol, @unchecked Sendable {
         }
 
         let url = baseURL.appendingPathComponent("/api/v1/session/\(sessionID)/watchpoint")
-        let watchpoint: Watchpoint = try await post(url: url, body: WatchpointRequest(address: address, type: type))
-        return watchpoint
+        return try await post(url: url, body: WatchpointRequest(address: address, type: type))
     }
 
     func removeWatchpoint(sessionID: String, watchpointID: Int) async throws {

swift-gui/ARMEmulator/ViewModels/EmulatorViewModel.swift

@@ -25,7 +25,7 @@ class EmulatorViewModel: ObservableObject {
     @Published var errorMessage: String?
     @Published var isConnected = false
 
-    // Callback for scrolling editor to current PC
+    /// Callback for scrolling editor to current PC
     var scrollToCurrentPC: (() -> Void)?
 
     // Memory state
@@ -34,16 +34,16 @@ class EmulatorViewModel: ObservableObject {
     @Published var lastMemoryWrite: UInt32?
     @Published var lastMemoryWriteSize: UInt32 = 4 // Size in bytes (1, 2, or 4)
 
-    // Disassembly state
+    /// Disassembly state
     @Published var disassembly: [DisassemblyInstruction] = []
 
-    // Source map: address -> source line (for display)
+    /// Source map: address -> source line (for display)
     @Published var sourceMap: [UInt32: String] = [:]
-    // Valid breakpoint lines (1-based line numbers that can have breakpoints)
+    /// Valid breakpoint lines (1-based line numbers that can have breakpoints)
     @Published var validBreakpointLines: Set<Int> = []
-    // Line number to address mapping for breakpoint setting
+    /// Line number to address mapping for breakpoint setting
     @Published var lineToAddress: [Int: UInt32] = [:]
-    // Address to line number mapping for breakpoint display
+    /// Address to line number mapping for breakpoint display
     @Published var addressToLine: [UInt32: Int] = [:]
 
     let apiClient: any APIClientProtocol
@@ -52,12 +52,12 @@ class EmulatorViewModel: ObservableObject {
     private var cancellables = Set<AnyCancellable>()
     private var isInitializing = false
 
-    // Computed property: determines if pause button should be enabled
+    /// Computed property: determines if pause button should be enabled
     var canPause: Bool {
         status == .running || status == .waitingForInput
     }
 
-    // Computed property: determines if step buttons should be enabled
+    /// Computed property: determines if step buttons should be enabled
     var canStep: Bool {
         status == .idle || status == .breakpoint
     }

swift-gui/ARMEmulator/Views/AboutView.swift

@@ -7,7 +7,7 @@ struct AboutView: View {
     @State private var errorMessage: String?
     private let apiClient = APIClient()
 
-    // Cache version info to avoid repeated API calls
+    /// Cache version info to avoid repeated API calls
     private static var cachedVersion: BackendVersion?
 
     var body: some View {
@@ -80,7 +80,8 @@ struct AboutView: View {
         }
     }
 
-    private func loadVersionAsync() async { isLoading = true
+    private func loadVersionAsync() async {
+        isLoading = true
         errorMessage = nil
 
         do {

swift-gui/ARMEmulator/Views/CustomGutterView.swift

@@ -44,7 +44,7 @@ class CustomGutterView: NSView {
         fatalError("init(coder:) not implemented")
     }
 
-    // Use flipped coordinates (origin at top-left) to match text view
+    /// Use flipped coordinates (origin at top-left) to match text view
     override var isFlipped: Bool {
         true
     }

swift-gui/ARMEmulator/Views/EditorView.swift

@@ -9,9 +9,9 @@ struct EditorView: View {
     @State private var scrollView: NSScrollView?
     @EnvironmentObject var viewModel: EmulatorViewModel
 
-    // Compute editor editability based on VM state
-    // Editor is editable only when VM is completely stopped (idle, halted, error)
-    // Editor is read-only during any form of execution (running, paused, breakpoint, waitingForInput)
+    /// Compute editor editability based on VM state
+    /// Editor is editable only when VM is completely stopped (idle, halted, error)
+    /// Editor is read-only during any form of execution (running, paused, breakpoint, waitingForInput)
     private var isEditable: Bool {
         viewModel.status == .idle || viewModel.status == .halted || viewModel.status == .error
     }
@@ -173,7 +173,7 @@ struct EditorView: View {
     }
 }
 
-// Custom text view wrapper for editor
+/// Custom text view wrapper for editor
 struct EditorWithGutterView: NSViewRepresentable {
     @Binding var text: String
     @Binding var breakpoints: Set<Int>

swift-gui/ARMEmulator/Views/ExpressionEvaluatorView.swift

@@ -84,7 +84,6 @@ struct ExpressionEvaluatorView: View {
         .navigationTitle("Expression Evaluator")
     }
 
-    @ViewBuilder
     private func resultRow(_ result: EvaluationResult) -> some View {
         VStack(alignment: .leading, spacing: 4) {
             HStack {
@@ -114,7 +113,6 @@ struct ExpressionEvaluatorView: View {
         .cornerRadius(8)
     }
 
-    @ViewBuilder
     private func resultValue(label: String, value: String) -> some View {
         VStack(alignment: .leading, spacing: 2) {
             Text(label)

swift-gui/ARMEmulator/Views/FileCommands.swift

@@ -75,12 +75,12 @@ struct FileCommands: Commands {
     }
 }
 
-// Notification for showing examples browser
+/// Notification for showing examples browser
 extension Notification.Name {
     static let showExamplesBrowser = Notification.Name("showExamplesBrowser")
 }
 
-// FocusedValue key for accessing ViewModel from commands
+/// FocusedValue key for accessing ViewModel from commands
 struct ViewModelKey: FocusedValueKey {
     typealias Value = EmulatorViewModel
 }

swift-gui/ARMEmulator/Views/MemoryView.swift

@@ -12,7 +12,9 @@ struct MemoryView: View {
 
     private let bytesPerRow = 16
     private let rowsToShow = 16
-    private var totalBytes: Int { bytesPerRow * rowsToShow }
+    private var totalBytes: Int {
+        bytesPerRow * rowsToShow
+    }
 
     var body: some View {
         VStack(spacing: 0) {