moonbitlang / core / 4145

94%

push

github

refactor(argparse): Map literal + spread + comprehension in build_*_index

`build_long_index`: replace `let index : Map[K,V] = {}` with `Map([])`
and `globals.iter() + args.iter()` with the array spread `[..globals, ..args]`.

`build_short_index`: collapse the two-loop population into a single Map
comprehension `Map([for arg in [..globals, ..args] if ... => (key, arg)])`.

Both transforms walk the same elements once. The spread allocates a
small temporary array (bounded by the global+local arg count, typically
<20 in CLI use), which is acceptable for argparse's non-hot-path
invocation pattern.

Semantics-preserving: `moon test -p argparse` (225/225) passes and
`moon info` produces no `.mbti` change.

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

2 of 2 new or added lines in 1 file covered. (100.0%)

14879 of 15737 relevant lines covered (94.55%)

218055.67 hits per line