refactor(docql): rename SmartSearchStrategy to SearchStrategy

phodal · phodal · commit 706202413ce1 · 2025-11-28T10:29:06.000+08:00
Rename interface and implementations for clarity and consistency.
diff --git a/mpp-core/src/commonMain/kotlin/cc/unitmesh/agent/tool/impl/docql/SearchStrategy.kt b/mpp-core/src/commonMain/kotlin/cc/unitmesh/agent/tool/impl/docql/SearchStrategy.kt
@@ -6,7 +6,7 @@ import cc.unitmesh.agent.scoring.ExpandedKeywords
 import cc.unitmesh.agent.scoring.KeywordExpander
 import cc.unitmesh.agent.tool.ToolResult
 
-interface SmartSearchStrategy {
+interface SearchStrategy {
     suspend fun execute(context: SmartSearchContext): ToolResult
 }
 
@@ -37,7 +37,7 @@ data class SearchLevelResult(
     val activeChannels: List<String>
 )
 
-class KeepStrategy : SmartSearchStrategy {
+class KeepStrategy : SearchStrategy {
     override suspend fun execute(context: SmartSearchContext): ToolResult {
         return context.resultBuilder(
             context.level1Results,
@@ -55,7 +55,7 @@ class KeepStrategy : SmartSearchStrategy {
     }
 }
 
-class FilterStrategy : SmartSearchStrategy {
+class FilterStrategy : SearchStrategy {
     override suspend fun execute(context: SmartSearchContext): ToolResult {
         val filterKeywords = if (context.secondaryKeyword != null) {
             listOf(context.secondaryKeyword) + context.expandedKeywords.secondary.take(3)
@@ -135,7 +135,7 @@ class FilterStrategy : SmartSearchStrategy {
     }
 }
 
-class ExpandStrategy : SmartSearchStrategy {
+class ExpandStrategy : SearchStrategy {
     override suspend fun execute(context: SmartSearchContext): ToolResult {
         // Too few results, expand to Level 2
         val level2Results = context.searchExecutor(

Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,7 @@ import cc.unitmesh.agent.scoring.ExpandedKeywords`
`6`	`6`	`import cc.unitmesh.agent.scoring.KeywordExpander`
`7`	`7`	`import cc.unitmesh.agent.tool.ToolResult`
`8`	`8`
`9`		`-interface SmartSearchStrategy {`
	`9`	`+interface SearchStrategy {`
`10`	`10`	`suspend fun execute(context: SmartSearchContext): ToolResult`
`11`	`11`	`}`
`12`	`12`
`@@ -37,7 +37,7 @@ data class SearchLevelResult(`
`37`	`37`	`val activeChannels: List<String>`
`38`	`38`	`)`
`39`	`39`
`40`		`-class KeepStrategy : SmartSearchStrategy {`
	`40`	`+class KeepStrategy : SearchStrategy {`
`41`	`41`	`override suspend fun execute(context: SmartSearchContext): ToolResult {`
`42`	`42`	`return context.resultBuilder(`
`43`	`43`	`context.level1Results,`
`@@ -55,7 +55,7 @@ class KeepStrategy : SmartSearchStrategy {`
`55`	`55`	`}`
`56`	`56`	`}`
`57`	`57`
`58`		`-class FilterStrategy : SmartSearchStrategy {`
	`58`	`+class FilterStrategy : SearchStrategy {`
`59`	`59`	`override suspend fun execute(context: SmartSearchContext): ToolResult {`
`60`	`60`	`val filterKeywords = if (context.secondaryKeyword != null) {`
`61`	`61`	`listOf(context.secondaryKeyword) + context.expandedKeywords.secondary.take(3)`
`@@ -135,7 +135,7 @@ class FilterStrategy : SmartSearchStrategy {`
`135`	`135`	`}`
`136`	`136`	`}`
`137`	`137`
`138`		`-class ExpandStrategy : SmartSearchStrategy {`
	`138`	`+class ExpandStrategy : SearchStrategy {`
`139`	`139`	`override suspend fun execute(context: SmartSearchContext): ToolResult {`
`140`	`140`	`// Too few results, expand to Level 2`
`141`	`141`	`val level2Results = context.searchExecutor(`