WellNuo/REVIEW_REPORT.md

Code Review Report — WellNuo Push Notifications System

Date: 2026-01-26 Reviewers: Claude Code Workers: @worker1 (Backend), @worker2 (Mobile App) Status: Partial Completion

---

Executive Summary

Параллельное выполнение задач завершено с частичным успехом. Backend часть выполнена хорошо, Mobile App часть выполнена частично — отсутствует ключевой компонент expo-notifications.

Оценка качества: 6/10

---

Task Completion Status

@worker1 — Backend Tasks

Task	Status	Comments
TASK-1: Улучшить sendPushNotifications с проверкой настроек	✅ Done	Отличная реализация в notifications.js
TASK-2: notification_history таблица и логирование	✅ Done	Миграция + сервис созданы
TASK-3: API для получения истории алертов	✅ Done	/api/notification-settings/history работает
TASK-4: Деплой backend изменений	⚠️ Not Verified	Нужен ручной деплой на сервер

@worker2 — Mobile App Tasks

Task	Status	Comments
TASK-5: Установить expo-notifications	❌ Not Done	Пакет НЕ установлен в package.json
TASK-6: Создать сервис pushNotifications.ts	❌ Not Done	Файл отсутствует
TASK-7: Интеграция при логине	❌ Not Done	Зависит от TASK-5/6
TASK-8: Обработка входящих push уведомлений	❌ Not Done	Зависит от TASK-5/6
TASK-9: UI настроек уведомлений	✅ Done	notifications.tsx создан и работает

---

Code Quality Analysis

Backend Code (backend/src/services/notifications.js)

Strengths:

1. ✅ Хорошая структура — модульный подход с отдельными функциями
2. ✅ Проверка notification settings перед отправкой
3. ✅ Quiet hours поддержка с учётом overnight периодов (22:00-07:00)
4. ✅ Emergency alerts обходят quiet hours — правильно
5. ✅ Batch отправка (chunks по 100) — согласно рекомендациям Expo
6. ✅ Логирование в notification_history для всех статусов (sent/skipped/failed)
7. ✅ Валидация Expo Push Token формата

Potential Issues:

1. ⚠️ isInQuietHours использует timezone = 'UTC' по умолчанию, но пользователь может быть в другом timezone. Timezone не передаётся из settings.
2. ⚠️ getUserPushTokens фильтрует по is_active = true, но нигде не видно логики деактивации токенов при ошибках.

Backend Code (backend/src/routes/notification-settings.js)

Strengths:

1. ✅ Правильный маппинг snake_case (DB) → camelCase (API)
2. ✅ Upsert для settings — работает корректно
3. ✅ Pagination с лимитом (max 100) в history endpoint
4. ✅ Фильтрация по type/status в history

Potential Issues:

1. ⚠️ Нет input validation — quietStart/quietEnd могут быть любыми строками

Backend Code (backend/src/services/mqtt.js)

Issues Found:

1. ❌ НЕ использует новый notifications.js сервис! Функция sendPushNotifications в mqtt.js — старая версия, которая НЕ проверяет notification_settings
2. ❌ Дублирование кода — есть две версии sendPushNotifications:
   • mqtt.js:216 — старая, без проверки настроек
   • notifications.js:294 — новая, с полной логикой

Migration (010_create_notification_history.sql)

Strengths:

1. ✅ Все необходимые поля (user_id, type, status, skip_reason, etc.)
2. ✅ Индексы для частых запросов
3. ✅ CHECK constraint на status
4. ✅ Комментарии на колонки

Mobile App (types/index.ts)

Strengths:

1. ✅ Типы NotificationSettings, NotificationHistoryItem, NotificationHistoryResponse корректны
2. ✅ Соответствуют API response формату

Mobile App (services/api.ts)

Strengths:

1. ✅ getNotificationSettings() — корректная реализация
2. ✅ updateNotificationSettings() — работает
3. ✅ getNotificationHistory() — с фильтрами и pagination

Mobile App (app/(tabs)/profile/notifications.tsx)

Strengths:

1. ✅ Хороший UI с Switch компонентами
2. ✅ Загрузка настроек с сервера
3. ✅ Сохранение через API
4. ✅ Quiet hours toggle с отображением времени

Issues:

1. ⚠️ Time picker для quiet hours — заглушка "Coming Soon"
2. ⚠️ "Send Test Notification" — только Alert, не отправляет реальный push

---

Security Analysis

Positive:

1. ✅ JWT authentication на всех endpoints
2. ✅ Expo Push Token validation
3. ✅ Admin-only endpoints в mqtt.js

Concerns:

1. ⚠️ quietStart/quietEnd не валидируются как HH:MM формат
2. ⚠️ MQTT credentials хардкодены в коде (хоть и через env)

---

Critical Missing Components

1. expo-notifications Package

# Ожидалось в package.json:
"expo-notifications": "~0.31.0"  # НЕ НАЙДЕНО!

2. pushNotifications.ts Service

Файл services/pushNotifications.ts не создан. Должен содержать:

• registerForPushNotificationsAsync()
• registerTokenOnServer(token)
• unregisterToken()

3. MQTT → Notifications Integration

mqtt.js НЕ интегрирован с новым notifications.js:

// mqtt.js:140 — ТЕКУЩИЙ КОД (неправильно):
await sendPushNotifications(alert);  // Локальная функция без проверки settings

// ДОЛЖНО БЫТЬ:
const { sendPushNotifications } = require('./notifications');
await sendPushNotifications({
  userIds: users.map(u => u.user_id),
  title: `Alert: ${beneficiaryName}`,
  body: alert.body,
  type: 'emergency', // или определять по содержимому
  beneficiaryId: alert.beneficiaryId,
});

---

Build & TypeScript Check

✅ TypeScript: No errors in main WellNuo project
❌ npm run build: Script not defined (expected — Expo project)

---

Recommendations

Critical (Must Fix):

1. Установить expo-notifications:

   npx expo install expo-notifications
   

2. Создать services/pushNotifications.ts:

   • Реализовать регистрацию push токена
   • Интегрировать в login flow

3. Интегрировать mqtt.js с notifications.js:

   • Заменить локальную sendPushNotifications на импорт из notifications.js
   • Определять тип алерта (emergency/activity/low_battery) по содержимому

Important:

4. Добавить input validation для quiet hours:

   const timeRegex = /^([01]\d|2[0-3]):([0-5]\d)$/;
   if (quietStart && !timeRegex.test(quietStart)) {
     return res.status(400).json({ error: 'Invalid quietStart format' });
   }
   

5. Добавить timezone в notification_settings таблицу:

   • Quiet hours должны работать в timezone пользователя

6. Деплой backend на сервер:

   rsync -avz backend/ root@91.98.205.156:/var/www/wellnuo-api/
   ssh root@91.98.205.156 "cd /var/www/wellnuo-api && npm install && pm2 restart wellnuo-api"
   

Nice to Have:

7. Реализовать time picker для quiet hours (вместо заглушки)
8. Добавить логику деактивации push токенов при ошибках доставки

---

Test Checklist

Test	Status
Push токен регистрируется при логине	❌ Не реализовано
MQTT алерты сохраняются в mqtt_alerts	✅ Работает
Push отправляется с учётом настроек	⚠️ Код есть, но не интегрирован
Notification history записывается	✅ Работает
UI настроек работает	✅ Работает
TypeScript без ошибок	✅ Проходит

---

Files Changed

File	Worker	Status	Quality
backend/migrations/010_create_notification_history.sql	@worker1	✅ Created	Good
backend/src/routes/notification-settings.js	@worker1	✅ Modified	Good
backend/src/services/notifications.js	@worker1	✅ Created	Excellent
services/api.ts	@worker2	✅ Modified	Good
types/index.ts	@worker2	✅ Modified	Good
app/(tabs)/profile/notifications.tsx	@worker2	✅ Exists	Good
services/pushNotifications.ts	@worker2	❌ Missing	—

---

Final Score

Category	Score	Max	Notes
Task Completion	5	10	5/9 tasks done
Code Quality	8	10	Clean, well-structured
TypeScript	10	10	No errors
Security	8	10	Good auth, minor validation gaps
Integration	4	10	mqtt.js not connected to notifications.js

Total: 6/10

---

Conclusion

Backend часть (@worker1) выполнена качественно — создан полноценный notification service с проверкой настроек, quiet hours, и логированием истории.

Mobile App часть (@worker2) выполнена частично — UI настроек работает, но ключевой функционал регистрации push токенов отсутствует.

Главная проблема: mqtt.js использует старую локальную версию sendPushNotifications вместо нового сервиса notifications.js. Это означает, что настройки уведомлений НЕ применяются к реальным MQTT алертам.

Для достижения минимального балла 8/10 необходимо:

1. Установить expo-notifications
2. Создать pushNotifications.ts сервис
3. Интегрировать mqtt.js с notifications.js
4. Задеплоить backend на сервер