Correct the branch write-back claim in the sync skill #31

Closed
opened 2026-08-10 12:32:04 +00:00 by claude · 0 comments
Collaborator

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

  1. Прочитать skills/sync/SKILL.md, раздел про branch::

    `branch:` is Gitea's `ref`, the branch the work actually lives on. Push fills
    an empty one with the current git branch (`git rev-parse --abbrev-ref HEAD`)
    and writes it back into the issue file; a value already there is never
    overwritten, neither on create nor on `--update`.
    
  2. Прочитать комментарий в skills/sync/scripts/push.py:260-271 и код ниже
    него: значение проставляется только на in-memory issue.

  3. Прогнать реальный сценарий:
    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 never
overwrites 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/ ничего не находит
  • остальные утверждения абзаца (пустое поле заполняется из git, непустое
    никогда не перезаписывается, detached HEAD -> warning) сохранены — они
    верны
<!-- tea:id correct-the-branch-write-back-claim-in-the-sync --> ## 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 1. Прочитать `skills/sync/SKILL.md`, раздел про `branch:`: ``` `branch:` is Gitea's `ref`, the branch the work actually lives on. Push fills an empty one with the current git branch (`git rev-parse --abbrev-ref HEAD`) and writes it back into the issue file; a value already there is never overwritten, neither on create nor on `--update`. ``` 2. Прочитать комментарий в `skills/sync/scripts/push.py:260-271` и код ниже него: значение проставляется только на in-memory issue. 3. Прогнать реальный сценарий: `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 never overwrites 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/` ничего не находит - [ ] остальные утверждения абзаца (пустое поле заполняется из git, непустое никогда не перезаписывается, detached HEAD -> warning) сохранены — они верны
claude added this to the Self-improvement milestone 2026-08-10 12:32:04 +00:00
claude added the comp/sync
type
bug
severity
low
labels 2026-08-10 12:32:04 +00:00
Sign in to join this conversation.