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

Follow-ups from #22 #25

New Issue
Closed
opened 2026-10-14 21:59:36 +03:00 by aep · 1 comment
aep commented 2026-10-14 21:59:36 +03:00
Owner
Copy Link

Отложенные находки финального ревью (opus) PR #23 (самообновление агента, хаб):

  • M-1 — OSError при записи jira-hub agent publish (диск полон, нет прав): трейсбек вместо сообщения; agent.py новый при старом манифесте → GET /agent/release 404 до следующей публикации; остаются .tmp. (release.py, cli.py)
  • M-2 — после перезапуска хаба до первого claim подсказка «агент запущен без run.py» и «без запускателя» ложные (launcher_version только в памяти). (release.hint, mcp_server)
  • M-3 — предупреждение о битом манифесте пишется на каждый claim и GET /agent (~3000 строк/сутки). (release.current)
  • M-4 — конфликт с #21 (web-app) в сигнатуре Hub.__init__ (web_dir/release_dir) при слиянии; CLAUDE.md «текущий change». (api.py)
  • M-5 — load: не-UTF-8 файл с совпадающим sha → 500 вместо 404. (release.py)
  • M-6 — release._write_atomic без fsync (в отличие от relay.write_atomic). (release.py)
  • M-7 — version манифеста приводится int() без проверки типа (6.9, true). (release.py)
Отложенные находки финального ревью (opus) PR #23 (самообновление агента, хаб): - **M-1** — OSError при записи `jira-hub agent publish` (диск полон, нет прав): трейсбек вместо сообщения; agent.py новый при старом манифесте → `GET /agent/release` 404 до следующей публикации; остаются `.tmp`. (release.py, cli.py) - **M-2** — после перезапуска хаба до первого claim подсказка «агент запущен без run.py» и «без запускателя» ложные (launcher_version только в памяти). (release.hint, mcp_server) - **M-3** — предупреждение о битом манифесте пишется на каждый claim и GET /agent (~3000 строк/сутки). (release.current) - **M-4** — конфликт с #21 (web-app) в сигнатуре `Hub.__init__` (`web_dir`/`release_dir`) при слиянии; CLAUDE.md «текущий change». (api.py) - **M-5** — `load`: не-UTF-8 файл с совпадающим sha → 500 вместо 404. (release.py) - **M-6** — `release._write_atomic` без fsync (в отличие от relay.write_atomic). (release.py) - **M-7** — `version` манифеста приводится `int()` без проверки типа (6.9, true). (release.py)
aep commented 2026-10-15 10:05:18 +03:00
Author
Owner
Copy Link

Закрыто PR #36 (merge cbc86be), архив #38: M-1, M-3, M-5, M-6, M-7 — f977d9b/5793438; M-2 — 8a319ea/7d3f648. M-4 исправлено ранее слиянием web-app (#27). Остаток по публикации (T2.1, T2.2, T2.4, M-6 нового ревью) — в follow-up из #32.

Закрыто PR #36 (merge cbc86be), архив #38: M-1, M-3, M-5, M-6, M-7 — f977d9b/5793438; M-2 — 8a319ea/7d3f648. M-4 исправлено ранее слиянием web-app (#27). Остаток по публикации (T2.1, T2.2, T2.4, M-6 нового ревью) — в follow-up из #32.
aep closed this issue 2026-10-15 10:05:19 +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#25
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?