Follow-ups from #129 #132

New Issue

aep · 2026-10-26T09:22:02+03:00

aep commented

2026-10-26 09:22:02 +03:00

Отложенные находки пакета C бэкенда хаба (#129, PR #130, merge 6f2cf1a, архив #131). Формат: маркер — суть — откуда.

Нормализация параметров (Task 1)

T1.1 — hooks.py:36: проверка filename стала недостижимой после нормализации в validate — ревью Task 1
T1.2 — нет теста retry команды с необрезанными params — ревью Task 1
T1.3 — MCP batch_create: нет теста обрезки key — ревью Task 1
T1.4 — дубль проверки обязательных параметров в validate (до и после нормализации) — ревью Task 1
M-1 — нет теста подстановки реального ключа вместо временного ключа с пробелами — финальное ревью
O-9 — тема ensure_fields с пробелами по краям не подтверждается по зеркалу (цена принята в R-2) — финальный ревьюер

ASCII-цифры (Task 2)

T2.2 — ссылка #<id> на вложение в MCP без теста — ревью Task 2
T2.3 — разбор дат (history/mcp) через is_digits без теста отката — ревью Task 2
O-6 — маршруты вики с int_group/\d принимают цифры Unicode ("١٢" → id 12); вне спеки — финальный ревьюер

Форма запросов

O-7 — нет проверки формы links и neighbor_keys в POST /agent/state; вне спеки — финальный ревьюер

Файлы и SDLC_HUB_HOME (Task 4)

T4.4 — предупреждение о нечитаемом файле пишется на каждый claim, без троттлинга — ревью Task 4
O-8 — files.put поверх каталога на месте файла — 500; вне спеки — финальный ревьюер
M-4 — тест SDLC_HUB_HOME="" проверяет только paths.home(), не db_path/backups_dir — финальное ревью

Публикация агента (Task 5)

T5.1 — нет теста на битый manifest в _recover — ревью Task 5
T5.3 — часть регрессионных тестов была зелёной и без правки — ревью Task 5
T5.4 — .tmp от оборванного _write_atomic не убирается — ревью Task 5
T5.5 — предупреждение _recover на каждую публикацию — ревью Task 5

Тесты

O-4 — флейк test_mcp::test_unbound_session повторился один раз (раньше вёлся в #116, закрыта) — наблюдение контроллера

🤖 Generated with Claude Code

Отложенные находки пакета C бэкенда хаба (#129, PR #130, merge 6f2cf1a, архив #131). Формат: маркер — суть — откуда. **Нормализация параметров (Task 1)** - **T1.1** — `hooks.py:36`: проверка `filename` стала недостижимой после нормализации в validate — ревью Task 1 - **T1.2** — нет теста retry команды с необрезанными params — ревью Task 1 - **T1.3** — MCP `batch_create`: нет теста обрезки key — ревью Task 1 - **T1.4** — дубль проверки обязательных параметров в `validate` (до и после нормализации) — ревью Task 1 - **M-1** — нет теста подстановки реального ключа вместо временного ключа с пробелами — финальное ревью - **O-9** — тема `ensure_fields` с пробелами по краям не подтверждается по зеркалу (цена принята в R-2) — финальный ревьюер **ASCII-цифры (Task 2)** - **T2.2** — ссылка `#<id>` на вложение в MCP без теста — ревью Task 2 - **T2.3** — разбор дат (history/mcp) через `is_digits` без теста отката — ревью Task 2 - **O-6** — маршруты вики с `int_group`/`\d` принимают цифры Unicode (`"١٢"` → id 12); вне спеки — финальный ревьюер **Форма запросов** - **O-7** — нет проверки формы `links` и `neighbor_keys` в `POST /agent/state`; вне спеки — финальный ревьюер **Файлы и SDLC_HUB_HOME (Task 4)** - **T4.4** — предупреждение о нечитаемом файле пишется на каждый claim, без троттлинга — ревью Task 4 - **O-8** — `files.put` поверх каталога на месте файла — 500; вне спеки — финальный ревьюер - **M-4** — тест `SDLC_HUB_HOME=""` проверяет только `paths.home()`, не `db_path`/`backups_dir` — финальное ревью **Публикация агента (Task 5)** - **T5.1** — нет теста на битый manifest в `_recover` — ревью Task 5 - **T5.3** — часть регрессионных тестов была зелёной и без правки — ревью Task 5 - **T5.4** — `.tmp` от оборванного `_write_atomic` не убирается — ревью Task 5 - **T5.5** — предупреждение `_recover` на каждую публикацию — ревью Task 5 **Тесты** - **O-4** — флейк `test_mcp::test_unbound_session` повторился один раз (раньше вёлся в #116, закрыта) — наблюдение контроллера 🤖 Generated with [Claude Code](https://claude.com/claude-code)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: aep/sdlc-hub#132