Improve Seq and remove newline from non-min empty in Immutable plugin

[pedrottimark]

Summary

• enclose entries of keyed Seq in braces
• enclose values of non-keyed Seq in brackets (not distinguishing indexed/non-indexed)
• represent lazy Seq by ellipsis

This was a brain-buster to write and rewrite. Again, many thanks to y’all who have reviewed.

• added printImmutableSeq to detect private properties and call public iterator methods unless collection is lazy
• deleted length/size exception (that is, newline within brackets/braces in non-minified empty collection) because although I sympathize with it, now that the plugin calls same helper functions as pretty-format core:
  • the code becomes too complicated to understand and maintain
  • although notation is diff-friendly for empty versus non-empty, the inconsistency can be diff-unfriendly if you convert from an empty ECMAScript to corresponding Immutable collection
• sorted SENTINEL constants
• added printAsLeaf helper function

Test plan

• Changed criterion for correctness in 8 existing tests of empty collections {min: false}
• Updated all tests of Seq that were added in Replace print with serialize in Immutable plugins #4189
• Added 1 test for maxDepth of Seq
• Added 6 tests for lazy Seq

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.