Fixup this refactoring

Discovered that *args weren't being sent correctly from
the main class that uses cinch to the actions. Pretty simple
mistake, but still I'm a little disappointed that I didn't
catch this earlier.
Remove calls to weird User() method

Cinch must define this as a class method, because it wasn't
available to the action classes that I setup. It seems unnecessary
because you can also just call #user on the message object that you
get, and it gives you the identical object.