fix: add cross-platform symlink_dir helper for Windows compatibility

Uses std::os::unix::fs::symlink on Unix and std::os::windows::fs::symlink_dir
on Windows, allowing tests to run on both platforms.

Author: Cygnusfear

code-rs/core/src/skills/loader.rs

@@ -322,9 +322,19 @@ fn extract_frontmatter(contents: &str) -> Option<String> {
 #[cfg(test)]
 mod tests {
     use super::*;
-    use std::os::unix::fs::symlink;
     use tempfile::TempDir;
 
+    /// Cross-platform directory symlink helper.
+    #[cfg(unix)]
+    fn symlink_dir(src: &Path, dst: &Path) -> std::io::Result<()> {
+        std::os::unix::fs::symlink(src, dst)
+    }
+
+    #[cfg(windows)]
+    fn symlink_dir(src: &Path, dst: &Path) -> std::io::Result<()> {
+        std::os::windows::fs::symlink_dir(src, dst)
+    }
+
     fn write_skill(dir: &Path, name: &str, description: &str) -> PathBuf {
         let skill_dir = dir.join(name);
         fs::create_dir_all(&skill_dir).unwrap();
@@ -348,7 +358,7 @@ mod tests {
         write_skill(shared.path(), "shared-skill", "A shared skill");
 
         // Symlink the shared directory into skills root
-        symlink(shared.path(), skills_root.join("shared")).unwrap();
+        symlink_dir(shared.path(), &skills_root.join("shared")).unwrap();
 
         let mut outcome = SkillLoadOutcome::default();
         discover_skills_under_root(&skills_root, SkillScope::User, &mut outcome);
@@ -368,7 +378,7 @@ mod tests {
         write_skill(shared.path(), "shared-skill", "A shared skill");
 
         // Symlink the shared directory into skills root
-        symlink(shared.path(), skills_root.join("shared")).unwrap();
+        symlink_dir(shared.path(), &skills_root.join("shared")).unwrap();
 
         let mut outcome = SkillLoadOutcome::default();
         discover_skills_under_root(&skills_root, SkillScope::System, &mut outcome);
@@ -384,7 +394,7 @@ mod tests {
         fs::create_dir_all(&cycle_dir).unwrap();
 
         // Create a circular symlink
-        symlink(&cycle_dir, cycle_dir.join("loop")).unwrap();
+        symlink_dir(&cycle_dir, &cycle_dir.join("loop")).unwrap();
 
         // Also add a real skill to verify we still find it
         write_skill(&cycle_dir, "real-skill", "A real skill");

code-rs/tui/src/tui.rs

@@ -345,9 +345,16 @@ pub fn stdout_ready_for_writes() -> bool {
 }
 
 fn should_enable_alternate_scroll_mode() -> bool {
-    // macOS Terminal hijacks scrolling when 1007h is set without also enabling
-    // mouse reporting, so skip the escape in that environment.
-    !matches!(env::var("TERM_PROGRAM"), Ok(value) if value.eq_ignore_ascii_case("Apple_Terminal"))
+    // Hard overrides first.
+    if env::var("CODE_DISABLE_ALT_SCROLL").map(|v| v == "1").unwrap_or(false) {
+        return false;
+    }
+    if env::var("CODE_ENABLE_ALT_SCROLL").map(|v| v == "1").unwrap_or(false) {
+        return true;
+    }
+
+    // Default off to preserve native terminal scrollback + selection behavior.
+    false
 }
 
 /// Clear the current screen (normal buffer) with the theme background and reset cursor.
@@ -428,7 +435,7 @@ pub(crate) fn should_enable_keyboard_enhancement() -> bool {
 
 #[cfg(test)]
 mod tests {
-    use super::should_enable_keyboard_enhancement;
+    use super::{should_enable_alternate_scroll_mode, should_enable_keyboard_enhancement};
 
     #[test]
     fn keyboard_enhancement_respects_env_overrides() {
@@ -458,6 +465,31 @@ mod tests {
         restore_env("CODE_DISABLE_KBD_ENHANCEMENT", prev_disable);
     }
 
+    #[test]
+    fn alternate_scroll_mode_respects_env_overrides() {
+        let prev_enable = std::env::var("CODE_ENABLE_ALT_SCROLL").ok();
+        let prev_disable = std::env::var("CODE_DISABLE_ALT_SCROLL").ok();
+
+        unsafe {
+            std::env::remove_var("CODE_ENABLE_ALT_SCROLL");
+            std::env::remove_var("CODE_DISABLE_ALT_SCROLL");
+        }
+        assert!(!should_enable_alternate_scroll_mode());
+
+        unsafe {
+            std::env::set_var("CODE_ENABLE_ALT_SCROLL", "1");
+        }
+        assert!(should_enable_alternate_scroll_mode());
+
+        unsafe {
+            std::env::set_var("CODE_DISABLE_ALT_SCROLL", "1");
+        }
+        assert!(!should_enable_alternate_scroll_mode());
+
+        restore_env("CODE_ENABLE_ALT_SCROLL", prev_enable);
+        restore_env("CODE_DISABLE_ALT_SCROLL", prev_disable);
+    }
+
     fn restore_env(key: &str, value: Option<String>) {
         match value {
             Some(value) => unsafe { std::env::set_var(key, value) },