feat: remove background animation due to cpu spike

Author: brucehsu

apps/desktop/src-tauri/Cargo.lock

@@ -126,8 +126,6 @@ struct AppSettings {
   terminal_font_size: u16,
   #[serde(default = "default_shortcut_modifier")]
   shortcut_modifier: String,
-  #[serde(default = "default_battery_saver", alias = "backgroundAnimation")]
-  battery_saver: bool,
   #[serde(default)]
   repo_defaults: HashMap<String, RepoDefaults>,
   #[serde(default)]
@@ -656,17 +654,7 @@ fn load_config() -> Result<AppConfig, String> {
 
   let raw = std::fs::read_to_string(&file)
     .map_err(|error| format!("Failed to read config: {error}"))?;
-  let mut value: serde_json::Value = serde_json::from_str(&raw)
-    .map_err(|error| format!("Failed to parse config: {error}"))?;
-  if let Some(settings) = value.get_mut("settings") {
-    let has_battery = settings.get("batterySaver").is_some();
-    if !has_battery {
-      if let Some(background) = settings.get("backgroundAnimation").and_then(|val| val.as_bool()) {
-        settings["batterySaver"] = serde_json::Value::Bool(!background);
-      }
-    }
-  }
-  let config: AppConfig = serde_json::from_value(value)
+  let config: AppConfig = serde_json::from_str(&raw)
     .map_err(|error| format!("Failed to parse config: {error}"))?;
   Ok(config)
 }
@@ -860,7 +848,6 @@ fn default_config() -> AppConfig {
       terminal_font_family: default_terminal_font_family(),
       terminal_font_size: default_terminal_font_size(),
       shortcut_modifier: default_shortcut_modifier(),
-      battery_saver: default_battery_saver(),
       repo_defaults: HashMap::new(),
       agent_args: HashMap::new(),
     },
@@ -953,10 +940,6 @@ fn default_terminal_font_size() -> u16 {
   13
 }
 
-fn default_battery_saver() -> bool {
-  false
-}
-
 fn default_shortcut_modifier() -> String {
   "Alt".to_string()
 }

apps/desktop/src-tauri/src/lib.rs

@@ -129,7 +129,6 @@ export default function App() {
     setActiveSessionId,
     updateRecentDirs,
     updateTerminalSettings,
-    updateBatterySaver,
     updateShortcutModifier,
     updateRepoDefaults,
     updateAgentSettings,
@@ -158,7 +157,6 @@ export default function App() {
   const [settingsOpen, setSettingsOpen] = useState(false);
   const [fontFamily, setFontFamily] = useState(config.settings.terminalFontFamily);
   const [fontSize, setFontSize] = useState(config.settings.terminalFontSize);
-  const [batterySaver, setBatterySaver] = useState(config.settings.batterySaver);
   const [agentArgs, setAgentArgs] = useState<Record<string, string>>(config.settings.agentArgs ?? {});
   const [renameOpen, setRenameOpen] = useState(false);
   const [renameSessionId, setRenameSessionId] = useState<string | null>(null);
@@ -204,11 +202,9 @@ export default function App() {
   useEffect(() => {
     setFontFamily(config.settings.terminalFontFamily);
     setFontSize(config.settings.terminalFontSize);
-    setBatterySaver(config.settings.batterySaver);
   }, [
     config.settings.terminalFontFamily,
     config.settings.terminalFontSize,
-    config.settings.batterySaver,
   ]);
 
   useEffect(() => {
@@ -453,11 +449,9 @@ export default function App() {
   const openSettings = useCallback(() => {
     setFontFamily(config.settings.terminalFontFamily);
     setFontSize(config.settings.terminalFontSize);
-    setBatterySaver(config.settings.batterySaver);
     setAgentArgs(config.settings.agentArgs ?? {});
     setSettingsOpen(true);
   }, [
-    config.settings.batterySaver,
     config.settings.agentArgs,
     config.settings.terminalFontFamily,
     config.settings.terminalFontSize,
@@ -622,7 +616,6 @@ export default function App() {
       terminalFontFamily: fontFamily.trim() || config.settings.terminalFontFamily,
       terminalFontSize: Number.isNaN(fontSize) ? config.settings.terminalFontSize : fontSize,
     });
-    updateBatterySaver(batterySaver);
     updateAgentSettings(cleanedArgs);
     setAgentArgs(cleanedArgs);
     setSettingsOpen(false);
@@ -886,12 +879,10 @@ export default function App() {
         open={settingsOpen}
         fontFamily={fontFamily}
         fontSize={fontSize}
-        batterySaver={batterySaver}
         shortcutModifier={shortcutModifier}
         agentArgs={agentArgs}
         onChangeFontFamily={setFontFamily}
         onChangeFontSize={setFontSize}
-        onToggleBatterySaver={setBatterySaver}
         onCommitShortcutModifier={handleShortcutModifierCommit}
         onAgentArgsChange={setAgentArgs}
         onClose={closeSettings}

apps/desktop/src/App.tsx

@@ -28,9 +28,6 @@
     radial-gradient(circle at 12% 55%, rgba(34, 197, 94, 0.18), transparent 60%),
     radial-gradient(circle at 65% 12%, rgba(59, 130, 246, 0.16), transparent 62%),
     radial-gradient(circle at 88% 75%, rgba(217, 70, 239, 0.14), transparent 62%);
-  background-size: 200% 200%;
-  animation: gradientDrift 16s ease-in-out infinite alternate;
-  animation-delay: 0s;
 }
 
 .main::after {
@@ -39,10 +36,7 @@
     radial-gradient(circle at 45% 45%, rgba(251, 191, 36, 0.16), transparent 62%),
     radial-gradient(circle at 20% 30%, rgba(248, 113, 113, 0.14), transparent 62%),
     radial-gradient(circle at 82% 35%, rgba(20, 184, 166, 0.12), transparent 62%);
-  background-size: 220% 220%;
   mix-blend-mode: screen;
-  animation: gradientDriftAlt 20.7s ease-in-out infinite alternate;
-  animation-delay: 0s;
 }
 
 .tabPane {
@@ -271,33 +265,6 @@
   display: none;
 }
 
-:global(body[data-battery-saver="on"]) .main::before,
-:global(body[data-battery-saver="on"]) .main::after {
-  animation: none;
-}
-
-@keyframes gradientDrift {
-  0% {
-    background-position: 0% 0%;
-    filter: hue-rotate(0deg);
-  }
-  100% {
-    background-position: 100% 100%;
-    filter: hue-rotate(40deg);
-  }
-}
-
-@keyframes gradientDriftAlt {
-  0% {
-    background-position: 100% 0%;
-    filter: hue-rotate(0deg);
-  }
-  100% {
-    background-position: 0% 100%;
-    filter: hue-rotate(-30deg);
-  }
-}
-
 @keyframes refreshSpin {
   0% {
     transform: rotate(0deg);

apps/desktop/src/components/MainPane/MainPane.module.css

@@ -15,12 +15,10 @@ interface SettingsDialogProps {
   open: boolean;
   fontFamily: string;
   fontSize: number;
-  batterySaver: boolean;
   shortcutModifier: string;
   agentArgs: Record<string, string>;
   onChangeFontFamily: (value: string) => void;
   onChangeFontSize: (value: number) => void;
-  onToggleBatterySaver: (value: boolean) => void;
   onCommitShortcutModifier: (value: string) => void;
   onAgentArgsChange: (next: Record<string, string>) => void;
   onClose: () => void;
@@ -31,12 +29,10 @@ export default function SettingsDialog({
   open,
   fontFamily,
   fontSize,
-  batterySaver,
   shortcutModifier,
   agentArgs,
   onChangeFontFamily,
   onChangeFontSize,
-  onToggleBatterySaver,
   onCommitShortcutModifier,
   onAgentArgsChange,
   onClose,
@@ -242,21 +238,6 @@ export default function SettingsDialog({
             />
             <p className={styles.fieldHint}>Click and press modifiers. Enter to save, Esc to cancel.</p>
           </label>
-          <div className={styles.toggleRow}>
-            <div>
-              <span>Battery saver</span>
-              <p className={styles.toggleHint}>Disable animated background to save battery.</p>
-            </div>
-            <button
-              type="button"
-              className={`${styles.toggle} ${batterySaver ? styles.toggleActive : ""}`}
-              aria-pressed={batterySaver}
-              onClick={() => onToggleBatterySaver(!batterySaver)}
-            >
-              <span className={styles.toggleKnob} />
-            </button>
-          </div>
-
           <div className={styles.section}>
             <div className={styles.sectionHeader}>
               <span className={styles.sectionTitle}>Agents</span>

apps/desktop/src/components/SettingsDialog/SettingsDialog.tsx

@@ -194,7 +194,6 @@ const defaultSettings = {
   terminalFontFamily: '"JetBrains Mono", "SF Mono", "Fira Code", monospace',
   terminalFontSize: 13,
   shortcutModifier: "Alt",
-  batterySaver: false,
   repoDefaults: {},
   agentArgs: {},
 };
@@ -549,10 +548,6 @@ export function useAppState(
     activeSessionRef.current = activeSessionId;
   }, [activeSessionId]);
 
-  const setBatterySaverDataset = useCallback((enabled: boolean) => {
-    document.body.dataset.batterySaver = enabled ? "on" : "off";
-  }, []);
-
   const markConfigReady = useCallback(() => {
     configReadyResolveRef.current?.();
     configReadyResolveRef.current = null;
@@ -616,23 +611,21 @@ export function useAppState(
         setConfig(nextConfig);
         setHasSavedConfig(hasSaved);
         applyTheme(nextConfig.settings.theme);
-        setBatterySaverDataset(nextConfig.settings.batterySaver);
       })
       .catch(() => {
         if (!mounted) {
           return;
         }
         setHasSavedConfig(false);
         applyTheme(defaultSettings.theme);
-        setBatterySaverDataset(false);
       })
       .finally(() => {
         markConfigReady();
       });
     return () => {
       mounted = false;
     };
-  }, [applyTheme, markConfigReady, setBatterySaverDataset]);
+  }, [applyTheme, markConfigReady]);
 
   const persistConfig = useCallback(async () => {
     try {
@@ -759,14 +752,6 @@ export function useAppState(
     [applyTerminalAppearanceToAll, toTerminalAppearance, updateSettings]
   );
 
-  const updateBatterySaver = useCallback((enabled: boolean) => {
-    updateSettings((settings) => ({
-      ...settings,
-      batterySaver: enabled,
-    }));
-    setBatterySaverDataset(enabled);
-  }, [setBatterySaverDataset, updateSettings]);
-
   const updateShortcutModifier = useCallback(
     (modifier: string) => {
       const normalized = normalizeShortcutModifier(modifier);
@@ -2113,7 +2098,6 @@ export function useAppState(
     setActiveSessionId,
     updateRecentDirs,
     updateTerminalSettings,
-    updateBatterySaver,
     updateShortcutModifier,
     updateRepoDefaults,
     updateAgentSettings,

apps/desktop/src/hooks/useAppState.ts

@@ -29,7 +29,6 @@ export interface AppSettings {
   terminalFontFamily: string;
   terminalFontSize: number;
   shortcutModifier: string;
-  batterySaver: boolean;
   repoDefaults?: Record<string, RepoSessionDefaults>;
   agentArgs?: Record<string, string>;
 }