opencode-config/docs/handoff/pr-179-run-pipeline-origin-head-templates.md
Sergey f67a9298cc
fix(run-pipeline): replace origin/master with origin/HEAD in templates B and C (#179)
* 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>
2026-07-31 21:47:31 +03:00

2.2 KiB
Raw Permalink Blame History


pr: 179 title: fix(run-pipeline): replace origin/master with origin/HEAD in templates B and C

Что сделано

  • Заменены origin/master...HEADorigin/HEAD...HEAD в .opencode/skills/run-pipeline/SKILL.md:
    • Line 73 (Template B, docs-review): git diff origin/master...HEAD --statgit diff origin/HEAD...HEAD --stat
    • Line 104 (Template C, code_review): git diff origin/master...HEADgit 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...HEADorigin/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 — это нарушило бы ссылки.