Еще как плохая ! Наследуйся от AndroidViewModel, там есть указатель на Application (context). ViewModel вообще ничего не должна знать о View и т.д.
Вот да, меня смутило, что viewModel получает контекст от активити, хотя по идее она от вьюхи никаких ссылок не должна получать. Я просто смотрел пример один, где mvvm реализован в приложении с несколькими Service и был удивлён открытием...