feat: implement right sidebar inspector panel

OpenTable/AppIcon.icon/icon.json

@@ -1,40 +1,40 @@
 {
-  "fill" : {
-    "automatic-gradient" : "srgb:1.00000,0.57637,0.00000,1.00000"
+  "fill": {
+    "automatic-gradient": "srgb:1.00000,0.57637,0.00000,1.00000"
   },
-  "groups" : [
+  "groups": [
     {
-      "layers" : [
+      "layers": [
         {
-          "fill" : {
-            "solid" : "display-p3:0.99736,1.00000,0.97886,1.00000"
+          "fill": {
+            "solid": "display-p3:0.99736,1.00000,0.97886,1.00000"
           },
-          "image-name" : "cylinder.split.1x2.fill 1.svg",
-          "name" : "cylinder.split.1x2.fill 1"
+          "image-name": "cylinder.split.1x2.fill 1.svg",
+          "name": "cylinder.split.1x2.fill 1"
         }
       ],
-      "position" : {
-        "scale" : 1.3,
-        "translation-in-points" : [
+      "position": {
+        "scale": 1.3,
+        "translation-in-points": [
           0,
           0
         ]
       },
-      "shadow" : {
-        "kind" : "neutral",
-        "opacity" : 0.5
+      "shadow": {
+        "kind": "neutral",
+        "opacity": 0.5
       },
-      "specular" : false,
-      "translucency" : {
-        "enabled" : true,
-        "value" : 0.5
+      "specular": false,
+      "translucency": {
+        "enabled": true,
+        "value": 0.5
       }
     }
   ],
-  "supported-platforms" : {
-    "circles" : [
+  "supported-platforms": {
+    "circles": [
       "watchOS"
     ],
-    "squares" : "shared"
+    "squares": "shared"
   }
 }

OpenTable/ContentView.swift

@@ -10,7 +10,7 @@ import SwiftUI
 struct ContentView: View {
     @StateObject private var dbManager = DatabaseManager.shared
     @State private var connections: [DatabaseConnection] = []
-    @State private var columnVisibility: NavigationSplitViewVisibility = .detailOnly
+    @State private var columnVisibility: NavigationSplitViewVisibility = .all
     @State private var showNewConnectionSheet = false
     @State private var showEditConnectionSheet = false
     @State private var connectionToEdit: DatabaseConnection?
@@ -20,6 +20,7 @@ struct ContentView: View {
     @State private var pendingCloseSessionId: UUID?
     @State private var hasLoaded = false
     @State private var escapeKeyMonitor: Any?
+    @State private var isInspectorPresented = false  // Right sidebar (inspector) visibility
 
     @Environment(\.openWindow) private var openWindow
     @Environment(\.dismissWindow) private var dismissWindow
@@ -91,10 +92,16 @@ struct ContentView: View {
                 guard currentSession != nil else { return }
                 Task { @MainActor in
                     withAnimation {
-                        columnVisibility = columnVisibility == .all ? .detailOnly : .all
+                        // Toggle left sidebar (2-column layout: sidebar + detail)
+                        if columnVisibility == .all {
+                            columnVisibility = .detailOnly
+                        } else {
+                            columnVisibility = .all
+                        }
                     }
                 }
             }
+            // Right sidebar toggle is handled by MainContentView (has the binding)
             .onChange(of: dbManager.currentSessionId) { _, newSessionId in
                 Task { @MainActor in
                     withAnimation {
@@ -117,6 +124,7 @@ struct ContentView: View {
     private var mainContent: some View {
         if currentSession != nil {
             NavigationSplitView(columnVisibility: $columnVisibility) {
+                // MARK: - Sidebar (Left) - Table Browser
                 VStack(spacing: 0) {
                     if !sessions.isEmpty {
                         ConnectionSidebarHeader(
@@ -152,13 +160,16 @@ struct ContentView: View {
                         pendingDeletes: sessionPendingDeletesBinding
                     )
                 }
+                .navigationSplitViewColumnWidth(min: 200, ideal: 250, max: 350)
             } detail: {
+                // MARK: - Detail (Main workspace with optional right sidebar)
                 MainContentView(
                     connection: currentSession!.connection,
                     tables: sessionTablesBinding,
                     selectedTables: sessionSelectedTablesBinding,
                     pendingTruncates: sessionPendingTruncatesBinding,
-                    pendingDeletes: sessionPendingDeletesBinding
+                    pendingDeletes: sessionPendingDeletesBinding,
+                    isInspectorPresented: $isInspectorPresented
                 )
                 .id(currentSession!.id)
             }

OpenTable/Core/Database/DatabaseDriver.swift

@@ -55,6 +55,9 @@ protocol DatabaseDriver: AnyObject {
 
     /// Fetch rows with LIMIT/OFFSET pagination
     func fetchRows(query: String, offset: Int, limit: Int) async throws -> QueryResult
+
+    /// Fetch table metadata (size, comment, engine, etc.)
+    func fetchTableMetadata(tableName: String) async throws -> TableMetadata
 }
 
 /// Default implementation for common operations

OpenTable/Core/Database/MySQLDriver.swift

@@ -15,6 +15,13 @@ final class MySQLDriver: DatabaseDriver {
 
     /// The underlying MariaDB connection
     private var mariadbConnection: MariaDBConnection?
+
+    /// Static date formatter for parsing MySQL dates (performance optimization)
+    private static let mysqlDateFormatter: DateFormatter = {
+        let formatter = DateFormatter()
+        formatter.dateFormat = "yyyy-MM-dd HH:mm:ss"
+        return formatter
+    }()
 
     init(connection: DatabaseConnection) {
         self.connection = connection
@@ -294,6 +301,77 @@ final class MySQLDriver: DatabaseDriver {
 
         return ddl
     }
+
+    func fetchTableMetadata(tableName: String) async throws -> TableMetadata {
+        let escapedTableName = tableName.replacingOccurrences(of: "'", with: "''")
+        // NOTE: `SHOW TABLE STATUS LIKE` expects a pattern string literal, not an
+        // identifier. For that reason we must use single-quoted string syntax here
+        // instead of the backtick identifier quoting used in other schema queries
+        // (e.g. `SHOW CREATE TABLE \`table\``). The table name is safely embedded
+        // by escaping single quotes above.
+        let query = "SHOW TABLE STATUS WHERE Name = '\(escapedTableName)'"
+        let result = try await execute(query: query)
+
+        guard let row = result.rows.first else {
+            return TableMetadata(
+                tableName: tableName,
+                dataSize: nil,
+                indexSize: nil,
+                totalSize: nil,
+                avgRowLength: nil,
+                rowCount: nil,
+                comment: nil,
+                engine: nil,
+                collation: nil,
+                createTime: nil,
+                updateTime: nil
+            )
+        }
+
+        // SHOW TABLE STATUS columns:
+        // 0: Name, 1: Engine, 2: Version, 3: Row_format, 4: Rows, 5: Avg_row_length,
+        // 6: Data_length, 7: Max_data_length, 8: Index_length, 9: Data_free,
+        // 10: Auto_increment, 11: Create_time, 12: Update_time, 13: Check_time,
+        // 14: Collation, 15: Checksum, 16: Create_options, 17: Comment
+
+        let engine = row.count > 1 ? row[1] : nil
+        let rowCount = row.count > 4 ? Int64(row[4] ?? "0") : nil
+        let avgRowLength = row.count > 5 ? Int64(row[5] ?? "0") : nil
+        let dataSize = row.count > 6 ? Int64(row[6] ?? "0") : nil
+        let indexSize = row.count > 8 ? Int64(row[8] ?? "0") : nil
+        let collation = row.count > 14 ? row[14] : nil
+        let comment = row.count > 17 ? row[17] : nil
+
+        // Parse dates using static formatter for performance
+        let createTime: Date? = {
+            guard row.count > 11, let dateStr = row[11] else { return nil }
+            return Self.mysqlDateFormatter.date(from: dateStr)
+        }()
+
+        let updateTime: Date? = {
+            guard row.count > 12, let dateStr = row[12] else { return nil }
+            return Self.mysqlDateFormatter.date(from: dateStr)
+        }()
+
+        let totalSize: Int64? = {
+            guard let data = dataSize, let index = indexSize else { return nil }
+            return data + index
+        }()
+
+        return TableMetadata(
+            tableName: tableName,
+            dataSize: dataSize,
+            indexSize: indexSize,
+            totalSize: totalSize,
+            avgRowLength: avgRowLength,
+            rowCount: rowCount,
+            comment: comment?.isEmpty == true ? nil : comment,
+            engine: engine,
+            collation: collation,
+            createTime: createTime,
+            updateTime: updateTime
+        )
+    }
 
     // MARK: - Paginated Query Support
 

OpenTable/Core/Database/PostgreSQLDriver.swift

@@ -309,6 +309,64 @@ final class PostgreSQLDriver: DatabaseDriver {
         let paginatedQuery = "\(baseQuery) LIMIT \(limit) OFFSET \(offset)"
         return try await execute(query: paginatedQuery)
     }
+
+    func fetchTableMetadata(tableName: String) async throws -> TableMetadata {
+        // Escape single quotes to prevent SQL injection (string literal context)
+        let safeTableName = tableName.replacingOccurrences(of: "'", with: "''")
+
+        let query = """
+            SELECT
+                pg_total_relation_size(c.oid) AS total_size,
+                pg_table_size(c.oid) AS data_size,
+                pg_indexes_size(c.oid) AS index_size,
+                c.reltuples::bigint AS row_count,
+                CASE WHEN c.reltuples > 0 THEN pg_table_size(c.oid) / GREATEST(c.reltuples, 1) ELSE 0 END AS avg_row_length,
+                obj_description(c.oid, 'pg_class') AS comment
+            FROM pg_class c
+            JOIN pg_namespace n ON n.oid = c.relnamespace
+            WHERE c.relname = '\(safeTableName)'
+              AND n.nspname = 'public'
+            """
+
+        let result = try await execute(query: query)
+
+        guard let row = result.rows.first else {
+            return TableMetadata(
+                tableName: tableName,
+                dataSize: nil,
+                indexSize: nil,
+                totalSize: nil,
+                avgRowLength: nil,
+                rowCount: nil,
+                comment: nil,
+                engine: nil,
+                collation: nil,
+                createTime: nil,
+                updateTime: nil
+            )
+        }
+
+        let totalSize = row.count > 0 ? Int64(row[0] ?? "0") : nil
+        let dataSize = row.count > 1 ? Int64(row[1] ?? "0") : nil
+        let indexSize = row.count > 2 ? Int64(row[2] ?? "0") : nil
+        let rowCount = row.count > 3 ? Int64(row[3] ?? "0") : nil
+        let avgRowLength = row.count > 4 ? Int64(row[4] ?? "0") : nil
+        let comment = row.count > 5 ? row[5] : nil
+
+        return TableMetadata(
+            tableName: tableName,
+            dataSize: dataSize,
+            indexSize: indexSize,
+            totalSize: totalSize,
+            avgRowLength: avgRowLength,
+            rowCount: rowCount,
+            comment: comment?.isEmpty == true ? nil : comment,
+            engine: "PostgreSQL",
+            collation: nil,
+            createTime: nil,
+            updateTime: nil
+        )
+    }
 
     private func stripLimitOffset(from query: String) -> String {
         var result = query

OpenTable/Core/Database/SQLiteDriver.swift

@@ -297,6 +297,39 @@ final class SQLiteDriver: DatabaseDriver {
         return try await execute(query: paginatedQuery)
     }
 
+    func fetchTableMetadata(tableName: String) async throws -> TableMetadata {
+        guard status == .connected else {
+            throw DatabaseError.notConnected
+        }
+
+        // Escape table name to prevent SQL injection (escape double quotes for identifier quoting)
+        let safeTableName = tableName.replacingOccurrences(of: "\"", with: "\"\"")
+
+        // Get row count
+        let countQuery = "SELECT COUNT(*) FROM \"\(safeTableName)\""
+        let countResult = try await execute(query: countQuery)
+        let rowCount: Int64? = {
+            guard let row = countResult.rows.first, let countStr = row.first else { return nil }
+            return Int64(countStr ?? "0")
+        }()
+
+        // SQLite does not expose accurate per-table size information.
+        // To avoid reporting misleading values, we leave size-related fields as nil.
+        return TableMetadata(
+            tableName: tableName,
+            dataSize: nil,
+            indexSize: nil,
+            totalSize: nil,
+            avgRowLength: nil,
+            rowCount: rowCount,
+            comment: nil,
+            engine: "SQLite",
+            collation: nil,
+            createTime: nil,
+            updateTime: nil
+        )
+    }
+
     private func stripLimitOffset(from query: String) -> String {
         var result = query
 

OpenTable/Models/TableMetadata.swift

@@ -0,0 +1,39 @@
+//
+//  TableMetadata.swift
+//  OpenTable
+//
+//  Model for table-level metadata
+//
+
+import Foundation
+
+/// Represents table-level metadata fetched from database
+struct TableMetadata {
+    let tableName: String
+    let dataSize: Int64?
+    let indexSize: Int64?
+    let totalSize: Int64?
+    let avgRowLength: Int64?
+    let rowCount: Int64?
+    let comment: String?
+    let engine: String?          // MySQL/MariaDB only
+    let collation: String?       // MySQL/MariaDB only
+    let createTime: Date?
+    let updateTime: Date?
+
+    /// Format a size in bytes to human readable format
+    static func formatSize(_ bytes: Int64?) -> String {
+        guard let bytes = bytes else { return "—" }
+        if bytes == 0 { return "0 B" }
+
+        let units = ["B", "KB", "MB", "GB", "TB"]
+        let exponent = min(Int(log(Double(bytes)) / log(1024)), units.count - 1)
+        let size = Double(bytes) / pow(1024, Double(exponent))
+
+        if exponent == 0 {
+            return "\(bytes) B"
+        } else {
+            return String(format: "%.1f %@", size, units[exponent])
+        }
+    }
+}