Follow-ups from #108 #111

Open
opened 2026-10-25 14:26:11 +03:00 by aep · 3 comments
Owner

Отложенное из #108 (PR #109, архив #110). Подробности — openspec/changes/archive/2026-10-25-sprints-backlog/design.md.

Код

  • T-F1 — validate не обрезает значения params после проверки на пустоту: before=" A-2 " уходит агенту с пробелами (финальное ревью).
  • R-33 — до выкатки агента нужной версии ensure_fields {story_points} агент v14 отвечает done без записи; исправление — 422 при постановке, если agent_version модуля jira < 15 (финальное ревью M-3).
  • M-6 — формат SP: веб String(n), MCP :g; дробные суммы (0.1+0.2) расходятся (финальное ревью).
  • T3.4 — issue_sprint читается целиком в GET /agile (ревью Task 3).

Тесты

  • T2.2 — нет теста: спринт переходит между досками в одной пачке (Task 2).
  • T3.1, T3.2 — нет тестов на границы R-30/R-17: board_type NULL, kanban без колонок, активный спринт другой доски исключает из бэклога, пустые/неразбираемые даты (Task 3).
  • T5.1 — нет теста неизменности входных params команд спринта (Task 5).
  • T9.4 — нет тестов загрузки/ошибки AgileGate и kanban-проекта в меню (Task 9).
  • T12.2–T12.4 — e2e: фиксированный jira_updated сида, нестрогие getByText, зависимость от порядка DOM (Task 12).
  • T1.1, T3.3, T3.5, T6.1 — мелочи тестов (хелпер без try/finally, дубль NOW, lambda с noqa, цепочка ==).

Документы и разметка

  • T1.3 — design.md архива: порядок R-28 перед R-13, лишний пробел в R-17.
  • T10.3 — заголовки колонок доски h2 вместо h3 (Task 10).

Наблюдения

  • O-3 — web/src/shell/activity/activity.test.tsx нестабилен в полном vitest под нагрузкой (≥4 раза), отдельно зелёный.
  • O-6 — исполнитель SDD записал правку src/agent.py в основной checkout jira-agent вместо worktree (незакоммиченная копия do_ensure_rank, совпадает с main); исполнитель хаба оставил зависший cat > /tmp/x на 20 ч. В брифах исполнителей — запрет записи вне worktree и команд, ждущих stdin.
  • O-7 — контроллер SDD фоновым сабагентом не получает hand-back своих исполнителей (приходят сессии); обход — Monitor по коммиту/отчёту + пересылка SendMessage (память feedback-sdd-no-sleep-polling).
Отложенное из #108 (PR #109, архив #110). Подробности — openspec/changes/archive/2026-10-25-sprints-backlog/design.md. **Код** - **T-F1** — validate не обрезает значения params после проверки на пустоту: `before=" A-2 "` уходит агенту с пробелами (финальное ревью). - **R-33** — до выкатки агента нужной версии `ensure_fields {story_points}` агент v14 отвечает `done` без записи; исправление — 422 при постановке, если `agent_version` модуля jira < 15 (финальное ревью M-3). - **M-6** — формат SP: веб `String(n)`, MCP `:g`; дробные суммы (0.1+0.2) расходятся (финальное ревью). - **T3.4** — `issue_sprint` читается целиком в `GET /agile` (ревью Task 3). **Тесты** - **T2.2** — нет теста: спринт переходит между досками в одной пачке (Task 2). - **T3.1, T3.2** — нет тестов на границы R-30/R-17: `board_type` NULL, kanban без колонок, активный спринт другой доски исключает из бэклога, пустые/неразбираемые даты (Task 3). - **T5.1** — нет теста неизменности входных params команд спринта (Task 5). - **T9.4** — нет тестов загрузки/ошибки AgileGate и kanban-проекта в меню (Task 9). - **T12.2–T12.4** — e2e: фиксированный `jira_updated` сида, нестрогие `getByText`, зависимость от порядка DOM (Task 12). - **T1.1, T3.3, T3.5, T6.1** — мелочи тестов (хелпер без try/finally, дубль `NOW`, lambda с noqa, цепочка `==`). **Документы и разметка** - **T1.3** — design.md архива: порядок R-28 перед R-13, лишний пробел в R-17. - **T10.3** — заголовки колонок доски h2 вместо h3 (Task 10). **Наблюдения** - **O-3** — `web/src/shell/activity/activity.test.tsx` нестабилен в полном vitest под нагрузкой (≥4 раза), отдельно зелёный. - **O-6** — исполнитель SDD записал правку `src/agent.py` в основной checkout jira-agent вместо worktree (незакоммиченная копия `do_ensure_rank`, совпадает с main); исполнитель хаба оставил зависший `cat > /tmp/x` на 20 ч. В брифах исполнителей — запрет записи вне worktree и команд, ждущих stdin. - **O-7** — контроллер SDD фоновым сабагентом не получает hand-back своих исполнителей (приходят сессии); обход — Monitor по коммиту/отчёту + пересылка SendMessage (память feedback-sdd-no-sleep-polling).
Author
Owner

PR #114 (merge e84f7b1) закрыл веб-маркеры: M-6 (веб-сторона: fmtSp — до двух знаков без хвостовых нулей; MCP :g не менялся), T10.3 (заголовки колонок доски — h3), T9.4 (тесты загрузки/ошибки AgileGate и меню kanban-проекта), T12.2–T12.4 (e2e доски: фиксированный jira_updated, строгие локаторы, без зависимости от порядка DOM). Остальные пункты issue открыты.

PR #114 (merge e84f7b1) закрыл веб-маркеры: **M-6** (веб-сторона: fmtSp — до двух знаков без хвостовых нулей; MCP `:g` не менялся), **T10.3** (заголовки колонок доски — h3), **T9.4** (тесты загрузки/ошибки AgileGate и меню kanban-проекта), **T12.2–T12.4** (e2e доски: фиксированный jira_updated, строгие локаторы, без зависимости от порядка DOM). Остальные пункты issue открыты.
Author
Owner

Сверка 25.10.2026: исправлены M-6 (веб, PR #114), T9.4 и T12.2–T12.4 (f21dea3), T10.3 (2cd69bd). R-33 — rejected: агент v16, min_agent_version 15, публикация только вверх. O-3 ведётся в #116; O-6, O-7 — процесс.

Остаются: T-F1 (воспроизведено: before=" A-2 " уходит агенту с пробелами, modules/jira/commands.py:145-146), T3.4, T3.1/T3.2 (частично), T5.1, T2.2, T1.1/T3.3/T3.5/T6.1, T1.3.

Сверка 25.10.2026: исправлены M-6 (веб, PR #114), T9.4 и T12.2–T12.4 (f21dea3), T10.3 (2cd69bd). R-33 — rejected: агент v16, min_agent_version 15, публикация только вверх. O-3 ведётся в #116; O-6, O-7 — процесс. Остаются: T-F1 (воспроизведено: `before=" A-2 "` уходит агенту с пробелами, `modules/jira/commands.py:145-146`), T3.4, T3.1/T3.2 (частично), T5.1, T2.2, T1.1/T3.3/T3.5/T6.1, T1.3.
Author
Owner

PR #130 (merge 6f2cf1a), архив #131: закрыт T-F1 — abbcf0c (строковые параметры, кроме summary/description/body/goal, обрезаются при постановке до всех проверок; ensure_fields из одних пробелов — 422, e20977e). Остальные маркеры (T3.4, T3.1/T3.2, T5.1, T2.2, T1.1/T3.3/T3.5/T6.1, T1.3) открыты — issue остаётся открытой.

PR #130 (merge 6f2cf1a), архив #131: закрыт **T-F1** — abbcf0c (строковые параметры, кроме summary/description/body/goal, обрезаются при постановке до всех проверок; ensure_fields из одних пробелов — 422, e20977e). Остальные маркеры (T3.4, T3.1/T3.2, T5.1, T2.2, T1.1/T3.3/T3.5/T6.1, T1.3) открыты — issue остаётся открытой.
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aep/sdlc-hub#111
No description provided.