spl-core 7.6.1__py3-none-any.whl → 7.8.0__py3-none-any.whl

spl_core/__init__.py

@@ -1 +1 @@
-__version__ = "7.6.1"
+__version__ = "7.8.0"

spl_core/steps/collect_pr_changes.py

@@ -0,0 +1,97 @@
+import json
+from dataclasses import dataclass
+from pathlib import Path
+from typing import List, Optional
+
+from mashumaro.mixins.json import DataClassJSONMixin
+from py_app_dev.core.logging import logger
+from py_app_dev.core.subprocess import SubprocessExecutor
+from pypeline.domain.pipeline import PipelineStep
+from pypeline_semantic_release.steps import CIContext
+
+
+@dataclass
+class PR_Changes(DataClassJSONMixin):
+    """Dataclass for storing PR changes."""
+
+    ci_system: str
+    target_branch: str
+    current_branch: str
+    commit_id: str
+    changed_files: List[str]
+
+
+class CollectPRChanges(PipelineStep):
+    """Collect changed files from a pull request."""
+
+    def run(self) -> None:
+        logger.info(f"{self.get_name()}")
+        ci_contexts = self.execution_context.data_registry.find_data(CIContext)
+        if len(ci_contexts) > 0:
+            ci_context = ci_contexts[0]
+            logger.info(f"CI context: {ci_context}")
+            if ci_context.is_pull_request:
+                logger.info(f"Collecting PR changes between branch {ci_context.target_branch} and {ci_context.current_branch}")
+                changed_files = self._get_changed_files(ci_context.target_branch, ci_context.current_branch)
+                output_file = self.get_outputs()[0]
+                output_file.parent.mkdir(parents=True, exist_ok=True)
+                output_file.write_text(json.dumps(
+                    PR_Changes(
+                        ci_system=ci_context.ci_system.name, target_branch=ci_context.target_branch, current_branch=ci_context.current_branch, commit_id=self._get_commit_id(ci_context), changed_files=changed_files
+                    ).to_dict(),
+                    indent=2
+                ))
+                logger.info(f"PR changes saved to {output_file}")
+
+    def _get_changed_files(self, target_branch: str, current_branch: str) -> List[str]:
+        """Get list of changed files in the current branch/PR"""
+        try:
+            result = SubprocessExecutor(["git", "fetch", "origin", current_branch]).execute(handle_errors=False)
+            if result and result.returncode == 0:
+                result = SubprocessExecutor(["git", "diff", "--name-only", f"origin/{target_branch}...origin/{current_branch}"]).execute(handle_errors=False)
+
+                if result and result.returncode == 0 and result.stdout.strip():
+                    files = [line.strip() for line in result.stdout.strip().split("\n") if line.strip()]
+                    return files
+
+        except Exception as e:
+            logger.error(f"Git command failed: {e}")
+            return []  # Ensure a list is always returned
+        return []
+
+    def get_inputs(self) -> List[Path]:
+        return []
+
+    def get_outputs(self) -> List[Path]:
+        return [self.output_dir.joinpath("pr_changes.json")]
+
+    def get_name(self) -> str:
+        return self.__class__.__name__
+
+    def update_execution_context(self) -> None:
+        pass
+
+    def get_config(self) -> dict[str, str] | None:
+        """
+        Get runnable configuration.
+
+        (!) Do NOT put sensitive information in the configuration. It will be stored in a file.
+        """
+        return {"latest_commit": self._get_commit_id()}
+
+    def _get_commit_id(self, ci_context: Optional[CIContext] = None) -> str:
+        """Get the latest commit ID in the current branch."""
+        try:
+            if not ci_context:
+                ci_contexts = self.execution_context.data_registry.find_data(CIContext)
+                if len(ci_contexts) == 0:
+                    logger.info("No CI context found.")
+                    return ""
+                ci_context = ci_contexts[0]
+            if isinstance(ci_context, CIContext) and ci_context.is_pull_request:
+                result = SubprocessExecutor(["git", "rev-parse", f"origin/{ci_context.current_branch}"]).execute(handle_errors=False)
+                if result and result.returncode == 0:
+                    return result.stdout.strip()
+        except Exception as e:
+            logger.info(f"Failed to get commit ID: {e}")
+        return ""

spl_core/test_utils/spl_build.py

@@ -6,8 +6,7 @@ from pathlib import Path
 from typing import Callable, ClassVar, List, Optional
 
 from py_app_dev.core.logging import time_it
-
-from spl_core.common.command_line_executor import CommandLineExecutor
+from py_app_dev.core.subprocess import SubprocessExecutor
 
 
 @dataclass
@@ -110,7 +109,7 @@ class SplBuild:
             target = self.target if self.target else "all"
         return_code = -1
         while True:
-            cmd = [
+            cmd: list[str | Path] = [
                 "build.bat",
                 "-build",
                 "-buildKit",
@@ -125,7 +124,11 @@ class SplBuild:
                 cmd.extend(["-buildType", self.build_type])
             if additional_args:
                 cmd.extend(additional_args)
-            result = CommandLineExecutor().execute(cmd)
+            # Cast to Union[str, List[Union[str, Path]]] to satisfy SubprocessExecutor type
+            result = SubprocessExecutor(command=cmd).execute(handle_errors=False)
+            if result is None:
+                return_code = -1
+                break
             return_code = result.returncode
             if result.returncode:
                 if result.stdout:

{spl_core-7.6.1.dist-info → spl_core-7.8.0.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: spl-core
-Version: 7.6.1
+Version: 7.8.0
 Summary: Software Product Line Support for CMake
 License: MIT
 Author: Avengineers
@@ -23,6 +23,8 @@ Requires-Dist: kconfiglib (>=14.1,<15.0)
 Requires-Dist: mlx-traceability (>=10.0,<11.0)
 Requires-Dist: myst-parser (>=0.16)
 Requires-Dist: py-app-dev (>=2.1,<3.0)
+Requires-Dist: pypeline-runner (>=1,<=2)
+Requires-Dist: pypeline-semantic-release (>=0.4.1,<=0.5.0)
 Requires-Dist: sphinx (>=7.3,<8.0)
 Requires-Dist: sphinx-book-theme (>=1.1,<2.0)
 Requires-Dist: sphinx-copybutton (>=0.5,<0.6)

{spl_core-7.6.1.dist-info → spl_core-7.8.0.dist-info}/RECORD

@@ -1,7 +1,6 @@
-spl_core/__init__.py,sha256=LhyfBkEBbOgB0Jk4g3WaFpdYuMxHYgQiZOy-lA4hMwE,22
+spl_core/__init__.py,sha256=vZw91QDi0kVrQT2yAQVzBUsHnFXKxQOJTwxJLeLNYNo,22
 spl_core/__run.py,sha256=DphnN7_Bjiw_mOOztsHxTDHS8snz1g2MMWAaJpZxPKM,361
 spl_core/common/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
-spl_core/common/command_line_executor.py,sha256=fvrzZRFmxgY9pMf2byr3ufyQoT7zMd4hzruqObMgRs0,2622
 spl_core/common/path.py,sha256=sDujd3n4XP1XGjHc7ImXEdjihO6A8BOIDbKCf7HgQ0Y,462
 spl_core/common.cmake,sha256=sRWqQeK04iUpeM3beQZXIi5mpEQNyO751O-yBoFCrqk,35160
 spl_core/conan.cmake,sha256=i1AuyN-e8cczX7TI1nl6e3Y8N-EP-QXPVY7LG6NUyJY,41958
@@ -60,11 +59,12 @@ spl_core/kickstart/templates/project/tools/toolchains/clang/toolchain.cmake,sha2
 spl_core/kickstart/templates/project/tools/toolchains/gcc/toolchain.cmake,sha256=AmLzPyhTgfc_Dsre4AlsvSOkVGW6VswWvrEnUL8JLhA,183
 spl_core/main.py,sha256=_hL4j155WZMXog_755bgAH1PeUwvTdJZvVdVw9EWhvo,1225
 spl_core/spl.cmake,sha256=W8h-Zj-N302qxMrRG8MhoEkJ0io92bWGp4Y5r8LBQnc,4535
+spl_core/steps/collect_pr_changes.py,sha256=0zqjF55K3P8V3ZfCNQc33pYrCxhJJhBmAG4cVxVuy1U,4182
 spl_core/test_utils/archive_artifacts_collection.py,sha256=x7LH5dGIvssyhXsTFzB6rjgb5D2efKvHVpnjId3MNDk,5126
 spl_core/test_utils/base_variant_test_runner.py,sha256=Oq27lkJlpB_y-p2_8S23F5zjn1438HW148q-hQNz3EY,3795
-spl_core/test_utils/spl_build.py,sha256=o3MCWYRb769bXut_9EM_ZEweRKH7SSAuajh9277mKuI,8233
-spl_core-7.6.1.dist-info/LICENSE,sha256=UjjA0o8f5tT3wVm7qodTLAhPWLl6kgVyn9FPAd1VeYY,1099
-spl_core-7.6.1.dist-info/METADATA,sha256=xbBF90UeFv1q1v6GxZiHpH1Rcj4naz6v6CLWMZh_SZI,5157
-spl_core-7.6.1.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-spl_core-7.6.1.dist-info/entry_points.txt,sha256=18_sdVY93N1GVBiAHxQ_F9ZM-bBvOmVMOMn7PNe2EqU,45
-spl_core-7.6.1.dist-info/RECORD,,
+spl_core/test_utils/spl_build.py,sha256=bSM6hwhTH9aRryvUvtSPDfk_zoZuKEO5g3QXK4SIrco,8442
+spl_core-7.8.0.dist-info/LICENSE,sha256=UjjA0o8f5tT3wVm7qodTLAhPWLl6kgVyn9FPAd1VeYY,1099
+spl_core-7.8.0.dist-info/METADATA,sha256=cI-eKefU7d5Y4TL740Uubo_11uPvfd8d2ufGp7-vy-s,5257
+spl_core-7.8.0.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+spl_core-7.8.0.dist-info/entry_points.txt,sha256=18_sdVY93N1GVBiAHxQ_F9ZM-bBvOmVMOMn7PNe2EqU,45
+spl_core-7.8.0.dist-info/RECORD,,

spl_core/common/command_line_executor.py

@@ -1,70 +0,0 @@
-import locale
-import subprocess
-from pathlib import Path
-from typing import Dict, List, Optional
-
-
-class CommandLineExecutor:
-    def __init__(
-        self,
-        cwd: Optional[Path] = None,
-        env: Optional[Dict[str, str]] = None,
-    ):
-        """
-        A class for executing command line commands.
-
-        Args:
-        - cmd: A string or list of strings representing the command to be executed.
-        - cwd: An optional Path object representing the current working directory.
-        - env: An optional dictionary of environment variables to be used in the command execution.
-        """
-        self.current_working_directory = cwd
-        self.env = env
-
-    def execute(self, cmd: str | List[str]) -> subprocess.CompletedProcess[str]:
-        """
-        Executes the command and returns a CompletedProcess object.
-
-        Returns:
-        - A subprocess.CompletedProcess object representing the result of the command execution.
-        """
-        if isinstance(cmd, str):
-            command: str | list[str] = cmd
-            use_shell = True
-        else:
-            # Check if any argument contains quotes, which indicates shell processing is needed
-            has_quotes = any('"' in arg or "'" in arg for arg in cmd)
-            if has_quotes:
-                command = " ".join(cmd)
-                use_shell = True
-            else:
-                command = cmd
-                use_shell = False
-
-        output = ""
-        try:
-            print("=" * 120)
-            print(f"= Running command: {command}")
-            print("=" * 120)
-            with subprocess.Popen(
-                command,
-                cwd=str(self.current_working_directory or Path.cwd()),
-                stdout=subprocess.PIPE,
-                stderr=subprocess.STDOUT,
-                bufsize=1,
-                text=True,
-                env=self.env,
-                universal_newlines=True,
-                encoding=locale.getpreferredencoding(False),
-                errors="replace",
-                shell=use_shell,
-            ) as process:
-                if process.stdout:
-                    for line in process.stdout:
-                        print(line, end="")
-                        # We have to store the stdout content.
-                        # This is necessary because the stdout object is closed after we printed its content
-                        output += line
-        except Exception:
-            raise RuntimeError(f"Command '{command}' failed.")  # noqa: B904
-        return subprocess.CompletedProcess(args=command, returncode=process.returncode, stdout=output, stderr=None)