Да, не стоит стучаться в сабджект, который в сервисе, его вообще приватные сделать и публичное поле добавить типа subject.asObservable()
Если же нужно менять что то в сабджекте, то сделать метод сервиса отдельный, который будет уже в сабджект данные писать и, возможно, проверки выполнять какие то