From 99110c44965751d4cc40c1e932905a0c9f0bbe03 Mon Sep 17 00:00:00 2001
From: Luan Vu <luanvuseo@gmail.com>
Date: Tue, 10 Mar 2026 16:51:23 +0700
Subject: [PATCH] fix: memory flush UI default should match backend (ON, not
 OFF) (#115)

Backend defaults memoryFlush.enabled to true when config is nil
(memoryflush.go:39-46), but UI used `?? false` showing OFF.
Also fix sub-fields visibility to show when flush is defaulted ON.

Co-authored-by: Luvu182 <208665161+Luvu182@users.noreply.github.com>
---
 .../agent-detail/config-sections/compaction-section.tsx       | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/ui/web/src/pages/agents/agent-detail/config-sections/compaction-section.tsx b/ui/web/src/pages/agents/agent-detail/config-sections/compaction-section.tsx
index 9d8d6248..51e0f106 100644
--- a/ui/web/src/pages/agents/agent-detail/config-sections/compaction-section.tsx
+++ b/ui/web/src/pages/agents/agent-detail/config-sections/compaction-section.tsx
@@ -65,14 +65,14 @@ export function CompactionSection({ enabled, value, onToggle, onChange }: Compac
       <div className="space-y-3">
         <div className="flex items-center gap-2">
           <Switch
-            checked={value.memoryFlush?.enabled ?? false}
+            checked={value.memoryFlush?.enabled ?? true}
             onCheckedChange={(v) =>
               onChange({ ...value, memoryFlush: { ...value.memoryFlush, enabled: v } })
             }
           />
           <InfoLabel tip="When enabled, compacted history is also saved to long-term memory for future retrieval.">{t(`${s}.memoryFlush`)}</InfoLabel>
         </div>
-        {value.memoryFlush?.enabled && (
+        {(value.memoryFlush?.enabled ?? true) && (
           <div className="space-y-2 pl-6">
             <InfoLabel tip="Token count threshold that triggers memory flush. When summary exceeds this, older memories are flushed to storage.">{t(`${s}.softThreshold`)}</InfoLabel>
             <Input