feat(env): migrate vp env packages to vp pm list -g + add vp list/vp ls aliases

Move global package listing from `vp env` to `vp pm list -g` where
package management belongs. Add `vp list` and `vp ls` as top-level
aliases that rewrite to `vp pm list`. When `-g` is passed, the command
intercepts early (before JS runtime init) and delegates to the managed
packages listing with optional pattern filtering.

Author: fengmk2

crates/vite_global_cli/src/cli.rs

@@ -718,13 +718,6 @@ pub enum EnvSubcommands {
         command: Vec<String>,
     },
 
-    /// List installed global packages
-    Packages {
-        /// Output as JSON
-        #[arg(long)]
-        json: bool,
-    },
-
     /// Uninstall a Node.js version
     #[command(alias = "uni")]
     Uninstall {

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

@@ -14,7 +14,7 @@ mod list_remote;
 mod off;
 mod on;
 pub mod package_metadata;
-mod packages;
+pub mod packages;
 mod pin;
 mod run;
 mod setup;
@@ -56,7 +56,6 @@ pub async fn execute(cwd: AbsolutePathBuf, args: EnvArgs) -> Result<ExitStatus,
             crate::cli::EnvSubcommands::Run { node, npm, command } => {
                 run::execute(node.as_deref(), npm.as_deref(), &command).await
             }
-            crate::cli::EnvSubcommands::Packages { json } => packages::execute(json).await,
             crate::cli::EnvSubcommands::Uninstall { version } => {
                 let provider = vite_js_runtime::NodeProvider::new();
                 let resolved = config::resolve_version_alias(&version, &provider).await?;
@@ -135,7 +134,6 @@ fn print_help() {
     println!("  list-remote [PAT]  List available Node.js versions from the registry");
     println!("  use [VERSION]      Use a Node.js version for this shell session");
     println!("  run [--node <VER>] Run a command (--node optional for shim tools)");
-    println!("  packages           List installed global packages");
     println!("  install [VERSION]  Install a Node.js version (reads project config if omitted)");
     println!("  uninstall <VERSION>  Uninstall a Node.js version");
     println!();
@@ -176,6 +174,7 @@ fn print_help() {
     println!("  vp install -g <package>       # Install a global package");
     println!("  vp uninstall -g <package>     # Uninstall a global package");
     println!("  vp update -g [package]        # Update global package(s)");
+    println!("  vp list -g [package]          # List installed global packages");
 }
 
 /// Print shell snippet for setting environment (--print flag)

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

@@ -6,12 +6,23 @@ use super::package_metadata::PackageMetadata;
 use crate::error::Error;
 
 /// Execute the packages command.
-pub async fn execute(json: bool) -> Result<ExitStatus, Error> {
-    let packages = PackageMetadata::list_all().await?;
+pub async fn execute(json: bool, pattern: Option<&str>) -> Result<ExitStatus, Error> {
+    let all_packages = PackageMetadata::list_all().await?;
+
+    let packages: Vec<_> = if let Some(pat) = pattern {
+        let pat_lower = pat.to_lowercase();
+        all_packages.into_iter().filter(|p| p.name.to_lowercase().contains(&pat_lower)).collect()
+    } else {
+        all_packages
+    };
 
     if packages.is_empty() {
         if json {
             println!("[]");
+        } else if pattern.is_some() {
+            println!("No global packages matching '{}'.", pattern.unwrap());
+            println!();
+            println!("Run 'vp list -g' to see all installed global packages.");
         } else {
             println!("No global packages installed.");
             println!();

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

@@ -34,7 +34,7 @@ pub async fn execute(cwd: AbsolutePathBuf, tool: &str) -> Result<ExitStatus, Err
     // Unknown tool
     eprintln!("vp: Unknown tool '{tool}'");
     eprintln!("Not a core tool (node, npm, npx) and not found in any installed global package.");
-    eprintln!("Run 'vp env packages' to see installed global packages.");
+    eprintln!("Run 'vp list -g' to see installed global packages.");
     Ok(exit_status(1))
 }
 

crates/vite_global_cli/src/commands/pm.rs

@@ -44,6 +44,11 @@ pub async fn execute_pm_subcommand(
     cwd: AbsolutePathBuf,
     command: PmCommands,
 ) -> Result<ExitStatus, Error> {
+    // Intercept `pm list -g` to use vite-plus managed global packages listing
+    if let PmCommands::List { global: true, json, ref pattern, .. } = command {
+        return crate::commands::env::packages::execute(json, pattern.as_deref()).await;
+    }
+
     prepend_js_runtime_to_path_env(&cwd).await?;
 
     let package_manager = PackageManager::builder(&cwd).build_with_default().await?;

crates/vite_global_cli/src/main.rs

@@ -17,21 +17,30 @@ use std::process::ExitCode;
 
 use crate::cli::{parse_args_from, run_command};
 
-/// Normalize help arguments: transform `help [command]` into `[command] --help`
-fn normalize_help_args() -> Vec<String> {
-    let args: Vec<String> = std::env::args().collect();
-
-    // Skip the binary name (args[0])
+/// Normalize CLI arguments:
+/// - `vp list ...` / `vp ls ...` → `vp pm list ...`
+/// - `vp help [command]` → `vp [command] --help`
+fn normalize_args(args: Vec<String>) -> Vec<String> {
     match args.get(1).map(String::as_str) {
+        // `vp list ...` → `vp pm list ...`
+        // `vp ls ...` → `vp pm list ...`
+        Some("list" | "ls") => {
+            let mut normalized = Vec::with_capacity(args.len() + 1);
+            normalized.push(args[0].clone());
+            normalized.push("pm".to_string());
+            normalized.push("list".to_string());
+            normalized.extend(args[2..].iter().cloned());
+            normalized
+        }
         // `vp help` alone -> show main help
         Some("help") if args.len() == 2 => vec![args[0].clone(), "--help".to_string()],
         // `vp help [command] [args...]` -> `vp [command] --help [args...]`
         Some("help") if args.len() > 2 => {
             let mut normalized = Vec::with_capacity(args.len());
-            normalized.push(args[0].clone()); // binary name
-            normalized.push(args[2].clone()); // command
+            normalized.push(args[0].clone());
+            normalized.push(args[2].clone());
             normalized.push("--help".to_string());
-            normalized.extend(args[3..].iter().cloned()); // remaining args
+            normalized.extend(args[3..].iter().cloned());
             normalized
         }
         // No transformation needed
@@ -64,8 +73,8 @@ async fn main() -> ExitCode {
         }
     };
 
-    // Normalize help arguments: transform `help [command]` into `[command] --help`
-    let normalized_args = normalize_help_args();
+    // Normalize arguments (list/ls aliases, help rewriting)
+    let normalized_args = normalize_args(args);
 
     // Parse CLI arguments (using custom help formatting)
     let args = parse_args_from(normalized_args);

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

@@ -35,4 +35,4 @@ added 41 packages in <variable>ms
 [1]> vp env which unknown-tool # Unknown tool - error message
 vp: Unknown tool 'unknown-tool'
 Not a core tool (node, npm, npx) and not found in any installed global package.
-Run 'vp env packages' to see installed global packages.
+Run 'vp list -g' to see installed global packages.

rfcs/env-command.md

@@ -167,8 +167,8 @@ vp install -g --node lts typescript
 vp install -g --force eslint-v9    # Removes 'eslint' if it provides same binary
 
 # List installed global packages
-vp env packages
-vp env packages --json
+vp list -g
+vp list -g --json
 
 # Uninstall a global package
 vp remove -g typescript
@@ -1240,7 +1240,7 @@ $ vp env which eslint
 $ vp env which unknown-tool
 vp: Unknown tool 'unknown-tool'
 Not a core tool (node, npm, npx) and not found in any installed global package.
-Run 'vp env packages' to see installed global packages.
+Run 'vp list -g' to see installed global packages.
 ```
 
 ## Pin Command
@@ -2021,7 +2021,7 @@ env-doctor/
 5. Implement `vp install -g` / `vp remove -g` / `vp update -g` for managed global packages
 6. Implement package metadata storage
 7. Implement per-package binary shims
-8. Implement `vp env packages` to list installed global packages
+8. Implement `vp list -g` / `vp pm list -g` to list installed global packages
 9. Implement `vp env install <VERSION>` to install Node.js versions
 10. Implement `vp env uninstall <VERSION>` to uninstall Node.js versions
 11. Implement per-binary config files (`bins/`) for conflict detection