python-dependency-linter 0.3.0__tar.gz → 0.4.0__tar.gz

{python_dependency_linter-0.3.0 → python_dependency_linter-0.4.0}/CHANGELOG.md

@@ -2,6 +2,27 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.3.0] - 2026-03-30
+
+### Bug Fixes
+
+- Use exit code 2 for config file not found (#11)
+
+### Documentation
+
+- Add CONTRIBUTING.md and CLAUDE.md
+- Add PR title convention to template and CONTRIBUTING.md
+- Add release process to CONTRIBUTING.md and /release skill
+
+### Features
+
+- Resolve relative imports to absolute module names (#10)
+- Add include/exclude file filtering options (#12)
+
+### Miscellaneous
+
+- Add /commit skill for Claude Code
+- Add uv.lock for reproducible builds
 ## [0.2.0] - 2026-03-30
 
 ### Documentation

{python_dependency_linter-0.3.0 → python_dependency_linter-0.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-dependency-linter
-Version: 0.3.0
+Version: 0.4.0
 Summary: A dependency linter for Python projects
 License-Expression: MIT
 License-File: LICENSE
@@ -309,14 +309,11 @@ third_party = ["boto3"]
 ## CLI
 
 ```bash
-# Check with default config (.python-dependency-linter.yaml)
+# Check with auto-discovered config (searches upward from cwd)
 pdl check
 
-# Specify config file
+# Specify config file (project root = config file's parent directory)
 pdl check --config path/to/config.yaml
-
-# Specify project root
-pdl check --project-root path/to/project
 ```
 
 Exit codes:
@@ -325,10 +322,10 @@ Exit codes:
 - `1` — Violations found
 - `2` — Config file not found
 
-If no `--config` is given, the tool looks for `.python-dependency-linter.yaml` in the current directory. If the config file does not exist, the tool prints an error and exits with code `2`:
+If no `--config` is given, the tool searches upward from the current directory for `.python-dependency-linter.yaml` or `pyproject.toml` (with `[tool.python-dependency-linter]`). The config file's parent directory is used as the project root. If no config file is found, the tool prints an error and exits with code `2`:
 
 ```
-Error: Config file not found: .python-dependency-linter.yaml
+Error: Config file not found. Create .python-dependency-linter.yaml or configure [tool.python-dependency-linter] in pyproject.toml.
 ```
 
 ## Pre-commit
@@ -342,14 +339,14 @@ Add to `.pre-commit-config.yaml`:
     - id: python-dependency-linter
 ```
 
-To pass custom options (e.g., a different config file or project root):
+To pass custom options (e.g., a different config file):
 
 ```yaml
 - repo: https://github.com/heumsi/python-dependency-linter
   rev: v0.1.0
   hooks:
     - id: python-dependency-linter
-      args: [--config, custom-config.yaml, --project-root, src]
+      args: [--config, custom-config.yaml]
 ```
 
 ## License

{python_dependency_linter-0.3.0 → python_dependency_linter-0.4.0}/README.md

@@ -284,14 +284,11 @@ third_party = ["boto3"]
 ## CLI
 
 ```bash
-# Check with default config (.python-dependency-linter.yaml)
+# Check with auto-discovered config (searches upward from cwd)
 pdl check
 
-# Specify config file
+# Specify config file (project root = config file's parent directory)
 pdl check --config path/to/config.yaml
-
-# Specify project root
-pdl check --project-root path/to/project
 ```
 
 Exit codes:
@@ -300,10 +297,10 @@ Exit codes:
 - `1` — Violations found
 - `2` — Config file not found
 
-If no `--config` is given, the tool looks for `.python-dependency-linter.yaml` in the current directory. If the config file does not exist, the tool prints an error and exits with code `2`:
+If no `--config` is given, the tool searches upward from the current directory for `.python-dependency-linter.yaml` or `pyproject.toml` (with `[tool.python-dependency-linter]`). The config file's parent directory is used as the project root. If no config file is found, the tool prints an error and exits with code `2`:
 
 ```
-Error: Config file not found: .python-dependency-linter.yaml
+Error: Config file not found. Create .python-dependency-linter.yaml or configure [tool.python-dependency-linter] in pyproject.toml.
 ```
 
 ## Pre-commit
@@ -317,14 +314,14 @@ Add to `.pre-commit-config.yaml`:
     - id: python-dependency-linter
 ```
 
-To pass custom options (e.g., a different config file or project root):
+To pass custom options (e.g., a different config file):
 
 ```yaml
 - repo: https://github.com/heumsi/python-dependency-linter
   rev: v0.1.0
   hooks:
     - id: python-dependency-linter
-      args: [--config, custom-config.yaml, --project-root, src]
+      args: [--config, custom-config.yaml]
 ```
 
 ## License

{python_dependency_linter-0.3.0 → python_dependency_linter-0.4.0}/python_dependency_linter/cli.py

@@ -1,11 +1,12 @@
 from __future__ import annotations
 
+import fnmatch
 from pathlib import Path
 
 import click
 
 from python_dependency_linter.checker import check_import
-from python_dependency_linter.config import load_config
+from python_dependency_linter.config import find_config, load_config
 from python_dependency_linter.matcher import find_matching_rules, merge_rules
 from python_dependency_linter.parser import parse_imports
 from python_dependency_linter.reporter import format_violations
@@ -46,7 +47,7 @@ def _normalize_pattern(pattern: str, project_root: Path) -> str:
 
 
 def _matches_any(path: Path, patterns: list[str]) -> bool:
-    return any(path.match(p) for p in patterns)
+    return any(fnmatch.fnmatch(str(path), p) for p in patterns)
 
 
 def _find_python_files(
@@ -84,19 +85,29 @@ def main():
 @click.option(
     "--config",
     "config_path",
-    default=".python-dependency-linter.yaml",
+    default=None,
     help="Path to config file.",
 )
-@click.option("--project-root", default=".", help="Project root directory.")
-def check(config_path: str, project_root: str):
-    root = Path(project_root).resolve()
-    config_file = Path(config_path)
-
-    try:
-        config = load_config(config_file)
-    except FileNotFoundError as e:
-        click.echo(f"Error: {e}", err=True)
-        raise SystemExit(2)
+def check(config_path: str | None):
+    if config_path is not None:
+        config_file = Path(config_path)
+        if not config_file.exists():
+            click.echo(f"Error: Config file not found: {config_file}", err=True)
+            raise SystemExit(2)
+        root = config_file.resolve().parent
+    else:
+        config_file = find_config()
+        if config_file is None:
+            click.echo(
+                "Error: Config file not found. "
+                "Create .python-dependency-linter.yaml or configure "
+                "[tool.python-dependency-linter] in pyproject.toml.",
+                err=True,
+            )
+            raise SystemExit(2)
+        root = config_file.resolve().parent
+
+    config = load_config(config_file)
 
     all_violations = []
     python_files = _find_python_files(root, config.include, config.exclude)

{python_dependency_linter-0.3.0 → python_dependency_linter-0.4.0}/python_dependency_linter/config.py

@@ -62,14 +62,18 @@ def _load_yaml(path: Path) -> Config:
     )
 
 
-def _load_pyproject_toml(path: Path) -> Config:
+def _load_toml(path: Path) -> dict:
     try:
         import tomllib
     except ImportError:
         import tomli as tomllib  # type: ignore[no-redef]
 
     with open(path, "rb") as f:
-        data = tomllib.load(f)
+        return tomllib.load(f)
+
+
+def _load_pyproject_toml(path: Path) -> Config:
+    data = _load_toml(path)
     tool_config = data["tool"]["python-dependency-linter"]
     return Config(
         rules=_parse_rules(tool_config["rules"]),
@@ -78,6 +82,31 @@ def _load_pyproject_toml(path: Path) -> Config:
     )
 
 
+def _has_pdl_section(path: Path) -> bool:
+    """Check if a pyproject.toml contains [tool.python-dependency-linter]."""
+    data = _load_toml(path)
+    return "python-dependency-linter" in data.get("tool", {})
+
+
+_CONFIG_FILENAMES = [".python-dependency-linter.yaml", "pyproject.toml"]
+
+
+def find_config() -> Path | None:
+    """Search upward from cwd for a config file. Returns None if not found."""
+    current = Path.cwd().resolve()
+    while True:
+        for name in _CONFIG_FILENAMES:
+            candidate = current / name
+            if candidate.is_file():
+                if name == "pyproject.toml" and not _has_pdl_section(candidate):
+                    continue
+                return candidate
+        parent = current.parent
+        if parent == current:
+            return None
+        current = parent
+
+
 def load_config(path: Path) -> Config:
     if not path.exists():
         raise FileNotFoundError(f"Config file not found: {path}")

{python_dependency_linter-0.3.0 → python_dependency_linter-0.4.0}/tests/test_cli.py

@@ -7,7 +7,7 @@ from python_dependency_linter.cli import main
 FIXTURES = Path(__file__).parent / "fixtures"
 
 
-def test_cli_check_with_violations(tmp_path):
+def test_cli_check_with_violations(tmp_path, monkeypatch):
     config_content = """\
 rules:
   - name: domain-isolation
@@ -27,16 +27,15 @@ rules:
     dst = tmp_path / "contexts"
     shutil.copytree(src, dst)
 
+    monkeypatch.chdir(tmp_path)
     runner = CliRunner()
-    result = runner.invoke(
-        main, ["check", "--config", str(config_file), "--project-root", str(tmp_path)]
-    )
+    result = runner.invoke(main, ["check"])
     assert result.exit_code == 1
     assert "[domain-isolation]" in result.output
     assert "Found" in result.output
 
 
-def test_cli_check_no_violations(tmp_path):
+def test_cli_check_no_violations(tmp_path, monkeypatch):
     config_content = """\
 rules:
   - name: allow-all
@@ -55,15 +54,14 @@ rules:
     dst = tmp_path / "contexts"
     shutil.copytree(src, dst)
 
+    monkeypatch.chdir(tmp_path)
     runner = CliRunner()
-    result = runner.invoke(
-        main, ["check", "--config", str(config_file), "--project-root", str(tmp_path)]
-    )
+    result = runner.invoke(main, ["check"])
     assert result.exit_code == 0
     assert "No violations found." in result.output
 
 
-def test_cli_check_with_include(tmp_path):
+def test_cli_check_with_include(tmp_path, monkeypatch):
     """Files outside include paths should be skipped."""
     config_content = """\
 include:
@@ -74,7 +72,7 @@ rules:
     deny:
       third_party: [pydantic]
 """
-    config_file = tmp_path / "config.yaml"
+    config_file = tmp_path / ".python-dependency-linter.yaml"
     config_file.write_text(config_content)
 
     # Create files inside and outside include path
@@ -88,16 +86,52 @@ rules:
     (other / "__init__.py").write_text("")
     (other / "app.py").write_text("import pydantic\n")
 
+    monkeypatch.chdir(tmp_path)
     runner = CliRunner()
-    result = runner.invoke(
-        main, ["check", "--config", str(config_file), "--project-root", str(tmp_path)]
-    )
+    result = runner.invoke(main, ["check"])
     assert result.exit_code == 1
     assert "src/app.py" in result.output
     assert "other/app.py" not in result.output
 
 
-def test_cli_check_with_exclude(tmp_path):
+def test_cli_check_with_include_nested(tmp_path, monkeypatch):
+    """Include should match files in deeply nested subdirectories."""
+    config_content = """\
+include:
+  - src
+rules:
+  - name: domain-isolation
+    modules: "**"
+    deny:
+      third_party: [pydantic]
+"""
+    config_file = tmp_path / ".python-dependency-linter.yaml"
+    config_file.write_text(config_content)
+
+    # Create deeply nested files inside include path
+    nested = tmp_path / "src" / "contexts" / "analytics" / "domain"
+    nested.mkdir(parents=True)
+    (tmp_path / "src" / "__init__.py").write_text("")
+    (tmp_path / "src" / "contexts" / "__init__.py").write_text("")
+    (tmp_path / "src" / "contexts" / "analytics" / "__init__.py").write_text("")
+    (nested / "__init__.py").write_text("")
+    (nested / "models.py").write_text("import pydantic\n")
+
+    # Create files outside include path
+    other = tmp_path / "other"
+    other.mkdir()
+    (other / "__init__.py").write_text("")
+    (other / "app.py").write_text("import pydantic\n")
+
+    monkeypatch.chdir(tmp_path)
+    runner = CliRunner()
+    result = runner.invoke(main, ["check"])
+    assert result.exit_code == 1
+    assert "src/contexts/analytics/domain/models.py" in result.output
+    assert "other/app.py" not in result.output
+
+
+def test_cli_check_with_exclude(tmp_path, monkeypatch):
     """Files matching exclude patterns should be skipped."""
     config_content = """\
 exclude:
@@ -108,7 +142,7 @@ rules:
     deny:
       third_party: [pydantic]
 """
-    config_file = tmp_path / "config.yaml"
+    config_file = tmp_path / ".python-dependency-linter.yaml"
     config_file.write_text(config_content)
 
     src = tmp_path / "src"
@@ -121,16 +155,15 @@ rules:
     (generated / "__init__.py").write_text("")
     (generated / "models.py").write_text("import pydantic\n")
 
+    monkeypatch.chdir(tmp_path)
     runner = CliRunner()
-    result = runner.invoke(
-        main, ["check", "--config", str(config_file), "--project-root", str(tmp_path)]
-    )
+    result = runner.invoke(main, ["check"])
     assert result.exit_code == 1
     assert "src/app.py" in result.output
     assert "generated/" not in result.output
 
 
-def test_cli_check_with_include_and_exclude(tmp_path):
+def test_cli_check_with_include_and_exclude(tmp_path, monkeypatch):
     """Exclude should filter within included paths."""
     config_content = """\
 include:
@@ -143,7 +176,7 @@ rules:
     deny:
       third_party: [pydantic]
 """
-    config_file = tmp_path / "config.yaml"
+    config_file = tmp_path / ".python-dependency-linter.yaml"
     config_file.write_text(config_content)
 
     app = tmp_path / "src"
@@ -156,16 +189,51 @@ rules:
     (generated / "__init__.py").write_text("")
     (generated / "models.py").write_text("import pydantic\n")
 
+    monkeypatch.chdir(tmp_path)
     runner = CliRunner()
-    result = runner.invoke(
-        main, ["check", "--config", str(config_file), "--project-root", str(tmp_path)]
-    )
+    result = runner.invoke(main, ["check"])
     assert result.exit_code == 1
     assert "src/app.py" in result.output
     assert "generated/" not in result.output
 
 
-def test_cli_check_config_not_found():
+def test_cli_check_config_not_found(tmp_path, monkeypatch):
+    monkeypatch.chdir(tmp_path)
+    runner = CliRunner()
+    result = runner.invoke(main, ["check"])
+    assert result.exit_code == 2
+
+
+def test_cli_check_explicit_config_not_found():
     runner = CliRunner()
     result = runner.invoke(main, ["check", "--config", "nonexistent.yaml"])
     assert result.exit_code == 2
+    assert "not found" in result.output.lower()
+
+
+def test_cli_check_with_explicit_config(tmp_path, monkeypatch):
+    """--config should use the config file's parent as project root."""
+    project_dir = tmp_path / "project"
+    project_dir.mkdir()
+
+    config_content = """\
+rules:
+  - name: domain-isolation
+    modules: "**"
+    deny:
+      third_party: [pydantic]
+"""
+    config_file = project_dir / "custom-config.yaml"
+    config_file.write_text(config_content)
+
+    src = project_dir / "src"
+    src.mkdir()
+    (src / "__init__.py").write_text("")
+    (src / "app.py").write_text("import pydantic\n")
+
+    # Run from a different directory, but point --config to project_dir
+    monkeypatch.chdir(tmp_path)
+    runner = CliRunner()
+    result = runner.invoke(main, ["check", "--config", str(config_file)])
+    assert result.exit_code == 1
+    assert "src/app.py" in result.output

{python_dependency_linter-0.3.0 → python_dependency_linter-0.4.0}/tests/test_config.py

@@ -1,6 +1,6 @@
 from pathlib import Path
 
-from python_dependency_linter.config import load_config
+from python_dependency_linter.config import find_config, load_config
 
 FIXTURES = Path(__file__).parent / "fixtures"
 
@@ -81,3 +81,46 @@ def test_load_config_file_not_found():
 
     with pytest.raises(FileNotFoundError):
         load_config(Path("nonexistent.yaml"))
+
+
+def test_find_config_yaml_in_cwd(tmp_path, monkeypatch):
+    (tmp_path / ".python-dependency-linter.yaml").write_text("rules: []\n")
+    monkeypatch.chdir(tmp_path)
+    assert find_config() == tmp_path / ".python-dependency-linter.yaml"
+
+
+def test_find_config_yaml_in_parent(tmp_path, monkeypatch):
+    (tmp_path / ".python-dependency-linter.yaml").write_text("rules: []\n")
+    child = tmp_path / "sub"
+    child.mkdir()
+    monkeypatch.chdir(child)
+    assert find_config() == tmp_path / ".python-dependency-linter.yaml"
+
+
+def test_find_config_pyproject_toml(tmp_path, monkeypatch):
+    toml_content = "[tool.python-dependency-linter]\nrules = []\n"
+    (tmp_path / "pyproject.toml").write_text(toml_content)
+    monkeypatch.chdir(tmp_path)
+    assert find_config() == tmp_path / "pyproject.toml"
+
+
+def test_find_config_yaml_preferred_over_toml(tmp_path, monkeypatch):
+    """When both exist in the same directory, YAML wins."""
+    (tmp_path / ".python-dependency-linter.yaml").write_text("rules: []\n")
+    toml_content = "[tool.python-dependency-linter]\nrules = []\n"
+    (tmp_path / "pyproject.toml").write_text(toml_content)
+    monkeypatch.chdir(tmp_path)
+    assert find_config() == tmp_path / ".python-dependency-linter.yaml"
+
+
+def test_find_config_not_found(tmp_path, monkeypatch):
+    monkeypatch.chdir(tmp_path)
+    result = find_config()
+    assert result is None
+
+
+def test_find_config_skips_pyproject_without_section(tmp_path, monkeypatch):
+    """pyproject.toml without [tool.python-dependency-linter] should be skipped."""
+    (tmp_path / "pyproject.toml").write_text("[tool.other]\nfoo = 1\n")
+    monkeypatch.chdir(tmp_path)
+    assert find_config() is None