Follow-ups from #42 #62

New Issue

aep · 2026-10-24T19:01:23+03:00

aep commented

2026-10-24 19:01:23 +03:00

Отложенные находки из #42 (DSH: OpenSpec integration). Источник — ledger ветки feat/42-dsh-openspec (удалён вместе с worktree; находки продублированы в PR #60 и комментарии к #42).

Чеклист

T2.1 / M-2 — негативный тест --check без openspec (dsh/tests/openspec.test.mjs, «warns but does not fail without the openspec CLI») подставляет PATH = bin:/usr/bin:/bin:/usr/sbin:/sbin и рассчитывает, что CLI там нет. На macOS верно (openspec в /opt/homebrew/bin), на Linux с npm -g (префикс /usr) CLI окажется в /usr/bin — тест найдёт его и упадёт. Предложение: вырезать openspec из PATH явно (временный каталог с симлинками на всё, кроме openspec) или проверять наличие CLI в тесте и скипать. Источник: ревью Task 2 (review-2-out.md), подтверждено финальным ревью.
T3.4 — процессная: implementer Task 3 написал реализацию раньше тестов (TDD-порядок нарушен). Тесты не тавтологичны, поведение проверено. Источник: review-3-out.md. Решение: усилить формулировку в брифах SDD или в sdd skill: шаг «тест падает» обязателен до кода.
O-2 — в окружении DSH-сессии экспортирован DSH_HOME=/Users/andrey.epifancev/.dsh; команда HOME=$(mktemp -d) bash dsh/install.sh --skills из брифа не изолирует и целится в реальный ~/.dsh (спасла песочница, каталог не изменён). Предложение: в брифах и рецептах всегда изолировать обе переменные (HOME=$tmp DSH_HOME=$tmp/.dsh), либо научить install.sh предупреждать, когда DSH_HOME указывает на существующий каталог при нестандартном HOME. Источник: отчёт Task 1, ledger O-2.

Где смотреть

PR #60 (мержнут): #60
Архив change: openspec/changes/archive/2026-10-24-dsh-openspec/design.md, §6 «Приёмка».

Отложенные находки из #42 (DSH: OpenSpec integration). Источник — ledger ветки `feat/42-dsh-openspec` (удалён вместе с worktree; находки продублированы в PR #60 и комментарии к #42). ## Чеклист - [ ] **T2.1 / M-2** — негативный тест `--check` без openspec (`dsh/tests/openspec.test.mjs`, «warns but does not fail without the openspec CLI») подставляет `PATH = bin:/usr/bin:/bin:/usr/sbin:/sbin` и рассчитывает, что CLI там нет. На macOS верно (`openspec` в `/opt/homebrew/bin`), на Linux с `npm -g` (префикс `/usr`) CLI окажется в `/usr/bin` — тест найдёт его и упадёт. Предложение: вырезать `openspec` из PATH явно (временный каталог с симлинками на всё, кроме `openspec`) или проверять наличие CLI в тесте и скипать. Источник: ревью Task 2 (review-2-out.md), подтверждено финальным ревью. - [ ] **T3.4** — процессная: implementer Task 3 написал реализацию раньше тестов (TDD-порядок нарушен). Тесты не тавтологичны, поведение проверено. Источник: review-3-out.md. Решение: усилить формулировку в брифах SDD или в `sdd` skill: шаг «тест падает» обязателен до кода. - [ ] **O-2** — в окружении DSH-сессии экспортирован `DSH_HOME=/Users/andrey.epifancev/.dsh`; команда `HOME=$(mktemp -d) bash dsh/install.sh --skills` из брифа не изолирует и целится в реальный `~/.dsh` (спасла песочница, каталог не изменён). Предложение: в брифах и рецептах всегда изолировать обе переменные (`HOME=$tmp DSH_HOME=$tmp/.dsh`), либо научить `install.sh` предупреждать, когда `DSH_HOME` указывает на существующий каталог при нестандартном `HOME`. Источник: отчёт Task 1, ledger O-2. ## Где смотреть - PR #60 (мержнут): https://git.aepif.ru/aep/claude-harness/pulls/60 - Архив change: `openspec/changes/archive/2026-10-24-dsh-openspec/design.md`, §6 «Приёмка».

aep added this to the DSH port milestone 2026-10-24 20:44:24 +03:00

aep referenced this issue

2026-10-24 20:44:35 +03:00

DSH port: roadmap #48

aep referenced this issue

2026-10-24 21:12:40 +03:00

DSH port v2: план адаптации обвязки под DSH (вариант C) #69

aep referenced this issue from a commit

2026-10-25 12:06:10 +03:00

fix(dsh): #62 — PATH-изоляция теста, WARN DSH_HOME, TDD-порядок в sdd (#62)

aep referenced this issue from a commit

2026-10-25 12:06:10 +03:00