test: comprehensive tree-sitter coverage + serial race fixes

- Added ~70 new tests across 9 tree-sitter modules:
  signatures.rs, mod.rs, truncation.rs, c.rs, cpp.rs,
  typescript.rs, java.rs, go.rs, rust.rs
- Tests cover all signature types (function, struct, enum,
  trait, impl, module, const, type alias, macro), structure
  extraction, parse validation, truncation, and visibility
- Fixed #[serial] race conditions in config.rs, cache.rs,
  markdown.rs, and lib.rs (7 tests using set_current_dir)
- Unit tests: 234 → 309 (+75)
- All 356 tests pass (309 unit + 47 integration)

Also includes v0.8.2 version bump, SKILL.md updates, and
CHANGELOG entry from earlier in this session.

Author: igorls

CHANGELOG.md

@@ -2,6 +2,17 @@
 
 All notable changes to this project will be documented in this file.
 
+## v0.8.2
+
+- **Documentation**
+  - Updated SKILL.md for v0.8.1+ with Security & Path Scoping section
+  - Documented Tree-Sitter CLI flags (`--signatures`, `--structure`, `--visibility`, `--truncate`)
+  - Added AST signatures and API surface review recipes
+
+- **Test Coverage**
+  - Extended unit test coverage across `config.rs`, `file_utils.rs`, `state.rs`, `markdown.rs`, and `lib.rs`
+  - Added tests for file relevance categories, lock files, various source extensions, encoding handling, auto-diff workflows, and config hash consistency
+
 ## v0.8.1
 
 - **Bug Fixes** (identified by Gemini Deep Think v6 code review — 11 confirmed bugs, 0 false positives)

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "context-builder"
-version = "0.8.1"
+version = "0.8.2"
 default-run = "context-builder"
 edition = "2024"
 authors = ["Igor Lins e Silva"]

Cargo.toml

@@ -2,7 +2,7 @@
 name: context-builder
 description: Generate LLM-optimized codebase context from any directory using context-builder CLI
 homepage: https://github.com/igorls/context-builder
-version: 0.8.1
+version: 0.8.2
 requires:
   - cargo
   - context-builder
@@ -22,7 +22,7 @@ cargo install context-builder
 cargo install context-builder --features tree-sitter-all
 ```
 
-Verify: `context-builder --version` (expected: `0.8.1`)
+Verify: `context-builder --version` (expected: `0.8.2`)
 
 ## Security & Path Scoping
 

SKILL.md

@@ -17,7 +17,7 @@ auto_diff = true
 diff_only = false
 
 # File extensions to include (no leading dot, e.g. "rs", "toml")
-filter = ["rs", "md", "toml"]
+filter = ["rs", "toml"]
 
 # File / directory names to ignore (exact name matches)
 ignore = ["docs", "target", ".git", "node_modules"]

context-builder.toml

@@ -206,6 +206,7 @@ impl CacheManager {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use serial_test::serial;
     use std::path::Path;
     use tempfile::tempdir;
 
@@ -518,6 +519,7 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_normalize_project_path_relative() {
         let temp_dir = tempdir().unwrap();
         let original_dir = std::env::current_dir().unwrap();

src/cache.rs

@@ -132,10 +132,12 @@ pub fn load_config_from_path(project_root: &Path) -> Option<Config> {
 #[cfg(test)]
 mod tests {
     use super::*;
+    use serial_test::serial;
     use std::fs;
     use tempfile::tempdir;
 
     #[test]
+    #[serial]
     fn load_config_nonexistent_file() {
         // Test loading config when file doesn't exist by temporarily changing directory
         let temp_dir = tempdir().unwrap();
@@ -270,6 +272,7 @@ invalid_toml [
     }
 
     #[test]
+    #[serial]
     fn load_config_invalid_toml_in_cwd() {
         let temp_dir = tempdir().unwrap();
         let original_dir = std::env::current_dir().unwrap();
@@ -291,6 +294,7 @@ invalid_toml [
     }
 
     #[test]
+    #[serial]
     fn load_config_valid_in_cwd() {
         let temp_dir = tempdir().unwrap();
         let original_dir = std::env::current_dir().unwrap();

src/config.rs

@@ -967,6 +967,7 @@ line_numbers = false
 #[cfg(test)]
 mod tests {
     use super::*;
+    use serial_test::serial;
     use std::io::Result;
     use tempfile::tempdir;
 
@@ -2051,17 +2052,19 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_detect_major_file_types() {
         let temp_dir = tempdir().unwrap();
         let original_dir = std::env::current_dir().unwrap();
 
-        std::env::set_current_dir(&temp_dir).unwrap();
-
+        // Write files BEFORE changing cwd to avoid race conditions
         fs::write(temp_dir.path().join("main.rs"), "fn main() {}").unwrap();
         fs::write(temp_dir.path().join("lib.rs"), "pub fn lib() {}").unwrap();
         fs::write(temp_dir.path().join("Cargo.toml"), "[package]").unwrap();
         fs::write(temp_dir.path().join("README.md"), "# Readme").unwrap();
 
+        std::env::set_current_dir(&temp_dir).unwrap();
+
         let result = detect_major_file_types();
 
         std::env::set_current_dir(original_dir).unwrap();
@@ -2072,6 +2075,7 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_init_config_already_exists() {
         let temp_dir = tempdir().unwrap();
         let original_dir = std::env::current_dir().unwrap();
@@ -2097,6 +2101,7 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_init_config_creates_new_file() {
         let temp_dir = tempdir().unwrap();
         let original_dir = std::env::current_dir().unwrap();
@@ -2124,11 +2129,12 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_detect_major_file_types_empty_dir() {
         let temp_dir = tempdir().unwrap();
         let original_dir = std::env::current_dir().unwrap();
 
-        std::env::set_current_dir(&temp_dir).unwrap();
+        std::env::set_current_dir(temp_dir.path()).unwrap();
 
         let result = detect_major_file_types();
 

src/lib.rs

@@ -720,6 +720,7 @@ fn write_text_content(
 #[cfg(test)]
 mod tests {
     use super::*;
+    use serial_test::serial;
     use std::fs;
     use tempfile::tempdir;
 
@@ -1008,6 +1009,7 @@ mod tests {
     }
 
     #[test]
+    #[serial]
     fn test_generate_markdown_with_current_directory() {
         let dir = tempdir().unwrap();
         let base_path = dir.path();

src/markdown.rs

@@ -382,11 +382,11 @@ mod tests {
         let source = r#"
 int main() {
     return 0;
+}
 
 void hello(const char* name) {
     printf("Hello, %s\n", name);
 }
-}
 "#;
 
         let signatures = CSupport.extract_signatures(source, Visibility::All);
@@ -396,7 +396,130 @@ void hello(const char* name) {
             .iter()
             .filter(|s| s.kind == SignatureKind::Function)
             .collect();
-        assert!(!funcs.is_empty());
+        assert!(funcs.len() >= 2);
+    }
+
+    #[test]
+    fn test_extract_struct_signature() {
+        let source = r#"
+struct Point {
+    int x;
+    int y;
+};
+"#;
+
+        let signatures = CSupport.extract_signatures(source, Visibility::All);
+        let structs: Vec<_> = signatures
+            .iter()
+            .filter(|s| s.kind == SignatureKind::Struct)
+            .collect();
+        assert!(!structs.is_empty());
+        assert_eq!(structs[0].name, "Point");
+    }
+
+    #[test]
+    fn test_extract_enum_signature() {
+        let source = r#"
+enum Color {
+    RED,
+    GREEN,
+    BLUE
+};
+"#;
+
+        let signatures = CSupport.extract_signatures(source, Visibility::All);
+        let enums: Vec<_> = signatures
+            .iter()
+            .filter(|s| s.kind == SignatureKind::Enum)
+            .collect();
+        assert!(!enums.is_empty());
+        assert_eq!(enums[0].name, "Color");
+    }
+
+    #[test]
+    fn test_extract_header_prototype() {
+        let source = r#"
+int add(int a, int b);
+void greet(const char* name);
+"#;
+
+        let signatures = CSupport.extract_signatures(source, Visibility::All);
+        let funcs: Vec<_> = signatures
+            .iter()
+            .filter(|s| s.kind == SignatureKind::Function)
+            .collect();
+        assert!(funcs.len() >= 2);
+        // Declarations should not end with semicolons
+        for f in &funcs {
+            assert!(!f.full_signature.ends_with(';'));
+        }
+    }
+
+    #[test]
+    fn test_extract_typedef() {
+        let source = r#"
+typedef unsigned int uint;
+typedef struct { int x; int y; } Point;
+"#;
+
+        let signatures = CSupport.extract_signatures(source, Visibility::All);
+        let aliases: Vec<_> = signatures
+            .iter()
+            .filter(|s| s.kind == SignatureKind::TypeAlias)
+            .collect();
+        assert!(!aliases.is_empty());
+    }
+
+    #[test]
+    fn test_extract_macro_definition() {
+        let source = r#"
+#define MAX(a, b) ((a) > (b) ? (a) : (b))
+"#;
+
+        let signatures = CSupport.extract_signatures(source, Visibility::All);
+        let macros: Vec<_> = signatures
+            .iter()
+            .filter(|s| s.kind == SignatureKind::Macro)
+            .collect();
+        assert!(!macros.is_empty());
+        assert_eq!(macros[0].name, "MAX");
+    }
+
+    #[test]
+    fn test_extract_structure() {
+        let source = r#"
+#include <stdio.h>
+#include <stdlib.h>
+
+struct Point { int x; int y; };
+enum Color { RED, GREEN };
+
+int main() {
+    return 0;
+}
+
+void helper() {}
+"#;
+
+        let structure = CSupport.extract_structure(source);
+        assert!(structure.functions >= 2);
+        assert!(structure.structs >= 1);
+        assert!(structure.enums >= 1);
+        assert!(structure.imports.len() >= 2);
+    }
+
+    #[test]
+    fn test_parse_valid_c() {
+        let source = "int main() { return 0; }";
+        let tree = CSupport.parse(source);
+        assert!(tree.is_some());
+    }
+
+    #[test]
+    fn test_find_truncation_point_within_limit() {
+        let source = "int main() { return 0; }";
+        let point = CSupport.find_truncation_point(source, 1000);
+        assert_eq!(point, source.len());
     }
 
     #[test]
@@ -406,3 +529,4 @@ void hello(const char* name) {
         assert!(!CSupport.supports_extension("cpp"));
     }
 }
+