chore: add pre-commit fmt+clippy hooks, fix clippy warnings

.claude/settings.json

@@ -0,0 +1,18 @@
+{
+  "hooks": {
+    "PreToolUse": [
+      {
+        "matcher": "Bash",
+        "hooks": [
+          {
+            "type": "command",
+            "if": "Bash(git commit*)",
+            "command": "cargo fmt --all 2>&1 && cargo clippy --workspace 2>&1 || echo '{\"continue\": false, \"stopReason\": \"cargo fmt or clippy failed — fix before committing\"}'",
+            "timeout": 120,
+            "statusMessage": "Running cargo fmt + clippy..."
+          }
+        ]
+      }
+    ]
+  }
+}

crates/adapters/activitypub-base/src/activities.rs

@@ -241,12 +241,12 @@ impl Activity for UndoActivity {
 
     async fn verify(&self, _data: &Data<Self::DataType>) -> Result<(), Self::Error> {
         // The actor undoing must be the same as the actor in the wrapped activity.
-        if let Some(inner_actor) = self.object.get("actor").and_then(|v| v.as_str()) {
-            if inner_actor != self.actor.inner().as_str() {
-                return Err(Error::bad_request(anyhow::anyhow!(
-                    "Undo actor does not match inner activity actor"
-                )));
-            }
+        if let Some(inner_actor) = self.object.get("actor").and_then(|v| v.as_str())
+            && inner_actor != self.actor.inner().as_str()
+        {
+            return Err(Error::bad_request(anyhow::anyhow!(
+                "Undo actor does not match inner activity actor"
+            )));
         }
         Ok(())
     }

crates/application/src/use_cases/thoughts/mod.rs

@@ -135,7 +135,9 @@ pub async fn get_thought_view(
         .find_by_id(&thought.user_id)
         .await?
         .ok_or(DomainError::NotFound)?;
-    let mut map = engagement.get_for_thoughts(&[id.clone()], viewer).await?;
+    let mut map = engagement
+        .get_for_thoughts(std::slice::from_ref(id), viewer)
+        .await?;
     let (stats, viewer_ctx) = map.remove(id).unwrap_or((
         EngagementStats {
             like_count: 0,