Consider moving hoisted dependencies into the controller actions

[matthewmueller]

Right now you're able to do something like this:

func Load(session *Session, pg *postgres.Client) *Controller {
  return &Controller{w, pg}
}

type Controller struct {
  pg *postgres.Client
  session *session.Session
}

func (c *Controller) Index() {}

And Bud is smart enough to initialize *postgres.Client once on boot and *session.Session per request. This was done to keep the function signature of the actions clean and focused on the inputs and outputs.

Suppose Load was an expensive operation though. It would be very surprising to learn that Load is called every request. I'm now leaning towards something like this:

func (c *Controller) Delete(session *session.Session) error { 
  return session.Logout() 
}

And then separating inputs from dependencies.

See for more discussion: #185 (comment)