Improve failover controls and preserve manual switching state

src/web/features/failover/FailoverFeature.tsx

@@ -3,23 +3,31 @@ import { flushSync } from 'react-dom';
 import { CONNECTIVITY_SITES, MAX_CUSTOM_DIAGNOSTIC_SERVICES, type DiagnosticSettings } from '../../../shared/connectivityDiagnostics.js';
 import { normalizeFailoverPolicy, type FailoverPolicy, type FailoverSnapshot } from '../../../shared/failover.js';
 import type { ProfileSnapshot } from '../../../shared/contracts/state.js';
+import { ConfirmationDialog } from '../../ui/ConfirmationDialog.js';
 import { Drawer } from '../../ui/Drawer.js';
 import { RailAction } from '../../ui/RailAction.js';
+import { Tooltip } from '../../ui/Tooltip.js';
 import { saveCustomDiagnosticService } from '../diagnostics/index.js';
-
 export function useFailoverFeature() {
   const [isOpen, setIsOpen] = useState(false);
   const panelRef = useRef<HTMLElement>(null);
   const toggleRef = useRef<HTMLButtonElement>(null);
   const closeRef = useRef<HTMLButtonElement>(null);
   const beforeCloseRef = useRef<() => boolean>(() => true);
+  const pendingTargetRef = useRef<(() => void) | null>(null);
   const close = () => {
-    if (beforeCloseRef.current()) setIsOpen(false);
+    if (beforeCloseRef.current()) {
+      pendingTargetRef.current = null;
+      setIsOpen(false);
+    }
   };
   useEffect(() => {
     if (!isOpen) return undefined;
     const frame = requestAnimationFrame(() => closeRef.current?.focus());
     const handleClose = (event: PointerEvent | KeyboardEvent) => {
+      if (event.defaultPrevented) return;
+      const target = event.target;
+      if (target instanceof Element && target.closest('.client-confirmation-popup')) return;
       if (event.type === 'keydown' && (event as KeyboardEvent).key !== 'Escape') return;
       if (event.type !== 'keydown' && (
         panelRef.current?.contains(event.target as Node) || toggleRef.current?.contains(event.target as Node)
@@ -38,7 +46,7 @@ export function useFailoverFeature() {
     };
   }, [isOpen]);
   return {
-    isOpen, panelRef, toggleRef, closeRef, beforeCloseRef, close,
+    isOpen, panelRef, toggleRef, closeRef, beforeCloseRef, pendingTargetRef, close,
     toggle: () => isOpen ? close() : setIsOpen(true),
   };
 }
@@ -67,6 +75,14 @@ export function FailoverToggle({ feature, open, onToggle }: {
 }
 
 const seconds = (milliseconds: number) => Math.round(milliseconds / 1000);
+const normalizeFailoverDraft = (policy: FailoverPolicy) => {
+  const normalized = normalizeFailoverPolicy(policy);
+  return {
+    ...normalized,
+    minimumReserveMs: Math.round(normalized.minimumReserveMs / 60_000) * 60_000,
+  };
+};
+
 
 function pulseNumber(event: FormEvent<HTMLInputElement>) {
   if (matchMedia('(prefers-reduced-motion: reduce)').matches) return;
@@ -309,7 +325,7 @@ export function FailoverPanel({
   onCheck: () => Promise<unknown>;
   onUpdateDiagnostics: (settings: unknown) => Promise<unknown>;
 }) {
-  const [draft, setDraft] = useState(() => snapshot.policy);
+  const [draft, setDraft] = useState(() => normalizeFailoverDraft(snapshot.policy));
   const [dirty, setDirty] = useState(false);
   const [confirmDiscard, setConfirmDiscard] = useState(false);
   const [addingService, setAddingService] = useState(false);
@@ -319,25 +335,35 @@ export function FailoverPanel({
   const [openPicker, setOpenPicker] = useState('');
   const [removingCheckId, setRemovingCheckId] = useState('');
   const [manualChecking, setManualChecking] = useState(false);
+  const discardContinuationFrameRef = useRef<number | null>(null);
   useEffect(() => {
-    if (!dirty) setDraft(snapshot.policy);
+    if (!dirty) setDraft(normalizeFailoverDraft(snapshot.policy));
   }, [snapshot.policy, dirty]);
   useEffect(() => {
     feature.beforeCloseRef.current = () => {
-      if (!dirty) return true;
+      if (!dirty) {
+        feature.pendingTargetRef.current = null;
+        return true;
+      }
       setConfirmDiscard(true);
       return false;
     };
     const beforeUnload = (event: BeforeUnloadEvent) => {
       if (!dirty) return;
       event.preventDefault();
+      event.returnValue = '';
     };
     window.addEventListener('beforeunload', beforeUnload);
     return () => {
       feature.beforeCloseRef.current = () => true;
       window.removeEventListener('beforeunload', beforeUnload);
     };
-  }, [dirty, feature.beforeCloseRef]);
+  }, [dirty, feature.beforeCloseRef, feature.pendingTargetRef]);
+  useEffect(() => () => {
+    if (discardContinuationFrameRef.current !== null) {
+      cancelAnimationFrame(discardContinuationFrameRef.current);
+    }
+  }, []);
   const services = useMemo(() => [
     ...CONNECTIVITY_SITES,
     ...diagnostics.customServices,
@@ -368,11 +394,11 @@ export function FailoverPanel({
     ...(diagnostics.customServices.length < MAX_CUSTOM_DIAGNOSTIC_SERVICES ? [{ id: '__new-service', label: 'Новый HTTPS-сервис…' }] : []),
   ], [diagnostics.customServices.length, draft.checks, services]);
   const update = (value: Partial<FailoverPolicy>) => {
-    setDraft((current) => normalizeFailoverPolicy({ ...current, ...value }));
+    setDraft((current) => normalizeFailoverDraft({ ...current, ...value }));
     setDirty(true);
   };
   const updateChecks = (change: (checks: FailoverPolicy['checks']) => FailoverPolicy['checks']) => {
-    setDraft((current) => normalizeFailoverPolicy({ ...current, checks: change(current.checks) }));
+    setDraft((current) => normalizeFailoverDraft({ ...current, checks: change(current.checks) }));
     setDirty(true);
   };
   const updateTarget = (channel: 'primary' | 'reserve', optionId: string) => {
@@ -383,6 +409,7 @@ export function FailoverPanel({
     ? snapshot.currentRole
     : null;
   const switchRole = role === 'primary' ? 'reserve' : role === 'reserve' ? 'primary' : null;
+  const checking = manualChecking || snapshot.reason === 'checking-channels';
   const activity = snapshot.trafficActivity;
   const targetExists = (target: FailoverPolicy['primary']) => profiles
     .find(({ id }) => id === target.profileId)?.servers.some(({ id }) => id === target.serverId);
@@ -437,27 +464,48 @@ export function FailoverPanel({
       <span>Gateway</span>
       <div className="client-failover-heading">
         <h2>Резервный канал</h2>
-        <HarborSwitch
-          checked={draft.enabled}
-          label={draft.enabled ? 'Выключить резервный канал' : 'Включить резервный канал'}
-          disabled={blocked}
-          onChange={(enabled) => {
-            setOpenPicker('');
-            if (!enabled) setAddingService(false);
-            update({ enabled });
-          }}
-        />
+        <div className="client-failover-header-actions">
+          <div className="client-failover-switch-setting">
+            <span>Канал</span>
+            <HarborSwitch
+              checked={draft.enabled}
+              label={draft.enabled ? 'Выключить резервный канал' : 'Включить резервный канал'}
+              disabled={blocked}
+              onChange={(enabled) => {
+                setOpenPicker('');
+                if (!enabled) setAddingService(false);
+                update({ enabled });
+              }}
+            />
+          </div>
+          <div className="client-failover-switch-setting">
+            <span>Автоматика</span>
+            <HarborSwitch
+              checked={!snapshot.paused}
+              label={snapshot.paused ? 'Включить автоматическое переключение' : 'Остановить автоматическое переключение'}
+              disabled={blocked || !snapshot.enabled}
+              onChange={(automatic) => void onPause(!automatic)}
+            />
+          </div>
+          <button
+            className="client-failover-save"
+            type="submit"
+            form="client-failover-form"
+            disabled={blocked || !dirty || !draftValid}
+          >
+            Сохранить
+          </button>
+        </div>
       </div>
-      <p>Если основной канал перестанет работать, Harbor направит новые подключения через резервный канал. Уже открытые подключения продолжат работать через прежний канал.</p>
-      {snapshot.enabled && <div className="client-failover-global-actions">
-        <button type="button" disabled={blocked} onClick={() => void onPause(!snapshot.paused)}>{snapshot.paused ? 'Продолжить автоматическое переключение' : 'Приостановить автоматическое переключение'}</button>
-        <small>{snapshot.paused ? 'Новые подключения продолжают идти через текущий канал.' : 'Harbor проверяет оба канала и при необходимости переключает новые подключения.'}</small>
-      </div>}
     </header>
+    <div className="client-failover-intro">
+      <p>Если основной канал перестанет работать, Harbor направит новые подключения через резервный канал. Уже открытые подключения продолжат работать через прежний канал.</p>
+      {snapshot.enabled && <small className="client-failover-pause-hint">{snapshot.paused ? 'Новые подключения продолжают идти через текущий канал.' : 'Harbor проверяет оба канала и при необходимости переключает новые подключения.'}</small>}
+    </div>
 
-    <form className="client-failover-form" onSubmit={(event) => {
+    <form id="client-failover-form" className="client-failover-form" onSubmit={(event) => {
       event.preventDefault();
-      void onSave(draft).then((result) => { if (result !== false) setDirty(false); });
+      void onSave(normalizeFailoverDraft(draft)).then((result) => { if (result !== false) setDirty(false); });
     }}>
       <section className="client-failover-channels" aria-label="Основной и резервный каналы">
       {(['primary', 'reserve'] as const).map((channel) => {
@@ -479,7 +527,7 @@ export function FailoverPanel({
             : channel === 'primary' && snapshot.currentRole === 'reserve' && health === 'healthy' && snapshot.reason === 'recovery-hold'
               ? 'Проверяем восстановление'
               : health === 'healthy' ? 'Работает' : health === 'unhealthy' ? 'Недоступен' : health === 'not-monitoring' ? 'Проверка выключена' : 'Ждёт проверки';
-        return <article className="client-failover-channel" data-channel={channel} key={channel}>
+        return <div className="client-failover-channel-slot" key={channel}><article className="client-failover-channel" data-channel={channel}>
           <div className="client-failover-channel-title">
             <h3>{channel === 'primary' ? 'Основной' : 'Резервный'}</h3>
             <strong className={health === 'healthy' ? 'is-healthy' : health === 'unhealthy' || missing ? 'is-unhealthy' : ''}>{healthLabel}</strong>
@@ -494,13 +542,63 @@ export function FailoverPanel({
             onOpenChange={(open) => setOpenPicker(open ? channel : '')}
             onChange={(optionId) => updateTarget(channel, optionId)}
           />
-        </article>;
+        </article>{channel === 'primary' && <div className="client-failover-channel-actions" aria-label="Действия с каналами">
+          <button
+            className={`client-failover-check-action client-tooltip-anchor${checking ? ' is-running' : ''}`}
+            type="button"
+            aria-label="Проверить оба канала"
+            aria-busy={checking}
+            disabled={blocked || checking || !snapshot.enabled || snapshot.activation !== 'active'}
+            onClick={() => {
+              setManualChecking(true);
+              void onCheck().finally(() => setManualChecking(false));
+            }}
+          >
+            <svg viewBox="0 0 24 24" aria-hidden="true">
+              <path d="M20 11a8 8 0 1 0-2.3 6.7M20 5v6h-6" />
+            </svg>
+            <Tooltip>Проверить оба канала</Tooltip>
+          </button>
+          <button
+            className="client-failover-direction-action client-tooltip-anchor"
+            type="button"
+            aria-label={switchRole ? `Переключить на ${switchRole === 'primary' ? 'основной' : 'резервный'} канал` : 'Переключить канал'}
+            disabled={blocked || !snapshot.enabled || snapshot.activation !== 'active' || !switchRole}
+            onClick={() => { if (switchRole) void onSwitch(switchRole); }}
+          >
+            <svg className={`client-failover-direction-icon ${switchRole === 'primary' ? 'is-primary' : 'is-reserve'}`} viewBox="0 0 24 24" aria-hidden="true">
+              <path d="M4 12h15M14 7l5 5-5 5" />
+            </svg>
+            <Tooltip>{switchRole ? `Переключить на ${switchRole === 'primary' ? 'основной' : 'резервный'} канал` : 'Переключение недоступно'}</Tooltip>
+          </button>
+        </div>}</div>;
       })}
         <div className="client-failover-checking" role="status" aria-live="polite">
           {(manualChecking || snapshot.reason === 'checking-channels' ? ['primary', 'reserve'] as const : []).map((checkingRole) => <span key={checkingRole}>Проверяем {checkingRole === 'primary' ? 'основной' : 'резервный'} канал</span>)}
         </div>
       </section>
 
+      <section className={`client-failover-runtime ${role === 'primary' ? 'is-primary' : role === 'reserve' ? 'is-reserve' : 'is-none'}`} aria-label="Текущее состояние каналов">
+        <span className="client-failover-runtime-label">Сейчас используется</span>
+        <strong role="status" aria-live="polite">{blocked
+          ? 'Harbor завершает текущее действие'
+          : role === 'primary'
+            ? 'Основной канал'
+            : role === 'reserve'
+              ? 'Резервный канал'
+              : 'Канал пока не выбран'}</strong>
+        <span>{blocked ? 'Подождите…' : reasonLabel(snapshot.reason)}</span>
+        <span>{role
+          ? `Новые подключения: ${targetLabel(profiles, snapshot[role].target)}`
+          : snapshot.reason === 'vpn-stopped'
+            ? `При запуске VPN будет выбран: ${targetLabel(profiles, draft.primary)}`
+            : `После перезапуска VPN будет выбран: ${targetLabel(profiles, draft.primary)}`}</span>
+        {snapshot.nextDecisionAt && <span>Проверим каналы снова через {seconds(Math.max(0, Date.parse(snapshot.nextDecisionAt) - Date.now()))} с</span>}
+        {activity && <span>{activity.state === 'active' ? `Сейчас передаётся ${Math.round(activity.totalBytesPerSecond / 1024)} КБ/с. Активных подключений: ${activity.transmittingConnections}` : activity.state === 'quiet' ? `Передачи данных нет: ${Math.min(seconds(quietElapsed), seconds(draft.trafficGuard.quietWindowMs))} из ${seconds(draft.trafficGuard.quietWindowMs)} с` : 'Не удалось проверить передачу данных. Автоматическое переключение остановлено'}</span>}
+        {activity?.blockers.slice(0, 2).map((blocker) => <small key={`${blocker.device}:${blocker.service}`}>{blocker.device} · {blocker.service} · {Math.round((blocker.uploadBytesPerSecond + blocker.downloadBytesPerSecond) / 1024)} КБ/с</small>)}
+        {activity && activity.blockers.length > 2 && <small>Ещё {activity.blockers.length - 2}</small>}
+      </section>
+
       <fieldset className="client-failover-services">
         <legend><span className="client-failover-section-header"><strong>Что проверять</strong><small>Harbor проверяет эти сайты через оба канала, чтобы понять, какой канал работает.</small></span></legend>
         {draft.checks.map((check) => {
@@ -595,42 +693,41 @@ export function FailoverPanel({
           <p>Если основной канал снова перестаёт работать, Harbor дольше оставляет новые подключения на резервном канале.</p>
         </header>
         <div className="client-failover-protection-options">
-          <FailoverNumberSetting before="Оставаться на резервном канале не меньше" after="секунд" min={60} max={86400} ariaLabel="Минимальное время на резервном канале" value={seconds(draft.minimumReserveMs)} disabled={blocked || !draft.enabled} onChange={(value) => update({ minimumReserveMs: value * 1_000 })} />
+          <FailoverNumberSetting before="Оставаться на резервном канале не меньше" after="минут" min={1} max={1440} step={1} ariaLabel="Минимальное время на резервном канале" value={draft.minimumReserveMs / 60_000} disabled={blocked || !draft.enabled} onChange={(value) => update({ minimumReserveMs: Math.round(value) * 60_000 })} />
           <FailoverNumberSetting before="Считать основной канал нестабильным после" after="повторных сбоев" min={2} max={10} step={1} ariaLabel="Количество повторных сбоев" value={draft.flapProtection.count} disabled={blocked || !draft.enabled} onChange={(value) => update({ flapProtection: { ...draft.flapProtection, count: value } })} />
           <FailoverNumberSetting before="Учитывать сбои, случившиеся не более" after="ч назад" min={1} max={72} step={1} ariaLabel="Период повторных сбоев" value={Math.round(draft.flapProtection.windowMs / 3_600_000)} disabled={blocked || !draft.enabled} onChange={(value) => update({ flapProtection: { ...draft.flapProtection, windowMs: value * 3_600_000 } })} />
           <FailoverNumberSetting before="После повторных сбоев не возвращаться на основной канал" after="минут" min={10} max={10080} ariaLabel="Длительность карантина основного канала" value={Math.round(draft.flapProtection.quarantineMs / 60_000)} disabled={blocked || !draft.enabled} onChange={(value) => update({ flapProtection: { ...draft.flapProtection, quarantineMs: value * 60_000 } })} />
         </div>
       </section>
 
-      <div className="client-failover-runtime">
-        <strong role="status" aria-live="polite">{blocked ? 'Подождите, Harbor завершает текущее действие…' : snapshot.activation === 'pending' ? 'Резервный канал включится при следующем запуске VPN' : reasonLabel(snapshot.reason)}</strong>
-        <span>{role
-          ? `Новые подключения: ${targetLabel(profiles, snapshot[role].target)}`
-          : snapshot.reason === 'vpn-stopped'
-            ? `При запуске VPN будет выбран: ${targetLabel(profiles, draft.primary)}`
-            : `После перезапуска VPN будет выбран: ${targetLabel(profiles, draft.primary)}`}</span>
-        {snapshot.nextDecisionAt && <span>Проверим каналы снова через {seconds(Math.max(0, Date.parse(snapshot.nextDecisionAt) - Date.now()))} с</span>}
-        {activity && <span>{activity.state === 'active' ? `Сейчас передаётся ${Math.round(activity.totalBytesPerSecond / 1024)} КБ/с. Активных подключений: ${activity.transmittingConnections}` : activity.state === 'quiet' ? `Передачи данных нет: ${Math.min(seconds(quietElapsed), seconds(draft.trafficGuard.quietWindowMs))} из ${seconds(draft.trafficGuard.quietWindowMs)} с` : 'Не удалось проверить передачу данных. Автоматическое переключение остановлено'}</span>}
-        {activity?.blockers.slice(0, 2).map((blocker) => <small key={`${blocker.device}:${blocker.service}`}>{blocker.device} · {blocker.service} · {Math.round((blocker.uploadBytesPerSecond + blocker.downloadBytesPerSecond) / 1024)} КБ/с</small>)}
-        {activity && activity.blockers.length > 2 && <small>Ещё {activity.blockers.length - 2}</small>}
-      </div>
-
       {!draftValid && <p className="client-failover-validation">Выберите разные серверы для основного и резервного каналов и хотя бы один сайт для проверки.</p>}
-      {confirmDiscard && <div className="client-failover-discard" role="alert">
-        <span>Отменить несохранённые изменения?</span>
-        <button type="button" onClick={() => { setDraft(snapshot.policy); setDirty(false); setConfirmDiscard(false); feature.beforeCloseRef.current = () => true; feature.close(); }}>Отменить изменения</button>
-        <button type="button" onClick={() => setConfirmDiscard(false)}>Продолжить настройку</button>
-      </div>}
-
-      <div className="client-failover-actions">
-        <button type="button" disabled={blocked || manualChecking || !snapshot.enabled || snapshot.activation !== 'active'} onClick={() => {
-          setManualChecking(true);
-          void onCheck().finally(() => setManualChecking(false));
-        }}>Проверить оба канала</button>
-        <button type="submit" disabled={blocked || !dirty || !draftValid}>Сохранить</button>
-        {switchRole && snapshot.enabled && snapshot.activation === 'active' && <button type="button" disabled={blocked} onClick={() => void onSwitch(switchRole)}>Переключить новые подключения на {switchRole === 'reserve' ? 'резервный канал' : 'основной канал'}</button>}
-        {(snapshot.currentRole === 'primary' || snapshot.currentRole === 'reserve') && <small>Открытые подключения останутся на прежнем канале.</small>}
-      </div>
     </form>
+    <ConfirmationDialog
+      open={confirmDiscard}
+      id="discard-failover"
+      title="Закрыть без сохранения?"
+      description="Несохранённые настройки резервного канала будут потеряны."
+      cancelLabel="Остаться"
+      confirmLabel="Закрыть без сохранения"
+      onCancel={() => {
+        feature.pendingTargetRef.current = null;
+        setConfirmDiscard(false);
+      }}
+      onConfirm={() => {
+        const continueNavigation = feature.pendingTargetRef.current;
+        feature.pendingTargetRef.current = null;
+        feature.beforeCloseRef.current = () => true;
+        flushSync(() => {
+          setDraft(normalizeFailoverDraft(snapshot.policy));
+          setDirty(false);
+          setConfirmDiscard(false);
+        });
+        discardContinuationFrameRef.current = requestAnimationFrame(() => {
+          discardContinuationFrameRef.current = null;
+          if (continueNavigation) continueNavigation();
+          else feature.close();
+        });
+      }}
+    />
   </Drawer>;
 }