IJHack / QtPass / 27475393351

55%

Build Type

push

github

Committed by web-flow

Commit Message

refactor: dedup RealPass::Move/Copy via passMoveOrCopy (#1513) (#1522)

RealPass::Move and RealPass::Copy were byte-identical apart from the
pass subcommand ("mv" vs "cp") and the process id (PASS_MOVE vs
PASS_COPY): same force-skip guard, same store-relative path mapping,
same .gpg-suffix stripping, same arg assembly. Extract a private
passMoveOrCopy(id, subcommand, src, dest, force); Move/Copy become
one-line delegations.

Scope note: only the RealPass internal duplication is addressed. The
cross-backend "normalizePaths in Pass" idea from #1513 doesn't fit —
ImitatePass::Move/Copy are structurally different (git mv/cp + direct
file ops + reencrypt) and don't share the store-relative/.gpg path
shape. The executeWrapper-indirection, Grep-regex-semantics, and
backend-factory parts of #1513 remain open.

Behaviour unchanged; tst_integration 20/20 (incl. RealPass paths).

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

Coverage Stats

0 of 5 new or added lines in 1 file covered. (0.0%)

10 existing lines in 1 file now uncovered.

3731 of 6714 relevant lines covered (55.57%)

36.9 hits per line