From f9e9ad5f4be6e3d76eee2b94c1217cf55639c7cb Mon Sep 17 00:00:00 2001
From: mruwnik <ai@ahiru.pl>
Date: Sun, 21 Dec 2025 16:22:22 +0000
Subject: [PATCH] Fix search enhancement UI to show actual server defaults
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Initialize checkboxes to match server defaults (BM25, HyDE, Reranking
enabled; Query Analysis disabled) so the UI accurately reflects what
the server is actually doing. Previously showed all as unchecked which
was confusing.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 frontend/src/components/search/SearchForm.tsx | 27 ++++++++++---------
 1 file changed, 14 insertions(+), 13 deletions(-)

diff --git a/frontend/src/components/search/SearchForm.tsx b/frontend/src/components/search/SearchForm.tsx
index 4bf3737..4eeae93 100644
--- a/frontend/src/components/search/SearchForm.tsx
+++ b/frontend/src/components/search/SearchForm.tsx
@@ -56,11 +56,12 @@ export const SearchForm = ({ isLoading, onSearch }: SearchFormProps) => {
     const [tags, setTags] = useState<Record<string, boolean>>({})
     const [dynamicFilters, setDynamicFilters] = useState<Record<string, any>>({})
     const [limit, setLimit] = useState(10)
-    // Search enhancement options (undefined = use server defaults)
-    const [useBm25, setUseBm25] = useState<boolean | undefined>(undefined)
-    const [useHyde, setUseHyde] = useState<boolean | undefined>(undefined)
-    const [useReranking, setUseReranking] = useState<boolean | undefined>(undefined)
-    const [useQueryAnalysis, setUseQueryAnalysis] = useState<boolean | undefined>(undefined)
+    // Search enhancement options - initialize to match server defaults
+    // Server defaults: BM25=true, HyDE=true, Reranking=true, QueryAnalysis=false
+    const [useBm25, setUseBm25] = useState<boolean | undefined>(true)
+    const [useHyde, setUseHyde] = useState<boolean | undefined>(true)
+    const [useReranking, setUseReranking] = useState<boolean | undefined>(true)
+    const [useQueryAnalysis, setUseQueryAnalysis] = useState<boolean | undefined>(false)
     const { getMetadataSchemas, getTags } = useMCP()
     
     useEffect(() => {
@@ -146,40 +147,40 @@ export const SearchForm = ({ isLoading, onSearch }: SearchFormProps) => {
                             <label>
                                 <input
                                     type="checkbox"
-                                    checked={useBm25 ?? false}
+                                    checked={useBm25}
                                     onChange={(e) => setUseBm25(e.target.checked)}
                                 />
-                                Enable BM25 (keyword search)
+                                BM25 keyword search
                             </label>
                         </div>
                         <div className="search-option">
                             <label>
                                 <input
                                     type="checkbox"
-                                    checked={useHyde ?? false}
+                                    checked={useHyde}
                                     onChange={(e) => setUseHyde(e.target.checked)}
                                 />
-                                Enable HyDE (hypothetical document expansion)
+                                HyDE (hypothetical document expansion)
                             </label>
                         </div>
                         <div className="search-option">
                             <label>
                                 <input
                                     type="checkbox"
-                                    checked={useReranking ?? false}
+                                    checked={useReranking}
                                     onChange={(e) => setUseReranking(e.target.checked)}
                                 />
-                                Enable reranking (cross-encoder)
+                                Reranking (cross-encoder)
                             </label>
                         </div>
                         <div className="search-option">
                             <label>
                                 <input
                                     type="checkbox"
-                                    checked={useQueryAnalysis ?? false}
+                                    checked={useQueryAnalysis}
                                     onChange={(e) => setUseQueryAnalysis(e.target.checked)}
                                 />
-                                Enable query analysis (LLM-based)
+                                Query analysis (LLM-based)
                             </label>
                         </div>
                     </div>