crackerjack 0.35.1__py3-none-any.whl → 0.36.0__py3-none-any.whl

crackerjack/core/workflow_orchestrator.py

@@ -424,17 +424,31 @@ class WorkflowPipeline:
             if self._is_publishing_workflow(options):
                 return False
 
-        if not await self._execute_publishing_workflow(options, workflow_id):
+        # Execute publishing workflow if requested
+        publishing_success = await self._execute_publishing_workflow(
+            options, workflow_id
+        )
+        if not publishing_success:
             success = False
-            return False
 
-        if not await self._execute_commit_workflow(options, workflow_id):
+        # Execute commit workflow independently if requested
+        # Note: Commit workflow runs regardless of publish success to ensure
+        # version bump changes are always committed when requested
+        commit_success = await self._execute_commit_workflow(options, workflow_id)
+        if not commit_success:
             success = False
 
+        # Only fail the overall workflow if publishing was explicitly requested and failed
+        if not publishing_success and (options.publish or options.all):
+            self.console.print(
+                "[red]❌ Publishing failed - overall workflow marked as failed[/red]"
+            )
+            return False
+
         return success
 
     def _is_publishing_workflow(self, options: OptionsProtocol) -> bool:
-        return bool(options.publish or options.all or options.commit)
+        return bool(options.publish or options.all)
 
     async def _execute_publishing_workflow(
         self, options: OptionsProtocol, workflow_id: str
@@ -1457,7 +1471,7 @@ class WorkflowPipeline:
     def _check_security_gates_for_publishing(
         self, options: OptionsProtocol
     ) -> tuple[bool, bool]:
-        publishing_requested = bool(options.publish or options.all or options.commit)
+        publishing_requested = bool(options.publish or options.all)
 
         if not publishing_requested:
             return False, False

crackerjack/managers/publish_manager.py

@@ -415,12 +415,28 @@ class PublishManagerImpl:
     def _execute_publish(self) -> bool:
         result = self._run_command(["uv", "publish"])
 
-        if result.returncode != 0:
-            self._handle_publish_failure(result.stderr)
-            return False
+        # Check for success indicators in output even if return code is non-zero
+        # UV can return non-zero codes for warnings while still succeeding
+        success_indicators = [
+            "Successfully uploaded",
+            "Package uploaded successfully",
+            "Upload successful",
+            "Successfully published",
+        ]
+
+        has_success_indicator = (
+            any(indicator in result.stdout for indicator in success_indicators)
+            if result.stdout
+            else False
+        )
 
-        self._handle_publish_success()
-        return True
+        # Consider it successful if either return code is 0 OR we find success indicators
+        if result.returncode == 0 or has_success_indicator:
+            self._handle_publish_success()
+            return True
+
+        self._handle_publish_failure(result.stderr)
+        return False
 
     def _handle_publish_failure(self, error_msg: str) -> None:
         self.console.print(f"[red]❌[/ red] Publish failed: {error_msg}")

crackerjack/ui/__init__.py

@@ -1 +1 @@
-"""UI components for Crackerjack."""
+"""UI components for Crackerjack."""

crackerjack/ui/server_panels.py

@@ -1,6 +1,5 @@
 """Rich panel utilities for MCP server operations with consistent styling."""
 
-import typing as t
 from pathlib import Path
 
 from rich.console import Console
@@ -60,7 +59,7 @@ class ServerPanels:
         self,
         http_endpoint: str | None = None,
         websocket_monitor: str | None = None,
-        process_id: int | None = None
+        process_id: int | None = None,
     ) -> None:
         """Display success panel with server details."""
         content = Text()
@@ -123,4 +122,4 @@ class ServerPanels:
 
 def create_server_panels(console: Console | None = None) -> ServerPanels:
     """Factory function to create ServerPanels instance."""
-    return ServerPanels(console)
+    return ServerPanels(console)

{crackerjack-0.35.1.dist-info → crackerjack-0.36.0.dist-info}/METADATA

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

{crackerjack-0.35.1.dist-info → crackerjack-0.36.0.dist-info}/RECORD

@@ -54,7 +54,7 @@ crackerjack/core/service_watchdog.py,sha256=Ttj1imOxvUea4Tkf5JO1e2dQtGIK7D-bX1xO
 crackerjack/core/session_coordinator.py,sha256=TgoGE9DfXe2x-OkH93Ld9dX9ROjx2_mZFkGXen-z5YI,15680
 crackerjack/core/timeout_manager.py,sha256=_sbEsfYDwWx7y0Pn89QCoAZ5DpWIbCdtR9qkG_Kqj5E,15013
 crackerjack/core/websocket_lifecycle.py,sha256=74kn6ugu6FLlDQhCNSPgqguCFwRoT1WFOvtl8G2OyFc,12860
-crackerjack/core/workflow_orchestrator.py,sha256=NgvOn6pTVV8aULxe-bbwT6GeIELRc-KNXU7bYKroGV8,70162
+crackerjack/core/workflow_orchestrator.py,sha256=jh8G1xKqxXwD0yGaFHgdESRjm23-kUZt4qriMJltowM,70789
 crackerjack/docs/INDEX.md,sha256=a6CGFEeL5DX_FRft_JFWd0nOxoBmCSSp-QHIC3B7ato,342
 crackerjack/docs/generated/api/API_REFERENCE.md,sha256=mWoqImZA7AhDvRqqF1MhUo70g_pnZr3NoBeZQRotqN8,155816
 crackerjack/docs/generated/api/CLI_REFERENCE.md,sha256=ikuG0hO5EjIiQlJtAUnvEuAhXDa-JHPULPXNNmUwvk4,2805
@@ -84,7 +84,7 @@ crackerjack/intelligence/integration.py,sha256=vVaC2Fp5RbbABpaohCePzGw1XANuRztGl
 crackerjack/managers/__init__.py,sha256=PFWccXx4hDQA76T02idAViOLVD-aPeVpgjdfSkh_Dmk,298
 crackerjack/managers/async_hook_manager.py,sha256=c0HFR98sFwfk0uZ3NmAe_6OVZpBrq9I570V8A2DoIxw,5129
 crackerjack/managers/hook_manager.py,sha256=_FT0ngwPwujqg0KZGpLz-pP07mwDmptJ5pVkiy5yS8k,7820
-crackerjack/managers/publish_manager.py,sha256=-sHzSVMF6fSUXuPdWA47gBFkFCT8sIb4cPj3GJB-uX4,21370
+crackerjack/managers/publish_manager.py,sha256=rcdKYgN2c4Nwj1JtG1HkaoSQxuN2ZM0_YVvUf9yx1U0,22013
 crackerjack/managers/test_command_builder.py,sha256=1TlPzddNcDDxRORH6UvAudcbRc6hKwFyknSEVLkiWAo,3459
 crackerjack/managers/test_executor.py,sha256=2837Ti4OaNsmLxnmELjbQ18hmfL0-Z2EW-W2UeFSDcE,13871
 crackerjack/managers/test_manager.py,sha256=BRPBWXx4flPDK0w96xyHhg-9dmUca1vpKQRM2VofSlI,13158
@@ -222,10 +222,10 @@ crackerjack/slash_commands/status.md,sha256=U3qqppVLtIIm2lEiMYaKagaHYLI9UplL7OH1
 crackerjack/tools/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 crackerjack/tools/validate_input_validator_patterns.py,sha256=NN7smYlXWrHLQXTb-81gRam2vjW-cJav92f1klPA0qA,8234
 crackerjack/tools/validate_regex_patterns.py,sha256=9ejFb7Tw1js_oydzuEeeeXvrU5ipHUEX9ATBfkLCCE8,5811
-crackerjack/ui/__init__.py,sha256=seLTuIcCzZDvUNfCvm1xNkiw0814JqUxoaQKmocJPpU,36
-crackerjack/ui/server_panels.py,sha256=1MsCbKVPUD6EOONw7miovL_r5CDvllTxjnVU14dKtIE,4249
-crackerjack-0.35.1.dist-info/METADATA,sha256=s6RH3Wqup4CHvmsEcd3fTtqBnshDsrGM9-0cZJkDMgE,37942
-crackerjack-0.35.1.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-crackerjack-0.35.1.dist-info/entry_points.txt,sha256=AJKNft0WXm9xoGUJ3Trl-iXHOWxRAYbagQiza3AILr4,57
-crackerjack-0.35.1.dist-info/licenses/LICENSE,sha256=fDt371P6_6sCu7RyqiZH_AhT1LdN3sN1zjBtqEhDYCk,1531
-crackerjack-0.35.1.dist-info/RECORD,,
+crackerjack/ui/__init__.py,sha256=eMb1OeTU-dSLICAACn0YdYB4Amdr8wHckjKfn0wOIZE,37
+crackerjack/ui/server_panels.py,sha256=F5IH6SNN06BaZQMsFx_D-OA286aojmaFPJ5kvvSRv_c,4232
+crackerjack-0.36.0.dist-info/METADATA,sha256=hWG3NeOyHJk16ZRz9ZMZ7HMxPGYm18YVW_RYfKPrzAc,37942
+crackerjack-0.36.0.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+crackerjack-0.36.0.dist-info/entry_points.txt,sha256=AJKNft0WXm9xoGUJ3Trl-iXHOWxRAYbagQiza3AILr4,57
+crackerjack-0.36.0.dist-info/licenses/LICENSE,sha256=fDt371P6_6sCu7RyqiZH_AhT1LdN3sN1zjBtqEhDYCk,1531
+crackerjack-0.36.0.dist-info/RECORD,,