Added in-memory caching for module artifact set
Now that we cache module artifact sets, can fail when offline
Remove a few stale/obvious TODOs
REVIEW-4447: Don't mention changes to stuff that we don't consider 'public'