🤖 fix: avoid overwriting task settings on load error

Skip debounced auto-save when the initial config fetch fails, so default Task settings never clobber the user's stored config.

Signed-off-by: Thomas Kosiewski <tk@coder.com>

---
_Generated with `codex cli` • Model: `gpt-5.2` • Thinking: `xhigh`_
<!-- mux-attribution: model=gpt-5.2 thinking=xhigh -->

Change-Id: I472af8699b1d377254720929926d7514629903b9

Author: ThomasK33

src/browser/components/Settings/sections/TasksSection.tsx

@@ -12,6 +12,7 @@ export function TasksSection() {
   const { api } = useAPI();
   const [settings, setSettings] = useState<TaskSettings>(DEFAULT_TASK_SETTINGS);
   const [loaded, setLoaded] = useState(false);
+  const [loadFailed, setLoadFailed] = useState(false);
   const [saveError, setSaveError] = useState<string | null>(null);
   const saveTimerRef = useRef<ReturnType<typeof setTimeout> | null>(null);
   const savingRef = useRef(false);
@@ -20,23 +21,27 @@ export function TasksSection() {
     if (!api) return;
 
     setLoaded(false);
+    setLoadFailed(false);
     setSaveError(null);
 
     void api.config
       .getConfig()
       .then((cfg) => {
         setSettings(normalizeTaskSettings(cfg.taskSettings));
+        setLoadFailed(false);
         setLoaded(true);
       })
       .catch((error: unknown) => {
         setSaveError(error instanceof Error ? error.message : String(error));
+        setLoadFailed(true);
         setLoaded(true);
       });
   }, [api]);
 
   useEffect(() => {
     if (!api) return;
     if (!loaded) return;
+    if (loadFailed) return;
     if (savingRef.current) return;
 
     if (saveTimerRef.current) {
@@ -62,7 +67,7 @@ export function TasksSection() {
         saveTimerRef.current = null;
       }
     };
-  }, [api, loaded, settings]);
+  }, [api, loaded, loadFailed, settings]);
 
   const setMaxParallelAgentTasks = (rawValue: string) => {
     const parsed = Number(rawValue);