fix(tray): refactor menu and update battery plugins

Tray Menu Fixes:
- Separate mode now uses tray_manager for crossbar icon + SNI for plugins
- Added Disable action to plugin menus (replaces Quit)
- Added debug logging for menu creation

Sample Plugins Updates:
- Battery upgraded from .30s to .2s (faster refresh)
- Removed Python/Node/Dart/Go/Rust variants (now sh+lua only)
- battery.2s.lua uses crossbarBridge.batterySync for cross-platform

Known Issues:
- tray_manager setContextMenu may not work on Linux (menu appears empty)
- This is a limitation of the tray_manager package on GNOME

Author: insign

lib/services/sample_plugins_service.dart

@@ -58,18 +58,13 @@ class SamplePluginsService {
     PluginMetadata(
       id: 'battery',
       name: 'Battery Status',
-      description: 'Shows battery level and charging status',
+      description: 'Shows battery level and charging status with dynamic icons',
       category: PluginCategory.system,
       mobileCompatible: true,
       tags: ['battery', 'power', 'charging'],
       variants: [
-        PluginMetadataVariant(language: PluginLanguage.bash, filename: 'battery.30s.sh', assetPath: 'plugins/battery/battery.30s.sh'),
-        PluginMetadataVariant(language: PluginLanguage.python, filename: 'battery.30s.py', assetPath: 'plugins/battery/battery.30s.py'),
-        PluginMetadataVariant(language: PluginLanguage.node, filename: 'battery.30s.js', assetPath: 'plugins/battery/battery.30s.js'),
-        PluginMetadataVariant(language: PluginLanguage.dart, filename: 'battery.30s.dart', assetPath: 'plugins/battery/battery.30s.dart'),
-        PluginMetadataVariant(language: PluginLanguage.go, filename: 'battery.30s.go', assetPath: 'plugins/battery/battery.30s.go'),
-        PluginMetadataVariant(language: PluginLanguage.rust, filename: 'battery.30s.rs', assetPath: 'plugins/battery/battery.30s.rs'),
-        PluginMetadataVariant(language: PluginLanguage.lua, filename: 'battery.30s.lua', assetPath: 'plugins/battery/battery.30s.lua'),
+        PluginMetadataVariant(language: PluginLanguage.bash, filename: 'battery.2s.sh', assetPath: 'plugins/battery/battery.2s.sh'),
+        PluginMetadataVariant(language: PluginLanguage.lua, filename: 'battery.2s.lua', assetPath: 'plugins/battery/battery.2s.lua'),
       ],
     ),
     PluginMetadata(

lib/services/tray_service.dart

@@ -9,6 +9,7 @@ import 'package:tray_manager/tray_manager.dart';
 import '../core/plugin_manager.dart';
 import 'package:crossbar_core/crossbar_core.dart' as plugin_model;
 import 'logger_service.dart';
+import 'refresh_service.dart';
 import 'scheduler_service.dart';
 import 'settings_service.dart';
 import 'tray/tray_backend.dart';
@@ -76,11 +77,13 @@ class TrayService with TrayListener {
 
     // Set up tray based on mode
     if (_useSeparateMode) {
-      // SEPARATE MODE: Use only SNI backend, no unified tray_manager
-      LoggerService().info('TrayService: Using separate mode (SNI)');
-      // Icons will be created when plugins output is received
+      // SEPARATE MODE: Use BOTH tray_manager (crossbar control) + SNI daemons (plugins)
+      LoggerService().info('TrayService: Using separate mode (mixed)');
+      // Initialize tray_manager for crossbar control icon (Show/Quit)
+      await _initUnifiedTray();
+      // Plugin icons will be created via daemons when plugins output is received
     } else {
-      // UNIFIED MODE: Use tray_manager for single icon
+      // UNIFIED MODE: Use tray_manager for single icon with all plugins
       LoggerService().info('TrayService: Using unified mode (tray_manager)');
       await _initUnifiedTray();
     }
@@ -208,39 +211,53 @@ class TrayService with TrayListener {
 
     final menuItems = <MenuItem>[];
 
-    // Plugin outputs
-    for (final plugin in _pluginManager.plugins.where((p) => p.enabled)) {
-      final output = _pluginOutputs[plugin.id];
-      if (output != null && output.text != null && output.text!.isNotEmpty) {
-        if (output.menu.isNotEmpty) {
-          final submenuItems = _convertMenuItems(output.menu);
-          menuItems.add(MenuItem.submenu(
-            label: '${output.icon} ${output.text}',
-            submenu: Menu(items: submenuItems),
-          ));
-        } else {
-          menuItems.add(MenuItem(
-            label: '${output.icon} ${output.text}',
-            disabled: true,
-          ));
+    // Plugin outputs - only show in unified mode (in separate mode they have their own icons)
+    if (!_useSeparateMode) {
+      for (final plugin in _pluginManager.plugins.where((p) => p.enabled)) {
+        final output = _pluginOutputs[plugin.id];
+        if (output != null && output.text != null && output.text!.isNotEmpty) {
+          if (output.menu.isNotEmpty) {
+            final submenuItems = _convertMenuItems(output.menu);
+            menuItems.add(MenuItem.submenu(
+              label: '${output.icon} ${output.text}',
+              submenu: Menu(items: submenuItems),
+            ));
+          } else {
+            menuItems.add(MenuItem(
+              label: '${output.icon} ${output.text}',
+              disabled: true,
+            ));
+          }
         }
       }
-    }
 
-    if (menuItems.isNotEmpty) {
-      menuItems.add(MenuItem.separator());
+      if (menuItems.isNotEmpty) {
+        menuItems.add(MenuItem.separator());
+      }
     }
 
     // Standard menu items
-    menuItems.addAll([
-      MenuItem(key: 'show', label: 'Show Crossbar'),
-      MenuItem(key: 'refresh', label: 'Refresh All Plugins'),
-      MenuItem.separator(),
-      MenuItem(key: 'quit', label: 'Quit'),
-    ]);
+    if (_useSeparateMode) {
+      // In separate mode, only Show and Quit (no Refresh All since plugins have their own)
+      menuItems.addAll([
+        MenuItem(key: 'show', label: 'Show'),
+        MenuItem.separator(),
+        MenuItem(key: 'quit', label: 'Quit'),
+      ]);
+    } else {
+      menuItems.addAll([
+        MenuItem(key: 'show', label: 'Show Crossbar'),
+        MenuItem(key: 'refresh', label: 'Refresh All Plugins'),
+        MenuItem.separator(),
+        MenuItem(key: 'quit', label: 'Quit'),
+      ]);
+    }
+
+    LoggerService().info('TrayService: Setting menu with ${menuItems.length} items, separateMode=$_useSeparateMode');
 
     try {
       await trayManager.setContextMenu(Menu(items: menuItems));
+      LoggerService().info('TrayService: Menu set successfully');
     } catch (e) {
       LoggerService().warning('Failed to set tray context menu: $e');
     }
@@ -370,9 +387,9 @@ class TrayService with TrayListener {
     // Standard actions
     items.addAll([
       TrayMenuItem(label: 'Refresh', key: 'refresh_$pluginId'),
+      TrayMenuItem(label: 'Disable', key: 'disable_$pluginId'),
       const TrayMenuItem.separator(),
       const TrayMenuItem(label: 'Show Crossbar', key: 'show'),
-      const TrayMenuItem(label: 'Quit', key: 'quit'),
     ]);
 
     return items;
@@ -487,7 +504,23 @@ class TrayService with TrayListener {
 
   @override
   void onTrayMenuItemClick(MenuItem menuItem) {
-    switch (menuItem.key) {
+    final key = menuItem.key ?? '';
+    
+    // Handle disable_pluginId actions
+    if (key.startsWith('disable_')) {
+      final pluginId = key.substring(8);
+      _disablePlugin(pluginId);
+      return;
+    }
+    
+    // Handle refresh_pluginId actions
+    if (key.startsWith('refresh_')) {
+      final pluginId = key.substring(8);
+      SchedulerService().runPluginNow(pluginId);
+      return;
+    }
+    
+    switch (key) {
       case 'show':
         WindowService().show();
       case 'refresh':
@@ -497,6 +530,23 @@ class TrayService with TrayListener {
     }
   }
 
+  /// Disables a plugin and removes its tray icon.
+  Future<void> _disablePlugin(String pluginId) async {
+    // Disable the plugin via RefreshService (handles outputs and notifications)
+    await RefreshService().disablePlugin(pluginId);
+    
+    // Remove the tray icon
+    final iconId = _pluginIconIds.remove(pluginId);
+    if (iconId != null && _backend != null) {
+      await _backend!.destroyIcon(iconId);
+    }
+    
+    // Clear output
+    _pluginOutputs.remove(pluginId);
+    
+    LoggerService().info('TrayService: Disabled plugin $pluginId');
+  }
+
   Future<void> dispose() async {
     if (!_initialized) return;
 

plugins/battery/battery.2s.lua

@@ -0,0 +1,70 @@
+-- battery.2s.lua - Battery monitor with dynamic icons (argos-compatible)
+-- Updates every 2 seconds for real-time monitoring
+--
+-- Features:
+-- - Uses Crossbar API (crossbarBridge.batterySync)
+-- - Dynamic Freedesktop icons (battery-level-X-symbolic)
+-- - Works on ALL platforms (Lua runs embedded)
+
+local bridge = crossbarBridge
+
+-- Get battery data
+local battery = bridge.batterySync()
+
+if type(battery) ~= "table" then
+    print("🔋 --")
+    print("---")
+    print("No battery detected")
+    return
+end
+
+local level = battery.level or 0
+local charging = battery.charging or false
+local state = charging and "charging" or "discharging"
+
+-- Determine icon based on level and state
+local function get_battery_icon(pct, st)
+    local l = math.floor(pct / 10) * 10
+    if l > 100 then l = 100 end
+    if l < 0 then l = 0 end
+
+    local suffix = ""
+    if st == "charging" then
+        suffix = "-charging"
+    elseif pct == 100 and st == "charging" then
+        suffix = "-charged"
+    end
+
+    if pct == 100 then
+        return "battery-level-100-charged-symbolic"
+    elseif pct <= 5 then
+        return "battery-level-0" .. suffix .. "-symbolic"
+    else
+        return "battery-level-" .. l .. suffix .. "-symbolic"
+    end
+end
+
+local icon_name = get_battery_icon(level, state)
+
+-- Determine color
+local color
+if state == "discharging" then
+    if level < 20 then
+        color = "#ff5555"
+    else
+        color = "#f8f8f2"
+    end
+else
+    color = "#8be9fd"
+end
+
+-- Emoji icon
+local emoji = charging and "⚡" or (level <= 20 and "🪫" or "🔋")
+
+-- Output
+print(emoji .. " " .. level .. "% | iconName=" .. icon_name .. " color=" .. color)
+print("---")
+print("State: " .. state)
+print("Level: " .. level .. "%")
+print("---")
+print("Refresh | refresh=true")