Stevens / Ivan Issue Report

Повестка внутренней встречи с Антоном и Иваном

Дата: завтра Цель: установить факты (особенно по git-истории), договориться о процессных изменениях, после этого готовить ответ Stevens. Stevens до встречи не пишем.

План (75 мин)

1. Контекст письма Alexis (5 мин)

  • Что Alexis написал 29.04
  • 4 ключевых тезиса
  • Что header упомянут прямо
  • Что PR #847 упомянут прямо

2. Сводка фактов по PR (20 мин)

Прошлись по всем 5 разобранным PR:

  • PR #837 — merged, 2 претензии (Contentful env, конфигурация поля в нарушение спеки ICUS-214 которую Андрей же приложил 11.03)
  • PR #842 — OPEN 16+ дней, 4 претензии (debug console.log, missing pre-merge tasks, invalid VideoObject, ~700 unsynced entries)
  • PR #832 — OPEN 2+ месяца, 9 претензий, REQUESTED CHANGES + Alexis уже 23.02 сигнализировал про pre-deployment (за 2 мес до письма)
  • PR #846 (header) — 11 претензий за 8 дней (8 self-QA + 3 пробела спеки)
  • PR #847 — 🚨 commit-after-approve schema (TODO "when approve PR")

3. PR #847 — git-история (15 мин)

Иван открывает Bitbucket / локальный git:

  • merge strategy PR #837 (был ли squash при закрытии 24.03)
  • состояние branch ICUS-216-faq после merge
  • force-push'и между PR #837 и PR #847
  • локальный rebase/squash перед открытием PR #847
  • git reflog ICUS-216-faq (если возможно)

→ Получаем ответ: что именно увидел Alexis под "squashing".

4. Корневые причины (15 мин)

По всем PR — повторяющиеся паттерны (см. cross-PR summary):

  • Self-QA дисциплина (визуальная сверка, responsive, dark mode, env-vars grep)
  • Чтение/выполнение спеки до старта (parent-тикета, attachments)
  • PR description (pre-merge tasks, миграции существующих данных)
  • Workflow дисциплина (no commit-after-approve, no rebase after share)
  • Размер батча и длительность PR

5. План процессных изменений (15 мин)

Что вводим у себя (предложения для обсуждения):

  1. Self-QA checklist перед "Open PR" (responsive, dark mode, mockup-overlay, env-vars)
  2. PR template с обязательными секциями (Pre-merge tasks, Migration plan, Validation done)
  3. Spec-read checkpoint перед стартом сабтаска (Иван явно подтверждает прочитанную parent-задачу)
  4. Workflow rules: no rebase после первого push; no commit-after-approve; if not ready — draft
  5. Internal validation pass — кто и когда сверяет реализацию с acceptance criteria
  6. Размер батча — договариваемся о макс. длительности PR / способе деления

6. Договоримся о следующих шагах (5 мин)

  • Кто и когда правит существующие открытые PR (#832, #842, #846, #847)
  • Когда формируем ответ Stevens (после встречи)
  • Кто пишет первый черновик ответа

К встрече иметь под рукой

  • Письмо Alexis — claims/screenshots/2026-04-29_alexis-watson_dev-workflow-email.png
  • Спеки: ICUS-214 PDF (FAQ), ICUS-220 user stories v2 + mockups v3 (header), ICUS-211/212 (YouTube)
  • Все скрины комментариев (по папкам PR)
  • Cross-PR summary (в работе)
  • Финальные questions-anton / questions-ivan (в работе)