crackerjack 0.22.2__tar.gz → 0.22.4__tar.gz

{crackerjack-0.22.2 → crackerjack-0.22.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: crackerjack
-Version: 0.22.2
+Version: 0.22.4
 Summary: Crackerjack: code quality toolkit
 Keywords: bandit,black,creosote,mypy,pyright,pytest,refurb,ruff
 Author-Email: lesleslie <les@wedgwoodwebworks.com>
@@ -24,7 +24,7 @@ Project-URL: repository, https://github.com/lesleslie/crackerjack
 Requires-Python: >=3.13
 Requires-Dist: autotyping>=24.9
 Requires-Dist: keyring>=25.6
-Requires-Dist: pdm>=2.25.3
+Requires-Dist: pdm>=2.25.4
 Requires-Dist: pdm-bump>=0.9.12
 Requires-Dist: pre-commit>=4.2
 Requires-Dist: pydantic>=2.11.7
@@ -34,12 +34,12 @@ Requires-Dist: pytest-benchmark>=5.1
 Requires-Dist: pytest-cov>=6.2.1
 Requires-Dist: pytest-mock>=3.14.1
 Requires-Dist: pytest-timeout>=2.4
-Requires-Dist: pytest-xdist>=3.7
+Requires-Dist: pytest-xdist>=3.8
 Requires-Dist: pyyaml>=6.0.2
 Requires-Dist: rich>=14
 Requires-Dist: tomli-w>=1.2
 Requires-Dist: typer>=0.16
-Requires-Dist: uv>=0.7.15
+Requires-Dist: uv>=0.7.20
 Description-Content-Type: text/markdown
 
 # Crackerjack: Elevate Your Python Development

{crackerjack-0.22.2 → crackerjack-0.22.4}/crackerjack/crackerjack.py

@@ -79,37 +79,52 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
         try:
             code = file_path.read_text(encoding="utf-8")
             original_code = code
+            cleaning_failed = False
             try:
                 code = self.remove_line_comments(code)
             except Exception as e:
                 self.console.print(
-                    f"[yellow]Warning: Failed to remove line comments from {file_path}: {e}[/yellow]"
+                    f"[bright_yellow]⚠️  Warning: Failed to remove line comments from {file_path}: {e}[/bright_yellow]"
                 )
                 code = original_code
+                cleaning_failed = True
             try:
                 code = self.remove_docstrings(code)
             except Exception as e:
                 self.console.print(
-                    f"[yellow]Warning: Failed to remove docstrings from {file_path}: {e}[/yellow]"
+                    f"[bright_yellow]⚠️  Warning: Failed to remove docstrings from {file_path}: {e}[/bright_yellow]"
                 )
                 code = original_code
+                cleaning_failed = True
             try:
                 code = self.remove_extra_whitespace(code)
             except Exception as e:
                 self.console.print(
-                    f"[yellow]Warning: Failed to remove extra whitespace from {file_path}: {e}[/yellow]"
+                    f"[bright_yellow]⚠️  Warning: Failed to remove extra whitespace from {file_path}: {e}[/bright_yellow]"
                 )
                 code = original_code
+                cleaning_failed = True
             try:
                 code = self.reformat_code(code)
             except Exception as e:
                 self.console.print(
-                    f"[yellow]Warning: Failed to reformat {file_path}: {e}[/yellow]"
+                    f"[bright_yellow]⚠️  Warning: Failed to reformat {file_path}: {e}[/bright_yellow]"
                 )
                 code = original_code
+                cleaning_failed = True
             file_path.write_text(code, encoding="utf-8")
-            self.console.print(f"Cleaned: {file_path}")
+            if cleaning_failed:
+                self.console.print(
+                    f"[bright_yellow]⚠️  Partially cleaned: {file_path}[/bright_yellow]"
+                )
+            else:
+                self.console.print(
+                    f"[bright_green]🧹 Cleaned: {file_path}[/bright_green]"
+                )
         except PermissionError as e:
+            self.console.print(
+                f"[bright_red]❌ Failed to clean: {file_path} (Permission denied)[/bright_red]"
+            )
             handle_error(
                 ExecutionError(
                     message=f"Permission denied while cleaning {file_path}",
@@ -121,6 +136,9 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
                 exit_on_error=False,
             )
         except OSError as e:
+            self.console.print(
+                f"[bright_red]❌ Failed to clean: {file_path} (File system error)[/bright_red]"
+            )
             handle_error(
                 ExecutionError(
                     message=f"File system error while cleaning {file_path}",
@@ -132,6 +150,9 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
                 exit_on_error=False,
             )
         except UnicodeDecodeError as e:
+            self.console.print(
+                f"[bright_red]❌ Failed to clean: {file_path} (Encoding error)[/bright_red]"
+            )
             handle_error(
                 ExecutionError(
                     message=f"Encoding error while reading {file_path}",
@@ -143,6 +164,9 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
                 exit_on_error=False,
             )
         except Exception as e:
+            self.console.print(
+                f"[bright_red]❌ Failed to clean: {file_path} (Unexpected error)[/bright_red]"
+            )
             handle_error(
                 ExecutionError(
                     message=f"Unexpected error while cleaning {file_path}",
@@ -591,7 +615,7 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
                     formatted_code = temp_path.read_text()
                 else:
                     self.console.print(
-                        f"[yellow]Ruff formatting failed: {result.stderr}[/yellow]"
+                        f"[bright_yellow]⚠️  Ruff formatting failed: {result.stderr}[/bright_yellow]"
                     )
                     handle_error(
                         ExecutionError(
@@ -601,10 +625,13 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
                             recovery="Check Ruff configuration and formatting rules",
                         ),
                         console=self.console,
+                        exit_on_error=False,
                     )
                     formatted_code = code
             except Exception as e:
-                self.console.print(f"[red]Error running Ruff: {e}[/red]")
+                self.console.print(
+                    f"[bright_red]❌ Error running Ruff: {e}[/bright_red]"
+                )
                 handle_error(
                     ExecutionError(
                         message="Error running Ruff",
@@ -613,6 +640,7 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
                         recovery="Verify Ruff is installed and configured correctly",
                     ),
                     console=self.console,
+                    exit_on_error=False,
                 )
                 formatted_code = code
             finally:
@@ -620,7 +648,9 @@ class CodeCleaner(BaseModel, arbitrary_types_allowed=True):
                     temp_path.unlink()
             return formatted_code
         except Exception as e:
-            self.console.print(f"[red]Error during reformatting: {e}[/red]")
+            self.console.print(
+                f"[bright_red]❌ Error during reformatting: {e}[/bright_red]"
+            )
             handle_error(
                 ExecutionError(
                     message="Error during reformatting",
@@ -792,7 +822,7 @@ class ConfigManager(BaseModel, arbitrary_types_allowed=True):
         self, cmd: list[str], **kwargs: t.Any
     ) -> subprocess.CompletedProcess[str]:
         if self.dry_run:
-            self.console.print(f"[yellow]Would run: {' '.join(cmd)}[/yellow]")
+            self.console.print(f"[dim cyan]🔍 Would run: {' '.join(cmd)}[/dim cyan]")
             return CompletedProcess(cmd, 0, "", "")
         return execute(cmd, **kwargs)
 
@@ -814,7 +844,7 @@ class ProjectManager(BaseModel, arbitrary_types_allowed=True):
             ["pdm", "list", "--freeze"], capture_output=True, text=True
         ).stdout.splitlines()
         if not len([pkg for pkg in installed_pkgs if "pre-commit" in pkg]):
-            self.console.print("Initializing project...")
+            self.console.print("[bright_blue]🚀 Initializing project...[/bright_blue]")
             self.execute_command(["pdm", "self", "add", "keyring"])
             self.execute_command(["pdm", "config", "python.use_uv", "true"])
             self.execute_command(["git", "init"])
@@ -829,7 +859,9 @@ class ProjectManager(BaseModel, arbitrary_types_allowed=True):
         self.config_manager.update_pyproject_configs()
 
     def run_pre_commit(self) -> None:
-        self.console.print("\nRunning pre-commit hooks...\n")
+        self.console.print(
+            "\n[bright_blue]🔍 Running pre-commit hooks...[/bright_blue]\n"
+        )
         cmd = ["pre-commit", "run", "--all-files"]
         if hasattr(self, "options") and getattr(self.options, "ai_agent", False):
             cmd.extend(["-c", ".pre-commit-config-ai.yaml"])
@@ -837,14 +869,16 @@ class ProjectManager(BaseModel, arbitrary_types_allowed=True):
         if check_all.returncode > 0:
             check_all = self.execute_command(cmd)
             if check_all.returncode > 0:
-                self.console.print("\n\nPre-commit failed. Please fix errors.\n")
+                self.console.print(
+                    "\n\n[bright_red]❌ Pre-commit failed. Please fix errors.[/bright_red]\n"
+                )
                 raise SystemExit(1)
 
     def execute_command(
         self, cmd: list[str], **kwargs: t.Any
     ) -> subprocess.CompletedProcess[str]:
         if self.dry_run:
-            self.console.print(f"[yellow]Would run: {' '.join(cmd)}[/yellow]")
+            self.console.print(f"[dim cyan]🔍 Would run: {' '.join(cmd)}[/dim cyan]")
             return CompletedProcess(cmd, 0, "", "")
         return execute(cmd, **kwargs)
 
@@ -887,7 +921,7 @@ class Crackerjack(BaseModel, arbitrary_types_allowed=True):
         self.pkg_name = self.pkg_path.stem.lower().replace("-", "_")
         self.pkg_dir = self.pkg_path / self.pkg_name
         self.pkg_dir.mkdir(exist_ok=True)
-        self.console.print("\nCrackerjacking...\n")
+        self.console.print("\n[bright_magenta]🎯 Crackerjacking...[/bright_magenta]\n")
         self.config_manager.pkg_name = self.pkg_name
         self.project_manager.pkg_name = self.pkg_name
         self.project_manager.pkg_dir = self.pkg_dir
@@ -899,9 +933,11 @@ class Crackerjack(BaseModel, arbitrary_types_allowed=True):
                 ["pdm", "install"], capture_output=True, text=True
             )
             if result.returncode == 0:
-                self.console.print("PDM installed: ✅\n")
+                self.console.print("[bright_green]✅ PDM installed[/bright_green]\n")
                 self.execute_command(["pdm", "lock"])
-                self.console.print("Lock file updated: ✅\n")
+                self.console.print(
+                    "[bright_green]✅ Lock file updated[/bright_green]\n"
+                )
             else:
                 self.console.print(
                     "\n\n❌ PDM installation failed. Is PDM is installed? Run `pipx install pdm` and try again.\n\n"
@@ -921,7 +957,9 @@ class Crackerjack(BaseModel, arbitrary_types_allowed=True):
             if self.pkg_path.stem == "crackerjack":
                 tests_dir = self.pkg_path / "tests"
                 if tests_dir.exists() and tests_dir.is_dir():
-                    self.console.print("\nCleaning tests directory...\n")
+                    self.console.print(
+                        "\n[bright_blue]🧹 Cleaning tests directory...[/bright_blue]\n"
+                    )
                     self.code_cleaner.clean_files(tests_dir)
 
     def _get_test_timeout(self, options: OptionsProtocol, project_size: str) -> int:
@@ -1018,26 +1056,34 @@ class Crackerjack(BaseModel, arbitrary_types_allowed=True):
 
     def _print_ai_agent_files(self, options: t.Any) -> None:
         if getattr(options, "ai_agent", False):
-            self.console.print("📄 Structured test results: test-results.xml")
-            self.console.print("📊 Coverage report: coverage.json")
+            self.console.print(
+                "[dim cyan]📄 Structured test results: test-results.xml[/dim cyan]"
+            )
+            self.console.print("[dim cyan]📊 Coverage report: coverage.json[/dim cyan]")
             if options.benchmark or options.benchmark_regression:
-                self.console.print("⏱️  Benchmark results: benchmark.json")
+                self.console.print(
+                    "[dim cyan]⏱️  Benchmark results: benchmark.json[/dim cyan]"
+                )
 
     def _handle_test_failure(self, result: t.Any, options: t.Any) -> None:
         if result.stderr:
             self.console.print(result.stderr)
-        self.console.print("\n\n❌ Tests failed. Please fix errors.\n")
+        self.console.print(
+            "\n\n[bright_red]❌ Tests failed. Please fix errors.[/bright_red]\n"
+        )
         self._print_ai_agent_files(options)
         raise SystemExit(1)
 
     def _handle_test_success(self, options: t.Any) -> None:
-        self.console.print("\n\n✅ Tests passed successfully!\n")
+        self.console.print(
+            "\n\n[bright_green]✅ Tests passed successfully![/bright_green]\n"
+        )
         self._print_ai_agent_files(options)
 
     def _run_tests(self, options: t.Any) -> None:
         if not options.test:
             return
-        self.console.print("\n\nRunning tests...\n")
+        self.console.print("\n\n[bright_blue]🧪 Running tests...[/bright_blue]\n")
         test_command = self._prepare_pytest_command(options)
         result = self.execute_command(test_command, capture_output=True, text=True)
         if result.stdout:
@@ -1058,7 +1104,7 @@ class Crackerjack(BaseModel, arbitrary_types_allowed=True):
                         default=False,
                     ):
                         self.console.print(
-                            f"[yellow]Skipping {option} version bump[/yellow]"
+                            f"[dim yellow]⏭️  Skipping {option} version bump[/dim yellow]"
                         )
                         return
                 self.execute_command(["pdm", "bump", option])
@@ -1072,7 +1118,9 @@ class Crackerjack(BaseModel, arbitrary_types_allowed=True):
             self.console.print(build.stdout)
             if build.returncode > 0:
                 self.console.print(build.stderr)
-                self.console.print("\n\nBuild failed. Please fix errors.\n")
+                self.console.print(
+                    "\n\n[bright_red]❌ Build failed. Please fix errors.[/bright_red]\n"
+                )
                 raise SystemExit(1)
             self.execute_command(["pdm", "publish", "--no-build"])
 
@@ -1088,7 +1136,7 @@ class Crackerjack(BaseModel, arbitrary_types_allowed=True):
         self, cmd: list[str], **kwargs: t.Any
     ) -> subprocess.CompletedProcess[str]:
         if self.dry_run:
-            self.console.print(f"[yellow]Would run: {' '.join(cmd)}[/yellow]")
+            self.console.print(f"[dim cyan]🔍 Would run: {' '.join(cmd)}[/dim cyan]")
             return CompletedProcess(cmd, 0, "", "")
         return execute(cmd, **kwargs)
 
@@ -1106,12 +1154,12 @@ class Crackerjack(BaseModel, arbitrary_types_allowed=True):
         if not options.skip_hooks:
             self.project_manager.run_pre_commit()
         else:
-            self.console.print("Skipping pre-commit hooks")
+            self.console.print("[dim yellow]⏭️  Skipping pre-commit hooks[/dim yellow]")
         self._run_tests(options)
         self._bump_version(options)
         self._publish_project(options)
         self._commit_and_push(options)
-        self.console.print("\n🍺 Crackerjack complete!\n")
+        self.console.print("\n[bright_green]🍺 Crackerjack complete![/bright_green]\n")
 
 
 crackerjack_it = Crackerjack().process

{crackerjack-0.22.2 → crackerjack-0.22.4}/crackerjack/pyproject.toml

@@ -4,7 +4,7 @@ requires = [ "pdm-backend" ]
 
 [project]
 name = "crackerjack"
-version = "0.22.1"
+version = "0.22.3"
 description = "Crackerjack: code quality toolkit"
 readme = "README.md"
 keywords = [
@@ -43,7 +43,7 @@ classifiers = [
 dependencies = [
     "autotyping>=24.9",
     "keyring>=25.6",
-    "pdm>=2.25.3",
+    "pdm>=2.25.4",
     "pdm-bump>=0.9.12",
     "pre-commit>=4.2",
     "pydantic>=2.11.7",
@@ -53,12 +53,12 @@ dependencies = [
     "pytest-cov>=6.2.1",
     "pytest-mock>=3.14.1",
     "pytest-timeout>=2.4",
-    "pytest-xdist>=3.7",
+    "pytest-xdist>=3.8",
     "pyyaml>=6.0.2",
     "rich>=14",
     "tomli-w>=1.2",
     "typer>=0.16",
-    "uv>=0.7.15",
+    "uv>=0.7.20",
 ]
 urls.documentation = "https://github.com/lesleslie/crackerjack"
 urls.homepage = "https://github.com/lesleslie/crackerjack"

{crackerjack-0.22.2 → crackerjack-0.22.4}/pyproject.toml

@@ -6,7 +6,7 @@ requires = [
 
 [project]
 name = "crackerjack"
-version = "0.22.2"
+version = "0.22.4"
 description = "Crackerjack: code quality toolkit"
 readme = "README.md"
 keywords = [
@@ -43,7 +43,7 @@ classifiers = [
 dependencies = [
     "autotyping>=24.9",
     "keyring>=25.6",
-    "pdm>=2.25.3",
+    "pdm>=2.25.4",
     "pdm-bump>=0.9.12",
     "pre-commit>=4.2",
     "pydantic>=2.11.7",
@@ -53,12 +53,12 @@ dependencies = [
     "pytest-cov>=6.2.1",
     "pytest-mock>=3.14.1",
     "pytest-timeout>=2.4",
-    "pytest-xdist>=3.7",
+    "pytest-xdist>=3.8",
     "pyyaml>=6.0.2",
     "rich>=14",
     "tomli-w>=1.2",
     "typer>=0.16",
-    "uv>=0.7.15",
+    "uv>=0.7.20",
 ]
 
 [project.license]

{crackerjack-0.22.2 → crackerjack-0.22.4}/tests/test_crackerjack.py

@@ -500,7 +500,9 @@ class TestCrackerjackProcess:
                                 mock_tests.assert_called_once()
                                 mock_publish.assert_called_once()
                                 mock_commit.assert_called_once()
-        mock_console_print.assert_any_call("\n🍺 Crackerjack complete!\n")
+        mock_console_print.assert_any_call(
+            "\n[bright_green]🍺 Crackerjack complete![/bright_green]\n"
+        )
 
     def test_process_with_all_option_sets_flags(
         self,
@@ -672,7 +674,9 @@ class TestCrackerjackProcess:
                 cj = Crackerjack(dry_run=True)
                 with suppress(SystemExit):
                     cj.process(options)
-        mock_console_print.assert_any_call("\n\nRunning tests...\n")
+        mock_console_print.assert_any_call(
+            "\n\n[bright_red]❌ Tests failed. Please fix errors.[/bright_red]\n"
+        )
         mock_cj_execute.assert_called_once()
 
     def test_process_with_failed_build(
@@ -701,7 +705,9 @@ class TestCrackerjackProcess:
                     cj = Crackerjack(dry_run=True)
                     with suppress(SystemExit):
                         cj.process(options)
-        mock_console_print.assert_any_call("\n\nBuild failed. Please fix errors.\n")
+        mock_console_print.assert_any_call(
+            "\n\n[bright_red]❌ Build failed. Please fix errors.[/bright_red]\n"
+        )
 
     def test_publish_project_darwin(self) -> None:
         options = OptionsForTesting(publish=BumpOption.micro)