Merge branch 'main' into perf/optimize-table-loading

Signed-off-by: Ngô Quốc Đạt <datlechin@gmail.com>

Author: datlechin

.claude/skills/fix-issue/SKILL.md

@@ -7,7 +7,7 @@ description: >-
   scope, no quick patches. It runs a parallel investigation team (codebase tracing + Apple
   platform research + competitor/UX research), synthesizes a refactor-aware implementation
   blueprint, gets the user's approval, then implements to TablePro's standards. Trigger on
-  things like "fix issue #1234", "fix this bug", "this should behave like TablePlus", "do this
+  things like "fix issue #1234", "fix this bug", "this should behave like a native app", "do this
   properly / natively", or any non-trivial defect or behaviour gap in the app. Prefer this over
   an ad-hoc fix when the change touches UI behaviour, architecture, or anything the user expects
   to match Apple conventions.
@@ -59,7 +59,7 @@ Then spawn the three roles in a single turn so they run concurrently. Full chart
 | --- | --- | --- |
 | Codebase Analyzer | `feature-dev:code-explorer` | How does the relevant code actually work today? Which files, types, and call paths are involved? Where is the real cause? |
 | Apple Platform Researcher | `general-purpose` | What does Apple's documentation (HIG, AppKit, SwiftUI) say the correct behaviour and the right API are? |
-| Competitor / UX Researcher | `general-purpose` | How do TablePlus, DataGrip, Postico, and similar native clients handle this? What's the expected UX? |
+| Competitor / UX Researcher | `general-purpose` | How do DataGrip, Postico, Sequel Ace, and similar native clients handle this? What's the expected UX? |
 
 Give each investigator the Phase 0 problem statement and a sharp question. Tell them to report findings as a structured message back to you, citing concrete evidence (`file:line` for code, doc URLs or API names for platform research). `references/research-sources.md` lists the documentation map and the tools each researcher should reach for (`mcp__xcode__DocumentationSearch`, `WebSearch`, `WebFetch`).
 

.claude/skills/fix-issue/references/research-sources.md

@@ -24,7 +24,6 @@ Prefer the Xcode docs tool for API questions: it's authoritative and version-mat
 
 Native macOS database clients worth checking for expected behaviour:
 
-- **TablePlus** — the closest comparison; TablePro positions against it. `https://docs.tableplus.com`.
 - **DataGrip** — JetBrains, feature-rich; good for data-grid and SQL-editor behaviour.
 - **Postico** — native macOS Postgres client, strong HIG conformance; good model for "what feels native on macOS".
 - **Sequel Ace** — open-source MySQL client; behaviour is inspectable.

.claude/skills/fix-issue/references/team-roles.md

@@ -72,13 +72,13 @@ structured message with citations.
 
 ```
 You are the Competitor / UX Researcher on a TablePro fix investigation. TablePro is a native
-macOS database client, a fast alternative to TablePlus.
+macOS database client.
 
 Problem statement:
 <PROBLEM STATEMENT>
 
 Research how mature native DB clients handle this interaction. I need:
-1. How TablePlus, DataGrip, Postico, and Sequel Ace handle this specific behaviour or UI, as
+1. How DataGrip, Postico, and Sequel Ace handle this specific behaviour or UI, as
    concretely as you can (documented behaviour, help docs, release notes, reviews, screenshots).
 2. The interaction pattern users expect: what the control looks like, what the keyboard/mouse
    affordances are, edge cases these tools handle.

CHANGELOG.md

@@ -9,7 +9,33 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
-- The tree sidebar can show only the databases you pick. Use the filter button to check the ones you want, with a search box for long lists. The choice is saved per connection. (#1667)
+- The table structure view has a Triggers tab for MySQL, MariaDB, PostgreSQL, SQLite, SQL Server, Oracle, libSQL, and Cloudflare D1. It lists each trigger with its timing and event (plus enabled state where the engine reports it), with a filter field and sortable columns. Selecting a trigger shows its full definition in a read-only syntax-highlighted viewer. (#1695)
+- Traditional Chinese (繁體中文) language in Settings > General with full UI translation
+- An Add button in the table status bar inserts a new row at the end of the grid and starts editing it.
+
+### Changed
+
+- Selecting a Redis namespace in the sidebar key tree now filters the open database view to that prefix, with paging, instead of opening a separate tab limited to one batch of keys. (#1701)
+
+### Fixed
+
+- Redis entries no longer disappear after the connection sits idle. The health check was running `SELECT 1`, which on Redis switches the active database, so a later refresh scanned the wrong database. (#1701)
+- Redis key browsing now lists every key in a database or namespace and pages through them correctly. It was reading only the first SCAN batch, so large keyspaces showed a partial, fixed set of keys. (#1701)
+- A dropped Redis connection now reconnects on the next command and replays auth and the selected database, instead of failing until the next health check. (#1701)
+- DuckDB VARIANT columns now show their value as text instead of an empty cell.
+
+## [0.51.1] - 2026-06-16
+
+### Added
+
+- The tree sidebar can filter to only the databases you pick, saved per connection. (#1667)
+- Closing a query tab no longer loses unsaved SQL. The next blank query tab for the same connection restores the last closed draft. (#1686)
+- A checkbox in the filter panel header turns every filter row on or off at once, with a dash when only some are on.
+
+### Changed
+
+- The filter panel's "Unset" button is now "Clear". It keeps your filter rows and only drops the applied state. To remove the rows, use "Remove All Filters" in the filter options menu.
+- A row's right-click menu now has "Apply Only This Filter". The inline per-row Apply button is gone.
 
 ### Changed
 
@@ -19,6 +45,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - Expanding or collapsing a database or schema in the tree sidebar while its tables were still loading could crash the app. The tree now updates its rows without rebuilding the outline structure.
 - MongoDB filters on `_id` and other ObjectId fields now match. A 24-character hex value is matched as an ObjectId as well as a string, so filtering by `_id` returns the row instead of nothing. (#1682)
+- Shift+Arrow in the data grid now starts and extends a cell selection from the focused cell. Cmd+Shift+Arrow extends to the row or column edge.
+- Delete key now removes all rows covered by a cell-range selection instead of ignoring it.
+- Right-clicking inside a multi-row or cell-range selection no longer collapses the selection first.
+- Oracle connections no longer crash during connect when the server sends a short or unexpected handshake packet. (#1683)
+- MongoDB filters on `_id` and other ObjectId fields now match. A 24-character hex value is matched as an ObjectId as well as a string. (#1682)
+- The sidebar and inspector keep their width per connection, the sidebar its collapsed state, and the inspector its selected tab, across quit and reopen.
 
 ## [0.51.0] - 2026-06-13
 
@@ -2281,7 +2313,8 @@ TablePro is a native macOS database client built with SwiftUI and AppKit, design
     - Custom SQL query templates
     - Performance optimized for large datasets
 
-[Unreleased]: https://github.com/TableProApp/TablePro/compare/v0.51.0...HEAD
+[Unreleased]: https://github.com/TableProApp/TablePro/compare/v0.51.1...HEAD
+[0.51.1]: https://github.com/TableProApp/TablePro/compare/v0.51.0...v0.51.1
 [0.51.0]: https://github.com/TableProApp/TablePro/compare/v0.50.0...v0.51.0
 [0.50.0]: https://github.com/TableProApp/TablePro/compare/v0.49.1...v0.50.0
 [0.49.1]: https://github.com/TableProApp/TablePro/compare/v0.49.0...v0.49.1

Plugins/CloudflareD1DriverPlugin/CloudflareD1Plugin.swift

@@ -26,6 +26,7 @@ final class CloudflareD1Plugin: NSObject, TableProPlugin, DriverPlugin {
     static let isDownloadable = true
     static let supportsImport = false
     static let supportsSchemaEditing = true
+    static let supportsTriggers = true
     static let databaseGroupingStrategy: GroupingStrategy = .flat
     static let brandColorHex = "#F6821F"
     static let urlSchemes: [String] = ["d1"]

Plugins/CloudflareD1DriverPlugin/CloudflareD1PluginDriver.swift

@@ -433,6 +433,27 @@ final class CloudflareD1PluginDriver: PluginDatabaseDriver, @unchecked Sendable
         }
     }
 
+    func fetchTriggers(table: String, schema: String?) async throws -> [PluginTriggerInfo] {
+        let safeTable = escapeStringLiteral(table)
+        let query = """
+            SELECT name, sql FROM sqlite_master
+            WHERE type = 'trigger' AND tbl_name = '\(safeTable)'
+                AND name NOT GLOB '_cf_*'
+            ORDER BY name
+            """
+        let result = try await execute(query: query)
+
+        return result.rows.compactMap { row -> PluginTriggerInfo? in
+            guard row.count >= 2,
+                  let name = row[0].asText,
+                  let sql = row[1].asText else {
+                return nil
+            }
+            let (timing, event) = TriggerSQLParser.timingAndEvent(from: sql)
+            return PluginTriggerInfo(name: name, timing: timing, event: event, statement: sql)
+        }
+    }
+
     func fetchTableDDL(table: String, schema: String?) async throws -> String {
         let safeTable = escapeStringLiteral(table)
         let query = """

Plugins/DuckDBDriverPlugin/DuckDBConnection.swift

@@ -91,7 +91,7 @@ actor DuckDBConnectionActor {
         }
 
         var raw = Self.extractResult(from: &result, startTime: startTime)
-        Self.patchTzColumns(&raw, query: query, connection: conn)
+        Self.patchCastedColumns(&raw, query: query, connection: conn)
         return raw
     }
 
@@ -163,7 +163,7 @@ actor DuckDBConnectionActor {
         }
 
         var raw = Self.extractResult(from: &result, startTime: startTime)
-        Self.patchTzColumns(&raw, query: query, connection: conn)
+        Self.patchCastedColumns(&raw, query: query, connection: conn)
         return raw
     }
 
@@ -204,7 +204,7 @@ actor DuckDBConnectionActor {
             columnTypeNames.append(Self.typeName(for: colType))
         }
 
-        if columnTypes.contains(where: Self.isUnrenderable) {
+        if columnTypes.contains(where: Self.requiresTextCast) {
             duckdb_destroy_result(&result)
             try Self.streamWrappedQuery(
                 query: query,
@@ -235,7 +235,7 @@ actor DuckDBConnectionActor {
         continuation: AsyncThrowingStream<PluginStreamElement, Error>.Continuation
     ) throws {
         let castExprs = columns.enumerated().map { i, name in
-            castExpression(for: columnTypes[i], column: name)
+            projection(for: columnTypes[i], column: name)
         }
         let wrappedQuery = buildWrappedQuery(originalQuery: query, castExprs: castExprs)
 
@@ -484,16 +484,16 @@ actor DuckDBConnectionActor {
         }
     }
 
-    static func patchTzColumns(
+    static func patchCastedColumns(
         _ raw: inout DuckDBRawResult, query: String, connection: duckdb_connection
     ) {
         let patchedColIndices = raw.columnTypes.enumerated().compactMap { idx, type in
-            isUnrenderable(type) ? idx : nil
+            requiresTextCast(type) ? idx : nil
         }
         guard !patchedColIndices.isEmpty, !raw.rows.isEmpty else { return }
 
         let castExprs = raw.columns.enumerated().map { i, name in
-            castExpression(for: raw.columnTypes[i], column: name)
+            projection(for: raw.columnTypes[i], column: name)
         }
         let wrappedQuery = buildWrappedQuery(originalQuery: query, castExprs: castExprs)
 
@@ -514,25 +514,36 @@ actor DuckDBConnectionActor {
         }
     }
 
-    static func isUnrenderable(_ type: duckdb_type) -> Bool {
+    static func isNativelyRenderable(_ type: duckdb_type) -> Bool {
         switch type {
-        case DUCKDB_TYPE_TIMESTAMP_TZ, DUCKDB_TYPE_TIME_TZ, DUCKDB_TYPE_GEOMETRY:
+        case DUCKDB_TYPE_BOOLEAN,
+             DUCKDB_TYPE_TINYINT, DUCKDB_TYPE_SMALLINT, DUCKDB_TYPE_INTEGER, DUCKDB_TYPE_BIGINT, DUCKDB_TYPE_HUGEINT,
+             DUCKDB_TYPE_UTINYINT, DUCKDB_TYPE_USMALLINT, DUCKDB_TYPE_UINTEGER, DUCKDB_TYPE_UBIGINT, DUCKDB_TYPE_UHUGEINT,
+             DUCKDB_TYPE_FLOAT, DUCKDB_TYPE_DOUBLE, DUCKDB_TYPE_DECIMAL,
+             DUCKDB_TYPE_VARCHAR, DUCKDB_TYPE_BLOB, DUCKDB_TYPE_UUID, DUCKDB_TYPE_BIT, DUCKDB_TYPE_ENUM,
+             DUCKDB_TYPE_DATE, DUCKDB_TYPE_TIME, DUCKDB_TYPE_TIME_NS, DUCKDB_TYPE_INTERVAL,
+             DUCKDB_TYPE_TIMESTAMP, DUCKDB_TYPE_TIMESTAMP_S, DUCKDB_TYPE_TIMESTAMP_MS, DUCKDB_TYPE_TIMESTAMP_NS,
+             DUCKDB_TYPE_LIST, DUCKDB_TYPE_STRUCT, DUCKDB_TYPE_MAP, DUCKDB_TYPE_ARRAY, DUCKDB_TYPE_UNION:
             return true
         default:
             return false
         }
     }
 
+    static func requiresTextCast(_ type: duckdb_type) -> Bool {
+        !isNativelyRenderable(type)
+    }
+
     static func castExpression(for type: duckdb_type, column: String) -> String {
         let quoted = quoteIdentifier(column)
-        switch type {
-        case DUCKDB_TYPE_GEOMETRY:
+        if type == DUCKDB_TYPE_GEOMETRY {
             return "CASE WHEN \(quoted) IS NULL THEN NULL ELSE ST_AsText(\(quoted)) END AS \(quoted)"
-        case DUCKDB_TYPE_TIMESTAMP_TZ, DUCKDB_TYPE_TIME_TZ:
-            return "CASE WHEN \(quoted) IS NULL THEN NULL ELSE CAST(\(quoted) AS VARCHAR) END AS \(quoted)"
-        default:
-            return quoted
         }
+        return "CASE WHEN \(quoted) IS NULL THEN NULL ELSE CAST(\(quoted) AS VARCHAR) END AS \(quoted)"
+    }
+
+    static func projection(for type: duckdb_type, column: String) -> String {
+        requiresTextCast(type) ? castExpression(for: type, column: column) : quoteIdentifier(column)
     }
 
     static func buildWrappedQuery(originalQuery: String, castExprs: [String]) -> String {

Plugins/LibSQLDriverPlugin/LibSQLPlugin.swift

@@ -28,6 +28,7 @@ final class LibSQLPlugin: NSObject, TableProPlugin, DriverPlugin {
     static let isDownloadable = true
     static let supportsImport = false
     static let supportsSchemaEditing = true
+    static let supportsTriggers = true
     static let supportsDropDatabase = false
     static let supportsDatabaseSwitching = false
     static let databaseGroupingStrategy: GroupingStrategy = .flat

Plugins/LibSQLDriverPlugin/LibSQLPluginDriver.swift

@@ -505,6 +505,26 @@ final class LibSQLPluginDriver: PluginDatabaseDriver, @unchecked Sendable {
         }
     }
 
+    func fetchTriggers(table: String, schema: String?) async throws -> [PluginTriggerInfo] {
+        let safeTable = escapeStringLiteral(table)
+        let query = """
+            SELECT name, sql FROM sqlite_master
+            WHERE type = 'trigger' AND tbl_name = '\(safeTable)'
+            ORDER BY name
+            """
+        let result = try await execute(query: query)
+
+        return result.rows.compactMap { row -> PluginTriggerInfo? in
+            guard row.count >= 2,
+                  let name = row[0].asText,
+                  let sql = row[1].asText else {
+                return nil
+            }
+            let (timing, event) = TriggerSQLParser.timingAndEvent(from: sql)
+            return PluginTriggerInfo(name: name, timing: timing, event: event, statement: sql)
+        }
+    }
+
     func fetchTableDDL(table: String, schema: String?) async throws -> String {
         let safeTable = escapeStringLiteral(table)
         let query = """

Plugins/MSSQLDriverPlugin/MSSQLPlugin.swift

@@ -150,6 +150,7 @@ final class MSSQLPlugin: NSObject, TableProPlugin, DriverPlugin {
     )
 
     static let supportsDropDatabase = true
+    static let supportsTriggers = true
 
     func createDriver(config: DriverConnectionConfig) -> any PluginDatabaseDriver {
         MSSQLPluginDriver(config: config)