Stevens / Ivan Issue Report

Вопросы Антону (тимлид)

Антон тегнут Alexis 23.02 в PR #832 (про pre-deployment steps) — то есть он был в курсе сигнала за 2 месяца до письма. Также Антон фигурирует как "Anton Moskalenko" — тимлид нашей команды.

A. Контекст и история сигналов

  1. Знал ли ты про комментарий Alexis от 23.02 в PR #832: "Are there any additional steps we need to take prior to deployment?"

    • Если да — что было сделано в ответ? Почему через 5 недель в PR #842 ровно эта же проблема повторилась?
    • Если нет — кто отвечает за то, чтобы такие сигналы доходили до тебя?

  2. Письмо Alexis от 29.04 — для тебя это сюрприз или ожидаемая эскалация по накопившимся замечаниям?

  3. Получали ли мы раньше похожие сигналы от Tom (iX) — устно, в Slack, в звонках?

B. Self-QA / internal validation

  1. Делается ли сейчас на нашей стороне internal validation pass перед отправкой PR на ревью Stevens?

    • Если да — кто, что проверяет, как фиксируется?
    • Если нет — кто должен это делать (ты, Андрей, Иван сам)?

  2. Согласны ли ввести обязательный self-QA checklist для Ивана перед "Open PR" (см. analysis/cross-pr-patterns.md — раздел A)?

    • Что бы ты добавил/убрал из этого checklist'а?

  3. Кто на нашей стороне сравнивает реализацию с acceptance criteria до отправки на ревью Stevens?

C. PR template и workflow

  1. Согласен с обязательной структурой PR description (Summary / Changes / Pre-merge tasks / Migration plan / Validation done / Compare URLs)?

  2. Какое у нас сейчас правило по squash/rebase? Формализовано ли где-то?

    • Готов ли ввести правило: после первого push на remote — никаких rebase/force-push?

  3. По кейсу PR #847 (commit-after-approve, TODO "when approve PR") — как ты это видишь?

    • Это разовая ошибка Ивана или знак, что у нас нет понимания, что approve привязан к конкретному коммиту?
    • Согласен ввести правило "если код не готов — Draft PR, никаких TODO в коде"?

D. Спека и parent-тикеты

  1. Есть ли у нас сейчас правило "перед стартом сабтаска прочитать parent-задачу и приложенные документы"?

  2. Согласен ввести Spec-read checkpoint — Иван перед стартом пишет в Jira-комменте, что прочитал и какие acceptance criteria выписал?

  3. Кто следит за фразами "to be confirmed" / "agreed before implementation begins" в спеке (Comment #11 PR #846 — data-cta)?

E. Размер батча и циклы ревью

  1. PR #832 — 2+ месяца OPEN, 27+ коммитов. PR #846 — 11 претензий за 2 недели. Можно ли было разбивать?

    • Если да — почему не разбивали?
    • Если нет — какая структурная причина?

  2. Согласен ли с лимитом "PR не висит OPEN дольше 2 недель"?

F. Языковой барьер

  1. PR #846 Comment #10 (alignment) — Иван задал вопрос на ломаном английском, Michael не понял, потеряли цикл (~3 дня).
    • Должен ли менеджер/тимлид перечитывать существенные комментарии Ивана к ревьюерам клиента перед публикацией?

G. Невыученные уроки между PR

  1. VideoObject без uploadDate — пойман в PR #832 и PR #842 в один день (31.03). Между PR урок не закрепился.
    • Согласен ввести lessons learned log (после каждого замержённого PR — 1-2 повторяемых замечания)?
    • Кто бы это вёл — ты или Иван?

H. Знание codebase

  1. PR #832 — Иван не использовал getExternalPath, Image Wrapper.credit (существующие абстракции).
    • Делается ли у нас онбординг по архитектуре проекта? Карта компонентов / utilities?
    • Нужна ли практика "перед фичей просмотреть похожие места в codebase"?

I. AI-tooling (вопрос из письма Alexis)

  1. Используем ли мы какие-то AI-инструменты при разработке (Copilot, Cursor, Codex)?

  2. Что думаешь про предложение Alexis "AI-assisted tooling has helped us during our internal experiments" — стоит ли пилотить?

J. Ответ Stevens — тон и стратегия

  1. Какой тон ответа выбираем — оборонительный, согласительный, со встречными предложениями?

  2. Что нельзя признать в ответе клиенту? (Например, может быть юридически или коммерчески чувствительно.)

  3. Что хотим сами получить от Stevens в ответе? (Например, ясность по спеке, регулярные sync'и, чёткий escalation channel.)

K. После встречи

  1. Кто пишет первый черновик ответа Tom/Stevens?
  2. К какому сроку нужно отправить ответ?
  3. Распределение работ по существующим открытым PR (#832, #842, #846, #847) — как двигаем дальше?