13 KiB
P22 — декабрь 2019: ответственность за код
Статус: принят в publication registry 31 июля 2026 года. Три revision применяются по стабильным slug и сохраняют дату и автора базового архива:
editorial-2019-12-practice-code-ownership;editorial-2019-12-mechanism-code-ownership;editorial-2019-12-field-code-ownership.
Модуль экспортирует ровно три revision без полей date и author. При
прямом запуске с --print-revisions stdout содержит только JSON, совпадающий
с import-safe export. Все псевдонимы @example, issue SIM-2019-12-17 и
события полевого разбора — учебная симуляция; они не описывают частный
репозиторий, команду, пользователей или реальный инцидент.
Проход 1. Факты и техника — пройдено
| Утверждение или решение | Официальный источник | Проверенная граница |
|---|---|---|
git blame показывает revision и автора, последними изменивших каждую строку |
Git: git-blame | Историческая атрибуция не выдана за назначение текущего владельца решения; в статьях сначала исследуется context, затем роль фиксируется отдельно |
git log нужен для истории узких путей и связанных commits |
Git: git-log | История файла не названа полной картой внешних зависимостей; в поле зрения остаются контракт и соседние modules |
CODEOWNERS определяет людей или teams для путей, а review request берётся по правилам base branch pull request |
GitHub Docs: About code owners | Механизм назван возможностью GitHub, а не частью стандарта Git; feature branch не объявлена источником route для собственного merge |
В GitHub последнее совпадающее CODEOWNERS-правило имеет приоритет, а ! и диапазоны [] не работают как в .gitignore |
GitHub Docs: About code owners | Пример использует общий и более узкий path без ложного «исключения»; отдельной строкой закрыт сам .github/CODEOWNERS |
| Pull request review имеет решения Comment, Approve и Request changes; защита ветки может требовать approval | GitHub Docs: Pull request reviews | Автоматический request reviewer не назван выполненным review, а approval не подменяет проверку результата после merge |
Честная граница исследования
Пакет использует публичную официальную документацию Git и GitHub. В нём нет
выводов о реальной конфигурации GitHub, branch protection, составе teams или
истории какого-либо закрытого репозитория. CODEOWNERS в примерах —
псевдоконфигурация с @example; она показывает порядок и границы механизма,
но не была отправлена в хостинг для проверки actual reviewer request.
Технический принцип в трёх статьях сознательно уже: история Git — источник факта, CODEOWNERS — маршрут запроса review, итог review — решение по diff, follow-up — самостоятельное действие после merge. Инструменты не объединены в фиктивного «единственного владельца».
Вердикт прохода: пройден. Нормативные свойства привязаны к первичным документам, а зависящие от конкретного хостинга и команды результаты помечены как будущая проверка интеграционного этапа.
Проход 2. Редактура и голос M2 / 2019 — пройдено
| Ревизия | Симптом и цена в начале | Главный технический вопрос | Артефакт и ограничение |
|---|---|---|---|
| Практика | Дефект стоит на месте, потому что автор строки, владелец решения и reviewer смешаны; цена — повторный bug и задержка изменения | Как разложить ownership одного change на решение, путь кода, review и follow-up | Таблица ролей, пример CODEOWNERS, маршрут из шести шагов; платформа не заявлена настроенной |
| Механизм | git blame уже назвал автора, но смысл статуса всё ещё не решён; цена — ложный выбор ответственного |
Где заканчивается история Git, что именно делает CODEOWNERS и что остаётся после review | Сопоставление следов, команды Git и схема потока; нет утверждения о правах или policy конкретного репозитория |
| Полевой разбор | Неизвестный gateway status превращён в success, а дефект пересекает два module; цена — неверное действие на экране | Как провести issue/review timeline без легенды о реальном инциденте | Анонимизированная fixture SIM-2019-12-17, таблица фактов и route; симуляция не выдана за production-проверку |
- Основной текст: практика — 7 118 знаков, механизм — 8 118, полевой разбор — 8 616. Все значения в коридоре 5 000–15 000 и рассчитаны gate без раздела источников.
- Во всех статьях есть ранние «симптом», «ошибка» или конкретный сбой с ценой; далее выдержана цепочка: симптом → причина смешения ролей → проверка → действие → граница знания.
- Голос соответствует M2 / 2019: автор говорит о Git, путях, тесте, контракте, pull request и review предметно. Он не приписывает себе SLO, организационные метрики, приватные процессы или поздний управленческий манифест.
- У каждой revision не менее пяти смысловых разделов, доступная таблица с
captionиthead, figure с развёрнутым alt и подписью, кодовый пример, упорядоченный маршрут и как минимум четыре официальных ссылки. - Из текста удалены шаблонные обещания и подмена факта словом «владелец»;
git blame, CODEOWNERS, review и follow-up описаны разными глаголами.
Вердикт прохода: пройден. Тексты стали практичными без перехода к поздней менеджерской риторике: каждый раздел ведёт к проверяемому действию.
Проход 3. Визуал и выпуск — пройдено в пределах автономного пакета
code-ownership-three-surfaces-2019.svgсначала был слишком широк для мобильного чтения. После отдельного визуального просмотра он переделан в узкую вертикальную схему: symptom, решение, путь кода, review, follow-up и Git history читаются последовательно.code-ownership-resolution-2019.svgпоказывает не «органиграмму», а маршрут артефактов: наблюдаемый сбой → исторический факт → route review → решение reviewer → проверка после merge.code-ownership-simulated-timeline-2019.svgкрупно помечает simulation и не содержит намёков на настоящую команду. Шкала отделяет поиск context от назначения ролей.- Все три SVG содержат
title,desc,role="img"иaria-labelledby. В них нет JavaScript,foreignObject, внешних URL или растровых data URI. У figure в статьях есть отдельные alt-тексты и captions. - SVG были отрендерены локально через Sharp в исходном размере и при ширине 375 px. После перестройки узкой композиции нет обрезания текста или горизонтального выхода за границы. Это проверка самих диаграмм, не браузерный e2e-прогон опубликованной страницы.
- Основной редактор независимо просмотрел финальные три растеризованных схемы на 375 px; вывод подтверждён: текст не обрезан, последовательность артефактов читается. Статьи не выдают это за проверку интерактивного поведения хостинга или реального review-flow.
- После подключения registry strict audit и production build пройдены; ручной browser-review остаётся отдельной проверкой и не заявлен как выполненный.
Фактические проверки
node --check web/scripts/upgrade-2019-12.mjs
cd web && npm run audit:draft -- scripts/upgrade-2019-12.mjs
xmllint --noout \
web/public/assets/editorial/2019/code-ownership-three-surfaces-2019.svg \
web/public/assets/editorial/2019/code-ownership-resolution-2019.svg \
web/public/assets/editorial/2019/code-ownership-simulated-timeline-2019.svg
| Проверка | Фактический результат |
|---|---|
node --check |
PASS, код 0 |
--print-revisions и import-safe export |
PASS внутри audit:draft: stdout JSON-only и ровно три revision |
npm run audit:draft -- scripts/upgrade-2019-12.mjs |
PASS: 7 118 / 8 118 / 8 616 знаков, структура и локальные assets найдены |
xmllint --noout для трёх SVG |
PASS, код 0 |
| Локальный visual review | PASS: исходный размер и 375 px, после перестройки нет clipping или horizontal overflow внутри SVG |
| Strict audit после подключения registry | PASS: 7 118 / 8 118 / 8 616 знаков; по одному figure и table, 1 / 2 / 1 code example |
npm run build |
PASS, code 0, 374 статические страницы |
| Scope/self-review | PASS: в revision нет date/author, а articles.json не перезаписан |
Выпусковой вердикт: тройное ревью пройдено, пакет принят к публикации. Registry заменяет только редакционные поля по stable slug. Production build и visual preflight не подменяют реальную проверку поведения на выбранной платформе review: её нужно выполнить отдельно при работе с конкретным репозиторием.