[Bug] Trailing commas are being removed from typedef declarations.

[alejandro-all-win-software]

If the bug is about the formatter producing output you don't like, please use the "Formatting issue" template.

Describe the issue

I have the trailing_commas option set to preserve, so I expect any commas I add to remain intact. However, when I try to add a comma to a typedef, regardless of its length, the comma is removed.

To Reproduce

For example, I have the following typedef: when I add a comma after url, it gets removed.

typedef RawPhoto = ({DateTime addedAt, String name, String url});

Expected behavior

I expect that when using the trailing_commas option set to preserve, no commas should be removed.

[androidseb]

EDITED: As requested here, moved this extra bug report to a separate new issue here.

Original message

I have another similar case to report, so I was thinking we could rename this issue's title to something like "the trailing_commas option being set to preserve is ignored in some cases" - the cases being the typedef showcased above, and enums with constructors (maybe there are more). Please let me know if I should create a separate issue instead, thanks.

Staying in the same theme as this initial bug report, here is an example of an enum declaration showcasing the same bug (i.e. comma removed when it shouldn't):

enum RawPhotoType {
  jpeg('image/jpeg'),
  png('image/png'),
  ;

  final String mimeType;
  const RawPhotoType(this.mimeType);
}

When formatted, the above code has its comma removed at the end of the png line, resulting in this:

enum RawPhotoType {
  jpeg('image/jpeg'),
  png('image/png');

  final String mimeType;
  const RawPhotoType(this.mimeType);
}

[munificent]

Please let me know if I should create a separate issue instead, thanks.

Filing separate issues will definitely help ensure that something doesn't slip through the cracks. Thanks. :)