python-dependency-linter 0.5.0__tar.gz → 0.5.1__tar.gz

{python_dependency_linter-0.5.0 → python_dependency_linter-0.5.1}/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.5.0] - 2026-03-30
+
+### Features
+
+- Add named capture in module patterns for back-referencing in allow/deny (#17)
 ## [0.4.0] - 2026-03-30
 
 ### Bug Fixes

{python_dependency_linter-0.5.0 → python_dependency_linter-0.5.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-dependency-linter
-Version: 0.5.0
+Version: 0.5.1
 Summary: A dependency linter for Python projects
 License-Expression: MIT
 License-File: LICENSE
@@ -283,14 +283,19 @@ Named captures coexist with `*` and `**` wildcards. `{name}` always matches exac
 
 ### Submodule Matching
 
-When a pattern is used in `allow` or `deny`, it also matches submodules of the matched module. For example:
+When a pattern is used in `modules`, `allow`, or `deny`, it also matches submodules of the matched module.
+
+For example, the following rule applies to `contexts.boards.domain` as well as its submodules like `contexts.boards.domain.models` or `contexts.boards.domain.entities.metric`:
 
 ```yaml
-allow:
-  local: [contexts.*.domain]
+rules:
+  - name: domain-layer
+    modules: contexts.*.domain
+    allow:
+      local: [contexts.*.domain]
 ```
 
-This allows imports of `contexts.boards.domain` as well as its submodules like `contexts.boards.domain.models` or `contexts.boards.domain.entities.metric`.
+> **Note:** `contexts.*.domain` matches the module itself (`__init__.py`) **and** all submodules beneath it, while `contexts.*.domain.**` matches submodules only.
 
 ### Rule Merging
 
@@ -368,7 +373,7 @@ Add to `.pre-commit-config.yaml`:
 
 ```yaml
 - repo: https://github.com/heumsi/python-dependency-linter
-  rev: v0.1.0
+  rev: ''  # Use the tag you want to point at (e.g., v0.5.0)
   hooks:
     - id: python-dependency-linter
 ```
@@ -377,7 +382,7 @@ To pass custom options (e.g., a different config file):
 
 ```yaml
 - repo: https://github.com/heumsi/python-dependency-linter
-  rev: v0.1.0
+  rev: ''  # Use the tag you want to point at (e.g., v0.5.0)
   hooks:
     - id: python-dependency-linter
       args: [--config, custom-config.yaml]

{python_dependency_linter-0.5.0 → python_dependency_linter-0.5.1}/README.md

@@ -258,14 +258,19 @@ Named captures coexist with `*` and `**` wildcards. `{name}` always matches exac
 
 ### Submodule Matching
 
-When a pattern is used in `allow` or `deny`, it also matches submodules of the matched module. For example:
+When a pattern is used in `modules`, `allow`, or `deny`, it also matches submodules of the matched module.
+
+For example, the following rule applies to `contexts.boards.domain` as well as its submodules like `contexts.boards.domain.models` or `contexts.boards.domain.entities.metric`:
 
 ```yaml
-allow:
-  local: [contexts.*.domain]
+rules:
+  - name: domain-layer
+    modules: contexts.*.domain
+    allow:
+      local: [contexts.*.domain]
 ```
 
-This allows imports of `contexts.boards.domain` as well as its submodules like `contexts.boards.domain.models` or `contexts.boards.domain.entities.metric`.
+> **Note:** `contexts.*.domain` matches the module itself (`__init__.py`) **and** all submodules beneath it, while `contexts.*.domain.**` matches submodules only.
 
 ### Rule Merging
 
@@ -343,7 +348,7 @@ Add to `.pre-commit-config.yaml`:
 
 ```yaml
 - repo: https://github.com/heumsi/python-dependency-linter
-  rev: v0.1.0
+  rev: ''  # Use the tag you want to point at (e.g., v0.5.0)
   hooks:
     - id: python-dependency-linter
 ```
@@ -352,7 +357,7 @@ To pass custom options (e.g., a different config file):
 
 ```yaml
 - repo: https://github.com/heumsi/python-dependency-linter
-  rev: v0.1.0
+  rev: ''  # Use the tag you want to point at (e.g., v0.5.0)
   hooks:
     - id: python-dependency-linter
       args: [--config, custom-config.yaml]

{python_dependency_linter-0.5.0 → python_dependency_linter-0.5.1}/python_dependency_linter/matcher.py

@@ -59,12 +59,31 @@ def _match_with_captures(
     return False
 
 
+def match_pattern_with_captures_or_submodule(
+    pattern: str, module: str
+) -> dict[str, str] | None:
+    """Match pattern exactly or treat module as a submodule of the pattern."""
+    captures = match_pattern_with_captures(pattern, module)
+    if captures is not None:
+        return captures
+    # Check if a prefix of the module matches the pattern.
+    # e.g. "contexts.*.domain" should match "contexts.boards.domain.models"
+    module_parts = module.split(".")
+    pattern_parts = pattern.split(".")
+    if len(module_parts) > len(pattern_parts):
+        prefix = ".".join(module_parts[: len(pattern_parts)])
+        captures = match_pattern_with_captures(pattern, prefix)
+        if captures is not None:
+            return captures
+    return None
+
+
 def find_matching_rules(
     module: str, rules: list[Rule]
 ) -> list[tuple[Rule, dict[str, str]]]:
     result = []
     for r in rules:
-        captures = match_pattern_with_captures(r.modules, module)
+        captures = match_pattern_with_captures_or_submodule(r.modules, module)
         if captures is not None:
             result.append((r, captures))
     return result

{python_dependency_linter-0.5.0 → python_dependency_linter-0.5.1}/tests/test_matcher.py

@@ -214,3 +214,62 @@ def test_capture_after_double_star():
 def test_capture_after_double_star_backtrack():
     result = match_pattern_with_captures("**.{x}.end", "a.b.c.end")
     assert result == {"x": "c"}
+
+
+def test_find_matching_rules_submodule():
+    """modules pattern should match submodules automatically."""
+    rules = [
+        Rule(
+            name="domain-layer",
+            modules="contexts.*.domain",
+            allow=AllowDeny(local=["contexts.*.domain"]),
+        ),
+    ]
+    # Exact match still works
+    matched = find_matching_rules("contexts.boards.domain", rules)
+    assert len(matched) == 1
+    assert matched[0][0].name == "domain-layer"
+
+    # Submodule should also match
+    matched = find_matching_rules("contexts.boards.domain.models", rules)
+    assert len(matched) == 1
+    assert matched[0][0].name == "domain-layer"
+
+    # Deeper submodule should also match
+    matched = find_matching_rules("contexts.boards.domain.entities.metric", rules)
+    assert len(matched) == 1
+    assert matched[0][0].name == "domain-layer"
+
+    # Non-matching module should not match
+    matched = find_matching_rules("contexts.boards.application.service", rules)
+    assert len(matched) == 0
+
+
+def test_find_matching_rules_submodule_with_captures():
+    """modules submodule matching should preserve captures."""
+    rules = [
+        Rule(
+            name="domain-layer",
+            modules="contexts.{context}.domain",
+            allow=AllowDeny(local=["contexts.{context}.domain"]),
+        ),
+    ]
+    matched = find_matching_rules("contexts.boards.domain.models", rules)
+    assert len(matched) == 1
+    rule, captures = matched[0]
+    assert rule.name == "domain-layer"
+    assert captures == {"context": "boards"}
+
+
+def test_find_matching_rules_submodule_exact_pattern():
+    """Exact (no wildcard) modules pattern should also match submodules."""
+    rules = [
+        Rule(
+            name="shared",
+            modules="src.shared.domain",
+            allow=AllowDeny(local=["src.shared.domain"]),
+        ),
+    ]
+    matched = find_matching_rules("src.shared.domain.entity.user", rules)
+    assert len(matched) == 1
+    assert matched[0][0].name == "shared"