c75fb35082
Both agents work on other machines and push straight to git. A8 and B7 were pinned to8cddc9fwhile main had already moved to7f912f1, and neither task said to pull first — branching from a stale checkout is how merges revert other people's work. Adds an explicit "update your local copy" section to A8 and B7: fetch, reset to origin/main, record the actual HEAD as BASE_SHA rather than the SHA printed in the document, and branch from fresh main. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
130 lines
11 KiB
Markdown
130 lines
11 KiB
Markdown
# Задание B7 (Codex): дефекты, найденные при живой эксплуатации
|
||
|
||
## Дата поступления
|
||
2026-08-22
|
||
|
||
## База
|
||
Проверочный HEAD на момент выдачи: **`7f912f1`**. Обязательно обновить локальную копию — см. следующий раздел.
|
||
|
||
## Ветка
|
||
`codex/usability-fixes`
|
||
|
||
## Отношение к B6
|
||
Задание **B6 (граф маршрутизации) остаётся в силе**, но это — приоритетнее. Владелец впервые прошёл сценарий вживую, и половина действий не сработала. Сначала чинится то, что он не смог сделать, потом граф.
|
||
|
||
---
|
||
|
||
## Перед началом: обновить локальную копию
|
||
|
||
Задание выдано, когда `origin/main` был `7f912f1`. Ваша рабочая копия на другой машине почти наверняка отстала — за последние сутки в `main` вошло 9 коммитов, включая работу Codex по интерфейсу и принятые задания Antigravity A6–A8.
|
||
|
||
Порядок:
|
||
|
||
```
|
||
cd <каталог репозитория>; git fetch origin --prune; git status
|
||
```
|
||
|
||
Если рабочее дерево чистое:
|
||
|
||
```
|
||
git checkout main; git reset --hard origin/main
|
||
```
|
||
|
||
Если есть незакоммиченные правки — сначала сохранить их отдельной веткой, вслепую сбрасывать нельзя.
|
||
|
||
После обновления **зафиксировать фактический `BASE_SHA`** командой `git rev-parse --short HEAD` и указать его в отчёте. Не считать `7f912f1` актуальным автоматически: пока вы работаете, `main` может уйти вперёд.
|
||
|
||
Ветку создавать **от свежего `origin/main`**, не от старого состояния. Иначе слияние принесёт откат чужой работы — так уже случалось: одна из веток содержала устаревшую распаковку `start_profile_oauth`, и при неаккуратном слиянии подключение Antigravity-аккаунта снова бы сломалось.
|
||
|
||
---
|
||
|
||
|
||
## Что сообщил владелец, дословно
|
||
|
||
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`.
|