Follow-ups from #23 #26

New Issue

aep · 2026-10-14T23:52:41+03:00

aep commented

2026-10-14 23:52:41 +03:00

Отложенные находки агента v9 / run.py v2 (PR #24, change agent-v9-hardening). Формат: маркер — суть — откуда.

T1.1 — рассогласованный пробел перед комментарием в test_download_failure_is_transient (tests/test_update.py) — ревью Task 1
T3.1 — не переотступлены продолжения словаря в self._hub(self.hub.state, {...}) (src/agent.py) — ревью Task 3
T4.1 — threading.Timer в tests/test_runner.py не отменяется в cleanup (addCleanup(t.cancel)) — ревью Task 4
T4.2 — разделить кейсы «давность» и «чужой проект» в test_ensure_issue_old_or_foreign_is_not_duplicate — ревью Task 4
T4.3 — claim_wait в fake_hub смотрит только queue, не in_flight — ревью Task 4
T4.4 — fake_jira молча игнорирует created >= -Nh/-Nw; лучше падать громко — ревью Task 4 (M-9 финального ревью — то же)
T5.1 / M-1 / O-3 — OSError при os.replace(prev, agent) в Launcher.rollback и reconcile (run.py) не перехвачен — ревью Task 5, финальное ревью
T5.2 / M-2 — reconcile не проверяет результат write_state(..., "trial") в ветке «подмена уже сделана» (сходится на следующем старте) — ревью Task 5, финальное ревью
M-5 / O-2 — не-ASCII токен хаба в agent.json роняет агента (UnicodeEncodeError в заголовке Authorization); проверять в check/при загрузке конфига — финальное ревью (было до ветки)
M-6 — rejected при незаписанном trial без настоящего текста ошибки (write_state теряет OSError) — финальное ревью
M-10 — trial_sha считается дважды (wait_child и run()) — скоуп-ревью фикса I-1

Поправки run.py требуют ручной установки на виртуалке — копить до следующей причины переставлять запускатель.

Отложенные находки агента v9 / run.py v2 (PR #24, change agent-v9-hardening). Формат: маркер — суть — откуда. - **T1.1** — рассогласованный пробел перед комментарием в `test_download_failure_is_transient` (tests/test_update.py) — ревью Task 1 - **T3.1** — не переотступлены продолжения словаря в `self._hub(self.hub.state, {...})` (src/agent.py) — ревью Task 3 - **T4.1** — `threading.Timer` в tests/test_runner.py не отменяется в cleanup (`addCleanup(t.cancel)`) — ревью Task 4 - **T4.2** — разделить кейсы «давность» и «чужой проект» в `test_ensure_issue_old_or_foreign_is_not_duplicate` — ревью Task 4 - **T4.3** — `claim_wait` в fake_hub смотрит только `queue`, не in_flight — ревью Task 4 - **T4.4** — fake_jira молча игнорирует `created >= -Nh/-Nw`; лучше падать громко — ревью Task 4 (M-9 финального ревью — то же) - **T5.1 / M-1 / O-3** — `OSError` при `os.replace(prev, agent)` в `Launcher.rollback` и `reconcile` (run.py) не перехвачен — ревью Task 5, финальное ревью - **T5.2 / M-2** — `reconcile` не проверяет результат `write_state(..., "trial")` в ветке «подмена уже сделана» (сходится на следующем старте) — ревью Task 5, финальное ревью - **M-5 / O-2** — не-ASCII токен хаба в agent.json роняет агента (`UnicodeEncodeError` в заголовке Authorization); проверять в `check`/при загрузке конфига — финальное ревью (было до ветки) - **M-6** — `rejected` при незаписанном `trial` без настоящего текста ошибки (`write_state` теряет `OSError`) — финальное ревью - **M-10** — `trial_sha` считается дважды (`wait_child` и `run()`) — скоуп-ревью фикса I-1 Поправки run.py требуют ручной установки на виртуалке — копить до следующей причины переставлять запускатель.

aep referenced this issue from aep/sdlc-hub

2026-10-15 00:29:08 +03:00

Пакет D хаба: публикация агента, статус, подтверждение CRLF, тесты #32

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: aep/jira-agent#26