Object pattern formatting

[MichaReiser]

Rome tries to fit all object pattern members on a single line. Prettier uses a more sophisticated approach to determine if the pattern should be formatted on a single line or broken across multiple ones. Playground

A few examples:

Input

// Break over multiple lines if it contains a sub pattern
const {b: {test}} = test;

// non-shorthand and whole statement exceeds line width
const { id, isStatic: isStatic, method, methodId, g } = privateNamesMap.get(name);

// default and whole statement exceeds line width
const { id, isStatic = true, method, methodId, gee } = privateNamesMap.get(name);

// sub pattern
const { id, isStatic: {sub}, method, methodId, gee } = privateNamesMap;

Prettier

// Break over multiple lines if it contains a sub pattern
const {
  b: { test },
} = test;

// non-shorthand and whole statement exceeds line width
const {
  id,
  isStatic: isStatic,
  method,
  methodId,
  g,
} = privateNamesMap.get(name);

// default and whole statement exceeds line width
const {
  id,
  isStatic = true,
  method,
  methodId,
  gee,
} = privateNamesMap.get(name);

// sub pattern
const {
  id,
  isStatic: { sub },
  method,
  methodId,
  gee,
} = privateNamesMap;

Rome

// Break over multiple lines if it contains a sub pattern
const { b: { test } } = test;

// non-shorthand and whole statement exceeds line width
const { id, isStatic: isStatic, method, methodId, g } = privateNamesMap.get(
  name,
);

// default and whole statement exceeds line width
const { id, isStatic = true, method, methodId, gee } = privateNamesMap.get(name);

// sub pattern
const { id, isStatic: { sub }, method, methodId, gee } = privateNamesMap;

Expected

Rome's formatting to match Prettier's

[ematipico]

I will look after this one

[ematipico]

Currently looking at implement this for type alias parameters

[ematipico]

Currently looking at class members