crackerjack 0.21.5__tar.gz → 0.21.7__tar.gz

{crackerjack-0.21.5 → crackerjack-0.21.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: crackerjack
-Version: 0.21.5
+Version: 0.21.7
 Summary: Crackerjack: code quality toolkit
 Keywords: bandit,black,creosote,mypy,pyright,pytest,refurb,ruff
 Author-Email: lesleslie <les@wedgwoodwebworks.com>
@@ -102,7 +102,7 @@ If you're new to Crackerjack, follow these steps:
 
     Or use the interactive Rich UI:
     ```
-    python -m crackerjack --rich-ui
+    python -m crackerjack --interactive
     ```
 
 ---
@@ -269,7 +269,7 @@ python -m crackerjack -t --benchmark-regression --benchmark-regression-threshold
 
     Or with the interactive Rich UI:
     ```
-    python -m crackerjack --rich-ui
+    python -m crackerjack --interactive
     ```
 
 ## Usage
@@ -361,7 +361,7 @@ runner.process(MyOptions())
 -   `--benchmark-regression`: Fail tests if benchmarks regress beyond threshold.
 -   `--benchmark-regression-threshold`: Set threshold percentage for benchmark regression (default 5.0%).
 -   `-a`, `--all`: Run with `-x -t -p <micro|minor|major> -c` development options.
--   `--rich-ui`: Enable the interactive Rich UI for a more user-friendly experience with visual progress tracking and interactive prompts.
+-   `--interactive`: Enable the interactive Rich UI for a more user-friendly experience with visual progress tracking and interactive prompts.
 -   `--ai-agent`: Enable AI agent mode with structured output (see [AI Agent Integration](#ai-agent-integration)).
 -   `--help`: Display help.
 
@@ -461,7 +461,7 @@ runner.process(MyOptions())
 
 - **Rich Interactive Mode** - Run with the interactive Rich UI:
   ```bash
-  python -m crackerjack --rich-ui
+  python -m crackerjack --interactive
   ```
 
 - **AI Integration** - Run with structured output for AI tools:
@@ -500,10 +500,10 @@ Crackerjack now offers an enhanced interactive experience through its Rich UI:
 - **Error Visualization:** Errors are presented in a structured, easy-to-understand format with recovery suggestions
 - **File Selection:** Interactive file browser for operations that require selecting files
 
-To use the Rich UI, run Crackerjack with the `--rich-ui` flag:
+To use the Rich UI, run Crackerjack with the `--interactive` flag:
 
 ```bash
-python -m crackerjack --rich-ui
+python -m crackerjack --interactive
 ```
 
 This launches an interactive terminal interface where you can:
@@ -542,7 +542,7 @@ python -m crackerjack -v
 For the most comprehensive error details with visual formatting, combine verbose mode with the Rich UI:
 
 ```bash
-python -m crackerjack --rich-ui -v
+python -m crackerjack --interactive -v
 ```
 
 ## Python 3.13+ Features

{crackerjack-0.21.5 → crackerjack-0.21.7}/README.md

@@ -58,7 +58,7 @@ If you're new to Crackerjack, follow these steps:
 
     Or use the interactive Rich UI:
     ```
-    python -m crackerjack --rich-ui
+    python -m crackerjack --interactive
     ```
 
 ---
@@ -225,7 +225,7 @@ python -m crackerjack -t --benchmark-regression --benchmark-regression-threshold
 
     Or with the interactive Rich UI:
     ```
-    python -m crackerjack --rich-ui
+    python -m crackerjack --interactive
     ```
 
 ## Usage
@@ -317,7 +317,7 @@ runner.process(MyOptions())
 -   `--benchmark-regression`: Fail tests if benchmarks regress beyond threshold.
 -   `--benchmark-regression-threshold`: Set threshold percentage for benchmark regression (default 5.0%).
 -   `-a`, `--all`: Run with `-x -t -p <micro|minor|major> -c` development options.
--   `--rich-ui`: Enable the interactive Rich UI for a more user-friendly experience with visual progress tracking and interactive prompts.
+-   `--interactive`: Enable the interactive Rich UI for a more user-friendly experience with visual progress tracking and interactive prompts.
 -   `--ai-agent`: Enable AI agent mode with structured output (see [AI Agent Integration](#ai-agent-integration)).
 -   `--help`: Display help.
 
@@ -417,7 +417,7 @@ runner.process(MyOptions())
 
 - **Rich Interactive Mode** - Run with the interactive Rich UI:
   ```bash
-  python -m crackerjack --rich-ui
+  python -m crackerjack --interactive
   ```
 
 - **AI Integration** - Run with structured output for AI tools:
@@ -456,10 +456,10 @@ Crackerjack now offers an enhanced interactive experience through its Rich UI:
 - **Error Visualization:** Errors are presented in a structured, easy-to-understand format with recovery suggestions
 - **File Selection:** Interactive file browser for operations that require selecting files
 
-To use the Rich UI, run Crackerjack with the `--rich-ui` flag:
+To use the Rich UI, run Crackerjack with the `--interactive` flag:
 
 ```bash
-python -m crackerjack --rich-ui
+python -m crackerjack --interactive
 ```
 
 This launches an interactive terminal interface where you can:
@@ -498,7 +498,7 @@ python -m crackerjack -v
 For the most comprehensive error details with visual formatting, combine verbose mode with the Rich UI:
 
 ```bash
-python -m crackerjack --rich-ui -v
+python -m crackerjack --interactive -v
 ```
 
 ## Python 3.13+ Features

{crackerjack-0.21.5 → crackerjack-0.21.7}/crackerjack/.gitignore

@@ -13,5 +13,8 @@
 /scratch/
 /.zencoder/
 /.benchmarks/
-
 **/.claude/settings.local.json
+/complexipy.json
+/coverage.json
+/test-results.xml
+/.coverage

{crackerjack-0.21.5 → crackerjack-0.21.7}/crackerjack/__init__.py

@@ -1,5 +1,4 @@
 import typing as t
-
 from .crackerjack import Crackerjack, create_crackerjack_runner
 from .errors import (
     CleaningError,

{crackerjack-0.21.5 → crackerjack-0.21.7}/crackerjack/__main__.py

@@ -1,5 +1,4 @@
 from enum import Enum
-
 import typer
 from pydantic import BaseModel, field_validator
 from rich.console import Console

{crackerjack-0.21.5 → crackerjack-0.21.7}/crackerjack/crackerjack.py

@@ -6,7 +6,6 @@ from pathlib import Path
 from subprocess import CompletedProcess
 from subprocess import run as execute
 from tomllib import loads
-
 from pydantic import BaseModel
 from rich.console import Console
 from tomli_w import dumps
@@ -86,28 +85,93 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
         except Exception as e:
             print(f"Error cleaning {file_path}: {e}")
 
+    def _initialize_docstring_state(self) -> dict[str, t.Any]:
+        return {
+            "in_docstring": False,
+            "delimiter": None,
+            "waiting": False,
+            "function_indent": 0,
+            "removed_docstring": False,
+            "in_multiline_def": False,
+        }
+
+    def _handle_function_definition(
+        self, line: str, stripped: str, state: dict[str, t.Any]
+    ) -> bool:
+        if self._is_function_or_class_definition(stripped):
+            state["waiting"] = True
+            state["function_indent"] = len(line) - len(line.lstrip())
+            state["removed_docstring"] = False
+            state["in_multiline_def"] = not stripped.endswith(":")
+            return True
+        return False
+
+    def _handle_multiline_definition(
+        self, line: str, stripped: str, state: dict[str, t.Any]
+    ) -> bool:
+        if state["in_multiline_def"]:
+            if stripped.endswith(":"):
+                state["in_multiline_def"] = False
+            return True
+        return False
+
+    def _handle_waiting_docstring(
+        self, lines: list[str], i: int, stripped: str, state: dict[str, t.Any]
+    ) -> tuple[bool, str | None]:
+        if state["waiting"] and stripped:
+            if self._handle_docstring_start(stripped, state):
+                pass_line = None
+                if not state["in_docstring"]:
+                    function_indent: int = state["function_indent"]
+                    if self._needs_pass_statement(lines, i + 1, function_indent):
+                        pass_line = " " * (function_indent + 4) + "pass"
+                state["removed_docstring"] = True
+                return True, pass_line
+            else:
+                state["waiting"] = False
+        return False, None
+
+    def _handle_docstring_content(
+        self, lines: list[str], i: int, stripped: str, state: dict[str, t.Any]
+    ) -> tuple[bool, str | None]:
+        if state["in_docstring"]:
+            if self._handle_docstring_end(stripped, state):
+                pass_line = None
+                function_indent: int = state["function_indent"]
+                if self._needs_pass_statement(lines, i + 1, function_indent):
+                    pass_line = " " * (function_indent + 4) + "pass"
+                state["removed_docstring"] = False
+                return True, pass_line
+            else:
+                return True, None
+        return False, None
+
+    def _process_line(
+        self, lines: list[str], i: int, line: str, state: dict[str, t.Any]
+    ) -> tuple[bool, str | None]:
+        stripped = line.strip()
+        if self._handle_function_definition(line, stripped, state):
+            return True, line
+        if self._handle_multiline_definition(line, stripped, state):
+            return True, line
+        handled, pass_line = self._handle_waiting_docstring(lines, i, stripped, state)
+        if handled:
+            return True, pass_line
+        handled, pass_line = self._handle_docstring_content(lines, i, stripped, state)
+        if handled:
+            return True, pass_line
+        if state["removed_docstring"] and stripped:
+            state["removed_docstring"] = False
+        return False, line
+
     def remove_docstrings(self, code: str) -> str:
         lines = code.split("\n")
         cleaned_lines = []
-        docstring_state = {"in_docstring": False, "delimiter": None, "waiting": False}
-        for line in lines:
-            stripped = line.strip()
-            if self._is_function_or_class_definition(stripped):
-                docstring_state["waiting"] = True
-                cleaned_lines.append(line)
-                continue
-            if docstring_state["waiting"] and stripped:
-                if self._handle_docstring_start(stripped, docstring_state):
-                    continue
-                else:
-                    docstring_state["waiting"] = False
-            if docstring_state["in_docstring"]:
-                if self._handle_docstring_end(stripped, docstring_state):
-                    continue
-                else:
-                    continue
-            cleaned_lines.append(line)
-
+        docstring_state = self._initialize_docstring_state()
+        for i, line in enumerate(lines):
+            _, result_line = self._process_line(lines, i, line, docstring_state)
+            if result_line:
+                cleaned_lines.append(result_line)
         return "\n".join(cleaned_lines)
 
     def _is_function_or_class_definition(self, stripped_line: str) -> bool:
@@ -137,6 +201,21 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
             state["in_docstring"] = False
             state["delimiter"] = None
             return True
+        return False
+
+    def _needs_pass_statement(
+        self, lines: list[str], start_index: int, function_indent: int
+    ) -> bool:
+        for i in range(start_index, len(lines)):
+            line = lines[i]
+            stripped = line.strip()
+            if not stripped:
+                continue
+            line_indent = len(line) - len(line.lstrip())
+            if line_indent <= function_indent:
+                return True
+            if line_indent > function_indent:
+                return False
         return True
 
     def remove_line_comments(self, code: str) -> str:
@@ -149,7 +228,6 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
             cleaned_line = self._process_line_for_comments(line)
             if cleaned_line or not line.strip():
                 cleaned_lines.append(cleaned_line or line)
-
         return "\n".join(cleaned_lines)
 
     def _process_line_for_comments(self, line: str) -> str:
@@ -162,7 +240,6 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
                 break
             else:
                 result.append(char)
-
         return "".join(result).rstrip()
 
     def _handle_string_character(
@@ -173,18 +250,14 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
         string_state: dict[str, t.Any],
         result: list[str],
     ) -> bool:
-        """Handle string quote characters. Returns True if character was handled."""
         if char not in ("'", '"'):
             return False
-
         if index > 0 and line[index - 1] == "\\":
             return False
-
         if string_state["in_string"] is None:
             string_state["in_string"] = char
         elif string_state["in_string"] == char:
             string_state["in_string"] = None
-
         result.append(char)
         return True
 
@@ -196,14 +269,11 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
         string_state: dict[str, t.Any],
         result: list[str],
     ) -> bool:
-        """Handle comment character. Returns True if comment was found."""
         if char != "#" or string_state["in_string"] is not None:
             return False
-
         comment = line[index:].strip()
         if self._is_special_comment_line(comment):
             result.append(line[index:])
-
         return True
 
     def _is_special_comment_line(self, comment: str) -> bool:
@@ -227,13 +297,11 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
                 ):
                     continue
             cleaned_lines.append(line)
-
         return "\n".join(self._remove_trailing_empty_lines(cleaned_lines))
 
     def _update_function_state(
         self, line: str, stripped_line: str, function_tracker: dict[str, t.Any]
     ) -> None:
-        """Update function tracking state based on current line."""
         if stripped_line.startswith(("def ", "async def ")):
             function_tracker["in_function"] = True
             function_tracker["function_indent"] = len(line) - len(stripped_line)
@@ -244,7 +312,6 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
     def _is_function_end(
         self, line: str, stripped_line: str, function_tracker: dict[str, t.Any]
     ) -> bool:
-        """Check if current line marks the end of a function."""
         return (
             function_tracker["in_function"]
             and bool(line)
@@ -259,13 +326,10 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
         cleaned_lines: list[str],
         function_tracker: dict[str, t.Any],
     ) -> bool:
-        """Determine if an empty line should be skipped."""
         if line_idx > 0 and cleaned_lines and (not cleaned_lines[-1]):
             return True
-
         if function_tracker["in_function"]:
             return self._should_skip_function_empty_line(line_idx, lines)
-
         return False
 
     def _should_skip_function_empty_line(self, line_idx: int, lines: list[str]) -> bool:
@@ -280,7 +344,6 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
             return True
         if next_line in ("pass", "break", "continue", "raise"):
             return True
-
         return self._is_special_comment(next_line)
 
     def _is_special_comment(self, line: str) -> bool:
@@ -420,15 +483,12 @@ class ConfigManager(BaseModel, arbitrary_types_allowed=True):
         for tool, settings in our_toml_config.get("tool", {}).items():
             if tool not in pkg_toml_config["tool"]:
                 pkg_toml_config["tool"][tool] = {}
-
             pkg_tool_config = pkg_toml_config["tool"][tool]
-
             self._merge_tool_config(settings, pkg_tool_config, tool)
 
     def _merge_tool_config(
         self, our_config: dict[str, t.Any], pkg_config: dict[str, t.Any], tool: str
     ) -> None:
-        """Recursively merge tool configuration, preserving existing project settings."""
         for setting, value in our_config.items():
             if isinstance(value, dict):
                 self._merge_nested_config(setting, value, pkg_config)
@@ -438,21 +498,17 @@ class ConfigManager(BaseModel, arbitrary_types_allowed=True):
     def _merge_nested_config(
         self, setting: str, value: dict[str, t.Any], pkg_config: dict[str, t.Any]
     ) -> None:
-        """Handle nested configuration merging."""
         if setting not in pkg_config:
             pkg_config[setting] = {}
         elif not isinstance(pkg_config[setting], dict):
             pkg_config[setting] = {}
-
         self._merge_tool_config(value, pkg_config[setting], "")
-
         for k, v in value.items():
             self._merge_nested_value(k, v, pkg_config[setting])
 
     def _merge_nested_value(
         self, key: str, value: t.Any, nested_config: dict[str, t.Any]
     ) -> None:
-        """Merge individual nested values."""
         if isinstance(value, str | list) and "crackerjack" in str(value):
             nested_config[key] = self.swap_package_name(value)
         elif self._is_mergeable_list(key, value):
@@ -467,7 +523,6 @@ class ConfigManager(BaseModel, arbitrary_types_allowed=True):
     def _merge_direct_config(
         self, setting: str, value: t.Any, pkg_config: dict[str, t.Any]
     ) -> None:
-        """Handle direct configuration merging."""
         if isinstance(value, str | list) and "crackerjack" in str(value):
             pkg_config[setting] = self.swap_package_name(value)
         elif self._is_mergeable_list(setting, value):
@@ -633,7 +688,7 @@ class Crackerjack(BaseModel, arbitrary_types_allowed=True):
             )
             if result.returncode == 0:
                 self.console.print("PDM installed: ✅\n")
-                self.execute_command(["pdm", "sync"])
+                self.execute_command(["pdm", "lock"])
                 self.console.print("Lock file updated: ✅\n")
             else:
                 self.console.print(
@@ -730,7 +785,6 @@ class Crackerjack(BaseModel, arbitrary_types_allowed=True):
             self._add_benchmark_flags(test, options)
         else:
             self._add_worker_flags(test, options, project_size)
-
         return test
 
     def _detect_project_size(self) -> str:

{crackerjack-0.21.5 → crackerjack-0.21.7}/crackerjack/errors.py

@@ -2,7 +2,6 @@ import sys
 import typing as t
 from enum import Enum
 from pathlib import Path
-
 from rich.console import Console
 from rich.panel import Panel
 

{crackerjack-0.21.5 → crackerjack-0.21.7}/crackerjack/interactive.py

@@ -2,7 +2,6 @@ import time
 import typing as t
 from enum import Enum, auto
 from pathlib import Path
-
 from rich.box import ROUNDED
 from rich.console import Console
 from rich.layout import Layout
@@ -19,7 +18,6 @@ from rich.prompt import Confirm, Prompt
 from rich.table import Table
 from rich.text import Text
 from rich.tree import Tree
-
 from .errors import CrackerjackError, ErrorCode, handle_error
 
 
@@ -289,7 +287,6 @@ class InteractiveCLI:
         )
         total_tasks = len(self.workflow.tasks)
         progress_task = progress.add_task("Running workflow", total=total_tasks)
-
         return {
             "progress": progress,
             "progress_task": progress_task,
@@ -299,14 +296,11 @@ class InteractiveCLI:
     def _execute_workflow_loop(
         self, layout: Layout, progress_tracker: dict[str, t.Any], live: Live
     ) -> None:
-        """Execute the main workflow loop."""
         while not self.workflow.all_tasks_completed():
             layout["tasks"].update(self.show_task_table())
             next_task = self.workflow.get_next_task()
-
             if not next_task:
                 break
-
             if self._should_execute_task(layout, next_task, live):
                 self._execute_task(layout, next_task, progress_tracker)
             else:
@@ -322,20 +316,16 @@ class InteractiveCLI:
     def _execute_task(
         self, layout: Layout, task: Task, progress_tracker: dict[str, t.Any]
     ) -> None:
-        """Execute a single task and update progress."""
         task.start()
         layout["details"].update(self.show_task_status(task))
         time.sleep(1)
-
         success = self._simulate_task_execution()
-
         if success:
             task.complete()
             progress_tracker["completed_tasks"] += 1
         else:
             error = self._create_task_error(task.name)
             task.fail(error)
-
         progress_tracker["progress"].update(
             progress_tracker["progress_task"],
             completed=progress_tracker["completed_tasks"],

{crackerjack-0.21.5 → crackerjack-0.21.7}/crackerjack/pyproject.toml

@@ -4,7 +4,7 @@ requires = [ "pdm-backend" ]
 
 [project]
 name = "crackerjack"
-version = "0.21.4"
+version = "0.21.6"
 description = "Crackerjack: code quality toolkit"
 readme = "README.md"
 keywords = [

{crackerjack-0.21.5 → crackerjack-0.21.7}/pyproject.toml

@@ -6,7 +6,7 @@ requires = [
 
 [project]
 name = "crackerjack"
-version = "0.21.5"
+version = "0.21.7"
 description = "Crackerjack: code quality toolkit"
 readme = "README.md"
 keywords = [

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/conftest.py

@@ -2,7 +2,6 @@ import os
 import time
 import typing as t
 from pathlib import Path
-
 import pytest
 from pytest import Config, Item, Parser
 

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/test_crackerjack.py

@@ -6,7 +6,6 @@ from dataclasses import dataclass
 from enum import Enum
 from pathlib import Path
 from unittest.mock import MagicMock, patch
-
 import pytest
 from rich.console import Console
 from crackerjack.crackerjack import (
@@ -317,12 +316,10 @@ class TestCrackerjackProcess:
     ) -> None:
         options = options_factory(bump="minor", no_config_updates=True)
         cj = Crackerjack(dry_run=True)
-
         with patch("rich.prompt.Confirm.ask", return_value=True) as mock_confirm:
             with patch.object(Crackerjack, "execute_command") as mock_exec:
                 mock_exec.return_value = MagicMock(returncode=0)
                 cj._bump_version(options)
-
                 mock_confirm.assert_called_once_with(
                     "Are you sure you want to bump the minor version?", default=False
                 )
@@ -339,12 +336,10 @@ class TestCrackerjackProcess:
     ) -> None:
         options = options_factory(bump="minor", no_config_updates=True)
         cj = Crackerjack(dry_run=True)
-
         with patch("rich.prompt.Confirm.ask", return_value=False) as mock_confirm:
             with patch.object(Crackerjack, "execute_command") as mock_exec:
                 mock_exec.return_value = MagicMock(returncode=0)
                 cj._bump_version(options)
-
                 mock_confirm.assert_called_once_with(
                     "Are you sure you want to bump the minor version?", default=False
                 )
@@ -361,12 +356,10 @@ class TestCrackerjackProcess:
     ) -> None:
         options = options_factory(bump="major", no_config_updates=True)
         cj = Crackerjack(dry_run=True)
-
         with patch("rich.prompt.Confirm.ask", return_value=True) as mock_confirm:
             with patch.object(Crackerjack, "execute_command") as mock_exec:
                 mock_exec.return_value = MagicMock(returncode=0)
                 cj._bump_version(options)
-
                 mock_confirm.assert_called_once_with(
                     "Are you sure you want to bump the major version?", default=False
                 )
@@ -383,12 +376,10 @@ class TestCrackerjackProcess:
     ) -> None:
         options = options_factory(bump="micro", no_config_updates=True)
         cj = Crackerjack(dry_run=True)
-
         with patch("rich.prompt.Confirm.ask") as mock_confirm:
             with patch.object(Crackerjack, "execute_command") as mock_exec:
                 mock_exec.return_value = MagicMock(returncode=0)
                 cj._bump_version(options)
-
                 mock_confirm.assert_not_called()
                 mock_exec.assert_called_once_with(["pdm", "bump", "micro"])
 
@@ -405,7 +396,6 @@ class TestCrackerjackProcess:
         pytest_command = (_cj := Crackerjack(dry_run=True))._prepare_pytest_command(
             options
         )
-
         assert "--junitxml=test-results.xml" in pytest_command
         assert "--cov-report=json:coverage.json" in pytest_command
         assert "--quiet" in pytest_command
@@ -427,7 +417,6 @@ class TestCrackerjackProcess:
         pytest_command = (_cj := Crackerjack(dry_run=True))._prepare_pytest_command(
             options
         )
-
         assert "--junitxml=test-results.xml" in pytest_command
         assert "--cov-report=json:coverage.json" in pytest_command
         assert "--benchmark-json=benchmark.json" in pytest_command
@@ -447,12 +436,10 @@ class TestCrackerjackProcess:
         pytest_command = (_cj := Crackerjack(dry_run=True))._prepare_pytest_command(
             options
         )
-
         assert "--junitxml=test-results.xml" not in pytest_command
         assert "--cov-report=json:coverage.json" not in pytest_command
         assert "--benchmark-json=benchmark.json" not in pytest_command
         assert "--quiet" not in pytest_command
-
         assert "--capture=fd" in pytest_command
         assert "--disable-warnings" in pytest_command
         assert "--durations=0" in pytest_command
@@ -943,16 +930,11 @@ class TestCrackerjackProcess:
         options_factory: t.Callable[..., OptionsForTesting],
     ) -> None:
         mock_project = MagicMock()
-
         options = options_factory(ai_agent=True)
         mock_project.options = options
-
         mock_project.console = MagicMock()
-
         mock_project.execute_command.return_value = MagicMock(returncode=0)
-
         ProjectManager.run_pre_commit(mock_project)
-
         mock_project.execute_command.assert_called_with(
             ["pre-commit", "run", "--all-files", "-c", ".pre-commit-config-ai.yaml"]
         )
@@ -964,24 +946,17 @@ class TestCrackerjackProcess:
         options_factory: t.Callable[..., OptionsForTesting],
     ) -> None:
         mock_project = MagicMock(spec=ProjectManager)
-
         options = options_factory(ai_agent=True)
         mock_project.options = options
-
         mock_project.console = MagicMock()
-
         mock_project.config_manager = MagicMock()
-
         with patch.object(mock_project, "execute_command") as mock_execute:
             mock_execute.return_value = MagicMock(
                 returncode=0,
                 stdout="package1\npackage2\n",
             )
-
             original_method = ProjectManager.update_pkg_configs
-
             original_method(mock_project)
-
             mock_execute.assert_any_call(
                 ["pre-commit", "install", "-c", ".pre-commit-config-ai.yaml"]
             )
@@ -1091,6 +1066,7 @@ class TestCrackerjackProcess:
             mock_clean_file.assert_called_once_with(py_file)
 
     def test_code_cleaner_remove_docstrings(self) -> None:
+        import ast
         from rich.console import Console
         from crackerjack.crackerjack import CodeCleaner
 
@@ -1108,6 +1084,43 @@ class TestCrackerjackProcess:
         assert "This is a multi-line docstring." not in cleaned_code, (
             f"Got: {cleaned_code!r}"
         )
+        try:
+            ast.parse(cleaned_code)
+        except SyntaxError as e:
+            raise AssertionError(
+                f"Cleaned code is not valid Python syntax: {e}\nCode: {cleaned_code!r}"
+            )
+
+    def test_code_cleaner_remove_docstrings_empty_functions(self) -> None:
+        import ast
+        from rich.console import Console
+        from crackerjack.crackerjack import CodeCleaner
+
+        code_cleaner = CodeCleaner(console=Console())
+        test_code = """
+def empty_function():
+    pass
+class TestClass:
+    def method_with_docstring_only(self):
+        pass
+    def method_with_code(self):
+        return True
+"""
+        cleaned_code = code_cleaner.remove_docstrings(test_code)
+        print(f"Cleaned code: {cleaned_code!r}")
+        assert '"""This function has only a docstring."""' not in cleaned_code
+        assert '"""Class docstring."""' not in cleaned_code
+        assert '"""Method with only docstring."""' not in cleaned_code
+        assert '"""This method has code after docstring."""' not in cleaned_code
+        assert "def empty_function():\n    pass" in cleaned_code
+        assert "def method_with_docstring_only(self):\n        pass" in cleaned_code
+        assert "def method_with_code(self):\n        return True" in cleaned_code
+        try:
+            ast.parse(cleaned_code)
+        except SyntaxError as e:
+            raise AssertionError(
+                f"Cleaned code is not valid Python syntax: {e}\nCode: {cleaned_code!r}"
+            )
 
     def test_code_cleaner_remove_line_comments(self) -> None:
         from pathlib import Path

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/test_crackerjack_runner.py

@@ -2,7 +2,6 @@ import tempfile
 import typing as t
 from pathlib import Path
 from unittest.mock import MagicMock, patch
-
 import pytest
 from rich.console import Console
 from crackerjack import create_crackerjack_runner

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/test_errors.py

@@ -1,6 +1,5 @@
 import typing as t
 from unittest.mock import MagicMock, patch
-
 import pytest
 from rich.console import Console
 from crackerjack.errors import (

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/test_interactive.py

@@ -2,7 +2,6 @@ import io
 import time
 from pathlib import Path
 from unittest.mock import MagicMock, patch
-
 import pytest
 from rich.console import Console
 from rich.panel import Panel

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/test_interactive_run.py

@@ -1,5 +1,4 @@
 from unittest.mock import MagicMock, patch
-
 import pytest
 from rich.console import Console
 from crackerjack.errors import ErrorCode, ExecutionError

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/test_main.py

@@ -1,6 +1,5 @@
 import typing as t
 from unittest.mock import MagicMock, patch
-
 import pytest
 from typer.testing import CliRunner
 from crackerjack.__main__ import BumpOption, Options, app
@@ -59,7 +58,6 @@ def test_interactive_option(
         assert result.exit_code == 0
         mock_interactive.assert_called_once()
         mock_crackerjack_process.process.assert_not_called()
-
         mock_interactive.reset_mock()
         result = runner.invoke(app, ["--interactive"])
         assert result.exit_code == 0
@@ -203,7 +201,6 @@ def test_multiple_options(
         assert result.exit_code == 0
         mock_interactive.assert_called_once()
         mock_crackerjack_process.process.assert_not_called()
-
     result = runner.invoke(app, ["-c", "-d", "-t", "-x"])
     assert result.exit_code == 0
     mock_crackerjack_process.process.assert_called_once()

crackerjack-0.21.7/tests/test_multiline_functions.py

@@ -0,0 +1,163 @@
+"""Test cases for multiline function definitions in docstring removal."""
+
+import ast
+from rich.console import Console
+from crackerjack.crackerjack import CodeCleaner
+
+
+class TestMultilineFunctions:
+    def setup_method(self) -> None:
+        self.console = Console()
+        self.cleaner = CodeCleaner(console=self.console)
+
+    def test_multiline_async_function_with_docstring(self) -> None:
+        code = """class BackgroundTaskManager:
+    async def __aexit__(
+        self,
+        exc_type: type[BaseException] | None,
+        exc_val: BaseException | None,
+        exc_tb: t.Any | None,
+    ) -> None:
+        del exc_type, exc_val, exc_tb
+        self.logger.debug("Background task manager shutting down")
+        await wait_for_background_tasks(self.cleanup_timeout)
+"""
+        result = self.cleaner.remove_docstrings(code)
+        ast.parse(result)
+        assert '"""Clean up background tasks on exit."""' not in result
+        assert "del exc_type, exc_val, exc_tb" in result
+        assert "self.logger.debug" in result
+
+    def test_multiline_function_with_docstring(self) -> None:
+        code = '''def complex_function(
+        param1: str,
+        param2: int,
+        param3: dict[str, Any],
+    ) -> bool:
+        """This is a complex function with multiple parameters."""
+        return True
+'''
+        result = self.cleaner.remove_docstrings(code)
+        ast.parse(result)
+        assert (
+            '"""This is a complex function with multiple parameters."""' not in result
+        )
+        assert "return True" in result
+
+    def test_multiline_function_no_docstring(self) -> None:
+        code = """def complex_function(
+        param1: str,
+        param2: int,
+        param3: dict[str, Any],
+    ) -> bool:
+        return True
+"""
+        result = self.cleaner.remove_docstrings(code)
+        ast.parse(result)
+        assert "return True" in result
+        assert "def complex_function(" in result
+
+    def test_nested_multiline_functions(self) -> None:
+        code = """class TestClass:
+    def outer_method(
+        self,
+        param: str,
+    ) -> None:
+        def inner_function(
+            x: int,
+            y: int,
+        ) -> int:
+            return x + y
+        return inner_function(1, 2)
+"""
+        result = self.cleaner.remove_docstrings(code)
+        ast.parse(result)
+        assert '"""Outer method docstring."""' not in result
+        assert '"""Inner function docstring."""' not in result
+        assert "return x + y" in result
+        assert "return inner_function(1, 2)" in result
+
+    def test_multiline_function_with_decorators(self) -> None:
+        code = """class TestClass:
+    @property
+    @some_decorator(
+        param1="value1",
+        param2="value2"
+    )
+    def complex_property(
+        self,
+    ) -> str:
+        return "test"
+"""
+        result = self.cleaner.remove_docstrings(code)
+        ast.parse(result)
+        assert (
+            '"""Property with complex decorator and multiline signature."""'
+            not in result
+        )
+        assert "@property" in result
+        assert "@some_decorator" in result
+        assert 'return "test"' in result
+
+    def test_class_with_multiline_init(self) -> None:
+        code = '''class ComplexClass:
+    """Class docstring."""
+    def __init__(
+        self,
+        param1: str,
+        param2: int = 42,
+        param3: Optional[dict] = None,
+    ) -> None:
+        self.param1 = param1
+        self.param2 = param2
+        self.param3 = param3 or {}
+'''
+        result = self.cleaner.remove_docstrings(code)
+        ast.parse(result)
+        assert '"""Class docstring."""' not in result
+        assert '"""Initialize the complex class."""' not in result
+        assert "self.param1 = param1" in result
+        assert "self.param2 = param2" in result
+        assert "self.param3 = param3 or {}" in result
+
+    def test_multiline_function_empty_body_gets_pass(self) -> None:
+        code = '''def empty_function(
+        param1: str,
+        param2: int,
+    ) -> None:
+        """This function only has a docstring."""
+'''
+        result = self.cleaner.remove_docstrings(code)
+        ast.parse(result)
+        assert '"""This function only has a docstring."""' not in result
+        assert "pass" in result
+
+    def test_complex_real_world_example(self) -> None:
+        code = '''class BackgroundTaskManager:
+    """Manage background tasks."""
+    def __init__(self, cleanup_timeout: float = 30.0) -> None:
+        self.cleanup_timeout = cleanup_timeout
+        self.logger = depends.get(Logger)
+    async def __aenter__(self) -> "BackgroundTaskManager":
+        self.logger.debug("Background task manager started")
+        return self
+    async def __aexit__(
+        self,
+        exc_type: type[BaseException] | None,
+        exc_val: BaseException | None,
+        exc_tb: t.Any | None,
+    ) -> None:
+        del exc_type, exc_val, exc_tb
+        self.logger.debug("Background task manager shutting down")
+        await wait_for_background_tasks(self.cleanup_timeout)
+'''
+        result = self.cleaner.remove_docstrings(code)
+        ast.parse(result)
+        assert '"""Manage background tasks."""' not in result
+        assert '"""Initialize the background task manager."""' not in result
+        assert '"""Enter async context."""' not in result
+        assert '"""Clean up background tasks on exit."""' not in result
+        assert "self.cleanup_timeout = cleanup_timeout" in result
+        assert 'self.logger.debug("Background task manager started")' in result
+        assert "del exc_type, exc_val, exc_tb" in result
+        assert "await wait_for_background_tasks" in result

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/test_py313_advanced.py

@@ -2,7 +2,6 @@ import subprocess
 from pathlib import Path
 from typing import Any
 from unittest import mock
-
 from crackerjack.py313 import (
     CommandResult,
     CommandRunner,

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/test_py313_features.py

@@ -1,5 +1,4 @@
 from pathlib import Path
-
 from crackerjack.py313 import (
     CommandResult,
     HookResult,

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/test_pytest_features.py

@@ -1,5 +1,4 @@
 import time
-
 import pytest
 
 

{crackerjack-0.21.5 → crackerjack-0.21.7}/tests/test_structured_errors.py

@@ -2,7 +2,6 @@ import io
 import typing as t
 from pathlib import Path
 from unittest.mock import MagicMock, patch
-
 import pytest
 from rich.console import Console