fix: update get deps function to work

Author: shenxianpeng

cpp_linter_hooks/util.py

@@ -20,10 +20,13 @@ def get_version_from_dependency(tool: str) -> Optional[str]:
         return None
     with open(pyproject_path, "rb") as f:
         data = tomllib.load(f)
-    dependencies = data.get("project", {}).get("dependencies", [])
-    for dep in dependencies:
+    # First try project.optional-dependencies.tools
+    optional_deps = data.get("project", {}).get("optional-dependencies", {})
+    tools_deps = optional_deps.get("tools", [])
+    for dep in tools_deps:
         if dep.startswith(f"{tool}=="):
             return dep.split("==")[1]
+
     return None
 
 

pyproject.toml

@@ -45,6 +45,7 @@ source =  "https://github.com/cpp-linter/cpp-linter-hooks"
 tracker = "https://github.com/cpp-linter/cpp-linter-hooks/issues"
 
 [project.optional-dependencies]
+# only clang tools can added to this section to make hooks work
 tools = [
     "clang-format==20.1.7",
     "clang-tidy==20.1.0",

tests/test_util.py

@@ -102,11 +102,13 @@ def test_get_version_from_dependency_success():
     """Test get_version_from_dependency with valid pyproject.toml."""
     mock_toml_content = {
         "project": {
-            "dependencies": [
-                "clang-format==20.1.7",
-                "clang-tidy==20.1.0",
-                "other-package==1.0.0",
-            ]
+            "optional-dependencies": {
+                "tools": [
+                    "clang-format==20.1.7",
+                    "clang-tidy==20.1.0",
+                    "other-package==1.0.0",
+                ]
+            }
         }
     }
 
@@ -132,7 +134,9 @@ def test_get_version_from_dependency_missing_file():
 @pytest.mark.benchmark
 def test_get_version_from_dependency_missing_dependency():
     """Test get_version_from_dependency with missing dependency."""
-    mock_toml_content = {"project": {"dependencies": ["other-package==1.0.0"]}}
+    mock_toml_content = {
+        "project": {"optional-dependencies": {"tools": ["other-package==1.0.0"]}}
+    }
 
     with (
         patch("pathlib.Path.exists", return_value=True),
@@ -155,6 +159,30 @@ def test_get_version_from_dependency_malformed_toml():
         assert result is None
 
 
+@pytest.mark.benchmark
+def test_get_version_from_dependency_fallback_to_dependencies():
+    """Test get_version_from_dependency falls back to project.dependencies."""
+    mock_toml_content = {
+        "project": {
+            "dependencies": [
+                "clang-format==20.1.7",
+                "clang-tidy==20.1.0",
+                "other-package==1.0.0",
+            ]
+        }
+    }
+
+    with (
+        patch("pathlib.Path.exists", return_value=True),
+        patch("cpp_linter_hooks.util.tomllib.load", return_value=mock_toml_content),
+    ):
+        result = get_version_from_dependency("clang-format")
+        assert result == "20.1.7"
+
+        result = get_version_from_dependency("clang-tidy")
+        assert result == "20.1.0"
+
+
 # Tests for _resolve_version
 @pytest.mark.benchmark
 @pytest.mark.parametrize(