mdify-cli 2.5.0__tar.gz → 2.6.0__tar.gz

{mdify_cli-2.5.0/mdify_cli.egg-info → mdify_cli-2.6.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mdify-cli
-Version: 2.5.0
+Version: 2.6.0
 Summary: Convert PDFs and document images into structured Markdown for LLM workflows
 Author: tiroq
 License-Expression: MIT

{mdify_cli-2.5.0 → mdify_cli-2.6.0}/mdify/__init__.py

@@ -1,3 +1,3 @@
 """mdify - Convert documents to Markdown via Docling container."""
 
-__version__ = "2.5.0"
+__version__ = "2.6.0"

{mdify_cli-2.5.0 → mdify_cli-2.6.0}/mdify/container.py

@@ -41,6 +41,39 @@ class DoclingContainer:
         """Return base URL for API requests."""
         return f"http://localhost:{self.port}"
 
+    def _cleanup_stale_containers(self) -> None:
+        """Stop any existing mdify-serve containers.
+
+        This handles the case where a previous run left a container running
+        (e.g., due to crash, interrupt, or timeout).
+        """
+        # Find running containers matching mdify-serve-* pattern
+        result = subprocess.run(
+            [
+                self.runtime,
+                "ps",
+                "--filter",
+                "name=mdify-serve-",
+                "--format",
+                "{{.Names}}",
+            ],
+            capture_output=True,
+            text=True,
+            check=False,
+        )
+
+        if result.returncode != 0 or not result.stdout.strip():
+            return
+
+        # Stop each stale container
+        for container_name in result.stdout.strip().split("\n"):
+            if container_name:
+                subprocess.run(
+                    [self.runtime, "stop", container_name],
+                    capture_output=True,
+                    check=False,
+                )
+
     def start(self, timeout: int = 120) -> None:
         """Start container and wait for health check.
 
@@ -51,6 +84,8 @@ class DoclingContainer:
             subprocess.CalledProcessError: If container fails to start
             TimeoutError: If health check doesn't pass within timeout
         """
+        self._cleanup_stale_containers()
+
         # Start container in detached mode
         cmd = [
             self.runtime,

{mdify_cli-2.5.0 → mdify_cli-2.6.0/mdify_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mdify-cli
-Version: 2.5.0
+Version: 2.6.0
 Summary: Convert PDFs and document images into structured Markdown for LLM workflows
 Author: tiroq
 License-Expression: MIT

{mdify_cli-2.5.0 → mdify_cli-2.6.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "mdify-cli"
-version = "2.5.0"
+version = "2.6.0"
 description = "Convert PDFs and document images into structured Markdown for LLM workflows"
 readme = "README.md"
 requires-python = ">=3.8"

{mdify_cli-2.5.0 → mdify_cli-2.6.0}/tests/test_container.py

@@ -311,7 +311,110 @@ class TestDoclingContainerIntegration:
             container1 = DoclingContainer("docker", "image1", port=5001)
             container2 = DoclingContainer("docker", "image2", port=5002)
 
-            # Each should be independent
             assert container1.port == 5001
             assert container2.port == 5002
             assert container1.container_name != container2.container_name
+
+
+class TestDoclingContainerCleanup:
+    """Test cleanup of stale containers."""
+
+    def test_cleanup_no_stale_containers(self):
+        """Test cleanup runs when no stale containers exist."""
+        with patch("mdify.container.subprocess.run") as mock_run, patch(
+            "mdify.container.check_health"
+        ) as mock_health:
+            ps_result = Mock()
+            ps_result.returncode = 0
+            ps_result.stdout = ""
+
+            run_result = Mock()
+            run_result.stdout = "container_id\n"
+
+            mock_run.side_effect = [ps_result, run_result]
+            mock_health.return_value = True
+
+            container = DoclingContainer("docker", "test-image")
+            container.start(timeout=5)
+
+            ps_call = mock_run.call_args_list[0][0][0]
+            assert "ps" in ps_call
+            assert "--filter" in ps_call
+            assert "name=mdify-serve-" in ps_call
+
+    def test_cleanup_stops_stale_containers(self):
+        """Test cleanup finds and stops stale containers."""
+        with patch("mdify.container.subprocess.run") as mock_run, patch(
+            "mdify.container.check_health"
+        ) as mock_health:
+            ps_result = Mock()
+            ps_result.returncode = 0
+            ps_result.stdout = "mdify-serve-abc123\nmdify-serve-def456\n"
+
+            stop_result1 = Mock()
+            stop_result2 = Mock()
+
+            run_result = Mock()
+            run_result.stdout = "container_id\n"
+
+            mock_run.side_effect = [ps_result, stop_result1, stop_result2, run_result]
+            mock_health.return_value = True
+
+            container = DoclingContainer("docker", "test-image")
+            container.start(timeout=5)
+
+            ps_call = mock_run.call_args_list[0][0][0]
+            assert "ps" in ps_call
+
+            stop_calls = [
+                call for call in mock_run.call_args_list if "stop" in str(call)
+            ]
+            assert len(stop_calls) == 2
+            assert "mdify-serve-abc123" in str(stop_calls[0])
+            assert "mdify-serve-def456" in str(stop_calls[1])
+
+    def test_cleanup_handles_subprocess_error(self):
+        """Test cleanup handles subprocess errors gracefully."""
+        with patch("mdify.container.subprocess.run") as mock_run, patch(
+            "mdify.container.check_health"
+        ) as mock_health:
+            ps_result = Mock()
+            ps_result.returncode = 1
+            ps_result.stdout = ""
+
+            run_result = Mock()
+            run_result.stdout = "container_id\n"
+
+            mock_run.side_effect = [ps_result, run_result]
+            mock_health.return_value = True
+
+            container = DoclingContainer("docker", "test-image")
+            container.start(timeout=5)
+
+            assert container.container_id == "container_id"
+
+    def test_start_calls_cleanup(self):
+        """Test that start() calls _cleanup_stale_containers()."""
+        with patch("mdify.container.subprocess.run") as mock_run, patch(
+            "mdify.container.check_health"
+        ) as mock_health:
+            ps_result = Mock()
+            ps_result.returncode = 0
+            ps_result.stdout = ""
+
+            run_result = Mock()
+            run_result.stdout = "new_container_id\n"
+
+            mock_run.side_effect = [ps_result, run_result]
+            mock_health.return_value = True
+
+            container = DoclingContainer("docker", "test-image")
+            container.start(timeout=5)
+
+            all_calls = mock_run.call_args_list
+            ps_called = any("ps" in str(call) for call in all_calls[:1])
+            run_called = any("run" in str(call) for call in all_calls)
+
+            assert ps_called
+            assert run_called
+            assert "ps" in all_calls[0][0][0]