Skip to content

Split remote/mod.rs (2315 LOC) into 7 focused modules#90

Merged
an0nn30 merged 1 commit intomainfrom
chore/split-remote-mod
Mar 27, 2026
Merged

Split remote/mod.rs (2315 LOC) into 7 focused modules#90
an0nn30 merged 1 commit intomainfrom
chore/split-remote-mod

Conversation

@an0nn30
Copy link
Copy Markdown
Owner

@an0nn30 an0nn30 commented Mar 27, 2026

What does this PR do?

Splits conch_tauri/src/remote/mod.rs from 2315 lines → 436 lines by extracting 6 new submodules:

Module Lines Contents
mod.rs 436 Core state types, shared helpers (establish_ssh_session, spawn_output_forwarder)
ssh_commands.rs 685 SSH connect/write/resize/disconnect, credential resolution
server_commands.rs 860 Server/folder CRUD, import/export, vault eager import
sftp_commands.rs 200 SFTP + local filesystem operations
transfer_commands.rs 74 Upload/download/cancel
tunnel_commands.rs 170 Tunnel lifecycle
auth.rs 39 Auth prompt response handlers

lib.rs updated to use full module paths in generate_handler!.

Type of change

  • Refactor / cleanup

Testing

  • Builds without errors
  • No regressions — all 119 tests pass

Pure refactor. No function signatures or behavior changed.

Break the 2315-line mod.rs into 7 focused modules:
- ssh_commands.rs: SSH connect, write, resize, disconnect, open channel
- server_commands.rs: server CRUD, folders, import/export, sessions
- sftp_commands.rs: SFTP operations and local filesystem commands
- transfer_commands.rs: file download, upload, cancel
- tunnel_commands.rs: tunnel start, stop, save, delete, list
- auth.rs: host key and password prompt responses
- mod.rs: core state types, shared helpers, module declarations

All 119 tests pass, no behavior changes.
@an0nn30 an0nn30 merged commit af039bf into main Mar 27, 2026
5 checks passed
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.

1 participant