refactor: Refactor SPI BeanProvider logic (#1208)

Author: tangcent

idea-plugin/src/main/kotlin/com/itangcent/ai/AIService.kt

@@ -1,8 +1,13 @@
 package com.itangcent.ai
 
+import com.google.inject.ProvidedBy
+import com.google.inject.Singleton
+import com.itangcent.spi.SpiSingleBeanProvider
+
 /**
  * Interface for AI service operations
  */
+@ProvidedBy(AIServiceProvider::class)
 interface AIService {
     /**
      * Sends a prompt to the AI service and returns the response
@@ -27,3 +32,5 @@ interface AIService {
     }
 }
 
+@Singleton
+class AIServiceProvider : SpiSingleBeanProvider<AIService>()

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/actions/YapiExportAction.kt

@@ -29,8 +29,6 @@ class YapiExportAction : ApiExportAction("Export Yapi") {
         builder.bindInstance(ExportChannel::class, ExportChannel.of("yapi"))
         builder.bindInstance(ExportDoc::class, ExportDoc.of("request", "methodDoc"))
 
-        builder.bind(MethodDocBuilderListener::class) { it.with(CompositeMethodDocBuilderListener::class).singleton() }
-
         builder.bind(MethodFilter::class) { it.with(ConfigurableMethodFilter::class).singleton() }
 
         builder.bindInstance("file.save.default", "yapi.json")

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/api/export/core/CompositeMethodDocBuilderListener.kt

@@ -1,41 +1,59 @@
 package com.itangcent.idea.plugin.api.export.core
 
-import com.google.inject.ImplementedBy
+import com.google.inject.ProvidedBy
+import com.google.inject.Singleton
 import com.itangcent.common.model.MethodDoc
 import com.itangcent.common.model.Param
-import com.itangcent.common.model.Request
+import com.itangcent.spi.SpiCompositeBeanProvider
 
-@ImplementedBy(DefaultMethodDocBuilderListener::class)
+@ProvidedBy(MethodDocBuilderListenerCompositeProvider::class)
 interface MethodDocBuilderListener {
 
-    fun setName(exportContext: ExportContext,
-                methodDoc: MethodDoc, name: String)
+    fun setName(
+        exportContext: ExportContext,
+        methodDoc: MethodDoc, name: String
+    )
 
-    fun appendDesc(exportContext: ExportContext,
-                   methodDoc: MethodDoc, desc: String?)
+    fun appendDesc(
+        exportContext: ExportContext,
+        methodDoc: MethodDoc, desc: String?
+    )
 
-    fun addParam(exportContext: ExportContext,
-                 methodDoc: MethodDoc, param: Param)
+    fun addParam(
+        exportContext: ExportContext,
+        methodDoc: MethodDoc, param: Param
+    )
 
-    fun setRet(exportContext: ExportContext,
-               methodDoc: MethodDoc, ret: Any?)
+    fun setRet(
+        exportContext: ExportContext,
+        methodDoc: MethodDoc, ret: Any?
+    )
 
-    fun appendRetDesc(exportContext: ExportContext,
-                      methodDoc: MethodDoc, retDesc: String?)
+    fun appendRetDesc(
+        exportContext: ExportContext,
+        methodDoc: MethodDoc, retDesc: String?
+    )
 
     fun startProcessMethod(methodExportContext: MethodExportContext, methodDoc: MethodDoc)
 
     fun processCompleted(methodExportContext: MethodExportContext, methodDoc: MethodDoc)
 }
 
+@Singleton
+class MethodDocBuilderListenerCompositeProvider : SpiCompositeBeanProvider<MethodDocBuilderListener>()
+
 //region utils------------------------------------------------------------------
-fun MethodDocBuilderListener.addParam(exportContext: ExportContext,
-                                      methodDoc: MethodDoc, paramName: String, value: Any?, desc: String?, required: Boolean) {
+fun MethodDocBuilderListener.addParam(
+    exportContext: ExportContext,
+    methodDoc: MethodDoc, paramName: String, value: Any?, desc: String?, required: Boolean
+) {
     addParam(exportContext, methodDoc, paramName, value, required, desc)
 }
 
-fun MethodDocBuilderListener.addParam(exportContext: ExportContext,
-                                      methodDoc: MethodDoc, paramName: String, value: Any?, required: Boolean, desc: String?) {
+fun MethodDocBuilderListener.addParam(
+    exportContext: ExportContext,
+    methodDoc: MethodDoc, paramName: String, value: Any?, required: Boolean, desc: String?
+) {
     val param = Param()
     param.name = paramName
     param.value = value

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/api/export/core/CompositeRequestBuilderListener.kt

@@ -1,9 +1,11 @@
 package com.itangcent.idea.plugin.api.export.core
 
-import com.google.inject.ImplementedBy
+import com.google.inject.ProvidedBy
+import com.google.inject.Singleton
 import com.itangcent.common.model.*
+import com.itangcent.spi.SpiCompositeBeanProvider
 
-@ImplementedBy(CompositeRequestBuilderListener::class)
+@ProvidedBy(RequestBuilderListenerCompositeProvider::class)
 interface RequestBuilderListener {
 
     fun setName(
@@ -128,6 +130,9 @@ interface RequestBuilderListener {
     fun processCompleted(methodExportContext: MethodExportContext, request: Request)
 }
 
+@Singleton
+class RequestBuilderListenerCompositeProvider : SpiCompositeBeanProvider<RequestBuilderListener>()
+
 //region utils------------------------------------------------------------------
 
 fun RequestBuilderListener.addParam(

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/api/export/core/MethodDocBuilderListener.kt

@@ -1,14 +1,20 @@
 package com.itangcent.idea.plugin.api.export.spring
 
-import com.google.inject.ImplementedBy
+import com.google.inject.ProvidedBy
+import com.google.inject.Singleton
 import com.intellij.psi.PsiClass
+import com.itangcent.spi.SpiCompositeBeanProvider
 
 /*
  * This interface defines a contract for resolving whether a given PsiClass
  * has a Spring controller annotation. It is implemented by various classes
  * to provide different strategies for determining the presence of controller annotations.
  */
-@ImplementedBy(DefaultSpringControllerAnnotationResolver::class)
+@ProvidedBy(SpringControllerAnnotationResolverCompositeProvider::class)
 interface SpringControllerAnnotationResolver {
     fun hasControllerAnnotation(psiClass: PsiClass): Boolean
-} 
+}
+
+@Singleton
+class SpringControllerAnnotationResolverCompositeProvider :
+    SpiCompositeBeanProvider<SpringControllerAnnotationResolver>()

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/api/export/core/RequestBuilderListener.kt

@@ -381,10 +381,6 @@ open class SuvApiExporter {
             builder.bindInstance(ExportChannel::class, ExportChannel.of("yapi"))
             builder.bindInstance(ExportDoc::class, ExportDoc.of("request", "methodDoc"))
 
-            builder.bind(MethodDocBuilderListener::class) {
-                it.with(CompositeMethodDocBuilderListener::class).singleton()
-            }
-
             builder.bindInstance("file.save.default", "api.json")
             builder.bindInstance("file.save.last.location.key", "com.itangcent.api.export.path")
 

idea-plugin/src/main/kotlin/com/itangcent/idea/plugin/api/export/spring/DefaultSpringControllerAnnotationResolver.kt

@@ -12,9 +12,7 @@ import com.itangcent.common.model.Request
 import com.itangcent.common.utils.GsonUtils
 import com.itangcent.common.utils.notNullOrEmpty
 import com.itangcent.idea.plugin.settings.helper.AISettingsHelper
-import com.itangcent.intellij.context.ActionContext
 import com.itangcent.intellij.logger.Logger
-import com.itangcent.spi.SpiCompositeLoader
 import java.util.concurrent.ConcurrentHashMap
 
 /**
@@ -32,11 +30,7 @@ class APITranslationHelper {
     private lateinit var aiSettingsHelper: AISettingsHelper
 
     @Inject
-    private lateinit var actionContext: ActionContext
-
-    private val aiService: AIService by lazy {
-        SpiCompositeLoader.load<AIService>(actionContext).first()
-    }
+    private lateinit var aiService: AIService
 
     // Cache for translated content to avoid duplicate translations
     private val translationCache = ConcurrentHashMap<String, String>()