[P0] 05.root. Сделать успешные CI checks условием merge #3

Closed
opened 2026-09-13 15:33:30 +03:00 by codex-bot · 1 comment
Owner

Источник: локальный todo/wealthledger-engineering-improvements.md, пункт 05.root, анализ от 13.09.2026. Исходные критерии приведены ниже, чтобы issue не зависела от наличия локального TODO.

Что осталось сделать

05.root. Сделать успешные CI checks условием merge. Размер: S. Владелец: root / координация настроек Git hosting.

После появления работающих jobs настроить protection/default branch и required
checks для root и согласовать общий принцип для остальных repos. Правила обхода и доступы определяет владелец проекта.
Не считать YAML pipeline доказательством включённой защиты на сервере.
Готово, когда: тестовый PR с failing required check не может пройти обычный
merge; подтверждены настройки и имена checks; новый commit требует новой проверки.
Если hosting не поддерживает нужную защиту, явно записать оставшееся ограничение.
Зависимость: 04. Основание: A, с. 20–21; B, с. 29.

Основание текущего статуса

В Gitea workspace представлен репозиторием WealthLedger/main. Локально есть .git, но root AGENTS утверждает обратное; root README отсутствует. Это конфликт описания topology, который нужно явно разрешить при 02. Вложенные frontend/backend/docs/specs остаются независимыми репозиториями.

Проверка 13.09.2026: статическая сверка текущих файлов и remediation registry; через MCP проверены репозитории, issues, ветки и Actions workflows. Все пять main-веток имеют protected=false, список workflows пуст, существующих issues не найдено. Внешние CI, server-side updater, история секретов и свежий npm audit этой сверкой не проверялись; полные application gates повторно не запускались.

Проверяемая revision main: ee2adc9fda6ce319422e5fe89143b9a1584d690f.

Remediation registry остаётся владельцем статусов audit findings. Создание issue не означает выполнение или изменение статуса finding. Закрыть после проверки критериев и ссылки на фактические результаты.

Связанные issues

<!-- wealthledger-engineering-improvements:main:05.root --> Источник: локальный `todo/wealthledger-engineering-improvements.md`, пункт 05.root, анализ от 13.09.2026. Исходные критерии приведены ниже, чтобы issue не зависела от наличия локального TODO. ## Что осталось сделать **05.root. Сделать успешные CI checks условием merge.** Размер: S. Владелец: root / координация настроек Git hosting. После появления работающих jobs настроить protection/default branch и required checks для root и согласовать общий принцип для остальных repos. Правила обхода и доступы определяет владелец проекта. Не считать YAML pipeline доказательством включённой защиты на сервере. **Готово, когда:** тестовый PR с failing required check не может пройти обычный merge; подтверждены настройки и имена checks; новый commit требует новой проверки. Если hosting не поддерживает нужную защиту, явно записать оставшееся ограничение. **Зависимость:** 04. **Основание:** A, с. 20–21; B, с. 29. ## Основание текущего статуса В Gitea workspace представлен репозиторием `WealthLedger/main`. Локально есть `.git`, но root AGENTS утверждает обратное; root README отсутствует. Это конфликт описания topology, который нужно явно разрешить при 02. Вложенные frontend/backend/docs/specs остаются независимыми репозиториями. Проверка 13.09.2026: статическая сверка текущих файлов и remediation registry; через MCP проверены репозитории, issues, ветки и Actions workflows. Все пять main-веток имеют protected=false, список workflows пуст, существующих issues не найдено. Внешние CI, server-side updater, история секретов и свежий npm audit этой сверкой не проверялись; полные application gates повторно не запускались. Проверяемая revision main: `ee2adc9fda6ce319422e5fe89143b9a1584d690f`. [Remediation registry](https://git.personal.ksv741.ru/WealthLedger/docs/src/commit/7bc8792b9853768734066c31a323b70886d73a23/development/audit-remediation.md) остаётся владельцем статусов audit findings. Создание issue не означает выполнение или изменение статуса finding. Закрыть после проверки критериев и ссылки на фактические результаты. ## Связанные issues - [backend: [P0] 05.backend. Включить required checks для backend](https://git.personal.ksv741.ru/WealthLedger/backend/issues/2) - [frontend: [P0] 05.frontend. Включить required checks для frontend](https://git.personal.ksv741.ru/WealthLedger/frontend/issues/2) - [specs: [P0] 05.specs. Включить required checks для specs](https://git.personal.ksv741.ru/WealthLedger/specs/issues/3) - [docs: [P0] 05.docs. Включить required checks для docs](https://git.personal.ksv741.ru/WealthLedger/docs/issues/2) - [main: [P0] 04.root. Согласовать CI для независимых репозиториев](https://git.personal.ksv741.ru/WealthLedger/main/issues/2)
Author
Owner

Закрываем задачу настройки required checks на основании подтверждения владельца проекта от 16.09.2026: required checks настроены во всех пяти репозиториях; падение обязательной проверки блокирует обычный merge, новый commit требует нового результата проверки.

Источник подтверждения — явное сообщение владельца в рабочем обсуждении («да, все подтверждено»). Независимая повторная проверка серверных настроек или тестовый failing PR агентом при закрытии не выполнялись.

Для root действует принятый scope CI: штатный commitlint. Удалённые verify/security workflows не являются требованием этой задачи.

Закрываем задачу настройки required checks на основании подтверждения владельца проекта от 16.09.2026: required checks настроены во всех пяти репозиториях; падение обязательной проверки блокирует обычный merge, новый commit требует нового результата проверки. Источник подтверждения — явное сообщение владельца в рабочем обсуждении («да, все подтверждено»). Независимая повторная проверка серверных настроек или тестовый failing PR агентом при закрытии не выполнялись. Для root действует принятый scope CI: штатный commitlint. Удалённые verify/security workflows не являются требованием этой задачи.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: WealthLedger/main#3