🟡 pokemon-tamagotchi: i18n — гонка синхронного translate по ленивому scope, осиротевшие ключи, дублированная эвристика ключа

[JsPowWow]

Почему создана

Аудит фичи pokemon-tamagotchi. Найдены проблемы i18n/Transloco: осиротевшие и мёртво-дублирующие ключи, синхронный translate() по лениво загружаемому scope (гонка) и хрупкая эвристика «строка с точкой = ключ перевода», продублированная в двух местах. Все подтверждены двумя верификаторами.

Проблематика

🟡 1. Синхронный translate() по лениво загружаемому scope в root-сервисе уведомлений

TamagotchiNotificationService (@Injectable({ providedIn:'root' })) переводит ключи тостов синхронно: this.transloco.translate(\${NOTIFICATION_SCOPE}.${key}`)(notification.service.ts:83-85). ScopepokemonTamagotchi — **лениво загружаемый** (provideTranslocoScopeв routes.ts:17 + HTTP-фетч JSON при инициализации директивы*transloco). Но публикация уведомлений запускается эффектами фасада независимо от готовности scope: эффект «evolution ready» (facade.ts:136-146) срабатывает сразу после bootstrapFromProfile, если восстановленное состояние уже удовлетворяет критериям; первый тик таймера (facade.ts:284) может принести alerts после офлайн-decay. Если JSON scope ещё не доехал, translate()возвращает **сам ключ** — Taiga-тост покажетpokemonTamagotchi.notifications.evolution.readyTitle` вместо текста. То же окно — при каждом переключении языка.

Нюанс: само отступление от паттерна «директива в шаблоне» здесь оправдано — тостам через AppNotificationService нужна готовая строка, а история в шаблоне корректно ре-переводится через t(). Проблема именно в синхронности до загрузки scope.

🫥 2. Осиротевшие i18n-ключи: disabledSleeping и мёртвая ветка alerts.evolutionReady

• actions.disabledSleeping («Pokémon is sleeping») есть в обоих JSON, но не упоминается нигде в src: когда покемон спит, awake-кнопки не рендерятся, а кнопке sleep/wakeUp назначается только disabledTraining.
• notifications.alerts.evolutionReady.title/message (en/ru.json:89-92) достижимы только через ALERT_TEMPLATES.evolutionReady, но ни один прод-путь не порождает StatusAlertType 'evolutionReady': STAT_ALERT_CONFIG покрывает только hunger/mood/energy/hydration, а единственный метод, способный доставить произвольный alert (notifyStatusAlerts), зовётся только из спека. Живой путь «готов к эволюции» идёт через notifyEvolutionReady → ключ notifications.evolution.readyTitle. Итог: две параллельные пары переводов одного события, одна мёртвая — мусор в обеих локалях и ловушка для переводчика.

🫥 3. Хрупкая эвристика «содержит точку = ключ перевода», продублированная в двух местах

NotificationModel.title/message несут либо i18n-ключ, либо сырой текст (имя покемона в notificationFromEvolutionReady), а различаются эвристикой value.includes('.'), скопированной в два независимых места: resolveText (notification.service.ts:75-81) и isTranslationKey (notification.component.ts:24-26). Контракт нигде не зафиксирован типом: любое будущее уведомление с сырым текстом, содержащим точку («Progress saved.», имя «Mr. Mime»), молча уйдёт в transloco.translate — в истории пользователь увидит pokemonTamagotchi.notifications.Progress saved.. Ошибка не ловится компилятором/линтом, чинить придётся в двух файлах.

Где

• src/app/features/pokemon-tamagotchi/ui/services/notification.service.ts:84 — синхронный translate()
• src/app/features/pokemon-tamagotchi/data/facades/tamagotchi.facade.ts:141 — публикация до готовности scope (evolution-ready)
• src/app/features/pokemon-tamagotchi/data/facades/tamagotchi.facade.ts:284 — публикация на первом тике (alerts)
• public/i18n/pokemonTamagotchi/en.json:47 — осиротевший actions.disabledSleeping
• public/i18n/pokemonTamagotchi/ru.json:47 — то же
• public/i18n/pokemonTamagotchi/en.json:89 — мёртвая ветка alerts.evolutionReady
• public/i18n/pokemonTamagotchi/ru.json:89 — то же
• src/app/features/pokemon-tamagotchi/data/helpers/notification-factory.helper.ts:24 — ALERT_TEMPLATES.evolutionReady
• src/app/features/pokemon-tamagotchi/ui/services/notification.service.ts:76 — эвристика .includes('.') (копия 1)
• src/app/features/pokemon-tamagotchi/ui/components/notifications/notification.component.ts:25 — эвристика .includes('.') (копия 2)

Варианты решения

1. Асинхронный перевод тостов (баг 1). Заменить синхронный translate на selectTranslate(key, params, 'pokemonTamagotchi') (ждёт загрузки scope; первую эмиссию через firstValueFrom/take(1)) и показывать тост с готовой строкой. Альтернатива — публиковать тосты только после transloco.events$ (scope loaded).
2. Убрать осиротевшие ключи (баг 2). Удалить actions.disabledSleeping, блок alerts.evolutionReady из en/ru.json, запись evolutionReady из ALERT_TEMPLATES, член 'evolutionReady' из StatusAlertType и метод notifyStatusAlerts. Либо реально задействовать (показывать disabledSleeping как hint у wakeUp, слать evolutionReady через processStatusAlerts) — но тогда убрать дубль notifications.evolution.readyTitle.
3. Зафиксировать контракт в модели (баг 3). Разделить поля (titleKey: string | null + titleText: string | null, либо дискриминатор isTranslated: boolean в NotificationModel) и убрать обе эвристики — сервис и компонент читают явный признак.

Рекомендация

Баг 1 — самый важный (user-facing: сырой ключ в тосте при офлайн-возврате и смене языка), вариант 1 (selectTranslate). Баг 3 — устраняет корень (угадывание по точке) и заодно снимает дублирование — вариант 3, делать вместе с багом 2 (оба про модель уведомлений). Баг 2 — вычистить мусор из локалей (вариант «удалить», т.к. живой путь эволюции уже работает через отдельный ключ). Все правки в data/ui-слое уведомлений + JSON.

Правила: CLAUDE.md (i18n — Transloco), скилл angular-best-practices-transloco (ленивые scope, selectTranslate); баги 2-3 — консистентность/устойчивость.