You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Merge discipline: up-to-date branches and a PR checklist (main went red after #67) #105
Сейчас: 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 включено (или явно отклонено с причиной в этом ишью)
Part of #87.
Что случилось
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 использовали именно такой текст.За всю историю форка это единственный красный main, но и защиты от повторения нет.
Сопутствующее, по аудиту 10 последних PR:
scripts/scan-stuck-reviewers.ts, а файл в.gitignore;sessionPhase2Generation, а в коде проверки по identity;Как проверить (один раз)
Открыть run #112 по ссылке выше. Или локально:
Сейчас: 2 failed (
skips teammate relays when picking firstUserMessage,picks a later real user message when the session opens with a relay).Что сделать
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;Acceptance
mainвключено (или явно отклонено с причиной в этом ишью)