Refactor: Improve code formatting and readability

Co-authored-by: script3r <script3r@gmail.com>

Author: cursoragent

crates/cli/src/main.rs

@@ -22,7 +22,6 @@ struct Args {
     #[arg(long, value_name = "FILE")]
     sarif: Option<PathBuf>,
 
-
     /// Number of threads
     #[arg(long, value_name = "N")]
     threads: Option<usize>,

crates/cli/tests/anchors.rs

@@ -14,7 +14,10 @@ fn write_file(dir: &Path, rel: &str, contents: &str) {
 
 fn tmp_dir(prefix: &str) -> PathBuf {
     let mut base = std::env::temp_dir();
-    let ts = SystemTime::now().duration_since(UNIX_EPOCH).unwrap().as_nanos();
+    let ts = SystemTime::now()
+        .duration_since(UNIX_EPOCH)
+        .unwrap()
+        .as_nanos();
     let pid = std::process::id();
     base.push(format!("cipherscope_test_{}_{}_{}", prefix, pid, ts));
     fs::create_dir_all(&base).unwrap();
@@ -48,9 +51,7 @@ public class ImportOnly {
     );
     let findings = scanner.run(std::slice::from_ref(&dir_import_only)).unwrap();
     assert!(
-        !findings
-            .iter()
-            .any(|f| f.library == "Google Tink (Java)"),
+        !findings.iter().any(|f| f.library == "Google Tink (Java)"),
         "Tink should not be reported with import only"
     );
 
@@ -71,9 +72,7 @@ public class ApiOnly {
     );
     let findings = scanner.run(std::slice::from_ref(&dir_api_only)).unwrap();
     assert!(
-        !findings
-            .iter()
-            .any(|f| f.library == "Google Tink (Java)"),
+        !findings.iter().any(|f| f.library == "Google Tink (Java)"),
         "Tink should not be reported with API mentions only"
     );
 
@@ -95,10 +94,7 @@ public class Both {
     );
     let findings = scanner.run(std::slice::from_ref(&dir_both)).unwrap();
     assert!(
-        findings
-            .iter()
-            .any(|f| f.library == "Google Tink (Java)"),
+        findings.iter().any(|f| f.library == "Google Tink (Java)"),
         "Tink should be reported when import and API are present"
     );
 }
-

crates/cli/tests/filtering.rs

@@ -14,7 +14,10 @@ fn write_file(dir: &Path, rel: &str, contents: &str) {
 
 fn tmp_dir(prefix: &str) -> PathBuf {
     let mut base = std::env::temp_dir();
-    let ts = SystemTime::now().duration_since(UNIX_EPOCH).unwrap().as_nanos();
+    let ts = SystemTime::now()
+        .duration_since(UNIX_EPOCH)
+        .unwrap()
+        .as_nanos();
     let pid = std::process::id();
     base.push(format!("cipherscope_test_{}_{}_{}", prefix, pid, ts));
     fs::create_dir_all(&base).unwrap();
@@ -54,9 +57,7 @@ public class Main {
     );
     let findings = scanner.run(std::slice::from_ref(&dir)).unwrap();
     assert!(
-        !findings
-            .iter()
-            .any(|f| f.library == "Java JCA/JCE"),
+        !findings.iter().any(|f| f.library == "Java JCA/JCE"),
         "JCA/JCE should not be reported when import is commented"
     );
 }
@@ -93,8 +94,16 @@ echo $ciphertext;
 fn include_glob_filters_file_types() {
     let reg = load_registry();
     let dets_java: Vec<Box<dyn Detector>> = vec![
-        Box::new(PatternDetector::new("detector-java", &[Language::Java], reg.clone())),
-        Box::new(PatternDetector::new("detector-php", &[Language::Php], reg.clone())),
+        Box::new(PatternDetector::new(
+            "detector-java",
+            &[Language::Java],
+            reg.clone(),
+        )),
+        Box::new(PatternDetector::new(
+            "detector-php",
+            &[Language::Php],
+            reg.clone(),
+        )),
     ];
 
     let dir = tmp_dir("include_glob_filters");
@@ -139,8 +148,16 @@ echo openssl_encrypt("data", "aes-256-cbc", "key", 0, "1234567890123456");
         ..Default::default()
     };
     let dets_php: Vec<Box<dyn Detector>> = vec![
-        Box::new(PatternDetector::new("detector-java", &[Language::Java], reg.clone())),
-        Box::new(PatternDetector::new("detector-php", &[Language::Php], reg.clone())),
+        Box::new(PatternDetector::new(
+            "detector-java",
+            &[Language::Java],
+            reg.clone(),
+        )),
+        Box::new(PatternDetector::new(
+            "detector-php",
+            &[Language::Php],
+            reg.clone(),
+        )),
     ];
     let scanner_php = Scanner::new(&reg, dets_php, cfg_php_only);
     let findings_php = scanner_php.run(std::slice::from_ref(&dir)).unwrap();
@@ -177,6 +194,8 @@ fn max_file_size_skips_large_files() {
     };
     let scanner = Scanner::new(&reg, dets, cfg_small_limit);
     let findings = scanner.run(std::slice::from_ref(&dir)).unwrap();
-    assert!(findings.is_empty(), "Large file should be skipped by max_file_size");
+    assert!(
+        findings.is_empty(),
+        "Large file should be skipped by max_file_size"
+    );
 }
-

crates/scanner-core/src/lib.rs

@@ -72,7 +72,6 @@ pub struct Span {
     pub column: usize,
 }
 
-
 #[derive(Debug, Clone, Serialize, Deserialize)]
 pub struct Finding {
     pub language: Language,
@@ -1024,8 +1023,13 @@ impl PatternDetector {
                 }
             }
             // Require anchor only if patterns define any; always require at least one API hit
-            let has_anchor_patterns = !lib.include.is_empty() || !lib.import.is_empty() || !lib.namespace.is_empty();
-            let anchor_satisfied = if has_anchor_patterns { matched_import } else { true };
+            let has_anchor_patterns =
+                !lib.include.is_empty() || !lib.import.is_empty() || !lib.namespace.is_empty();
+            let anchor_satisfied = if has_anchor_patterns {
+                matched_import
+            } else {
+                true
+            };
             let should_report = anchor_satisfied && api_hits > 0;
             if should_report {
                 let finding = Finding {