Skip to content

Commit 6fd87fb

Browse files
committed
refactor(context): remove @NotNull annotations and add error handling #324
- Remove redundant @NotNull annotations from MethodContextProvider. - Add try-catch block in ClassContext to handle potential exceptions when retrieving method signatures.
1 parent 668b098 commit 6fd87fb

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

core/src/main/kotlin/cc/unitmesh/devti/context/ClassContext.kt

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,11 @@ class ClassContext(
2222
}
2323

2424
private fun getMethodSignatures(): List<String> = methods.mapNotNull {
25-
MethodContextProvider(false, gatherUsages = false).from(it).signature
25+
try {
26+
MethodContextProvider(false, gatherUsages = false).from(it).signature
27+
} catch (e: Exception) {
28+
null
29+
}
2630
}
2731

2832
override fun format(): String {

core/src/main/kotlin/cc/unitmesh/devti/context/MethodContextProvider.kt

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,24 +5,20 @@ import cc.unitmesh.devti.context.builder.MethodContextBuilder
55
import com.intellij.lang.Language
66
import com.intellij.lang.LanguageExtension
77
import com.intellij.psi.PsiElement
8-
import org.jetbrains.annotations.NotNull
98

109
class MethodContextProvider(private val includeClassContext: Boolean, private val gatherUsages: Boolean):
1110
LLMCodeContextProvider<PsiElement> {
12-
@NotNull
1311
private val languageExtension: LanguageExtension<MethodContextBuilder> =
1412
LanguageExtension("cc.unitmesh.methodContextBuilder")
1513

16-
@NotNull
1714
private var providers: List<MethodContextBuilder> = emptyList()
1815

1916
init {
2017
val registeredLanguages = Language.getRegisteredLanguages()
2118
providers = registeredLanguages.mapNotNull(languageExtension::forLanguage)
2219
}
2320

24-
@NotNull
25-
override fun from(@NotNull psiElement: PsiElement): MethodContext {
21+
override fun from(psiElement: PsiElement): MethodContext {
2622
val iterator = providers.iterator()
2723
while (iterator.hasNext()) {
2824
val provider = iterator.next()

0 commit comments

Comments
 (0)