fix: resolve clippy warnings (collapsible_if, unused doc comments, next_back)

Author: 1337Xcode

CHANGELOG.md

@@ -2,7 +2,7 @@
 
 All notable changes to Cortex are documented here.
 
-## [1.0.3] - 2026-06-01
+## [1.0.3] - 2026-05-25
 
 ### Added
 - `cortex update` self-update command: downloads latest release from GitHub, verifies SHA-256 checksum, replaces binary (with Windows rename-then-replace pattern), and triggers reindex

src/cli/commands/reindex.rs

@@ -20,15 +20,13 @@ pub fn run_reindex(repo_root: &Path, data_dir: &Path) {
     // Delete existing database files (graph.db, graph.db-wal, graph.db-shm)
     for suffix in &["", "-wal", "-shm"] {
         let db_file = data_dir.join(format!("graph.db{}", suffix));
-        if db_file.exists() {
-            if let Err(e) = fs::remove_file(&db_file) {
-                eprintln!(
-                    "error: cannot delete '{}': {}. Close any running cortex processes.",
-                    db_file.display(),
-                    e
-                );
-                process::exit(1);
-            }
+        if db_file.exists() && let Err(e) = fs::remove_file(&db_file) {
+            eprintln!(
+                "error: cannot delete '{}': {}. Close any running cortex processes.",
+                db_file.display(),
+                e
+            );
+            process::exit(1);
         }
     }
 

src/indexer/pipeline.rs

@@ -1620,7 +1620,7 @@ def get_aws_config():
 
             // The excluded file should NOT appear in results
             for path in &results {
-                let filename = path.split('/').last().unwrap_or("");
+                let filename = path.split('/').next_back().unwrap_or("");
                 prop_assert!(
                     !EXCLUDED_FILES.contains(&filename),
                     "Excluded file '{}' should not appear in walk results, but found path '{}'",
@@ -1671,13 +1671,12 @@ def get_aws_config():
         "[a-z][a-z0-9_]{0,11}"
     }
 
-    /// **Validates: Requirements 1.5, 2.1, 2.2**
-    ///
-    /// Property 1: Gitignore-style pattern matching
-    ///
-    /// Patterns loaded from .cortexignore should correctly match paths using
-    /// gitignore syntax: directory patterns (`dir/`), wildcard extension patterns
-    /// (`*.ext`), path prefix patterns (containing `/`), and simple name patterns.
+    // Property 1: Gitignore-style pattern matching
+    // Validates: Requirements 1.5, 2.1, 2.2
+    //
+    // Patterns loaded from .cortexignore should correctly match paths using
+    // gitignore syntax: directory patterns (`dir/`), wildcard extension patterns
+    // (`*.ext`), path prefix patterns (containing `/`), and simple name patterns.
     proptest! {
         /// Directory patterns ending in `/` match paths starting with the prefix.
         #[test]
@@ -1824,12 +1823,11 @@ def get_aws_config():
     // Property 2: Pattern file parsing filters non-pattern lines
     // -----------------------------------------------------------------------
 
-    /// **Validates: Requirements 2.3, 2.4**
-    ///
-    /// Property 2: Pattern file parsing filters non-pattern lines
-    ///
-    /// Comments (lines starting with #) and empty/whitespace lines should be
-    /// filtered out during parsing. Only valid pattern lines remain.
+    // Property 2: Pattern file parsing filters non-pattern lines
+    // Validates: Requirements 2.3, 2.4
+    //
+    // Comments (lines starting with #) and empty/whitespace lines should be
+    // filtered out during parsing. Only valid pattern lines remain.
     proptest! {
         /// Mixed content: valid patterns, comments, and blanks. Only valid patterns
         /// should appear in the result.