From e45c2f0bc8d501ee8224dfc102386fe48a81819c Mon Sep 17 00:00:00 2001
From: Yann Amsellem <yann@agnostic.engineering>
Date: Tue, 14 Jan 2025 17:51:57 +0100
Subject: [PATCH] refactor(tabs): add constraint to ensure that only one tab
 can be active

---
 src/lib/migrations/003_create_tabs_table.sql | 2 +-
 src/lib/repositories/tabs.ts                 | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/lib/migrations/003_create_tabs_table.sql b/src/lib/migrations/003_create_tabs_table.sql
index 79bb78b..35fcf01 100644
--- a/src/lib/migrations/003_create_tabs_table.sql
+++ b/src/lib/migrations/003_create_tabs_table.sql
@@ -4,5 +4,5 @@ CREATE TABLE IF NOT EXISTS tabs (
     contents TEXT NOT NULL,
     query_id INTEGER,
     tab_index INTEGER NOT NULL,
-    active BOOL NOT NULL DEFAULT FALSE
+    active BOOL DEFAULT FALSE UNIQUE
 );
diff --git a/src/lib/repositories/tabs.ts b/src/lib/repositories/tabs.ts
index 0407b7f..e5c2a11 100644
--- a/src/lib/repositories/tabs.ts
+++ b/src/lib/repositories/tabs.ts
@@ -36,7 +36,7 @@ VALUES ${Array.from({ length: rows.length }).fill('(?,?,?,?,?, ?)').join(',\n')}
 					r.contents,
 					r.query_id ?? null,
 					r.tab_index,
-					r.tab_index === activeIndex
+					r.tab_index === activeIndex || null
 				])
 				.flat()
 		);