fix: use continue instead of break on low-relevance KB search results
All checks were successful
BotServer CI/CD / build (push) Successful in 4m9s

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.
This commit is contained in:
Rodrigo Rodriguez (Pragmatismo) 2026-04-15 10:19:13 -03:00
parent dd15899ac3
commit 5338ffab12

View file

@ -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;
}
}