From 4cb6991fc6789b9d68a546cfc4af942e50417438 Mon Sep 17 00:00:00 2001
From: viettranx <viettranx@gmail.com>
Date: Sun, 29 Mar 2026 19:41:35 +0700
Subject: [PATCH] fix: default noisy team notifications to off
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Disable progress, failed, commented, and slow_tool notifications
by default — these are too verbose for most teams.
---
 internal/tools/team_notify_config.go         |  8 ++++----
 ui/web/src/pages/teams/team-settings-tab.tsx | 16 ++++++++--------
 2 files changed, 12 insertions(+), 12 deletions(-)

diff --git a/internal/tools/team_notify_config.go b/internal/tools/team_notify_config.go
index 54d000b6..d03edd6c 100644
--- a/internal/tools/team_notify_config.go
+++ b/internal/tools/team_notify_config.go
@@ -18,12 +18,12 @@ type TeamNotifyConfig struct {
 func DefaultTeamNotifyConfig() TeamNotifyConfig {
 	return TeamNotifyConfig{
 		Dispatched: true,
-		Progress:   true,
-		Failed:     true,
+		Progress:   false,
+		Failed:     false,
 		Completed:  true,
-		Commented:  true,
+		Commented:  false,
 		NewTask:    true,
-		SlowTool:   true,
+		SlowTool:   false,
 		Mode:       "direct",
 	}
 }
diff --git a/ui/web/src/pages/teams/team-settings-tab.tsx b/ui/web/src/pages/teams/team-settings-tab.tsx
index 4d30816c..0a6064c8 100644
--- a/ui/web/src/pages/teams/team-settings-tab.tsx
+++ b/ui/web/src/pages/teams/team-settings-tab.tsx
@@ -71,12 +71,12 @@ export function TeamSettingsTab({ teamId, team, onSaved }: TeamSettingsTabProps)
   const [denyChannels, setDenyChannels] = useState<string[]>(initial.deny_channels ?? []);
   const initNotify = initial.notifications ?? {};
   const [notifyDispatched, setNotifyDispatched] = useState(initNotify.dispatched ?? true);
-  const [notifyProgress, setNotifyProgress] = useState(initNotify.progress ?? true);
-  const [notifyFailed, setNotifyFailed] = useState(initNotify.failed ?? true);
+  const [notifyProgress, setNotifyProgress] = useState(initNotify.progress ?? false);
+  const [notifyFailed, setNotifyFailed] = useState(initNotify.failed ?? false);
   const [notifyCompleted, setNotifyCompleted] = useState(initNotify.completed ?? true);
-  const [notifyCommented, setNotifyCommented] = useState(initNotify.commented ?? true);
+  const [notifyCommented, setNotifyCommented] = useState(initNotify.commented ?? false);
   const [notifyNewTask, setNotifyNewTask] = useState(initNotify.new_task ?? true);
-  const [notifySlowTool, setNotifySlowTool] = useState(initNotify.slow_tool ?? true);
+  const [notifySlowTool, setNotifySlowTool] = useState(initNotify.slow_tool ?? false);
   const [notifyMode, setNotifyMode] = useState<"direct" | "leader">(initNotify.mode ?? "direct");
   const initMemberRequests = initial.member_requests ?? {};
   const [memberRequestsEnabled, setMemberRequestsEnabled] = useState(initMemberRequests.enabled ?? false);
@@ -105,12 +105,12 @@ export function TeamSettingsTab({ teamId, team, onSaved }: TeamSettingsTabProps)
     setDenyChannels(s.deny_channels ?? []);
     const sn = s.notifications ?? {};
     setNotifyDispatched(sn.dispatched ?? true);
-    setNotifyProgress(sn.progress ?? true);
-    setNotifyFailed(sn.failed ?? true);
+    setNotifyProgress(sn.progress ?? false);
+    setNotifyFailed(sn.failed ?? false);
     setNotifyCompleted(sn.completed ?? true);
-    setNotifyCommented(sn.commented ?? true);
+    setNotifyCommented(sn.commented ?? false);
     setNotifyNewTask(sn.new_task ?? true);
-    setNotifySlowTool(sn.slow_tool ?? true);
+    setNotifySlowTool(sn.slow_tool ?? false);
     setNotifyMode(sn.mode ?? "direct");
     const smr = s.member_requests ?? {};
     setMemberRequestsEnabled(smr.enabled ?? false);