GeezerVison

Summary

Behavior-preserving simplification pass over the CLI command layer: −575 lines across
14 command files, args.rs, and 2 test files, plus one documented core visibility widening.

Per-file highlights (each verified against its module's tests when authored):

• scan.rs (−~180): 3× discovery loop, 4× GC dispatch, 3× DownloadParams, 2× staging +
  partition helpers dedup'd; pub(crate) helpers privatized; boxed_* Windows-stack frames
  deliberately kept.
• get.rs: 4× files-map + 2× client/apply/drift blocks dedup'd; dead DownloadParams.one_off
  removed.
• apply.rs: rollback's is_local_go dedup'd into apply; as_patch_sources() adopted;
  helpers privatized.
• fetch_stage.rs: offline diagnostics dedup'd; reuses get.rs is_valid_blob_hash;
  3× MemStagedSources literals → 1; pub → pub(crate).
• remove.rs: patch_matches predicate shared across 3 sites; emit_not_found dedup;
  reuses get.rs short_uuid.
• repair.rs / repair_vendor.rs: error emits reuse lock_cli::error_envelope; cleanup loop
  dedup; write-only RepairVendorCounts deleted; Missing/Corrupt arms merged; 3× remove-dir
  dedup; persist branches unified.
• rollback.rs: reuses core get_before_hash_blobs + remove's patch_matches; one-off
  emits merged.
• lock_cli.rs (−64), unlock.rs, vendor.rs, vex.rs: same treatment (shared
  envelope helpers, dedup'd emits/dispatch blocks).
• commands/mod.rs: fetch_stage/lock_cli/repair_vendor → pub(crate) (internal
  libraries, not commands).
• core go_redirect.rs: are_safe_redirect_coords pub(crate) → pub with doc comment —
  the CLI's VEX go-patches synthesis now applies the exact core guard instead of a
  drift-prone local mirror (vex.rs test retargeted accordingly).

No user-visible behavior, flags, JSON shapes, or exit codes change.

Test plan

• cargo test --workspace green (e2e_cargo re-verified in isolation after a
  parallel-load flake)
• cargo clippy --workspace --all-targets zero warnings

🤖 Generated with Claude Code