cecli-dev 0.100.6rc6__tar.gz → 0.100.6rc7__tar.gz

{cecli_dev-0.100.6rc6 → cecli_dev-0.100.6rc7}/PKG-INFO

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

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.100.6rc6'
-__version_tuple__ = version_tuple = (0, 100, 6, 'rc6')
+__version__ = version = '0.100.6rc7'
+__version_tuple__ = version_tuple = (0, 100, 6, 'rc7')
 
-__commit_id__ = commit_id = 'gb2b12bdf9'
+__commit_id__ = commit_id = 'g7f1d0176c'

{cecli_dev-0.100.6rc6 → cecli_dev-0.100.6rc7}/cecli/helpers/conversation/files.py

@@ -281,8 +281,8 @@ class ConversationFiles:
             diff_message = {
                 "role": "user",
                 "content": (
-                    f"{rel_fname} has been updated. Review this git diff of the changes to"
-                    f" ensure the modifications are intended:\n\n{diff}"
+                    f"{rel_fname} has been updated. Review this diff of the changes to"
+                    f" ensure all modifications are appropriate:\n\n{diff}"
                 ),
             }
 
@@ -290,7 +290,7 @@ class ConversationFiles:
                 "role": "assistant",
                 "content": (
                     f"Thank you for sharing this diff of the updates to {rel_fname}."
-                    " I will review their contents next turn."
+                    " I will review their contents."
                 ),
             }
 

{cecli_dev-0.100.6rc6 → cecli_dev-0.100.6rc7}/cecli/tools/edit_text.py

@@ -42,7 +42,8 @@ class Tool(BaseTool):
                 "Each edit must include its own file_path and operation type. "
                 "Use content ID ranges with the start_line and end_line parameters with format "
                 "`content_id::` (the content id with the :: demarcator). For empty files, use `@000` as the "
-                "content ID references."
+                "content ID references. "
+                "Edits within a file must not be adjacent or overlapping."
             ),
             "parameters": {
                 "type": "object",
@@ -91,7 +92,6 @@ class Tool(BaseTool):
                         "description": "Array of edits to apply.",
                     },
                     "change_id": {"type": "string"},
-                    "dry_run": {"type": "boolean", "default": False},
                 },
                 "required": ["edits"],
             },

{cecli_dev-0.100.6rc6 → cecli_dev-0.100.6rc7}/cecli/tools/explore_code.py

@@ -120,7 +120,10 @@ class Tool(BaseTool):
 
                 try:
                     if action == "search":
-                        results = c.search(symbol, limit=limit)
+                        # Sanitize symbol: Cymbal's CLI interprets hyphens as SQL operators.
+                        # Replace hyphens with underscores (common in code) and strip special chars.
+                        safe_symbol = symbol.replace("-", "_") if symbol else symbol
+                        results = c.search(safe_symbol, limit=limit)
                         all_results.append(cls._format_search_results(results, symbol))
                     elif action == "investigate":
                         symbol_name = symbol
@@ -131,8 +134,11 @@ class Tool(BaseTool):
                                 file_hint = parts[0]
                                 symbol_name = parts[1]
 
+                        # Sanitize for Cymbal search
+                        safe_name = symbol_name.replace("-", "_") if symbol_name else symbol_name
+
                         try:
-                            investigation = c.investigate(symbol_name, file_hint)
+                            investigation = c.investigate(safe_name, file_hint)
                             all_results.append(
                                 cls._format_investigation_results(investigation, symbol)
                             )
@@ -151,7 +157,8 @@ class Tool(BaseTool):
                             else:
                                 raise e
                     elif action == "find_references":
-                        references = c.find_references(symbol, limit=limit)
+                        safe_symbol = symbol.replace("-", "_") if symbol else symbol
+                        references = c.find_references(safe_symbol, limit=limit)
                         all_results.append(cls._format_reference_results(references, symbol))
                     else:
                         all_failed_queries.append(

{cecli_dev-0.100.6rc6 → cecli_dev-0.100.6rc7}/cecli/tools/read_range.py

@@ -36,12 +36,13 @@ class Tool(BaseTool):
                 " respectively. Line numbers may also be used for range lookups."
                 " It is best to use function names, variable declarations and other meaningful identifiers"
                 " as range_start and range_end values."
-                " Do not use both of the special markers together on non-empty file."
                 " Do not use the same pattern for the range_start and range_end."
                 " Do not use empty strings for the range_start and range_end."
-                " Prefer using this tool over cli tools for reading files."
-                " Calling this tool sequentially on increasingly finer grained searches "
-                " will help with understanding important structural features."
+                " Use this tool instead of cli tools for reading file contents."
+                " Line number and special marker ranges greater than 200 lines will return"
+                " preview content for further, more scoped investigation."
+                " Call this tool sequentially on increasingly finer grained searches "
+                " to help with understanding important structural features in large files."
             ),
             "parameters": {
                 "type": "object",
@@ -436,7 +437,7 @@ class Tool(BaseTool):
                 # For structured searches (line numbers, special markers) or mixed searches
                 # (one special marker, one text pattern), cap large ranges with preview
                 # Text pattern searches are not subject to capping
-                if (both_structured or mixed_special_search) and (e_idx - s_idx > 200):
+                if both_structured or (mixed_special_search and (e_idx - s_idx > 200)):
                     preview = cls._get_range_preview(
                         abs_path, coder.io, start_idx=s_idx, end_idx=e_idx, line_numbers=True
                     )

{cecli_dev-0.100.6rc6 → cecli_dev-0.100.6rc7}/cecli/tools/utils/helpers.py

@@ -338,6 +338,9 @@ def format_tool_result(
             result_for_llm += f" Change ID: {change_id}."
         if diff_snippet:
             result_for_llm += f" Diff snippet:\n{diff_snippet}"
+        else:
+            result_for_llm += " A diff will be provided in a future message."
+
         return result_for_llm
 
 

{cecli_dev-0.100.6rc6 → cecli_dev-0.100.6rc7}/cecli_dev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 0.100.6rc6
+Version: 0.100.6rc7
 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.6rc6 → cecli_dev-0.100.6rc7}/tests/tools/test_read_range_execute.py

@@ -162,7 +162,7 @@ class TestReadRangeExecute:
         try:
             show = [{"file_path": self.test_file, "range_start": "5", "range_end": "10"}]
             result = self.Tool.execute(self.coder, show)
-            assert "Snapshot" in result
+            assert "File range too large" in result
             assert "line5" in result
             assert "line10" in result
         finally:
@@ -456,7 +456,7 @@ class TestReadRangeExecute:
         cs_patch = patch("cecli.helpers.conversation.ConversationService", mock_cs)
         cs_patch.start()
 
-        mock_coder.io.read_text.side_effect = [content1, content2]
+        mock_coder.io.read_text.side_effect = [content1, content1, content2, content2]
 
         try:
             from cecli.tools.read_range import Tool