minor fixes to reduce qodana baseline

vldF · vldF · commit 7b3c13cce2bf · 2025-10-24T14:40:10.000+03:00
diff --git a/src/main/kotlin/com/vk/modulite/actions/usages/yaml/FindAllowedSymbolUsagesInModuleAction.kt b/src/main/kotlin/com/vk/modulite/actions/usages/yaml/FindAllowedSymbolUsagesInModuleAction.kt
@@ -1,9 +1,7 @@
 package com.vk.modulite.actions.usages.yaml
 
-import com.intellij.openapi.actionSystem.ActionUpdateThread
 import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.actionSystem.CommonDataKeys
-import com.intellij.util.SlowOperations
 import com.vk.modulite.actions.YamlPsiElementAction
 import com.vk.modulite.modulite.Modulite
 import com.vk.modulite.psi.extensions.files.containingModulite
diff --git a/src/main/kotlin/com/vk/modulite/actions/usages/yaml/FindModuleUsagesInCurrentModuleAction.kt b/src/main/kotlin/com/vk/modulite/actions/usages/yaml/FindModuleUsagesInCurrentModuleAction.kt
@@ -1,10 +1,8 @@
 package com.vk.modulite.actions.usages.yaml
 
-import com.intellij.openapi.actionSystem.ActionUpdateThread
 import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.actionSystem.CommonDataKeys
 import com.intellij.psi.PsiElement
-import com.intellij.util.SlowOperations
 import com.vk.modulite.actions.YamlPsiElementAction
 import com.vk.modulite.actions.panels.ModuliteSymbolDialog
 import com.vk.modulite.actions.usages.base.UsagesBaseFinder
diff --git a/src/main/kotlin/com/vk/modulite/actions/usages/yaml/FindSymbolUsagesInCurrentModuleAction.kt b/src/main/kotlin/com/vk/modulite/actions/usages/yaml/FindSymbolUsagesInCurrentModuleAction.kt
@@ -1,11 +1,8 @@
 package com.vk.modulite.actions.usages.yaml
 
-import com.intellij.openapi.actionSystem.ActionUpdateThread
 import com.intellij.openapi.actionSystem.AnActionEvent
 import com.intellij.openapi.actionSystem.CommonDataKeys
-import com.intellij.util.SlowOperations
 import com.vk.modulite.actions.YamlPsiElementAction
-import com.vk.modulite.modulite.Modulite
 import com.vk.modulite.psi.extensions.files.containingModulite
 import com.vk.modulite.utils.YamlUtils
 import org.jetbrains.yaml.psi.YAMLQuotedText
diff --git a/src/main/kotlin/com/vk/modulite/completion/ModuliteYamlReferenceContributor.kt b/src/main/kotlin/com/vk/modulite/completion/ModuliteYamlReferenceContributor.kt
@@ -317,3 +317,8 @@ class ModuliteYamlReferenceContributor : PsiReferenceContributor() {
         override fun isSoft() = true
     }
 }
+
+fun YAMLPsiElement.references(): Array<PsiReference> {
+    return ModuliteYamlReferenceContributor.PhpPsiReferenceProvider()
+        .getReferencesByElement(this, ProcessingContext())
+}
diff --git a/src/main/kotlin/com/vk/modulite/index/ModuliteFilesIndex.kt b/src/main/kotlin/com/vk/modulite/index/ModuliteFilesIndex.kt
@@ -115,7 +115,7 @@ class ModuliteFilesIndex : FileBasedIndexExtension<String, Modulite>() {
                     val type = input.readInt()
                     val text = stringer.read(input)
 
-                    val kind = SymbolName.Kind.values()[type]
+                    val kind = SymbolName.Kind.entries[type]
                     val name = SymbolName(text, kind = kind)
 
                     public.add(name)
diff --git a/src/main/kotlin/com/vk/modulite/inspections/config/UnknownComposerPackageInspection.kt b/src/main/kotlin/com/vk/modulite/inspections/config/UnknownComposerPackageInspection.kt
@@ -3,7 +3,7 @@ package com.vk.modulite.inspections.config
 import com.intellij.codeInspection.ProblemHighlightType
 import com.intellij.codeInspection.ProblemsHolder
 import com.intellij.psi.PsiElementVisitor
-import com.vk.modulite.completion.ModuliteYamlReferenceContributor.Companion.references
+import com.vk.modulite.completion.references
 import com.vk.modulite.completion.ModuliteYamlReferenceContributor.UnknownComposerPackageReference
 import com.vk.modulite.utils.registerModuliteProblem
 import com.vk.modulite.utils.unquote
diff --git a/src/main/kotlin/com/vk/modulite/inspections/config/UnknownModuliteInspection.kt b/src/main/kotlin/com/vk/modulite/inspections/config/UnknownModuliteInspection.kt
@@ -4,7 +4,7 @@ import com.intellij.codeInspection.InspectionManager
 import com.intellij.codeInspection.ProblemHighlightType
 import com.intellij.codeInspection.ProblemsHolder
 import com.intellij.psi.PsiElementVisitor
-import com.vk.modulite.completion.ModuliteYamlReferenceContributor.Companion.references
+import com.vk.modulite.completion.references
 import com.vk.modulite.psi.ModuliteNamePsi
 import com.vk.modulite.utils.createModuliteProblemDescriptor
 import com.vk.modulite.utils.unquote
diff --git a/src/main/kotlin/com/vk/modulite/inspections/config/UnknownSymbolInspection.kt b/src/main/kotlin/com/vk/modulite/inspections/config/UnknownSymbolInspection.kt
@@ -4,7 +4,7 @@ import com.intellij.codeInspection.ProblemHighlightType
 import com.intellij.codeInspection.ProblemsHolder
 import com.intellij.psi.PsiElementVisitor
 import com.vk.modulite.SymbolName
-import com.vk.modulite.completion.ModuliteYamlReferenceContributor.Companion.references
+import com.vk.modulite.completion.references
 import com.vk.modulite.completion.ModuliteYamlReferenceContributor.PhpUnknownElementReference
 import com.vk.modulite.psi.extensions.yaml.containingModulite
 import com.vk.modulite.utils.YamlUtils.insideDescription
diff --git a/src/main/kotlin/com/vk/modulite/modulite/Modulite.kt b/src/main/kotlin/com/vk/modulite/modulite/Modulite.kt
@@ -133,7 +133,7 @@ data class Modulite(
     // А также все прямые потомки своих родителей и экспортированные
     // подмодули этих потомков.
     fun canUse(other: Modulite): Boolean {
-        // Если переданный модуль является родителем текущего
+        // Если переданный модуль является родителем текущего,
         // то переданный модуль может быть использован всегда.
         if (other.contains(this)) {
             return true
diff --git a/src/main/kotlin/com/vk/modulite/services/ModuliteDependenciesCollector.kt b/src/main/kotlin/com/vk/modulite/services/ModuliteDependenciesCollector.kt
@@ -144,7 +144,6 @@ data class ModuliteDeps(
 }
 
 @Service(Service.Level.PROJECT)
-@Suppress("UnstableApiUsage")
 class ModuliteDependenciesCollector(val project: Project) {
     companion object {
         private val LOG = logger<ModuliteDependenciesCollector>()
@@ -363,7 +362,7 @@ class ModuliteDependenciesCollector(val project: Project) {
                     if (reference is Variable) {
                         // Глобальные переменные не могут содержаться в некотором модуле,
                         // они всегда считаются глобальными и обязательным для их использования
-                        // является их запись в require модуля.
+                        // является запись в require модуля.
                         return false
                     }
 
diff --git a/src/main/kotlin/com/vk/modulite/services/ModuliteSymbolsCollector.kt b/src/main/kotlin/com/vk/modulite/services/ModuliteSymbolsCollector.kt
@@ -22,7 +22,6 @@ import com.vk.modulite.psi.extensions.php.symbolName
  * Собирает все символы определенные в текущей папке и всех подпапках.
  */
 @Service(Service.Level.PROJECT)
-@Suppress("UnstableApiUsage")
 class ModuliteSymbolsCollector(val project: Project) {
     companion object {
         fun getInstance(project: Project) = project.service<ModuliteSymbolsCollector>()
diff --git a/src/test/fixtures/golden/ModuliteAvailability/Compiler/Compiler.php b/src/test/fixtures/golden/ModuliteAvailability/Compiler/Compiler.php
@@ -18,7 +18,7 @@ public function compile(string $filename) {
     $lexer->lex($filename);
 
     // Мы не можем использовать InternalImplFlexLexer, так как @compiler1/lexer/impl2
-    // не жкспортирован из @compiler1/lexer.
+    // не экспортирован из @compiler1/lexer.
     $lexer = new InternalImplFlexLexer();
     //           ^^^^^^^^^^^^^^^^^^^^^
     //           error: restricted to use Compiler\Lexer\Impl2\InternalImplFlexLexer, it belongs to @compiler1/lexer/impl2,
diff --git a/src/test/kotlin/com/vk/modulite/dsl/integrationTestBase.kt b/src/test/kotlin/com/vk/modulite/dsl/integrationTestBase.kt
@@ -224,7 +224,7 @@ open class ElementContextBase(val element: PsiElement, val ctx: BaseFileContext)
         }
 
         val usages = runInEdtAndGet {
-            NonBlockingReadActionImpl.waitForAsyncTaskCompletion();
+            NonBlockingReadActionImpl.waitForAsyncTaskCompletion()
             val startMillis = System.currentTimeMillis()
             var view: UsageView?
             var viewWasInitialized = false
diff --git a/src/test/kotlin/com/vk/modulite/tests/ExportListSortTest.kt b/src/test/kotlin/com/vk/modulite/tests/ExportListSortTest.kt
@@ -21,16 +21,16 @@ class ExportListSortTest : ConfigListTestBase() {
     fun `test add to unsorted list`() = doExportTest(
         "\\foo()",
         listOf(
-            "Foo::\$field",
+            $$"Foo::$field",
             "@mod",
-            "\$Global",
+            $$"$Global",
             "Foo",
             "CONSTANT",
             "Foo::CONSTANT",
             "#vk/pack",
             "#vk/pack2",
             "VK\\Message\\CONSTANT",
-            "\$Global2",
+            $$"$Global2",
             "Foo::method()",
         ),
         listOf(
@@ -39,13 +39,13 @@ class ExportListSortTest : ConfigListTestBase() {
             "#vk/pack2",
             "Foo",
             "Foo::method()",
-            "Foo::\$field",
+            $$"Foo::$field",
             "Foo::CONSTANT",
             "foo()",
             "CONSTANT",
             "VK\\Message\\CONSTANT",
-            "\$Global",
-            "\$Global2",
+            $$"$Global",
+            $$"$Global2",
         ),
     )
 
@@ -56,25 +56,25 @@ class ExportListSortTest : ConfigListTestBase() {
             "#vk/pack",
             "#vk/pack2",
             "Foo",
-            "Foo::\$field",
+            $$"Foo::$field",
             "Foo::CONSTANT",
             "CONSTANT",
             "VK\\Message\\CONSTANT",
-            "\$Global",
-            "\$Global2",
+            $$"$Global",
+            $$"$Global2",
         ),
         listOf(
             "@mod",
             "#vk/pack",
             "#vk/pack2",
             "Foo",
-            "Foo::\$field",
+            $$"Foo::$field",
             "Foo::CONSTANT",
             "foo()",
             "CONSTANT",
             "VK\\Message\\CONSTANT",
-            "\$Global",
-            "\$Global2",
+            $$"$Global",
+            $$"$Global2",
         ),
     )
 
@@ -85,25 +85,25 @@ class ExportListSortTest : ConfigListTestBase() {
             "#vk/pack",
             "#vk/pack2",
             "Foo",
-            "Foo::\$field",
+            $$"Foo::$field",
             "Foo::CONSTANT",
             "foo()",
             "VK\\Message\\CONSTANT",
-            "\$Global",
-            "\$Global2",
+            $$"$Global",
+            $$"$Global2",
         ),
         listOf(
             "@mod",
             "#vk/pack",
             "#vk/pack2",
             "Foo",
-            "Foo::\$field",
+            $$"Foo::$field",
             "Foo::CONSTANT",
             "foo()",
             "CONSTANT",
             "VK\\Message\\CONSTANT",
-            "\$Global",
-            "\$Global2",
+            $$"$Global",
+            $$"$Global2",
         ),
     )
 
@@ -116,7 +116,7 @@ class ExportListSortTest : ConfigListTestBase() {
 
     // Add field do nothing for export list
     fun `test add field`() = doExportTest(
-        "Foo::\$field",
+        $$"Foo::$field",
         listOf(),
         listOf(),
     )
@@ -130,7 +130,7 @@ class ExportListSortTest : ConfigListTestBase() {
 
     // Add global variable do nothing for export list
     fun `test add global`() = doExportTest(
-        "\\\$GlobalVariable",
+        $$"\\$GlobalVariable",
         listOf(),
         listOf(),
     )
diff --git a/src/test/kotlin/com/vk/modulite/tests/RequiresListTest.kt b/src/test/kotlin/com/vk/modulite/tests/RequiresListTest.kt
@@ -7,16 +7,16 @@ class RequiresListTest : ConfigListTestBase() {
     fun `test add to unsorted list`() = doRequiresTest(
         "\\foo()",
         listOf(
-            "\\Foo::\$field",
+            $$"\\Foo::$field",
             "@mod",
-            "\$Global",
+            $$"$Global",
             "\\Foo",
             "\\CONSTANT",
             "\\Foo::CONSTANT",
             "#vk/pack",
             "#vk/pack2",
             "\\VK\\Message\\CONSTANT",
-            "\$Global2",
+            $$"$Global2",
             "\\Foo::method()",
         ),
         listOf(
@@ -25,13 +25,13 @@ class RequiresListTest : ConfigListTestBase() {
             "#vk/pack2",
             "\\Foo",
             "\\Foo::method()",
-            "\\Foo::\$field",
+            $$"\\Foo::$field",
             "\\Foo::CONSTANT",
             "\\foo()",
             "\\CONSTANT",
             "\\VK\\Message\\CONSTANT",
-            "\$Global",
-            "\$Global2",
+            $$"$Global",
+            $$"$Global2",
         ),
     )
-}
+}
diff --git a/src/test/kotlin/com/vk/modulite/tests/SymbolKindTest.kt b/src/test/kotlin/com/vk/modulite/tests/SymbolKindTest.kt
@@ -14,8 +14,8 @@ class SymbolKindTest : BasePlatformTestCase() {
         compareKind("\\Foo", Kind.Class)
         compareKind("\\VK\\Messages\\Foo", Kind.Class)
 
-        compareKind("Foo::\$field", Kind.Field)
-        compareKind("\\Foo::\$field", Kind.Field)
+        compareKind($$"Foo::$field", Kind.Field)
+        compareKind($$"\\Foo::$field", Kind.Field)
 
         compareKind("Foo::method()", Kind.Method)
         compareKind("\\Foo::method()", Kind.Method)
@@ -26,9 +26,9 @@ class SymbolKindTest : BasePlatformTestCase() {
         compareKind("foo()", Kind.Function)
         compareKind("\\foo()", Kind.Function)
 
-        compareKind("\$GlobalVariable", Kind.GlobalVariable)
-        compareKind("\\\$GlobalVariable", Kind.GlobalVariable)
-        compareKind("\\\\\$GlobalVariable", Kind.GlobalVariable)
+        compareKind($$"$GlobalVariable", Kind.GlobalVariable)
+        compareKind($$"\\$GlobalVariable", Kind.GlobalVariable)
+        compareKind($$"\\\\$GlobalVariable", Kind.GlobalVariable)
 
         compareKind("CONSTANT", Kind.Constant)
         compareKind("\\CONSTANT", Kind.Constant)
diff --git a/src/test/kotlin/com/vk/modulite/tests/integration/IntentionsTest.kt b/src/test/kotlin/com/vk/modulite/tests/integration/IntentionsTest.kt
@@ -67,7 +67,7 @@ class IntentionsTest : IntegrationTestBase() {
         step("invoke intention") {
             php("Module/functions.php") {
                 element("module_func") {
-                    // This intention requires input from the user, so we check it directly.
+                    // This intention requires an input from the user, so we check it directly.
                     // runIntention("Allow internal symbol")
                     allowForModulite("module2")
                 }
diff --git a/src/test/kotlin/com/vk/modulite/tests/integration/ModuliteNestedAvailabilityTest.kt b/src/test/kotlin/com/vk/modulite/tests/integration/ModuliteNestedAvailabilityTest.kt
@@ -113,7 +113,7 @@ class ModuliteNestedAvailabilityTest : IntegrationTestBase() {
             modulite("messages/core/impl") canUse modulite("messages/core/impl") shouldBe true
             modulite("messages/core")      canUse modulite("messages/core/impl") shouldBe true
 
-            // Можно использовать несмотря на то что messages/core/impl не экспортируется из
+            // Можно использовать, несмотря на то что messages/core/impl не экспортируется из
             // messages/core, так как есть allow-internal-access и messages/core можно использовать.
             modulite("messages")           canUse modulite("messages/core/impl") shouldBe true
         }
@@ -137,8 +137,8 @@ class ModuliteNestedAvailabilityTest : IntegrationTestBase() {
 
             modulite("messages")           canUse modulite("messages/core") shouldBe true
             modulite("messages")           canUse modulite("messages/core/impl") shouldBe true
-            // Можно использовать несмотря на то что messages/core/impl/deep не экспортируется из
-            // messages/core/impl, так как есть allow-internal-access и messages/core/impl можно использовать
+            // Можно использовать, несмотря на то что messages/core/impl/deep не экспортируется из
+            // messages/core/impl, так как есть allow-internal-access и messages/core/impl можно использовать,
             // так как он экспортируется из messages/core, который является прямым потомком.
             modulite("messages")           canUse modulite("messages/core/impl/deep") shouldBe true
         }

Original file line number	Diff line number	Diff line change
`@@ -317,3 +317,8 @@ class ModuliteYamlReferenceContributor : PsiReferenceContributor() {`
`317`	`317`	`override fun isSoft() = true`
`318`	`318`	`}`
`319`	`319`	`}`
	`320`	`+`
	`321`	`+fun YAMLPsiElement.references(): Array<PsiReference> {`
	`322`	`+ return ModuliteYamlReferenceContributor.PhpPsiReferenceProvider()`
	`323`	`+ .getReferencesByElement(this, ProcessingContext())`
	`324`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -144,7 +144,6 @@ data class ModuliteDeps(`
`144`	`144`	`}`
`145`	`145`
`146`	`146`	`@Service(Service.Level.PROJECT)`
`147`		`-@Suppress("UnstableApiUsage")`
`148`	`147`	`class ModuliteDependenciesCollector(val project: Project) {`
`149`	`148`	`companion object {`
`150`	`149`	`private val LOG = logger<ModuliteDependenciesCollector>()`
`@@ -363,7 +362,7 @@ class ModuliteDependenciesCollector(val project: Project) {`
`363`	`362`	`if (reference is Variable) {`
`364`	`363`	`// Глобальные переменные не могут содержаться в некотором модуле,`
`365`	`364`	`// они всегда считаются глобальными и обязательным для их использования`
`366`		`- // является их запись в require модуля.`
	`365`	`+ // является запись в require модуля.`
`367`	`366`	`return false`
`368`	`367`	`}`
`369`	`368`