Follow-ups from #49 #66

Closed
opened 2026-10-24 20:25:22 +03:00 by aep · 1 comment
Owner

Отложенные находки из #49 (PR #64, архив #65). Маркеры присвоены в финальном ревью ветки.

  • M-1. dsh/bundle/index.js — предупреждение о legacy-ролях в apply не покрыто интеграционным тестом (примитивы isLegacyClaudePath/warnLegacyOnce и проводка bootstrap покрыты). Источник: ревью ветки #49, Minor.
  • M-2. dsh/install.sh --global зависит от python3 для тела AGENTS.md: при отсутствии python3 возможен усечённый файл (python3 — документированное требование и проверяется --check, но --global сам --check не вызывает). Источник: ревью ветки #49, Minor.
  • M-3. Старые каталоги скиллов не удаляются: замена per-skill оставляет скилл, исчезнувший в апстриме (пре-существующее поведение, не регрессия #49). Источник: ревью ветки #49, Minor.
  • O-1. guard-main: git worktree add <path> -b chore/x блокируется, а -b x с последующим git branch -m chore/x проходит. Похоже на разбор -b со слэшем; воспроизведено 24.10.2026 при создании archive-worktree. Источник: наблюдение контроллера вне ревью.
Отложенные находки из #49 (PR #64, архив #65). Маркеры присвоены в финальном ревью ветки. - [ ] **M-1.** `dsh/bundle/index.js` — предупреждение о legacy-ролях в `apply` не покрыто интеграционным тестом (примитивы `isLegacyClaudePath`/`warnLegacyOnce` и проводка bootstrap покрыты). Источник: ревью ветки #49, Minor. - [ ] **M-2.** `dsh/install.sh --global` зависит от `python3` для тела `AGENTS.md`: при отсутствии python3 возможен усечённый файл (python3 — документированное требование и проверяется `--check`, но `--global` сам `--check` не вызывает). Источник: ревью ветки #49, Minor. - [ ] **M-3.** Старые каталоги скиллов не удаляются: замена per-skill оставляет скилл, исчезнувший в апстриме (пре-существующее поведение, не регрессия #49). Источник: ревью ветки #49, Minor. - [ ] **O-1.** guard-main: `git worktree add <path> -b chore/x` блокируется, а `-b x` с последующим `git branch -m chore/x` проходит. Похоже на разбор `-b` со слэшем; воспроизведено 24.10.2026 при создании archive-worktree. Источник: наблюдение контроллера вне ревью.
aep added this to the DSH port milestone 2026-10-24 20:44:26 +03:00
aep closed this issue 2026-10-25 13:41:07 +03:00
Author
Owner

Закрыто PR #84 (feat/66-followups-49, коммит 58d48b0): M-1 (тест legacy-ролей), M-2 (python3-guard в --global), M-3 (манифест скиллов + удаление исчезнувших), O-1 (регрессия; баг был закрыт #63, коммиты 8d8dd89/e064de7). Верификация: dsh/tests 205/205, tests 641/0.

Закрыто PR #84 (feat/66-followups-49, коммит 58d48b0): M-1 (тест legacy-ролей), M-2 (python3-guard в --global), M-3 (манифест скиллов + удаление исчезнувших), O-1 (регрессия; баг был закрыт #63, коммиты 8d8dd89/e064de7). Верификация: dsh/tests 205/205, tests 641/0.
Sign in to join this conversation.
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: aep/claude-harness#66
No description provided.