cecli-dev 0.100.0rc1__tar.gz → 0.100.0rc4__tar.gz

{cecli_dev-0.100.0rc1 → cecli_dev-0.100.0rc4}/.github/workflows/release.yml

@@ -4,7 +4,7 @@ on:
   workflow_dispatch:
   push:
     tags:
-      - 'v[0-9]+.[0-9]+.[0-9]+'
+      - 'v[0-9]*.[0-9]*.[0-9]*'
 
 jobs:
   publish_cecli:

{cecli_dev-0.100.0rc1 → cecli_dev-0.100.0rc4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 0.100.0rc1
+Version: 0.100.0rc4
 Summary: we also can't pronounce cecli
 Project-URL: Homepage, https://github.com/dwash96/cecli
 Project-URL: Documentation, https://github.com/dwash96/cecli

{cecli_dev-0.100.0rc1 → cecli_dev-0.100.0rc4}/cecli/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.100.0rc1'
-__version_tuple__ = version_tuple = (0, 100, 0, 'rc1')
+__version__ = version = '0.100.0rc4'
+__version_tuple__ = version_tuple = (0, 100, 0, 'rc4')
 
-__commit_id__ = commit_id = 'gd703941b6'
+__commit_id__ = commit_id = 'g20f9c40e6'

{cecli_dev-0.100.0rc1 → cecli_dev-0.100.0rc4}/cecli/tools/read_range.py

@@ -254,7 +254,10 @@ class Tool(BaseTool):
                                 candidates.append((dist_sum, s, e))
                         # Sort by distance sum, then prefer ranges after the last range
                         candidates.sort(key=lambda x: (x[0], x[1] < last_s, x[1], x[2]))
-                        best_pair = (candidates[0][1], candidates[0][2])
+                        if candidates:
+                            best_pair = (candidates[0][1], candidates[0][2])
+                        else:
+                            best_pair = None
                     else:
                         best_pair = None
                         min_dist = float("inf")

{cecli_dev-0.100.0rc1 → cecli_dev-0.100.0rc4}/cecli_dev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 0.100.0rc1
+Version: 0.100.0rc4
 Summary: we also can't pronounce cecli
 Project-URL: Homepage, https://github.com/dwash96/cecli
 Project-URL: Documentation, https://github.com/dwash96/cecli

{cecli_dev-0.100.0rc1 → cecli_dev-0.100.0rc4}/tests/subagents/test_commands.py

@@ -104,7 +104,11 @@ class TestInvokeAgentCommand:
             mock_instance.invoke = AsyncMock(return_value="task done")
             MockSvc.get_instance.return_value = mock_instance
 
-            await InvokeAgentCommand.execute(io, coder, "reviewer do it")
+            with patch("cecli.helpers.conversation.service.ConversationService") as MockCS:
+                mock_manager = MagicMock()
+                MockCS.get_manager.return_value = mock_manager
+
+                await InvokeAgentCommand.execute(io, coder, "reviewer do it")
 
         io.tool_output.assert_called_once()
         assert "task done" in io.tool_output.call_args[0][0]

{cecli_dev-0.100.0rc1 → cecli_dev-0.100.0rc4}/tests/subagents/test_delegate.py

@@ -15,7 +15,7 @@ class TestDelegateTool:
         """Missing name returns error string."""
         from cecli.tools.delegate import Tool
 
-        result = await Tool.execute(None, name="", prompt="do it")
+        result = await Tool.execute(None, delegations=[{"name": "", "prompt": "do it"}])
         assert "Error" in result
         assert "name" in result
 
@@ -24,7 +24,7 @@ class TestDelegateTool:
         """Missing prompt returns error string."""
         from cecli.tools.delegate import Tool
 
-        result = await Tool.execute(None, name="reviewer", prompt="")
+        result = await Tool.execute(None, delegations=[{"name": "reviewer", "prompt": ""}])
         assert "Error" in result
         assert "prompt" in result
 
@@ -33,7 +33,7 @@ class TestDelegateTool:
         """Both empty — name error comes first."""
         from cecli.tools.delegate import Tool
 
-        result = await Tool.execute(None, name="", prompt="")
+        result = await Tool.execute(None, delegations=[{"name": "", "prompt": ""}])
         assert "Error" in result
         assert "name" in result
 
@@ -50,7 +50,9 @@ class TestDelegateTool:
             mock_instance.invoke = AsyncMock(return_value="review summary")
             MockService.get_instance.return_value = mock_instance
 
-            result = await Tool.execute(mock_coder, name="reviewer", prompt="review this")
+            result = await Tool.execute(
+                mock_coder, delegations=[{"name": "reviewer", "prompt": "review this"}]
+            )
 
             MockService.get_instance.assert_called_once_with(mock_coder)
             mock_instance.invoke.assert_called_once_with("reviewer", "review this", blocking=True)
@@ -67,7 +69,9 @@ class TestDelegateTool:
             mock_instance.invoke = AsyncMock(return_value=None)
             MockService.get_instance.return_value = mock_instance
 
-            result = await Tool.execute(mock_coder, name="tester", prompt="test")
+            result = await Tool.execute(
+                mock_coder, delegations=[{"name": "tester", "prompt": "test"}]
+            )
             assert "completed (no summary)" in result
 
     @pytest.mark.asyncio
@@ -81,8 +85,8 @@ class TestDelegateTool:
             mock_instance.invoke = AsyncMock(side_effect=ValueError("unknown agent"))
             MockService.get_instance.return_value = mock_instance
 
-            result = await Tool.execute(mock_coder, name="ghost", prompt="x")
-            assert "Error" in result
+            result = await Tool.execute(mock_coder, delegations=[{"name": "ghost", "prompt": "x"}])
+            assert "failed" in result
             assert "unknown agent" in result
 
     @pytest.mark.asyncio
@@ -96,8 +100,10 @@ class TestDelegateTool:
             mock_instance.invoke = AsyncMock(side_effect=RuntimeError("max reached"))
             MockService.get_instance.return_value = mock_instance
 
-            result = await Tool.execute(mock_coder, name="reviewer", prompt="x")
-            assert "Error" in result
+            result = await Tool.execute(
+                mock_coder, delegations=[{"name": "reviewer", "prompt": "x"}]
+            )
+            assert "failed" in result
             assert "max reached" in result
 
     @pytest.mark.asyncio
@@ -111,6 +117,8 @@ class TestDelegateTool:
             mock_instance.invoke = AsyncMock(side_effect=Exception("unexpected"))
             MockService.get_instance.return_value = mock_instance
 
-            result = await Tool.execute(mock_coder, name="reviewer", prompt="x")
-            assert "Error" in result
+            result = await Tool.execute(
+                mock_coder, delegations=[{"name": "reviewer", "prompt": "x"}]
+            )
+            assert "failed with unexpected error" in result
             assert "unexpected" in result

{cecli_dev-0.100.0rc1 → cecli_dev-0.100.0rc4}/tests/subagents/test_sub_agent_coder.py

@@ -133,7 +133,7 @@ class TestSubAgentCoder:
         coder.use_enhanced_context = True
         coder.choose_fence = MagicMock()
 
-        with patch("cecli.coders.sub_agent_coder.ConversationService") as MockCS:
+        with patch("cecli.coders.agent_coder.ConversationService") as MockCS:
             mock_chunks = MagicMock()
             mock_manager = MagicMock()
             MockCS.get_chunks.return_value = mock_chunks