fix: cross-platform CI -- tree-sitter API compat, pin >=0.24, Windows path separators in tests

Author: DevanshuNEU

CLAUDE.md

@@ -36,8 +36,8 @@ These files have the most dependents -- understand them before editing:
 
 - `saar/models.py` (14 dependents)
 - `saar/formatters/_tribal.py` (4 dependents)
-- `saar/formatters/agents_md.py` (3 dependents)
 - `saar/formatters/claude_md.py` (3 dependents)
+- `saar/formatters/agents_md.py` (3 dependents)
 - `saar/cli.py` (3 dependents)
 - `saar/interview.py` (3 dependents)
 - `saar/dependency_analyzer.py` (2 dependents)

pyproject.toml

@@ -29,9 +29,9 @@ classifiers = [
     "Topic :: Software Development :: Code Generators",
 ]
 dependencies = [
-    "tree-sitter>=0.23.0",
-    "tree-sitter-python>=0.23.0",
-    "tree-sitter-javascript>=0.23.0",
+    "tree-sitter>=0.24.0,<0.26.0",
+    "tree-sitter-python>=0.23.0,<0.26.0",
+    "tree-sitter-javascript>=0.23.0,<0.26.0",
     "typer>=0.12.0",
     "rich>=13.0.0",
     "questionary>=2.0.0",

saar/dependency_analyzer.py

@@ -49,11 +49,20 @@ class DependencyAnalyzer:
     }
 
     def __init__(self) -> None:
-        self.parsers = {
-            "python": Parser(Language(tspython.language())),
-            "javascript": Parser(Language(tsjavascript.language())),
-            "typescript": Parser(Language(tsjavascript.language())),
-        }
+        try:
+            self.parsers = {
+                "python": Parser(Language(tspython.language())),
+                "javascript": Parser(Language(tsjavascript.language())),
+                "typescript": Parser(Language(tsjavascript.language())),
+            }
+        except TypeError:
+            py_lang = Language(tspython.language())
+            js_lang = Language(tsjavascript.language())
+            self.parsers = {}
+            for name, lang in [("python", py_lang), ("javascript", js_lang), ("typescript", js_lang)]:
+                p = Parser()
+                p.language = lang
+                self.parsers[name] = p
 
     def _detect_language(self, file_path: str) -> str:
         ext = Path(file_path).suffix.lower()

saar/extractor.py

@@ -90,11 +90,21 @@ class DNAExtractor:
     ]
 
     def __init__(self) -> None:
-        self.parsers = {
-            "python": Parser(Language(tspython.language())),
-            "javascript": Parser(Language(tsjavascript.language())),
-            "typescript": Parser(Language(tsjavascript.language())),
-        }
+        try:
+            self.parsers = {
+                "python": Parser(Language(tspython.language())),
+                "javascript": Parser(Language(tsjavascript.language())),
+                "typescript": Parser(Language(tsjavascript.language())),
+            }
+        except TypeError:
+            # tree-sitter <0.24 uses Parser().set_language() API
+            py_lang = Language(tspython.language())
+            js_lang = Language(tsjavascript.language())
+            self.parsers = {}
+            for name, lang in [("python", py_lang), ("javascript", js_lang), ("typescript", js_lang)]:
+                p = Parser()
+                p.language = lang
+                self.parsers[name] = p
         self._file_cache: Dict[Path, str] = {}
         self._stats = {"files_read": 0, "files_skipped": 0, "read_errors": 0}
         self._active_skip_dirs = set(self.SKIP_DIRS)

saar/style_analyzer.py

@@ -48,11 +48,20 @@ class StyleAnalyzer:
     }
 
     def __init__(self) -> None:
-        self.parsers = {
-            "python": Parser(Language(tspython.language())),
-            "javascript": Parser(Language(tsjavascript.language())),
-            "typescript": Parser(Language(tsjavascript.language())),
-        }
+        try:
+            self.parsers = {
+                "python": Parser(Language(tspython.language())),
+                "javascript": Parser(Language(tsjavascript.language())),
+                "typescript": Parser(Language(tsjavascript.language())),
+            }
+        except TypeError:
+            py_lang = Language(tspython.language())
+            js_lang = Language(tsjavascript.language())
+            self.parsers = {}
+            for name, lang in [("python", py_lang), ("javascript", js_lang), ("typescript", js_lang)]:
+                p = Parser()
+                p.language = lang
+                self.parsers[name] = p
 
     def _detect_language(self, file_path: str) -> str:
         ext = Path(file_path).suffix.lower()

tests/test_dependency_analyzer.py

@@ -15,14 +15,15 @@ def test_build_graph_returns_dict(self, tmp_repo: Path):
     def test_finds_nodes(self, tmp_repo: Path):
         result = DependencyAnalyzer().build_graph(str(tmp_repo))
         file_ids = [n["id"] for n in result["nodes"]]
-        assert any("main.py" in f for f in file_ids)
+        assert any("main.py" in str(Path(f)) for f in file_ids)
 
     def test_resolves_internal_imports(self, tmp_repo: Path):
         result = DependencyAnalyzer().build_graph(str(tmp_repo))
         # dependencies.py imports from services/user_service.py
         edges = result["edges"]
         sources = [e["source"] for e in edges]
-        assert any("dependencies.py" in s for s in sources)
+        # use Path for cross-platform comparison
+        assert any(Path(s).name == "dependencies.py" for s in sources)
 
     def test_detects_circular_deps(self, tmp_path: Path):
         """Two files importing each other should be flagged."""