aep/sdlc-hub
1
0
Fork 0
Code Issues 24 Pull Requests Actions Packages Projects Releases Wiki Activity

Follow-ups from #37 #42

New Issue
Open
opened 2026-10-15 14:33:47 +03:00 by aep · 2 comments
aep commented 2026-10-15 14:33:47 +03:00
Owner
Copy Link

Отложенные находки change issue-card-modal (#37, PR #40). Источник: ревью задач плана (T*), финальное ревью ветки (M-*).

  • T2.1 — check_comments не проверяет, что элемент issues в POST /agent/state — объект с key (500 вместо 422) (ревью Task 2).
  • T4.3 — ветка «браузер сам закрыл окно» (Esc без отменяемого cancel) без автотеста; вручную проверена в Chromium (ревью Task 4, финальное ревью).
  • T5.4 — логический тип из link_types вне блокеров и «связанных» попадает в «Прочие» под логическим именем (ревью Task 5).
  • M-1 — отложенная запись поиска (300 мс) закрывает только что открытое окно карточки (финальное ревью).
  • M-2 — двойной щелчок по фону или ✕ вызывает navigate(-depth) дважды (финальное ревью).
  • M-3 — фокус не переводится в окно и не возвращается на ссылку после закрытия (финальное ревью).
  • M-4 — у подтверждения «Уйти без сохранения?» нет доступного имени и фокуса (финальное ревью).
  • M-5 — ссылка на задачу вне зеркала открывает 404, «Закрыть» закрывает всю стопку окон (финальное ревью).
  • M-6 — узкий экран: окно на 2 px шире/выше экрана (рамка), 100vh на мобильных (финальное ревью).
  • M-7 — e2e «заметка из окна видна на доске» сравнивает прокрутку 0 с 0 (финальное ревью).
  • M-8 — e2e прямого адреса не проверяет «назад»; догруженные порции списка под окном не проверены (финальное ревью).
  • M-9 — хаб принимает body: null у комментария, спека — «строка» (финальное ревью).
  • M-10 — склеенный комментарий в web/src/issues/IssueModal.test.tsx (финальное ревью).

🤖 Generated with Claude Code

Отложенные находки change `issue-card-modal` (#37, PR #40). Источник: ревью задач плана (T*), финальное ревью ветки (M-*). - **T2.1** — `check_comments` не проверяет, что элемент `issues` в `POST /agent/state` — объект с `key` (500 вместо 422) (ревью Task 2). - **T4.3** — ветка «браузер сам закрыл окно» (Esc без отменяемого `cancel`) без автотеста; вручную проверена в Chromium (ревью Task 4, финальное ревью). - **T5.4** — логический тип из `link_types` вне блокеров и «связанных» попадает в «Прочие» под логическим именем (ревью Task 5). - **M-1** — отложенная запись поиска (300 мс) закрывает только что открытое окно карточки (финальное ревью). - **M-2** — двойной щелчок по фону или ✕ вызывает `navigate(-depth)` дважды (финальное ревью). - **M-3** — фокус не переводится в окно и не возвращается на ссылку после закрытия (финальное ревью). - **M-4** — у подтверждения «Уйти без сохранения?» нет доступного имени и фокуса (финальное ревью). - **M-5** — ссылка на задачу вне зеркала открывает 404, «Закрыть» закрывает всю стопку окон (финальное ревью). - **M-6** — узкий экран: окно на 2 px шире/выше экрана (рамка), `100vh` на мобильных (финальное ревью). - **M-7** — e2e «заметка из окна видна на доске» сравнивает прокрутку 0 с 0 (финальное ревью). - **M-8** — e2e прямого адреса не проверяет «назад»; догруженные порции списка под окном не проверены (финальное ревью). - **M-9** — хаб принимает `body: null` у комментария, спека — «строка» (финальное ревью). - **M-10** — склеенный комментарий в `web/src/issues/IssueModal.test.tsx` (финальное ревью). 🤖 Generated with [Claude Code](https://claude.com/claude-code)
aep commented 2026-10-26 00:56:30 +03:00
Author
Owner
Copy Link

PR #125 (merge 9b98420), архив #126: закрыты M-1, M-2, M-4 — 154cc88 (поиск не закрывает окно, повторное закрытие игнорируется, фокус подтверждения ухода); M-5 — 7b297fe (связь «нет в зеркале» ведёт в Jira, «Назад» у окна 404). Остальные маркеры (в т.ч. M-3, M-6, T2.1) открыты — issue остаётся открытой.

PR #125 (merge 9b98420), архив #126: закрыты M-1, M-2, M-4 — 154cc88 (поиск не закрывает окно, повторное закрытие игнорируется, фокус подтверждения ухода); M-5 — 7b297fe (связь «нет в зеркале» ведёт в Jira, «Назад» у окна 404). Остальные маркеры (в т.ч. M-3, M-6, T2.1) открыты — issue остаётся открытой.
aep commented 2026-10-26 09:21:33 +03:00
Author
Owner
Copy Link

PR #130 (merge 6f2cf1a), архив #131: закрыт T2.1 — f1dae5a, e20977e (POST /agent/state с issues неверной формы — 422 bad_state, а не 500). Остальные маркеры (в т.ч. M-3, M-6) открыты — issue остаётся открытой.

PR #130 (merge 6f2cf1a), архив #131: закрыт **T2.1** — f1dae5a, e20977e (POST /agent/state с issues неверной формы — 422 bad_state, а не 500). Остальные маркеры (в т.ч. M-3, M-6) открыты — issue остаётся открытой.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
aep
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aep/sdlc-hub#42
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?