chore: ensure we use unix path

DanielleMaywood · DanielleMaywood · commit 21845a952944 · 2025-04-02T09:22:21.000+01:00
diff --git a/cli/open.go b/cli/open.go
@@ -428,6 +428,8 @@ func buildVSCodeWorkspaceDevContainerLink(
 	containerName string,
 	containerFolder string,
 ) (*url.URL, url.Values) {
+	containerFolder = windowsToUnixPath(containerFolder)
+
 	qp := url.Values{}
 	qp.Add("url", clientURL)
 	qp.Add("owner", workspace.OwnerName)
@@ -520,6 +522,10 @@ func unixToWindowsPath(p string) string {
 	return strings.ReplaceAll(p, "/", "\\")
 }
 
+func windowsToUnixPath(p string) string {
+	return strings.ReplaceAll(p, "\\", "/")
+}
+
 // resolveAgentAbsPath resolves the absolute path to a file or directory in the
 // workspace. If the path is relative, it will be resolved relative to the
 // workspace's expanded directory. If the path is absolute, it will be returned
diff --git a/cli/open_test.go b/cli/open_test.go
@@ -185,11 +185,6 @@ func TestOpenVSCode_NoAgentDirectory(t *testing.T) {
 	wd, err := os.Getwd()
 	require.NoError(t, err)
 
-	absPath := "/home/coder"
-	if runtime.GOOS == "windows" {
-		absPath = "C:\\home\\coder"
-	}
-
 	tests := []struct {
 		name      string
 		args      []string
@@ -210,8 +205,8 @@ func TestOpenVSCode_NoAgentDirectory(t *testing.T) {
 		},
 		{
 			name:    "ok with absolute path",
-			args:    []string{"--test.open-error", workspace.Name, absPath},
-			wantDir: absPath,
+			args:    []string{"--test.open-error", workspace.Name, "/home/coder"},
+			wantDir: "/home/coder",
 		},
 		{
 			name:      "ok with token",
