fix: get deps func

Author: shenxianpeng

cpp_linter_hooks/util.py

@@ -27,11 +27,16 @@ def get_version_from_dependency(tool: str) -> Optional[str]:
         if dep.startswith(f"{tool}=="):
             return dep.split("==")[1]
 
+    # Fallback to project.dependencies for backward compatibility
+    dependencies = data.get("project", {}).get("dependencies", [])
+    for dep in dependencies:
+        if dep.startswith(f"{tool}=="):
+            return dep.split("==")[1]
     return None
 
 
-DEFAULT_CLANG_FORMAT_VERSION = get_version_from_dependency("clang-format")
-DEFAULT_CLANG_TIDY_VERSION = get_version_from_dependency("clang-tidy")
+DEFAULT_CLANG_FORMAT_VERSION = get_version_from_dependency("clang-format") or "20.1.7"
+DEFAULT_CLANG_TIDY_VERSION = get_version_from_dependency("clang-tidy") or "20.1.0"
 
 
 CLANG_FORMAT_VERSIONS = [
@@ -179,6 +184,10 @@ def _resolve_install(tool: str, version: Optional[str]) -> Optional[Path]:
             else DEFAULT_CLANG_TIDY_VERSION
         )
 
+    # Additional safety check in case DEFAULT versions are None
+    if user_version is None:
+        user_version = "20.1.7" if tool == "clang-format" else "20.1.0"
+
     path = shutil.which(tool)
     if path:
         runtime_version = _get_runtime_version(tool)

tests/test_util.py

@@ -427,3 +427,22 @@ def test_version_lists_not_empty():
     assert len(CLANG_TIDY_VERSIONS) > 0
     assert all(isinstance(v, str) for v in CLANG_FORMAT_VERSIONS)
     assert all(isinstance(v, str) for v in CLANG_TIDY_VERSIONS)
+
+
+@pytest.mark.benchmark
+def test_resolve_install_with_none_default_version():
+    """Test _resolve_install when DEFAULT versions are None."""
+    with (
+        patch("shutil.which", return_value=None),
+        patch("cpp_linter_hooks.util.DEFAULT_CLANG_FORMAT_VERSION", None),
+        patch("cpp_linter_hooks.util.DEFAULT_CLANG_TIDY_VERSION", None),
+        patch(
+            "cpp_linter_hooks.util._install_tool",
+            return_value=Path("/usr/bin/clang-format"),
+        ) as mock_install,
+    ):
+        result = _resolve_install("clang-format", None)
+        assert result == Path("/usr/bin/clang-format")
+
+        # Should fallback to hardcoded version when DEFAULT is None
+        mock_install.assert_called_once_with("clang-format", "20.1.7")