Refactor: Use Path instead of PathBuf in test helpers

Co-authored-by: script3r <[email protected]>

Author: cursoragent

crates/cli/tests/anchors.rs

@@ -1,10 +1,10 @@
 use scanner_core::*;
 use std::fs;
-use std::path::PathBuf;
+use std::path::{Path, PathBuf};
 use std::sync::Arc;
 use std::time::{SystemTime, UNIX_EPOCH};
 
-fn write_file(dir: &PathBuf, rel: &str, contents: &str) {
+fn write_file(dir: &Path, rel: &str, contents: &str) {
     let path = dir.join(rel);
     if let Some(parent) = path.parent() {
         fs::create_dir_all(parent).unwrap();

crates/cli/tests/filtering.rs

@@ -1,10 +1,10 @@
 use scanner_core::*;
 use std::fs;
-use std::path::PathBuf;
+use std::path::{Path, PathBuf};
 use std::sync::Arc;
 use std::time::{SystemTime, UNIX_EPOCH};
 
-fn write_file(dir: &PathBuf, rel: &str, contents: &str) {
+fn write_file(dir: &Path, rel: &str, contents: &str) {
     let path = dir.join(rel);
     if let Some(parent) = path.parent() {
         fs::create_dir_all(parent).unwrap();

crates/scanner-core/src/lib.rs

@@ -1016,13 +1016,11 @@ impl PatternDetector {
                     last_api = Some((m.start(), re.as_str().to_string()));
                 }
             }
-            if api_hits > 0 {
-                if first_symbol.is_empty() {
-                    if let Some((pos, sym)) = last_api.clone() {
-                        first_span = index.to_line_col(pos);
-                        first_symbol = sym;
-                        first_snippet = extract_line(stripped_s, pos);
-                    }
+            if api_hits > 0 && first_symbol.is_empty() {
+                if let Some((pos, sym)) = last_api.clone() {
+                    first_span = index.to_line_col(pos);
+                    first_symbol = sym;
+                    first_snippet = extract_line(stripped_s, pos);
                 }
             }
             // Require anchor only if patterns define any; always require at least one API hit