f67a9298cc
* fix(run-pipeline): replace origin/master with origin/HEAD in templates B and C * docs(handoff): add handoff and ADR for run-pipeline origin/HEAD fix * docs(handoff): set PR number --------- Co-authored-by: opencode-agent <agent@opencode.local>
20 lines
No EOL
2.2 KiB
Markdown
20 lines
No EOL
2.2 KiB
Markdown
---
|
||
pr: 179
|
||
title: fix(run-pipeline): replace origin/master with origin/HEAD in templates B and C
|
||
---
|
||
|
||
## Что сделано
|
||
- Заменены `origin/master...HEAD` → `origin/HEAD...HEAD` в `.opencode/skills/run-pipeline/SKILL.md`:
|
||
- Line 73 (Template B, docs-review): `git diff origin/master...HEAD --stat` → `git diff origin/HEAD...HEAD --stat`
|
||
- Line 104 (Template C, code_review): `git diff origin/master...HEAD` → `git diff origin/HEAD...HEAD`
|
||
- Синхронизировано зеркало `~/.config/opencode/skills/run-pipeline/SKILL.md` (идентичные замены на lines 73 и 104).
|
||
- Подтверждено: `grep -rn "origin/master" .opencode/` → 0 совпадений; `grep -rn "origin/HEAD" .opencode/skills/run-pipeline/SKILL.md` → 2 совпадения (lines 73, 104).
|
||
|
||
## Почему
|
||
PR #163 устранил `origin/master...HEAD` → `origin/HEAD...HEAD` в `docs-reviewer.md`, но в `run-pipeline/SKILL.md` остались два устаревших вхождения (Template B и Template C). Эти шаблоны копируются оркестратором в invocation-prompt саб-агентам. На репо с default branch `main` (как текущий репо) `origin/master...HEAD` падает с `fatal: ambiguous argument`. `origin/HEAD` детерминированно указывает на default branch независимо от его имени (`master`/`main`).
|
||
|
||
## Pending
|
||
—
|
||
|
||
## Watch out
|
||
- **ADR #076 коллизия нумерации (issue #176).** scaffold-handoff.sh вычисляет следующий номер ADR как `count(files) + 1`, а не `max(existing numbers) + 1`. В репо уже существовал `075-pr-178-*` (для PR #178 — хронологически позже), поэтому новый ADR для PR #164 получил `076` — номер больше, чем у PR #178. Это не блокирующая косметическая коллизия: ADR остаётся читаемым и однозначно связан с этим PR через slug. Не чинил (вне scope, см. issue #176). Не ренумеровал существующие ADR — это нарушило бы ссылки. |