Conversation
6f91a92 to
429f4ea
Compare
4f7eb07 to
7b15440
Compare
SeanSilke
left a comment
There was a problem hiding this comment.
Необходимые изменения (обновленный план)
-
Выровнять поведение ручной и авто-остановки
WebTelemetryResources:
вend()добавить дочитывание очереди observer (takeRecords()), обработку черезprocessResources(...), затем финальныйflushBufferedEvents()иdisconnect(). -
Убрать расхождение shutdown-путей:
выделить единый приватный shutdown-метод и использовать его и вend(), и вfinalizeAfterDocumentReady(). -
Синхронизировать async-поведение завершения:
так какprocessResources(...)асинхронный, сделать путь остановки явно консистентным (предпочтительно через единый async shutdown с централизованной обработкой ошибок). -
Переименовать флаг для явной области действия:
сobserveAfterLoadна имя, которое явно говорит про ресурсы (рекомендуемоobserveResourcesAfterLoad), чтобы исключить трактовку как “глобальный” флаг всей телеметрии. -
Обновить тесты под финальную семантику:
- продолжение сбора ресурсов после
loadприobserveResourcesAfterLoad: true; - корректная финальная отправка при ручном
end(); - отсутствие потерь записей из внутренней очереди observer при остановке;
- продолжение сбора ресурсов после
📦 Published PR as canary version:
2.0.18--canary.44.24510571290.0✨ Test out this PR locally via:
npm install @salutejs/web-telemetry@2.0.18--canary.44.24510571290.0 # or yarn add @salutejs/web-telemetry@2.0.18--canary.44.24510571290.0