mechanical-refactor-verify

Make mechanical refactoring (file splits, function moves, module extractions, renames) machine-checkable instead of eyeballed. Reproduce a relocation commit byte-for-byte from faithful primitives, and split an extraction into a verifiable prepare + move + postpare. Use when doing or reviewing such changes.

Install
npx skills add 'https://github.com/sgl-project/sglang/tree/main/.claude/skills/mechanical-refactor-verify'
Download bundle ↓
main · a9fb1c3Scanned 2026-09-17

Contributors

GitHub-linked commit authors for this SKILL.md at the saved revision. Co-authors and history before file renames are not included.

File history ↗

scripts/tests/reproduction_cli/test_proof_discovery.py

scripts/tests/reproduction_cli/test_proof_discovery.pyBrowse 41 files
View on GitHub
← Back to SKILL.md
import sysfrom pathlib import Path sys.path.insert(0, str(Path(__file__).resolve().parents[2])) from cli_testlib import _chain, _write_stub_prooffrom mechanical_refactor_reproduction_cli import (    VERDICT_AMBIGUOUS_PROOF,    VERDICT_MISSING_PROOF,    VERDICT_PASS,    verify_chain,) _MSG = "mechanical_provable: move foo"  def _run_single(repo: Path, proof: Path):    base, shas = _chain(repo, [_MSG])    return shas[0], verify_chain(        base=base, branch="chain", proof=proof, repo_root=str(repo)    )  def test_proof_is_found_under_repro_scripts_by_sha_prefix(    repo: Path, tmp_path: Path) -> None:    """The generator layout repro_scripts/<sha9>.py resolves to the commit's proof."""    proof = tmp_path / "proof"    base, shas = _chain(repo, [_MSG])    _write_stub_proof(proof, shas[0], stem_len=9)    result = verify_chain(base=base, branch="chain", proof=proof, repo_root=str(repo))    assert result.verdicts[0].verdict == VERDICT_PASS  def test_proof_is_found_flat_in_the_proof_folder_by_full_sha(    repo: Path, tmp_path: Path) -> None:    """A flat <proof>/<full-sha>.py layout is also accepted."""    proof = tmp_path / "proof"    base, shas = _chain(repo, [_MSG])    _write_stub_proof(proof, shas[0], flat=True, stem_len=40)    result = verify_chain(base=base, branch="chain", proof=proof, repo_root=str(repo))    assert result.verdicts[0].verdict == VERDICT_PASS  def test_provable_commit_without_a_proof_script_is_missing_proof(    repo: Path, tmp_path: Path) -> None:    """A mechanical_provable commit with no matching script fails as MISSING_PROOF."""    proof = tmp_path / "proof"    proof.mkdir()    sha, result = _run_single(repo, proof)    assert result.verdicts[0].verdict == VERDICT_MISSING_PROOF    assert not result.passed  def test_unrelated_and_non_hex_scripts_do_not_match(repo: Path, tmp_path: Path) -> None:    """Scripts named for another sha or with a non-hex stem are not this commit's proof."""    proof = tmp_path / "proof"    scripts = proof / "repro_scripts"    scripts.mkdir(parents=True)    (scripts / "0123456789abcdef.py").write_text("raise SystemExit(1)\n")    (scripts / "not_a_sha.py").write_text("raise SystemExit(1)\n")    sha, result = _run_single(repo, proof)    assert result.verdicts[0].verdict == VERDICT_MISSING_PROOF  def test_two_scripts_matching_one_commit_are_ambiguous(    repo: Path, tmp_path: Path) -> None:    """A commit matched by both a nested and a flat script fails as AMBIGUOUS_PROOF."""    proof = tmp_path / "proof"    base, shas = _chain(repo, [_MSG])    _write_stub_proof(proof, shas[0], stem_len=9)    _write_stub_proof(proof, shas[0], flat=True, stem_len=12)    result = verify_chain(base=base, branch="chain", proof=proof, repo_root=str(repo))    assert result.verdicts[0].verdict == VERDICT_AMBIGUOUS_PROOF    assert not result.passed  def test_short_hex_stem_below_minimum_length_is_ignored(    repo: Path, tmp_path: Path) -> None:    """A 6-char hex stem is too short to name a commit and is not treated as a proof."""    proof = tmp_path / "proof"    base, shas = _chain(repo, [_MSG])    _write_stub_proof(proof, shas[0], stem_len=6)    result = verify_chain(base=base, branch="chain", proof=proof, repo_root=str(repo))    assert result.verdicts[0].verdict == VERDICT_MISSING_PROOF