2й читается лучше на мой взгляд, а так разницы никакой.
Разница лишь в возможностях обоих подходов.
Во 2м можно написать некую свою логику до return и создать или заменить исходное значение свойства на новое.
Как пример:
const mapStateToProps = ({ user, data }) => {
const myProp = localStorage.get('myProp');
return {
myProp,
user,
data
}
};
Исходя из вашего 1го примера, его можно ещё короче записать.
const mapStateToProps = props => props;
Какой только смысл в этом!!! Так вообще не читается, что внутри props лежит, фиг его знает... Зато коротко и экономит несколько байт