Skip to content

fix: browse remote Linux workspaces from a Windows client - #25

Merged
Golden-Pigeon merged 1 commit into
mainfrom
fix/21-windows-remote-paths
Nov 9, 2026
Merged

Golden-Pigeon merged 1 commit into
mainfrom
fix/21-windows-remote-paths

Conversation

@Golden-Pigeon

Copy link
Copy Markdown
Owner

Fixes #21.

Original problem

On a Windows client, opening a mirrored remote workspace resolved the remote POSIX path with the client platform path implementation. /home/ubuntu became C:\home\ubuntu, was sent as remotePath to the remote agent, and file list/stat/read failed with not found: C:\home\ubuntu.

Change

  • Split local anchor resolution (platform path) from remote workspace resolution (POSIX path) in a pure resolver used by the workspace-files adapter.
  • Convert the workspace-relative path to / separators before joining the remote root.
  • Added 9 path tests covering Windows anchors, relative paths, the workspace root, remote absolute paths outside the root, and escape rejection.

Fixed during this change

  • None.

Resolved

  • Windows clients can list/stat/read mirrored workspace files under POSIX remote roots.
  • The adapter no longer sends a Windows-resolved path to the remote agent.

Unresolved

Testing

  • pnpm --filter dsh-remote-local-plugin test (126/126)
  • pnpm --filter dsh-remote-local-plugin typecheck
  • pnpm --filter dsh-remote-local-plugin build
  • Real Windows host: ran the compiled resolver with the Windows node:path; /home/ubuntu stays /home/ubuntu, the local anchor uses backslashes, and all cases pass.

The workspace-files adapter used the client platform path for both the local anchor and the remote POSIX root, so a Windows client turned /home/ubuntu into C:\home\ubuntu before calling the remote agent. Split the two path namespaces in a pure resolver and cover the Windows and POSIX cases, including the real Windows host run.

Refs #21
@Golden-Pigeon
Golden-Pigeon merged commit 2a97a02 into main Nov 9, 2026
2 checks passed
@Golden-Pigeon
Golden-Pigeon deleted the fix/21-windows-remote-paths branch November 9, 2026 11:33
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.

Windows client: remote file list/stat/read resolves POSIX paths to C:\home\... and fails

1 participant