so should this be `copyOf(subscriptions)` rather than `copyOf(getSubscriptions())' to ensure the safety?

(or if they are guaranteed to be the same we should just use `getSubscriptions()` everywhere and not introduce `subscriptions` right?)