chore: fix cargo fmt formatting issues

Author: RoyLin

core/src/agent_api.rs

@@ -185,7 +185,8 @@ pub struct SessionOptions {
     ///
     /// When set, enables custom document format support (PDF, Excel, Word, etc.)
     /// for the agentic_search tool. If not set, only plain text files are searched.
-    pub document_parser_registry: Option<Arc<crate::tools::document_parser::DocumentParserRegistry>>,
+    pub document_parser_registry:
+        Option<Arc<crate::tools::document_parser::DocumentParserRegistry>>,
 }
 
 impl std::fmt::Debug for SessionOptions {

core/src/context/ripgrep_provider.rs

@@ -131,7 +131,11 @@ impl RipgrepContextProvider {
     }
 
     /// Search files for pattern matches
-    async fn search_files(&self, query: &str, max_results: usize) -> anyhow::Result<Vec<FileMatch>> {
+    async fn search_files(
+        &self,
+        query: &str,
+        max_results: usize,
+    ) -> anyhow::Result<Vec<FileMatch>> {
         let root = self.config.root_path.clone();
         let max_file_size = self.config.max_file_size;
         let include = self.config.include_patterns.clone();
@@ -202,10 +206,7 @@ impl RipgrepContextProvider {
                                 .map(|s| s.to_string())
                                 .collect()
                         } else {
-                            lines[0..line_idx]
-                                .iter()
-                                .map(|s| s.to_string())
-                                .collect()
+                            lines[0..line_idx].iter().map(|s| s.to_string()).collect()
                         };
 
                         let context_after = if line_idx + context_lines < lines.len() {
@@ -242,7 +243,11 @@ impl RipgrepContextProvider {
             }
 
             // Sort by relevance (descending)
-            file_matches.sort_by(|a, b| b.relevance.partial_cmp(&a.relevance).unwrap_or(std::cmp::Ordering::Equal));
+            file_matches.sort_by(|a, b| {
+                b.relevance
+                    .partial_cmp(&a.relevance)
+                    .unwrap_or(std::cmp::Ordering::Equal)
+            });
             file_matches.truncate(max_results);
 
             Ok::<_, anyhow::Error>(file_matches)
@@ -259,7 +264,11 @@ impl RipgrepContextProvider {
         match depth {
             crate::context::ContextDepth::Abstract => {
                 // Just show file path and match count
-                output.push_str(&format!("{}: {} matches\n", path_str, file_match.matches.len()));
+                output.push_str(&format!(
+                    "{}: {} matches\n",
+                    path_str,
+                    file_match.matches.len()
+                ));
             }
             crate::context::ContextDepth::Overview => {
                 // Show first few matches with limited context
@@ -272,7 +281,10 @@ impl RipgrepContextProvider {
                     output.push_str(&format!("    {}\n", m.line_content));
                 }
                 if file_match.matches.len() > 3 {
-                    output.push_str(&format!("  ... and {} more matches\n", file_match.matches.len() - 3));
+                    output.push_str(&format!(
+                        "  ... and {} more matches\n",
+                        file_match.matches.len() - 3
+                    ));
                 }
             }
             crate::context::ContextDepth::Full => {
@@ -443,7 +455,10 @@ mod tests {
         assert_eq!(result.provider, "ripgrep");
         assert!(!result.items.is_empty());
         // Should find "Rust" in README.md
-        assert!(result.items.iter().any(|item| item.content.contains("Rust")));
+        assert!(result
+            .items
+            .iter()
+            .any(|item| item.content.contains("Rust")));
     }
 
     #[tokio::test]
@@ -492,6 +507,10 @@ mod tests {
     fn test_matches_patterns_include() {
         let patterns = vec!["**/*.rs".to_string()];
         assert!(matches_patterns(Path::new("src/main.rs"), &patterns, false));
-        assert!(!matches_patterns(Path::new("src/main.py"), &patterns, false));
+        assert!(!matches_patterns(
+            Path::new("src/main.py"),
+            &patterns,
+            false
+        ));
     }
 }

core/src/tools/builtin/agentic_search.rs

@@ -67,7 +67,8 @@ impl FileType {
     fn from_path(path: &std::path::Path) -> Self {
         if let Some(ext) = path.extension().and_then(|e| e.to_str()) {
             match ext {
-                "rs" | "py" | "ts" | "tsx" | "js" | "jsx" | "go" | "java" | "c" | "cpp" | "h" | "hpp" => Self::Code,
+                "rs" | "py" | "ts" | "tsx" | "js" | "jsx" | "go" | "java" | "c" | "cpp" | "h"
+                | "hpp" => Self::Code,
                 "toml" | "yaml" | "yml" | "json" | "ini" | "conf" => Self::Config,
                 "md" | "txt" | "rst" | "adoc" => Self::Documentation,
                 _ => Self::Other,
@@ -216,7 +217,14 @@ impl AgenticSearchTool {
 
         // Phase 1+2: Parallel search across all keywords
         let matches = tokio::task::spawn_blocking(move || {
-            search_workspace(&workspace, &keywords, max_results, include.as_deref(), context_lines, registry.as_deref())
+            search_workspace(
+                &workspace,
+                &keywords,
+                max_results,
+                include.as_deref(),
+                context_lines,
+                registry.as_deref(),
+            )
         })
         .await
         .map_err(|e| anyhow::anyhow!("Search task failed: {}", e))??;
@@ -257,7 +265,14 @@ impl AgenticSearchTool {
 
         // Phase 1: Initial broad search (2x max_results for sampling pool)
         let initial_matches = tokio::task::spawn_blocking(move || {
-            search_workspace(&workspace, &keywords, max_results * 2, include.as_deref(), context_lines, registry.as_deref())
+            search_workspace(
+                &workspace,
+                &keywords,
+                max_results * 2,
+                include.as_deref(),
+                context_lines,
+                registry.as_deref(),
+            )
         })
         .await
         .map_err(|e| anyhow::anyhow!("Search task failed: {}", e))??;
@@ -331,14 +346,13 @@ impl AgenticSearchTool {
 /// - Individual words (for broad match)
 fn extract_keywords(query: &str) -> Vec<String> {
     const STOP_WORDS: &[&str] = &[
-        "a", "an", "the", "is", "are", "was", "were", "be", "been", "being",
-        "have", "has", "had", "do", "does", "did", "will", "would", "could",
-        "should", "may", "might", "shall", "can", "need", "dare", "ought",
-        "how", "what", "where", "when", "why", "who", "which", "that", "this",
-        "these", "those", "it", "its", "in", "on", "at", "to", "for", "of",
-        "with", "by", "from", "up", "about", "into", "through", "during",
-        "and", "or", "but", "not", "no", "nor", "so", "yet", "both", "either",
-        "work", "works", "working", "find", "get", "use", "make", "look",
+        "a", "an", "the", "is", "are", "was", "were", "be", "been", "being", "have", "has", "had",
+        "do", "does", "did", "will", "would", "could", "should", "may", "might", "shall", "can",
+        "need", "dare", "ought", "how", "what", "where", "when", "why", "who", "which", "that",
+        "this", "these", "those", "it", "its", "in", "on", "at", "to", "for", "of", "with", "by",
+        "from", "up", "about", "into", "through", "during", "and", "or", "but", "not", "no", "nor",
+        "so", "yet", "both", "either", "work", "works", "working", "find", "get", "use", "make",
+        "look",
     ];
 
     let mut keywords: Vec<String> = Vec::new();
@@ -397,9 +411,7 @@ fn search_workspace(
     // Build regex patterns for each keyword
     let patterns: Vec<Regex> = keywords
         .iter()
-        .filter_map(|kw| {
-            Regex::new(&format!("(?i){}", regex::escape(kw))).ok()
-        })
+        .filter_map(|kw| Regex::new(&format!("(?i){}", regex::escape(kw))).ok())
         .collect();
 
     if patterns.is_empty() {
@@ -581,11 +593,7 @@ fn find_matching_files(
 // ============================================================================
 
 fn format_results(matches: &[FileMatch], query: &str) -> String {
-    let mut out = format!(
-        "Found {} file(s) matching \"{}\"\n\n",
-        matches.len(),
-        query
-    );
+    let mut out = format!("Found {} file(s) matching \"{}\"\n\n", matches.len(), query);
 
     for (i, fm) in matches.iter().enumerate() {
         let rel_path = fm.path.display();
@@ -598,9 +606,7 @@ fn format_results(matches: &[FileMatch], query: &str) -> String {
 
         out.push_str(&format!(
             "─── {} ({}, relevance: {:.2}) ───\n",
-            rel_path,
-            type_label,
-            fm.relevance
+            rel_path, type_label, fm.relevance
         ));
 
         for m in fm.matches.iter().take(5) {
@@ -825,7 +831,11 @@ mod tests {
         writeln!(f, "pub struct Session {{\n    pub user_id: String,\n    pub token: String,\n}}\n\nimpl Session {{\n    pub fn new(user_id: &str, token: &str) -> Self {{\n        Self {{ user_id: user_id.to_string(), token: token.to_string() }}\n    }}\n}}").unwrap();
 
         let mut f = File::create(root.join("README.md")).unwrap();
-        writeln!(f, "# Auth Service\n\nHandles JWT authentication and session management.").unwrap();
+        writeln!(
+            f,
+            "# Auth Service\n\nHandles JWT authentication and session management."
+        )
+        .unwrap();
 
         dir
     }
@@ -855,9 +865,18 @@ mod tests {
 
     #[test]
     fn test_file_type_detection() {
-        assert_eq!(FileType::from_path(std::path::Path::new("main.rs")), FileType::Code);
-        assert_eq!(FileType::from_path(std::path::Path::new("config.toml")), FileType::Config);
-        assert_eq!(FileType::from_path(std::path::Path::new("README.md")), FileType::Documentation);
+        assert_eq!(
+            FileType::from_path(std::path::Path::new("main.rs")),
+            FileType::Code
+        );
+        assert_eq!(
+            FileType::from_path(std::path::Path::new("config.toml")),
+            FileType::Config
+        );
+        assert_eq!(
+            FileType::from_path(std::path::Path::new("README.md")),
+            FileType::Documentation
+        );
     }
 
     #[tokio::test]
@@ -916,7 +935,10 @@ mod tests {
 
         assert!(output.success);
         if let Some(meta) = &output.metadata {
-            let count = meta.get("result_count").and_then(|v| v.as_u64()).unwrap_or(0);
+            let count = meta
+                .get("result_count")
+                .and_then(|v| v.as_u64())
+                .unwrap_or(0);
             assert!(count <= 1);
         }
     }

core/src/tools/document_parser.rs

@@ -93,22 +93,17 @@ impl DocumentParser for PlainTextParser {
     fn supported_extensions(&self) -> &[&str] {
         &[
             // Code
-            "rs", "py", "ts", "tsx", "js", "jsx", "go", "java", "c", "cpp", "h", "hpp",
-            "cs", "rb", "php", "swift", "kt", "scala", "sh", "bash", "zsh",
-            // Config
-            "toml", "yaml", "yml", "json", "ini", "conf", "cfg", "env",
-            // Documentation
-            "md", "txt", "rst", "adoc", "org",
-            // Web
-            "html", "htm", "xml", "css", "scss", "sass", "less",
-            // Data
+            "rs", "py", "ts", "tsx", "js", "jsx", "go", "java", "c", "cpp", "h", "hpp", "cs", "rb",
+            "php", "swift", "kt", "scala", "sh", "bash", "zsh", // Config
+            "toml", "yaml", "yml", "json", "ini", "conf", "cfg", "env", // Documentation
+            "md", "txt", "rst", "adoc", "org", // Web
+            "html", "htm", "xml", "css", "scss", "sass", "less", // Data
             "csv", "tsv", "log",
         ]
     }
 
     fn parse(&self, path: &Path) -> Result<String> {
-        std::fs::read_to_string(path)
-            .map_err(|e| anyhow::anyhow!("Failed to read file: {}", e))
+        std::fs::read_to_string(path).map_err(|e| anyhow::anyhow!("Failed to read file: {}", e))
     }
 
     fn max_file_size(&self) -> u64 {

scripts/pre-commit

@@ -0,0 +1,24 @@
+#!/bin/bash
+# Pre-commit hook for a3s-code
+# Ensures cargo fmt passes before allowing commit
+#
+# Installation:
+#   cp scripts/pre-commit .git/hooks/pre-commit
+#   chmod +x .git/hooks/pre-commit
+
+set -e
+
+echo "🔍 Running cargo fmt check..."
+
+# Run cargo fmt check
+if ! cargo fmt --all -- --check; then
+    echo ""
+    echo "❌ Formatting check failed!"
+    echo ""
+    echo "Please run 'cargo fmt --all' to format your code before committing."
+    echo ""
+    exit 1
+fi
+
+echo "✅ Formatting check passed!"
+exit 0