Fix clippy: expect attributes, map_or_else, safety comment

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: branchseer

crates/vite_task_bin/src/main.rs

@@ -13,6 +13,7 @@ fn main() -> ! {
         tokio::runtime::Builder::new_multi_thread().enable_all().build().unwrap().block_on(async {
             match run().await {
                 Ok(status) => i32::from(status.0),
+                #[expect(clippy::print_stderr, reason = "top-level error reporting")]
                 Err(err) => {
                     eprintln!("Error: {err:?}");
                     1

crates/vite_task_bin/tests/e2e_snapshots/main.rs

@@ -86,24 +86,26 @@ impl Step {
     fn spawn_mode(&self) -> StepSpawn<'_> {
         match self {
             Self::Command(command) => StepSpawn::Shell(command.as_str()),
-            Self::Detailed(config) => {
-                if let Some(argv) = &config.argv {
-                    StepSpawn::Direct(argv)
-                } else if let Some(command) = &config.command {
-                    StepSpawn::Shell(command.as_str())
-                } else {
-                    panic!("step must have either 'command' or 'argv'");
-                }
-            }
+            Self::Detailed(config) => config.argv.as_deref().map_or_else(
+                || {
+                    StepSpawn::Shell(
+                        config
+                            .command
+                            .as_ref()
+                            .expect("step must have either 'command' or 'argv'")
+                            .as_str(),
+                    )
+                },
+                StepSpawn::Direct,
+            ),
         }
     }
 
+    #[expect(clippy::disallowed_types, reason = "String required by join")]
     fn display_command(&self) -> String {
         match self.spawn_mode() {
             StepSpawn::Shell(cmd) => cmd.to_string(),
-            StepSpawn::Direct(argv) => {
-                argv.iter().map(|a| a.as_str()).collect::<Vec<_>>().join(" ")
-            }
+            StepSpawn::Direct(argv) => argv.iter().map(Str::as_str).collect::<Vec<_>>().join(" "),
         }
     }
 
@@ -467,12 +469,12 @@ fn run_case_inner(tmpdir: &AbsolutePath, fixture_path: &std::path::Path, fixture
 }
 
 fn main() {
-    // SAFETY: Called before any threads are spawned; insta reads this lazily on first assertion.
     // Skip INSTA_REQUIRE_FULL_MATCH when running cross-platform (via cargo-xtest): the snapshot
     // source metadata changes when the test binary runs on a remote machine, causing insta to
     // write .snap.new even when content matches. INSTA_UPDATE=always in this mode accepts
     // metadata-only changes.
     if std::env::var_os("INSTA_UPDATE").is_none() {
+        // SAFETY: Called before any threads are spawned; insta reads this lazily on first assertion.
         unsafe { std::env::set_var("INSTA_REQUIRE_FULL_MATCH", "1") };
     }