feat(macos): implement onMenuOpened callback for tray menu

- Add menu opened callback storage and dispatch in TrayContext (Swift)
- Invoke callback in InstanceButtonClickHandler.handleClick() before menu.popUp()
- Expose tray_set_menu_opened_callback() C API with per-instance support
- Add JNI bridge and trampoline for menu opened callback
- Wire callback through MacNativeBridge, MacTrayManager, and MacTrayInitializer
- Initialize callback after tray_init() to ensure TrayContext exists
- Clear native library cache in macOS build script (darwin-aarch64/x86-64)

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

Author: kdroidFilter

src/jvmMain/kotlin/com/kdroid/composetray/lib/mac/MacNativeBridge.kt

@@ -35,6 +35,11 @@ internal object MacNativeBridge {
         callback: Runnable?,
     )
 
+    @JvmStatic external fun nativeSetMenuOpenedCallback(
+        handle: Long,
+        callback: Runnable?,
+    )
+
     @JvmStatic external fun nativeSetTrayMenu(
         trayHandle: Long,
         menuHandle: Long,

src/jvmMain/kotlin/com/kdroid/composetray/lib/mac/MacTrayManager.kt

@@ -15,6 +15,7 @@ internal class MacTrayManager(
     private var iconPath: String,
     private var tooltip: String = "",
     onLeftClick: (() -> Unit)? = null,
+    onMenuOpened: (() -> Unit)? = null,
 ) {
     private var trayHandle: Long = 0L
     private var menuHandle: Long = 0L
@@ -33,6 +34,7 @@ internal class MacTrayManager(
     private val submenuHandles: MutableList<Pair<Long, Int>> = mutableListOf()
 
     private val onLeftClickCallback = mutableStateOf(onLeftClick)
+    private var onMenuOpenedCallback: (() -> Unit)? = onMenuOpened
 
     // Top level MenuItem class
     data class MenuItem(
@@ -72,6 +74,7 @@ internal class MacTrayManager(
         newTooltip: String,
         newOnLeftClick: (() -> Unit)?,
         newMenuItems: List<MenuItem>? = null,
+        newOnMenuOpened: (() -> Unit)? = null,
     ) {
         lock.withLock {
             if (!running.get() || trayHandle == 0L) return
@@ -80,11 +83,13 @@ internal class MacTrayManager(
             val iconChanged = this.iconPath != newIconPath
             val tooltipChanged = this.tooltip != newTooltip
             val onLeftClickChanged = this.onLeftClickCallback.value != newOnLeftClick
+            val onMenuOpenedChanged = this.onMenuOpenedCallback != newOnMenuOpened
 
             // Update icon path and tooltip
             this.iconPath = newIconPath
             this.tooltip = newTooltip
             this.onLeftClickCallback.value = newOnLeftClick
+            this.onMenuOpenedCallback = newOnMenuOpened
 
             if (iconChanged) {
                 MacNativeBridge.nativeSetTrayIcon(trayHandle, newIconPath)
@@ -95,6 +100,9 @@ internal class MacTrayManager(
             if (onLeftClickChanged) {
                 initializeOnLeftClickCallback()
             }
+            if (onMenuOpenedChanged) {
+                initializeOnMenuOpenedCallback()
+            }
 
             // Update menu items if provided
             if (newMenuItems != null) {
@@ -167,6 +175,9 @@ internal class MacTrayManager(
                             throw IllegalStateException("Failed to initialize tray: $initResult")
                         }
 
+                        // Set menu-opened callback after init (TrayContext must exist)
+                        initializeOnMenuOpenedCallback()
+
                         // Signal that initialization is complete
                         initLatch.countDown()
 
@@ -218,6 +229,26 @@ internal class MacTrayManager(
         }
     }
 
+    private fun initializeOnMenuOpenedCallback() {
+        if (trayHandle == 0L) return
+
+        val callback = onMenuOpenedCallback
+        if (callback != null) {
+            MacNativeBridge.nativeSetMenuOpenedCallback(
+                trayHandle,
+                Runnable {
+                    mainScope?.launch {
+                        ioScope?.launch {
+                            callback()
+                        }
+                    }
+                },
+            )
+        } else {
+            MacNativeBridge.nativeSetMenuOpenedCallback(trayHandle, null)
+        }
+    }
+
     private fun initializeTrayMenu() {
         if (trayHandle == 0L) return
 

src/jvmMain/kotlin/com/kdroid/composetray/tray/impl/MacTrayInitializer.kt

@@ -25,12 +25,12 @@ object MacTrayInitializer {
         tooltip: String,
         onLeftClick: (() -> Unit)? = null,
         menuContent: (TrayMenuBuilder.() -> Unit)? = null,
-        @Suppress("UNUSED_PARAMETER") onMenuOpened: (() -> Unit)? = null,
+        onMenuOpened: (() -> Unit)? = null,
     ) {
         var manager = trayManagers[id]
         if (manager == null) {
             // Create a new manager for this ID
-            manager = MacTrayManager(iconPath, tooltip, onLeftClick)
+            manager = MacTrayManager(iconPath, tooltip, onLeftClick, onMenuOpened)
             trayManagers[id] = manager
 
             // Build menu for this manager
@@ -55,7 +55,7 @@ object MacTrayInitializer {
             manager.startTray()
         } else {
             // Existing manager: delegate to update with the provided content
-            update(id, iconPath, tooltip, onLeftClick, menuContent)
+            update(id, iconPath, tooltip, onLeftClick, menuContent, onMenuOpened)
         }
     }
 
@@ -66,12 +66,12 @@ object MacTrayInitializer {
         tooltip: String,
         onLeftClick: (() -> Unit)? = null,
         menuContent: (TrayMenuBuilder.() -> Unit)? = null,
-        @Suppress("UNUSED_PARAMETER") onMenuOpened: (() -> Unit)? = null,
+        onMenuOpened: (() -> Unit)? = null,
     ) {
         val manager = trayManagers[id]
         if (manager == null) {
             // If manager doesn't exist, initialize it
-            initialize(id, iconPath, tooltip, onLeftClick, menuContent)
+            initialize(id, iconPath, tooltip, onLeftClick, menuContent, onMenuOpened)
             return
         }
 
@@ -94,7 +94,7 @@ object MacTrayInitializer {
                 null
             }
 
-        manager.update(iconPath, tooltip, onLeftClick, newMenuItems)
+        manager.update(iconPath, tooltip, onLeftClick, newMenuItems, onMenuOpened)
     }
 
     @Synchronized

src/native/macos/MacTrayBridge.m

@@ -61,6 +61,7 @@ JNIEXPORT jint JNICALL JNI_OnLoad(JavaVM *vm, void *reserved) {
 
 static CallbackEntry *g_trayCallbacks = NULL;      /* tray left-click */
 static CallbackEntry *g_menuCallbacks = NULL;       /* menu item click */
+static CallbackEntry *g_menuOpenedCallbacks = NULL;  /* menu opened */
 static CallbackEntry *g_themeCallback = NULL;        /* theme change (single) */
 
 static void storeCallback(CallbackEntry **list, void *key, JNIEnv *env, jobject callback) {
@@ -177,6 +178,18 @@ static void menuItemCbTrampoline(struct tray_menu_item *item) {
     if ((*env)->ExceptionCheck(env)) (*env)->ExceptionClear(env);
 }
 
+/* Called by the Swift click handler when the menu is about to open */
+static void menuOpenedCbTrampoline(struct tray *t) {
+    JNIEnv *env = getJNIEnv();
+    if (!env) return;
+    jobject runnable = findCallback(g_menuOpenedCallbacks, t);
+    if (!runnable) return;
+    jmethodID run = getRunnableRunMethod(env);
+    if (!run) return;
+    (*env)->CallVoidMethod(env, runnable, run);
+    if ((*env)->ExceptionCheck(env)) (*env)->ExceptionClear(env);
+}
+
 /* Called by the Swift appearance observer when the theme changes */
 static void themeCbTrampoline(int isDark) {
     JNIEnv *env = getJNIEnv();
@@ -260,6 +273,16 @@ JNIEXPORT void JNICALL Java_com_kdroid_composetray_lib_mac_MacNativeBridge_nativ
     t->cb = (callback != NULL) ? trayCbTrampoline : NULL;
 }
 
+JNIEXPORT void JNICALL Java_com_kdroid_composetray_lib_mac_MacNativeBridge_nativeSetMenuOpenedCallback(
+    JNIEnv *env, jclass clazz, jlong handle, jobject callback)
+{
+    (void)clazz;
+    struct tray *t = (struct tray *)(uintptr_t)handle;
+    if (!t) return;
+    storeCallback(&g_menuOpenedCallbacks, t, env, callback);
+    tray_set_menu_opened_callback(t, (callback != NULL) ? menuOpenedCbTrampoline : NULL);
+}
+
 JNIEXPORT void JNICALL Java_com_kdroid_composetray_lib_mac_MacNativeBridge_nativeSetTrayMenu(
     JNIEnv *env, jclass clazz, jlong trayHandle, jlong menuHandle)
 {
@@ -312,6 +335,7 @@ JNIEXPORT void JNICALL Java_com_kdroid_composetray_lib_mac_MacNativeBridge_nativ
     tray_dispose(t);
     /* Clean up callback refs for this tray */
     removeCallback(&g_trayCallbacks, t);
+    removeCallback(&g_menuOpenedCallbacks, t);
     /* Free the struct and its strings */
     free((void *)t->icon_filepath);
     free((void *)t->tooltip);
@@ -325,6 +349,7 @@ JNIEXPORT void JNICALL Java_com_kdroid_composetray_lib_mac_MacNativeBridge_nativ
     tray_exit();
     clearAllCallbacks(&g_trayCallbacks);
     clearAllCallbacks(&g_menuCallbacks);
+    clearAllCallbacks(&g_menuOpenedCallbacks);
 }
 
 /* ========================================================================== */

src/native/macos/build.sh

@@ -68,4 +68,14 @@ build_arch() {
 build_arch "arm64"  "arm64-apple-macosx11.0"  "$OUTPUT_DIR/darwin-aarch64"
 build_arch "x86_64" "x86_64-apple-macosx11.0" "$OUTPUT_DIR/darwin-x86-64"
 
+# Invalidate runtime cache (NativeLibraryLoader validates by size only,
+# so a same-size rebuild would serve the stale cached copy)
+for PLATFORM_DIR in darwin-aarch64 darwin-x86-64; do
+    CACHE_FILE="$HOME/.cache/composetray/native/$PLATFORM_DIR/libMacTray.dylib"
+    if [ -f "$CACHE_FILE" ]; then
+        rm -f "$CACHE_FILE"
+        echo "Cleared cached library: $CACHE_FILE"
+    fi
+done
+
 echo "Build completed successfully."

src/native/macos/tray.h

@@ -65,6 +65,7 @@ TRAY_API void tray_exit (void);                /* free everything and exit   */
 /*  Additional options / information                                          */
 /* -------------------------------------------------------------------------- */
 TRAY_API void tray_set_theme_callback(theme_callback cb);
+TRAY_API void tray_set_menu_opened_callback(struct tray *tray, tray_callback cb);
 TRAY_API int  tray_is_menu_dark(void);         /* 1 = dark mode              */
 
 /* Windows: corner and coordinates of notification area                       */

src/native/macos/tray.swift

@@ -21,6 +21,7 @@ private class TrayContext {
     let appearanceObserver: MenuBarAppearanceObserver
     var lightImage: NSImage?
     var darkImage: NSImage?
+    var menuOpenedCallback: TrayCallback?
     init(statusItem: NSStatusItem, clickHandler: InstanceButtonClickHandler, appearanceObserver: MenuBarAppearanceObserver) {
         self.statusItem = statusItem
         self.clickHandler = clickHandler
@@ -51,6 +52,7 @@ private class MenuDelegate: NSObject, NSMenuDelegate {
 
         if event.type == .rightMouseUp || event.modifierFlags.contains(.control) {
             if let menu = ctx.contextMenu {
+                ctx.menuOpenedCallback?(trayPtr)
                 let menuLocation = NSPoint(
                     x: sender.frame.minX,
                     y: sender.frame.minY - 5
@@ -384,6 +386,19 @@ public func tray_exit() {
     menuDelegate = nil
 }
 
+@_cdecl("tray_set_menu_opened_callback")
+public func tray_set_menu_opened_callback(
+    _ tray: UnsafeMutableRawPointer?,
+    _ cb: TrayCallback?
+) {
+    let doWork = {
+        guard let tray = tray, let ctx = contexts[tray] else { return }
+        ctx.menuOpenedCallback = cb
+    }
+    if Thread.isMainThread { doWork() }
+    else { DispatchQueue.main.sync { doWork() } }
+}
+
 @_cdecl("tray_set_theme_callback")
 public func tray_set_theme_callback(_ cb: @escaping ThemeCallback) {
     themeCallback = cb