fix(jetbrains): avoid crash when vfs kind service is unresolvable during plugin unload

[kirillk]

What

KiloVirtualFile.kind() used service<KiloVirtualFileKindRegistry>(), which throws a RuntimeException when the application service can't be resolved. Switched it to serviceOrNull<KiloVirtualFileKindRegistry>(), which returns null in that case.

Why

Reported crash:

java.lang.RuntimeException: Cannot find service ai.kilocode.client.vfs.KiloVirtualFileKindRegistry
    (classloader=PluginClassLoader(... state=unload in progress ...))
    at ai.kilocode.client.vfs.KiloVirtualFile.kind(KiloVirtualFile.kt:62)
    at ai.kilocode.client.vfs.KiloVirtualFile.isValid(KiloVirtualFile.kt:32)
    at com.intellij.openapi.fileEditor.impl.EditorHistoryManager.updateHistoryEntry(EditorHistoryManager.kt:213)
    ...

EditorHistoryManager can still hold a reference to a KiloVirtualFile (from previously opened editor history) and calls isValid() on it during a selection-changed event. If this happens while the Kilo Code plugin's classloader is mid-unload (e.g. plugin update/reload), the KiloVirtualFileKindRegistry light service is no longer resolvable, and service<T>() throws instead of returning null, crashing on the EDT.

serviceOrNull<T>() returns null in that situation, so kind() returns null and isValid() correctly reports false — which is also the semantically correct answer for a file backed by an unloading plugin.

Other service<KiloVirtualFileKindRegistry>() call sites (KiloVirtualFileSystem, KiloFileIconProvider, KiloEditorKindRegistry) are only reached through active path resolution/registration while the plugin is loaded, not through stale references surviving into an unload window, so they were left unchanged.

Testing

• ./gradlew typecheck from packages/kilo-jetbrains/
• ./gradlew :frontend:test --tests "*vfs*" from packages/kilo-jetbrains/

[kilo-code-bot]

Code Review Summary

Status: No Issues Found | Recommendation: Merge

The fix is correct and well-scoped: serviceOrNull<KiloVirtualFileKindRegistry>()?.get(...) avoids the RuntimeException from service<T>() during classloader unload, and returning null from kind() correctly yields isValid() == false for a file backed by an unloading plugin. The registry is instantiated at kind-registration time, so normal operation is unaffected. Changeset is present and user-facing.

Files Reviewed (2 files)

• .changeset/fix-jetbrains-vfs-unload-crash.md
• packages/kilo-jetbrains/frontend/src/main/kotlin/ai/kilocode/client/vfs/KiloVirtualFile.kt

---

_{Reviewed by deepseek-v4.1-flash · Input: 0 · Output: 0 · Cached: 0}

_{Review guidance: REVIEW.md from base branch main}