chore: configure clippy rules for non-vite crates via .non-vite.clippy.toml

Non-vite crates previously used blanket `#![allow(clippy::disallowed_types,
disallowed_methods, disallowed_macros)]` to opt out of vite_str/vite_path rules,
but this also silenced generic rules like HashMap→FxHashMap and cow_utils.

Replace the blanket allows with a dedicated `.non-vite.clippy.toml` that keeps
generic rules but omits vite-specific ones. Each non-vite crate gets a symlinked
`.clippy.toml` pointing to the shared config.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: wan9chi

.non-vite.clippy.toml

@@ -0,0 +1,24 @@
+# Clippy configuration for non-vite crates (fspy_*, subprocess_test, vite_pty, etc.)
+# that don't depend on vite_str/vite_path.
+#
+# This is a subset of the root .clippy.toml, with rules recommending vite_str/vite_path
+# alternatives removed. Generic rules (cow_utils, rustc-hash) still apply.
+#
+# To use: symlink as `.clippy.toml` in the crate's directory:
+#   ln -s ../../.non-vite.clippy.toml .clippy.toml
+
+avoid-breaking-exported-api = false
+
+disallowed-methods = [
+  { path = "str::to_ascii_lowercase", reason = "To avoid memory allocation, use `cow_utils::CowUtils::cow_to_ascii_lowercase` instead." },
+  { path = "str::to_ascii_uppercase", reason = "To avoid memory allocation, use `cow_utils::CowUtils::cow_to_ascii_uppercase` instead." },
+  { path = "str::to_lowercase", reason = "To avoid memory allocation, use `cow_utils::CowUtils::cow_to_lowercase` instead." },
+  { path = "str::to_uppercase", reason = "To avoid memory allocation, use `cow_utils::CowUtils::cow_to_uppercase` instead." },
+  { path = "str::replace", reason = "To avoid memory allocation, use `cow_utils::CowUtils::cow_replace` instead." },
+  { path = "str::replacen", reason = "To avoid memory allocation, use `cow_utils::CowUtils::cow_replacen` instead." },
+]
+
+disallowed-types = [
+  { path = "std::collections::HashMap", reason = "Use `rustc_hash::FxHashMap` instead, which is typically faster." },
+  { path = "std::collections::HashSet", reason = "Use `rustc_hash::FxHashSet` instead, which is typically faster." },
+]

Cargo.lock

@@ -0,0 +1 @@
+../../.non-vite.clippy.toml

crates/fspy/.clippy.toml

@@ -16,6 +16,7 @@ futures-util = { workspace = true }
 libc = { workspace = true }
 ouroboros = { workspace = true }
 rand = { workspace = true }
+rustc-hash = { workspace = true }
 tempfile = { workspace = true }
 thiserror = { workspace = true }
 tokio = { workspace = true, features = ["net", "process", "io-util", "sync", "rt"] }

crates/fspy/Cargo.toml

@@ -1,10 +1,3 @@
-#![allow(
-    clippy::disallowed_types,
-    clippy::disallowed_methods,
-    clippy::disallowed_macros,
-    reason = "non-vite crate"
-)]
-
 use std::{
     env::{self, current_dir},
     fs,

crates/fspy/build.rs

@@ -1,10 +1,3 @@
-#![allow(
-    clippy::disallowed_types,
-    clippy::disallowed_methods,
-    clippy::disallowed_macros,
-    reason = "non-vite crate"
-)]
-
 use std::{env::args_os, ffi::OsStr, path::PathBuf, pin::Pin};
 
 use tokio::{

crates/fspy/examples/cli.rs

@@ -1,12 +1,12 @@
 use std::{
-    collections::HashMap,
     ffi::{OsStr, OsString},
     path::{Path, PathBuf},
     process::Stdio,
 };
 
 #[cfg(unix)]
 use fspy_shared_unix::exec::Exec;
+use rustc_hash::FxHashMap;
 use tokio::process::Command as TokioCommand;
 
 use crate::{SPY_IMPL, TrackedChild, error::SpawnError};
@@ -15,7 +15,7 @@ use crate::{SPY_IMPL, TrackedChild, error::SpawnError};
 pub struct Command {
     program: OsString,
     args: Vec<OsString>,
-    envs: HashMap<OsString, OsString>,
+    envs: FxHashMap<OsString, OsString>,
     cwd: Option<PathBuf>,
     #[cfg(unix)]
     arg0: Option<OsString>,
@@ -37,7 +37,7 @@ impl Command {
         Self {
             program: program.as_ref().to_os_string(),
             args: Vec::new(),
-            envs: HashMap::new(),
+            envs: FxHashMap::default(),
             cwd: None,
             #[cfg(unix)]
             arg0: None,

crates/fspy/src/command.rs

@@ -1,11 +1,5 @@
 #![cfg_attr(target_os = "windows", feature(windows_process_extensions_main_thread_handle))]
 #![feature(once_cell_try)]
-#![allow(
-    clippy::disallowed_types,
-    clippy::disallowed_methods,
-    clippy::disallowed_macros,
-    reason = "non-vite crate"
-)]
 
 // Persist the injected DLL/shared library somewhere in the filesystem.
 mod artifact;

crates/fspy/src/lib.rs

@@ -1,10 +1,3 @@
-#![allow(
-    clippy::disallowed_types,
-    clippy::disallowed_methods,
-    clippy::disallowed_macros,
-    reason = "non-vite crate"
-)]
-
 mod test_utils;
 
 use std::{

crates/fspy/tests/node_fs.rs

@@ -1,10 +1,3 @@
-#![allow(
-    clippy::disallowed_types,
-    clippy::disallowed_methods,
-    clippy::disallowed_macros,
-    reason = "non-vite crate"
-)]
-
 mod test_utils;
 
 use std::{env::vars_os, ffi::OsString};