Skip to content

fix(remote-workspace): wire file CRUD through remote connection context#1415

Merged
bobleer merged 2 commits into
GCWing:mainfrom
bobleer:fix/remote-workspace-file-operations
Jul 4, 2026
Merged

fix(remote-workspace): wire file CRUD through remote connection context#1415
bobleer merged 2 commits into
GCWing:mainfrom
bobleer:fix/remote-workspace-file-operations

Conversation

@bobleer

@bobleer bobleer commented Jul 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Route remoteConnectionId through desktop file commands (create, rename, delete) and the file explorer so CRUD operations work correctly in SSH remote workspaces.
  • Fix remote workspace path resolution: only use the preferred connection hint when multiple registry candidates match, and add remote-specific path normalization and filename validation.
  • Improve file explorer UX: use confirmDanger for delete confirmation, platform-aware paste shortcut labels, Windows-native clipboard paths, and Linux/macOS reveal-in-explorer behavior.

Test plan

  • Open a local workspace and verify create/rename/delete file and folder still work
  • Open an SSH remote workspace and verify create/rename/delete file and folder work
  • Copy path on Windows-style paths and confirm backslashes in clipboard
  • Delete a file/folder and confirm the styled danger dialog appears
  • Reveal in explorer on Linux (file highlight via D-Bus when available)
  • pnpm run type-check:web and cargo check -p bitfun-desktop pass locally

bobleer added 2 commits July 4, 2026 17:08
Route remote_connection_id through desktop file commands and the file
explorer so create, rename, and delete work in SSH workspaces, and improve
path normalization, reveal-in-explorer, and delete confirmation UX.
Fix E0308 on ubuntu-latest where Slice::join expects &str, not char.
@bobleer bobleer merged commit 1cce339 into GCWing:main Jul 4, 2026
4 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