chore: remove dead global logic for update command (#1478)

Author: liangmiQwQ

crates/vite_global_cli/src/cli.rs

@@ -1840,7 +1840,6 @@ pub async fn run_command_with_options(
                 .execute(
                     &packages,
                     latest,
-                    global,
                     recursive,
                     filter.as_deref(),
                     workspace_root,

crates/vite_global_cli/src/commands/update.rs

@@ -24,7 +24,6 @@ impl UpdateCommand {
         self,
         packages: &[String],
         latest: bool,
-        global: bool,
         recursive: bool,
         filters: Option<&[String]>,
         workspace_root: bool,
@@ -43,7 +42,6 @@ impl UpdateCommand {
         let update_command_options = UpdateCommandOptions {
             packages,
             latest,
-            global,
             recursive,
             filters,
             workspace_root,

crates/vite_install/src/commands/update.rs

@@ -14,7 +14,6 @@ use crate::package_manager::{
 pub struct UpdateCommandOptions<'a> {
     pub packages: &'a [String],
     pub latest: bool,
-    pub global: bool,
     pub recursive: bool,
     pub filters: Option<&'a [String]>,
     pub workspace_root: bool,
@@ -48,19 +47,6 @@ impl PackageManager {
         let envs = HashMap::from([("PATH".to_string(), format_path_env(self.get_bin_prefix()))]);
         let mut args: Vec<String> = Vec::new();
 
-        // global packages should use npm cli only
-        if options.global {
-            bin_name = "npm".into();
-            args.push("update".into());
-            args.push("--global".into());
-            if let Some(pass_through_args) = options.pass_through_args {
-                args.extend_from_slice(pass_through_args);
-            }
-            args.extend_from_slice(options.packages);
-
-            return ResolveCommandResult { bin_path: bin_name, args, envs };
-        }
-
         match self.client {
             PackageManagerType::Pnpm => {
                 bin_name = "pnpm".into();
@@ -249,7 +235,6 @@ mod tests {
         let result = pm.resolve_update_command(&UpdateCommandOptions {
             packages: &["react".to_string()],
             latest: false,
-            global: false,
             recursive: false,
             filters: None,
             workspace_root: false,
@@ -539,18 +524,6 @@ mod tests {
         assert_eq!(result.bin_path, "npm");
     }
 
-    #[test]
-    fn test_global_update() {
-        let pm = create_mock_package_manager(PackageManagerType::Pnpm, "10.0.0");
-        let result = pm.resolve_update_command(&UpdateCommandOptions {
-            packages: &["typescript".to_string()],
-            global: true,
-            ..Default::default()
-        });
-        assert_eq!(result.args, vec!["update", "--global", "typescript"]);
-        assert_eq!(result.bin_path, "npm");
-    }
-
     #[test]
     fn test_pnpm_update_multiple_packages() {
         let pm = create_mock_package_manager(PackageManagerType::Pnpm, "10.0.0");