fix(install): address PR review comments for bun commands

- view.rs: use native `bun info` instead of npm fallback
- dlx.rs: forward `--package` flag to `bun x` (before package spec)
- install.rs: forward `--lockfile-only` to bun
- outdated.rs: forward `--production` and `--omit optional` to bun
- login/logout/owner/ping/search/token: remove unnecessary warning
  logs when falling back to npm (the fallback itself is correct)

Author: fengmk2

crates/vite_install/src/commands/dlx.rs

@@ -200,16 +200,18 @@ impl PackageManager {
         // Some installation methods (e.g. mise) don't add bunx to PATH on Windows.
         args.push("x".into());
 
+        // --packages flags must come before the package spec
+        for pkg in options.packages {
+            args.push("--package".into());
+            args.push(pkg.clone());
+        }
+
         // Add package spec
         args.push(options.package_spec.into());
 
         // Add command arguments
         args.extend(options.args.iter().cloned());
 
-        // Warn about unsupported flags
-        if !options.packages.is_empty() {
-            output::warn("bun x does not support --package");
-        }
         if options.shell_mode {
             output::warn("bun x does not support shell mode (-c)");
         }

crates/vite_install/src/commands/install.rs

@@ -317,7 +317,7 @@ impl PackageManager {
                     args.push("--ignore-scripts".into());
                 }
                 if options.lockfile_only {
-                    output::warn("bun does not support --lockfile-only");
+                    args.push("--lockfile-only".into());
                 }
                 if options.prefer_offline {
                     output::warn("bun does not support --prefer-offline");

crates/vite_install/src/commands/login.rs

@@ -3,7 +3,6 @@ use std::{collections::HashMap, process::ExitStatus};
 use vite_command::run_command;
 use vite_error::Error;
 use vite_path::AbsolutePath;
-use vite_shared::output;
 
 use crate::package_manager::{
     PackageManager, PackageManagerType, ResolveCommandResult, format_path_env,
@@ -57,7 +56,6 @@ impl PackageManager {
                 }
             }
             PackageManagerType::Bun => {
-                output::warn("bun does not have a login command, falling back to npm login");
                 bin_name = "npm".into();
                 args.push("login".into());
             }

crates/vite_install/src/commands/logout.rs

@@ -3,7 +3,6 @@ use std::{collections::HashMap, process::ExitStatus};
 use vite_command::run_command;
 use vite_error::Error;
 use vite_path::AbsolutePath;
-use vite_shared::output;
 
 use crate::package_manager::{
     PackageManager, PackageManagerType, ResolveCommandResult, format_path_env,
@@ -57,7 +56,6 @@ impl PackageManager {
                 }
             }
             PackageManagerType::Bun => {
-                output::warn("bun does not have a logout command, falling back to npm logout");
                 bin_name = "npm".into();
                 args.push("logout".into());
             }

crates/vite_install/src/commands/outdated.rs

@@ -249,11 +249,15 @@ impl PackageManager {
                     if options.workspace_root {
                         output::warn("bun outdated does not support --workspace-root");
                     }
-                    if options.prod || options.dev {
-                        output::warn("bun outdated does not support --prod/--dev");
+                    if options.prod {
+                        args.push("--production".into());
+                    }
+                    if options.dev {
+                        output::warn("bun outdated does not support --dev");
                     }
                     if options.no_optional {
-                        output::warn("bun outdated does not support --no-optional");
+                        args.push("--omit".into());
+                        args.push("optional".into());
                     }
                     if options.compatible {
                         output::warn("bun outdated does not support --compatible");

crates/vite_install/src/commands/owner.rs

@@ -3,11 +3,8 @@ use std::{collections::HashMap, process::ExitStatus};
 use vite_command::run_command;
 use vite_error::Error;
 use vite_path::AbsolutePath;
-use vite_shared::output;
 
-use crate::package_manager::{
-    PackageManager, PackageManagerType, ResolveCommandResult, format_path_env,
-};
+use crate::package_manager::{PackageManager, ResolveCommandResult, format_path_env};
 
 /// Owner subcommand type.
 #[derive(Debug, Clone)]
@@ -39,10 +36,6 @@ impl PackageManager {
         let envs = HashMap::from([("PATH".to_string(), format_path_env(self.get_bin_prefix()))]);
         let mut args: Vec<String> = Vec::new();
 
-        if self.client == PackageManagerType::Bun {
-            output::warn("bun does not support the owner command, falling back to npm owner");
-        }
-
         args.push("owner".into());
 
         match subcommand {

crates/vite_install/src/commands/ping.rs

@@ -3,11 +3,8 @@ use std::{collections::HashMap, process::ExitStatus};
 use vite_command::run_command;
 use vite_error::Error;
 use vite_path::AbsolutePath;
-use vite_shared::output;
 
-use crate::package_manager::{
-    PackageManager, PackageManagerType, ResolveCommandResult, format_path_env,
-};
+use crate::package_manager::{PackageManager, ResolveCommandResult, format_path_env};
 
 /// Options for the ping command.
 #[derive(Debug, Default)]
@@ -38,10 +35,6 @@ impl PackageManager {
         let envs = HashMap::from([("PATH".to_string(), format_path_env(self.get_bin_prefix()))]);
         let mut args: Vec<String> = Vec::new();
 
-        if self.client == PackageManagerType::Bun {
-            output::warn("bun does not support the ping command, falling back to npm ping");
-        }
-
         args.push("ping".into());
 
         if let Some(registry_value) = options.registry {

crates/vite_install/src/commands/search.rs

@@ -3,11 +3,8 @@ use std::{collections::HashMap, process::ExitStatus};
 use vite_command::run_command;
 use vite_error::Error;
 use vite_path::AbsolutePath;
-use vite_shared::output;
 
-use crate::package_manager::{
-    PackageManager, PackageManagerType, ResolveCommandResult, format_path_env,
-};
+use crate::package_manager::{PackageManager, ResolveCommandResult, format_path_env};
 
 /// Options for the search command.
 #[derive(Debug, Default)]
@@ -41,10 +38,6 @@ impl PackageManager {
         let envs = HashMap::from([("PATH".to_string(), format_path_env(self.get_bin_prefix()))]);
         let mut args: Vec<String> = Vec::new();
 
-        if self.client == PackageManagerType::Bun {
-            output::warn("bun does not support the search command, falling back to npm search");
-        }
-
         args.push("search".into());
 
         for term in options.terms {

crates/vite_install/src/commands/token.rs

@@ -3,11 +3,8 @@ use std::{collections::HashMap, process::ExitStatus};
 use vite_command::run_command;
 use vite_error::Error;
 use vite_path::AbsolutePath;
-use vite_shared::output;
 
-use crate::package_manager::{
-    PackageManager, PackageManagerType, ResolveCommandResult, format_path_env,
-};
+use crate::package_manager::{PackageManager, ResolveCommandResult, format_path_env};
 
 /// Token subcommand type.
 #[derive(Debug, Clone)]
@@ -53,10 +50,6 @@ impl PackageManager {
         let envs = HashMap::from([("PATH".to_string(), format_path_env(self.get_bin_prefix()))]);
         let mut args: Vec<String> = Vec::new();
 
-        if self.client == PackageManagerType::Bun {
-            output::warn("bun does not support the token command, falling back to npm token");
-        }
-
         args.push("token".into());
 
         match subcommand {

crates/vite_install/src/commands/view.rs

@@ -3,7 +3,6 @@ use std::{collections::HashMap, process::ExitStatus};
 use vite_command::run_command;
 use vite_error::Error;
 use vite_path::AbsolutePath;
-use vite_shared::output;
 
 use crate::package_manager::{
     PackageManager, PackageManagerType, ResolveCommandResult, format_path_env,
@@ -33,28 +32,39 @@ impl PackageManager {
 
     /// Resolve the view command.
     /// All package managers delegate to npm view (pnpm and yarn use npm internally).
-    /// Bun does not have a direct equivalent, so we fall back to npm view.
+    /// Bun uses `bun info` as a native alternative.
     #[must_use]
     pub fn resolve_view_command(&self, options: &ViewCommandOptions) -> ResolveCommandResult {
-        let bin_name: String = "npm".to_string();
         let envs = HashMap::from([("PATH".to_string(), format_path_env(self.get_bin_prefix()))]);
         let mut args: Vec<String> = Vec::new();
 
-        if self.client == PackageManagerType::Bun {
-            output::warn("bun does not have a view command, falling back to npm view");
-        }
+        let bin_name: String = if self.client == PackageManagerType::Bun {
+            args.push("info".into());
+            args.push(options.package.to_string());
 
-        args.push("view".into());
+            if let Some(field) = options.field {
+                args.push(field.to_string());
+            }
 
-        args.push(options.package.to_string());
+            if options.json {
+                args.push("--json".into());
+            }
 
-        if let Some(field) = options.field {
-            args.push(field.to_string());
-        }
+            "bun".into()
+        } else {
+            args.push("view".into());
+            args.push(options.package.to_string());
 
-        if options.json {
-            args.push("--json".into());
-        }
+            if let Some(field) = options.field {
+                args.push(field.to_string());
+            }
+
+            if options.json {
+                args.push("--json".into());
+            }
+
+            "npm".into()
+        };
 
         // Add pass-through args
         if let Some(pass_through_args) = options.pass_through_args {