fix(activate): strictly use forward slashes and colons for POSIX scripts on Windows

Author: snowdream

cmd/10.deactivate.go

@@ -113,6 +113,10 @@ func generateDeactivationScript(shellType string) string {
 
 // generatePosixDeactivationScript generates a POSIX-compatible deactivation script.
 func generatePosixDeactivationScript(shimsDir string) string {
+	posixShimsDir := shimsDir
+	if runtime.GOOS == "windows" {
+		posixShimsDir = filepath.ToSlash(posixShimsDir)
+	}
 	return fmt.Sprintf(`# UniRTM deactivation script
 # 1. Clean up shims and injected paths from PATH
 _unirtm_clean_path() {
@@ -142,7 +146,7 @@ done
 # 4. Remove hook if present
 unset -f unirtm
 unset -f _unirtm_hook
-`, shimsDir)
+`, posixShimsDir)
 }
 
 // generateFishDeactivationScript generates a fish shell deactivation script.

internal/service/activation.go

@@ -279,12 +279,24 @@ func (m *ActivationManager) generatePosixScript(config ActivationConfig) (*Activ
 		// Add shims directory to PATH
 		sb.WriteString("# Add UniRTM shims to PATH\n")
 		// Clean up existing shims from PATH to avoid duplicates
-		sb.WriteString(fmt.Sprintf(`export PATH="%s:$(echo "$PATH" | sed -E 's|%s:?||g' | sed 's|:$||')"`+"\n", config.ShimsDir, config.ShimsDir))
+		posixShimsDir := config.ShimsDir
+		if runtime.GOOS == "windows" {
+			posixShimsDir = filepath.ToSlash(posixShimsDir)
+		}
+		sb.WriteString(fmt.Sprintf(`export PATH="%s:$(echo "$PATH" | sed -E 's|%s:?||g' | sed 's|:$||')"`+"\n", posixShimsDir, posixShimsDir))
 		sb.WriteString("\n")
 	} else if len(config.InjectedPaths) > 0 {
 		// PATH mode activation
 		sb.WriteString("# UniRTM PATH mode activation\n")
-		injectedPath := strings.Join(config.InjectedPaths, string(os.PathListSeparator))
+		var posixPaths []string
+		for _, p := range config.InjectedPaths {
+			if runtime.GOOS == "windows" {
+				posixPaths = append(posixPaths, filepath.ToSlash(p))
+			} else {
+				posixPaths = append(posixPaths, p)
+			}
+		}
+		injectedPath := strings.Join(posixPaths, ":")
 
 		// Use UNIRTM_PATH to track injected paths.
 		// Use a shell loop to filter out existing UNIRTM-managed entries from PATH,

internal/service/auto_activation.go

@@ -418,13 +418,16 @@ func (m *AutoActivationManager) generatePosixDeactivation(sb *strings.Builder, s
 		sb.WriteString("\n")
 	} else {
 		// No previous path known, clean up what we injected
-		shimsDir := m.activationManager.shimsDir
+		posixShimsDir := m.activationManager.shimsDir
+		if runtime.GOOS == "windows" {
+			posixShimsDir = filepath.ToSlash(posixShimsDir)
+		}
 		sb.WriteString("# Clean up UniRTM paths from PATH\n")
 		sb.WriteString("_unirtm_clean_path() {\n")
 		sb.WriteString("  local result=\"\"\n")
 		sb.WriteString("  local IFS=:\n")
 		sb.WriteString("  for _p in $PATH; do\n")
-		sb.WriteString(fmt.Sprintf("    case \":$UNIRTM_PATH:%s:\" in\n", shimsDir))
+		sb.WriteString(fmt.Sprintf("    case \":$UNIRTM_PATH:%s:\" in\n", posixShimsDir))
 		sb.WriteString("      *\":$_p:\"*) ;;\n")
 		sb.WriteString("      *) result=\"${result:+$result:}$_p\" ;;\n")
 		sb.WriteString("    esac\n")