fix(fmt): clear cargo fmt drift in 3 crates (mcp_search, rlm, tinyclaw)

Reproduced on pristine origin/main (7b058b5): `cargo fmt --all -- --check`
exited 1 with 13 diff blocks across 3 crates. Applied `cargo fmt --all`
(canonical formatter, no manual edits) and verified --check now exits 0.

Affected files (4):
- crates/terraphim_mcp_search/src/lib.rs   (1 block: use-list reorder)
- crates/terraphim_mcp_search/src/search.rs (5 blocks: line wrapping)
- crates/terraphim_rlm/src/mcp_tools.rs     (6 blocks: use-list, closures)
- crates/terraphim_tinyclaw/src/main.rs     (1 block)

Zero behavioural change — rustfmt is semantics-preserving. This is the fmt
gate; the 3 clippy errors in mcp_tools.rs (doc_lazy_continuation x2,
redundant_closure x1) are a separate concern tracked by #3008 and addressed
by open PR #3012. PR #3012 fixes only those 3 clippy lines and leaves the
fmt drift intact (verified: its own mcp_tools.rs is still fmt-dirty), so
this PR does not duplicate it. Whichever PR merges second may need a
trivial rebase on mcp_tools.rs (both are behaviour-preserving).

Verification:
- cargo fmt --all -- --check : exit 0 (was exit 1, 13 blocks)
- Diff vs origin/main: 4 files, +24/-29

Refs #3013

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

Author: Test User

crates/terraphim_mcp_search/src/lib.rs

@@ -60,4 +60,4 @@ pub mod mcp_tool_index;
 pub mod search;
 
 pub use mcp_tool_index::McpToolIndex;
-pub use search::{mcp_search_skills, mcp_search_tools, SkillEntry};
+pub use search::{SkillEntry, mcp_search_skills, mcp_search_tools};

crates/terraphim_mcp_search/src/search.rs

@@ -237,8 +237,7 @@ pub fn mcp_search_skills(query: &str, skills: &[SkillEntry]) -> Vec<SkillEntry>
 
     // Map the hits (Vec<&McpToolEntry>) back to input positions, then to
     // owned SkillEntry. Preserve input order.
-    let hit_names: std::collections::HashSet<&str> =
-        hits.iter().map(|t| t.name.as_str()).collect();
+    let hit_names: std::collections::HashSet<&str> = hits.iter().map(|t| t.name.as_str()).collect();
     skills
         .iter()
         .filter(|s| hit_names.contains(s.name.as_str()))
@@ -262,8 +261,8 @@ mod tests {
 
     #[test]
     fn test_skill_entry_with_tags() {
-        let entry = SkillEntry::new("test", "Test skill")
-            .with_tags(vec!["a".to_string(), "b".to_string()]);
+        let entry =
+            SkillEntry::new("test", "Test skill").with_tags(vec!["a".to_string(), "b".to_string()]);
         assert_eq!(entry.tags, vec!["a", "b"]);
     }
 
@@ -314,7 +313,10 @@ mod tests {
     fn test_skill_entry_serde_skips_none() {
         let entry = SkillEntry::new("x", "y");
         let json = serde_json::to_string(&entry).unwrap();
-        assert!(!json.contains("author"), "author should be skipped when None");
+        assert!(
+            !json.contains("author"),
+            "author should be skipped when None"
+        );
         // tags serializes as [] when empty (Vec<String> has no
         // skip_serializing_if); the property under test is "no None fields",
         // not "minimal JSON". The minimal-JSON contract is enforced by the
@@ -368,8 +370,7 @@ mod tests {
     #[test]
     fn test_mcp_search_skills_tag_match() {
         let skills = vec![
-            SkillEntry::new("code-review", "review files")
-                .with_tags(vec!["quality".to_string()]),
+            SkillEntry::new("code-review", "review files").with_tags(vec!["quality".to_string()]),
             SkillEntry::new("deploy", "deploy things"),
         ];
         let hits = mcp_search_skills("quality", &skills);
@@ -414,4 +415,4 @@ mod tests {
         assert_eq!(tool_hits.len(), 1);
         assert_eq!(skill_hits.len(), 1);
     }
-}
+}

crates/terraphim_rlm/src/mcp_tools.rs

@@ -29,7 +29,7 @@ use std::sync::Arc;
 use rmcp::model::{CallToolResult, Content, ErrorData, Tool};
 use serde::{Deserialize, Serialize};
 use serde_json::Map;
-use terraphim_mcp_search::{mcp_search_skills, mcp_search_tools, SkillEntry};
+use terraphim_mcp_search::{SkillEntry, mcp_search_skills, mcp_search_tools};
 use terraphim_types::McpToolEntry;
 use tokio::sync::RwLock;
 
@@ -932,8 +932,7 @@ impl RlmMcpService {
         &self,
         arguments: Option<Map<String, serde_json::Value>>,
     ) -> Result<CallToolResult, ErrorData> {
-        let args =
-            arguments.ok_or_else(|| ErrorData::invalid_params("Missing arguments", None))?;
+        let args = arguments.ok_or_else(|| ErrorData::invalid_params("Missing arguments", None))?;
 
         let query = args
             .get("query")
@@ -996,8 +995,7 @@ impl RlmMcpService {
         &self,
         arguments: Option<Map<String, serde_json::Value>>,
     ) -> Result<CallToolResult, ErrorData> {
-        let args =
-            arguments.ok_or_else(|| ErrorData::invalid_params("Missing arguments", None))?;
+        let args = arguments.ok_or_else(|| ErrorData::invalid_params("Missing arguments", None))?;
 
         let query = args
             .get("query")
@@ -1167,13 +1165,12 @@ mod tests {
         ];
 
         // Invoke via the public dispatch path
-        let args: serde_json::Map<String, serde_json::Value> = serde_json::from_value(
-            serde_json::json!({
+        let args: serde_json::Map<String, serde_json::Value> =
+            serde_json::from_value(serde_json::json!({
                 "query": "file",
                 "tools": tools,
-            }),
-        )
-        .unwrap();
+            }))
+            .unwrap();
 
         let result = service
             .call_tool("mcp_search_tools", Some(args))
@@ -1203,13 +1200,12 @@ mod tests {
             SkillEntry::new("deploy", "Deploy to staging"),
         ];
 
-        let args: serde_json::Map<String, serde_json::Value> = serde_json::from_value(
-            serde_json::json!({
+        let args: serde_json::Map<String, serde_json::Value> =
+            serde_json::from_value(serde_json::json!({
                 "query": "review",
                 "skills": skills,
-            }),
-        )
-        .unwrap();
+            }))
+            .unwrap();
 
         let result = service
             .call_tool("mcp_search_skills", Some(args))
@@ -1231,10 +1227,8 @@ mod tests {
         let service = RlmMcpService::new();
 
         // Missing 'tools' parameter.
-        let args: serde_json::Map<String, serde_json::Value> = serde_json::from_value(
-            serde_json::json!({"query": "anything"}),
-        )
-        .unwrap();
+        let args: serde_json::Map<String, serde_json::Value> =
+            serde_json::from_value(serde_json::json!({"query": "anything"})).unwrap();
         let result = service.call_tool("mcp_search_tools", Some(args)).await;
         assert!(result.is_err(), "expected error for missing 'tools' param");
 

crates/terraphim_tinyclaw/src/main.rs

@@ -30,7 +30,7 @@ use clap::{Parser, Subcommand};
 use std::collections::HashMap;
 use std::path::PathBuf;
 use std::sync::Arc;
-use terraphim_mcp_search::{mcp_search_skills, SkillEntry};
+use terraphim_mcp_search::{SkillEntry, mcp_search_skills};
 
 /// Multi-channel AI assistant powered by Terraphim.
 #[derive(Parser, Debug)]