refactor(windows): migrate from JNA to JNI for native tray and mouse hook

Replace JNA-based Windows implementation with pure JNI, following the same
patterns as Mac and Linux bridges. Add WindowsNativeBridge.kt with external
JNI declarations and jni_bridge.c wrapping the tray.h C API. Rewrite
WindowsTrayManager and WindowsOutsideClickWatcher to use JNI calls instead
of JNA structures and callbacks. Remove all JNA dependencies from build
configuration. Rename DLL output from tray.dll to WinTray.dll. Update
GraalVM reachability metadata and CI workflow for JDK requirement.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: kdroidFilter

.github/workflows/build-natives.yaml

@@ -83,6 +83,12 @@ jobs:
       - name: Checkout code
         uses: actions/checkout@v4
 
+      - name: Set up JDK
+        uses: actions/setup-java@v4
+        with:
+          java-version: '17'
+          distribution: 'temurin'
+
       - name: Build Windows native libraries
         working-directory: src/native/windows
         run: cmd /c build.bat
@@ -92,21 +98,21 @@ jobs:
       - name: Verify Windows natives
         shell: bash
         run: |
-          test -f build/nativeLibs/win32-x86-64/tray.dll
-          test -f build/nativeLibs/win32-arm64/tray.dll
+          test -f build/nativeLibs/win32-x86-64/WinTray.dll
+          test -f build/nativeLibs/win32-arm64/WinTray.dll
           ls -la build/nativeLibs/win32-x86-64/
           ls -la build/nativeLibs/win32-arm64/
 
       - name: Upload Windows x64 library
         uses: actions/upload-artifact@v4
         with:
           name: native-win32-x86-64
-          path: build/nativeLibs/win32-x86-64/tray.dll
+          path: build/nativeLibs/win32-x86-64/WinTray.dll
           retention-days: 1
 
       - name: Upload Windows ARM64 library
         uses: actions/upload-artifact@v4
         with:
           name: native-win32-arm64
-          path: build/nativeLibs/win32-arm64/tray.dll
+          path: build/nativeLibs/win32-arm64/WinTray.dll
           retention-days: 1

build.gradle.kts

@@ -39,11 +39,10 @@ kotlin {
             implementation(compose.foundation)
             implementation(compose.ui)
             implementation(compose.components.resources)
-            implementation(libs.jna)
-            implementation(libs.jna.platform)
             implementation(libs.kotlinx.coroutines.core)
             implementation(libs.kotlinx.coroutines.swing)
             implementation(libs.platformtools.core)
+            implementation(libs.nucleus.core.runtime)
             implementation(libs.nucleus.darkmode.detector)
         }
     }

gradle/libs.versions.toml

@@ -3,7 +3,6 @@ kermit = "2.0.8"
 kotlin = "2.3.10"
 kotlinx-coroutines = "1.10.2"
 compose = "1.10.0"
-jna = "5.18.1"
 platformtools = "0.7.5"
 detekt = "1.23.7"
 ktlint = "12.1.2"
@@ -13,8 +12,6 @@ nucleus = "1.9.0"
 kermit = { module = "co.touchlab:kermit", version.ref = "kermit" }
 kotlinx-coroutines-core = { module = "org.jetbrains.kotlinx:kotlinx-coroutines-core", version.ref = "kotlinx-coroutines" }
 kotlinx-coroutines-swing = { module = "org.jetbrains.kotlinx:kotlinx-coroutines-swing", version.ref = "kotlinx-coroutines" }
-jna = { module = "net.java.dev.jna:jna-jpms", version.ref = "jna" }
-jna-platform = { module = "net.java.dev.jna:jna-platform-jpms", version.ref = "jna" }
 platformtools-core = { module = "io.github.kdroidfilter:platformtools.core", version.ref = "platformtools" }
 nucleus-core-runtime = { module = "io.github.kdroidfilter:nucleus.core-runtime", version.ref = "nucleus" }
 nucleus-darkmode-detector = { module = "io.github.kdroidfilter:nucleus.darkmode-detector", version.ref = "nucleus" }

src/jvmMain/kotlin/com/kdroid/composetray/lib/windows/StdCallCallback.kt

@@ -0,0 +1,57 @@
+package com.kdroid.composetray.lib.windows
+
+import com.kdroid.composetray.utils.NativeLibraryLoader
+
+/**
+ * JNI bridge to the native Windows tray library (WinTray.dll).
+ * Replaces the previous JNA-based approach.
+ * Follows the same patterns as MacNativeBridge and LinuxNativeBridge.
+ */
+internal object WindowsNativeBridge {
+
+    init {
+        NativeLibraryLoader.load("WinTray", WindowsNativeBridge::class.java)
+    }
+
+    // -- Tray lifecycle --
+
+    @JvmStatic external fun nativeCreateTray(iconPath: String, tooltip: String): Long
+    @JvmStatic external fun nativeFreeTray(handle: Long)
+    @JvmStatic external fun nativeSetTrayIcon(handle: Long, iconPath: String)
+    @JvmStatic external fun nativeSetTrayTooltip(handle: Long, tooltip: String)
+    @JvmStatic external fun nativeSetTrayCallback(handle: Long, callback: Runnable?)
+    @JvmStatic external fun nativeSetTrayMenu(trayHandle: Long, menuHandle: Long)
+    @JvmStatic external fun nativeClearTrayMenu(trayHandle: Long)
+    @JvmStatic external fun nativeInitTray(handle: Long): Int
+    @JvmStatic external fun nativeLoopTray(blocking: Int): Int
+    @JvmStatic external fun nativeUpdateTray(handle: Long)
+    @JvmStatic external fun nativeExitTray()
+
+    // -- Menu items --
+
+    @JvmStatic external fun nativeCreateMenuItems(count: Int): Long
+    @JvmStatic external fun nativeSetMenuItem(
+        menuHandle: Long,
+        index: Int,
+        text: String,
+        iconPath: String?,
+        disabled: Int,
+        checked: Int,
+    )
+    @JvmStatic external fun nativeSetMenuItemCallback(menuHandle: Long, index: Int, callback: Runnable?)
+    @JvmStatic external fun nativeSetMenuItemSubmenu(menuHandle: Long, index: Int, submenuHandle: Long)
+    @JvmStatic external fun nativeFreeMenuItems(menuHandle: Long, count: Int)
+
+    // -- Position --
+
+    /** Writes [x, y] into outXY. Returns non-zero if precise. */
+    @JvmStatic external fun nativeGetNotificationIconsPosition(outXY: IntArray): Int
+    @JvmStatic external fun nativeGetNotificationIconsRegion(): String?
+
+    // -- Mouse hook (for outside-click detection) --
+
+    @JvmStatic external fun nativeInstallMouseHook(callback: Runnable): Long
+    @JvmStatic external fun nativeRunMouseHookLoop(hookId: Long)
+    @JvmStatic external fun nativeStopMouseHook(hookId: Long)
+    @JvmStatic external fun nativeGetLastMouseHookClick(outXY: IntArray)
+}