Skip to content

Improve WSL & Windows compatibility for SSH tunnels and editors #210

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

oliverholworthy
Copy link

This PR enhances compatibility with Windows and Windows Subsystem for Linux (WSL) environments by addressing several issues:

SSH Tunneling Improvements

  • Fix SSH tunnel configuration in WSL environments by properly managing both Windows and WSL SSH config files
  • Update SSH tunnel naming to use a simpler format (removing devspace name duplication)
  • Use proper path formatting with as_posix() in tunnel server code

Editor Support Enhancements

  • Add robust editor detection that works correctly in Windows, Linux, and WSL environments
  • Properly handle Windows executables from WSL for VS Code and Cursor editors
  • Add detailed error messages to guide users through editor setup issues
  • Ensure environment variables are set correctly for each platform

Testing & Dependencies

  • Add tests for the editor detection functionality
  • Add WSL-specific tests for the SSH config handling

This allows users on any platform to seamlessly use SSH tunnels and launch editors, particularly improving the experience for those using WSL.

@@ -0,0 +1,163 @@
import os
import platform
import subprocess

Check notice

Code scanning / CodeQL

Unused import Note test

Import of 'subprocess' is not used.
import pytest
from pathlib import Path
import shutil
from unittest.mock import patch, MagicMock

Check notice

Code scanning / CodeQL

Unused import Note test

Import of 'patch' is not used.
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