From 7674ef07d3f53ffcb556798eda8e92e75fbac685 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Sun, 27 Jul 2025 13:40:39 +0200
Subject: [PATCH] fix: change default notification preference to false to
 prevent misleading UI state

- Notifications now default to disabled until user explicitly enables them
- Fixes issue where UI showed notifications as enabled without browser permission
- Ensures config state matches actual browser permission state from the start
- Provides better UX with opt-in notifications rather than false enabled state
---
 web/src/types/config.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/web/src/types/config.ts b/web/src/types/config.ts
index 31ab3fbf..62eb4690 100644
--- a/web/src/types/config.ts
+++ b/web/src/types/config.ts
@@ -70,7 +70,7 @@ export const DEFAULT_QUICK_START_COMMANDS: QuickStartCommand[] = [
 ];
 
 export const DEFAULT_NOTIFICATION_PREFERENCES: NotificationPreferences = {
-  enabled: true,
+  enabled: false,
   sessionStart: true,
   sessionExit: true,
   commandCompletion: true,