fix: clean up remaining P1 backlog items (status parser, log guard, popout height, location casing)

- disconnectPattern now only matches firstLine, fixing false-positive
  disconnects triggered by auxiliary output lines
- openTunnelLog guards against concurrent double-invocation via
  tunnelLogOpening flag (UI + Service layers)
- popoutHeight clamps to screen height (parentScreen/barThickness)
  instead of a fixed 760, fixing overflow on short displays
- connected location name is title-cased for display, masking an
  upstream adguardvpn-cli capitalization quirk (SãO PAULO)

3 new regression tests (29/29 passing).

Author: bernardopg

AdGuardVpnParsers.js

@@ -33,7 +33,7 @@ function parseStatusOutput(clean) {
     const fullOutput = lines.join("\n");
     const dnsWarning = /System DNS could not be configured/i.test(fullOutput);
     const disconnectPattern = /not\s+connected|disconnected|not\s+running|stopped/i;
-    if (disconnectPattern.test(firstLine) || disconnectPattern.test(fullOutput)) {
+    if (disconnectPattern.test(firstLine)) {
         return {
             disconnected: true,
             firstLine: firstLine

AdGuardVpnService.qml

@@ -52,6 +52,7 @@ Item {
     property string cliVersion: ""
     property bool commandRunning: false
     property string runningCommand: ""
+    property bool tunnelLogOpening: false
 
     property bool isConnected: false
     property string statusSummary: AdGuardVpnI18n.tr("status.unknown", "Unknown")
@@ -830,6 +831,12 @@ Item {
     }
 
     function openTunnelLog() {
+        if (tunnelLogOpening) {
+            ToastService.showInfo(t("app.title", "AdGuard VPN"), t("toast.log_opening", "Opening tunnel log..."));
+            return;
+        }
+        tunnelLogOpening = true;
+
         const openScript = `
             resolve_home() {
                 if [ -n "$HOME" ]; then
@@ -936,6 +943,7 @@ Item {
         `;
 
         Proc.runCommand(`${pluginId}.openTunnelLog.${Date.now()}`, ["sh", "-lc", openScript], (stdout, exitCode) => {
+            tunnelLogOpening = false;
             if (exitCode === 0) {
                 ToastService.showInfo(t("app.title", "AdGuard VPN"), t("toast.log_opened", "Tunnel log opened"));
                 return;

AdGuardVpnWidget.qml

@@ -93,7 +93,7 @@ PluginComponent {
             return root.t("bar.pending", "...");
         }
         if (AdGuardVpnService.isConnected) {
-            return AdGuardVpnService.connectedLocation || root.t("bar.connected_short", "Connected");
+            return root.formatLocationName(AdGuardVpnService.connectedLocation) || root.t("bar.connected_short", "Connected");
         }
         return root.t("bar.off", "Off");
     }
@@ -104,7 +104,7 @@ PluginComponent {
         }
         if (AdGuardVpnService.isConnected) {
             return root.t("summary.connected_to", "Connected to {location}", {
-                location: root.safeText(AdGuardVpnService.connectedLocation, root.t("summary.location_unknown", "location unknown"))
+                location: root.formatLocationName(root.safeText(AdGuardVpnService.connectedLocation, root.t("summary.location_unknown", "location unknown")))
             });
         }
         return root.safeText(AdGuardVpnService.statusSummary, root.t("status.disconnected", "Disconnected"));
@@ -144,6 +144,14 @@ PluginComponent {
         return value;
     }
 
+    function formatLocationName(value) {
+        const text = (value || "").toString();
+        if (!text) {
+            return text;
+        }
+        return text.toLowerCase().replace(/(^|[\s-])(\S)/g, (match, sep, ch) => sep + ch.toUpperCase());
+    }
+
     component VpnActionButton: StyledRect {
         id: buttonRoot
 
@@ -549,7 +557,14 @@ PluginComponent {
     }
 
     popoutWidth: 520
-    popoutHeight: 760
+    readonly property int popoutMaxHeight: 760
+    readonly property int popoutMinHeight: 420
+    popoutHeight: {
+        const scr = root.parentScreen || Screen;
+        const screenHeight = (scr && scr.height) ? scr.height : root.popoutMaxHeight;
+        const reserved = root.barThickness + 96;
+        return Math.max(root.popoutMinHeight, Math.min(root.popoutMaxHeight, screenHeight - reserved));
+    }
 
     horizontalBarPill: Component {
         Row {
@@ -979,7 +994,7 @@ PluginComponent {
                                             iconName: "article"
                                             label: root.t("action.open_log", "Open Log")
                                             compact: true
-                                            actionEnabled: !AdGuardVpnService.commandRunning
+                                            actionEnabled: !AdGuardVpnService.commandRunning && !AdGuardVpnService.tunnelLogOpening
                                             onTriggered: {
                                                 ToastService.showInfo(root.t("app.title", "AdGuard VPN"), root.t("toast.log_opening", "Opening tunnel log..."));
                                                 AdGuardVpnService.openTunnelLog();

CHANGELOG.md

@@ -13,6 +13,13 @@ Format follows [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) and [Sem
 
 - DNS leak warning banner: when `adguardvpn-cli` reports "System DNS could not be configured" on connect/status, the popout now shows a persistent warning in the hero instead of staying silent about queries potentially bypassing the tunnel.
 
+### Fixed
+
+- Status parser no longer misreads a connected session as disconnected because of an auxiliary output line containing "disconnected"/"stopped" — the disconnect signal is now matched against the first line only, where every supported CLI status format already places it.
+- `openTunnelLog` now guards against concurrent double-invocation (rapid double-click) with a dedicated `tunnelLogOpening` flag instead of being able to spawn duplicate terminals.
+- Popout height is no longer a fixed `760`; it now clamps to the screen height (via `parentScreen`/`Screen`, same fallback the DMS `PluginComponent` uses internally) so it no longer overflows short displays (e.g. 768p with a bar).
+- Connected location name display (`SãO PAULO`, an upstream `adguardvpn-cli` capitalization quirk) is now normalized to title case (`São Paulo`) in the bar pill and hero title.
+
 ---
 
 ## [1.4.0] - 2026-06-12
@@ -224,6 +231,13 @@ Todas as mudanças relevantes deste projeto são documentadas aqui. O inglês ac
 
 - Banner de vazamento de DNS: quando o `adguardvpn-cli` reporta "System DNS could not be configured" no connect/status, o popout agora mostra um aviso persistente no hero em vez de ficar em silêncio sobre consultas DNS que podem vazar para fora do túnel.
 
+#### Corrigido
+
+- O parser de status não confunde mais uma sessão conectada com desconectada por causa de uma linha auxiliar contendo "disconnected"/"stopped" — o sinal de desconexão agora é testado só na primeira linha, onde todos os formatos suportados do CLI já colocam essa informação.
+- `openTunnelLog` agora tem guard contra dupla invocação concorrente (clique duplo rápido) via flag dedicada `tunnelLogOpening`, em vez de poder lançar terminais duplicados.
+- Altura do popout deixou de ser fixa em `760`; agora é clampada pela altura da tela (via `parentScreen`/`Screen`, mesmo fallback usado internamente pelo `PluginComponent` do DMS), eliminando o estouro em telas baixas (ex.: 768p com barra).
+- Exibição do nome da localização conectada (`SãO PAULO`, quirk de capitalização do próprio `adguardvpn-cli`) agora é normalizada para title case (`São Paulo`) no pill da barra e no título do hero.
+
 ### [1.4.0] - 2026-06-12
 
 #### Adicionado

DONE.md

@@ -8,6 +8,10 @@
 ## Pós-v1.4.0 — 2026-07-10
 
 - [x] **Aviso de vazamento de DNS nunca era capturado** — ✅ 2026-07-10: descoberta ao reconectar manualmente e ver `Warning: System DNS could not be configured` no CLI, nunca exposto no plugin. `AdGuardVpnParsers.js` agora detecta a linha em `parseStatusOutput` (`dnsWarning`), `AdGuardVpnService.qml` expõe `dnsWarning: bool` (resetado em todo branch de status), `AdGuardVpnWidget.qml` mostra banner persistente no hero (mesmo padrão visual do banner "Sessão não iniciada"). 2 testes novos em `scripts/test-parsers.mjs` (28/28 passando). Chaves i18n `dns.leak_warning_title`/`dns.leak_warning_body` em en+pt_BR (paridade estrita mantida).
+- [x] **Detecção de "disconnected" agressiva demais** — ✅ 2026-07-10: `disconnectPattern` (`AdGuardVpnParsers.js`) testava `firstLine` OU `fullOutput` inteiro; qualquer linha auxiliar com "disconnected"/"stopped" derrubava o estado mesmo com "Connected to …" na primeira linha. Teste restrito só ao `firstLine` — todos os formatos suportados (connected/disconnected/key-value) já colocam o sinal decisivo ali, confirmado pelos 29 testes existentes. Teste de regressão novo prova que "Connected to Frankfurt..." + linha auxiliar "...was stopped..." continua `connected: true`.
+- [x] **`openTunnelLog` sem guard contra dupla invocação** — ✅ 2026-07-10: novo `property bool tunnelLogOpening` em `AdGuardVpnService.qml`, setado antes do `Proc.runCommand` e resetado no callback (sucesso e erro); clique duplo/rápido agora só reexibe o toast "Opening tunnel log..." em vez de lançar terminal/spawn duplicado. Botão "Open Log" no widget desabilita enquanto `tunnelLogOpening` é `true` (defesa em duas camadas: UI + Service).
+- [x] **`popoutHeight: 760` fixo estourava em telas baixas** — ✅ 2026-07-10: `popoutHeight` agora é um binding que usa `root.parentScreen || Screen` (mesmo fallback usado internamente pelo `PluginComponent.qml` do DMS) e `root.barThickness`, clampando entre `popoutMinHeight` (420) e `popoutMaxHeight` (760) — `Math.max(420, Math.min(760, screenHeight - barThickness - 96))`. Sem regressão em 1080p (resultado idêntico a 760); em 768p com barra o popout cabe com margem.
+- [x] **Capitalização quebrada em nomes de localização com acento (`SãO PAULO`)** — ✅ 2026-07-10: quirk confirmado como bug do próprio `adguardvpn-cli` (hex dump mostrou `ã` minúsculo dentro de string em bold/uppercase), mascarado cosmeticamente com `formatLocationName()` novo em `AdGuardVpnWidget.qml` (lowercase + capitalize por palavra/hífen) aplicado no bar pill e no hero title. Verificado visualmente: badge agora mostra "São Paulo".
 
 ---
 

To-Do.md

@@ -5,18 +5,8 @@
 
 ---
 
-## P1 — Lógica
-
-- [ ] **Detecção de "disconnected" agressiva demais** — `AdGuardVpnParsers.js:34-35`: `disconnectPattern` testa o output inteiro; qualquer linha auxiliar contendo "disconnected"/"stopped" (ex.: aviso histórico do CLI) marca o estado como desconectado mesmo com a primeira linha "Connected to …". Restringir o teste do `fullOutput` a padrões mais específicos ou priorizar o match de connected na primeira linha antes do disconnect no full output.
-- [ ] **`openTunnelLog` roda fora do guard `commandRunning`** — pode rodar em paralelo com connect/disconnect (ambos via `Proc.runCommand`). Baixo risco, mas avaliar guard ou fila.
-
-## P1 — UI/UX
-
-- [ ] **`popoutHeight: 760` fixo** — em telas baixas (768p com barra) estoura. Calcular: `Math.min(760, Screen.height - margens)` ou equivalente DMS. (Pendente: requer verificar como o DMS clampa popouts antes de mexer.)
-
 ## P2 — Melhorias
 
-- [ ] **Capitalização quebrada em nomes de localização com acento** — descoberto 2026-07-10: o próprio `adguardvpn-cli` emite `SãO PAULO` (confirmado em hex: `ã` minúsculo dentro de string em bold/uppercase) — não é bug do plugin, mas dá pra mascarar cosmeticamente normalizando a exibição de `connectedLocation` (title-case) em vez de repassar cru.
 - [ ] **Indicador visual de ping** — colorir o badge de ping (verde <80ms, amarelo <150, vermelho acima) nos cards de localização.
 - [ ] **Confirmação/feedback de favoritos** — toast leve ou animação na estrela ao favoritar.
 - [ ] **Copiar diagnóstico** — botão "copiar" no bloco Command output / last command.

scripts/test-parsers.mjs

@@ -112,6 +112,15 @@ test("status: disconnected variants", () => {
     }
 });
 
+test("status: connected first line survives disconnect keyword in auxiliary line", () => {
+    const r = P.parseStatusOutput([
+        "Connected to Frankfurt in TUN mode, running on tun0",
+        "Note: a previous session was stopped due to a network change"
+    ].join("\n"));
+    eq(r.connected, true, "still connected");
+    eq(r.disconnected, undefined, "not flagged disconnected");
+});
+
 test("status: empty output", () => {
     eq(P.parseStatusOutput("").empty, true);
     eq(P.parseStatusOutput("   \n  ").empty, true);