Какой вариант написания лучше?
2й читается лучше на мой взгляд, а так разницы никакой. 

Разница лишь в возможностях обоих подходов.

Во 2м можно написать некую свою логику до return и создать или заменить исходное значение свойства на новое.

Как пример:
const mapStateToProps = ({ user, data }) => {
  const myProp = localStorage.get('myProp');

  return {
    myProp,
    user,
    data
  }
};

Исходя из вашего 1го примера, его можно ещё короче записать.
const mapStateToProps = props => props;

Какой только смысл в этом!!! Так вообще не читается, что внутри props лежит, фиг его знает... Зато коротко и экономит несколько байт
Лучше для чего? )
Я бы использовал первый вариант