fix(activate): use os.PathListSeparator for PowerShell scripts to support Linux/macOS pwsh

Author: snowdream

cmd/10.deactivate.go

@@ -184,10 +184,10 @@ func generatePowerShellDeactivationScript(shimsDir string) string {
 # 1. Clean up PATH
 $unirtmPaths = @()
 if ($env:UNIRTM_PATH) {
-    $unirtmPaths = $env:UNIRTM_PATH -split ';'
+    $unirtmPaths = $env:UNIRTM_PATH -split '%c'
 }
 $shimsDir = "%s"
-$env:PATH = ($env:PATH -split ';' | Where-Object { $unirtmPaths -notcontains $_ -and $_ -ne $shimsDir }) -join ';'
+$env:PATH = ($env:PATH -split '%c' | Where-Object { $unirtmPaths -notcontains $_ -and $_ -ne $shimsDir }) -join '%c'
 
 # 2. Unset environment variables
 Remove-Item Env:\UNIRTM_PATH -ErrorAction SilentlyContinue
@@ -201,5 +201,5 @@ Get-ChildItem Env: | Where-Object { $_.Name -match '^UNIRTM_.*_VERSION$' } | For
 
 # 4. Remove hook
 if (Test-Path Function:\unirtm) { Remove-Item Function:\unirtm }
-`, shimsDir)
+`, os.PathListSeparator, shimsDir, os.PathListSeparator, os.PathListSeparator)
 }

internal/service/activation.go

@@ -481,7 +481,7 @@ func (m *ActivationManager) generatePowerShellScript(config ActivationConfig) (*
 			shimsDir = filepath.FromSlash(shimsDir)
 		}
 		sb.WriteString(fmt.Sprintf("$shimsDir = \"%s\"\n", shimsDir))
-		sb.WriteString("$env:PATH = \"$shimsDir;\" + (($env:PATH -split ';') | Where-Object { $_ -ne $shimsDir } -join ';')\n")
+		sb.WriteString(fmt.Sprintf("$env:PATH = \"$shimsDir%c\" + (($env:PATH -split '%c') | Where-Object { $_ -ne $shimsDir } -join '%c')\n", os.PathListSeparator, os.PathListSeparator, os.PathListSeparator))
 		sb.WriteString("\n")
 	} else if len(config.InjectedPaths) > 0 {
 		// PATH mode activation
@@ -494,10 +494,10 @@ func (m *ActivationManager) generatePowerShellScript(config ActivationConfig) (*
 			}
 			paths = append(paths, path)
 		}
-		injectedPath := strings.Join(paths, ";")
-		sb.WriteString(fmt.Sprintf("$unirtmPaths = \"%s\" -split ';'\n", injectedPath))
-		sb.WriteString("$env:UNIRTM_PATH = $unirtmPaths -join ';'\n")
-		sb.WriteString("$env:PATH = ($env:UNIRTM_PATH + ';' + (($env:PATH -split ';') | Where-Object { $unirtmPaths -notcontains $_ } -join ';'))\n")
+		injectedPath := strings.Join(paths, string(os.PathListSeparator))
+		sb.WriteString(fmt.Sprintf("$unirtmPaths = \"%s\" -split '%c'\n", injectedPath, os.PathListSeparator))
+		sb.WriteString(fmt.Sprintf("$env:UNIRTM_PATH = $unirtmPaths -join '%c'\n", os.PathListSeparator))
+		sb.WriteString(fmt.Sprintf("$env:PATH = ($env:UNIRTM_PATH + '%c' + (($env:PATH -split '%c') | Where-Object { $unirtmPaths -notcontains $_ } -join '%c'))\n", os.PathListSeparator, os.PathListSeparator, os.PathListSeparator))
 		sb.WriteString("\n")
 	}
 

internal/service/auto_activation.go

@@ -518,9 +518,9 @@ func (m *AutoActivationManager) generatePowerShellDeactivation(sb *strings.Build
 		}
 		sb.WriteString("# Clean up UniRTM paths from PATH\n")
 		sb.WriteString("$unirtmPaths = @()\n")
-		sb.WriteString("if ($env:UNIRTM_PATH) { $unirtmPaths = $env:UNIRTM_PATH -split ';' }\n")
+		sb.WriteString(fmt.Sprintf("if ($env:UNIRTM_PATH) { $unirtmPaths = $env:UNIRTM_PATH -split '%c' }\n", os.PathListSeparator))
 		sb.WriteString(fmt.Sprintf("$shimsDir = \"%s\"\n", shimsDir))
-		sb.WriteString("$env:PATH = ($env:PATH -split ';' | Where-Object { $unirtmPaths -notcontains $_ -and $_ -ne $shimsDir }) -join ';'\n")
+		sb.WriteString(fmt.Sprintf("$env:PATH = ($env:PATH -split '%c' | Where-Object { $unirtmPaths -notcontains $_ -and $_ -ne $shimsDir }) -join '%c'\n", os.PathListSeparator, os.PathListSeparator))
 		sb.WriteString("\n")
 	}