Yeah, if the intent is really "not read-only", then I agree we should make it a property of the modulestore. But at least from the comments in these files, it's not entirely clear to me. I think we could introduce that abstraction in the future.