Skip to content

Commit

Permalink
refactor(context): remove @NotNull annotations and add error handling #…
Browse files Browse the repository at this point in the history
…324

- Remove redundant @NotNull annotations from MethodContextProvider.
- Add try-catch block in ClassContext to handle potential exceptions when retrieving method signatures.
  • Loading branch information
phodal committed Mar 7, 2025
1 parent 668b098 commit 6fd87fb
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,11 @@ class ClassContext(
}

private fun getMethodSignatures(): List<String> = methods.mapNotNull {
MethodContextProvider(false, gatherUsages = false).from(it).signature
try {
MethodContextProvider(false, gatherUsages = false).from(it).signature
} catch (e: Exception) {
null
}
}

override fun format(): String {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,24 +5,20 @@ import cc.unitmesh.devti.context.builder.MethodContextBuilder
import com.intellij.lang.Language
import com.intellij.lang.LanguageExtension
import com.intellij.psi.PsiElement
import org.jetbrains.annotations.NotNull

class MethodContextProvider(private val includeClassContext: Boolean, private val gatherUsages: Boolean):
LLMCodeContextProvider<PsiElement> {
@NotNull
private val languageExtension: LanguageExtension<MethodContextBuilder> =
LanguageExtension("cc.unitmesh.methodContextBuilder")

@NotNull
private var providers: List<MethodContextBuilder> = emptyList()

init {
val registeredLanguages = Language.getRegisteredLanguages()
providers = registeredLanguages.mapNotNull(languageExtension::forLanguage)
}

@NotNull
override fun from(@NotNull psiElement: PsiElement): MethodContext {
override fun from(psiElement: PsiElement): MethodContext {
val iterator = providers.iterator()
while (iterator.hasNext()) {
val provider = iterator.next()
Expand Down

0 comments on commit 6fd87fb

Please sign in to comment.