Hermes Team
20078f5ff6
The owner used the product and the router proved itself: the log shows a real failover chain codex-orch -> ag-orch-fallback -> opengo-3 driven by actual quota errors. Everything around that core failed. Hub stopped launching because customtkinter vanished from the Hermes venv, most likely wiped by that app's repair-install; pythonw swallows the traceback, so it looks like the window never opens. The wizard offers five providers while the router config defines profiles for three, so find_free_slot returns a non-existent claude-orch. The test action still reaches the agy subprocess, which opens a browser on expired credentials. The routing "configure" button only raises a toast about drag-and-drop. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
105 lines
8.9 KiB
Markdown
105 lines
8.9 KiB
Markdown
# Задание B7 (Codex): дефекты, найденные при живой эксплуатации
|
||
|
||
## Дата поступления
|
||
2026-08-22
|
||
|
||
## База
|
||
Проверочный HEAD: **`8cddc9f`**, `origin/main` = `8cddc9f`. `git fetch`, зафиксировать `BASE_SHA`.
|
||
|
||
## Ветка
|
||
`codex/usability-fixes`
|
||
|
||
## Отношение к B6
|
||
Задание **B6 (граф маршрутизации) остаётся в силе**, но это — приоритетнее. Владелец впервые прошёл сценарий вживую, и половина действий не сработала. Сначала чинится то, что он не смог сделать, потом граф.
|
||
|
||
---
|
||
|
||
## Что сообщил владелец, дословно
|
||
|
||
1. «завершить не нажимается. пишет что все исчерпано, но нет»
|
||
2. «при тесте открывается опять окно авторизации и ничего»
|
||
3. «появляется код, куда его вставлять, не понятно»
|
||
4. «назначить роль не получается, ничего не видно»
|
||
5. «в маршрутизации при нажатии на кнопку настроить ничего не происходит»
|
||
|
||
Каждое проверено по коду. Ниже — что относится к вам; пункты 1 (частично) и 2 уходят в backend отдельным заданием A8.
|
||
|
||
## P0-1. «Настроить» в маршрутизации не делает ничего
|
||
|
||
`hermes_hub_app.py:632`:
|
||
|
||
```python
|
||
elif action == "edit_route":
|
||
self._show_toast("Редактор цепочки использует кнопки и селекторы; drag-and-drop отключён.")
|
||
```
|
||
|
||
Кнопка показывает сообщение про отключённый drag-and-drop — и всё. С точки зрения пользователя нажатие не делает ничего, а текст объясняет то, чего он не спрашивал.
|
||
|
||
Требуется настоящий редактор цепочки для роли: список профилей в порядке приоритета, изменение порядка, добавление и удаление профиля из цепочки, сохранение через `AutoAssigner`. Кнопочный и селекторный, без drag-and-drop — это оговорено и допустимо. Но он должен существовать.
|
||
|
||
## P0-2. Результат действия не виден
|
||
|
||
«назначить роль не получается, ничего не видно».
|
||
|
||
Проверил: модальное окно назначения роли **открывается** и содержит семь вариантов — здесь дефекта нет. Проблема в обратной связи: результат уходит в `_show_toast`, то есть в строку состояния внизу окна, где его легко не заметить. Если профиль не найден (а при выборе Claude или Grok он сейчас действительно не найден — см. A8), пользователь видит ровно ничего.
|
||
|
||
Требуется:
|
||
- результат действия показывать заметно: в самой модалке до закрытия либо всплывающим уведомлением рядом с местом действия;
|
||
- **при ошибке модалку не закрывать** — сейчас `modal.destroy()` вызывается до показа результата, и человек остаётся без контекста;
|
||
- то же для остальных действий карточки: «Тест», «Основной», «Оркестратор».
|
||
|
||
## P0-3. Код авторизации: непонятно, что с ним делать
|
||
|
||
«появляется код, куда его вставлять, не понятно».
|
||
|
||
Сейчас в мастере для Codex и Grok показывается поле со ссылкой, отдельная метка с кодом и статус «Ожидание подтверждения кода XXX в браузере…». Ни одной фразы о том, что нужно сделать.
|
||
|
||
Требуется явная пронумерованная последовательность прямо в шаге:
|
||
|
||
```
|
||
1. Откройте ссылку — [кнопка «Открыть в браузере»] [копировать]
|
||
2. Введите на странице код: ABCD-1234 [копировать]
|
||
3. Подтвердите доступ — окно закроется само
|
||
```
|
||
|
||
Код — крупно, моноширинным, с кнопкой копирования. Статус ожидания — ниже, отдельной строкой. Пользователь не должен догадываться о порядке действий.
|
||
|
||
## P0-4. Мастер: честное поведение при отсутствии свободного слота
|
||
|
||
Backend вернёт `None`, когда свободных слотов действительно нет (A8 это чинит). Сейчас мастер подставляет `f"{provider[:3]}-spare-1"` — придуманный идентификатор, который может не существовать:
|
||
|
||
```python
|
||
AutoAssigner.find_free_slot(self.selected_provider) or f"{self.selected_provider[:3]}-spare-1"
|
||
```
|
||
|
||
Требуется: если слот не найден — не выдумывать, а показать понятное объяснение («все слоты этого провайдера заняты, освободите один или удалите неиспользуемый аккаунт») и не давать пройти дальше. Кнопка «Завершить» должна быть либо активной и работающей, либо отключённой с подсказкой почему — но не «нажимается и ничего не происходит».
|
||
|
||
## P1-5. Первый запуск должен вести пользователя
|
||
|
||
Остаётся из B6: на пустой конфигурации «Обзор» показывает «Критическое состояние» и пять `Н/Д` без единой подсказки. Первый экран обязан объяснять следующий шаг и вести к нему.
|
||
|
||
---
|
||
|
||
## Ограничения
|
||
|
||
- Граница: ваша зона — `src/antigravity_provider/router/ui/**`, `hermes_hub_app.py`, `tests/test_ui_*.py`.
|
||
- Не выдумывать идентификаторы, значения и метрики. Нет данных — «Н/Д» либо блок отсутствует.
|
||
- Три темы сохранить.
|
||
- Мастер не ломать: шесть рабочих потоков подключения и трёхэлементная распаковка `start_profile_oauth`.
|
||
- Тег `v0.1.1` не создавать.
|
||
|
||
## Критерии приёмки
|
||
|
||
1. Ни один файл чужой зоны не изменён.
|
||
2. «Настроить» открывает работающий редактор цепочки; изменение сохраняется и видно после перезапуска.
|
||
3. Ошибка любого действия видна пользователю в месте действия; модалка при ошибке остаётся открытой.
|
||
4. Шаг с кодом устройства содержит пронумерованную инструкцию и кнопки копирования для ссылки и кода.
|
||
5. При отсутствии свободного слота мастер объясняет причину и не подставляет выдуманный идентификатор.
|
||
6. Пустое состояние «Обзора» ведёт к подключению аккаунта.
|
||
7. Тесты на каждый пункт: редактор цепочки сохраняет порядок; ошибка действия отображается; мастер без свободных слотов не завершается молча.
|
||
8. Прогон **в обоих окружениях** — без UI-зависимостей и с `customtkinter`/`pillow`/`psutil`; обе команды и оба результата в отчёте.
|
||
9. `ruff check .` чисто; release gate не ухудшен.
|
||
10. **Скриншоты живого сценария**: подключение аккаунта, назначение роли, редактор цепочки — с реальными данными.
|
||
|
||
## Порядок сдачи
|
||
Передать точный `FINAL_COMMIT_SHA`.
|