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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 1.2.3rc2
+Version: 1.2.3rc4
 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.3rc4}/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.3rc4'
+__version_tuple__ = version_tuple = (1, 2, 3, 'rc4')
 
-__commit_id__ = commit_id = 'ga5ff62c22'
+__commit_id__ = commit_id = 'g0cce57c1b'

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/cecli/coders/base_coder.py

@@ -3363,7 +3363,7 @@ class Coder(metaclass=UsageMeta):
                     continue
 
                 for tool in server_tools:
-                    if server_name == "Local":
+                    if server_name.lower() == "local":
                         # Apply per-instance tool name filtering
                         tool_name = tool.get("function", {}).get("name", "")
                         if (

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/cecli/commands/load_mcp.py

@@ -60,13 +60,13 @@ class LoadMcpCommand(BaseCommand):
         # This moves them from "implicitly include all" to explicit state-machine
         # management, preventing the new server from being implicitly available
         # to all coders.
-        connected_names = {s.name for s in coder.mcp_manager.connected_servers}
+        connected_names = {s.name.lower() for s in coder.mcp_manager.connected_servers}
         if connected_names:
             for c in iter_all_coders(coder):
                 if not c.registered_servers["included"]:
                     included = set(connected_names) - c.registered_servers["excluded"]
                     if c.edit_format in ("agent", "subagent"):
-                        included.add("Local")  # "local" is always available
+                        included.add("local")  # "local" is always available
                     c.registered_servers["included"] = included
 
         # Process connections with interrupt support

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/cecli/commands/remove_mcp.py

@@ -49,7 +49,7 @@ class RemoveMcpCommand(BaseCommand):
             server_name = item.name if hasattr(item, "name") else item
 
             # Never remove the "local" server
-            if server_name == "Local":
+            if server_name.lower() == "local":
                 results.append("Cannot remove 'Local' server")
                 continue
 

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/cecli/commands/utils/helpers.py

@@ -348,7 +348,7 @@ def update_server_registration(coder, server_name, operation, force=False):
                When False, respect the opposing set (excluded wins
                for include, included wins for exclude).
     """
-    name = server_name
+    name = server_name.lower()
     included = coder.registered_servers["included"]
     excluded = coder.registered_servers["excluded"]
 
@@ -379,7 +379,7 @@ def is_server_globally_excluded(coder, server_name):
     Returns:
         True if the server is excluded from every coder.
     """
-    name = server_name
+    name = server_name.lower()
     for other in iter_all_coders(coder):
         incl = other.registered_servers["included"]
         excl = other.registered_servers["excluded"]

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/cecli/helpers/agents/service.py

@@ -565,11 +565,19 @@ class AgentService:
         )
 
         if agent_config:
+            # Reset the per-instance tool/server filters so AgentCoder.post_init()
+            # rebuilds them from *this* sub-agent's own agent-config (tools_includelist/
+            # excludelist, servers_includelist/excludelist) instead of inheriting the
+            # parent's filters verbatim. Deliberately do NOT reset mcp_manager here:
+            # doing so used to discard the parent's already-connected MCP servers and
+            # replace them with a brand new, empty manager, making it impossible for a
+            # sub-agent's servers_includelist to ever match anything. Leaving mcp_manager
+            # unset lets it inherit from from_coder (see Coder.create()) so the same
+            # connected MCP servers remain available and can then be filtered.
             kwargs.update(
                 dict(
                     registered_tools=None,
                     registered_servers=None,
-                    mcp_manager=None,
                 )
             )
 

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/cecli/helpers/responses.py

@@ -366,11 +366,18 @@ def unprefix_tool_call(tool_call):
     return server_name, result
 
 
-def parse_tool_arguments(args_string: str) -> dict:
+def parse_tool_arguments(args_string: str) -> dict | list:
     """Parse tool-call arguments, merging glued ``{…}{} {…}`` object fragments.
 
     Also unwraps a single ``arguments``/``parameters``/``params`` wrapper key
     that some models emit when mirroring the OpenAI wire format.
+
+    Returns a dict in almost all cases. A bare top-level JSON array (e.g. a
+    model emitting ``[...]`` directly as the arguments instead of wrapping it
+    under the tool's single required array parameter, as EditFile's `edits`
+    or ReadFile's `read`) is returned as-is rather than collapsed to ``{}``,
+    so ``BaseTool.process_response`` can still wrap it using the tool's
+    schema instead of silently losing the data.
     """
     text = (args_string or "").strip()
     if not text:
@@ -379,6 +386,8 @@ def parse_tool_arguments(args_string: str) -> dict:
         parsed = json.loads(text)
         if isinstance(parsed, dict):
             return coerce_tool_structure(parsed)
+        if isinstance(parsed, list):
+            return parsed
     except json.JSONDecodeError:
         pass
 
@@ -393,12 +402,18 @@ def parse_tool_arguments(args_string: str) -> dict:
         lone = try_parse_json_value(chunks[0])
         if isinstance(lone, dict):
             return coerce_tool_structure(lone)
+        if isinstance(lone, list):
+            return lone
         try:
             json_string = json_repair.repair_json(chunks[0], ensure_ascii=False)
             single = json.loads(json_string)
         except json.JSONDecodeError as err:
             return {"@error": f"Malformed JSON arguments: {err}"}
-        return coerce_tool_structure(single) if isinstance(single, dict) else {}
+        if isinstance(single, dict):
+            return coerce_tool_structure(single)
+        if isinstance(single, list):
+            return single
+        return {}
 
     merged = merge_glued_json_objects(chunks)
 

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/cecli/mcp/manager.py

@@ -99,7 +99,7 @@ class McpServerManager:
             The server instance or None if not found
         """
         try:
-            return next(server for server in self._servers if server.name == name)
+            return next(server for server in self._servers if server.name.lower() == name.lower())
         except StopIteration:
             return None
 
@@ -254,7 +254,7 @@ class McpServerManager:
         """
         existing_server = self.get_server(server.name)
         if existing_server:
-            if server.name not in ["unnamed-server", "Local"]:
+            if server.name.lower() not in ["unnamed-server", "local"]:
                 self._log_warning(f"MCP server with name '{server.name}' already exists")
             return False
 
@@ -332,7 +332,7 @@ class McpServerManager:
 
         results = await asyncio.gather(*tasks)
         for server, did_connect in results:
-            if not did_connect and server.name not in ["unnamed-server", "Local"]:
+            if not did_connect and server.name.lower() not in ["unnamed-server", "local"]:
                 io.tool_warning(
                     f"MCP tool initialization failed after multiple retries: {server.name}"
                 )

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/cecli/mcp/server.py

@@ -54,7 +54,7 @@ class McpServer:
             verbose: Whether to output verbose logging
         """
         self.config = server_config
-        self.name = server_config.get("name", "unnamed-server")
+        self.name = str(server_config.get("name", "unnamed-server")).lower()
         self.io = io
         self.verbose = verbose
         self.session = None

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/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.3rc4}/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.3rc4}/cecli/tools/command.py

@@ -722,6 +722,14 @@ class Tool(BaseTool):
         elif command:
             coder.io.tool_output(f"{color_start}Command:{color_end}")
             coder.io.tool_output(coder.format_command_with_prefix(command))
+        else:
+            # No command and no background_key/action pair: this call will
+            # be rejected by execute() before it ever reaches confirmation.
+            # Say so explicitly instead of leaving the panel looking blank,
+            # which is otherwise indistinguishable from a display bug.
+            coder.io.tool_output(
+                f"{color_start}(no command provided — call will be rejected){color_end}"
+            )
 
         coder.io.tool_output("")
 

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/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.3rc4}/cecli/tools/resource_manager.py

@@ -199,15 +199,15 @@ class Tool(BaseTool):
         # included sets to explicit include lists.
         if load_mcp_servers and coder.mcp_manager:
             if isinstance(coder.mcp_manager.connected_servers, dict):
-                connected_names = set(coder.mcp_manager.connected_servers.keys())
+                connected_names = {k.lower() for k in coder.mcp_manager.connected_servers.keys()}
             else:
-                connected_names = {s.name for s in coder.mcp_manager.connected_servers}
+                connected_names = {s.name.lower() for s in coder.mcp_manager.connected_servers}
             if connected_names:
                 for c in iter_all_coders(coder):
                     if not c.registered_servers["included"]:
                         included = set(connected_names) - c.registered_servers["excluded"]
                         if c.edit_format in ("agent", "subagent"):
-                            included.add("Local")
+                            included.add("local")
                         c.registered_servers["included"] = included
 
         for f in create_files:
@@ -546,7 +546,7 @@ class Tool(BaseTool):
             if not coder.mcp_manager or not coder.mcp_manager.servers:
                 return "No MCP servers are configured."
 
-            if server_name == "Local":
+            if server_name.lower() == "local":
                 return "Cannot remove 'Local' server"
 
             server = coder.mcp_manager.get_server(server_name)

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/cecli/tools/utils/base_tool.py

@@ -63,25 +63,29 @@ class BaseTool(ABC):
                 required_params = function_schema["parameters"]["required"]
                 properties = function_schema["parameters"].get("properties", {})
 
-                # Auto-correction: If a required parameter is missing but it's an array,
-                # and the current params look like a single item of that array, wrap it.
-                if len(required_params) == 1:
-                    missing_param = required_params[0]
-                    if missing_param not in params and params:
-                        param_schema = properties.get(missing_param, {})
-                        if param_schema.get("type") == "array":
-                            params = {missing_param: [params]}
+                # Auto-correction: fix common shape mistakes (a bare array or
+                # a single item of that array sent directly as the whole
+                # arguments, instead of wrapped under the expected key) BEFORE
+                # checking for missing required parameters. Otherwise a
+                # recoverable shape (e.g. a bare `[...]` array) gets rejected
+                # before it has a chance to be normalized.
+                params = ToolValidations._basic_validations(params, cls.SCHEMA)
 
                 # Auto-correction: If a required parameter is present but is a dict instead of an array
-                for param_name in required_params:
-                    if param_name in params:
-                        param_schema = properties.get(param_name, {})
-                        if param_schema.get("type") == "array" and isinstance(
-                            params[param_name], dict
-                        ):
-                            params[param_name] = [params[param_name]]
-
-                missing_params = [param for param in required_params if param not in params]
+                if isinstance(params, dict):
+                    for param_name in required_params:
+                        if param_name in params:
+                            param_schema = properties.get(param_name, {})
+                            if param_schema.get("type") == "array" and isinstance(
+                                params[param_name], dict
+                            ):
+                                params[param_name] = [params[param_name]]
+
+                missing_params = [
+                    param
+                    for param in required_params
+                    if not isinstance(params, dict) or param not in params
+                ]
                 if missing_params:
                     tool_name = function_schema.get("name", "Unknown Tool")
                     error_msg = (

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/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)
 
@@ -198,16 +204,28 @@ class ToolValidations:
 
         parameters = function_schema["parameters"]
         properties = parameters.get("properties", {})
-
-        # Only auto-correct when there is exactly one property and it is an array
-        if len(properties) == 1:
+        required = parameters.get("required", [])
+
+        # Determine the single array-typed parameter to auto-correct into,
+        # if any. Prefer the schema's `required` list (covers tools that also
+        # declare optional properties alongside their one required array,
+        # e.g. EditFile's optional `change_id`); fall back to "exactly one
+        # property total" when the schema doesn't declare `required` at all.
+        single_param_name = None
+        if len(required) == 1:
+            single_param_name = required[0]
+        elif not required and len(properties) == 1:
             single_param_name = next(iter(properties.keys()))
-            param_schema = properties[single_param_name]
+
+        if single_param_name is not None:
+            param_schema = properties.get(single_param_name, {})
             if param_schema.get("type") == "array":
-                # Case 1: LLM emitted the array directly (bare list)
+                # Case 1: LLM emitted the array directly (bare list) → wrap
+                # it as-is under the expected key, don't nest it again.
                 if isinstance(params, list):
                     return {single_param_name: params}
-                # Case 2: LLM emitted a dict missing the expected key → wrap it
+                # Case 2: LLM emitted a single item of the array directly
+                # as a dict, missing the expected wrapper key → wrap it.
                 if isinstance(params, dict) and single_param_name not in params:
                     return {single_param_name: [params]}
 

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 1.2.3rc2
+Version: 1.2.3rc4
 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.3rc4}/cecli_dev.egg-info/SOURCES.txt

@@ -787,6 +787,7 @@ tests/mcp/test_keepalive_resilience.py
 tests/mcp/test_keepalive_unit.py
 tests/mcp/test_manager.py
 tests/mcp/test_manager_retry.py
+tests/mcp/test_server_name_normalization.py
 tests/mcp/test_server_sdk_compat.py
 tests/scrape/test_playwright_disable.py
 tests/scrape/test_scrape.py
@@ -799,6 +800,7 @@ tests/subagents/test_finished.py
 tests/subagents/test_io_proxy.py
 tests/subagents/test_service.py
 tests/subagents/test_sub_agent_coder.py
+tests/tools/test_command_format_output.py
 tests/tools/test_edit_file_format_output.py
 tests/tools/test_extractions.py
 tests/tools/test_get_lines.py

{cecli_dev-1.2.3rc2 → cecli_dev-1.2.3rc4}/cecli_dev.egg-info/scm_file_list.json

@@ -1028,6 +1028,7 @@
     "tests/mcp/test_keepalive_unit.py",
     "tests/mcp/test_manager.py",
     "tests/mcp/test_manager_retry.py",
+    "tests/mcp/test_server_name_normalization.py",
     "tests/mcp/test_server_sdk_compat.py",
     "tests/scrape/test_playwright_disable.py",
     "tests/scrape/test_scrape.py",
@@ -1041,6 +1042,7 @@
     "tests/subagents/test_service.py",
     "tests/subagents/test_sub_agent_coder.py",
     "tests/test_requests.py",
+    "tests/tools/test_command_format_output.py",
     "tests/tools/test_edit_file_format_output.py",
     "tests/tools/test_extractions.py",
     "tests/tools/test_get_lines.py",

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

@@ -0,0 +1,8 @@
+{
+  "tag": "1.2.3rc4",
+  "distance": 0,
+  "node": "g0cce57c1b8208c7067fb545241fa84b5bf17f790",
+  "dirty": false,
+  "branch": "HEAD",
+  "node_date": "2026-08-19"
+}

cecli_dev-1.2.3rc4/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()