[spacesarmat]

Логи NUC больше не рисуются под списком контейнеров. То же окно используется для проверки источников, отчёта DHT, отчёта о сбое и подробностей мониторинга.

Окно: заголовок, «Закрыть», Esc, клик по затемнению, фокус не уходит наружу. Текст моноширинный и прокручивается. Есть фильтр строк и «Копировать». У лога контейнера есть «Обновить». На узком экране окно прижато к низу.

Не влито и на NUC не выкатывалось.

Проверено: ops/billing/test/adminLog.test.ts, adminModal.test.tsx, monitorView.test.ts. Сборка админки совпадает с ops/billing/public.

Лог NUC на широком экране

Лог NUC на узком экране

Подробности проверки

Отчёт о сбое

Проверка источников

_{To show artifacts inline, enable in settings.}

  

Confidence Score: 4/5

The PR appears safe to merge, with non-blocking fixes for late log replies and backward keyboard navigation.

Findings

1.  Closed logs open again ▶
2.  Focus leaves the window ▶

Fix with agent prompt

### Issue 1
ops/billing/admin/src/panels/DeskViews.tsx:472-474
Closing a log while its request is running clears `log`, but `openLog` sets it again when the request finishes or fails. The window pops open again, and an older reply can replace a newer log. `CrashesDesk` has the same issue.

Ignore replies after the user closes the window or starts another request.

### Issue 2
ops/billing/admin/src/components/modal.tsx:34
Pressing Shift+Tab immediately after opening `Modal` can move focus onto the page behind it. The effect focuses `root`, but the backward Tab check handles only the first control or an element outside `root`. Since `root.contains(root)` is true, neither case matches the initial focus.

Include `active === root` in this check, or focus the first control when opening.

```suggestion
      if (e.shiftKey && (active === root || active === first || !root.contains(active))) {
```

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

The admin page now opens NUC logs, source checks, DHT reports, crash logs, and monitor details in a shared window.

• Container logs open in a searchable window over the admin page.
• Source checks show their results in a window over the admin page.
• DHT probe results open in a window with readable details.
• Crash logs open in a window over the crash list.
• Monitor checks offer a window with readable details and the full check.

_{Reviews (1) · Last reviewed commit: "Админка: логи и подробности в окне" · Reviewed by Greptile}

[greptile-apps]

Closed logs open again

Closing a log while its request is running clears log, but openLog sets it again when the request finishes or fails. The window pops open again, and an older reply can replace a newer log. CrashesDesk has the same issue.

Ignore replies after the user closes the window or starts another request.

Prompt To Fix With AI

This is a comment left during a code review.
Path: ops/billing/admin/src/panels/DeskViews.tsx
Line: 472-474

Comment:
**Closed logs open again**

Closing a log while its request is running clears `log`, but `openLog` sets it again when the request finishes or fails. The window pops open again, and an older reply can replace a newer log. `CrashesDesk` has the same issue.

Ignore replies after the user closes the window or starts another request.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

[greptile-apps]

Focus leaves the window

Pressing Shift+Tab immediately after opening Modal can move focus onto the page behind it. The effect focuses root, but the backward Tab check handles only the first control or an element outside root. Since root.contains(root) is true, neither case matches the initial focus.

Include active === root in this check, or focus the first control when opening.

Suggested change

	if (e.shiftKey && (active === first || !root.contains(active))) {
	if (e.shiftKey && (active === root || active === first || !root.contains(active))) {

Prompt To Fix With AI

This is a comment left during a code review.
Path: ops/billing/admin/src/components/modal.tsx
Line: 34

Comment:
**Focus leaves the window**

Pressing Shift+Tab immediately after opening `Modal` can move focus onto the page behind it. The effect focuses `root`, but the backward Tab check handles only the first control or an element outside `root`. Since `root.contains(root)` is true, neither case matches the initial focus.

Include `active === root` in this check, or focus the first control when opening.

```suggestion
      if (e.shiftKey && (active === root || active === first || !root.contains(active))) {
```

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!