I don't think you should modify the definition of modules that are declared in other files. The need to do so arose from collapsing the distinction between module registration and module loading. 'targets' specifies the module's suitability for mobile; it is not tantamount to loading it. The suitability ought to be an immutable fact and distinct from the act of loading modules, IMO.

This stems from a paranoia we seem to have to have control of other modules outside our control (we do something similar for CentralNotice)

Let's kill the code as Ori says. We should have more trust (and do same for Central Notice)

https://gerrit.wikimedia.org/r/73434