sandialabs / LoQS / 32227180573

88%

push

github

Introduce `loqs.tools.migrate` and the `loqs-migrate` CLI (#114)

* Fix PatchDictCastableTypes's rename target in the version-2 table

It pointed to PatchDictLike in loqs.core.recordables.patchdict, but that
module no longer has a real implementation to rename within -- only a
construction-only legacy shim exporting PatchDict alone, with no
PatchDictLike attribute at all. Redirects to PatchLayoutLike in
loqs.core.recordables.patchlayout instead, the same real module PatchDict
itself already redirects to. Found via a new regression test asserting
every IMPORT_LOCATION_CHANGES_BY_VERSION target is a live, importable
attribute, not from a bug report.

* Add the loqs.tools.migrate core detect/resolve/rewrite library

A library for rewriting a user's own .py/MyST Markdown source files
still using pre-1.2 LoQS APIs, distinct from Serializable's own
decode-time compatibility machinery (which only keeps already-serialized
data readable, not source code).

Three independent passes, chained together by migrate_source: straight
(module, name) renames/removals (renames.py, built on libcst's
RenameCommand plus Serializable's own IMPORT_LOCATION_CHANGES_BY_VERSION
table as data); pre-#104 positional InstructionLabel construction,
rewritten to keyword form when the target instruction can be resolved
against a caller-supplied registry (labels.py, config.py), flagged for
manual review otherwise; and detect-only patterns whose replacement is a
semantic change rather than a pure rename (.cast(), include_idles=, an
"Iz" string -- flags.py). notebook.py extracts/splices MyST code-cell
fences so the same passes run against docs/notebooks/*.md too.

Two real correctness bugs were avoided by testing against real files,
not synthetic ones: reusing Serializable._update_imports directly on
whole files corrupts unrelated code shaped like its multi-line-import
heuristic, and a blind text substitution for renamed-name usages
corrupts docstrings that narrate a rename's own history ... (continued)

2507 of 3056 branches covered (82.04%)

Branch coverage included in aggregate %.

8069 of 9001 relevant lines covered (89.65%)

0.9 hits per line