Possible memory leak в eventemitter из-за подписки на `theme.change`

[ItMaga]

Проблема

Если полазить по разным страницам с включенным keepAlive, то в какой-то момент стреляет ошибка Possible memory leak из eventemitter, связано с тем, что мы подписываемся в каждом компоненте на изменение темы для синхронизации модификатора.

Происходит подписка здесь

Client/src/components/super/i-block/state/index.ts

Lines 645 to 646 in c239c64

	this.sync.mod('theme', 'remoteState.theme.emitter:theme.change', {immediate: true}, (theme?: Theme) =>
	(theme ?? this.remoteState.theme.get()).value);

Тут нужно уточнить у @kobezzza, по какой причине мы не могли отказать от слушателя, Артем говорил, что подобный вопрос уже поднимался.

Возможные решения

1. Использовать глобальный селектор на теге html, для определения текущей темы. Если есть необходимость в модификаторе на отдельном компоненте, то добавить возможность точечной синхронизации темы с локальным модификатор компонента.
2. Использовать sharedMods?

[ItMaga]

@kobezzza Подскажешь тут, пожалуйста, помнишь почему мы не могли отказаться от слушателя в каждом компоненте? Что скажешь по возможным решениям?

[kobezzza]

Тут не утечка, т.к. при смерти компонента все равно память очиститься.

1. Это нарушает инкапсуляцию и потребует использовать каскад.

2. Вариант с sharedMods не подходит, из-за того, что сущность определяется 3-й стороной, а не самим компонентом. Да и работать оно будет хуже.

НО, на самом деле есть правильное решение.

Когда компонент деактивируется, то его эвент-эмиттеры ставятся в mute (async). Сейчас асинк просто не пускает эти события дальше по цепочке, а надо сделать, чтобы mute реально отменял событие и потом его восстанавливал.