cecli-dev 1.2.3rc2__tar.gz → 1.2.3rc3__tar.gz

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 1.2.3rc2
+Version: 1.2.3rc3
 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-1.2.3rc2 → cecli_dev-1.2.3rc3}/cecli/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '1.2.3rc2'
-__version_tuple__ = version_tuple = (1, 2, 3, 'rc2')
+__version__ = version = '1.2.3rc3'
+__version_tuple__ = version_tuple = (1, 2, 3, 'rc3')
 
-__commit_id__ = commit_id = 'ga5ff62c22'
+__commit_id__ = commit_id = 'g3477cad89'

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc3}/cecli/prompts/agent.yml

@@ -31,8 +31,6 @@ main_system: |
   - **Unique Lines (`——`):** Lines that appear only once in the file are prefixed with `——`. To target these lines for edits, you can simply reference the exact literal text of the line, excluding the prefix.
   - **Duplicate Lines (e.g., `—“0车加—`):** Lines that appear multiple times are prefixed with an opaque identifier. You MUST include this exact identifier when targeting these lines to disambiguate which specific instance you want to edit.
 
-  Do not attempt to generate, guess, or calculate these identifiers yourself. Always use the exact line contents and prefixes provided from the most recent file read. 
-
   **Example File**
   ```
   ——#!/usr/bin/env python3
@@ -52,6 +50,9 @@ main_system: |
   —니0车加—        return None
   ```
 
+  Do not attempt to generate, guess, or calculate these identifiers yourself. Always use the exact line contents and prefixes provided from the most recent file read.  
+  This system removes the need to recite full file contents.
+
   ## Core Workflow
   1. **Plan**: Start by using `UpdateTodoList` to outline the task.
   2. **Explore**: Use discovery tools (`ExploreCode`, `Grep`, `Ls`) to research and gather understanding for you task. Modify search terms when errors are encountered.

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc3}/cecli/prompts/subagent.yml

@@ -16,8 +16,6 @@ main_system: |
   - **Unique Lines (`——`):** Lines that appear only once in the file are prefixed with `——`. To target these lines for edits, you can simply reference the exact literal text of the line, excluding the prefix.
   - **Duplicate Lines (e.g., `—“0车加—`):** Lines that appear multiple times are prefixed with an opaque identifier. You MUST include this exact identifier when targeting these lines to disambiguate which specific instance you want to edit.
 
-  Do not attempt to generate, guess, or calculate these identifiers yourself. Always use the exact line contents and prefixes provided from the most recent file read. 
-
   **Example File**
   ```
   ——#!/usr/bin/env python3
@@ -37,6 +35,9 @@ main_system: |
   —니0车加—        return None
   ```
 
+  Do not attempt to generate, guess, or calculate these identifiers yourself. Always use the exact line contents and prefixes provided from the most recent file read.  
+  This system removes the need to recite full file contents.
+
   ## Core Workflow
   1. **Plan**: Start by using `UpdateTodoList` to outline the task.
   2. **Explore**: Use discovery tools (`ExploreCode`, `Grep`, `Ls`) to research and gather understanding for you task. Modify search terms when errors are encountered.

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc3}/cecli/tools/read_file.py

@@ -50,8 +50,7 @@ class Tool(BaseTool):
                 "  - when range_start matches one location, range_end accepts '@C{num}' (context both sides), "
                 "    '@P{num}' (lines before the match), '@N{num}' (lines after the match)"
                 ""
-                "Identifiers are deterministic per line content, so adding or removing lines can re-prefix "
-                "identical lines elsewhere in the file; re-read after editing to get fresh identifiers."
+                "File edits may update prefixes of identical lines, requiring re-reading to get fresh identifiers."
                 ""
                 "Large structured ranges (line-number or boundary reads) return a structural outline "
                 "instead of full contents; read in smaller targeted ranges for full detail."

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc3}/cecli/tools/validations/validations.py

@@ -60,6 +60,12 @@ class ToolValidations:
         if not isinstance(params, (dict, list)):
             raise ToolError("Invalid Tool Input - Unparsable JSON")
 
+        # Mirror the execution path (responses.parse_tool_arguments): some models
+        # double-wrap the real params under a single "arguments"/"parameters"/"params"
+        # key. Unwrap so previews resolve the same args that will actually execute.
+        if isinstance(params, dict):
+            params = responses.coerce_tool_structure(params)
+
         # Apply basic structural corrections before declarative validations
         params = cls._basic_validations(params, schema)
 

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc3}/cecli_dev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 1.2.3rc2
+Version: 1.2.3rc3
 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-1.2.3rc3/cecli_dev.egg-info/scm_version.json

@@ -0,0 +1,8 @@
+{
+  "tag": "1.2.3rc3",
+  "distance": 0,
+  "node": "g3477cad895e755095f1c9e5c4d569e814a8173c4",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-08-18"
+}

cecli_dev-1.2.3rc3/tests/basic/test_model_info_manager.py

@@ -0,0 +1,43 @@
+import os
+import tempfile
+from pathlib import Path
+from unittest import TestCase
+from unittest.mock import patch
+
+from cecli.models import ModelInfoManager
+
+
+class TestModelInfoManager(TestCase):
+    def setUp(self):
+        self.original_env = os.environ.copy()
+        self.manager = ModelInfoManager()
+        # Create a temporary directory for cache
+        self.temp_dir = tempfile.TemporaryDirectory()
+        self.manager.cache_dir = Path(self.temp_dir.name)
+        self.manager.cache_file = self.manager.cache_dir / "model_prices_and_context_window.json"
+        self.manager.cache_dir.mkdir(exist_ok=True)
+
+    def tearDown(self):
+        self.temp_dir.cleanup()
+        os.environ.clear()
+        os.environ.update(self.original_env)
+
+    def test_lazy_loading_cache(self):
+        # Create a cache file
+        self.manager.cache_file.write_text('{"test_model": {"max_tokens": 4096}}')
+
+        # Verify cache is not loaded on initialization
+        self.assertFalse(self.manager._cache_loaded)
+        self.assertIsNone(self.manager.content)
+
+        # Access content through get_model_from_cached_json_db
+        with patch.object(self.manager, "_load_cache", wraps=self.manager._load_cache) as mock_load:
+            result = self.manager.get_model_from_cached_json_db("test_model")
+
+            # Verify cache was loaded lazily on first access
+            self.assertTrue(self.manager._cache_loaded)
+            self.assertIsNotNone(self.manager._raw_content)
+            self.assertEqual(result, {"max_tokens": 4096})
+
+            # Verify _load_cache was called exactly once (lazy load on demand)
+            mock_load.assert_called_once()

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc3}/tests/tools/test_edit_file_format_output.py

@@ -101,23 +101,28 @@ def coder_with_file(tmp_path):
     return coder, file_path
 
 
-def make_tool_response(edits):
+def make_tool_response(edits, wrap_arguments=False):
+    arguments = {"edits": edits}
+    if wrap_arguments:
+        # Some models mirror the OpenAI wire format and double-wrap the real
+        # params under a single top-level "arguments" key.
+        arguments = {"arguments": json.dumps(arguments)}
     return SimpleNamespace(
         id="test-id",
         type="function",
         function=SimpleNamespace(
             name="EditFile",
-            arguments=json.dumps({"edits": edits}),
+            arguments=json.dumps(arguments),
         ),
     )
 
 
-def preview_output(coder, edits):
+def preview_output(coder, edits, wrap_arguments=False):
     """Run format_output (as base_coder does before execute) and capture output."""
     edit_file.Tool.format_output(
         coder,
         mcp_server=SimpleNamespace(name="Local"),
-        tool_response=make_tool_response(edits),
+        tool_response=make_tool_response(edits, wrap_arguments=wrap_arguments),
     )
     return "\n".join(coder.io.outputs)
 
@@ -236,3 +241,29 @@ def test_format_output_mixed_selectors_in_batch(coder_with_file):
 
     result = edit_file.Tool.execute(coder, edits=edits)
     assert result.to_dict()["errors"] == []
+
+
+def test_format_output_unwraps_double_wrapped_arguments(coder_with_file):
+    """Double-wrapped {"arguments": "{\"edits\": [...]}"} previews a real diff.
+
+    The execution path unwraps this via responses.parse_tool_arguments, so the
+    preview (format_output) must resolve the same edits instead of rendering
+    nothing.
+    """
+    coder, _ = coder_with_file
+    edits = [
+        {
+            "file_path": "example.txt",
+            "operation": "replace",
+            "start_line": "@L2",
+            "end_line": "@L3",
+            "text": '    print("hello")  # edited',
+        }
+    ]
+
+    output = preview_output(coder, edits, wrap_arguments=True)
+    assert "Preview Unavailable" not in output
+    assert "@@" in output, f"expected a unified diff in preview output:\n{output}"
+
+    result = edit_file.Tool.execute(coder, edits=edits)
+    assert result.to_dict()["errors"] == []

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc3}/tests/tools/validations.py

@@ -311,6 +311,37 @@ class TestValidateParams:
         result = ToolValidations.validate_params(params, None)
         assert result == {"key": "value"}
 
+    def test_arguments_wrapper_string_value_unwrapped(self):
+        """A single top-level "arguments" wrapper (as JSON string) is unwrapped.
+
+        Mirrors the execution path (responses.parse_tool_arguments) so previews
+        resolve the same args that will actually run.
+        """
+        params = '{"arguments": "{"edits": [{"file_path": "a.txt"}]}"}'
+        result = ToolValidations.validate_params(
+            params,
+            {"edits": ["coerce_list"], "edits[]": ["coerce_dict"]},
+        )
+        assert result == {"edits": [{"file_path": "a.txt"}]}
+
+    def test_parameters_wrapper_dict_value_unwrapped(self):
+        """A single top-level "parameters" wrapper (as dict) is unwrapped."""
+        params = {"parameters": {"edits": [{"file_path": "a.txt"}]}}
+        result = ToolValidations.validate_params(
+            params,
+            {"edits": ["coerce_list"], "edits[]": ["coerce_dict"]},
+        )
+        assert result == {"edits": [{"file_path": "a.txt"}]}
+
+    def test_multi_key_params_not_unwrapped(self):
+        """Params with more than one key are never treated as a wrapper."""
+        params = {"edits": [{"file_path": "a.txt"}], "change_id": "x"}
+        result = ToolValidations.validate_params(
+            params,
+            {"edits": ["coerce_list"], "edits[]": ["coerce_dict"]},
+        )
+        assert result == {"edits": [{"file_path": "a.txt"}], "change_id": "x"}
+
     # ---- simple keys ----
 
     def test_simple_key_coerce_list(self):

cecli_dev-1.2.3rc2/cecli_dev.egg-info/scm_version.json

@@ -1,8 +0,0 @@
-{
-  "tag": "1.2.3rc2",
-  "distance": 0,
-  "node": "ga5ff62c2279a65bcbec68fffb7f36d5e4ec8571b",
-  "dirty": false,
-  "branch": "HEAD",
-  "node_date": "2026-08-17"
-}

cecli_dev-1.2.3rc2/tests/basic/test_model_info_manager.py

@@ -1,80 +0,0 @@
-import os
-import tempfile
-from pathlib import Path
-from unittest import TestCase
-from unittest.mock import MagicMock, patch
-
-from cecli.models import ModelInfoManager
-
-
-class TestModelInfoManager(TestCase):
-    def setUp(self):
-        self.original_env = os.environ.copy()
-        self.manager = ModelInfoManager()
-        # Create a temporary directory for cache
-        self.temp_dir = tempfile.TemporaryDirectory()
-        self.manager.cache_dir = Path(self.temp_dir.name)
-        self.manager.cache_file = self.manager.cache_dir / "model_prices_and_context_window.json"
-        self.manager.cache_dir.mkdir(exist_ok=True)
-
-    def tearDown(self):
-        self.temp_dir.cleanup()
-        os.environ.clear()
-        os.environ.update(self.original_env)
-
-    @patch("requests.get")
-    def test_update_cache_respects_verify_ssl(self, mock_get):
-        # Setup mock response
-        mock_response = MagicMock()
-        mock_response.status_code = 200
-        mock_response.json.return_value = {"test_model": {"max_tokens": 4096}}
-        mock_get.return_value = mock_response
-
-        # Test with default verify_ssl=True
-        self.manager._update_cache()
-        mock_get.assert_called_with(self.manager.MODEL_INFO_URL, timeout=5, verify=True)
-
-        # Test with verify_ssl=False
-        mock_get.reset_mock()
-        self.manager.set_verify_ssl(False)
-        self.manager._update_cache()
-        mock_get.assert_called_with(self.manager.MODEL_INFO_URL, timeout=5, verify=False)
-
-    def test_lazy_loading_cache(self):
-        # Create a cache file
-        self.manager.cache_file.write_text('{"test_model": {"max_tokens": 4096}}')
-
-        # Verify cache is not loaded on initialization
-        self.assertFalse(self.manager._cache_loaded)
-        self.assertIsNone(self.manager.content)
-
-        # Access content through get_model_from_cached_json_db
-        with patch.object(self.manager, "_update_cache") as mock_update:
-            result = self.manager.get_model_from_cached_json_db("test_model")
-
-            # Verify cache was loaded
-            self.assertTrue(self.manager._cache_loaded)
-            self.assertIsNotNone(self.manager._raw_content)
-            self.assertEqual(result, {"max_tokens": 4096})
-
-            # Verify _update_cache was not called since cache exists and is valid
-            mock_update.assert_not_called()
-
-    @patch("requests.get")
-    def test_verify_ssl_setting_before_cache_loading(self, mock_get):
-        # Setup mock response
-        mock_response = MagicMock()
-        mock_response.status_code = 200
-        mock_response.json.return_value = {"test_model": {"max_tokens": 4096}}
-        mock_get.return_value = mock_response
-
-        # Set verify_ssl to False before any cache operations
-        self.manager.set_verify_ssl(False)
-
-        # Force cache update by making it look expired
-        with patch("time.time", return_value=9999999999):
-            # This should trigger _update_cache
-            self.manager.get_model_from_cached_json_db("test_model")
-
-            # Verify _update_cache was called with verify=False
-            mock_get.assert_called_with(self.manager.MODEL_INFO_URL, timeout=5, verify=False)