fix(global): prevent finding package.json for global outdated and why commands

crates/vite_install/src/commands/why.rs

@@ -180,6 +180,10 @@ impl PackageManager {
                     args.push("--json".into());
                 }
 
+                if options.global {
+                    args.push("--global".into());
+                }
+
                 // Add packages (npm supports multiple packages)
                 args.extend_from_slice(options.packages);
 
@@ -354,6 +358,19 @@ mod tests {
         assert_eq!(result.args, vec!["explain", "--workspace", "app", "react"]);
     }
 
+    #[test]
+    fn test_npm_explain_global() {
+        let pm = create_mock_package_manager(PackageManagerType::Npm, "11.0.0");
+        let packages = vec!["react".to_string()];
+        let result = pm.resolve_why_command(&WhyCommandOptions {
+            packages: &packages,
+            global: true,
+            ..Default::default()
+        });
+        assert_eq!(result.bin_path, "npm");
+        assert_eq!(result.args, vec!["explain", "--global", "react"]);
+    }
+
     #[test]
     fn test_yarn_why_basic() {
         let pm = create_mock_package_manager(PackageManagerType::Yarn, "4.0.0");

crates/vite_pm_cli/src/handlers.rs

@@ -46,7 +46,10 @@ use crate::{
         ConfigCommands, DistTagCommands, OwnerCommands, PluginCommands, PmCommands, TokenCommands,
     },
     error::Error,
-    helpers::{build_package_manager, build_package_manager_or_npm_default, ensure_package_json},
+    helpers::{
+        build_package_manager, build_package_manager_or_npm_default, default_npm_package_manager,
+        ensure_package_json,
+    },
 };
 
 pub async fn run_add(
@@ -95,15 +98,23 @@ pub async fn run_outdated(
     cwd: &AbsolutePath,
     options: &OutdatedCommandOptions<'_>,
 ) -> Result<ExitStatus, Error> {
-    let pm = build_package_manager(cwd).await?;
+    let pm = if options.global {
+        default_npm_package_manager(cwd)
+    } else {
+        build_package_manager(cwd).await?
+    };
     Ok(pm.run_outdated_command(options, cwd).await?)
 }
 
 pub async fn run_why(
     cwd: &AbsolutePath,
     options: &WhyCommandOptions<'_>,
 ) -> Result<ExitStatus, Error> {
-    let pm = build_package_manager(cwd).await?;
+    let pm = if options.global {
+        default_npm_package_manager(cwd)
+    } else {
+        build_package_manager(cwd).await?
+    };
     Ok(pm.run_why_command(options, cwd).await?)
 }
 

crates/vite_pm_cli/src/helpers.rs

@@ -37,7 +37,7 @@ pub async fn build_package_manager_or_npm_default(
     }
 }
 
-fn default_npm_package_manager(cwd: &AbsolutePath) -> PackageManager {
+pub(crate) fn default_npm_package_manager(cwd: &AbsolutePath) -> PackageManager {
     PackageManager {
         client: PackageManagerType::Npm,
         package_name: "npm".into(),

packages/cli/snap-tests/command-pm-no-package-json/snap.txt

@@ -9,3 +9,16 @@ error: No package.json found.
 
 [1]> vp why lodash # should show friendly error
 error: No package.json found.
+
+> vp outdated definitely-not-installed-vite-plus-snap-pkg -g --format json # should not require package.json for global outdated
+{}
+
+[1]> vp why definitely-not-installed-vite-plus-snap-pkg -g --json # should not require package.json for global why
+npm error No dependencies found matching definitely-not-installed-vite-plus-snap-pkg
+{
+  "error": {
+    "summary": "No dependencies found matching definitely-not-installed-vite-plus-snap-pkg",
+    "detail": ""
+  }
+}
+npm error A complete log of this run can be found in: <homedir>/.npm/_logs/<timestamp>-debug.log

packages/cli/snap-tests/command-pm-no-package-json/steps.json

@@ -3,6 +3,8 @@
     "vp pm ls # should show friendly error",
     "vp pm prune # should show friendly error",
     "vp outdated # should show friendly error",
-    "vp why lodash # should show friendly error"
+    "vp why lodash # should show friendly error",
+    "vp outdated definitely-not-installed-vite-plus-snap-pkg -g --format json # should not require package.json for global outdated",
+    "vp why definitely-not-installed-vite-plus-snap-pkg -g --json # should not require package.json for global why"
   ]
 }