Repository navigation
Merge discipline: up-to-date branches and a PR checklist (main went red after #67) #105
Copy link
Copy link
Closed
Labels
enhancementNew feature or requestNew feature or request
Description
Part of #87.
⚠️ Выводы не проверены человеком. Ишью заведено по аудиту, который сделал Claude Code. Проверка ниже — одна ссылка или одна команда.
Что случилось
- 2026-10-08 13:44 UTC смержен fix(#64): skip teammate relays in session preview firstUserMessage #66. Через 25 минут смержен fix(#59): classify mixed relay-XML content as user text #67 — из ветки, которая не была обновлена после fix(#64): skip teammate relays in session preview firstUserMessage #66.
- По отдельности оба PR были зелёными. Вместе они сломали 2 теста fix(#64): skip teammate relays in session preview firstUserMessage #66 в
test/main/utils/jsonl.test.ts: fix(#59): classify mixed relay-XML content as user text #67 перестал считать «смешанный» текст релеем, а тесты fix(#64): skip teammate relays in session preview firstUserMessage #66 использовали именно такой текст. - CI на main — run #112: failure. main был красным ~4,7 ч, пока fix(turn-budget): one turn, one number — hook, bell and panel parity #68 не переписал тесты.
За всю историю форка это единственный красный main, но и защиты от повторения нет.
Сопутствующее, по аудиту 10 последних PR:
- в 3 PR описание расходится со смерженным кодом:
- feat(bell): catch up on incidents that happened while the app was closed #51 заявляет
scripts/scan-stuck-reviewers.ts, а файл в.gitignore; - fix(#54): recompute context stats on in-place session refresh #57 описывает
sessionPhase2Generation, а в коде проверки по identity; - заголовок fix(turn): compaction is not a turn — one numbering everywhere #69 «one numbering everywhere» не выполняется (CLI turn numbers differ from chat chips: relays not counted, sidechain prompts counted #99);
- feat(bell): catch up on incidents that happened while the app was closed #51 заявляет
- UI-изменения мержатся без тестов на уровне отображения — fix(turn): task-notification opens no user turn #70 убрал карточки фоновых задач (Background task cards disappeared from the chat after #70 #104).
Как проверить (один раз)
Открыть run #112 по ссылке выше. Или локально:
git checkout 70e3053 && pnpm vitest run test/main/utils/jsonl.test.ts; git checkout -Сейчас: 2 failed (skips teammate relays when picking firstUserMessage, picks a later real user message when the session opens with a relay).
Что сделать
- Settings → Branches → правило для
main: «Require status checks to pass» (проверкаci) и «Require branches to be up to date before merging» — или merge queue. В CI: typecheck / lint / test on push + PR #6 branch protection названа необязательной для репо с одним мейнтейнером; этот случай показывает её цену, решение стоит принять явно. .github/pull_request_template.mdс коротким чек-листом:- ветка обновлена с
main; - описание совпадает с итоговым кодом;
- UI-изменение — есть тест на уровне элементов отображения;
- для оставленных компромиссов заведены follow-up issue.
- ветка обновлена с
Acceptance
- Правило для
mainвключено (или явно отклонено с причиной в этом ишью) - Шаблон PR есть в репозитории
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request
Activity
ci: compare PR head sha in freshness check, not merge commit (#105)
ci: branch freshness check and PR checklist template (#105) (#126)