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

Follow-ups from #13 #16

New Issue
Closed
opened 2026-10-14 12:24:20 +03:00 by aep · 2 comments
aep commented 2026-10-14 12:24:20 +03:00
Owner
Copy Link

Отложенные находки пакета C bridge-hygiene (#13, PR #14). Формат: маркер — суть — откуда.

  • T3.1 — тяжёлая строка ok, want = ... and ..., "..." в commands._type_error — ревью Task 3
  • T3.2 — нет теста на bool в строковом параметре и на пустой список — ревью Task 3
  • T5.2 — тест предела выдач не проверяет, что sync_run.finished_at не ставится для не-refresh — ревью Task 5
  • M-8 — нет тестов: 501 у HTTP API на прочие методы, epics.csv без ключа — финальное ревью
  • ре-ревью minor-1 — relay._cleaned сравнивает пути по написанию: другая запись той же папки чистится ещё раз — ре-ревью fix-волны
  • ре-ревью minor-2 — secure_dir в Hub.__init__ и в backup может дважды писать предупреждение — ре-ревью fix-волны
  • ре-ревью minor-3 — в test_vanished_tree_keeps_work нет положительной проверки «подготовлена заново» — ре-ревью fix-волны
  • O-9 — commands.check_confirmed сравнивает description без нормализации \r\n: потерянный отчёт по ensure_fields на тексте с CRLF закроется довыдачей, а не подтверждением по состоянию — финальное ревью агента (#15)

Агентская часть — aep/jira-agent «Follow-ups from #15».

Отложенные находки пакета C bridge-hygiene (#13, PR #14). Формат: маркер — суть — откуда. - **T3.1** — тяжёлая строка `ok, want = ... and ..., "..."` в `commands._type_error` — ревью Task 3 - **T3.2** — нет теста на bool в строковом параметре и на пустой список — ревью Task 3 - **T5.2** — тест предела выдач не проверяет, что `sync_run.finished_at` не ставится для не-refresh — ревью Task 5 - **M-8** — нет тестов: 501 у HTTP API на прочие методы, `epics.csv` без ключа — финальное ревью - **ре-ревью minor-1** — `relay._cleaned` сравнивает пути по написанию: другая запись той же папки чистится ещё раз — ре-ревью fix-волны - **ре-ревью minor-2** — `secure_dir` в `Hub.__init__` и в `backup` может дважды писать предупреждение — ре-ревью fix-волны - **ре-ревью minor-3** — в `test_vanished_tree_keeps_work` нет положительной проверки «подготовлена заново» — ре-ревью fix-волны - **O-9** — `commands.check_confirmed` сравнивает `description` без нормализации `\r\n`: потерянный отчёт по `ensure_fields` на тексте с CRLF закроется довыдачей, а не подтверждением по состоянию — финальное ревью агента (#15) Агентская часть — aep/jira-agent «Follow-ups from #15».
aep commented 2026-10-14 14:45:15 +03:00
Author
Owner
Copy Link

Найдено при живой проверке 14.10.2026 (наблюдения):

  • O-10 — у зависимой команды, пропущенной из-за force-отменённой ensure_issue, error = «задача new:901 не создана», хотя задача создана (ключ в result отменённой команды); точнее — «создание задачи new:N отменено» — queue._next
  • O-11 — ответ PUT /settings не содержит relay_dir (настройка записывается, но в ответе её не видно — путает при ручной проверке) — api/settings
Найдено при живой проверке 14.10.2026 (наблюдения): - **O-10** — у зависимой команды, пропущенной из-за force-отменённой ensure_issue, error = «задача new:901 не создана», хотя задача создана (ключ в result отменённой команды); точнее — «создание задачи new:N отменено» — queue._next - **O-11** — ответ PUT /settings не содержит relay_dir (настройка записывается, но в ответе её не видно — путает при ручной проверке) — api/settings
aep commented 2026-10-15 10:05:19 +03:00
Author
Owner
Copy Link

Закрыто PR #36 (merge cbc86be), архив #38: O-9, O-10, O-11, minor-1, T3.1 — e180bb4; T3.2, T5.2, M-8, minor-3 — 1f1dad6. minor-2 — rejected (решение владельца 15.10.2026: дубль предупреждения при неудачном chmod не стоит правки).

Закрыто PR #36 (merge cbc86be), архив #38: O-9, O-10, O-11, minor-1, T3.1 — e180bb4; T3.2, T5.2, M-8, minor-3 — 1f1dad6. minor-2 — rejected (решение владельца 15.10.2026: дубль предупреждения при неудачном chmod не стоит правки).
aep closed this issue 2026-10-15 10:05:20 +03:00
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#16
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?