Compact arguments with object literal

[andrey-shostik]

Prettier 1.14.3
Playground link

Input:

export default handleAction({
    ['MOCK_ACTION']: state => ({ ...state, counter: state.counter + 1 })
}, initialState.mockReducer);

Output:

export default handleAction(
  {
    ["MOCK_ACTION"]: state => ({ ...state, counter: state.counter + 1 })
  },
  initialState.mockReducer
);

Expected behavior:
Input shouldn't be changed. I don't like output format, its takes more lines and looks worse for me.

[nmussy]

It seems this would be solved by the --pure option (#2068), as Prettier respected this hard line break:

export default handleAction({
                             ^

Without any line breaks, there is a cleaner output:

Input:

export default handleAction({['MOCK_ACTION']: state => ({ ...state, counter: state.counter + 1 })}, initialState.mockReducer);

Output:

export default handleAction(
  { ["MOCK_ACTION"]: state => ({ ...state, counter: state.counter + 1 }) },
  initialState.mockReducer
);

Which is consistent with other function calls with long arguments:

Input:

export default handleAction(justACoupleOfArgumentsButWithLongContents, andAnotherLongVariableName);

Output:

export default handleAction(
  justACoupleOfArgumentsButWithLongContents,
  andAnotherLongVariableName
);