sandialabs / LoQS / 41141528855

90%

push

github

Fix four `loqs.tools` bugs: state phases, legacy migration speed, dry-run exit code, `repeats` guard (#159)

* Fix state phases, dry-run exit code and repeats guard

Fix three loqs.tools bugs from #155: phases at any qubit count with no
wrong signs, a --dry-run that fails a CI gate when it should, and an
up-front refusal of a bad repeats.

- get_state_probs_phases walks single flips between nonzero strings,
  signs unequal magnitudes too, and raises ValueError where it can't.
- loqs-migrate --dry-run exits 1 when any file would be rewritten; the
  module docstring and --help now state every exit code.
- profile_strategies raises ValueError for repeats < 1 before any run.
- Minor: tests for linked, unlinked, complex and no-|0..0> states, the
  pending-rewrite dry run and the repeats guard; docstrings.

Co-authored-by: claude-sonnet-5-5 <noreply@anthropic.com>
Co-authored-by: claude-opus-5-5 <noreply@anthropic.com>

* Speed up legacy-function migration

Reading a v1.0/v1.1 program with migrate_legacy_fns=True drops from
about 17 s to about 2.5 s, and rewrite_renames skips files that only
mention old names as text, with the same output as before.

- Cache Serializable._prepare_function_source per (src, version) with
  lru_cache(256), returning review items as a tuple so hits can't drift.
- rewrite_renames runs a RenameCommand only for names the finder sees
  in code; the finder now also sees attribute access (loqs.core.X).
- Minor: tests for attribute-access rewrites and the gate on a cache
  hit; docstrings; one mypy annotation; CHANGELOG entry for #155.

Co-authored-by: claude-sonnet-5-5 <noreply@anthropic.com>
Co-authored-by: claude-opus-5-5 <noreply@anthropic.com>

---------

Co-authored-by: claude-sonnet-5-5 <noreply@anthropic.com>

3655 of 4398 branches covered (83.11%)

Branch coverage included in aggregate %.

11882 of 13090 relevant lines covered (90.77%)

0.91 hits per line