[feature] Support Python projects using pyproject.toml #706

Added support for pyproject.toml as a package type indicator and version
source for the releaser tool.

Changes:
- Detect pyproject.toml as a Python package marker (setup.py still takes priority)
- Read version from [project] version field in pyproject.toml (PEP 621)
- Bump version by updating the version string in pyproject.toml
- Graceful fallback when tomllib is not available (Python <3.11)
- Added tests for detection, version reading, version bumping, and priority

Author: BHARATH0153

openwisp_utils/releaser/config.py

@@ -4,6 +4,14 @@
 import re
 import subprocess
 
+try:
+    import tomllib
+except ImportError:
+    try:
+        import tomli as tomllib
+    except ImportError:
+        tomllib = None
+
 
 def get_package_name_from_setup():
     """Parses setup.py to find the package name without raising an error."""
@@ -22,6 +30,7 @@ def get_package_type_from_setup():
     """Detects package type based on config files present in the project."""
     package_type_files = {
         "setup.py": "python",
+        "pyproject.toml": "python",
         "package.json": "npm",
         "docker-compose.yml": "docker",
         ".ansible-lint": "ansible",
@@ -52,22 +61,55 @@ def detect_changelog_style(changelog_path):
 def _handle_python_version(config):
     """Handles version detection for Python packages."""
     project_name = get_package_name_from_setup()
-    if not project_name:
+    if project_name:
+        package_directory = project_name.replace("-", "_")
+        init_py_path = os.path.join(package_directory, "__init__.py")
+        if os.path.exists(init_py_path):
+            with open(init_py_path, "r") as f:
+                content = f.read()
+                version_match = re.search(r"^VERSION\s*=\s*\((.*)\)", content, re.M)
+                if version_match:
+                    config["version_path"] = init_py_path
+                    try:
+                        version_tuple = ast.literal_eval(
+                            f"({version_match.group(1)})"
+                        )
+                        config["CURRENT_VERSION"] = list(version_tuple)
+                    except (ValueError, SyntaxError):
+                        config["CURRENT_VERSION"] = None
+                    return
+    _handle_pyproject_toml_version(config)
+
+
+def _handle_pyproject_toml_version(config):
+    """Handles version detection from pyproject.toml."""
+    if not os.path.exists("pyproject.toml"):
         return
-    package_directory = project_name.replace("-", "_")
-    init_py_path = os.path.join(package_directory, "__init__.py")
-    if not os.path.exists(init_py_path):
+    if tomllib is None:
         return
-    with open(init_py_path, "r") as f:
-        content = f.read()
-        version_match = re.search(r"^VERSION\s*=\s*\((.*)\)", content, re.M)
-        if version_match:
-            config["version_path"] = init_py_path
-            try:
-                version_tuple = ast.literal_eval(f"({version_match.group(1)})")
-                config["CURRENT_VERSION"] = list(version_tuple)
-            except (ValueError, SyntaxError):
-                config["CURRENT_VERSION"] = None
+    with open("pyproject.toml", "rb") as f:
+        try:
+            data = tomllib.load(f)
+        except Exception:
+            return
+    project = data.get("project", {})
+    version_str = project.get("version")
+    if not version_str:
+        return
+    try:
+        parts = version_str.split(".")
+        if len(parts) != 3:
+            return
+        config["package_type"] = "pyproject"
+        config["version_path"] = "pyproject.toml"
+        config["CURRENT_VERSION"] = [
+            int(parts[0]),
+            int(parts[1]),
+            int(parts[2]),
+            "final",
+        ]
+    except (ValueError, TypeError):
+        config["CURRENT_VERSION"] = None
 
 
 def _handle_npm_version(config):
@@ -194,6 +236,7 @@ def _handle_openwrt_version(config):
 # Maps package types to their version detection handlers
 PACKAGE_VERSION_HANDLERS = {
     "python": _handle_python_version,
+    "pyproject": _handle_pyproject_toml_version,
     "npm": _handle_npm_version,
     "docker": _handle_docker_version,
     "ansible": _handle_ansible_version,

openwisp_utils/releaser/tests/test_config.py

@@ -424,3 +424,66 @@ def test_no_package_type_detected(project_dir, create_changelog, init_git_repo):
     assert config["package_type"] is None
     assert config["version_path"] is None
     assert config["CURRENT_VERSION"] is None
+
+
+# pyproject.toml Package Tests
+def test_pyproject_toml_package_detection(
+    project_dir, create_changelog, init_git_repo
+):
+    """Tests that python package type is detected when pyproject.toml exists."""
+    (project_dir / "pyproject.toml").write_text(
+        '[project]\nname = "my-package"\nversion = "1.2.3"\n'
+    )
+    create_changelog(project_dir)
+    init_git_repo(project_dir)
+    config = load_config()
+    assert config["package_type"] == "pyproject"
+    assert config["version_path"] == "pyproject.toml"
+    assert config["CURRENT_VERSION"] == [1, 2, 3, "final"]
+
+
+def test_pyproject_toml_missing_version(
+    project_dir, create_changelog, init_git_repo
+):
+    """Tests pyproject.toml without a version field."""
+    (project_dir / "pyproject.toml").write_text('[project]\nname = "my-package"\n')
+    create_changelog(project_dir)
+    init_git_repo(project_dir)
+    config = load_config()
+    assert config["version_path"] is None
+    assert config["CURRENT_VERSION"] is None
+
+
+def test_pyproject_toml_invalid_version(
+    project_dir, create_changelog, init_git_repo
+):
+    """Tests pyproject.toml with an invalid version format."""
+    (project_dir / "pyproject.toml").write_text(
+        '[project]\nname = "my-package"\nversion = "1.2"\n'
+    )
+    create_changelog(project_dir)
+    init_git_repo(project_dir)
+    config = load_config()
+    assert config["version_path"] is None
+    assert config["CURRENT_VERSION"] is None
+
+
+def test_setup_py_takes_priority_over_pyproject_toml(
+    project_dir,
+    create_setup_py,
+    create_package_dir_with_version,
+    create_changelog,
+    init_git_repo,
+):
+    """Tests that setup.py takes priority over pyproject.toml."""
+    create_setup_py(project_dir)
+    create_package_dir_with_version(project_dir)
+    (project_dir / "pyproject.toml").write_text(
+        '[project]\nname = "my-package"\nversion = "9.9.9"\n'
+    )
+    create_changelog(project_dir)
+    init_git_repo(project_dir)
+    config = load_config()
+    assert config["package_type"] == "python"
+    assert config["version_path"] == "my_test_package/__init__.py"
+    assert config["CURRENT_VERSION"] == [1, 2, 3, "final"]

openwisp_utils/releaser/tests/test_version_bumping.py

@@ -368,3 +368,73 @@ def test_bump_version_openwrt():
     assert result is True
     written_content = m_open().write.call_args[0][0]
     assert "1.2.4" in written_content
+
+
+# pyproject.toml Package Version Tests
+PYPROJECT_TOML_CONTENT = """[build-system]
+requires = ["setuptools"]
+build-backend = "setuptools.backends._legacy:_Backend"
+
+[project]
+name = "my-package"
+version = "1.2.3"
+description = "A test package"
+"""
+
+PYPROJECT_TOML_BUMPED = """[build-system]
+requires = ["setuptools"]
+build-backend = "setuptools.backends._legacy:_Backend"
+
+[project]
+name = "my-package"
+version = "1.2.4"
+description = "A test package"
+"""
+
+
+def test_get_current_version_pyproject():
+    """Tests getting current version from pyproject.toml."""
+    config = {
+        "package_type": "pyproject",
+        "version_path": "pyproject.toml",
+        "CURRENT_VERSION": [1, 2, 3, "final"],
+    }
+    version, version_type = get_current_version(config)
+    assert version == "1.2.3"
+    assert version_type == "final"
+
+
+def test_bump_version_pyproject_toml():
+    """Tests bumping version in pyproject.toml."""
+    config = {
+        "package_type": "pyproject",
+        "version_path": "pyproject.toml",
+        "CURRENT_VERSION": [1, 2, 3, "final"],
+    }
+    m_open = mock_open(read_data=PYPROJECT_TOML_CONTENT)
+    with patch("os.path.exists", return_value=True), patch("builtins.open", m_open):
+        result = bump_version(config, "1.2.4")
+    assert result is True
+    written_content = m_open().write.call_args[0][0]
+    assert 'version = "1.2.4"' in written_content
+
+
+def test_bump_version_pyproject_toml_not_found():
+    """Tests error when version field is missing in pyproject.toml."""
+    config = {
+        "package_type": "pyproject",
+        "version_path": "pyproject.toml",
+        "CURRENT_VERSION": [1, 2, 3, "final"],
+    }
+    content_no_version = """[build-system]
+requires = ["setuptools"]
+build-backend = "setuptools.backends._legacy:_Backend"
+
+[project]
+name = "my-package"
+description = "No version here"
+"""
+    m_open = mock_open(read_data=content_no_version)
+    with patch("os.path.exists", return_value=True), patch("builtins.open", m_open):
+        with pytest.raises(RuntimeError, match="Failed to find"):
+            bump_version(config, "1.2.4")

openwisp_utils/releaser/version.py

@@ -3,6 +3,14 @@
 import subprocess
 import sys
 
+try:
+    import tomllib
+except ImportError:
+    try:
+        import tomli as tomllib
+    except ImportError:
+        tomllib = None
+
 import questionary
 
 
@@ -54,6 +62,28 @@ def _bump_python_version(content, new_version, version_path):
     )
 
 
+def _bump_pyproject_toml_version(content, new_version, version_path):
+    """Handles version bumping in pyproject.toml."""
+    if tomllib is not None:
+        try:
+            data = tomllib.loads(content)
+        except Exception:
+            pass
+        else:
+            current = data.get("project", {}).get("version", "")
+            if current:
+                return content.replace(
+                    f'version = "{current}"', f'version = "{new_version}"'
+                )
+    return _bump_with_regex(
+        content,
+        r'^version\s*=\s*"([^"]+)"',
+        f'version = "{new_version}"',
+        version_path,
+        "version in pyproject.toml",
+    )
+
+
 def _bump_npm_version(content, new_version, version_path):
     """Handles version bumping for NPM packages."""
     try:
@@ -119,6 +149,7 @@ def _bump_openwrt_version(content, new_version, version_path):
 # Maps package types to their version bump handlers
 VERSION_BUMP_HANDLERS = {
     "python": _bump_python_version,
+    "pyproject": _bump_pyproject_toml_version,
     "npm": _bump_npm_version,
     "docker": _bump_docker_version,
     "ansible": _bump_ansible_version,