fix: resolve SwiftLint errors in terminal, MCP, and JSON viewer

Author: datlechin

TablePro/Core/MCP/MCPMessageTypes.swift

@@ -423,4 +423,3 @@ struct MCPResourceContent: Codable, Sendable {
 struct MCPResourceReadResult: Codable, Sendable {
     let contents: [MCPResourceContent]
 }
-

TablePro/Core/Terminal/TerminalProcessManager.swift

@@ -12,17 +12,17 @@ final class TerminalProcessManager {
     private static let logger = Logger(subsystem: "com.TablePro", category: "TerminalProcessManager")
 
     private let fdLock = NSLock()
-    private nonisolated(unsafe) var _ptyFD: Int32 = -1
+    nonisolated(unsafe) private var _ptyFD: Int32 = -1
 
     private var ptyFD: Int32 {
         get { fdLock.withLock { _ptyFD } }
         set { fdLock.withLock { _ptyFD = newValue } }
     }
 
     private let stateLock = NSLock()
-    private nonisolated(unsafe) var _childPID: pid_t = 0
-    private nonisolated(unsafe) var _readSource: DispatchSourceRead?
-    private nonisolated(unsafe) var _processMonitor: DispatchSourceProcess?
+    nonisolated(unsafe) private var _childPID: pid_t = 0
+    nonisolated(unsafe) private var _readSource: DispatchSourceRead?
+    nonisolated(unsafe) private var _processMonitor: DispatchSourceProcess?
 
     var onData: ((Data) -> Void)?
     var onExit: ((Int32) -> Void)?
@@ -66,7 +66,7 @@ final class TerminalProcessManager {
 
         if pid == 0 {
             // Child process: ONLY async-signal-safe POSIX calls, no Swift
-            execve(cArgs[0]!, cArgs, cEnv)
+            execve(cArgs[0]!, cArgs, cEnv) // swiftlint:disable:this force_unwrapping
             _exit(127)
         }
 
@@ -105,8 +105,8 @@ final class TerminalProcessManager {
 
     // MARK: - Resize (called from libghostty threads)
 
-    private nonisolated(unsafe) var lastCols: Int = 0
-    private nonisolated(unsafe) var lastRows: Int = 0
+    nonisolated(unsafe) private var lastCols: Int = 0
+    nonisolated(unsafe) private var lastRows: Int = 0
     private let resizeLock = NSLock()
 
     nonisolated func resize(cols: Int, rows: Int) {
@@ -154,7 +154,7 @@ final class TerminalProcessManager {
         }
     }
 
-    private nonisolated func killAndReap() {
+    nonisolated private func killAndReap() {
         let pid = stateLock.withLock {
             let p = _childPID
             _childPID = 0
@@ -169,7 +169,7 @@ final class TerminalProcessManager {
         }
     }
 
-    private nonisolated func cancelSources() {
+    nonisolated private func cancelSources() {
         stateLock.withLock {
             _readSource?.cancel()
             _readSource = nil

TablePro/Models/Settings/MCPSettings.swift

@@ -17,7 +17,7 @@ struct MCPSettings: Codable, Equatable {
 
     static let `default` = MCPSettings(
         enabled: false,
-        port: 23508,
+        port: 23_508,
         defaultRowLimit: 500,
         maxRowLimit: 10_000,
         queryTimeoutSeconds: 30,
@@ -26,7 +26,7 @@ struct MCPSettings: Codable, Equatable {
 
     init(
         enabled: Bool = false,
-        port: Int = 23508,
+        port: Int = 23_508,
         defaultRowLimit: Int = 500,
         maxRowLimit: Int = 10_000,
         queryTimeoutSeconds: Int = 30,
@@ -43,8 +43,8 @@ struct MCPSettings: Codable, Equatable {
     init(from decoder: Decoder) throws {
         let container = try decoder.container(keyedBy: CodingKeys.self)
         enabled = try container.decodeIfPresent(Bool.self, forKey: .enabled) ?? false
-        let rawPort = try container.decodeIfPresent(Int.self, forKey: .port) ?? 23508
-        port = (1...65535).contains(rawPort) ? rawPort : 23508
+        let rawPort = try container.decodeIfPresent(Int.self, forKey: .port) ?? 23_508
+        port = (1...65_535).contains(rawPort) ? rawPort : 23_508
         defaultRowLimit = try container.decodeIfPresent(Int.self, forKey: .defaultRowLimit) ?? 500
         maxRowLimit = try container.decodeIfPresent(Int.self, forKey: .maxRowLimit) ?? 10_000
         queryTimeoutSeconds = try container.decodeIfPresent(Int.self, forKey: .queryTimeoutSeconds) ?? 30

TablePro/Views/Results/JSONTreeView.swift

@@ -149,8 +149,7 @@ private struct JSONTreeContentView: View {
                     isExpanded: Binding(
                         get: { expandedNodeIDs.contains(node.id) },
                         set: { expanded in
-                            if expanded { expandedNodeIDs.insert(node.id) }
-                            else { expandedNodeIDs.remove(node.id) }
+                            if expanded { expandedNodeIDs.insert(node.id) } else { expandedNodeIDs.remove(node.id) }
                         }
                     )
                 ) {