[feature] Add support for npm, Docker, Ansible, and OpenWRT packages #123

- Added package type detection for npm, docker, ansible, and openwrt-agents
- Enhanced version parsing for all package types
- Updated bump_version() to handle all package type version formats
- Added 25 new tests for non-Python packages

Fixes #522

Author: pushpit kamboj

README.rst

@@ -81,4 +81,4 @@ Attribution
 - `Roboto webfont <https://www.google.com/fonts/specimen/Roboto>`_ is
   licensed under the `Apache License, Version 2.0
   <https://www.apache.org/licenses/LICENSE-2.0>`_. WOFF files extracted
-  using https://github.com/majodev/google-webfonts-helper.
+  using https://github.com/majodev/google-webfonts-helper.

openwisp_utils/releaser/config.py

@@ -1,8 +1,8 @@
 import ast
+import json
 import os
 import re
 import subprocess
-import json
 
 
 def get_package_name_from_setup():
@@ -22,19 +22,19 @@ def get_package_type_from_setup():
     """Detects package type based on config files present in the project."""
     if os.path.exists("setup.py"):
         return "python"
-    
+
     if os.path.exists("package.json"):
         return "npm"
-    
+
     if os.path.exists("docker-compose.yml"):
         return "docker"
-    
+
     if os.path.exists(".ansible-lint"):
         return "ansible"
-    
+
     if os.path.exists(".luacheckrc"):
         return "openwrt-agents"
-    
+
     return None
 
 
@@ -91,7 +91,9 @@ def load_config():
                     if version_match:
                         config["version_path"] = init_py_path
                         try:
-                            version_tuple = ast.literal_eval(f"({version_match.group(1)})")
+                            version_tuple = ast.literal_eval(
+                                f"({version_match.group(1)})"
+                            )
                             config["CURRENT_VERSION"] = list(version_tuple)
                         except (ValueError, SyntaxError):
                             config["CURRENT_VERSION"] = None
@@ -109,45 +111,67 @@ def load_config():
                             version_tuple, version_type = version_str.split("_", 1)
                         else:
                             version_tuple, version_type = version_str, "final"
-                        
+
                         parts = version_tuple.split(".")
                         if len(parts) < 3:
-                            raise ValueError(f"Version '{version_str}' does not have expected 3 parts (X.Y.Z)")  
-                        
+                            raise ValueError(
+                                f"Version '{version_str}' does not have expected 3 parts (X.Y.Z)"
+                            )
+
                         config["CURRENT_VERSION"] = [
                             int(parts[0]),
                             int(parts[1]),
                             int(parts[2]),
-                            version_type
+                            version_type,
                         ]
-                    except(ValueError, SyntaxError):
+                    except (ValueError, SyntaxError):
                         config["CURRENT_VERSION"] = None
     elif config["package_type"] == "docker":
         if os.path.exists("Makefile"):
             with open("Makefile", "r") as f:
                 content = f.read()
-                version_match = re.search(r"^OPENWISP_VERSION\s*=\s*([^\s]+)", content, re.MULTILINE)
+                version_match = re.search(
+                    r"^OPENWISP_VERSION\s*=\s*([^\s]+)", content, re.MULTILINE
+                )
                 if version_match:
                     config["version_path"] = "Makefile"
                     version_str = version_match.group(1)
                     parts = version_str.split(".")
                     if len(parts) < 3:
-                        raise ValueError(f"Version '{version_str}' does not have expected 3 parts (X.Y.Z)")
-                    config["CURRENT_VERSION"] = [int(parts[0]), int(parts[1]), int(parts[2]), "final"]
+                        raise ValueError(
+                            f"Version '{version_str}' does not have expected 3 parts (X.Y.Z)"
+                        )
+                    config["CURRENT_VERSION"] = [
+                        int(parts[0]),
+                        int(parts[1]),
+                        int(parts[2]),
+                        "final",
+                    ]
     elif config["package_type"] == "ansible":
         version_py_path = os.path.join("templates", "openwisp2", "version.py")
         if os.path.exists(version_py_path):
             with open(version_py_path, "r") as f:
                 content = f.read()
-                version_match = re.search(r"^__openwisp_version__\s*=\s*['\"]([^'\"]+)['\"]", content, re.MULTILINE)
+                version_match = re.search(
+                    r"^__openwisp_version__\s*=\s*['\"]([^'\"]+)['\"]",
+                    content,
+                    re.MULTILINE,
+                )
                 if version_match:
                     config["version_path"] = version_py_path
                     try:
                         version_str = version_match.group(1)
                         parts = version_str.split(".")
                         if len(parts) < 3:
-                            raise ValueError(f"Version '{version_str}' does not have expected 3 parts (X.Y.Z)")
-                        config["CURRENT_VERSION"] = [int(parts[0]), int(parts[1]), int(parts[2]), "final"]
+                            raise ValueError(
+                                f"Version '{version_str}' does not have expected 3 parts (X.Y.Z)"
+                            )
+                        config["CURRENT_VERSION"] = [
+                            int(parts[0]),
+                            int(parts[1]),
+                            int(parts[2]),
+                            "final",
+                        ]
                     except (ValueError, SyntaxError):
                         config["CURRENT_VERSION"] = None
     elif config["package_type"] == "openwrt-agents":
@@ -158,8 +182,15 @@ def load_config():
                     config["version_path"] = "VERSION"
                     parts = version_str.split(".")
                     if len(parts) < 3:
-                        raise ValueError(f"Version '{version_str}' does not have expected 3 parts (X.Y.Z)")
-                    config["CURRENT_VERSION"] = [int(parts[0]), int(parts[1]), int(parts[2]), "final"]
+                        raise ValueError(
+                            f"Version '{version_str}' does not have expected 3 parts (X.Y.Z)"
+                        )
+                    config["CURRENT_VERSION"] = [
+                        int(parts[0]),
+                        int(parts[1]),
+                        int(parts[2]),
+                        "final",
+                    ]
 
     possible_changelog_names = [
         "CHANGES.rst",

openwisp_utils/releaser/tests/conftest.py

@@ -65,10 +65,11 @@ def init_git_repo():
 def _create_package_json(path: Path, version="1.2.3"):
     """Helper to create a package.json file for npm projects."""
     import json
+
     package_json_content = {
         "name": "my-npm-package",
         "version": version,
-        "description": "Test npm package"
+        "description": "Test npm package",
     }
     (path / "package.json").write_text(json.dumps(package_json_content, indent=2))
 
@@ -96,7 +97,9 @@ def create_makefile():
 
 def _create_docker_compose(path: Path):
     """Helper to create a docker-compose.yml file."""
-    (path / "docker-compose.yml").write_text("version: '3'\nservices:\n  app:\n    image: test")
+    (path / "docker-compose.yml").write_text(
+        "version: '3'\nservices:\n  app:\n    image: test"
+    )
 
 
 @pytest.fixture

openwisp_utils/releaser/tests/test_config.py

@@ -221,14 +221,11 @@ def test_npm_package_with_underscore_suffix(
     assert config["CURRENT_VERSION"] == [1, 2, 3, "rc1"]
 
 
-def test_npm_package_missing_version(
-    project_dir, create_changelog, init_git_repo
-):
+def test_npm_package_missing_version(project_dir, create_changelog, init_git_repo):
     """Tests that npm package handles missing version gracefully."""
     import json
-    (project_dir / "package.json").write_text(
-        json.dumps({"name": "test-package"})
-    )
+
+    (project_dir / "package.json").write_text(json.dumps({"name": "test-package"}))
     create_changelog(project_dir)
     init_git_repo(project_dir)
 
@@ -290,7 +287,9 @@ def test_docker_package_invalid_version(
 ):
     """Tests docker package with invalid version in Makefile."""
     create_docker_compose(project_dir)
-    (project_dir / "Makefile").write_text("OPENWISP_VERSION = 1.2\n")  # Invalid: only 2 parts
+    (project_dir / "Makefile").write_text(
+        "OPENWISP_VERSION = 1.2\n"
+    )  # Invalid: only 2 parts
     create_changelog(project_dir)
     init_git_repo(project_dir)
 
@@ -300,7 +299,11 @@ def test_docker_package_invalid_version(
 
 # Ansible Package Tests
 def test_ansible_package_detection(
-    project_dir, create_ansible_lint, create_ansible_version_file, create_changelog, init_git_repo
+    project_dir,
+    create_ansible_lint,
+    create_ansible_version_file,
+    create_changelog,
+    init_git_repo,
 ):
     """Tests that ansible package type is detected when .ansible-lint exists."""
     create_ansible_lint(project_dir)
@@ -394,8 +397,12 @@ def test_openwrt_agents_invalid_version(
 
 # Package Type Priority Tests
 def test_package_type_priority_python_over_npm(
-    project_dir, create_setup_py, create_package_dir_with_version, 
-    create_package_json, create_changelog, init_git_repo
+    project_dir,
+    create_setup_py,
+    create_package_dir_with_version,
+    create_package_json,
+    create_changelog,
+    init_git_repo,
 ):
     """Tests that Python is detected first when both setup.py and package.json exist."""
     create_setup_py(project_dir)