fix(cli): Abort functionality, autocomplete, and fixed memory prompts (#246)

* fix(cli): Fix @ file autocomplete - directory navigation and backspace

- Directories now auto-add / and continue showing contents
- Backspace updates completions instead of closing menu
- File parsing stops at punctuation (@README.md? → README.md)

DirectoryAwarePathCompleter wraps PathCompleter to detect and add / to directories.
Backspace key binding retriggers completions. Regex: [A-Za-z0-9._/~-]+

File: input.py

* fix(cli): Fix bash command interrupt handling

Use signal handlers to properly handle Ctrl+C during bash command execution.
Previously, Ctrl+C would exit the entire CLI; now it only stops the command.

- Install custom SIGINT handler during bash command execution
- Kill subprocess immediately when Ctrl+C is pressed
- Set stdin=DEVNULL to prevent interactive commands from hanging
- Restore asyncio's signal handler after command completes

* fix(cli): Improve HITL approval UI and fix double rendering

- Add "User Approval Required:" header with cyan color
- Add visual separation for better readability
- Deduplicate action_requests to prevent double rendering
- Keep minimal arrow-based navigation

* fix(prompt): Tone down aggressive memory-first protocol

- Remove "at session start" automatic memory check
- Remove all-caps ALWAYS/FIRST emphasis
- Make memory checks contextual rather than mandatory
- Keep functionality but reduce eagerness

* fix(cli): Fix double rendering in approval menu cursor positioning

Remove conditional cursor movement logic that caused options to render twice.
Now consistently returns cursor to top of options after each render.

* fix(cli): Hide cursor during agent execution and approval menus

Hides blinking cursor during:
- Agent streaming and tool execution
- Approval menu navigation

Cursor is restored when returning to user input prompt.

* fix(memory): Remove aggressive memory-checking prompts

- Remove "CRITICAL - do this FIRST" and session-start triggers
- Remove "Memory-first response pattern" that forced checking before every question
- Remove all-caps "IMMEDIATELY" and "HIGH PRIORITY" language
- Make memory checks contextual rather than automatic
- Keep learning and update functionality but with gentler language

* fix(cli): Fix default prompt path in reset command

Updated path to correctly locate default_agent_prompt.md in src/deepagents/
instead of looking in libs/deepagents-cli/

* fix(cli): Move default_agent_prompt.md to correct location

Moved default_agent_prompt.md from src/deepagents/ to libs/deepagents-cli/
where the config.py expects to find it. Reverted the path hack.

* fix(cli): Remove extra blank line between messages

Removed duplicate console.print() that was creating excessive spacing
between agent response and next prompt.

* fix(cli): Fix terminal resize bug causing repeated prompts

Added prompt_continuation parameter to PromptSession to prevent the main
prompt (>) from being repeated on continuation lines during terminal resize.
Now continuation lines show two spaces instead of repeating >.

* fix(cli): Remove bottom toolbar to fix terminal resize bug

Removed bottom_toolbar from PromptSession which was causing prompts to
render on every line during terminal resize. The toolbar + multiline
combination was confusing prompt_toolkit's space calculation.

Changes:
- Removed bottom_toolbar and style parameters from PromptSession
- Updated Ctrl+T to show status message when toggled
- Auto-approve status still shown at startup in welcome message

* fix(cli): Add terminal state reset and reserve_space_for_menu=0

Additional fixes for terminal resize bug:
- Set prompt_continuation to empty string (no continuation prompts)
- Add reserve_space_for_menu=0 to prevent extra vertical space allocation
- Flush stdout/stderr and reset ANSI attributes before entering prompt loop
- Ensures clean terminal state for prompt_toolkit

* fix(cli): Restore bottom toolbar for auto-approve status

Brought back the bottom toolbar with colored status indicator:
- Green background when auto-approve is ON
- Orange background when manual approve (OFF)
- Updates automatically when toggling with Ctrl+T
- Kept other resize fixes (reserve_space_for_menu=0, empty continuation, state reset)

* add memory tests and fix unnecessary printing of warnings

* fix(cli): Add terminal state flush to prevent prompt desync

- Flush stdout/stderr after agent execution
- Reset ANSI attributes to clean state
- Prevents Rich console from corrupting terminal state

* fix(cli): Remove approval requirement for web_search tool

web_search is a read-only tool that fetches public information and should
not require user approval before execution. This change makes it behave
like http_request - executing automatically and showing results.

Previously, web_search would pause for approval, and sending a new message
would cancel the pending approval, creating a frustrating loop where users
could never get search results.

Changes:
- Removed web_search from interrupt_on configuration
- Deleted unused format_web_search_description function
- Added comment explaining why web_search doesn't require approval

* fix(cli): Show diffs in auto-approve mode for file operations

When auto-approve is enabled, file edits and writes now display their
diffs before executing, providing visibility into what changes are being
made automatically.

Previously, auto-approve mode would only show a brief message without
any diff preview, making it hard to see what the agent was changing.

Changes:
- Build approval preview in auto-approve flow
- Display diff using render_diff_block for file operations
- Show preview title and details before auto-approving
- Maintains same UX for non-file operations

* fix(cli): Reserve space for autocomplete menu

Calculate dynamic rows for menu visibility based on terminal height

* fix(cli): Flush console output before resuming in auto-approve

Prevents Rich console from interfering with agent resume

Author: vtrivedy

libs/deepagents-cli/deepagents_cli/agent.py

@@ -199,14 +199,6 @@ def format_edit_file_description(tool_call: dict) -> str:
             f"Snippet delta: {delta:+} characters"
         )
 
-    def format_web_search_description(tool_call: dict) -> str:
-        """Format web_search tool call for approval prompt."""
-        args = tool_call.get("args", {})
-        query = args.get("query", "unknown")
-        max_results = args.get("max_results", 5)
-
-        return f"Query: {query}\nMax results: {max_results}\n\n⚠️  This will use Tavily API credits"
-
     def format_task_description(tool_call: dict) -> str:
         """Format task (subagent) tool call for approval prompt."""
         args = tool_call.get("args", {})
@@ -248,11 +240,6 @@ def format_task_description(tool_call: dict) -> str:
         "description": lambda tool_call, state, runtime: format_edit_file_description(tool_call),
     }
 
-    web_search_interrupt_config: InterruptOnConfig = {
-        "allowed_decisions": ["approve", "reject"],
-        "description": lambda tool_call, state, runtime: format_web_search_description(tool_call),
-    }
-
     task_interrupt_config: InterruptOnConfig = {
         "allowed_decisions": ["approve", "reject"],
         "description": lambda tool_call, state, runtime: format_task_description(tool_call),
@@ -268,8 +255,9 @@ def format_task_description(tool_call: dict) -> str:
             "shell": shell_interrupt_config,
             "write_file": write_file_interrupt_config,
             "edit_file": edit_file_interrupt_config,
-            "web_search": web_search_interrupt_config,
             "task": task_interrupt_config,
+            # Note: web_search is NOT here - it's a read-only tool like http_request
+            # and should execute automatically without requiring approval
         },
     ).with_config(config)
 

libs/deepagents-cli/deepagents_cli/commands.py

@@ -1,5 +1,7 @@
 """Command handlers for slash commands and bash execution."""
 
+import os
+import signal
 import subprocess
 from pathlib import Path
 
@@ -57,33 +59,92 @@ def execute_bash_command(command: str) -> bool:
     if not cmd:
         return True
 
+    process = None
+    interrupted = False
+    original_handler = None
+
+    def sigint_handler(signum, frame):
+        """Custom SIGINT handler - kills subprocess immediately and sets flag."""
+        nonlocal interrupted, process
+        interrupted = True
+
+        # Kill subprocess immediately when Ctrl+C is pressed
+        if process and process.poll() is None:
+            try:
+                if hasattr(os, "killpg"):
+                    os.killpg(os.getpgid(process.pid), signal.SIGINT)
+                else:
+                    process.send_signal(signal.SIGINT)
+            except (ProcessLookupError, OSError):
+                pass
+
     try:
+        # Install our signal handler (temporarily overrides asyncio's handler)
+        original_handler = signal.signal(signal.SIGINT, sigint_handler)
+
         console.print()
         console.print(f"[dim]$ {cmd}[/dim]")
 
-        # Execute the command
-        result = subprocess.run(
-            cmd, check=False, shell=True, capture_output=True, text=True, timeout=30, cwd=Path.cwd()
+        # Create subprocess in new session (process group) to isolate from parent's signals
+        process = subprocess.Popen(
+            cmd,
+            shell=True,
+            text=True,
+            stdin=subprocess.DEVNULL,  # Close stdin to prevent interactive commands from hanging
+            stdout=subprocess.PIPE,
+            stderr=subprocess.PIPE,
+            start_new_session=True,  # Isolate subprocess in its own process group
+            cwd=Path.cwd(),
         )
 
-        # Display output
-        if result.stdout:
-            console.print(result.stdout, style=COLORS["dim"], markup=False)
-        if result.stderr:
-            console.print(result.stderr, style="red", markup=False)
-
-        # Show return code if non-zero
-        if result.returncode != 0:
-            console.print(f"[dim]Exit code: {result.returncode}[/dim]")
+        try:
+            # Wait for command to complete with timeout
+            stdout, stderr = process.communicate(timeout=30)
+
+            # Display output
+            if stdout:
+                console.print(stdout, style=COLORS["dim"], markup=False)
+            if stderr:
+                console.print(stderr, style="red", markup=False)
+
+            # Check if interrupted via our flag
+            if interrupted:
+                console.print("\n[yellow]Command interrupted by user[/yellow]\n")
+            elif process.returncode != 0:
+                # Exit code 130 = 128 + SIGINT (2) - command was interrupted
+                # Exit code -2 also indicates interrupt in some shells
+                if process.returncode == 130 or process.returncode == -2:
+                    console.print("[yellow]Command interrupted[/yellow]")
+                else:
+                    console.print(f"[dim]Exit code: {process.returncode}[/dim]")
+
+        except subprocess.TimeoutExpired:
+            # Timeout - kill the process group
+            if hasattr(os, "killpg"):
+                try:
+                    os.killpg(os.getpgid(process.pid), signal.SIGKILL)
+                except (ProcessLookupError, OSError):
+                    pass
+            else:
+                process.kill()
+
+            # Clean up zombie process
+            try:
+                process.wait(timeout=1)
+            except subprocess.TimeoutExpired:
+                pass
+
+            console.print("[red]Command timed out after 30 seconds[/red]")
 
         console.print()
         return True
 
-    except subprocess.TimeoutExpired:
-        console.print("[red]Command timed out after 30 seconds[/red]")
-        console.print()
-        return True
     except Exception as e:
         console.print(f"[red]Error executing command: {e}[/red]")
         console.print()
         return True
+
+    finally:
+        # CRITICAL: Always restore original signal handler so asyncio can handle Ctrl+C at prompt
+        if original_handler is not None:
+            signal.signal(signal.SIGINT, original_handler)

libs/deepagents-cli/deepagents_cli/config.py

@@ -48,7 +48,6 @@
 # Common bash commands for autocomplete
 COMMON_BASH_COMMANDS = {
     "ls": "List directory contents",
-    "ls -la": "List all files with details",
     "cd": "Change directory",
     "pwd": "Print working directory",
     "cat": "Display file contents",

libs/deepagents-cli/deepagents_cli/default_agent_prompt.md

@@ -3,16 +3,12 @@ You are an AI assistant that helps users with various tasks including coding, re
 # Core Role
 Your core role and behavior may be updated based on user feedback and instructions. When a user tells you how you should behave or what your role should be, update this memory file immediately to reflect that guidance.
 
-## Memory-First Protocol
-You have access to a persistent memory system. ALWAYS follow this protocol:
+## Memory System
+You have access to persistent memory in `/memories/`.
 
-**At session start:**
-- Check `ls /memories/` to see what knowledge you have stored
-- If your role description references specific topics, check /memories/ for relevant guides
-
-**Before answering questions:**
-- If asked "what do you know about X?" or "how do I do Y?" → Check `ls /memories/` FIRST
-- If relevant memory files exist → Read them and base your answer on saved knowledge
+**When answering questions:**
+- If asked "what do you know about X?" or similar questions about specific topics → Check `/memories/` for relevant knowledge
+- If you recall working on or saving information about the current topic → Read those memory files
 - Prefer saved knowledge over general knowledge when available
 
 **When learning new information:**