Finalize Mini EQ 0.7.3

Author: bhack

CHANGELOG.md

@@ -9,6 +9,8 @@
   routing.
 - Avoid a headless startup hang when system-wide EQ fails synchronously before
   the GLib main loop starts.
+- Hide the GNOME Shell extension's mini analyzer while monitoring is turned
+  off, keeping the panel item available without showing stale meter bars.
 
 ## 0.7.2 - 2026-05-10
 

data/io.github.bhack.mini-eq.metainfo.xml

@@ -51,6 +51,7 @@
           <li>Require pipewire-gobject 0.3.6 and use its synchronous PipeWire registry, metadata, node, and device APIs for startup and routing state.</li>
           <li>Replace Mini EQ's PipeWire startup and routing sleep loops with bounded sync or registry-event waits for route params, virtual sink creation, and stream routing.</li>
           <li>Avoid a headless startup hang when system-wide EQ fails synchronously before the GLib main loop starts.</li>
+          <li>Hide the GNOME Shell extension's mini analyzer while monitoring is turned off.</li>
         </ul>
       </description>
     </release>

extensions/gnome-shell/mini-eq@bhack.github.io/extension.js

@@ -69,6 +69,7 @@ class MiniEqIndicator extends PanelMenu.Button {
         this._running = false;
         this._routed = false;
         this._eqEnabled = false;
+        this._analyzerEnabled = false;
         this._capabilities = new Set();
         this.visible = false;
 
@@ -83,7 +84,8 @@ class MiniEqIndicator extends PanelMenu.Button {
             y_align: Clutter.ActorAlign.CENTER,
         });
         box.add_child(this._icon);
-        box.add_child(this._buildPanelAnalyzer());
+        this._panelAnalyzer = this._buildPanelAnalyzer();
+        box.add_child(this._panelAnalyzer);
         this.add_child(box);
 
         this._statusItem = new PopupMenu.PopupMenuItem(_('Mini EQ is not running'));
@@ -336,6 +338,9 @@ class MiniEqIndicator extends PanelMenu.Button {
         const running = Boolean(unpackValue(state.running));
         const eqEnabled = Boolean(unpackValue(state.eq_enabled));
         const routed = Boolean(unpackValue(state.routed));
+        const analyzerEnabled = 'analyzer_enabled' in state
+            ? Boolean(unpackValue(state.analyzer_enabled))
+            : true;
         const presetName = unpackValue(state.preset_name) || _('Current State');
         const outputPresetName = unpackValue(state.output_preset_name) || '';
         const capabilities = unpackValue(state.capabilities) || [];
@@ -345,8 +350,9 @@ class MiniEqIndicator extends PanelMenu.Button {
         this._running = running;
         this._routed = routed;
         this._eqEnabled = eqEnabled;
+        this._analyzerEnabled = analyzerEnabled;
         this.visible = running;
-        this._syncPanelStateStyle(running, routed, eqEnabled);
+        this._syncPanelStateStyle(running, routed, eqEnabled, analyzerEnabled);
         this._updating = true;
         try {
             this._routingItem.setToggleState(routed);
@@ -355,7 +361,7 @@ class MiniEqIndicator extends PanelMenu.Button {
             this._updating = false;
         }
 
-        if (!running || !routed || !eqEnabled)
+        if (!running || !routed || !eqEnabled || !analyzerEnabled)
             this._setAnalyzerLevels([]);
 
         this._routingItem.setSensitive(running);
@@ -371,9 +377,10 @@ class MiniEqIndicator extends PanelMenu.Button {
         this._running = false;
         this._routed = false;
         this._eqEnabled = false;
+        this._analyzerEnabled = false;
         this._capabilities = new Set();
         this.visible = false;
-        this._syncPanelStateStyle(false, false, false);
+        this._syncPanelStateStyle(false, false, false, false);
         this._updating = true;
         try {
             this._routingItem.setToggleState(false);
@@ -413,10 +420,12 @@ class MiniEqIndicator extends PanelMenu.Button {
         if (this._disposed)
             return;
 
-        this._setAnalyzerLevels(this._running && this._routed && this._eqEnabled ? levels : []);
+        this._setAnalyzerLevels(this._running && this._routed && this._eqEnabled && this._analyzerEnabled ? levels : []);
     }
 
-    _syncPanelStateStyle(running, routed, eqEnabled) {
+    _syncPanelStateStyle(running, routed, eqEnabled, analyzerEnabled) {
+        this._panelAnalyzer.visible = running && analyzerEnabled;
+
         if (!running) {
             this._icon.opacity = 0;
             this._panelAnalyzerDimColor = PANEL_ANALYZER_STANDBY_COLOR;

src/mini_eq/dbus_control.py

@@ -170,6 +170,7 @@ def state(self) -> dict[str, GLib.Variant]:
                 "b",
                 bool(window and getattr(window, "output_preset_auto_applied", False)),
             ),
+            "analyzer_enabled": GLib.Variant("b", bool(window and getattr(window, "analyzer_enabled", False))),
             "background_mode": GLib.Variant("b", bool(getattr(self.app, "background_mode", False))),
             "start_at_login": GLib.Variant("b", bool(getattr(self.app, "start_at_login", False))),
             "start_active_at_login": GLib.Variant(

tests/test_mini_eq_dbus_control.py

@@ -217,6 +217,7 @@ def test_dbus_control_state_contains_shell_summary() -> None:
         "output_sink": "alsa_output.test",
         "output_preset_name": "Headphones",
         "output_preset_auto_applied": False,
+        "analyzer_enabled": False,
         "background_mode": True,
         "start_at_login": False,
         "start_active_at_login": False,

tools/gnome-shell-extension/fake_mini_eq_control.py

@@ -1,6 +1,7 @@
 #!/usr/bin/env python3
 from __future__ import annotations
 
+import argparse
 import math
 import signal
 import warnings
@@ -93,12 +94,13 @@ def display_level(level: float) -> float:
 
 
 class FakeMiniEqControl:
-    def __init__(self) -> None:
+    def __init__(self, *, analyzer_enabled: bool = True) -> None:
         self.eq_enabled = True
         self.routed = True
         self.preset_name = "Studio Reference"
         self.output_preset_name = "Demo Output Link"
         self.presets = ["Studio Reference", "Flat", "Voice Focus"]
+        self.analyzer_enabled = analyzer_enabled
         self.analyzer_levels = [0.0] * 10
         self.animation_step = 0
         self.loop = GLib.MainLoop()
@@ -120,6 +122,7 @@ def state(self) -> dict[str, GLib.Variant]:
             "output_sink": GLib.Variant("s", "Demo Output"),
             "output_preset_name": GLib.Variant("s", self.output_preset_name),
             "output_preset_auto_applied": GLib.Variant("b", True),
+            "analyzer_enabled": GLib.Variant("b", self.analyzer_enabled),
             "background_mode": GLib.Variant("b", True),
             "start_at_login": GLib.Variant("b", False),
             "window_visible": GLib.Variant("b", False),
@@ -142,7 +145,9 @@ def emit_analyzer_levels_changed(self) -> None:
             return
 
         levels = (
-            [display_level(level) for level in self.analyzer_levels] if self.eq_enabled and self.routed else [0.0] * 10
+            [display_level(level) for level in self.analyzer_levels]
+            if self.eq_enabled and self.routed and self.analyzer_enabled
+            else [0.0] * 10
         )
         self.connection.emit_signal(
             None,
@@ -238,5 +243,25 @@ def run(self) -> None:
                 Gio.bus_unown_name(self.owner_id)
 
 
+def parse_args() -> argparse.Namespace:
+    parser = argparse.ArgumentParser(description="Fake Mini EQ D-Bus service for GNOME Shell extension development.")
+    monitor_group = parser.add_mutually_exclusive_group()
+    monitor_group.add_argument(
+        "--monitor-on",
+        dest="analyzer_enabled",
+        action="store_true",
+        default=True,
+        help="report analyzer monitoring as enabled; this is the default",
+    )
+    monitor_group.add_argument(
+        "--monitor-off",
+        dest="analyzer_enabled",
+        action="store_false",
+        help="report analyzer monitoring as disabled",
+    )
+    return parser.parse_args()
+
+
 if __name__ == "__main__":
-    FakeMiniEqControl().run()
+    args = parse_args()
+    FakeMiniEqControl(analyzer_enabled=args.analyzer_enabled).run()

tools/run_gnome_extension_dev_shell.sh

@@ -15,11 +15,12 @@ mode="fake"
 
 usage() {
     cat >&2 <<EOF
-Usage: $0 [--fake-control|--no-fake-control|--real-session-install]
+Usage: $0 [--fake-control|--fake-control-monitor-off|--no-fake-control|--real-session-install]
 
-  --fake-control          Start isolated devkit Shell with fake Mini EQ D-Bus control service. Default.
-  --no-fake-control       Start isolated devkit Shell without a control service; tests disconnected UI.
-  --real-session-install  Install/reload the extension in the real GNOME session for real app integration.
+  --fake-control              Start isolated devkit Shell with fake Mini EQ D-Bus control service. Default.
+  --fake-control-monitor-off  Start fake control service with analyzer monitoring disabled.
+  --no-fake-control           Start isolated devkit Shell without a control service; tests disconnected UI.
+  --real-session-install      Install/reload the extension in the real GNOME session for real app integration.
 EOF
 }
 
@@ -28,6 +29,9 @@ while (($# > 0)); do
         --fake-control)
             mode="fake"
             ;;
+        --fake-control-monitor-off)
+            mode="fake-monitor-off"
+            ;;
         --no-fake-control)
             mode="no-fake"
             ;;
@@ -56,7 +60,7 @@ if [[ ! -x "$pack_extension" ]]; then
     exit 1
 fi
 
-if [[ "$mode" == "fake" && ! -x "$fake_control" ]]; then
+if [[ "$mode" != "no-fake" && "$mode" != "real-session" && ! -x "$fake_control" ]]; then
     echo "Fake control service not found or not executable: $fake_control" >&2
     exit 1
 fi
@@ -126,7 +130,9 @@ echo >&2
 echo "When the nested Shell starts, look for a virtual monitor/devkit surface." >&2
 echo "The top bar inside that nested Shell should show a 'Mini EQ' item." >&2
 if [[ "$mode" == "fake" ]]; then
-    echo "A fake Mini EQ D-Bus service is started inside the nested session." >&2
+    echo "A fake Mini EQ D-Bus service is started inside the nested session with monitoring enabled." >&2
+elif [[ "$mode" == "fake-monitor-off" ]]; then
+    echo "A fake Mini EQ D-Bus service is started inside the nested session with monitoring disabled." >&2
 else
     echo "No Mini EQ D-Bus service is started; controls should show disconnected/disabled state." >&2
 fi
@@ -149,12 +155,20 @@ run_in_dev_bus() {
         }
         trap cleanup EXIT INT TERM
 
-        "$1" &
+        fake_control="$1"
+        fake_mode="$2"
+        shift 2
+
+        fake_args=()
+        if [[ "$fake_mode" == "fake-monitor-off" ]]; then
+            fake_args+=(--monitor-off)
+        fi
+
+        "$fake_control" "${fake_args[@]}" &
         fake_pid=$!
         sleep 0.5
-        shift
         "$@"
-    ' bash "$fake_control" "${shell_command[@]}"
+    ' bash "$fake_control" "$mode" "${shell_command[@]}"
 }
 
 if gnome-shell --help 2>&1 | grep -q -- '--devkit'; then