Replace serde-yaml with serde-saphyr (again) (#1520)

serde-saphyr has way better error message, and it is written in safe
Rust.

Previous try: #1087
Previous revert: #1106 and #1112

TODO:
Replace serialize_yaml_scalar with serde-saphyr as well:


https://github.com/j178/prek/blob/0a4be22599076b8c6273940b2760828b442c78f4/crates/prek/src/yaml.rs#L12-L13

Author: j178

Cargo.lock

@@ -75,7 +75,7 @@ semver = { version = "1.0.24", features = ["serde"] }
 serde = { version = "1.0.210", features = ["derive"] }
 serde_json = { version = "1.0.132", features = ["unbounded_depth"] }
 serde_stacker = { version = "0.1.12" }
-serde_yaml = { version = "0.9.34" }
+serde-saphyr = { version = "0.0.17", default-features = false }
 shlex = { version = "1.3.0" }
 globset = { version = "0.4.18" }
 smallvec = { version = "1.15.1" }
@@ -136,6 +136,7 @@ similar_names = "allow"
 too_many_arguments = "allow"
 too_many_lines = "allow"
 used_underscore_binding = "allow"
+items_after_statements = "allow"
 # Disallowed restriction lints
 print_stdout = "warn"
 print_stderr = "warn"

Cargo.toml

@@ -77,7 +77,7 @@ semver = { workspace = true, features = ["serde"] }
 serde = { workspace = true, features = ["derive"] }
 serde_json = { workspace = true, features = ["unbounded_depth"] }
 serde_stacker = { workspace = true }
-serde_yaml = { workspace = true }
+serde-saphyr = { workspace = true, default-features = false }
 shlex = { workspace = true }
 smallvec = { workspace = true }
 target-lexicon = { workspace = true }

crates/prek/Cargo.toml

@@ -381,10 +381,10 @@ async fn collect_files_from_args(
 #[cfg(test)]
 mod tests {
     use super::*;
+    use crate::config::GlobPatterns;
 
     fn glob_pattern(pattern: &str) -> FilePattern {
-        serde_yaml::from_str::<FilePattern>(&format!("glob: {pattern}"))
-            .expect("glob pattern should deserialize")
+        FilePattern::Glob(GlobPatterns::new(vec![pattern.to_string()]).unwrap())
     }
 
     #[test]