crackerjack 0.31.9__py3-none-any.whl → 0.31.10__py3-none-any.whl

crackerjack/core/phase_coordinator.py

@@ -157,7 +157,7 @@ class PhaseCoordinator:
                 "CLAUDE.md",
                 "RULES.md",
                 ".gitignore",
-                "mcp.json",
+                "example.mcp.json",
                 "uv.lock",
             ]
 

crackerjack/core/workflow_orchestrator.py

@@ -172,10 +172,12 @@ class WorkflowPipeline:
     async def _execute_workflow_phases(self, options: OptionsProtocol) -> bool:
         success = True
         self.phases.run_configuration_phase(options)
+
         if not self.phases.run_cleaning_phase(options):
             success = False
             self.session.fail_task("workflow", "Cleaning phase failed")
             return False
+
         if not await self._execute_quality_phase(options):
             success = False
             return False
@@ -263,6 +265,21 @@ class WorkflowPipeline:
     ) -> bool:
         """Handle standard workflow where all phases must pass."""
         success = testing_passed and comprehensive_passed
+
+        # Debug information for workflow continuation issues
+        if not success and getattr(options, "verbose", False):
+            self.console.print(
+                f"[yellow]⚠️  Workflow stopped - testing_passed: {testing_passed}, comprehensive_passed: {comprehensive_passed}[/yellow]"
+            )
+            if not testing_passed:
+                self.console.print(
+                    "[yellow]   → Tests reported failure despite appearing successful[/yellow]"
+                )
+            if not comprehensive_passed:
+                self.console.print(
+                    "[yellow]   → Comprehensive hooks reported failure despite appearing successful[/yellow]"
+                )
+
         if options.ai_agent and self._should_debug():
             self.debugger.log_iteration_end(iteration, success)
         return success

crackerjack/managers/test_manager.py

@@ -136,6 +136,36 @@ class TestManager:
         except Exception:
             return None
 
+    def get_coverage(self) -> dict[str, t.Any]:
+        """Get coverage information as required by TestManagerProtocol."""
+        try:
+            # Get the ratchet status which includes coverage information
+            status = self.coverage_ratchet.get_status_report()
+
+            if status.get("status") == "not_initialized":
+                return {
+                    "status": "not_initialized",
+                    "coverage_percent": 0.0,
+                    "message": "Coverage ratchet not initialized",
+                }
+
+            return {
+                "status": "active",
+                "coverage_percent": status.get("current_coverage", 0.0),
+                "target_coverage": status.get("target_coverage", 100.0),
+                "next_milestone": status.get("next_milestone"),
+                "progress_percent": status.get("progress_percent", 0.0),
+                "last_updated": status.get("last_updated"),
+                "milestones_achieved": status.get("milestones_achieved", []),
+            }
+        except Exception as e:
+            return {
+                "status": "error",
+                "coverage_percent": 0.0,
+                "error": str(e),
+                "message": "Failed to get coverage information",
+            }
+
     def has_tests(self) -> bool:
         """Check if project has tests."""
         test_directories = ["tests", "test"]
@@ -219,11 +249,17 @@ class TestManager:
                 self._handle_coverage_improvement(ratchet_result)
             return True
         else:
-            self.console.print(
-                f"[red]📉[/red] Coverage regression: "
-                f"{ratchet_result.get('current_coverage', 0):.2f}% < "
-                f"{ratchet_result.get('previous_coverage', 0):.2f}%"
-            )
+            # Use the message from the ratchet result if available, or construct from available data
+            if "message" in ratchet_result:
+                self.console.print(f"[red]📉[/red] {ratchet_result['message']}")
+            else:
+                # Fallback to constructing message from available keys
+                current = ratchet_result.get("current_coverage", 0)
+                previous = ratchet_result.get("previous_coverage", 0)
+                self.console.print(
+                    f"[red]📉[/red] Coverage regression: "
+                    f"{current:.2f}% < {previous:.2f}%"
+                )
             return False
 
     def _handle_coverage_improvement(self, ratchet_result: dict[str, t.Any]) -> None:

crackerjack/services/coverage_ratchet.py

@@ -332,11 +332,13 @@ class CoverageRatchetService:
             # Look for .coverage file or coverage.json
             coverage_file = self.pkg_path / "coverage.json"
             if not coverage_file.exists():
-                # Look for coverage data in htmlcov/index.html or other standard locations
+                # No coverage data - this is acceptable, return success
                 return {
-                    "success": False,
-                    "error": "No coverage data found",
-                    "message": "Run tests with coverage enabled first",
+                    "success": True,
+                    "status": "no_coverage_data",
+                    "message": "No coverage data found - tests passed without coverage",
+                    "allowed": True,
+                    "baseline_updated": False,
                 }
 
             # Parse coverage data (simplified for now)
@@ -346,7 +348,9 @@ class CoverageRatchetService:
             )
 
             # Update the ratchet
-            return self.update_coverage(current_coverage)
+            result = self.update_coverage(current_coverage)
+            result["success"] = result.get("allowed", True)
+            return result
 
         except Exception as e:
             return {

crackerjack/services/git.py

@@ -29,18 +29,27 @@ class GitService:
 
     def get_changed_files(self) -> list[str]:
         try:
-            staged_result = self._run_git_command(["diff", "--cached", "--name-only"])
+            # Get staged files excluding deletions
+            staged_result = self._run_git_command(
+                ["diff", "--cached", "--name-only", "--diff-filter=ACMRT"]
+            )
             staged_files = (
                 staged_result.stdout.strip().split("\n")
                 if staged_result.stdout.strip()
                 else []
             )
-            unstaged_result = self._run_git_command(["diff", "--name-only"])
+
+            # Get unstaged files excluding deletions
+            unstaged_result = self._run_git_command(
+                ["diff", "--name-only", "--diff-filter=ACMRT"]
+            )
             unstaged_files = (
                 unstaged_result.stdout.strip().split("\n")
                 if unstaged_result.stdout.strip()
                 else []
             )
+
+            # Get untracked files
             untracked_result = self._run_git_command(
                 ["ls-files", "--others", "--exclude-standard"],
             )
@@ -49,6 +58,7 @@ class GitService:
                 if untracked_result.stdout.strip()
                 else []
             )
+
             all_files = set(staged_files + unstaged_files + untracked_files)
             return [f for f in all_files if f]
         except Exception as e:

crackerjack/services/initialization.py

@@ -72,7 +72,7 @@ class InitializationService:
             "pyproject.toml": "smart_merge",
             "CLAUDE.md": "smart_append",
             "RULES.md": "replace_if_missing",
-            "mcp.json": "special",  # Special handling: mcp.json -> .mcp.json with merging
+            "example.mcp.json": "special",  # Special handling: example.mcp.json -> .mcp.json with merging
         }
 
     def _process_config_file(
@@ -84,8 +84,8 @@ class InitializationService:
         force: bool,
         results: dict[str, t.Any],
     ) -> None:
-        # Special handling for mcp.json -> .mcp.json
-        if file_name == "mcp.json":
+        # Special handling for example.mcp.json -> .mcp.json
+        if file_name == "example.mcp.json":
             self._process_mcp_config(target_path, force, results)
             return
 
@@ -248,14 +248,14 @@ class InitializationService:
         force: bool,
         results: dict[str, t.Any],
     ) -> None:
-        """Handle special processing for mcp.json -> .mcp.json with merging."""
-        # Source: mcp.json in crackerjack package (contains servers to add to projects)
-        source_file = self.pkg_path / "mcp.json"
+        """Handle special processing for example.mcp.json -> .mcp.json with merging."""
+        # Source: example.mcp.json in crackerjack package (contains servers to add to projects)
+        source_file = self.pkg_path / "example.mcp.json"
         # Target: .mcp.json in target project
         target_file = target_path / ".mcp.json"
 
         if not source_file.exists():
-            self._handle_missing_source_file("mcp.json", results)
+            self._handle_missing_source_file("example.mcp.json", results)
             return
 
         try:
@@ -265,8 +265,8 @@ class InitializationService:
 
             if not isinstance(source_config.get("mcpServers"), dict):
                 self._handle_file_processing_error(
-                    "mcp.json",
-                    ValueError("Invalid mcp.json format: missing mcpServers"),
+                    "example.mcp.json",
+                    ValueError("Invalid example.mcp.json format: missing mcpServers"),
                     results,
                 )
                 return

{crackerjack-0.31.9.dist-info → crackerjack-0.31.10.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: crackerjack
-Version: 0.31.9
+Version: 0.31.10
 Summary: Opinionated Python project management tool
 Project-URL: documentation, https://github.com/lesleslie/crackerjack
 Project-URL: homepage, https://github.com/lesleslie/crackerjack

{crackerjack-0.31.9.dist-info → crackerjack-0.31.10.dist-info}/RECORD

@@ -37,10 +37,10 @@ crackerjack/core/autofix_coordinator.py,sha256=txbXTeGm-gJwuyRekVDbvvKgG0gQ1GqB3
 crackerjack/core/container.py,sha256=e9_1YnWHijUJ0yl23lpgf9mennVQy8NkgJBKtZstG-M,2889
 crackerjack/core/enhanced_container.py,sha256=fl5XvhNY0fzDnD5hSr16yQXGtL_AW01Wf4F4PL1L0P4,18169
 crackerjack/core/performance.py,sha256=sL9g2-_JflofnsXW6LUCwp9MaUDQprfV8lSG18s9mns,7601
-crackerjack/core/phase_coordinator.py,sha256=iWPVB3A5CFfHwqc3wFXGEQwBhzaa-VlGsvhvRid5FKE,21236
+crackerjack/core/phase_coordinator.py,sha256=L0-OXrxNDfcqSQnhyaeXot1fDk_zdHEWBl-thcbz65Y,21244
 crackerjack/core/proactive_workflow.py,sha256=ML1amNJI4Gx0dFJK5AKdvB0zNc1chbq-ZyqnhUi4tms,12677
 crackerjack/core/session_coordinator.py,sha256=hJKLthZBzX7fXm8AmNMFLEjITNmKxDGqM58Om6p7fr0,9893
-crackerjack/core/workflow_orchestrator.py,sha256=YfqnVartb7qIHEF5qRpy_Fw1ViyICfBf2r0Le9UCiIM,37749
+crackerjack/core/workflow_orchestrator.py,sha256=YHG-qp8J76fqj_poGRvTvuimljbJBvVwPT1r1OjF95M,38459
 crackerjack/executors/__init__.py,sha256=HF-DmXvKN45uKKDdiMxOT9bYxuy1B-Z91BihOhkK5lg,322
 crackerjack/executors/async_hook_executor.py,sha256=3U-AHToGNBojnlDsXK6HLv4CfJvv64UqTmCWYAoLcb8,15958
 crackerjack/executors/cached_hook_executor.py,sha256=LyrFINWbixB-0xEnaU0F2ZUBFUWrAdaTKvj_JW1Wss0,8186
@@ -58,7 +58,7 @@ crackerjack/managers/hook_manager.py,sha256=3qAKLYqoJGPJ8NAUB1KEoWHZafjs6564P9ud
 crackerjack/managers/publish_manager.py,sha256=7bBXkaHm1Ou-tMLvUqNQZScp_onZ2SJgB0o3kwThUDE,16084
 crackerjack/managers/test_command_builder.py,sha256=MdDz9AYSLOoLmI-8zoq3zd2SXF3DeuIkANa76h9cINI,5295
 crackerjack/managers/test_executor.py,sha256=XCMuJPssTV2Glb0hDPHvoDbVrJHVJpZrsAIltegMMfE,16337
-crackerjack/managers/test_manager.py,sha256=BqCaMCUhXx_MJV8CmqXQ-54gZErfdCU5ejgvps4vXPo,9735
+crackerjack/managers/test_manager.py,sha256=4HfnkWy2Sqbu-YW9zkIK0UNc1bnk0SUspK0ldg5RFGI,11387
 crackerjack/managers/test_manager_backup.py,sha256=tptpX99nw-caLJMVga4Hss7grJRqcFHz1JkRBqro4sE,41307
 crackerjack/managers/test_progress.py,sha256=gCNKdE7Bh7RS3K7Ekj2MGKCqiY4px54AzPNi0gMhAL0,3908
 crackerjack/mcp/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
@@ -117,15 +117,15 @@ crackerjack/services/cache.py,sha256=LKZIa8xZ6SzwdTBttO1W6VEBxaTgMNI5Paz_IgrqoaI
 crackerjack/services/config.py,sha256=BrIJoKKi0qyQg2lcZZhTTUpAVB6ej6gkbPv95o2rVvc,14050
 crackerjack/services/config_integrity.py,sha256=wn6b0x90wXNRSj64J_kOQ5JMQvG_9FwtOno0ITVJGwM,3416
 crackerjack/services/contextual_ai_assistant.py,sha256=QnJvJNdGimk-u6XxPm9D7F516AmbueNnAmky42D2caQ,19206
-crackerjack/services/coverage_ratchet.py,sha256=i12mOg5kB2iU3i4hn0LVLcVXJSPkBorO6Xu_nIiGrfs,13567
+crackerjack/services/coverage_ratchet.py,sha256=XM4iCe_Bckh1j378FgAG2zlbiJNCF0VeH1qVWG-fu1s,13731
 crackerjack/services/debug.py,sha256=crjsUZwVjP92aCEOEmtpEJwNf5gtlwNeZF_eB9JSfiI,24018
 crackerjack/services/dependency_monitor.py,sha256=axBXFGBdezoPK9ph5_ZGxIwhSJhurgdvCSiuaCWSrKY,21085
 crackerjack/services/enhanced_filesystem.py,sha256=MQj5zqvkNc5U6ZUmSVzgFQWKfnizD1lv4SJ2pt-w8W4,15424
 crackerjack/services/file_hasher.py,sha256=vHSJ6QbWU5Q5JPLYuQkyRMRXCpDC_hsxToaM83vI58U,5201
 crackerjack/services/filesystem.py,sha256=Re5VyP7H8W6T2tpDakoaghEivdR2VmshJgnZ9Y3QkH8,17932
-crackerjack/services/git.py,sha256=Jth_GiufRkEkUACUyOXELVx92CUXoPAdW5_6G-Mr8Sc,8366
+crackerjack/services/git.py,sha256=ZsIrsYhTuDyRSz4Lhtvh2cEFose7EPSpuDg74Vh8npI,8613
 crackerjack/services/health_metrics.py,sha256=M2OBqwwnGvnJB3eXIXXh5SgMuckYCjHIrD0RkYFAbQU,21458
-crackerjack/services/initialization.py,sha256=xcMqZmIi6pKufwYksJI17Tl7bZ7X7tOLqzNW7cvVZhg,33375
+crackerjack/services/initialization.py,sha256=l3CagjRvBD9AQWkzD_JWQciSCQpYOPduRkPdUprE6MA,33455
 crackerjack/services/log_manager.py,sha256=deM_i97biZVyuZJoHaGlnBitc5QV4WaaZHEb70N5LV0,8388
 crackerjack/services/logging.py,sha256=c15gVCLR_yRhqaza7f1pLLYL-xQ3Oi_OMWL_mR5G46k,5354
 crackerjack/services/metrics.py,sha256=kInkb2G0ML8hAtmEG1jK04b-F1hT_fZjHvYJKisyr1Y,22894
@@ -142,8 +142,8 @@ crackerjack/slash_commands/__init__.py,sha256=ZHfKjluj9dX88zDYN6Saj7tGUMdMnh37Q8
 crackerjack/slash_commands/init.md,sha256=mANRdCiFAzaTw29lKNrI1JFthK4pxVdtiFC5lN2SDSQ,4581
 crackerjack/slash_commands/run.md,sha256=bf_mEtnXagUuw3w8os5h3t1Yi3vjpfiNbkMJvuFEu-Y,6500
 crackerjack/slash_commands/status.md,sha256=U3qqppVLtIIm2lEiMYaKagaHYLI9UplL7OH1j6SRJGw,3921
-crackerjack-0.31.9.dist-info/METADATA,sha256=Y15hY2DSOtPmmEzByI4qpJBfAjPAg2GdnGxCj8SoEvw,22437
-crackerjack-0.31.9.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-crackerjack-0.31.9.dist-info/entry_points.txt,sha256=AJKNft0WXm9xoGUJ3Trl-iXHOWxRAYbagQiza3AILr4,57
-crackerjack-0.31.9.dist-info/licenses/LICENSE,sha256=fDt371P6_6sCu7RyqiZH_AhT1LdN3sN1zjBtqEhDYCk,1531
-crackerjack-0.31.9.dist-info/RECORD,,
+crackerjack-0.31.10.dist-info/METADATA,sha256=4VBwoRFPEEXi1M6Lh6wY7wCEFTAmVKnyRowdCjDT4V8,22438
+crackerjack-0.31.10.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+crackerjack-0.31.10.dist-info/entry_points.txt,sha256=AJKNft0WXm9xoGUJ3Trl-iXHOWxRAYbagQiza3AILr4,57
+crackerjack-0.31.10.dist-info/licenses/LICENSE,sha256=fDt371P6_6sCu7RyqiZH_AhT1LdN3sN1zjBtqEhDYCk,1531
+crackerjack-0.31.10.dist-info/RECORD,,