Follow-ups from #1 #4

Open
opened 2026-10-24 09:07:01 +03:00 by aep · 3 comments
Owner

Отложенные находки MVP агента (PR #2). Подробности в архиве openspec/changes/archive/2026-10-24-eva-agent-mvp/design.md.

Проверить на первой живой синхронизации

  • M-5 — SSO-редирект на файлах: страница входа сохранится как вложение; проверить Content-Type (финальное ревью).
  • Решение 7 — отдаёт ли Eva файл без ?version=1 и принимает ли Bearer на /files/….
  • M-4 — соблюдает ли Eva slice (иначе дерево неполное, видно только в логе).

Код и тесты

  • T1.1 — нет теста паузы на 403, только 401 (ревью Task 1).
  • T2.1 — EvaClient.call переопределяет Http.call с другой сигнатурой (Task 2).
  • T2.2 — тест скраба токена в ConnectionError проверяет не по-настоящему (Task 2).
  • T3.1 — regex DOC-кода без якоря сегмента пути (Task 3).
  • T3.2 — сравнение хоста регистрозависимо, без учёта порта по умолчанию (Task 3).
  • T3.3 — незакрытый <a> копит текст в имя вложения (Task 3).
  • T6.2 — HUB_SRC в e2e переопределяется окружением без проверки (Task 6).
  • T6.3 — токены e2e в argv curl (Task 6).
  • O-3 — устаревший абзац CLAUDE.md о test_runner (наблюдение контроллера).
Отложенные находки MVP агента (PR #2). Подробности в архиве `openspec/changes/archive/2026-10-24-eva-agent-mvp/design.md`. **Проверить на первой живой синхронизации** - M-5 — SSO-редирект на файлах: страница входа сохранится как вложение; проверить Content-Type (финальное ревью). - Решение 7 — отдаёт ли Eva файл без `?version=1` и принимает ли Bearer на `/files/…`. - M-4 — соблюдает ли Eva `slice` (иначе дерево неполное, видно только в логе). **Код и тесты** - T1.1 — нет теста паузы на 403, только 401 (ревью Task 1). - T2.1 — EvaClient.call переопределяет Http.call с другой сигнатурой (Task 2). - T2.2 — тест скраба токена в ConnectionError проверяет не по-настоящему (Task 2). - T3.1 — regex DOC-кода без якоря сегмента пути (Task 3). - T3.2 — сравнение хоста регистрозависимо, без учёта порта по умолчанию (Task 3). - T3.3 — незакрытый `<a>` копит текст в имя вложения (Task 3). - T6.2 — HUB_SRC в e2e переопределяется окружением без проверки (Task 6). - T6.3 — токены e2e в argv curl (Task 6). - O-3 — устаревший абзац CLAUDE.md о test_runner (наблюдение контроллера).
Author
Owner

Добавлено 25.10.2026 при установке на виртуалку: битый agent.json (управляющий символ в строке токена) роняет check трейсбеком JSONDecodeError из load_config; нужно понятное сообщение «agent.json: ошибка JSON в строке N, столбце M: …» и код выхода, как при отсутствии файла.

Добавлено 25.10.2026 при установке на виртуалку: битый agent.json (управляющий символ в строке токена) роняет `check` трейсбеком `JSONDecodeError` из `load_config`; нужно понятное сообщение «agent.json: ошибка JSON в строке N, столбце M: …» и код выхода, как при отсутствии файла.
Author
Owner

Живая проверка 25.10.2026 (eva-agent v1, хаб v7):

  • wiki_sync AWPCA, корень DOC-019855: 28 страниц, 10 ссылок page:, 37 вложений, сырого HTML в копии нет, поиск работает.
  • wiki_fetch вложения 1 (xlsx, 560 832 байт): файл настоящий (сигнатура PK), т.е. Bearer на /files/… принимается, без ?version=1 файл отдаётся, SSO-редиректа нет — пункты «Решение 7» и M-5 для этого случая сняты (M-5 оставить на протухший токен).
  • slice: дерево полное по сравнению со структурой ТЗ (выгрузка awp-ca 22 страницы — старее), явных потерь не видно.
Живая проверка 25.10.2026 (eva-agent v1, хаб v7): - wiki_sync AWPCA, корень DOC-019855: 28 страниц, 10 ссылок page:, 37 вложений, сырого HTML в копии нет, поиск работает. - wiki_fetch вложения 1 (xlsx, 560 832 байт): файл настоящий (сигнатура PK), т.е. Bearer на /files/… принимается, без ?version=1 файл отдаётся, SSO-редиректа нет — пункты «Решение 7» и M-5 для этого случая сняты (M-5 оставить на протухший токен). - slice: дерево полное по сравнению со структурой ТЗ (выгрузка awp-ca 22 страницы — старее), явных потерь не видно.
Author
Owner

Сверка 25.10.2026: решение 7 и M-4 сняты живой проверкой 25.10 (M-4 условно — постраничный обход на 28 страницах по сути не проверен). M-5 остаётся: Content-Type не проверяется (src/agent.py:198-232), проверить на протухшем токене или сразу добавить проверку.

Актуальны: Н-1 (битый agent.json → трейсбек JSONDecodeError, src/agent.py:51-56), T1.1, T2.1, T2.2, T3.1–T3.3, T6.2, T6.3, O-3.

Сверка 25.10.2026: решение 7 и M-4 сняты живой проверкой 25.10 (M-4 условно — постраничный обход на 28 страницах по сути не проверен). M-5 остаётся: Content-Type не проверяется (`src/agent.py:198-232`), проверить на протухшем токене или сразу добавить проверку. Актуальны: Н-1 (битый `agent.json` → трейсбек `JSONDecodeError`, `src/agent.py:51-56`), T1.1, T2.1, T2.2, T3.1–T3.3, T6.2, T6.3, O-3.
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aep/eva-agent#4
No description provided.