From 1f6fb863f475b948bcd9acc385f3aa78ff73dce9 Mon Sep 17 00:00:00 2001
From: t8y2 <1156263951@qq.com>
Date: Thu, 14 May 2026 22:11:03 +0800
Subject: [PATCH] fix(clickhouse): validate auth in test_connection and disable
 transactions

- test_connection now uses SELECT 1 instead of /ping which doesn't
  require authentication, so wrong passwords are caught early
- Disable transaction support for ClickHouse (no BEGIN/COMMIT)
- Mark ClickHouse tables as non-insertable (no row-level operations)
---
 src/lib/databaseTableDataCapabilities.ts | 9 +++++++++
 1 file changed, 9 insertions(+)

diff --git a/src/lib/databaseTableDataCapabilities.ts b/src/lib/databaseTableDataCapabilities.ts
index 9e91c919c2..573e1d4020 100644
--- a/src/lib/databaseTableDataCapabilities.ts
+++ b/src/lib/databaseTableDataCapabilities.ts
@@ -66,6 +66,15 @@ const DATABASE_CAPABILITY_OVERRIDES: Partial<Record<DatabaseType, Partial<Databa
       transaction: false,
     },
   },
+  clickhouse: {
+    tableData: {
+      insert: false,
+      updateRequiresPrimaryKey: true,
+      deleteRequiresPrimaryKey: true,
+      requiresTransactionalTableForExistingRows: false,
+      transaction: false,
+    },
+  },
 };
 
 export function getDatabaseCapability(dbType?: DatabaseType): DatabaseCapability {