Follow-ups from #2 #5

New Issue

aep · 2026-10-12T14:03:17+03:00

aep commented

2026-10-12 14:03:17 +03:00

Отложенные находки из #2, часть 1 (PR #4, merge 2c599a1). Маркеры из отчёта SDD; источник указан в скобках: Tn.k — ревью задачи n плана, M-n — финальное ревью ветки (opus).

Для части 2 (awp-ca) — планировать вместе с ней, не здесь:

T4.3 / M-4 — «Читать на старте» в dev.md: дубль brief:/tea issues ls, строки не подходят для cwd workspace, OQ у analyst не команда (ревью Task 4 + финальное ревью)
M-6 — правило codegraph «на период замера» и /harness-retro лежат в общем слое dev.md (финальное ревью)
T10.2 — notes/agent-memory.md упомянут, шаблона в role-repo нет (ревью Task 10)

Обвязка:

T1.1 — причина «симлинк → read-only» подтверждена одним контролем; 6.5 в части 2 как предосторожность (ревью Task 1, spike)
T2.1 — нет явного теста cc-qa … для role-hint (ревью Task 2)
T3.1 — tests/test-roles.sh проверяет cd голой подстрокой (ревью Task 3)
T4.1 — в таблице адресов нет пометки про убранное «кроме архивации OpenSpec» у adopt-project.sh (ревью Task 4)
T4.2 — в строке «Новое» нет require-plan/.bounded и «если хук мешает — сказать владельцу» (ревью Task 4)
T5.2 — шаг 2 skill /role зависит от пути auto-memory в системном промпте (ревью Task 5)
T6.2 — link-roles --map не проверяет роль admin|analyst|qa (ревью Task 6)
T6.3 — link-roles молча заменяет чужой/битый симлинк, в шапке не сказано (ревью Task 6)
T6.4 — ячейки каталога ../x и абсолютные пути не отклоняются (ревью Task 6)
T7.1 — $NAME в замене sed ломается на / и & (ревью Task 7)
T7.2 — new-project.sh --role без значения — unbound $2 (ревью Task 7)
T8.2 — tests/test-doctor.sh: проверка workspace слишком слабая (ревью Task 8)
T8.3 — не покрыта ветка WARN «нет проектного файла роли» в doctor (ревью Task 8)
T9.1 — фильтр ours_ss в install.sh удаляет всю запись SessionStart, если в ней есть наш хук; поведение было до ветки (ревью Task 9)

Без маркера (re-review финального ревью, маркер в отчёте не присвоен):

спека, раздел 9: «Проверяется первой» про симлинк устарело после spike
HARNESS.md: кавычки в «dev, admin, analyst или qa»

Отложенные находки из #2, часть 1 (PR #4, merge 2c599a1). Маркеры из отчёта SDD; источник указан в скобках: Tn.k — ревью задачи n плана, M-n — финальное ревью ветки (opus). Для части 2 (awp-ca) — планировать вместе с ней, не здесь: - T4.3 / M-4 — «Читать на старте» в dev.md: дубль `brief:`/`tea issues ls`, строки не подходят для cwd workspace, OQ у analyst не команда (ревью Task 4 + финальное ревью) - M-6 — правило codegraph «на период замера» и `/harness-retro` лежат в общем слое dev.md (финальное ревью) - T10.2 — `notes/agent-memory.md` упомянут, шаблона в role-repo нет (ревью Task 10) Обвязка: - T1.1 — причина «симлинк → read-only» подтверждена одним контролем; 6.5 в части 2 как предосторожность (ревью Task 1, spike) - T2.1 — нет явного теста `cc-qa …` для role-hint (ревью Task 2) - T3.1 — `tests/test-roles.sh` проверяет `cd` голой подстрокой (ревью Task 3) - T4.1 — в таблице адресов нет пометки про убранное «кроме архивации OpenSpec» у adopt-project.sh (ревью Task 4) - T4.2 — в строке «Новое» нет require-plan/.bounded и «если хук мешает — сказать владельцу» (ревью Task 4) - T5.2 — шаг 2 skill /role зависит от пути auto-memory в системном промпте (ревью Task 5) - T6.2 — link-roles `--map` не проверяет роль admin|analyst|qa (ревью Task 6) - T6.3 — link-roles молча заменяет чужой/битый симлинк, в шапке не сказано (ревью Task 6) - T6.4 — ячейки каталога `../x` и абсолютные пути не отклоняются (ревью Task 6) - T7.1 — `$NAME` в замене `sed` ломается на `/` и `&` (ревью Task 7) - T7.2 — `new-project.sh --role` без значения — unbound `$2` (ревью Task 7) - T8.2 — `tests/test-doctor.sh`: проверка `workspace` слишком слабая (ревью Task 8) - T8.3 — не покрыта ветка WARN «нет проектного файла роли» в doctor (ревью Task 8) - T9.1 — фильтр `ours_ss` в install.sh удаляет всю запись SessionStart, если в ней есть наш хук; поведение было до ветки (ревью Task 9) Без маркера (re-review финального ревью, маркер в отчёте не присвоен): - спека, раздел 9: «Проверяется первой» про симлинк устарело после spike - HARNESS.md: кавычки в «dev, admin, analyst или qa»

aep referenced this issue

2026-10-24 21:15:13 +03:00

Сводка отложенных находок обвязки (бывш. #5, #13, #21, #28, #31) #70

aep commented

2026-10-24 21:15:31 +03:00

Сведено в #70 (консолидация follow-ups обвязки, 24.10.2026). Все маркеры и статусы перенесены туда.

aep closed this issue

2026-10-24 21:15:32 +03:00

Sign in to join this conversation.