mirror of
https://github.com/Matysh/houseplan-card
synced 2026-10-01 20:29:00 +00:00
ci: ребейз виден автору до пуша, конфликт называет файлы, уход dev отмечается
Конвейер приводит ветку к dev сам (#257) и при конфликте возвращает задачу, не тратя цикл ревью. Оставались три щели, и все три про то, что человек узнаёт поздно и без подробностей. Первое. Отставание теперь видно в scripts/pre-push-gate.mjs до пуша, с числом коммитов и готовой командой. Это предупреждение, а не гейт: гейтом остаётся конвейер, который забыть не может. Смысл в цене — после любого ребейза разбор становится полным, а не по дельте (§7.2), а конфликт всё равно чинится на машине автора. Отключается --no-rebase-check. Второе. Конфликт называет файлы. Список снимается ДО `git rebase --abort`: abort снимает состояние конфликта вместе с ним, и раньше автору доставалось «не ребейзится» без единого имени. Логика проверена на настоящем конфликте в одноразовом репозитории — два файла названы. Третье. Если dev ушёл вперёд, пока шло ревью, это записывается в summary прогона, а при зелёном вердикте ещё и комментарием: вердикт вынесен по дереву, которое уже не совпадает с вершиной линии, и слияние приведёт ветку к dev. Комментарий только при зелёном — шуметь на каждом прогоне ни к чему, а вот молчать перед слиянием нельзя. Чего задача не делает: не заставляет dev стоять на месте, пока идёт ревью. Если возвраты частые именно из-за темпа, лечится очередью слияний, а это решение о процессе, не о скрипте. Два мутанта проверены руками — «советовать ребейз всегда» и «никогда не сообщать про уход dev», — каждый убит. Issue: #364 User-Visible: no
This commit is contained in:
@@ -0,0 +1,50 @@
|
||||
/**
|
||||
* Состояние ветки задачи относительно `dev` (#364).
|
||||
*
|
||||
* Конвейер приводит ветку к `dev` сам (#257) и при конфликте возвращает задачу,
|
||||
* не тратя цикл ревью. Но конфликт при этом обнаруживается в комментарии через
|
||||
* сорок минут, а чинится всё равно на машине автора. Дешевле сказать ему это до
|
||||
* пуша — тогда конвейеру нечего ребейзить, а конфликт виден там, где есть руки.
|
||||
*
|
||||
* Отставание — предупреждение, а не запрет: гейтом остаётся конвейер, который
|
||||
* забыть не может. Локальная проверка лишь убирает лишний круг.
|
||||
*/
|
||||
|
||||
/**
|
||||
* Совет по ребейзу. `null` означает «ветка содержит весь dev, делать нечего».
|
||||
*
|
||||
* @param {{ behind: number, base?: string }} state
|
||||
*/
|
||||
export function rebaseAdvice({ behind, base = 'origin/dev' }) {
|
||||
const count = Number(behind);
|
||||
if (!Number.isFinite(count) || count <= 0) return null;
|
||||
const commits = count === 1 ? 'коммит' : count < 5 ? 'коммита' : 'коммитов';
|
||||
return `ветка отстала от ${base} на ${count} ${commits}.`
|
||||
+ ` Конвейер приведёт её сам перед ревью, но после ребейза разбор станет`
|
||||
+ ` полным, а не по дельте (§7.2). Дешевле сделать это здесь:`
|
||||
+ ` git fetch origin && git rebase ${base}`;
|
||||
}
|
||||
|
||||
/**
|
||||
* Пути, на которых встал ребейз, из вывода `git diff --name-only --diff-filter=U`.
|
||||
*
|
||||
* Читается ДО `git rebase --abort`: abort снимает состояние конфликта вместе с
|
||||
* этим списком, и тогда автору остаётся только «не ребейзится», без единого
|
||||
* имени файла.
|
||||
*/
|
||||
export function conflictingPaths(output) {
|
||||
return [...new Set(String(output ?? '').split('\n')
|
||||
.map((line) => line.trim())
|
||||
.filter(Boolean))].sort();
|
||||
}
|
||||
|
||||
/** Строка о том, что `dev` ушёл вперёд, пока шло ревью. `null`, если не ушёл. */
|
||||
export function devMovedNote({ moved, sha }) {
|
||||
const count = Number(moved);
|
||||
if (!Number.isFinite(count) || count <= 0) return null;
|
||||
const commits = count === 1 ? 'коммит' : count < 5 ? 'коммита' : 'коммитов';
|
||||
const at = sha ? ` Материал ревью — \`${sha}\`.` : '';
|
||||
return `Пока шло ревью, \`dev\` продвинулся на ${count} ${commits}.${at}`
|
||||
+ ' Вердикт вынесен по дереву, которое уже не совпадает с вершиной линии:'
|
||||
+ ' слияние приведёт ветку к dev, и это другой код (§7.2).';
|
||||
}
|
||||
@@ -31,6 +31,7 @@ import { spawnSync } from 'node:child_process';
|
||||
import { existsSync } from 'node:fs';
|
||||
import { dirname, resolve } from 'node:path';
|
||||
import { fileURLToPath } from 'node:url';
|
||||
import { rebaseAdvice } from './branch-state.mjs';
|
||||
|
||||
const ROOT = resolve(dirname(fileURLToPath(import.meta.url)), '..');
|
||||
const argv = process.argv.slice(2);
|
||||
@@ -63,6 +64,26 @@ const capture = (command, args) => spawnSync(command, args, { cwd: ROOT, encodin
|
||||
const steps = [];
|
||||
const skipped = [];
|
||||
|
||||
// ---- приведена ли ветка к dev ---------------------------------------------
|
||||
// Конвейер ребейзит сам (#257), но после ребейза разбор становится полным, а не
|
||||
// по дельте (§7.2), и конфликт всплывает в комментарии через сорок минут вместо
|
||||
// машины автора. Поэтому предупреждение, а не гейт: гейтом остаётся конвейер.
|
||||
if (!flag('no-rebase-check')) {
|
||||
const fetched = capture('git', ['fetch', '-q', 'origin', base.replace(/^origin\//, '')]);
|
||||
if (fetched.status !== 0) {
|
||||
skipped.push(`проверка отставания от ${base} — git fetch не удался`);
|
||||
} else {
|
||||
const ancestor = capture('git', ['merge-base', '--is-ancestor', base, head]);
|
||||
if (ancestor.status === 0) {
|
||||
console.log(`\n── Ветка\n содержит весь ${base}, ребейз не нужен`);
|
||||
} else {
|
||||
const counted = capture('git', ['rev-list', '--count', `${head}..${base}`]);
|
||||
const advice = rebaseAdvice({ behind: Number(counted.stdout.trim()), base });
|
||||
if (advice) skipped.push(advice);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
// ---- что тронуто ----------------------------------------------------------
|
||||
const diff = capture('git', ['diff', '--name-only', `${base}..${head}`]);
|
||||
if (diff.status !== 0) {
|
||||
|
||||
Reference in New Issue
Block a user