[camgunz]

Shouldn't you have some shared context when working on code together, even at different times? Maybe they terms of art, maybe it's a style guide, but there oughta be something.

Like, isn't it fine to change:

    var basicAuthenticationController = new BasicAuthenticationController()

to:

    var ctrl = new BasicAuthenticationController()

I find that if code authors do work upfront to contextualize things for me that it helps tremendously. Like do I need to know at every point that this is a BasicAuthenticationController? Or is this the basic auth module and we're always dealing w that controller? I prefer it when engineers set the table for me like that, it helps me narrow my focus to the purpose of the code.