Follow-ups from #97 #102

Open
opened 2026-10-24 14:59:36 +03:00 by aep · 2 comments
Owner

Отложенные находки переименования (#97, PR #99). Подробности — архив openspec/changes/archive/2026-10-24-sdlc-rename/design.md.

Код и тесты

  • T2.1 — help sdlc-hub mcp в core/cli.py перечисляет модули «(jira, wiki)» вручную (ревью Task 2).
  • T2.2 — пробел в конце строки help в core/cli.py (ревью Task 2).
  • T2.4 — нет теста кода 2 для старой записи .mcp.json (… mcp --project X без модуля) (ревью Task 2).
  • T2.5 — список модулей строится дважды в core/mcp.py run() (ревью Task 2).
  • T3.1 — test_both_homes_warn не проверяет код возврата (ревью Task 3).
  • T3.2 — SDLC_HUB_HOME="" трактуется как незаданная (ревью Task 3).
  • T4.2 — лишние фигурные скобки в readStored (web/src/auth/token.ts) (ревью Task 4).
  • T4.3 — нет отдельных тестов переноса ключей для legendPref/commentsPref (ревью Task 4).
  • O-1 — codegraph init после переезда пакета дал пустой индекс у контроллера (наблюдение).

Скрипт переключения (scripts/cutover.py; переключение выполнено 24.10.2026 — актуально только при повторном использовании скрипта)

  • T5.7 — существующая копия *.pre-sdlc перезаписывается (недостижимо из-за preflight) (ревью Task 5).
  • T5.9 — «остановлен» не различает «не был загружен» (ревью Task 5).
  • M-4 — порт ожидания из CUTOVER_PORT/8790, а не из настроек хаба; «ответ» = открытый TCP (финальное ревью).
  • M-5 — при сбое шага 6/7 печатается полный откат (финальное ревью).
  • W-1 — derive_remote не понимает scp-адрес git@host:aep/… (re-review).
  • W-2 — сухой прогон не печатает вычисленный новый origin (re-review).
  • W-3 — текст отката предлагает mv plist.pre-sdlc, если шаг 2 упал до переименования (re-review).
  • W-4 — совет «запускать вне checkout» только в финальной сводке (re-review).
  • W-5 — потребители ищутся только по .mcp.json (re-review).

Документация

  • T6.1 — откат п. 3 README оставляет detached HEAD (ревью Task 6).
  • T6.3 — нумерация вложенных пунктов README может рендериться иначе (ревью Task 6).

T6.2 (перенаправление Gitea по SSH) подтвердилось при переключении и исправлено в eb86563.

Отложенные находки переименования (#97, PR #99). Подробности — архив `openspec/changes/archive/2026-10-24-sdlc-rename/design.md`. **Код и тесты** - T2.1 — help `sdlc-hub mcp` в `core/cli.py` перечисляет модули «(jira, wiki)» вручную (ревью Task 2). - T2.2 — пробел в конце строки help в `core/cli.py` (ревью Task 2). - T2.4 — нет теста кода 2 для старой записи `.mcp.json` (`… mcp --project X` без модуля) (ревью Task 2). - T2.5 — список модулей строится дважды в `core/mcp.py run()` (ревью Task 2). - T3.1 — `test_both_homes_warn` не проверяет код возврата (ревью Task 3). - T3.2 — `SDLC_HUB_HOME=""` трактуется как незаданная (ревью Task 3). - T4.2 — лишние фигурные скобки в `readStored` (`web/src/auth/token.ts`) (ревью Task 4). - T4.3 — нет отдельных тестов переноса ключей для `legendPref`/`commentsPref` (ревью Task 4). - O-1 — `codegraph init` после переезда пакета дал пустой индекс у контроллера (наблюдение). **Скрипт переключения** (`scripts/cutover.py`; переключение выполнено 24.10.2026 — актуально только при повторном использовании скрипта) - T5.7 — существующая копия `*.pre-sdlc` перезаписывается (недостижимо из-за preflight) (ревью Task 5). - T5.9 — «остановлен» не различает «не был загружен» (ревью Task 5). - M-4 — порт ожидания из `CUTOVER_PORT`/8790, а не из настроек хаба; «ответ» = открытый TCP (финальное ревью). - M-5 — при сбое шага 6/7 печатается полный откат (финальное ревью). - W-1 — `derive_remote` не понимает scp-адрес `git@host:aep/…` (re-review). - W-2 — сухой прогон не печатает вычисленный новый `origin` (re-review). - W-3 — текст отката предлагает `mv plist.pre-sdlc`, если шаг 2 упал до переименования (re-review). - W-4 — совет «запускать вне checkout» только в финальной сводке (re-review). - W-5 — потребители ищутся только по `.mcp.json` (re-review). **Документация** - T6.1 — откат п. 3 README оставляет detached HEAD (ревью Task 6). - T6.3 — нумерация вложенных пунктов README может рендериться иначе (ревью Task 6). T6.2 (перенаправление Gitea по SSH) подтвердилось при переключении и исправлено в eb86563.
Author
Owner

Сверка 25.10.2026: T2.4 исправлен (tests/test_cli.py:70-78). Блоки «Скрипт переключения» и «Документация» (T5.7, T5.9, M-4, M-5, W-1…W-5, T6.1, T6.3) — rejected: переключение выполнено 24.10.2026, повторного запуска не будет. O-1 — наблюдение.

Остаются: T3.2 (SDLC_HUB_HOME="" даёт Path(""), core/paths.py:10), T2.1, T2.2, T2.5, T3.1, T4.2, T4.3.

Сверка 25.10.2026: T2.4 исправлен (`tests/test_cli.py:70-78`). Блоки «Скрипт переключения» и «Документация» (T5.7, T5.9, M-4, M-5, W-1…W-5, T6.1, T6.3) — rejected: переключение выполнено 24.10.2026, повторного запуска не будет. O-1 — наблюдение. Остаются: T3.2 (`SDLC_HUB_HOME=""` даёт `Path("")`, `core/paths.py:10`), T2.1, T2.2, T2.5, T3.1, T4.2, T4.3.
Author
Owner

PR #130 (merge 6f2cf1a), архив #131: закрыт T3.2 — 75b1559 (пустой SDLC_HUB_HOME равен незаданному: данные в ~/.local/share/sdlc-hub). Остаются T2.1, T2.2, T2.5, T3.1, T4.2, T4.3 — issue остаётся открытой.

PR #130 (merge 6f2cf1a), архив #131: закрыт **T3.2** — 75b1559 (пустой SDLC_HUB_HOME равен незаданному: данные в ~/.local/share/sdlc-hub). Остаются T2.1, T2.2, T2.5, T3.1, T4.2, T4.3 — issue остаётся открытой.
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aep/sdlc-hub#102
No description provided.