fix(dsh): /approve и гейты резолвят change/план в worktree (#76) #82

aep · 2026-10-25T13:18:19+03:00

aep commented

2026-10-25 13:18:19 +03:00

Что

/approve и гейты A/B резолвили openspec/changes/<id>/ и docs/superpowers/plans/ только от cwd сессии. Из основного checkout change/план в связанном worktree не были видны — /approve design отвечал «missing: proposal.md, design.md …» при существующих файлах.

Как

Артефакты резолвятся по списку корней-кандидатов: корень репозитория сессии → worktree .claude/worktrees/*/ и .wt/*/ (только каталоги с .git), фолбэк на cwd. Порядок детерминированный.

gates.js: findRepoRoot, candidateRoots, findChangeRoot, isDir; checkPlanWrite/checkDispatch/checkSddWrite/resolvePlanFromSdd учитывают worktree.
approval.js: runApproval нормализует корень; approveDesign/approvePlan пишут approval.md в корень change.
spec.md: замена абзаца T3.8 на worktree-резолюцию + 3 scenario.

Верификация

bash dsh/tests/run.sh — 216/216
bash tests/run.sh — 640/0
openspec validate approval-gates --strict — valid

Rulings I made

Change в нескольких корнях → первый по порядку [корень сессии, .claude/worktrees/*, .wt/*].
План в нескольких корнях → отказ с перечислением (контракт «ровно один план» сохранён).
arguments.cwd диспатча игнорируется; план в worktree гейтится по approval его change.

Close: #76

## Что `/approve` и гейты A/B резолвили `openspec/changes/<id>/` и `docs/superpowers/plans/` только от cwd сессии. Из основного checkout change/план в связанном worktree не были видны — `/approve design` отвечал «missing: proposal.md, design.md …» при существующих файлах. ## Как Артефакты резолвятся по списку корней-кандидатов: корень репозитория сессии → worktree `.claude/worktrees/*/` и `.wt/*/` (только каталоги с `.git`), фолбэк на cwd. Порядок детерминированный. - `gates.js`: `findRepoRoot`, `candidateRoots`, `findChangeRoot`, `isDir`; `checkPlanWrite`/`checkDispatch`/`checkSddWrite`/`resolvePlanFromSdd` учитывают worktree. - `approval.js`: `runApproval` нормализует корень; `approveDesign`/`approvePlan` пишут `approval.md` в корень change. - `spec.md`: замена абзаца T3.8 на worktree-резолюцию + 3 scenario. ## Верификация - `bash dsh/tests/run.sh` — 216/216 - `bash tests/run.sh` — 640/0 - `openspec validate approval-gates --strict` — valid ## Rulings I made - Change в нескольких корнях → первый по порядку [корень сессии, `.claude/worktrees/*`, `.wt/*`]. - План в нескольких корнях → отказ с перечислением (контракт «ровно один план» сохранён). - `arguments.cwd` диспатча игнорируется; план в worktree гейтится по approval его change. Close: #76

aep added 1 commit 2026-10-25 13:18:20 +03:00

fix(dsh): /approve и гейты резолвят change/план в worktree (.claude/worktrees, .wt) (#76 ) 885c451c22

aep merged commit 148ca131a7 into main

2026-10-25 13:40:56 +03:00

aep referenced this issue from a commit

2026-10-25 13:40:57 +03:00

Merge pull request 'fix(dsh): /approve и гейты резолвят change/план в worktree (#76)' (#82) from feat/76-approve-resolve into main

aep referenced this pull request

2026-10-25 13:41:21 +03:00

approval-gates: /approve и гейты не видят change/план в worktree (приёмка A2) #76

Sign in to join this conversation.