Stop using nil internally to signal "commit not found"

Through an accident of history, our internal commit lookup method log.GetCommit returns nil when the commit cannot be found. I think this is an unergonomic interface. It forces us to do nil checks in addition to err != nil.