Files

T

Дмитрий Соловьев 294b17757d pcp: время — сквозной naive LocalDateTime как локальное настенное время (без UTC)

Отказ от остаточной UTC-семантики: naive LocalDateTime трактуется как
локальное время единой зоны всех сервисов.

Backend (границы внешних систем):
- VisibilityPayloadParser, SyncSpacecraftFromNsiUseCase: ZoneOffset.UTC →
  ZoneId.systemDefault() для проекции внешних Instant/ISO-с-Z в naive.
- тесты границ перевёрнуты на systemDefault (под Europe/Moscow ждут +3).
- чистка UTC-формулировок в DTO-комментариях, мёртвых ссылок на
  docs/standards/DATETIME_UTC.md.

Frontend (pcp-tgu-ops-ui):
- utils/utcTime.ts → utils/localTime.ts: parseLocal/formatLocal*/toLocalIso/
  toLocalInputValue с локальной семантикой (без дописывания Z и timeZone:UTC).
- замена getUTC*/Date.UTC/toISOString-сериализации на локальные аналоги.
- убраны пользовательские подписи «UTC» в редакторе/таймлайне.
- удалён utcTime.guard.test.ts (форсил обратный UTC-инвариант); тест-фикстуры
  сделаны зоно-независимыми. tsc + vitest зелёные под TZ=UTC и TZ=Europe/Moscow.

Включает также прочие незакоммиченные правки рабочего дерева.

2026-06-04 09:50:39 +03:00

5.9 KiB

Raw Blame History

CHECKLIST.md

Назначение

Этот документ содержит краткие checklists для review/merge, Definition of Done, обязательных тестов и validation.

Пункты ниже применяются по принципу where relevant:

для обычных Kotlin/Spring-изменений достаточно общего checklist;
для изменений в BPM / Zeebe / Camunda / Kafka / retries / DLQ / compensation / schema evolution дополнительно применяются профильные пункты;
для критичных процессов глубина проверок и тестов должна быть пропорциональна риску изменения.

Review checklist

Для всех нетривиальных изменений

Перед merge необходимо подтвердить:

запрошенное поведение реализовано;
решение осталось в корректной архитектурной границе;
существующие публичные контракты не сломаны без явного запроса;
добавлены или обновлены релевантные тесты;
не внесено несвязанных правок;
время на API — naive LocalDateTime без зоны; backend штампует now через LocalDateTime.now(), фронт парсит/форматирует через utils/utcTime без new Date("<строка>");
egress фронта к бэку — naive ISO (YYYY-MM-DDTHH:mm:ss, без Z);
validation выполнен по правилам AGENTS.md, либо пропуски явно объяснены.

Дополнительно для BPM / Kafka / Zeebe / external integration изменений

Перед merge необходимо подтвердить, где это релевантно:

business key определён и используется консистентно;
operation id принимается от вызывающей стороны и используется для dedup;
ошибки классифицированы семантически;
side effects идемпотентны или явно компенсируются;
доменные события публикуются через Outbox;
logs / metrics / traces содержат обязательные поля корреляции;
тесты покрывают duplicate delivery и retry behavior;
изменения деплоя и конфигурации трассируемы в VCS.

Definition of Done

Задача считается завершённой только если одновременно выполнено всё ниже, где это релевантно:

Общее

Запрошенное поведение реализовано.
Решение осталось в корректной архитектурной границе.
Нужные тесты добавлены или обновлены.
Нужный validation выполнен, либо невозможность запуска объяснена.
Не внесено несвязанных правок.

Для BPM / Kafka / Zeebe / external integration

BPM остаётся только оркестратором, если BPM участвует.
Доменные правила остаются внутри сервисной границы.
Flows остаются безопасными при at-least-once delivery.
Идемпотентность обеспечена.
operation_id сохранён или добавлен, где это требуется контрактом.
Outbox сохранён или использован, где операция меняет состояние и публикует события.
Retries ограничены и управляемы.
DLQ handling сохранён для критичных consumers.
Versioning / backward compatibility сохранены.
Observability сохранена или улучшена.

Обязательные тесты для значимых изменений

Базовый минимум

Как минимум покрывать, где применимо:

happy path;
изменённые ветки branching / validation logic;
regression scenario для bug fix.

Дополнительно для BPM / Kafka / Zeebe / external integration

Покрывать, где применимо:

retryable failure с последующим успехом;
исчерпание retries;
ветку business error;
обработку timeout;
сценарий compensation;
duplicate delivery / duplicate event handling;
contract compatibility;
schema compatibility.

Для критичных процессов и high-risk изменений

Покрывать пропорционально риску изменения через automated tests и/или явный resilience test plan:

перезапуск воркера во время выполнения;
временную недоступность БД;
временную недоступность внешней зависимости;
разрыв связности Kafka / BPM;
сценарий load / lag recovery.

Validation

Следуй разделу Validation commands из AGENTS.md.

Если для изменения есть module-specific команды, используй минимальный релевантный прогон, а затем более широкий, когда это необходимо.

Если какая-либо команда не была запущена:

точно укажи, какая именно команда не запускалась;
объясни почему;
опиши остаточный риск.

5.9 KiB Raw Blame History