Hey Ian I'm glad you like it!

That is strictly true, you don't need to split out the require calls. The nice thing about your PR is that it reduces the amount of lines in general, which is always the clean way of doing things. The only disadvantage is that it tends to not be the way you and I would normally do module imports in production, as you'll probably need the required modules again later on. Having said all that, I'm pretty comfortable bringing this in, but I'll give others a day or two to chime in and see what they think!

FWIW, Node.js docs use this type of ad hoc `require'. For example:
https://nodejs.org/api/child_process.html#child_process_example_sending_a_server_object
https://nodejs.org/api/child_process.html#child_process_example_sending_a_socket_object
https://nodejs.org/api/crypto.html#crypto_certificate_exportchallenge_spkac
https://nodejs.org/api/cluster.html#cluster_cluster

What this all about

Thanks Ian for your help!