fix(jcef): use official JCEF builder and handle exceptions #51

Author: phodal

src/main/kotlin/cc/unitmesh/devti/counit/view/WebViewWindow.kt

@@ -13,8 +13,16 @@ import java.awt.Component
  * for custom webview can refs: https://github.com/mucharafal/jcef_example
  */
 class WebViewWindow {
-    private val browser: JBCefBrowser = JBCefBrowser().also {
+    // offical doc: https://plugins.jetbrains.com/docs/intellij/jcef.html#executing-javascript
+    private val browser: JBCefBrowser
 
+    init {
+        browser = try {
+            JBCefBrowser.createBuilder()
+                .build()
+        } catch (e: Exception) {
+            JBCefBrowser()
+        }
     }
 
     val component: Component = browser.component

src/main/kotlin/cc/unitmesh/devti/gui/chat/AutoDevInputSection.kt

@@ -34,7 +34,6 @@ import com.intellij.util.ui.JBEmptyBorder
 import com.intellij.util.ui.JBUI
 import com.intellij.util.ui.UIUtil
 import com.intellij.util.ui.components.BorderLayoutPanel
-import kotlinx.serialization.decodeFromString
 import kotlinx.serialization.json.Json
 import java.awt.Color
 import java.awt.Component
@@ -147,7 +146,7 @@ class AutoDevInputSection(private val project: Project, val disposable: Disposab
             }
         })
 
-        tokenizer = TokenizerImpl.INSTANCESupplier.get()
+        tokenizer = TokenizerImpl.INSTANCE
     }
 
 

src/main/kotlin/cc/unitmesh/devti/llms/tokenizer/TokenizerImpl.kt

@@ -1,13 +1,11 @@
 package cc.unitmesh.devti.llms.tokenizer
 
 import com.intellij.openapi.application.ApplicationManager
-import com.intellij.openapi.application.CachedSingletonsRegistry
 import com.intellij.openapi.components.Service
 import com.knuddels.jtokkit.Encodings
 import com.knuddels.jtokkit.api.Encoding
 import com.knuddels.jtokkit.api.EncodingRegistry
 import com.knuddels.jtokkit.api.EncodingType
-import java.util.function.Supplier
 
 @Service(Service.Level.APP)
 class TokenizerImpl(private val maxTokenLength: Int = 8192) : Tokenizer {
@@ -22,7 +20,7 @@ class TokenizerImpl(private val maxTokenLength: Int = 8192) : Tokenizer {
     }
 
     companion object {
-        val INSTANCESupplier: Supplier<TokenizerImpl> =
-            CachedSingletonsRegistry.lazy { ApplicationManager.getApplication().getService(TokenizerImpl::class.java) }
+        val INSTANCE =
+            ApplicationManager.getApplication().getService(TokenizerImpl::class.java)
     }
 }

src/main/kotlin/cc/unitmesh/devti/provider/PromptStrategy.kt

@@ -26,7 +26,7 @@ abstract class PromptStrategy : LazyExtensionInstance<PromptStrategy>() {
     @Attribute("implementationClass")
     var implementationClass: String? = null
 
-    private val tokenizer: Tokenizer = TokenizerImpl.INSTANCESupplier.get()
+    private val tokenizer: Tokenizer = TokenizerImpl.INSTANCE.get()
 
     override fun getImplementationClassName(): String? = implementationClass
 

src/main/kotlin/com/intellij/temporary/error/ErrorMessageProcessor.kt

@@ -71,7 +71,7 @@ object ErrorMessageProcessor {
             extractErrorPlaces(project, description.consoleLineFrom, description.consoleLineTo, description.editor)
 
         val errorPromptBuilder =
-            ErrorPromptBuilder(AutoDevSettingsState.maxTokenLength, TokenizerImpl.INSTANCESupplier.get())
+            ErrorPromptBuilder(AutoDevSettingsState.maxTokenLength, TokenizerImpl.INSTANCE)
         return errorPromptBuilder.buildPrompt(extractedText, extractedErrorPlaces)
     }