Improve failover status feedback and controls

src/web/features/failover/FailoverFeature.tsx

@@ -67,10 +67,6 @@ export function FailoverToggle({ feature, open, onToggle }: {
 }
 
 const seconds = (milliseconds: number) => Math.round(milliseconds / 1000);
-const milliseconds = (value: string, fallback: number) => {
-  const parsed = Number(value);
-  return Number.isFinite(parsed) ? Math.round(parsed * 1000) : fallback;
-};
 
 function pulseNumber(event: FormEvent<HTMLInputElement>) {
   if (matchMedia('(prefers-reduced-motion: reduce)').matches) return;
@@ -80,9 +76,46 @@ function pulseNumber(event: FormEvent<HTMLInputElement>) {
   ], { duration: 320, easing: 'cubic-bezier(0.16, 1, 0.3, 1)' });
 }
 
+function FailoverNumberSetting({ before, after, value, min, max, step = 10, disabled, ariaLabel, onChange, compact = false }: {
+  before: string;
+  after: string;
+  value: number;
+  min: number;
+  max: number;
+  step?: number;
+  disabled?: boolean;
+  ariaLabel: string;
+  onChange: (value: number) => void;
+  compact?: boolean;
+}) {
+  const set = (next: number) => onChange(Math.min(max, Math.max(min, next)));
+  return <div className={`client-failover-number-setting${compact ? ' is-compact' : ''}`}>
+    <span>{before}</span>
+    <input
+      type="number"
+      min={min}
+      max={max}
+      aria-label={ariaLabel}
+      value={value}
+      disabled={disabled}
+      onInput={pulseNumber}
+      onChange={(event) => {
+        const next = Number(event.target.value);
+        if (Number.isFinite(next)) onChange(next);
+      }}
+    />
+    <span>{after}</span>
+    <span className="client-failover-number-steps">
+      <button type="button" aria-label={`${ariaLabel}: уменьшить на ${step}`} disabled={disabled || value <= min} onClick={() => set(value - step)}>−{step}</button>
+      <button type="button" aria-label={`${ariaLabel}: увеличить на ${step}`} disabled={disabled || value >= max} onClick={() => set(value + step)}>+{step}</button>
+    </span>
+  </div>;
+}
+
 const reasonLabel = (reason: string | null) => ({
   'primary-healthy': 'Основной канал работает',
   'health-unknown': 'Ожидаем результаты проверки',
+  'checking-channels': 'Проверяем основной и резервный каналы',
   'failure-window': 'Подтверждаем сбой основного канала',
   'reserve-not-healthy': 'Резервный канал ещё не подтверждён',
   'both-unhealthy': 'Оба канала недоступны',
@@ -90,7 +123,7 @@ const reasonLabel = (reason: string | null) => ({
   'recovery-hold': 'Проверяем стабильность основного канала',
   'activity-unknown': 'Не удалось определить активность',
   'active-traffic': 'Ждём завершения активной работы',
-  'quiet-window': 'Проверяем тишину перед переключением',
+  'quiet-window': 'Ждём паузу в передаче данных',
   'primary-failed': 'Основной канал недоступен',
   'primary-recovered': 'Основной канал восстановился',
   'pending-activation': 'Изменения ожидают следующего запуска VPN',
@@ -107,9 +140,13 @@ const reasonLabel = (reason: string | null) => ({
 function targetLabel(profiles: ProfileSnapshot[], target: { profileId: string; serverId: string }) {
   const profile = profiles.find(({ id }) => id === target.profileId);
   const server = profile?.servers.find(({ id }) => id === target.serverId);
-  return profile && server ? `${profile.label} · ${server.label}` : 'Не выбран';
+  return profile && server ? channelLabel(profile.label, server.label) : 'Не выбран';
 }
 
+const channelLabel = (profile: string, server: string) => profile.trim() === server.trim()
+  ? server
+  : `${profile} · ${server}`;
+
 function HarborSwitch({ checked, label, disabled, onChange }: {
   checked: boolean;
   label: string;
@@ -280,6 +317,7 @@ export function FailoverPanel({
   const [removingCheckId, setRemovingCheckId] = useState('');
   const [advancedOpen, setAdvancedOpen] = useState(false);
   const [advancedClosing, setAdvancedClosing] = useState(false);
+  const [manualChecking, setManualChecking] = useState(false);
   useEffect(() => {
     if (!dirty) setDraft(snapshot.policy);
   }, [snapshot.policy, dirty]);
@@ -305,7 +343,7 @@ export function FailoverPanel({
   ], [diagnostics.customServices]);
   const channelOptions = useMemo(() => profiles.flatMap((profile) => profile.servers.map((server) => ({
     id: `${profile.id}\u001f${server.id}`,
-    label: `${profile.label} · ${server.label}`,
+    label: channelLabel(profile.label, server.label),
     target: { profileId: profile.id, serverId: server.id },
   }))), [profiles]);
   const channelPickerOptions = useMemo(() => ({
@@ -419,12 +457,17 @@ export function FailoverPanel({
         />
       </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 ? 'VPN продолжает работать через текущий канал.' : 'Harbor сам проверяет оба канала.'}</small>
+      </div>}
     </header>
 
     <form className="client-failover-form" onSubmit={(event) => {
       event.preventDefault();
       void onSave(draft).then((result) => { if (result !== false) setDirty(false); });
     }}>
+      <section className="client-failover-channels" aria-label="Основной и резервный каналы">
       {(['primary', 'reserve'] as const).map((channel) => {
         const health = snapshot[channel].health;
         const missing = Boolean(snapshot[channel].target.profileId) && !targetExists(snapshot[channel].target);
@@ -444,9 +487,9 @@ export function FailoverPanel({
             : channel === 'primary' && snapshot.currentRole === 'reserve' && health === 'healthy' && snapshot.reason === 'recovery-hold'
               ? 'Восстанавливается'
               : health === 'healthy' ? 'Работает' : health === 'unhealthy' ? 'Недоступен' : health === 'not-monitoring' ? 'Не проверяется' : 'Ещё не проверен';
-        return <section className="client-failover-channel" data-channel={channel} key={channel}>
+        return <article className="client-failover-channel" data-channel={channel} key={channel}>
           <div className="client-failover-channel-title">
-            <span><b>{channel === 'primary' ? 'Основной' : 'Резервный'}</b><i aria-hidden="true">=</i></span>
+            <h3>{channel === 'primary' ? 'Основной' : 'Резервный'}</h3>
             <strong className={health === 'healthy' ? 'is-healthy' : health === 'unhealthy' || missing ? 'is-unhealthy' : ''}>{healthLabel}</strong>
           </div>
           <FailoverPicker
@@ -459,8 +502,12 @@ export function FailoverPanel({
             onOpenChange={(open) => setOpenPicker(open ? channel : '')}
             onChange={(optionId) => updateTarget(channel, optionId)}
           />
-        </section>;
+        </article>;
       })}
+        <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>
 
       <fieldset className="client-failover-services">
         <legend>Что проверять</legend>
@@ -477,21 +524,20 @@ export function FailoverPanel({
             }}
           >
             <span className="client-failover-service-name">{service?.label || 'Сервис недоступен'}</span>
-            <label className="client-failover-service-timeout">
-              <span>таймаут, сек</span>
-              <input
-                type="number"
-                min="2"
-                max="30"
-                aria-label={`Таймаут проверки: ${service?.label || 'сервис недоступен'}`}
-                value={seconds(check.timeoutMs)}
-                disabled={blocked || !draft.enabled}
-                onInput={pulseNumber}
-                onChange={(event) => updateChecks((checks) => checks.map((item) => item.serviceId === check.serviceId
-                  ? { ...item, timeoutMs: milliseconds(event.target.value, item.timeoutMs) }
-                  : item))}
-              />
-            </label>
+            <FailoverNumberSetting
+              compact
+              before="Ждать ответ не дольше"
+              after="секунд"
+              min={2}
+              max={30}
+              step={1}
+              ariaLabel={`Время ожидания ответа: ${service?.label || 'сервис недоступен'}`}
+              value={seconds(check.timeoutMs)}
+              disabled={blocked || !draft.enabled}
+              onChange={(value) => updateChecks((checks) => checks.map((item) => item.serviceId === check.serviceId
+                ? { ...item, timeoutMs: value * 1_000 }
+                : item))}
+            />
             <button
               className="client-failover-remove-service"
               type="button"
@@ -532,20 +578,20 @@ export function FailoverPanel({
       </fieldset>
 
       <section className="client-failover-timing" aria-label="Пороги переключения">
-        <label><span>Проверять каждые, сек</span><input type="number" min="15" max="900" disabled={blocked || !draft.enabled} value={seconds(draft.intervalMs)} onInput={pulseNumber} onChange={(event) => update({ intervalMs: milliseconds(event.target.value, draft.intervalMs) })} /></label>
-        <label><span>Сбой должен длиться, сек</span><input type="number" min={seconds(draft.intervalMs * 2)} max="1800" disabled={blocked || !draft.enabled} value={seconds(draft.failureWindowMs)} onInput={pulseNumber} onChange={(event) => update({ failureWindowMs: milliseconds(event.target.value, draft.failureWindowMs) })} /></label>
-        <label><span>Восстановление, сек</span><input type="number" min="60" max="86400" disabled={blocked || !draft.enabled} value={seconds(draft.recoveryWindowMs)} onInput={pulseNumber} onChange={(event) => update({ recoveryWindowMs: milliseconds(event.target.value, draft.recoveryWindowMs) })} /></label>
+        <FailoverNumberSetting before="Проверять каналы каждые" after="секунд" min={15} max={900} ariaLabel="Интервал проверки каналов" value={seconds(draft.intervalMs)} disabled={blocked || !draft.enabled} onChange={(value) => update({ intervalMs: value * 1_000 })} />
+        <FailoverNumberSetting before="Считать основной недоступным после" after="секунд непрерывного сбоя" min={seconds(draft.intervalMs * 2)} max={1800} ariaLabel="Длительность сбоя основного канала" value={seconds(draft.failureWindowMs)} disabled={blocked || !draft.enabled} onChange={(value) => update({ failureWindowMs: value * 1_000 })} />
+        <FailoverNumberSetting before="Вернуться на основной после" after="секунд стабильной работы" min={60} max={86400} ariaLabel="Время стабильной работы основного канала" value={seconds(draft.recoveryWindowMs)} disabled={blocked || !draft.enabled} onChange={(value) => update({ recoveryWindowMs: value * 1_000 })} />
         <div className="client-failover-traffic-switch">
-          <span><strong>Переключение во время работы</strong><small>{draft.trafficGuard.enabled ? 'Ждать тишины перед автоматическим переключением' : 'Переключать сразу после подтверждённого сбоя'}</small></span>
+          <span><strong>Не переключать во время активной работы</strong><small>{draft.trafficGuard.enabled ? 'Harbor дождётся снижения трафика, чтобы не мешать новым подключениям.' : 'Новые соединения переключатся сразу после подтверждённого сбоя.'}</small></span>
           <HarborSwitch
             checked={draft.trafficGuard.enabled}
-            label={draft.trafficGuard.enabled ? 'Не ждать завершения активной работы' : 'Ждать завершения активной работы'}
+            label={draft.trafficGuard.enabled ? 'Разрешить переключение во время активной работы' : 'Не переключать во время активной работы'}
             disabled={blocked || !draft.enabled}
             onChange={(enabled) => update({ trafficGuard: { ...draft.trafficGuard, enabled } })}
           />
         </div>
-        <label><span>Тишина перед переключением, сек</span><input type="number" min="5" max="600" disabled={blocked || !draft.enabled || !draft.trafficGuard.enabled} value={seconds(draft.trafficGuard.quietWindowMs)} onInput={pulseNumber} onChange={(event) => update({ trafficGuard: { ...draft.trafficGuard, quietWindowMs: milliseconds(event.target.value, draft.trafficGuard.quietWindowMs) } })} /></label>
-        <label><span>Активный трафик, КБ/с</span><input type="number" min="1" max="102400" disabled={blocked || !draft.enabled || !draft.trafficGuard.enabled} value={Math.round(draft.trafficGuard.thresholdBytesPerSecond / 1024)} onInput={pulseNumber} onChange={(event) => update({ trafficGuard: { ...draft.trafficGuard, thresholdBytesPerSecond: Number(event.target.value) * 1024 } })} /></label>
+        <FailoverNumberSetting before="Переключать после" after="секунд без активной передачи данных" min={5} max={600} ariaLabel="Время без активной передачи данных" value={seconds(draft.trafficGuard.quietWindowMs)} disabled={blocked || !draft.enabled || !draft.trafficGuard.enabled} onChange={(value) => update({ trafficGuard: { ...draft.trafficGuard, quietWindowMs: value * 1_000 } })} />
+        <FailoverNumberSetting before="Считать работу активной при скорости от" after="КБ/с" min={1} max={102400} ariaLabel="Скорость активной работы" value={Math.round(draft.trafficGuard.thresholdBytesPerSecond / 1024)} disabled={blocked || !draft.enabled || !draft.trafficGuard.enabled} onChange={(value) => update({ trafficGuard: { ...draft.trafficGuard, thresholdBytesPerSecond: value * 1024 } })} />
       </section>
 
       <section className={`client-failover-advanced${advancedOpen ? ' is-open' : ''}${advancedClosing ? ' is-closing' : ''}`}>
@@ -563,10 +609,10 @@ export function FailoverPanel({
             setAdvancedOpen(false);
           }}
         >
-          <label><span>Минимум на резерве, сек</span><input type="number" min="60" max="86400" disabled={blocked || !draft.enabled} value={seconds(draft.minimumReserveMs)} onInput={pulseNumber} onChange={(event) => update({ minimumReserveMs: milliseconds(event.target.value, draft.minimumReserveMs) })} /></label>
-          <label><span>Падений до карантина</span><input type="number" min="2" max="10" disabled={blocked || !draft.enabled} value={draft.flapProtection.count} onInput={pulseNumber} onChange={(event) => update({ flapProtection: { ...draft.flapProtection, count: Number(event.target.value) } })} /></label>
-          <label><span>Окно повторных сбоев, ч</span><input type="number" min="1" max="72" disabled={blocked || !draft.enabled} value={Math.round(draft.flapProtection.windowMs / 3_600_000)} onInput={pulseNumber} onChange={(event) => update({ flapProtection: { ...draft.flapProtection, windowMs: Number(event.target.value) * 3_600_000 } })} /></label>
-          <label><span>Карантин основного, мин</span><input type="number" min="10" max="10080" disabled={blocked || !draft.enabled} value={Math.round(draft.flapProtection.quarantineMs / 60_000)} onInput={pulseNumber} onChange={(event) => update({ flapProtection: { ...draft.flapProtection, quarantineMs: Number(event.target.value) * 60_000 } })} /></label>
+          <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={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>
 
@@ -578,7 +624,7 @@ export function FailoverPanel({
             ? `При запуске 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 && <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>
@@ -591,11 +637,13 @@ export function FailoverPanel({
       </div>}
 
       <div className="client-failover-actions">
-        <button type="button" disabled={blocked || !snapshot.enabled || snapshot.activation !== 'active'} onClick={() => void onCheck()}>Проверить оба канала</button>
+        <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>
-        {snapshot.enabled && <button type="button" disabled={blocked} onClick={() => void onPause(!snapshot.paused)}>{snapshot.paused ? 'Возобновить' : 'Пауза'}</button>}
-        {switchRole && snapshot.enabled && snapshot.activation === 'active' && <button type="button" disabled={blocked} aria-label={`Переключить новые соединения на ${switchRole === 'reserve' ? 'резервный' : 'основной'} сейчас`} onClick={() => void onSwitch(switchRole)}>Переключить новые сейчас</button>}
-        {(snapshot.currentRole === 'primary' || snapshot.currentRole === 'reserve') && <small>Текущие соединения Harbor не закроет</small>}
+        {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>
   </Drawer>;