fix(cli): suppress VITE+ header for lint/fmt LSP and stdin flows (#1619)

## Summary

The cosmetic `VITE+ - The Unified Toolchain for the Web` banner is
printed to stdout before `vp lint` / `vp fmt` delegate to oxlint/oxfmt.
For IDE-launched subprocesses (`--lsp`, `--stdin-filepath`) the editor
parses stdout as the LSP protocol or as formatted source, so the banner
corrupts the stream.

Skip the banner exactly when:
- `vp lint --lsp`
- `vp fmt --lsp`
- `vp fmt --stdin-filepath` (also `--stdin-filepath=<path>`)

All other invocations of `vp lint` / `vp fmt` are unchanged and remain
subject to the existing TTY / `GIT_INDEX_FILE` suppressions in
`vite_shared::header::should_print_header()`.

Implementation lives in `crates/vite_global_cli/src/cli.rs`:
- New `should_suppress_header_for_subcommand("lint"|"fmt", &args)`
predicate, mirroring the existing `should_force_global_delegate` shape
and reusing `has_flag_before_terminator` (so `--lsp=...`,
`--stdin-filepath=...`, and the `--` option terminator are all handled
correctly).
- New `maybe_print_runtime_header(command, args, show_header)` wrapper
around `print_runtime_header`, used at the two call sites.

Refs #1557 (first checkbox; the bin-wrapper deprecation and
editor-extension updates remain).

## Test plan

- [x] `cargo test -p vite_global_cli suppresses_header` — 2 positive
tests pass
- [x] `cargo test -p vite_global_cli does_not_suppress` — 4 negative
tests pass (incl. `--` option terminator, unknown subcommand, fmt
`--check`)
- [x] `cargo fmt -p vite_global_cli -- --check` clean
- [x] `cargo clippy -p vite_global_cli --tests` clean
- [ ] Manual smoke (in a Vite+ project, attached TTY):
  - [ ] `vp lint` → banner printed
  - [ ] `vp lint --lsp` → no banner; stdout is pure LSP protocol
  - [ ] `vp fmt` → banner printed
  - [ ] `vp fmt --lsp` → no banner
- [ ] `echo 'const a=1' | vp fmt --stdin-filepath foo.ts` → no banner;
stdout is just formatted source
  - [ ] `echo 'const a=1' | vp fmt --stdin-filepath=foo.ts` → same
  - [ ] `vp check` (composite) → unaffected

Author: fengmk2

crates/vite_global_cli/src/cli.rs

@@ -468,6 +468,22 @@ fn should_force_global_delegate(command: &str, args: &[String]) -> bool {
     }
 }
 
+/// Whether the Vite+ banner should be suppressed for a lint/fmt invocation.
+///
+/// IDE extensions invoke `vp lint --lsp`, `vp fmt --lsp`, and
+/// `vp fmt --stdin-filepath` and parse the subprocess's stdout as the LSP
+/// protocol / formatted source; the cosmetic banner would corrupt that stream.
+fn should_suppress_header_for_subcommand(command: &str, args: &[String]) -> bool {
+    match command {
+        "lint" => has_flag_before_terminator(args, "--lsp"),
+        "fmt" => {
+            has_flag_before_terminator(args, "--lsp")
+                || has_flag_before_terminator(args, "--stdin-filepath")
+        }
+        _ => false,
+    }
+}
+
 /// Get available tasks for shell completion.
 ///
 /// Delegates to the local vite-plus CLI to run `vp run` without arguments,
@@ -777,7 +793,7 @@ pub async fn run_command_with_options(
             if help::maybe_print_unified_delegate_help("lint", &args, render_options.show_header) {
                 return Ok(ExitStatus::default());
             }
-            print_runtime_header(render_options.show_header);
+            maybe_print_runtime_header("lint", &args, render_options.show_header);
             if should_force_global_delegate("lint", &args) {
                 commands::delegate::execute_global(cwd, "lint", &args).await
             } else {
@@ -789,7 +805,7 @@ pub async fn run_command_with_options(
             if help::maybe_print_unified_delegate_help("fmt", &args, render_options.show_header) {
                 return Ok(ExitStatus::default());
             }
-            print_runtime_header(render_options.show_header);
+            maybe_print_runtime_header("fmt", &args, render_options.show_header);
             if should_force_global_delegate("fmt", &args) {
                 commands::delegate::execute_global(cwd, "fmt", &args).await
             } else {
@@ -886,6 +902,13 @@ fn print_runtime_header(show_header: bool) {
     vite_shared::header::print_header();
 }
 
+fn maybe_print_runtime_header(command: &str, args: &[String], show_header: bool) {
+    if should_suppress_header_for_subcommand(command, args) {
+        return;
+    }
+    print_runtime_header(show_header);
+}
+
 /// Build a clap Command with custom help formatting matching the JS CLI output.
 pub fn command_with_help() -> clap::Command {
     command_with_help_with_options(RenderOptions::default())
@@ -933,6 +956,7 @@ pub fn try_parse_args_from_with_options(
 mod tests {
     use super::{
         has_flag_before_terminator, is_global_package_up_to_date, should_force_global_delegate,
+        should_suppress_header_for_subcommand,
     };
 
     #[test]
@@ -981,4 +1005,53 @@ mod tests {
         assert!(!should_force_global_delegate("lint", &["src/index.ts".to_string()]));
         assert!(!should_force_global_delegate("fmt", &["--check".to_string()]));
     }
+
+    #[test]
+    fn lint_lsp_suppresses_header() {
+        assert!(should_suppress_header_for_subcommand("lint", &["--lsp".to_string()]));
+        assert!(should_suppress_header_for_subcommand(
+            "lint",
+            &["--fix".to_string(), "--lsp".to_string()]
+        ));
+    }
+
+    #[test]
+    fn lint_without_lsp_does_not_suppress_header() {
+        assert!(!should_suppress_header_for_subcommand("lint", &[]));
+        assert!(!should_suppress_header_for_subcommand("lint", &["src".to_string()]));
+        assert!(!should_suppress_header_for_subcommand("lint", &["--fix".to_string()]));
+    }
+
+    #[test]
+    fn lint_lsp_after_terminator_does_not_suppress_header() {
+        assert!(!should_suppress_header_for_subcommand(
+            "lint",
+            &["--".to_string(), "--lsp".to_string()]
+        ));
+    }
+
+    #[test]
+    fn fmt_lsp_or_stdin_filepath_suppresses_header() {
+        assert!(should_suppress_header_for_subcommand("fmt", &["--lsp".to_string()]));
+        assert!(should_suppress_header_for_subcommand(
+            "fmt",
+            &["--stdin-filepath".to_string(), "foo.ts".to_string()]
+        ));
+        assert!(should_suppress_header_for_subcommand(
+            "fmt",
+            &["--stdin-filepath=foo.ts".to_string()]
+        ));
+    }
+
+    #[test]
+    fn fmt_without_lsp_or_stdin_does_not_suppress_header() {
+        assert!(!should_suppress_header_for_subcommand("fmt", &[]));
+        assert!(!should_suppress_header_for_subcommand("fmt", &["src".to_string()]));
+        assert!(!should_suppress_header_for_subcommand("fmt", &["--check".to_string()]));
+    }
+
+    #[test]
+    fn unknown_subcommand_does_not_suppress_header() {
+        assert!(!should_suppress_header_for_subcommand("test", &["--lsp".to_string()]));
+    }
 }