cecli-dev 0.99.4__tar.gz → 0.99.5__tar.gz

{cecli_dev-0.99.4 → cecli_dev-0.99.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 0.99.4
+Version: 0.99.5
 Summary: we also can't pronounce cecli
 Project-URL: Homepage, https://github.com/dwash96/cecli
 Project-URL: Documentation, https://github.com/dwash96/cecli
@@ -52,7 +52,7 @@ Requires-Dist: textual>=6.0.0
 Requires-Dist: tomlkit>=0.14.0
 Requires-Dist: truststore
 Requires-Dist: xxhash>=3.6.0
-Requires-Dist: py-cymbal>=0.1.6
+Requires-Dist: py-cymbal>=0.1.24
 Requires-Dist: rustworkx>=0.15.0
 Requires-Dist: scipy>=1.15.3
 Requires-Dist: importlib-metadata>=7.2.1

{cecli_dev-0.99.4 → cecli_dev-0.99.5}/cecli/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.99.4'
-__version_tuple__ = version_tuple = (0, 99, 4)
+__version__ = version = '0.99.5'
+__version_tuple__ = version_tuple = (0, 99, 5)
 
-__commit_id__ = commit_id = 'g25cca8975'
+__commit_id__ = commit_id = 'g4a4e977c3'

{cecli_dev-0.99.4 → cecli_dev-0.99.5}/cecli/tools/explore_symbols.py

@@ -5,12 +5,16 @@ from cecli.tools.utils.base_tool import BaseTool
 from cecli.tools.utils.helpers import ToolError
 from cecli.tools.utils.output import color_markers, tool_footer, tool_header
 
+cwd = os.getcwd()
+
 try:
     import cymbal
 
     CYMBAL_AVAILABLE = True
 except ImportError:
     CYMBAL_AVAILABLE = False
+finally:
+    os.chdir(cwd)
 
 
 class Tool(BaseTool):
@@ -95,14 +99,12 @@ class Tool(BaseTool):
             repo_path = getattr(coder, "root", ".")
 
             try:
-                # If we can't get a db_path or it doesn't exist, index it.
-                if not os.path.exists(c.db_path):
-                    c.index(repo_path)
+                # Always index to ensure we have the latest data
+                c.index(repo_path)
             except Exception as e:
                 error_msg = f"Failed to index repository: {str(e)}"
                 coder.io.tool_error(error_msg)
                 return f"Error: {error_msg}"
-
             all_results = []
             all_failed_queries = []
             total_successful_queries = 0
@@ -117,11 +119,8 @@ class Tool(BaseTool):
                         results = c.search(symbol, limit=limit)
                         all_results.append(cls._format_search_results(results, symbol))
                     elif action == "investigate":
-                        # Parse symbol for file hint format: {file}:{symbol} or {package}.{symbol}
                         symbol_name = symbol
                         file_hint = ""
-
-                        # Check for file:symbol format (e.g., "config.go:Config")
                         if ":" in symbol:
                             parts = symbol.split(":", 1)
                             if len(parts) == 2:
@@ -135,7 +134,6 @@ class Tool(BaseTool):
                             )
                         except Exception as e:
                             if "multiple matches" in str(e).lower():
-                                # Fallback to search to show locations
                                 results = c.search(symbol_name, limit=10)
                                 locations = "\n".join(
                                     [f"- {r['file']}:{r['start_line']}" for r in results]
@@ -178,6 +176,9 @@ class Tool(BaseTool):
         except Exception as e:
             coder.io.tool_error(f"Error in ExploreSymbols: {str(e)}")
             return f"Error: {str(e)}"
+        finally:
+            if "c" in locals():
+                c.close()
 
     @classmethod
     def _format_search_results(cls, results, symbol):
@@ -187,14 +188,18 @@ class Tool(BaseTool):
 
         formatted = [f"Found {len(results)} symbols matching '{symbol}':"]
         for i, result in enumerate(results[:15], 1):
-            # Extract symbol attributes (adjust based on actual cymbal result structure)
-            # Extract symbol attributes from dictionary
             name = result.get("name", "Unknown")
             kind = result.get("kind", "unknown")
-            file = result.get("file", "Unknown")
+            file = result.get("rel_path") or result.get("file", "Unknown")
             start_line = result.get("start_line", 0)
+            signature = result.get("signature", "")
+            parent = result.get("parent")
 
-            formatted.append(f"{i}. {name} ({kind}) at {file}:{start_line}")
+            location = f"{file}:{start_line}"
+            if parent:
+                location = f"{location} (in {parent})"
+
+            formatted.append(f"{i}. {name}{signature} ({kind}) at {location}")
 
         if len(results) > 15:
             formatted.append(f"... and {len(results) - 15} more results")
@@ -207,17 +212,33 @@ class Tool(BaseTool):
         if not investigation:
             return f"No information found for symbol '{symbol}'"
 
+        # Handle nested structure if present
+        if "results" in investigation and "result" in investigation["results"]:
+            investigation = investigation["results"]["result"]
+
         formatted = [f"Investigation of symbol '{symbol}':"]
 
         # Extract definition information
         definition = investigation.get("symbol")
         if definition:
             def_name = definition.get("name", symbol)
-            def_file = definition.get("file", "Unknown")
+            def_file = definition.get("rel_path") or definition.get("file", "Unknown")
             def_line = definition.get("start_line", 0)
             def_kind = definition.get("kind", "unknown")
-            formatted.append(f"Definition: {def_name} ({def_kind}) at {def_file}:{def_line}")
-
+            def_sig = definition.get("signature", "")
+            formatted.append(
+                f"Definition: {def_name}{def_sig} ({def_kind}) at {def_file}:{def_line}"
+            )
+
+        # Source code snippet
+        source = investigation.get("source")
+        if source:
+            formatted.append("\nSource Code:")
+            formatted.append("```python")
+            formatted.append(source.strip())
+            formatted.append("```")
+
+        # References
         references = investigation.get("refs", [])
         ref_count = len(references) if references else 0
         formatted.append(f"\nReferences found: {ref_count}")
@@ -225,13 +246,26 @@ class Tool(BaseTool):
         if references and ref_count > 0:
             formatted.append("Top references:")
             for i, ref in enumerate(references[:10], 1):
-                ref_file = ref.get("file", "Unknown")
+                ref_file = ref.get("rel_path") or ref.get("file", "Unknown")
                 ref_line = ref.get("line", 0)
                 formatted.append(f"{i}. {ref_file}:{ref_line}")
 
             if ref_count > 10:
                 formatted.append(f"... and {ref_count - 10} more references")
 
+        # Impact / Callers
+        impact = investigation.get("impact", [])
+        if impact:
+            formatted.append("\nImpact (Callers):")
+            for i, imp in enumerate(impact[:10], 1):
+                imp_file = imp.get("rel_path") or imp.get("file", "Unknown")
+                imp_line = imp.get("line", 0)
+                imp_caller = imp.get("caller", "unknown")
+                formatted.append(f"{i}. {imp_caller} at {imp_file}:{imp_line}")
+
+            if len(impact) > 10:
+                formatted.append(f"... and {len(impact) - 10} more callers")
+
         return "\n".join(formatted)
 
     @classmethod
@@ -242,11 +276,15 @@ class Tool(BaseTool):
 
         formatted = [f"Found {len(references)} references to '{symbol}':"]
         for i, ref in enumerate(references[:15], 1):
-            # Extract reference attributes from dictionary
-            file = ref.get("file", "Unknown")
+            file = ref.get("rel_path") or ref.get("file", "Unknown")
             line = ref.get("line", 0)
+            context = ref.get("context", [])
 
             formatted.append(f"{i}. {file}:{line}")
+            if context:
+                formatted.append("   Context:")
+                for line_text in context:
+                    formatted.append(f"     {line_text.strip()}")
 
         if len(references) > 15:
             formatted.append(f"... and {len(references) - 15} more references")
@@ -266,8 +304,6 @@ class Tool(BaseTool):
 
         # Output header
         tool_header(coder=coder, mcp_server=mcp_server, tool_response=tool_response)
-
-        # Output each query with the requested format
         queries = params.get("queries", [])
         if queries:
             coder.io.tool_output("")

cecli_dev-0.99.5/cecli/tools/ls.py

@@ -0,0 +1,114 @@
+import json
+import os
+
+from cecli.tools.utils.base_tool import BaseTool
+from cecli.tools.utils.output import color_markers, tool_footer, tool_header
+
+
+class Tool(BaseTool):
+    NORM_NAME = "ls"
+    TRACK_INVOCATIONS = False
+    SCHEMA = {
+        "type": "function",
+        "function": {
+            "name": "ls",
+            "description": "List files in a directory. Paths are relative to the project root.",
+            "parameters": {
+                "type": "object",
+                "properties": {
+                    "path": {
+                        "type": "string",
+                        "description": (
+                            "The path of the directory to list, relative to the project root. "
+                            "Defaults to the project root."
+                        ),
+                        "default": ".",
+                    }
+                },
+                "required": [],
+            },
+        },
+    }
+
+    @classmethod
+    def execute(cls, coder, path=None, **kwargs):
+        """
+        List files in directory and optionally add some to context.
+
+        This provides information about the structure of the codebase,
+        similar to how a developer would explore directories.
+        """
+        # Handle both positional and keyword arguments for backward compatibility
+        dir_path = path or "."
+
+        try:
+            # Create an absolute path from the provided relative path
+            abs_path = os.path.abspath(os.path.join(coder.root, dir_path))
+
+            # Security check: ensure the resolved path is within the project root
+            if not abs_path.startswith(os.path.abspath(coder.root)):
+                coder.io.tool_error(
+                    f"Error: Path '{dir_path}' attempts to access files outside the project root."
+                )
+                return "Error: Path is outside the project root."
+
+            # Check if path exists
+            if not os.path.exists(abs_path):
+                coder.io.tool_output(f"⚠️ Path '{dir_path}' not found")
+                return "Directory not found"
+
+            # Get directory contents
+            contents = []
+            if os.path.isdir(abs_path):
+                # It's a directory, list its contents
+                try:
+                    with os.scandir(abs_path) as entries:
+                        for entry in entries:
+                            if entry.is_file() and not entry.name.startswith("."):
+                                rel_path = os.path.relpath(entry.path, coder.root)
+                                contents.append(rel_path)
+                except OSError as e:
+                    coder.io.tool_error(f"Error listing directory '{dir_path}': {e}")
+                    return f"Error: {e}"
+            elif os.path.isfile(abs_path):
+                # It's a file, just return its relative path
+                contents.append(os.path.relpath(abs_path, coder.root))
+
+            if contents:
+                coder.io.tool_output(f"📋 Listed {len(contents)} file(s) in '{dir_path}'")
+                sorted_contents = sorted(contents)
+                if len(sorted_contents) > 10:
+                    return (
+                        f"Found {len(sorted_contents)} files: {', '.join(sorted_contents[:10])}..."
+                    )
+                else:
+                    return f"Found {len(sorted_contents)} files: {', '.join(sorted_contents)}"
+            else:
+                coder.io.tool_output(f"📋 No files found in '{dir_path}'")
+                return "No files found in directory"
+        except Exception as e:
+            coder.io.tool_error(f"Error in ls: {str(e)}")
+            return f"Error: {str(e)}"
+
+    @classmethod
+    def format_output(cls, coder, mcp_server, tool_response):
+        """Format output for Ls tool."""
+        color_start, color_end = color_markers(coder)
+
+        try:
+            params = json.loads(tool_response.function.arguments)
+        except json.JSONDecodeError:
+            coder.io.tool_error("Invalid Tool JSON")
+            return
+
+        tool_header(coder=coder, mcp_server=mcp_server, tool_response=tool_response)
+
+        # Output the directory parameter with the requested format
+        directory = params.get("path", "")
+        if directory:
+            # Format as "ls: • directory"
+            formatted_query = f"{color_start}path:{color_end} {directory}"
+            coder.io.tool_output(formatted_query)
+            coder.io.tool_output("")
+
+        tool_footer(coder=coder, tool_response=tool_response)

{cecli_dev-0.99.4 → cecli_dev-0.99.5}/cecli_dev.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 0.99.4
+Version: 0.99.5
 Summary: we also can't pronounce cecli
 Project-URL: Homepage, https://github.com/dwash96/cecli
 Project-URL: Documentation, https://github.com/dwash96/cecli
@@ -52,7 +52,7 @@ Requires-Dist: textual>=6.0.0
 Requires-Dist: tomlkit>=0.14.0
 Requires-Dist: truststore
 Requires-Dist: xxhash>=3.6.0
-Requires-Dist: py-cymbal>=0.1.6
+Requires-Dist: py-cymbal>=0.1.24
 Requires-Dist: rustworkx>=0.15.0
 Requires-Dist: scipy>=1.15.3
 Requires-Dist: importlib-metadata>=7.2.1

{cecli_dev-0.99.4 → cecli_dev-0.99.5}/cecli_dev.egg-info/requires.txt

@@ -32,7 +32,7 @@ textual>=6.0.0
 tomlkit>=0.14.0
 truststore
 xxhash>=3.6.0
-py-cymbal>=0.1.6
+py-cymbal>=0.1.24
 rustworkx>=0.15.0
 scipy>=1.15.3
 importlib-metadata>=7.2.1

{cecli_dev-0.99.4 → cecli_dev-0.99.5}/requirements/common-constraints.txt

@@ -345,7 +345,7 @@ psutil==7.1.3
     # via -r requirements/requirements.in
 ptyprocess==0.7.0
     # via pexpect
-py-cymbal==0.1.6
+py-cymbal==0.1.24
     # via -r requirements/requirements.in
 pycodestyle==2.14.0
     # via flake8

{cecli_dev-0.99.4 → cecli_dev-0.99.5}/requirements/requirements.in

@@ -32,7 +32,7 @@ textual>=6.0.0
 tomlkit>=0.14.0
 truststore
 xxhash>=3.6.0
-py-cymbal>=0.1.6
+py-cymbal>=0.1.24
 
 # Replaced networkx with rustworkx for better performance in repomap
 rustworkx>=0.15.0

{cecli_dev-0.99.4 → cecli_dev-0.99.5}/requirements.txt

@@ -274,7 +274,7 @@ ptyprocess==0.7.0
     # via
     #   -c requirements/common-constraints.txt
     #   pexpect
-py-cymbal==0.1.6
+py-cymbal==0.1.24
     # via
     #   -c requirements/common-constraints.txt
     #   -r requirements/requirements.in

cecli_dev-0.99.4/cecli/tools/ls.py

@@ -1,103 +0,0 @@
-import json
-import os
-
-from cecli.tools.utils.base_tool import BaseTool
-from cecli.tools.utils.output import color_markers, tool_footer, tool_header
-
-
-class Tool(BaseTool):
-    NORM_NAME = "ls"
-    TRACK_INVOCATIONS = False
-    SCHEMA = {
-        "type": "function",
-        "function": {
-            "name": "Ls",
-            "description": "List files in a directory.",
-            "parameters": {
-                "type": "object",
-                "properties": {
-                    "directory": {
-                        "type": "string",
-                        "description": "The directory to list.",
-                    },
-                },
-                "required": ["directory"],
-            },
-        },
-    }
-
-    @classmethod
-    def execute(cls, coder, dir_path=None, directory=None, **kwargs):
-        # Handle both positional and keyword arguments for backward compatibility
-        if dir_path is None and directory is not None:
-            dir_path = directory
-        elif dir_path is None:
-            return "Error: Missing directory parameter"
-        """
-        List files in directory and optionally add some to context.
-
-        This provides information about the structure of the codebase,
-        similar to how a developer would explore directories.
-        """
-        try:
-            # Make the path relative to root if it's absolute
-            if dir_path.startswith("/"):
-                rel_dir = os.path.relpath(dir_path, coder.root)
-            else:
-                rel_dir = dir_path
-
-            # Get absolute path
-            abs_dir = coder.abs_root_path(rel_dir)
-
-            # Check if path exists
-            if not os.path.exists(abs_dir):
-                coder.io.tool_output(f"⚠️ Directory '{dir_path}' not found")
-                return "Directory not found"
-
-            # Get directory contents
-            contents = []
-            try:
-                with os.scandir(abs_dir) as entries:
-                    for entry in entries:
-                        if entry.is_file() and not entry.name.startswith("."):
-                            rel_path = os.path.join(rel_dir, entry.name)
-                            contents.append(rel_path)
-            except NotADirectoryError:
-                # If it's a file, just return the file
-                contents = [rel_dir]
-
-            if contents:
-                coder.io.tool_output(f"📋 Listed {len(contents)} file(s) in '{dir_path}'")
-                if len(contents) > 10:
-                    return f"Found {len(contents)} files: {', '.join(contents[:10])}..."
-                else:
-                    return f"Found {len(contents)} files: {', '.join(contents)}"
-            else:
-                coder.io.tool_output(f"📋 No files found in '{dir_path}'")
-                return "No files found in directory"
-        except Exception as e:
-            coder.io.tool_error(f"Error in ls: {str(e)}")
-            return f"Error: {str(e)}"
-
-    @classmethod
-    def format_output(cls, coder, mcp_server, tool_response):
-        """Format output for Ls tool."""
-        color_start, color_end = color_markers(coder)
-
-        try:
-            params = json.loads(tool_response.function.arguments)
-        except json.JSONDecodeError:
-            coder.io.tool_error("Invalid Tool JSON")
-            return
-
-        tool_header(coder=coder, mcp_server=mcp_server, tool_response=tool_response)
-
-        # Output the directory parameter with the requested format
-        directory = params.get("directory", "")
-        if directory:
-            # Format as "ls: • directory"
-            formatted_query = f"{color_start}directory:{color_end} {directory}"
-            coder.io.tool_output(formatted_query)
-            coder.io.tool_output("")
-
-        tool_footer(coder=coder, tool_response=tool_response)