From 5338ffab1274fa0f30fddb8d3dce7a74aa88a8ec Mon Sep 17 00:00:00 2001 From: "Rodrigo Rodriguez (Pragmatismo)" Date: Wed, 15 Apr 2026 10:19:13 -0300 Subject: [PATCH] fix: use continue instead of break on low-relevance KB search results Bug: Using break instead of continue when encountering low-relevance results caused the search to stop prematurely, missing potentially relevant chunks in subsequent results. - Changed break to continue when score < 0.4 in search_single_collection - Changed break to continue when score < 0.4 in search_single_kb - Lowered threshold from 0.5 to 0.4 for consistency This ensures all search results are evaluated, not just those before the first low-relevance result. --- src/core/bot/kb_context.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/src/core/bot/kb_context.rs b/src/core/bot/kb_context.rs index 29092424..d241085e 100644 --- a/src/core/bot/kb_context.rs +++ b/src/core/bot/kb_context.rs @@ -343,10 +343,10 @@ impl KbContextManager { total_tokens += tokens; - if result.score < 0.4 { - debug!("Skipping low-relevance result (score: {})", result.score); - break; - } + if result.score < 0.4 { + debug!("Skipping low-relevance result (score: {})", result.score); + continue; + } } Ok(KbContext { @@ -409,9 +409,9 @@ impl KbContextManager { total_tokens += tokens; - if result.score < 0.5 { + if result.score < 0.4 { debug!("Skipping low-relevance result (score: {})", result.score); - break; + continue; } }