vrubel/vcom
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: skip local PathExists check when target is remote (SFTP copy)

Browse source
This commit is contained in:
vrubelroman 2026-05-13 12:43:03 +03:00
parent 7d2a2dd547
commit c598350fc5
1 changed files with 13 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

23
internal/ui/model.go
View file

@ -2358,17 +2358,20 @@ func (m *Model) handleTransfer(kind fileOpKind) (tea.Model, tea.Cmd) {
kind, m.active, srcPane.Path, dstPane.Path, srcHasRemote, dstHasRemote, sources) kind, m.active, srcPane.Path, dstPane.Path, srcHasRemote, dstHasRemote, sources)
// Check for existing targets (fast — one Stat per top-level item) // Check for existing targets (fast — one Stat per top-level item)
// Skip when target is remote — local fs check doesn't apply.
existingTargets := 0 existingTargets := 0
for _, sourcePath := range sources { if !dstHasRemote {
targetPath := filepath.Join(targetDir, filepath.Base(sourcePath)) for _, sourcePath := range sources {
exists, err := vfs.PathExists(targetPath) targetPath := filepath.Join(targetDir, filepath.Base(sourcePath))
if err != nil { exists, err := vfs.PathExists(targetPath)
log.Printf("[ERROR] Transfer: path check failed: %s err=%v", targetPath, err) if err != nil {
m.status = err.Error() log.Printf("[ERROR] Transfer: path check failed: %s err=%v", targetPath, err)
return m, nil m.status = err.Error()
} return m, nil
if exists { }
existingTargets++ if exists {
existingTargets++
}
} }
} }
overwrite := existingTargets > 0 overwrite := existingTargets > 0