chore: add cargo fmt and clippy to pre-commit hooks

fix: apply clippy suggestions (search_is_some in tests, fmt in main.rs)

Assisted-by: Claude Code (Claude Sonnet 4.6)

Author: jdc-pub

.config/pre-commit.yaml

@@ -19,6 +19,20 @@ repos:
     hooks:
       - id: typos
         args: [--write-changes, --force-exclude, --config, .config/typos.toml]
+  - repo: local
+    hooks:
+      - id: fmt
+        name: cargo fmt
+        language: system
+        entry: cargo fmt --all --
+        types: [rust]
+        pass_filenames: false
+      - id: clippy
+        name: cargo clippy
+        language: system
+        entry: cargo clippy --all-targets --all-features -- -D warnings
+        types: [rust]
+        pass_filenames: false
   - repo: https://github.com/crate-ci/committed
     rev: v1.1.10
     hooks:

crates/git-ledger/src/tests.rs

@@ -175,8 +175,8 @@ fn update_delete_field() {
         )
         .unwrap();
 
-    assert!(updated.fields.iter().find(|(k, _)| k == "status").is_none());
-    assert!(updated.fields.iter().find(|(k, _)| k == "title").is_some());
+    assert!(!updated.fields.iter().any(|(k, _)| k == "status"));
+    assert!(updated.fields.iter().any(|(k, _)| k == "title"));
 }
 
 #[test]
@@ -295,15 +295,9 @@ fn delete_nested_field() {
         .unwrap();
 
     // The nested field should be gone
-    assert!(
-        updated
-            .fields
-            .iter()
-            .find(|(k, _)| k == "meta/priority")
-            .is_none()
-    );
+    assert!(!updated.fields.iter().any(|(k, _)| k == "meta/priority"));
     // The non-nested field should remain
-    assert!(updated.fields.iter().find(|(k, _)| k == "title").is_some());
+    assert!(updated.fields.iter().any(|(k, _)| k == "title"));
 }
 
 #[test]

crates/git-metadata/src/main.rs

@@ -91,9 +91,10 @@ fn run(cli: &Cli) -> Result<(), Box<dyn std::error::Error>> {
 
             if !allow_empty
                 && let Some(ref c) = content
-                    && c.is_empty() {
-                        return Err("refusing to add empty content (use --allow-empty)".into());
-                    }
+                && c.is_empty()
+            {
+                return Err("refusing to add empty content (use --allow-empty)".into());
+            }
 
             let opts = MetadataOptions {
                 shard_level: *shard_level,