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

Follow-ups from #83 #86

New Issue
Open
opened 2026-10-22 15:07:20 +03:00 by aep · 3 comments
aep commented 2026-10-22 15:07:20 +03:00
Owner
Copy Link

Отложенные находки change attachment-fetch (PR #84):

  • T2.1 — тай-брейк id и равные created_at в выборе «самого нового» вложения (тесты) — ревью Task 2
  • T2.2 — N+1 запросов command в get_issue (состояние каждого вложения) — ревью Task 2
  • T2.4 — нет тестов: fetching поверх старого failed, ready поверх нового failed — ревью Task 2
  • T3.5 — import threading и FSHA посреди test_queue.py, длинная строка has_writes — ревью Task 3
  • T4.4 — нет теста «ready, но файл исчез из files/» — ревью Task 4
  • T5.1 — в тестах MCP размеры 239821 вместо 240000 из спеки — ревью Task 5
  • T5.4 — нет тестов обрезки wait_seconds и повторного вызова attachment — ревью Task 5
  • T5.5 — нет теста пути опроса MCP до ready — ревью Task 5
  • T6.2 — после успешного повтора мигает устаревшая ошибка failed — ревью Task 6
  • T6.3 — AuthError/HubDownError при опросе скачивания превращаются в текст вместо экрана входа — ревью Task 6
  • T6.6 — двойной import из одного модуля — ревью Task 6
  • T6.8 — нет тестов бейджа «у хаба» после invalidate и порядка секций на узком экране — ревью Task 6
  • M-4 — веб пересортирует вложения и выбирает дубли имён по Date.parse, а не порядком хаба — финальное ревью
  • F.1 — isdigit() принимает Unicode-цифры в attachment_id — повторное ревью
Отложенные находки change attachment-fetch (PR #84): - T2.1 — тай-брейк id и равные created_at в выборе «самого нового» вложения (тесты) — ревью Task 2 - T2.2 — N+1 запросов command в get_issue (состояние каждого вложения) — ревью Task 2 - T2.4 — нет тестов: fetching поверх старого failed, ready поверх нового failed — ревью Task 2 - T3.5 — import threading и FSHA посреди test_queue.py, длинная строка has_writes — ревью Task 3 - T4.4 — нет теста «ready, но файл исчез из files/» — ревью Task 4 - T5.1 — в тестах MCP размеры 239821 вместо 240000 из спеки — ревью Task 5 - T5.4 — нет тестов обрезки wait_seconds и повторного вызова attachment — ревью Task 5 - T5.5 — нет теста пути опроса MCP до ready — ревью Task 5 - T6.2 — после успешного повтора мигает устаревшая ошибка failed — ревью Task 6 - T6.3 — AuthError/HubDownError при опросе скачивания превращаются в текст вместо экрана входа — ревью Task 6 - T6.6 — двойной import из одного модуля — ревью Task 6 - T6.8 — нет тестов бейджа «у хаба» после invalidate и порядка секций на узком экране — ревью Task 6 - M-4 — веб пересортирует вложения и выбирает дубли имён по Date.parse, а не порядком хаба — финальное ревью - F.1 — isdigit() принимает Unicode-цифры в attachment_id — повторное ревью
aep commented 2026-10-25 21:41:53 +03:00
Author
Owner
Copy Link

Сверка 25.10.2026: исправлены T4.4 (на уровне state, tests/test_attachments.py:75-82) и T5.5 (tests/test_mcp.py:800-806). Остальные маркеры актуальны.

Сверка 25.10.2026: исправлены T4.4 (на уровне state, `tests/test_attachments.py:75-82`) и T5.5 (`tests/test_mcp.py:800-806`). Остальные маркеры актуальны.
aep commented 2026-10-26 00:56:37 +03:00
Author
Owner
Copy Link

PR #125 (merge 9b98420), архив #126: закрыты T6.2, T6.3, M-4 — 43c1f83 (нет прежней ошибки после повтора, 401 при опросе — экран входа, порядок вложений и выбор при одинаковых именах — как у хаба). F.1 и остальные маркеры открыты — issue остаётся открытой.

PR #125 (merge 9b98420), архив #126: закрыты T6.2, T6.3, M-4 — 43c1f83 (нет прежней ошибки после повтора, 401 при опросе — экран входа, порядок вложений и выбор при одинаковых именах — как у хаба). F.1 и остальные маркеры открыты — issue остаётся открытой.
aep commented 2026-10-26 09:21:31 +03:00
Author
Owner
Copy Link

PR #130 (merge 6f2cf1a), архив #131: закрыт F.1 — 4a6855d (цифровые id и версия агента — только ASCII 0-9 через core/http.is_digits, «²» больше не даёт 500). Остальные маркеры открыты — issue остаётся открытой.

PR #130 (merge 6f2cf1a), архив #131: закрыт **F.1** — 4a6855d (цифровые id и версия агента — только ASCII 0-9 через core/http.is_digits, «²» больше не даёт 500). Остальные маркеры открыты — 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#86
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?