cecli-dev 0.99.0__tar.gz → 0.99.1__tar.gz

{cecli_dev-0.99.0 → cecli_dev-0.99.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cecli-dev
-Version: 0.99.0
+Version: 0.99.1
 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.99.0 → cecli_dev-0.99.1}/cecli/__init__.py

@@ -1,6 +1,6 @@
 from packaging import version
 
-__version__ = "0.99.0.dev"
+__version__ = "0.99.1.dev"
 safe_version = __version__
 
 try:

{cecli_dev-0.99.0 → cecli_dev-0.99.1}/cecli/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '0.99.0'
-__version_tuple__ = version_tuple = (0, 99, 0)
+__version__ = version = '0.99.1'
+__version_tuple__ = version_tuple = (0, 99, 1)
 
-__commit_id__ = commit_id = 'g12b89af88'
+__commit_id__ = commit_id = 'g587390916'

{cecli_dev-0.99.0 → cecli_dev-0.99.1}/cecli/coders/agent_coder.py

@@ -1009,37 +1009,44 @@ I will proceed based on the tool results and updated context.""")
         repetition_warning = None
 
         if repetitive_tools:
+            default_temp = (
+                float(self.get_active_model().use_temperature)
+                if isinstance(self.get_active_model().use_temperature, (int, float, str))
+                else 1
+            )
+            default_fp = 0
+
             if not self.model_kwargs:
                 self.model_kwargs = {
-                    "temperature": (
-                        1
-                        if isinstance(self.get_active_model().use_temperature, bool)
-                        else float(self.get_active_model().use_temperature)
-                    ) + 0.1,
-                    "frequency_penalty": 0.2,
+                    "temperature": default_temp + 0.1,
+                    "frequency_penalty": default_fp + 0.2,
                     "presence_penalty": 0.1,
                 }
             else:
-                temperature = nested.getter(self.model_kwargs, "temperature")
-                freq_penalty = nested.getter(self.model_kwargs, "frequency_penalty")
-                if temperature and freq_penalty:
-                    self.model_kwargs["temperature"] = min(temperature + 0.1, 2)
-                    self.model_kwargs["frequency_penalty"] = min(freq_penalty + 0.1, 1)
+                temperature = nested.getter(self.model_kwargs, "temperature", default_temp)
+                freq_penalty = nested.getter(self.model_kwargs, "frequency_penalty", default_fp)
+
+                self.model_kwargs["temperature"] = temperature + 0.1
+                self.model_kwargs["frequency_penalty"] = freq_penalty + 0.1
 
                 if random.random() < 0.2:
-                    self.model_kwargs["temperature"] = min(
-                        (
-                            1
-                            if isinstance(self.get_active_model().use_temperature, bool)
-                            else float(self.get_active_model().use_temperature)
-                        ),
-                        max(temperature - 0.15, 1),
+                    self.model_kwargs["temperature"] = max(
+                        default_temp,
+                        temperature - 0.15,
+                    )
+                    self.model_kwargs["frequency_penalty"] = max(
+                        default_fp,
+                        freq_penalty - 0.15,
                     )
-                    self.model_kwargs["frequency_penalty"] = min(0, max(freq_penalty - 0.15, 0))
 
             self.model_kwargs["temperature"] = max(
-                0, min(nested.getter(self.model_kwargs, "temperature", 1), 1)
+                0, min(nested.getter(self.model_kwargs, "temperature", default_temp), 1)
             )
+
+            self.model_kwargs["frequency_penalty"] = max(
+                0, min(nested.getter(self.model_kwargs, "frequency_penalty", default_fp), 1)
+            )
+
             # One twentieth of the time, just straight reset the randomness
             if random.random() < 0.05:
                 self.model_kwargs = {}

{cecli_dev-0.99.0 → cecli_dev-0.99.1}/cecli/coders/base_coder.py

@@ -3065,7 +3065,7 @@ class Coder:
                 self.temperature,
                 # This could include any tools, but for now it is just MCP tools
                 tools=tools,
-                override_kwargs=self.model_kwargs,
+                override_kwargs=self.model_kwargs.copy(),
             )
             self.chat_completion_call_hashes.append(hash_object.hexdigest())
 

{cecli_dev-0.99.0 → cecli_dev-0.99.1}/cecli/helpers/background_commands.py

@@ -5,11 +5,22 @@ Provides a static BackgroundCommandManager class for running shell commands
 in the background and capturing their output for injection into chat streams.
 """
 
+import codecs
+import os
+import platform
 import subprocess
 import threading
 from collections import deque
 from typing import Dict, Optional, Tuple
 
+try:
+    import pty
+    import termios
+
+    HAS_PTY = True
+except ImportError:
+    HAS_PTY = False
+
 
 class CircularBuffer:
     """
@@ -89,14 +100,48 @@ class CircularBuffer:
             return sum(len(chunk) for chunk in self.buffer)
 
 
+class InputBuffer:
+    """
+    Thread-safe buffer for queuing input to be sent to a process.
+    """
+
+    def __init__(self):
+        self.queue = deque()
+        self.lock = threading.Lock()
+
+    def append(self, text: str) -> None:
+        """Add text to the input queue."""
+        with self.lock:
+            self.queue.append(text)
+
+    def pop_all(self) -> str:
+        """Get and clear all queued input."""
+        with self.lock:
+            result = "".join(self.queue)
+            self.queue.clear()
+            return result
+
+    def has_input(self) -> bool:
+        """Check if there is queued input."""
+        with self.lock:
+            return len(self.queue) > 0
+
+
 class BackgroundProcess:
     """
     Represents a background process with output capture.
     """
 
     def __init__(
-        self, command: str, process: subprocess.Popen, buffer: CircularBuffer, persist: bool = False
+        self,
+        command: str,
+        process: subprocess.Popen,
+        buffer: CircularBuffer,
+        persist: bool = False,
+        input_buffer: Optional[InputBuffer] = None,
+        master_fd: Optional[int] = None,
     ):
+        self.master_fd = master_fd
         """
         Initialize background process wrapper.
 
@@ -116,7 +161,11 @@ class BackgroundProcess:
         self.start_time = time.time()
         self.end_time = None
         self.persist = persist
+        self.input_buffer = input_buffer or InputBuffer()
+        self.writer_thread = None
+        self._stop_event = threading.Event()
         self._start_output_reader()
+        self._start_input_writer()
 
     def _start_output_reader(self) -> None:
         """Start thread to read process output."""
@@ -128,15 +177,25 @@ class BackgroundProcess:
                 # we're in a separate thread and the buffer will capture
                 # output as soon as it's available
 
-                # Read stdout
-                for line in iter(self.process.stdout.readline, ""):
-                    if line:
-                        self.buffer.append(line)
+                if self.master_fd is not None:
+                    while not self._stop_event.is_set():
+                        try:
+                            data = os.read(self.master_fd, 4096).decode(errors="replace")
+                            if not data:
+                                break
+                            self.buffer.append(data)
+                        except (OSError, EOFError):
+                            break
+                else:
+                    # Read stdout
+                    for line in iter(self.process.stdout.readline, ""):
+                        if line:
+                            self.buffer.append(line)
 
-                # Read stderr
-                for line in iter(self.process.stderr.readline, ""):
-                    if line:
-                        self.buffer.append(line)
+                    # Read stderr
+                    for line in iter(self.process.stderr.readline, ""):
+                        if line:
+                            self.buffer.append(line)
 
             except Exception as e:
                 self.buffer.append(f"\n[Error reading process output: {str(e)}]\n")
@@ -144,6 +203,45 @@ class BackgroundProcess:
         self.reader_thread = threading.Thread(target=reader, daemon=True)
         self.reader_thread.start()
 
+    def _start_input_writer(self) -> None:
+        """Start thread to write input to process stdin."""
+
+        def writer():
+            try:
+                while not self._stop_event.is_set() and self.is_alive():
+                    if self.input_buffer.has_input():
+                        text = self.input_buffer.pop_all()
+                        if text:
+                            if self.master_fd is not None:
+                                os.write(self.master_fd, text.encode("latin-1"))
+                            else:
+                                try:
+                                    # Try to write to the binary buffer for lossless propagation
+                                    self.process.stdin.buffer.write(text.encode("latin-1"))
+                                    self.process.stdin.buffer.flush()
+                                except (AttributeError, ValueError):
+                                    # Fallback to text mode if buffer is not available
+                                    self.process.stdin.write(text)
+                                    self.process.stdin.flush()
+                    import time
+
+                    time.sleep(0.1)
+            except (BrokenPipeError, OSError):
+                pass
+            except Exception as e:
+                self.buffer.append(f"\n[Error writing to process input: {str(e)}]\n")
+            finally:
+                try:
+                    if self.master_fd is not None:
+                        os.close(self.master_fd)
+                    else:
+                        self.process.stdin.close()
+                except Exception:
+                    pass
+
+        self.writer_thread = threading.Thread(target=writer, daemon=True)
+        self.writer_thread.start()
+
     def get_output(self, clear: bool = False) -> str:
         """
         Get current output buffer.
@@ -171,6 +269,11 @@ class BackgroundProcess:
         """Check if process is running."""
         return self.process.poll() is None
 
+    def send_input(self, text: str) -> None:
+        """Queue input to be sent to the process."""
+        if self.input_buffer:
+            self.input_buffer.append(text)
+
     def stop(self, timeout: float = 5.0) -> Tuple[bool, str, Optional[int]]:
         """
         Stop the process gracefully.
@@ -184,6 +287,9 @@ class BackgroundProcess:
         import time
 
         try:
+            # Signal threads to stop
+            self._stop_event.set()
+
             # Try SIGTERM first
             self.process.terminate()
             self.process.wait(timeout=timeout)
@@ -192,7 +298,6 @@ class BackgroundProcess:
             output = self.get_output(clear=True)
             exit_code = self.process.returncode
             self.end_time = time.time()
-
             return True, output, exit_code
 
         except subprocess.TimeoutExpired:
@@ -262,6 +367,8 @@ class BackgroundCommandManager:
         existing_process: Optional[subprocess.Popen] = None,
         existing_buffer: Optional[CircularBuffer] = None,
         persist: bool = False,
+        existing_input_buffer: Optional[InputBuffer] = None,
+        use_pty: bool = False,
     ) -> str:
         """
         Start a command in background.
@@ -283,7 +390,29 @@ class BackgroundCommandManager:
             buffer = existing_buffer or CircularBuffer(max_size=max_buffer_size)
 
             # Use existing process or start new one
-            if existing_process:
+            master_fd = None
+            if use_pty and HAS_PTY and platform.system() != "Windows":
+                master_fd, slave_fd = pty.openpty()
+
+                # Disable echo on the slave PTY
+                attr = termios.tcgetattr(slave_fd)
+                attr[3] = attr[3] & ~termios.ECHO
+                termios.tcsetattr(slave_fd, termios.TCSANOW, attr)
+
+                process = subprocess.Popen(
+                    command,
+                    shell=True,
+                    stdout=slave_fd,
+                    stderr=slave_fd,
+                    stdin=slave_fd,
+                    cwd=cwd,
+                    close_fds=True,
+                    text=True,
+                    bufsize=1,
+                    universal_newlines=True,
+                )
+                os.close(slave_fd)
+            elif existing_process:
                 process = existing_process
             else:
                 process = subprocess.Popen(
@@ -291,15 +420,22 @@ class BackgroundCommandManager:
                     shell=True,
                     stdout=subprocess.PIPE,
                     stderr=subprocess.PIPE,
-                    stdin=subprocess.DEVNULL,  # No stdin for background commands
+                    stdin=subprocess.PIPE,
                     cwd=cwd,
-                    text=True,  # Use text mode for easier handling
-                    bufsize=1,  # Line buffered
+                    text=True,
+                    bufsize=1,
                     universal_newlines=True,
                 )
 
             # Create background process wrapper
-            bg_process = BackgroundProcess(command, process, buffer, persist=persist)
+            bg_process = BackgroundProcess(
+                command,
+                process,
+                buffer,
+                persist=persist,
+                input_buffer=existing_input_buffer,
+                master_fd=master_fd,
+            )
 
             # Generate unique key and store
             command_key = cls._generate_command_key(command)
@@ -367,6 +503,30 @@ class BackgroundCommandManager:
                 return f"[Error] No background command found with key: {command_key}"
             return bg_process.get_new_output()
 
+    @classmethod
+    def send_command_input(cls, command_key: str, text: str) -> bool:
+        """
+        Send input to a background command.
+
+        Args:
+            command_key: Command key returned by start_background_command
+            text: Text to send to the command's stdin
+
+        Returns:
+            True if input was queued, False if command not found
+        """
+        with cls._lock:
+            bg_process = cls._background_commands.get(command_key)
+            if not bg_process:
+                return False
+            # Decode escape sequences (like \x1b) if present in the string
+            try:
+                text = codecs.decode(text, "unicode_escape")
+            except Exception:
+                pass
+            bg_process.send_input(text)
+            return True
+
     @classmethod
     def get_all_command_outputs(cls, clear: bool = False) -> Dict[str, str]:
         """

{cecli_dev-0.99.0 → cecli_dev-0.99.1}/cecli/repo.py

@@ -128,9 +128,8 @@ class GitRepo:
             self.io.tool_error("Files are in different git repos.")
             raise FileNotFoundError
 
-        # https://github.com/gitpython-developers/GitPython/issues/427
-        self.repo = git.Repo(repo_paths.pop(), odbt=git.GitCmdObjectDB)
-        self.root = utils.safe_abs_path(self.repo.working_tree_dir)
+        self._init_repo_path = repo_paths.pop()
+        self.init_repo()
 
         if cecli_ignore_file:
             self.cecli_ignore_file = Path(cecli_ignore_file)
@@ -140,6 +139,18 @@ class GitRepo:
         if self.workspace_path:
             self.io.tool_output(f"Working in workspace: {self.workspace_path.name}")
 
+    def init_repo(self):
+        if not self.repo:
+            self.repo = git.Repo(self._init_repo_path, odbt=git.GitCmdObjectDB)
+            self.root = utils.safe_abs_path(self.repo.working_tree_dir)
+
+        try:
+            commit = self.repo.head.commit
+            return commit
+        except ANY_GIT_ERROR:
+            self.repo = git.Repo(self._init_repo_path, odbt=git.GitCmdObjectDB)
+            self.root = utils.safe_abs_path(self.repo.working_tree_dir)
+
     def _detect_workspace_path(self, start_path: str):
         """Check if current directory is within a workspace"""
         current = Path(start_path).resolve()
@@ -478,6 +489,8 @@ class GitRepo:
         if not self.repo:
             return []
 
+        self.init_repo()
+
         try:
             commit = self.repo.head.commit
         except ValueError:

{cecli_dev-0.99.0 → cecli_dev-0.99.1}/cecli/tools/command.py

@@ -17,42 +17,99 @@ class Tool(BaseTool):
             "parameters": {
                 "type": "object",
                 "properties": {
-                    "command_string": {
+                    "command": {
                         "type": "string",
-                        "description": "The shell command to execute.",
+                        "description": (
+                            "The shell command to execute. To send stdin to an existing background"
+                            " command, use the format 'command_key::{key}'."
+                        ),
                     },
                     "background": {
                         "type": "boolean",
                         "description": "Run command in background (non-blocking).",
                         "default": False,
                     },
-                    "stop_background": {
+                    "stop": {
+                        "type": "boolean",
+                        "description": (
+                            "If true, stop the background command specified in the 'command'"
+                            " parameter (format 'command_key::{key}')."
+                        ),
+                        "default": False,
+                    },
+                    "pty": {
+                        "type": "boolean",
+                        "description": (
+                            "Run the command in a pseudo-terminal (PTY). Useful for interactive"
+                            " programs like 'vi' or 'top'."
+                        ),
+                        "default": False,
+                    },
+                    "stdin": {
                         "type": "string",
-                        "description": "Command string to stop if running in background.",
+                        "description": (
+                            "Input to send to the command's stdin. Supports escape sequences like"
+                            " \\n, \\r, \\t, and hex escapes like \\x1b."
+                        ),
                     },
                 },
-                "required": ["command_string"],
+                "required": ["command"],
             },
         },
     }
 
+    @staticmethod
+    def _parse_command_key(command):
+        """Extract command key from command string if it follows the pattern."""
+        if command and command.startswith("command_key::"):
+            return command.split("::", 1)[1].strip()
+        return None
+
     @classmethod
-    async def execute(cls, coder, command_string, background=False, stop_background=None, **kwargs):
+    async def execute(
+        cls, coder, command, background=False, stop=None, stdin=None, pty=False, **kwargs
+    ):
         """
         Execute a shell command, optionally in background.
         Commands run with timeout based on agent_config['command_timeout'] (default: 30 seconds).
         """
+        command_key = cls._parse_command_key(command)
+
         # Handle stopping background commands
-        if stop_background:
-            return await cls._stop_background_command(coder, stop_background)
+        if stop:
+            if not command_key:
+                return (
+                    "Error: 'command' in format 'command_key::{key}' is required when 'stop' is"
+                    " true."
+                )
+            return await cls._stop_background_command(coder, command_key)
+
+        # Handle sending stdin to an existing background command
+        if stdin:
+            if not command_key:
+                return (
+                    "Error: 'command' in format 'command_key::{key}' is required when using"
+                    " 'stdin'."
+                )
+
+            cls.clear_invocation_cache()
+
+            success = BackgroundCommandManager.send_command_input(command_key, stdin)
+            if success:
+                return f"Sent input to background command {command_key}: {stdin}"
+            else:
+                return f"Error: Background command {command_key} not found or not running."
+
+        if not command:
+            return "Error: 'command' must be provided."
 
         # Check for implicit background (trailing & on Linux)
-        if not background and command_string.strip().endswith("&"):
+        if not background and command.strip().endswith("&"):
             background = True
-            command_string = command_string.strip()[:-1].strip()
+            command = command.strip()[:-1].strip()
 
         # Get user confirmation
-        confirmed = await cls._get_confirmation(coder, command_string, background)
+        confirmed = await cls._get_confirmation(coder, command, background)
         if not confirmed:
             return "Command execution skipped by user."
 
@@ -62,11 +119,11 @@ class Tool(BaseTool):
             timeout = coder.agent_config.get("command_timeout", 30)
 
         if background:
-            return await cls._execute_background(coder, command_string)
+            return await cls._execute_background(coder, command, use_pty=pty)
         elif timeout > 0:
-            return await cls._execute_with_timeout(coder, command_string, timeout)
+            return await cls._execute_with_timeout(coder, command, timeout, use_pty=pty)
         else:
-            return await cls._execute_foreground(coder, command_string)
+            return await cls._execute_foreground(coder, command)
 
     @classmethod
     async def _get_confirmation(cls, coder, command_string, background):
@@ -90,7 +147,7 @@ class Tool(BaseTool):
         )
 
     @classmethod
-    async def _execute_background(cls, coder, command_string):
+    async def _execute_background(cls, coder, command_string, use_pty=False):
         """
         Execute command in background.
         """
@@ -98,7 +155,11 @@ class Tool(BaseTool):
 
         # Use static manager to start background command
         command_key = BackgroundCommandManager.start_background_command(
-            command_string, verbose=coder.verbose, cwd=coder.root, max_buffer_size=4096
+            command_string,
+            verbose=coder.verbose,
+            cwd=coder.root,
+            max_buffer_size=4096,
+            use_pty=use_pty,
         )
 
         return (
@@ -108,7 +169,7 @@ class Tool(BaseTool):
         )
 
     @classmethod
-    async def _execute_with_timeout(cls, coder, command_string, timeout):
+    async def _execute_with_timeout(cls, coder, command_string, timeout, use_pty=False):
         """
         Execute command with timeout. If timeout elapses, move to background.
 
@@ -136,7 +197,7 @@ class Tool(BaseTool):
             executable=shell if platform.system() != "Windows" else None,
             stdout=subprocess.PIPE,
             stderr=subprocess.PIPE,
-            stdin=subprocess.DEVNULL,
+            stdin=subprocess.PIPE,
             cwd=coder.root,
             text=True,
             bufsize=1,

{cecli_dev-0.99.0 → cecli_dev-0.99.1}/cecli/tools/command_interactive.py

@@ -7,6 +7,7 @@ from cecli.tools.utils.base_tool import BaseTool
 
 class Tool(BaseTool):
     NORM_NAME = "commandinteractive"
+    TRACK_INVOCATIONS = False
     SCHEMA = {
         "type": "function",
         "function": {
@@ -92,6 +93,8 @@ class Tool(BaseTool):
                     " large_file_token_threshold)"
                 )
 
+            cls.clear_invocation_cache()
+
             if exit_status == 0:
                 return (
                     "Interactive command finished successfully (exit code 0)."

{cecli_dev-0.99.0 → cecli_dev-0.99.1}/cecli/tools/delete_text.py

@@ -64,8 +64,6 @@ class Tool(BaseTool):
             raise ToolError(
                 "Please call `ShowContext` first to make sure edits are appropriately scoped"
             )
-        else:
-            coder.edit_allowed = False
 
         tool_name = "DeleteText"
         try:
@@ -120,6 +118,8 @@ class Tool(BaseTool):
             )
 
             coder.files_edited_by_tools.add(rel_path)
+            cls.clear_invocation_cache()
+
             # 5. Format and return result
             success_message = f"Deleted lines {start_line} to {end_line} in {file_path}"
             return format_tool_result(
@@ -131,7 +131,9 @@ class Tool(BaseTool):
 
         except ToolError as e:
             # Handle errors raised by utility functions (expected errors)
+            coder.edit_allowed = False
             return handle_tool_error(coder, tool_name, e, add_traceback=False)
         except Exception as e:
             # Handle unexpected errors
+            coder.edit_allowed = False
             return handle_tool_error(coder, tool_name, e)

{cecli_dev-0.99.0 → cecli_dev-0.99.1}/cecli/tools/finished.py

@@ -26,6 +26,7 @@ class Tool(BaseTool):
 
         This gives the LLM explicit control over when it can stop looping
         """
+        cls.clear_invocation_cache()
 
         if coder:
             coder.agent_finished = True