Skip to content

feat(tui): add dual-pane SFTP file browser#34

Open
vybraan wants to merge 4 commits into
mainfrom
feat/sftp
Open

feat(tui): add dual-pane SFTP file browser#34
vybraan wants to merge 4 commits into
mainfrom
feat/sftp

Conversation

@vybraan
Copy link
Copy Markdown
Collaborator

@vybraan vybraan commented May 16, 2026

Implements the initial SFTP file transfer view for #8.

This adds dual-pane file browser with:

  • local and remote directory navigation
  • upload from local to remote
  • download from remote to local

- Merge upstream/main (go 1.26.2, charm.land imports, tests, linting, security)
- Update sftp.go to use charm.land import paths
- Fix sftpModel.View() to return tea.View
@vybraan vybraan requested a review from lfaoro May 16, 2026 23:59
@vybraan vybraan marked this pull request as ready for review May 17, 2026 00:00
@lfaoro
Copy link
Copy Markdown
Owner

lfaoro commented May 17, 2026

image should add ctrl+s to the list of commands expanded with ? pressing ctrl+s again should toggle back

@lfaoro
Copy link
Copy Markdown
Owner

lfaoro commented May 17, 2026

would it be hard to have the ability to select multiple files to transfer using spacebar for selection ?

@lfaoro
Copy link
Copy Markdown
Owner

lfaoro commented May 17, 2026

should add ctrl+s to readme, and update changelog with the new feature

@vybraan
Copy link
Copy Markdown
Collaborator Author

vybraan commented May 17, 2026

should add ctrl+s to readme, and update changelog with the new feature

sure,

would it be hard to have the ability to select multiple files to transfer via ?

i dont believe so, will check it.

@vybraan
Copy link
Copy Markdown
Collaborator Author

vybraan commented May 17, 2026

would it be hard to have the ability to select multiple files to transfer using spacebar for selection ?

@lfaoro I think for now i'm going to keep it imp for multiple individual files but excluding the folders, i guess it can become messy pretty quick if i happen to do for thousands of folders and nested folders. So to transfer files in a dir you'd have to select all files inside and transfer them. what do think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants