moonbitlang / core / 4908

91%

push

github

feat(immut/sorted_map): add SortedMap constructor, deprecate from_array

Mirror the `List([...])` constructor convention for the immutable sorted
map: add `SortedMap::SortedMap(ArrayView[(K, V)]) -> SortedMap[K, V]` so a map
can be built with `@sorted_map.SortedMap([...])`, and deprecate the old
`from_array` (and its `of` alias) in favour of it.

`SortedMap` is an `enum` (`Empty` / `Tree`), so there is no constructor-name
conflict. Migrate all `@sorted_map.from_array(...)` / `from_array(...)` usages
across the package (impl, tests, README) to the new constructor. The
`Show`/`output` format string and its snapshot tests intentionally keep
emitting `@immut/sorted_map.from_array([...])`.

Found via a repo-wide scan for collections that have `from_array` but no
`Type([...])` constructor.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

5 of 5 new or added lines in 2 files covered. (100.0%)

15341 of 16800 relevant lines covered (91.32%)

195679.45 hits per line