Repository navigation
Add TRAMP / remote host support - #270
Merged
Merged
Conversation
Merge explicit prefix-map entries with implicit mappings from TRAMP folders. Strip TRAMP prefixes in local-to-remote so the server sees local paths.
Add a "TRAMP / remote hosts" section explaining that auto-install is local only, users must install eca on the remote PATH or set eca-custom-command, and that path mappings are derived from TRAMP workspace folders automatically.
Refuse auto-download on remote TRAMP hosts and show a clear error message directing users to install eca on the remote PATH or set eca-custom-command. Use executable-find with REMOTE argument to search the remote PATH, and make server-version detection TRAMP-aware by running --version through the remote shell when the binary lives on a remote host.
ericdallo
added a commit
that referenced
this pull request
Jul 11, 2026
Path mappings were derived via (eca-session), which resolves from the current buffer, so server messages handled while an unrelated buffer was current could pick the wrong session's mappings (or none). Bind the owning session dynamically while handling server messages. Also anchor the server version check at the session workspace so eca-version/eca-doctor report the remote binary for TRAMP sessions, restore defcustom indentation, and add a changelog entry for #270. 🤖 Generated with [ECA](https://eca.dev) (nubank-anthropic/fable-5 - max) Co-Authored-By: eca-agent <git@eca.dev>
Member
|
Thank you, will do small changes but overall looks good |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds support for launching ECA in TRAMP buffers when the buffer's
default-directoryis remote.TRAMP prefixes are stripped before sending paths to the server. Inbound, server paths are matched against the session's workspace folders to reconstruct the correct TRAMP paths. Explicit entries in
eca-local-to-remote-prefix-mapalways take priority over auto-derived mappings.Auto-install is skipped in TRAMP buffers. I was struggling to get this working.
Changes