From 4b267e4fa7e6455a41a9b87f9bf4e757ff8ce74b Mon Sep 17 00:00:00 2001
From: blankll <zilisheng1996@gmail.com>
Date: Tue, 28 Jul 2026 11:01:47 +0800
Subject: [PATCH 1/4] fix: sidebar navigation click handler not calling
 navigate

Ternary expression returned navigate function reference without invoking it. Vue 3 compiles simple @click='navigate' as  => navigate(), but complex expressions like 'condition ? fn() : navigate' evaluate to the function without calling it.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
---
 src/components/layout/AppSidebar.vue | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/layout/AppSidebar.vue b/src/components/layout/AppSidebar.vue
index a3b9b4a..b24f019 100644
--- a/src/components/layout/AppSidebar.vue
+++ b/src/components/layout/AppSidebar.vue
@@ -47,7 +47,7 @@ const { t } = useI18n()
                       ? 'bg-secondary text-primary border border-border shadow-sm'
                       : 'text-muted-foreground hover:bg-secondary hover:text-foreground',
                 ]"
-                @click="item.id === 'github' ? openUrl('https://github.com/geek-fun/sqlkit') : navigate"
+                @click="item.id === 'github' ? openUrl('https://github.com/geek-fun/sqlkit') : navigate()"
               >
                 <!-- DNS icon -->
                 <svg

From af432c620a21793873a7f1343d9071002bbce825 Mon Sep 17 00:00:00 2001
From: blankll <zilisheng1996@gmail.com>
Date: Tue, 28 Jul 2026 11:01:51 +0800
Subject: [PATCH 2/4] ui: unify result panel icons with editor toolbar style

Match editor toolbar patterns: colored semantic icons (sky/amber/emerald), h-7 w-7 buttons, h-3.5 w-3.5 icon sizes, i-lucide icon set, proper divider, consistent hover effects.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
---
 .../database-browser/QueryResultPanel.vue     | 26 +++++++++----------
 1 file changed, 13 insertions(+), 13 deletions(-)

diff --git a/src/components/database-browser/QueryResultPanel.vue b/src/components/database-browser/QueryResultPanel.vue
index 9db5fb6..70b69f9 100644
--- a/src/components/database-browser/QueryResultPanel.vue
+++ b/src/components/database-browser/QueryResultPanel.vue
@@ -257,8 +257,8 @@ const displayExecutionTime = computed(() => gridExecutionTimeMs.value ?? props.e
         <TooltipProvider>
           <Tooltip>
             <TooltipTrigger as-child>
-              <Button variant="ghost" size="sm" class="text-foreground p-0 h-8 w-8 hover:bg-muted" @click="copyUtil.copyRowsAs(displayResults.rows, displayResults.columns, 'csv')">
-                <span class="i-carbon-table-split h-4.5 w-4.5" />
+              <Button variant="ghost" size="icon" class="text-sky-600 h-7 w-7 dark:text-sky-400 hover:bg-sky-500/10 dark:hover:bg-sky-500/20" @click="copyUtil.copyRowsAs(displayResults.rows, displayResults.columns, 'csv')">
+                <span class="i-lucide-file-spreadsheet shrink-0 h-3.5 w-3.5" />
               </Button>
             </TooltipTrigger>
             <TooltipContent>{{ t('components.dataGrid.export.copyAllCsv') }}</TooltipContent>
@@ -267,8 +267,8 @@ const displayExecutionTime = computed(() => gridExecutionTimeMs.value ?? props.e
         <TooltipProvider>
           <Tooltip>
             <TooltipTrigger as-child>
-              <Button variant="ghost" size="sm" class="text-foreground p-0 h-8 w-8 hover:bg-muted" @click="copyUtil.copyRowsAs(displayResults.rows, displayResults.columns, 'json')">
-                <span class="i-carbon-code h-4.5 w-4.5" />
+              <Button variant="ghost" size="icon" class="text-amber-600 h-7 w-7 dark:text-amber-400 hover:bg-amber-500/10 dark:hover:bg-amber-500/20" @click="copyUtil.copyRowsAs(displayResults.rows, displayResults.columns, 'json')">
+                <span class="i-lucide-file-json shrink-0 h-3.5 w-3.5" />
               </Button>
             </TooltipTrigger>
             <TooltipContent>{{ t('components.dataGrid.export.copyAllJson') }}</TooltipContent>
@@ -277,19 +277,19 @@ const displayExecutionTime = computed(() => gridExecutionTimeMs.value ?? props.e
         <TooltipProvider>
           <Tooltip>
             <TooltipTrigger as-child>
-              <Button variant="ghost" size="sm" class="text-foreground p-0 h-8 w-8 hover:bg-muted" @click="copyUtil.copyRowsAs(displayResults.rows, displayResults.columns, 'insert')">
-                <span class="i-carbon-sql h-4.5 w-4.5" />
+              <Button variant="ghost" size="icon" class="text-emerald-600 h-7 w-7 dark:text-emerald-400 hover:bg-emerald-500/10 dark:hover:bg-emerald-500/20" @click="copyUtil.copyRowsAs(displayResults.rows, displayResults.columns, 'insert')">
+                <span class="i-lucide-database shrink-0 h-3.5 w-3.5" />
               </Button>
             </TooltipTrigger>
             <TooltipContent>{{ t('components.dataGrid.export.copyAllInsert') }}</TooltipContent>
           </Tooltip>
         </TooltipProvider>
-        <span class="text-xs text-muted-foreground mx-1">|</span>
+        <div class="mx-0.5 bg-border h-4 w-px" />
         <TooltipProvider>
           <Tooltip>
             <TooltipTrigger as-child>
-              <Button variant="ghost" size="sm" class="text-foreground p-0 h-8 w-8 hover:bg-muted" @click="copyUtil.exportToFile(displayResults.rows, displayResults.columns, 'csv')">
-                <span class="i-carbon-document-export h-4.5 w-4.5" />
+              <Button variant="ghost" size="icon" class="text-sky-600 h-7 w-7 dark:text-sky-400 hover:bg-sky-500/10 dark:hover:bg-sky-500/20" @click="copyUtil.exportToFile(displayResults.rows, displayResults.columns, 'csv')">
+                <span class="i-lucide-file-down shrink-0 h-3.5 w-3.5" />
               </Button>
             </TooltipTrigger>
             <TooltipContent>{{ t('components.dataGrid.export.exportCsv') }}</TooltipContent>
@@ -298,8 +298,8 @@ const displayExecutionTime = computed(() => gridExecutionTimeMs.value ?? props.e
         <TooltipProvider>
           <Tooltip>
             <TooltipTrigger as-child>
-              <Button variant="ghost" size="sm" class="text-foreground p-0 h-8 w-8 hover:bg-muted" @click="copyUtil.exportToFile(displayResults.rows, displayResults.columns, 'json')">
-                <span class="i-carbon-document-export h-4.5 w-4.5" />
+              <Button variant="ghost" size="icon" class="text-amber-600 h-7 w-7 dark:text-amber-400 hover:bg-amber-500/10 dark:hover:bg-amber-500/20" @click="copyUtil.exportToFile(displayResults.rows, displayResults.columns, 'json')">
+                <span class="i-lucide-file-down shrink-0 h-3.5 w-3.5" />
               </Button>
             </TooltipTrigger>
             <TooltipContent>{{ t('components.dataGrid.export.exportJson') }}</TooltipContent>
@@ -315,11 +315,11 @@ const displayExecutionTime = computed(() => gridExecutionTimeMs.value ?? props.e
       <Button
         variant="ghost"
         size="icon"
-        class="h-6 w-6"
+        class="text-muted-foreground h-7 w-7 hover:text-foreground"
         :title="t('components.queryResult.close')"
         @click="close"
       >
-        <span class="i-carbon-close h-3.5 w-3.5" />
+        <span class="i-lucide-x shrink-0 h-3.5 w-3.5" />
       </Button>
     </div>
 

From 266ecf0768ed082066f2e4e7b44dae3590e3a440 Mon Sep 17 00:00:00 2001
From: blankll <zilisheng1996@gmail.com>
Date: Tue, 28 Jul 2026 11:01:55 +0800
Subject: [PATCH 3/4] fix: DataGrid row selection only via checkbox, actions
 always visible

Removed @click toggleRow from data cells so selection only triggers from checkbox. Removed opacity-0/group-hover from actions column so Edit/Delete menu is always visible.

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
---
 src/components/grid/DataGrid.vue | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/components/grid/DataGrid.vue b/src/components/grid/DataGrid.vue
index 7905b38..d821775 100644
--- a/src/components/grid/DataGrid.vue
+++ b/src/components/grid/DataGrid.vue
@@ -634,7 +634,6 @@ watch(() => [props.rows, props.columns], () => {
               :class="getCellClass(columnTypes?.[col])"
               :style="{ width: `${getColumnWidth(col)}px` }"
               :title="getCellTooltip(rows[virtualRow.index][col])"
-              @click="selection.toggleRow(virtualRow.index, false)"
               @contextmenu.prevent="openCellContextMenu($event, virtualRow.index, col)"
             >
               <!-- NULL -->
@@ -695,7 +694,7 @@ watch(() => [props.rows, props.columns], () => {
             <!-- Row Actions -->
             <div
               v-if="connectionId"
-              class="bg-background/80 opacity-0 flex flex-shrink-0 w-10 transition-opacity items-center right-0 justify-center sticky z-10 group-hover:opacity-100"
+              class="bg-background/80 flex flex-shrink-0 w-10 items-center right-0 justify-center sticky z-10"
               @click.stop
             >
               <DropdownMenu>

From e437b956eed0509fa626b3f6c35ef7ff18b28762 Mon Sep 17 00:00:00 2001
From: blankll <zilisheng1996@gmail.com>
Date: Tue, 28 Jul 2026 11:01:58 +0800
Subject: [PATCH 4/4] chore: update Cargo.lock

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
---
 src-tauri/Cargo.lock | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src-tauri/Cargo.lock b/src-tauri/Cargo.lock
index daf5af1..8fbb4ac 100644
--- a/src-tauri/Cargo.lock
+++ b/src-tauri/Cargo.lock
@@ -6433,7 +6433,7 @@ dependencies = [
 
 [[package]]
 name = "sqlkit"
-version = "0.7.9"
+version = "0.8.0"
 dependencies = [
  "async-trait",
  "base64 0.22.1",