Fix misplaced comma in auto imported specifier

[andrewbranch]

Fixes #40219

Note: insertNodeAfter and insertNodeAtEndOfList are pretty dumb, and should probably be deleted, because insertNodeInLastAfter is very smart.

[andrewbranch]

I think this would be a good candidate for VS Code's next servicing release.

[DanielRosenwasser]

@typescript-bot cherry-pick this to release-4.0

[typescript-bot]

Heya @DanielRosenwasser, I've started to run the task to cherry-pick this into release-4.0 on this PR at 2c0e01a. You can monitor the build here.

[DanielRosenwasser]

Note: insertNodeAfter and insertNodeAtEndOfList are pretty dumb, and should probably be deleted, because insertNodeInLastAfter is very smart.

Can you explain a bit more here?

[typescript-bot]

Hey @DanielRosenwasser, I've opened #40303 for you.

[andrewbranch]

@DanielRosenwasser insertNodeInListAfter has a bunch of quite complex logic where it examines the previous list item to deduce how the list is formatted (i.e., what the separator is, and where line breaks and spaces should go), whereas insertNodeAtEndOfList just blindly inserts ", " + yourListItemText. But their names don’t suggest this discrepancy—insertNodeAtEndOfList just sounds like a special case of insertNodeInListAfter—which is how I made the mistake of using the dumb one in the first place.