moonbitlang / core / 5554

91%

push

github

refactor(debug): deprecate free `to_repr`, migrate call sites to `Repr(x)`

Follows the `Repr(x)` constructor: `to_repr` is now
`#as_free_fn(deprecated="Use `Repr(x)` instead")`, and every call site is
migrated.

Spelling follows normal prelude scoping — core's own library/whitebox code sits
below prelude in the dep graph and never gets its re-exports:

- user code, blackbox tests, and packages that already have `Repr` in scope:
  `Repr(x)`
- the two places that don't (`bigint/bigint_nonjs_wbtest.mbt`,
  `lazy_list/debug.mbt`): `@debug.Repr(x)`

The `#deprecated` messages on the per-package `X::to_repr` wrappers are updated
from "use `to_repr(a)`" to "use `Repr(a)`" so they point at the surviving
spelling.

Prelude keeps re-exporting `to_repr` (with `#warnings("-deprecated")` at the
re-export) so existing user code gets a deprecation warning and a migration path
rather than an unbound identifier.

Verified: `moon check --deny-warn --target all` clean; `moon test --target all`
green (6785/6785/6741/6696).

44 of 47 new or added lines in 27 files covered. (93.62%)

15640 of 17162 relevant lines covered (91.13%)

191700.63 hits per line