Make ignore_paths actually work

Author: blopker

crates/codebook-config/src/lib.rs

@@ -375,6 +375,27 @@ impl CodebookConfig {
         Ok(true)
     }
 
+    /// Add a file to the ignore list
+    pub fn add_ignore(&self, file: &str) -> Result<bool, io::Error> {
+        {
+            let mut project_settings = self.project_settings.write().unwrap();
+            let file = file.to_string();
+            // Check if file already exists
+            if project_settings.ignore_paths.contains(&file) {
+                return Ok(false);
+            }
+
+            // Add the file
+            project_settings.ignore_paths.push(file);
+            // Sort/dedup for consistency
+            project_settings.ignore_paths.sort();
+            project_settings.ignore_paths.dedup();
+        }
+        self.recalculate_effective_settings();
+
+        Ok(true)
+    }
+
     /// Save the project configuration to its file
     pub fn save(&self) -> Result<(), io::Error> {
         let project_config_path = match self.project_config_path.as_ref() {

crates/codebook/src/lib.rs

@@ -35,6 +35,11 @@ impl Codebook {
         language: Option<queries::LanguageType>,
         file_path: Option<&str>,
     ) -> Vec<parser::WordLocation> {
+        if file_path.is_some() {
+            if self.config.should_ignore_path(file_path.unwrap()) {
+                return Vec::new();
+            }
+        }
         // get needed dictionary names
         // get needed dictionaries
         // call spell check on each dictionary
@@ -102,7 +107,7 @@ impl Codebook {
     pub fn spell_check_file(&self, path: &str) -> Vec<WordLocation> {
         let lang_type = queries::get_language_name_from_filename(path);
         let file_text = std::fs::read_to_string(path).unwrap();
-        return self.spell_check(&file_text, Some(lang_type), None);
+        return self.spell_check(&file_text, Some(lang_type), Some(&path));
     }
 
     pub fn get_suggestions(&self, word: &str) -> Option<Vec<String>> {

crates/codebook/tests/test_files.rs

@@ -10,6 +10,14 @@ fn example_file_path(file: &str) -> String {
     format!("tests/examples/{}", file)
 }
 
+#[test]
+fn test_ignore_file() {
+    utils::init_logging();
+    let processor = utils::get_processor();
+    let results = processor.spell_check("badword", None, Some("ignore.txt"));
+    assert_eq!(results.len(), 0);
+}
+
 #[test]
 fn test_example_files_word_locations() {
     utils::init_logging();

crates/codebook/tests/utils/mod.rs

@@ -4,6 +4,9 @@ use codebook::Codebook;
 
 pub fn get_processor() -> Codebook {
     let config = Arc::new(codebook_config::CodebookConfig::default());
+    config
+        .add_ignore("**/ignore.txt")
+        .expect("Should ignore file");
     let dict = Codebook::new(config).unwrap();
     dict
 }

tsconfig.json

@@ -1,6 +1,6 @@
 {
   "compilerOptions": {
-    // Enable latest features
+    // Enable latest featurez
     "lib": ["ESNext", "DOM"],
     "target": "ESNext",
     "module": "ESNext",