mcp-souschef 2.1.2__py3-none-any.whl → 2.5.3__py3-none-any.whl

souschef/parsers/metadata.py

@@ -45,6 +45,94 @@ def read_cookbook_metadata(path: str) -> str:
         return f"An error occurred: {e}"
 
 
+def parse_cookbook_metadata(path: str) -> dict[str, str | list[str]]:
+    """
+    Parse Chef cookbook metadata.rb file and return as dictionary.
+
+    Args:
+        path: Path to the metadata.rb file.
+
+    Returns:
+        Dictionary containing extracted metadata fields.
+
+    """
+    try:
+        file_path = _normalize_path(path)
+        content = file_path.read_text(encoding="utf-8")
+
+        metadata = _extract_metadata(content)
+        return metadata
+
+    except ValueError as e:
+        return {"error": str(e)}
+    except FileNotFoundError:
+        return {"error": ERROR_FILE_NOT_FOUND.format(path=path)}
+    except IsADirectoryError:
+        return {"error": ERROR_IS_DIRECTORY.format(path=path)}
+    except PermissionError:
+        return {"error": ERROR_PERMISSION_DENIED.format(path=path)}
+    except Exception as e:
+        return {"error": f"An error occurred: {e}"}
+
+
+def _scan_cookbook_directory(
+    cookbook_path, dir_name: str
+) -> tuple[str, list[str]] | None:
+    """
+    Scan a single cookbook directory for files.
+
+    Args:
+        cookbook_path: Path to the cookbook root.
+        dir_name: Name of the subdirectory to scan.
+
+    Returns:
+        Tuple of (dir_name, files) if directory exists and has files, None otherwise.
+
+    """
+    dir_path = _safe_join(cookbook_path, dir_name)
+    if not dir_path.exists() or not dir_path.is_dir():
+        return None
+
+    files = [f.name for f in dir_path.iterdir() if f.is_file()]
+    return (dir_name, files) if files else None
+
+
+def _collect_cookbook_structure(cookbook_path) -> dict[str, list[str]]:
+    """
+    Collect all standard cookbook directories and their files.
+
+    Args:
+        cookbook_path: Path to the cookbook root.
+
+    Returns:
+        Dictionary mapping directory names to file lists.
+
+    """
+    structure = {}
+    common_dirs = [
+        "recipes",
+        "attributes",
+        "templates",
+        "files",
+        "resources",
+        "providers",
+        "libraries",
+        "definitions",
+    ]
+
+    for dir_name in common_dirs:
+        result = _scan_cookbook_directory(cookbook_path, dir_name)
+        if result:
+            structure[result[0]] = result[1]
+
+    # Check for metadata.rb
+    metadata_path = _safe_join(cookbook_path, METADATA_FILENAME)
+    if metadata_path.exists():
+        structure["metadata"] = [METADATA_FILENAME]
+
+    return structure
+
+
 def list_cookbook_structure(path: str) -> str:
     """
     List the structure of a Chef cookbook directory.
@@ -62,29 +150,7 @@ def list_cookbook_structure(path: str) -> str:
         if not cookbook_path.is_dir():
             return f"Error: {path} is not a directory"
 
-        structure = {}
-        common_dirs = [
-            "recipes",
-            "attributes",
-            "templates",
-            "files",
-            "resources",
-            "providers",
-            "libraries",
-            "definitions",
-        ]
-
-        for dir_name in common_dirs:
-            dir_path = _safe_join(cookbook_path, dir_name)
-            if dir_path.exists() and dir_path.is_dir():
-                files = [f.name for f in dir_path.iterdir() if f.is_file()]
-                if files:
-                    structure[dir_name] = files
-
-        # Check for metadata.rb
-        metadata_path = _safe_join(cookbook_path, METADATA_FILENAME)
-        if metadata_path.exists():
-            structure["metadata"] = [METADATA_FILENAME]
+        structure = _collect_cookbook_structure(cookbook_path)
 
         if not structure:
             return f"Warning: No standard cookbook structure found in {path}"

souschef/profiling.py

@@ -0,0 +1,568 @@
+"""
+Performance profiling and optimization utilities for SousChef.
+
+This module provides tools for profiling parsing operations, identifying bottlenecks,
+and generating performance reports for large cookbook migrations.
+"""
+
+import cProfile
+import io
+import pstats
+import time
+import tracemalloc
+from collections.abc import Callable, Iterator
+from contextlib import contextmanager
+from dataclasses import dataclass, field
+from pathlib import Path
+from typing import Any
+
+from souschef.core.errors import SousChefError
+
+
+@dataclass
+class ProfileResult:
+    """
+    Results from a profiling operation.
+
+    Attributes:
+        operation_name: Name of the operation profiled.
+        execution_time: Total execution time in seconds.
+        peak_memory: Peak memory usage in bytes.
+        function_stats: Statistics for individual function calls.
+        context: Additional context about the operation.
+
+    """
+
+    operation_name: str
+    execution_time: float
+    peak_memory: int
+    function_stats: dict[str, Any] = field(default_factory=dict)
+    context: dict[str, Any] = field(default_factory=dict)
+
+    def __str__(self) -> str:
+        """
+        Format profile result as readable string.
+
+        Returns:
+            Formatted profile summary.
+
+        """
+        return f"""
+Profile: {self.operation_name}
+Execution Time: {self.execution_time:.3f}s
+Peak Memory: {self.peak_memory / 1024 / 1024:.2f}MB
+Context: {self.context}
+"""
+
+
+@dataclass
+class PerformanceReport:
+    """
+    Comprehensive performance report for cookbook migration.
+
+    Attributes:
+        cookbook_name: Name of the cookbook profiled.
+        total_time: Total processing time in seconds.
+        total_memory: Total peak memory usage in bytes.
+        operation_results: Results for individual operations.
+        recommendations: Performance optimization recommendations.
+
+    """
+
+    cookbook_name: str
+    total_time: float
+    total_memory: int
+    operation_results: list[ProfileResult] = field(default_factory=list)
+    recommendations: list[str] = field(default_factory=list)
+
+    def add_result(self, result: ProfileResult) -> None:
+        """
+        Add a profile result to the report.
+
+        Args:
+            result: Profile result to add.
+
+        """
+        self.operation_results.append(result)
+
+    def add_recommendation(self, recommendation: str) -> None:
+        """
+        Add a performance recommendation.
+
+        Args:
+            recommendation: Recommendation text.
+
+        """
+        self.recommendations.append(recommendation)
+
+    def __str__(self) -> str:
+        """
+        Format performance report as readable string.
+
+        Returns:
+            Formatted report.
+
+        """
+        report_lines = [
+            f"\n{'=' * 80}",
+            f"Performance Report: {self.cookbook_name}",
+            f"{'=' * 80}",
+            f"Total Time: {self.total_time:.3f}s",
+            f"Total Peak Memory: {self.total_memory / 1024 / 1024:.2f}MB",
+            "\nOperation Breakdown:",
+            f"{'-' * 80}",
+        ]
+
+        for result in self.operation_results:
+            report_lines.extend(
+                [
+                    f"\n{result.operation_name}:",
+                    (
+                        f"  Time: {result.execution_time:.3f}s "
+                        f"({result.execution_time / self.total_time * 100:.1f}%)"
+                    ),
+                    f"  Memory: {result.peak_memory / 1024 / 1024:.2f}MB",
+                ]
+            )
+            if result.context:
+                for key, value in result.context.items():
+                    report_lines.append(f"  {key}: {value}")
+
+        if self.recommendations:
+            report_lines.extend([f"\n{'-' * 80}", "Recommendations:", f"{'-' * 80}"])
+            for i, rec in enumerate(self.recommendations, 1):
+                report_lines.append(f"{i}. {rec}")
+
+        report_lines.append(f"{'=' * 80}\n")
+        return "\n".join(report_lines)
+
+
+@contextmanager
+def profile_operation(
+    operation_name: str, context: dict[str, Any] | None = None
+) -> Iterator[ProfileResult]:
+    """
+    Context manager for profiling an operation.
+
+    Args:
+        operation_name: Name of the operation being profiled.
+        context: Additional context information.
+
+    Yields:
+        ProfileResult that will be populated with profiling data.
+
+    Example:
+        >>> with profile_operation("parse_recipe", {"file": "default.rb"}) as result:
+        ...     parse_recipe("/path/to/recipe.rb")
+        >>> print(result.execution_time)
+
+    """
+    result = ProfileResult(
+        operation_name=operation_name,
+        execution_time=0.0,
+        peak_memory=0,
+        context=context or {},
+    )
+
+    tracemalloc.start()
+    start_time = time.perf_counter()
+
+    try:
+        yield result
+    finally:
+        end_time = time.perf_counter()
+        _, peak = tracemalloc.get_traced_memory()
+        tracemalloc.stop()
+
+        result.execution_time = end_time - start_time
+        result.peak_memory = peak
+
+
+def profile_function(
+    func: Callable[..., Any],
+    *args: Any,
+    operation_name: str | None = None,
+    **kwargs: Any,
+) -> tuple[Any, ProfileResult]:
+    """
+    Profile a function call and return result with profiling data.
+
+    Args:
+        func: Function to profile.
+        *args: Positional arguments for the function.
+        operation_name: Name for the operation (defaults to function name).
+        **kwargs: Keyword arguments for the function.
+
+    Returns:
+        Tuple of (function_result, profile_result).
+
+    Example:
+        >>> result, profile = profile_function(parse_recipe, "/path/to/recipe.rb")
+        >>> print(f"Took {profile.execution_time:.3f}s")
+
+    """
+    op_name = operation_name or func.__name__
+
+    with profile_operation(op_name) as profile:
+        result = func(*args, **kwargs)
+
+    return result, profile
+
+
+def detailed_profile_function(
+    func: Callable[..., Any],
+    *args: Any,
+    operation_name: str | None = None,
+    top_n: int = 20,
+    **kwargs: Any,
+) -> tuple[Any, ProfileResult]:
+    """
+    Profile a function with detailed call statistics.
+
+    Args:
+        func: Function to profile.
+        *args: Positional arguments for the function.
+        operation_name: Name for the operation (defaults to function name).
+        top_n: Number of top functions to include in stats.
+        **kwargs: Keyword arguments for the function.
+
+    Returns:
+        Tuple of (function_result, profile_result with detailed stats).
+
+    Example:
+        >>> result, profile = detailed_profile_function(parse_recipe, path)
+        >>> print(profile.function_stats["total_calls"])
+
+    """
+    op_name = operation_name or func.__name__
+
+    # Memory and time profiling
+    tracemalloc.start()
+    start_time = time.perf_counter()
+
+    # Function call profiling
+    profiler = cProfile.Profile()
+    profiler.enable()
+
+    try:
+        result = func(*args, **kwargs)
+    finally:
+        profiler.disable()
+        end_time = time.perf_counter()
+        _, peak = tracemalloc.get_traced_memory()
+        tracemalloc.stop()
+
+    # Extract statistics
+    stats_buffer = io.StringIO()
+    stats = pstats.Stats(profiler, stream=stats_buffer)
+    stats.strip_dirs()
+    stats.sort_stats("cumulative")
+    stats.print_stats(top_n)
+
+    # Get call counts from stats
+    # pstats.Stats has a stats attribute that's a dict in runtime,
+    # but mypy doesn't recognize it. We use getattr to bypass type checking.
+    stats_dict = getattr(stats, "stats", {})
+    total_calls = sum(cc[0] for cc in stats_dict.values()) if stats_dict else 0
+    primitive_calls = sum(cc[1] for cc in stats_dict.values()) if stats_dict else 0
+
+    profile_result = ProfileResult(
+        operation_name=op_name,
+        execution_time=end_time - start_time,
+        peak_memory=peak,
+        function_stats={
+            "total_calls": total_calls,
+            "primitive_calls": primitive_calls,
+            "top_functions": stats_buffer.getvalue(),
+        },
+    )
+
+    return result, profile_result
+
+
+def _profile_directory_files(
+    directory: Path,
+    parse_func: Callable[[str], str],
+    operation_name: str,
+    file_pattern: str = "*.rb",
+) -> ProfileResult | None:
+    """
+    Profile parsing operations for files in a directory.
+
+    Args:
+        directory: Directory containing files to parse.
+        parse_func: Function to parse individual files.
+        operation_name: Name for the profiling operation.
+        file_pattern: Glob pattern to match files.
+
+    Returns:
+        ProfileResult with aggregated stats, or None if no files found.
+
+    """
+    if not directory.exists():
+        return None
+
+    file_count = 0
+    total_time = 0.0
+
+    for file_path in directory.glob(file_pattern):
+        with profile_operation(operation_name, {"file": file_path.name}) as result:
+            parse_func(str(file_path))
+        file_count += 1
+        total_time += result.execution_time
+
+    if file_count > 0:
+        return ProfileResult(
+            operation_name=f"{operation_name} (total: {file_count})",
+            execution_time=total_time,
+            peak_memory=0,
+            context={"avg_per_file": total_time / file_count},
+        )
+
+    return None
+
+
+def generate_cookbook_performance_report(
+    cookbook_path: str,
+) -> PerformanceReport:
+    """
+    Generate comprehensive performance report for a cookbook.
+
+    Profiles all parsing operations for a cookbook and provides
+    optimization recommendations based on the results.
+
+    Args:
+        cookbook_path: Path to the cookbook to profile.
+
+    Returns:
+        PerformanceReport with detailed profiling information.
+
+    Raises:
+        SousChefError: If cookbook path is invalid or profiling fails.
+
+    Example:
+        >>> report = generate_cookbook_performance_report("/path/to/cookbook")
+        >>> print(report)
+
+    """
+    from souschef.parsers.attributes import parse_attributes
+    from souschef.parsers.metadata import (
+        list_cookbook_structure,
+        read_cookbook_metadata,
+    )
+    from souschef.parsers.recipe import parse_recipe
+    from souschef.parsers.resource import parse_custom_resource
+    from souschef.parsers.template import parse_template
+
+    path = Path(cookbook_path)
+    if not path.exists():
+        raise SousChefError(
+            f"Cookbook path not found: {cookbook_path}",
+            suggestion="Verify the cookbook path exists and is accessible",
+        )
+
+    report = PerformanceReport(cookbook_name=path.name, total_time=0.0, total_memory=0)
+
+    overall_start = time.perf_counter()
+    overall_memory_start = 0
+
+    tracemalloc.start()
+    overall_memory_start = tracemalloc.get_traced_memory()[1]
+
+    try:
+        # Profile cookbook structure analysis
+        with profile_operation(
+            "list_cookbook_structure", {"path": cookbook_path}
+        ) as result:
+            list_cookbook_structure(cookbook_path)
+        report.add_result(result)
+
+        # Profile metadata parsing
+        metadata_path = path / "metadata.rb"
+        if metadata_path.exists():
+            with profile_operation(
+                "read_cookbook_metadata", {"file": str(metadata_path)}
+            ) as result:
+                read_cookbook_metadata(str(metadata_path))
+            report.add_result(result)
+
+        # Profile recipe parsing
+        recipe_result = _profile_directory_files(
+            path / "recipes", parse_recipe, "parse_recipes"
+        )
+        if recipe_result:
+            report.add_result(recipe_result)
+
+        # Profile attribute parsing
+        attr_result = _profile_directory_files(
+            path / "attributes", parse_attributes, "parse_attributes"
+        )
+        if attr_result:
+            report.add_result(attr_result)
+
+        # Profile custom resource parsing
+        resource_result = _profile_directory_files(
+            path / "resources", parse_custom_resource, "parse_custom_resources"
+        )
+        if resource_result:
+            report.add_result(resource_result)
+
+        # Profile template parsing
+        templates_dir = path / "templates"
+        if templates_dir.exists():
+            template_count = 0
+            template_total_time = 0.0
+
+            for template_file in templates_dir.rglob("*.erb"):
+                with profile_operation(
+                    "parse_template", {"file": template_file.name}
+                ) as result:
+                    parse_template(str(template_file))
+                template_count += 1
+                template_total_time += result.execution_time
+
+            if template_count > 0:
+                report.add_result(
+                    ProfileResult(
+                        operation_name=f"parse_templates (total: {template_count})",
+                        execution_time=template_total_time,
+                        peak_memory=0,
+                        context={
+                            "avg_per_template": template_total_time / template_count
+                        },
+                    )
+                )
+
+    finally:
+        _, overall_peak = tracemalloc.get_traced_memory()
+        tracemalloc.stop()
+
+    # Total memory is the peak during profiling, ensuring non-negative
+    report.total_time = time.perf_counter() - overall_start
+    report.total_memory = max(0, overall_peak - overall_memory_start)
+
+    # Generate recommendations
+    _add_performance_recommendations(report)
+
+    return report
+
+
+def _add_performance_recommendations(report: PerformanceReport) -> None:
+    """
+    Add performance recommendations based on profiling results.
+
+    Args:
+        report: Performance report to add recommendations to.
+
+    """
+    # Check for slow operations
+    slow_threshold = 1.0  # 1 second
+    for result in report.operation_results:
+        if result.execution_time > slow_threshold:
+            report.add_recommendation(
+                f"Operation '{result.operation_name}' took "
+                f"{result.execution_time:.2f}s. "
+                f"Consider optimizing or parallelizing this operation."
+            )
+
+    # Check for high memory usage
+    memory_threshold = 100 * 1024 * 1024  # 100MB
+    if report.total_memory > memory_threshold:
+        report.add_recommendation(
+            f"Peak memory usage is {report.total_memory / 1024 / 1024:.0f}MB. "
+            "Consider processing files in batches or implementing streaming "
+            "for large cookbooks."
+        )
+
+    # Check for many files
+    for result in report.operation_results:
+        if "total:" in result.operation_name:
+            # Extract count from operation name like "parse_recipes (total: 50)"
+            if "parse_recipes" in result.operation_name and result.execution_time > 5.0:
+                report.add_recommendation(
+                    "Large number of recipes detected. Consider using parallel "
+                    "processing or caching intermediate results for faster re-runs."
+                )
+            elif (
+                "parse_templates" in result.operation_name
+                and result.execution_time > 3.0
+            ):
+                report.add_recommendation(
+                    "Template parsing is time-consuming. Consider caching parsed "
+                    "templates or using a template compilation cache."
+                )
+
+    # Always provide general recommendations
+    report.add_recommendation(
+        "Use the --cache option (if available) to cache parsing results for "
+        "faster subsequent runs."
+    )
+    report.add_recommendation(
+        "For very large cookbooks, consider splitting into smaller, focused cookbooks."
+    )
+
+
+def compare_performance(
+    before: ProfileResult | PerformanceReport,
+    after: ProfileResult | PerformanceReport,
+) -> str:
+    """
+    Compare performance before and after optimization.
+
+    Args:
+        before: Performance metrics before optimization.
+        after: Performance metrics after optimization.
+
+    Returns:
+        Formatted comparison report.
+
+    Example:
+        >>> comparison = compare_performance(before_profile, after_profile)
+        >>> print(comparison)
+
+    """
+    time_before = (
+        before.total_time
+        if isinstance(before, PerformanceReport)
+        else before.execution_time
+    )
+    time_after = (
+        after.total_time
+        if isinstance(after, PerformanceReport)
+        else after.execution_time
+    )
+
+    memory_before = (
+        before.total_memory
+        if isinstance(before, PerformanceReport)
+        else before.peak_memory
+    )
+    memory_after = (
+        after.total_memory
+        if isinstance(after, PerformanceReport)
+        else after.peak_memory
+    )
+
+    time_improvement = (
+        ((time_before - time_after) / time_before * 100) if time_before > 0 else 0
+    )
+    memory_improvement = (
+        ((memory_before - memory_after) / memory_before * 100)
+        if memory_before > 0
+        else 0
+    )
+
+    return f"""
+Performance Comparison
+{"=" * 80}
+Execution Time:
+  Before: {time_before:.3f}s
+  After:  {time_after:.3f}s
+  Change: {time_improvement:+.1f}% {"(faster)" if time_improvement > 0 else "(slower)"}
+
+Memory Usage:
+  Before: {memory_before / 1024 / 1024:.2f}MB
+  After:  {memory_after / 1024 / 1024:.2f}MB
+  Change: {memory_improvement:+.1f}% {"(less)" if memory_improvement > 0 else "(more)"}
+{"=" * 80}
+"""