feat(env): show message when switching version automatically

Author: hyf0

crates/vite_global_cli/src/commands/env/config.rs

@@ -230,6 +230,15 @@ pub async fn resolve_version(cwd: &AbsolutePath) -> Result<VersionResolution, Er
         });
     }
 
+    resolve_version_from_project(cwd).await
+}
+
+/// Resolve the Node.js version from project files only (`.node-version`, `engines.node`, etc.).
+///
+/// Unlike [`resolve_version`], this skips session overrides (env var and session file).
+/// Use this when the caller IS the session setter (e.g., `vp env use` resolving which
+/// version a project needs, independent of what's currently active).
+pub async fn resolve_version_from_project(cwd: &AbsolutePath) -> Result<VersionResolution, Error> {
     let provider = NodeProvider::new();
 
     // Use shared version resolution with directory walking

crates/vite_global_cli/src/commands/env/setup.rs

@@ -319,6 +319,23 @@ vp() {
         command vp "$@"
     fi
 }
+
+# Auto-switch Node.js version on directory change
+__vp_use_on_cd() {
+    vp env use --silent-if-unchanged
+}
+if [ -n "${ZSH_VERSION-}" ]; then
+    if [[ ! " ${chpwd_functions[*]-} " == *" __vp_use_on_cd "* ]]; then
+        chpwd_functions+=(__vp_use_on_cd)
+    fi
+else
+    __vpcd() {
+        \cd "$@" || return $?
+        __vp_use_on_cd
+    }
+    alias cd=__vpcd
+fi
+__vp_use_on_cd
 "#
     .replace("__VP_BIN__", &bin_path_ref);
     let env_file = vite_plus_home.join("env");
@@ -344,6 +361,15 @@ function vp
         command vp $argv
     end
 end
+
+# Auto-switch Node.js version on directory change
+function __vp_use_on_cd --on-variable PWD --description 'Auto-switch Node.js version'
+    if status is-command-substitution
+        return
+    end
+    vp env use --silent-if-unchanged
+end
+__vp_use_on_cd
 "#
     .replace("__VP_BIN__", &bin_path_ref);
     let env_fish_file = vite_plus_home.join("env.fish");
@@ -379,6 +405,19 @@ function vp {
         & (Join-Path $__vp_bin "vp.exe") @args
     }
 }
+
+# Auto-switch Node.js version on directory change
+$__vp_original_prompt = $function:prompt
+function prompt {
+    $loc = Get-Location
+    if ($global:__vp_last_dir -ne $loc.Path) {
+        $global:__vp_last_dir = $loc.Path
+        vp env use --silent-if-unchanged
+    }
+    & $__vp_original_prompt
+}
+$global:__vp_last_dir = (Get-Location).Path
+vp env use --silent-if-unchanged
 "#;
 
     // For PowerShell, use the actual absolute path (not $HOME-relative)
@@ -688,6 +727,70 @@ mod tests {
         );
     }
 
+    #[tokio::test]
+    async fn test_create_env_files_posix_contains_cd_hook() {
+        let temp_dir = TempDir::new().unwrap();
+        let home = AbsolutePathBuf::new(temp_dir.path().to_path_buf()).unwrap();
+        let _guard = home_guard(temp_dir.path());
+
+        create_env_files(&home).await.unwrap();
+
+        let env_content = tokio::fs::read_to_string(home.join("env")).await.unwrap();
+
+        // Verify cd hook components
+        assert!(
+            env_content.contains("__vp_use_on_cd"),
+            "env file should contain __vp_use_on_cd function"
+        );
+        assert!(
+            env_content.contains("chpwd_functions"),
+            "env file should use chpwd_functions for zsh"
+        );
+        assert!(env_content.contains("alias cd=__vpcd"), "env file should alias cd for bash");
+    }
+
+    #[tokio::test]
+    async fn test_create_env_files_fish_contains_cd_hook() {
+        let temp_dir = TempDir::new().unwrap();
+        let home = AbsolutePathBuf::new(temp_dir.path().to_path_buf()).unwrap();
+        let _guard = home_guard(temp_dir.path());
+
+        create_env_files(&home).await.unwrap();
+
+        let fish_content = tokio::fs::read_to_string(home.join("env.fish")).await.unwrap();
+
+        // Verify fish cd hook
+        assert!(
+            fish_content.contains("__vp_use_on_cd"),
+            "env.fish should contain __vp_use_on_cd function"
+        );
+        assert!(
+            fish_content.contains("--on-variable PWD"),
+            "env.fish should use --on-variable PWD for cd detection"
+        );
+    }
+
+    #[tokio::test]
+    async fn test_create_env_files_ps1_contains_cd_hook() {
+        let temp_dir = TempDir::new().unwrap();
+        let home = AbsolutePathBuf::new(temp_dir.path().to_path_buf()).unwrap();
+        let _guard = home_guard(temp_dir.path());
+
+        create_env_files(&home).await.unwrap();
+
+        let ps1_content = tokio::fs::read_to_string(home.join("env.ps1")).await.unwrap();
+
+        // Verify PowerShell cd hook
+        assert!(
+            ps1_content.contains("__vp_last_dir"),
+            "env.ps1 should contain __vp_last_dir for directory change detection"
+        );
+        assert!(
+            ps1_content.contains("--silent-if-unchanged"),
+            "env.ps1 should call vp env use --silent-if-unchanged"
+        );
+    }
+
     #[tokio::test]
     async fn test_execute_env_only_creates_home_dir_and_env_files() {
         let temp_dir = TempDir::new().unwrap();

crates/vite_global_cli/src/commands/env/use.rs

@@ -98,7 +98,7 @@ pub async fn execute(
         let resolved = config::resolve_version_alias(ver, &provider).await?;
         (resolved, format!("{ver}"))
     } else {
-        let resolution = config::resolve_version(&cwd).await?;
+        let resolution = config::resolve_version_from_project(&cwd).await?;
         let source = resolution.source.clone();
         (resolution.version, source)
     };
@@ -154,7 +154,7 @@ pub async fn execute(
     }
 
     // Status message to stderr (visible to user)
-    eprintln!("Using Node.js v{} (resolved from {})", resolved_version, source_desc);
+    eprintln!("vp: Using Node.js v{} (from {})", resolved_version, source_desc);
 
     Ok(ExitStatus::default())
 }

packages/global/snap-tests/command-env-use-silent-if-unchanged/.node-version

@@ -0,0 +1 @@
+20.18.0

packages/global/snap-tests/command-env-use-silent-if-unchanged/package.json

@@ -0,0 +1 @@
+{ "name": "command-env-use-silent-if-unchanged", "version": "1.0.0", "private": true }

packages/global/snap-tests/command-env-use-silent-if-unchanged/snap.txt

@@ -0,0 +1,10 @@
+> vp env use --silent-if-unchanged # first call - should show message
+export VITE_PLUS_NODE_VERSION=20.18.0
+vp: Using Node.js v<semver> (from .node-version)
+
+> VITE_PLUS_NODE_VERSION=20.18.0 vp env use --silent-if-unchanged # same version - should be silent
+export VITE_PLUS_NODE_VERSION=20.18.0
+
+> VITE_PLUS_NODE_VERSION=22.0.0 vp env use --silent-if-unchanged # different version - should show message
+export VITE_PLUS_NODE_VERSION=20.18.0
+vp: Using Node.js v<semver> (from .node-version)

packages/global/snap-tests/command-env-use-silent-if-unchanged/steps.json

@@ -0,0 +1,11 @@
+{
+  "env": {
+    "VITE_PLUS_ENV_USE_EVAL_ENABLE": "1"
+  },
+  "ignoredPlatforms": ["win32"],
+  "commands": [
+    "vp env use --silent-if-unchanged # first call - should show message",
+    "VITE_PLUS_NODE_VERSION=20.18.0 vp env use --silent-if-unchanged # same version - should be silent",
+    "VITE_PLUS_NODE_VERSION=22.0.0 vp env use --silent-if-unchanged # different version - should show message"
+  ]
+}

packages/global/snap-tests/command-env-use/snap.txt

@@ -14,7 +14,7 @@ Options:
 
 > vp env use 20.18.0 --no-install # should output export command to stdout
 export VITE_PLUS_NODE_VERSION=20.18.0
-Using Node.js v<semver> (resolved from <semver>)
+vp: Using Node.js v<semver> (from <semver>)
 
 > vp env use --unset # should output unset command to stdout
 unset VITE_PLUS_NODE_VERSION