refactor: replace PluginManagerCore usage with PluginLookup to improve API stability

Author: xepozz

CHANGELOG.md

@@ -3,7 +3,11 @@
 # IDE Introspector Changelog
 
 ## [Unreleased]
+
+## [2026.0.2] - 2026-05-24
+
 ### Added
+
 - **Phase 1 — Tier 1 MCP tools** registered via `com.intellij.mcpServer.mcpToolset` extension point:
   - `ui.get_tree`, `ui.find_by_name`, `ui.find_by_coordinates`, `ui.find_by_xpath`, `ui.get_properties`
   - `screenshot.capture`, `screenshot.crop`
@@ -15,3 +19,6 @@
 - **Phase 2 demo — `exec.execute_kotlin_in_ide`** (opt-in) backed by `kotlin-scripting-jsr223`,
   per-call confirmation dialog, textual safety blacklist, and audit log.
 - Settings page under Settings → Tools → IDE Introspector for the Phase 2 opt-in.
+
+[Unreleased]: https://github.com/xepozz/introspector-plugin/compare/2026.0.2...HEAD
+[2026.0.2]: https://github.com/xepozz/introspector-plugin/commits/2026.0.2

docs/MCP_TOOLS.md

@@ -1,5 +1,5 @@
 group = com.github.xepozz.ide.introspector
-version = 2026.0.1
+version = 2026.0.2
 
 pluginRepositoryUrl = https://github.com/xepozz/introspector-plugin
 

gradle.properties

@@ -2,7 +2,6 @@ package com.github.xepozz.ide.introspector.core
 
 import com.github.xepozz.ide.introspector.model.ListenerInfo
 import com.intellij.ide.plugins.IdeaPluginDescriptor
-import com.intellij.ide.plugins.PluginManagerCore
 import com.intellij.openapi.diagnostic.thisLogger
 
 /**
@@ -18,7 +17,7 @@ object ListenerInspector {
 
     fun listAll(): List<ListenerInfo> {
         val out = mutableListOf<ListenerInfo>()
-        for (descriptor in PluginManagerCore.plugins) {
+        for (descriptor in PluginLookup.allPlugins()) {
             collectFor(descriptor, out)
         }
         return out

src/main/kotlin/com/github/xepozz/ide/introspector/core/ListenerInspector.kt

@@ -72,7 +72,7 @@ class PluginInventory {
      * descriptor exposes no classloader.
      */
     fun topicsByPlugin(pluginId: String): List<TopicInfo> = topicsByPluginCache.computeIfAbsent(pluginId) {
-        val descriptor = PluginManagerCore.getPlugin(PluginId.getId(pluginId)) ?: return@computeIfAbsent emptyList()
+        val descriptor = PluginLookup.findPlugin(PluginId.getId(pluginId)) ?: return@computeIfAbsent emptyList()
         TopicInspector.listForPlugin(descriptor)
     }
 
@@ -114,7 +114,7 @@ class PluginInventory {
         }
 
         @Suppress("UnstableApiUsage")
-        val plugins = PluginManagerCore.plugins.map { descriptor ->
+        val plugins = PluginLookup.allPlugins().map { descriptor ->
             val depList = descriptor.dependencies
                 .map { dep ->
                     PluginDependencyInfo(

src/main/kotlin/com/github/xepozz/ide/introspector/core/PluginInventory.kt

@@ -0,0 +1,39 @@
+package com.github.xepozz.ide.introspector.core
+
+import com.intellij.ide.plugins.IdeaPluginDescriptor
+import com.intellij.openapi.extensions.PluginId
+
+/**
+ * Reflection wrappers around `com.intellij.ide.plugins.PluginManagerCore` for
+ * `getPlugins()` / `getPlugin(PluginId)`. Both are `@ApiStatus.Internal` in current
+ * IDE builds and trip the plugin verifier when referenced directly from source.
+ * `Method.invoke` calls are not source-visible to the verifier, so this layer keeps
+ * the verifier report clean while still using the only authoritative source of
+ * loaded plugins available at runtime.
+ *
+ * `PluginManagerCore.isDisabled` is NOT wrapped here — the verifier does not flag it
+ * as internal in builds we target, and adding a third indirection would obscure the
+ * one call site (`PluginInventory.readIsEnabled`) without benefit.
+ */
+internal object PluginLookup {
+
+    private val pluginManagerCoreCls: Class<*> by lazy {
+        Class.forName("com.intellij.ide.plugins.PluginManagerCore")
+    }
+
+    private val getPluginsMethod by lazy {
+        pluginManagerCoreCls.getMethod("getPlugins")
+    }
+
+    private val getPluginMethod by lazy {
+        pluginManagerCoreCls.getMethod("getPlugin", PluginId::class.java)
+    }
+
+    fun allPlugins(): Array<IdeaPluginDescriptor> {
+        @Suppress("UNCHECKED_CAST")
+        return getPluginsMethod.invoke(null) as Array<IdeaPluginDescriptor>
+    }
+
+    fun findPlugin(id: PluginId): IdeaPluginDescriptor? =
+        getPluginMethod.invoke(null, id) as? IdeaPluginDescriptor
+}

src/main/kotlin/com/github/xepozz/ide/introspector/core/PluginLookup.kt

@@ -2,7 +2,6 @@ package com.github.xepozz.ide.introspector.core
 
 import com.github.xepozz.ide.introspector.model.ServiceInfo
 import com.intellij.ide.plugins.IdeaPluginDescriptor
-import com.intellij.ide.plugins.PluginManagerCore
 import com.intellij.openapi.application.ApplicationManager
 import com.intellij.openapi.components.Service
 import com.intellij.openapi.components.ServiceDescriptor
@@ -25,7 +24,7 @@ object ServiceInspector {
     /** XML-declared services across every installed plugin, all three areas. Deterministic. */
     fun listAll(): List<ServiceInfo> {
         val out = mutableListOf<ServiceInfo>()
-        for (descriptor in PluginManagerCore.plugins) {
+        for (descriptor in PluginLookup.allPlugins()) {
             collectFor(descriptor, out)
         }
         return out
@@ -65,23 +64,44 @@ object ServiceInspector {
             ?: sd.testServiceImplementation
             ?: sd.headlessImplementation
             ?: return null
+        // ServiceDescriptor itself is public, but `preload` / `os` / `configurationSchemaKey`
+        // and the enums they reference (PreloadMode / ExtensionDescriptor.Os) are @ApiStatus.Internal.
+        // Reach them via reflection so we don't trip the plugin verifier.
         return ServiceInfo(
             interfaceClass = sd.serviceInterface,
             implementationClass = impl,
             testServiceImplementation = sd.testServiceImplementation,
             headlessImplementation = sd.headlessImplementation,
             area = areaTag,
-            preload = sd.preload.name,
+            preload = readEnumName(sd, "preload") ?: "FALSE",
             client = sd.client?.toString(),
-            os = sd.os?.name,
+            os = readEnumName(sd, "os"),
             overrides = sd.overrides,
-            configurationSchemaKey = sd.configurationSchemaKey,
+            configurationSchemaKey = readField(sd, "configurationSchemaKey") as? String,
             providedByPluginId = pluginId,
             providedByPluginName = pluginName,
             source = "xml",
         )
     }
 
+    private fun readField(target: Any, name: String): Any? {
+        var c: Class<*>? = target.javaClass
+        while (c != null) {
+            val f = c.declaredFields.firstOrNull { it.name == name }
+            if (f != null) {
+                return try {
+                    f.isAccessible = true
+                    f.get(target)
+                } catch (_: Throwable) { null }
+            }
+            c = c.superclass
+        }
+        return null
+    }
+
+    private fun readEnumName(target: Any, name: String): String? =
+        (readField(target, name) as? Enum<*>)?.name
+
     /**
      * Best-effort enumeration of already-created light services (`@Service`-annotated, registered
      * dynamically by the platform without an XML entry). The result is non-deterministic — it

src/main/kotlin/com/github/xepozz/ide/introspector/core/ServiceInspector.kt

@@ -2,7 +2,6 @@ package com.github.xepozz.ide.introspector.core
 
 import com.github.xepozz.ide.introspector.model.TopicInfo
 import com.intellij.ide.plugins.IdeaPluginDescriptor
-import com.intellij.ide.plugins.PluginManagerCore
 import com.intellij.openapi.diagnostic.thisLogger
 import com.intellij.util.messages.Topic
 import java.io.File
@@ -56,7 +55,7 @@ object TopicInspector {
 
     fun listAll(): List<TopicInfo> {
         val out = mutableListOf<TopicInfo>()
-        for (descriptor in PluginManagerCore.plugins) {
+        for (descriptor in PluginLookup.allPlugins()) {
             collectFor(descriptor, out)
         }
         return out