Correct the branch write-back claim in the sync skill #31
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
skills/sync/SKILL.md:304-306утверждает, чтоpush.pyзаписывает вычисленныйbranch:обратно в файл issue. С тех пор как push удаляет файл после успешнойотправки, записывать некуда —
push.py:260-271говорит об этом прямо. Skill —единственный источник команд для агента, и здесь он описывает поведение,
которого больше нет.
Spec
skills/sync/SKILL.md
Steps to reproduce
Прочитать
skills/sync/SKILL.md, раздел проbranch::Прочитать комментарий в
skills/sync/scripts/push.py:260-271и код ниженего: значение проставляется только на in-memory issue.
Прогнать реальный сценарий:
push.py <slug>-> файл удалён ->pull.py <n>->branch:пришёл ссервера, а не из локальной записи.
Expected
Skill описывает то, что делает код: push отправляет
refнаверх, локальноничего не переписывает, и
branch:возвращается на диск следующимpull.py.Это уже правильно сформулировано в docstring
push.py:66-71и вreferences/format.md(branch| sync | «push fills an empty one … and neveroverwrites a filled one»).
Actual
Skill обещает запись в файл. Агент, который поверил тексту, будет искать в
tmp/issues/<id>.mdполе, появившееся после push, — а файла после push нетвообще. Формулировка пережила коммит
e629d14 feat: drop the local copy after a successful push, который переписал соседние абзацы того же файла, но этотпропустил.
Environment
Ветка
main, коммит2ac3015. Расхождение чисто документационное, кода некасается.
Acceptance criteria
skills/sync/SKILL.mdпроbranch:написано, что push отправляетrefи не пишет в файл; фраза «writes it back into the issue file» убрана
grep -n 'writes it back into the issue file' skills/ничего не находитникогда не перезаписывается, detached HEAD -> warning) сохранены — они
верны