Fix: voice_confirm setting was ignored (guided voice flow never triggered); skip STUN monitor when on (0.1.12)

- get_settings/set_setting теперь знают voice_confirm — флаг сохраняется и читается,
  без этого кнопка «Глубокий поиск» не видела его и гайд-поток не запускался (фича «ничего не делала»)
- при voice_confirm ON не поднимаем STUN-монитор голоса: на сетях, где Google STUN мёртв,
  он ложно «ронял» и переключал даже подтверждённую человеком стратегию (watchdog по сайтам остаётся)
- регресс-тесты на оба случая

Author: cold-hell

freeconnect/__init__.py

@@ -1,3 +1,3 @@
 """FreeConnect — обёртка над zapret с автоподбором стратегии обхода."""
 
-__version__ = "0.1.11"
+__version__ = "0.1.12"

freeconnect/app.py

@@ -666,10 +666,12 @@ def get_settings(self) -> dict:
             "auto_enable": self.cfg.get("auto_enable", True),
             "game_filter": self.cfg.get("game_filter", False),
             "doh": self.cfg.get("doh", False),
+            "voice_confirm": self.cfg.get("voice_confirm", False),
         }
 
     def set_setting(self, key: str, value) -> dict:
-        if key not in ("autostart", "monitor", "auto_enable", "game_filter", "doh"):
+        if key not in ("autostart", "monitor", "auto_enable", "game_filter", "doh",
+                       "voice_confirm"):
             return self.get_settings()
         val = bool(value)
         if key == "autostart":
@@ -1004,9 +1006,15 @@ def on_result(sc: StrategyScore):
 
     # ---- мониторы (голос по UDP + доступность сервисов по TCP/TLS) ----
     def _start_monitors(self) -> None:
-        if self.cfg.get("monitor", True):
+        if not self.cfg.get("monitor", True):
+            return
+        # STUN-монитор голоса запускаем ТОЛЬКО когда точная проверка голоса выключена:
+        # его сигнал (пинг Google STUN) ненадёжен и на части сетей STUN мёртв вовсе —
+        # тогда он ложно «роняет» и переключает даже стратегию, где голос реально живой
+        # (в т.ч. подтверждённую человеком). С voice_confirm доверяем человеку + watchdog.
+        if not self.cfg.get("voice_confirm", False):
             self.monitor.start()
-            self.watchdog.start()
+        self.watchdog.start()
 
     def _stop_monitors(self) -> None:
         self.monitor.stop()

installer/FreeConnect.iss

@@ -3,7 +3,7 @@
 ; Сборка: ISCC.exe installer\FreeConnect.iss  ->  installer\Output\FreeConnect-Setup.exe
 
 #define MyAppName "FreeConnect"
-#define MyAppVersion "0.1.11"
+#define MyAppVersion "0.1.12"
 #define MyAppPublisher "FreeConnect"
 #define MyAppExeName "FreeConnect.exe"
 

tests/test_voice_confirm.py

@@ -137,5 +137,56 @@ def test_cancel_during_wait(self):
         self.assertFalse(t.is_alive())
 
 
+class TestVoiceConfirmSetting(unittest.TestCase):
+    """Регресс: флаг voice_confirm должен читаться get_settings и сохраняться
+    set_setting — иначе кнопка «Глубокий поиск» не увидит его и гайд не запустится
+    (ровно тот баг, что фича «ничего не делала»)."""
+
+    def test_roundtrip(self):
+        from freeconnect import config
+        api = fcapp.Api.__new__(fcapp.Api)
+        api.cfg = {"voice_confirm": False, "monitor": True, "auto_enable": True,
+                   "game_filter": False, "doh": False}
+        api.enabled = False
+        orig = config.save
+        config.save = lambda cfg: None
+        try:
+            self.assertIn("voice_confirm", api.get_settings())
+            self.assertFalse(api.get_settings()["voice_confirm"])
+            api.set_setting("voice_confirm", True)
+            self.assertTrue(api.cfg["voice_confirm"])                 # сохранилось
+            self.assertTrue(api.get_settings()["voice_confirm"])      # и читается
+        finally:
+            config.save = orig
+
+
+class TestMonitorGating(unittest.TestCase):
+    """При voice_confirm ON ненадёжный STUN-монитор голоса НЕ поднимается (иначе он
+    ложно уронил бы подтверждённую человеком стратегию); watchdog по сайтам — да."""
+
+    def _api(self, voice_confirm):
+        api = fcapp.Api.__new__(fcapp.Api)
+        api.cfg = {"monitor": True, "voice_confirm": voice_confirm}
+
+        class _M:
+            def __init__(s): s.started = False
+            def start(s): s.started = True
+        api.monitor = _M()
+        api.watchdog = _M()
+        return api
+
+    def test_voice_confirm_skips_stun_monitor(self):
+        api = self._api(True)
+        api._start_monitors()
+        self.assertFalse(api.monitor.started)   # STUN-монитор НЕ поднят
+        self.assertTrue(api.watchdog.started)    # watchdog поднят
+
+    def test_off_starts_both(self):
+        api = self._api(False)
+        api._start_monitors()
+        self.assertTrue(api.monitor.started)
+        self.assertTrue(api.watchdog.started)
+
+
 if __name__ == "__main__":
     unittest.main()