refactor(repl): remove /list command and adopt per-domain list subcommands (Tracer-Cloud#2713)

* refactor(repl): remove /list command and adopt per-domain list subcommands

* resolving the greptile feed back

* refactor(command_registry): remove 'list' subcommands and associated completions

* chore(session_cmds): add a blank line for improved readability

* refactor(command_registry): implement a new root handler for the /tools command

* refactor(planner): remove informational question detection and simplify fallback logic

* refactor(command_registry): update references to slash_invoke tool description in documentation and tests

* refactor(planner): streamline context length overflow handling and improve error classification

* refactor(failure_explain): add blank lines for improved readability and maintainability

Author: Devesh36

app/cli/interactive_shell/chat/cli_agent.py

@@ -125,7 +125,7 @@ def _command_selection_response() -> str:
     '`{"action":"switch_toolcall_model","model":"claude-opus-4-7"}` '
     "to change ONLY the toolcall model on the currently active provider; "
     '`{"action":"slash","command":"/model show"}` where command is one of '
-    "/model show, /list models, /health, /doctor, /version; "
+    "/model show, /health, /doctor, /version; "
     '`{"action":"run_cli_command","args":"<subcommand> <flags>"}` '
     "to run any opensre subcommand (agent is blocked). For ordinary "
     "questions, return normal Markdown. Do not return action JSON for vague "
@@ -136,7 +136,6 @@ def _command_selection_response() -> str:
 _ALLOWED_SLASH_ACTIONS = frozenset(
     {
         "/model show",
-        "/list models",
         "/health",
         "/doctor",
         "/version",

app/cli/interactive_shell/command_registry/__init__.py

@@ -37,6 +37,7 @@
 from app.cli.interactive_shell.command_registry.suggestions import closest_choice
 from app.cli.interactive_shell.command_registry.system import COMMANDS as SYSTEM_COMMANDS
 from app.cli.interactive_shell.command_registry.tasks_cmds import COMMANDS as TASK_COMMANDS
+from app.cli.interactive_shell.command_registry.tools_cmds import COMMANDS as TOOLS_COMMANDS
 from app.cli.interactive_shell.command_registry.types import SlashCommand
 from app.cli.interactive_shell.command_registry.watch_cmds import COMMANDS as WATCH_COMMANDS
 from app.cli.interactive_shell.routing.handle_message_with_agent.orchestration.execution_policy import (
@@ -53,6 +54,7 @@
         SESSION_COMMANDS,
         INTEGRATIONS_COMMANDS,
         MODEL_COMMANDS,
+        TOOLS_COMMANDS,
         INVESTIGATION_COMMANDS,
         TASK_COMMANDS,
         WATCH_COMMANDS,

app/cli/interactive_shell/command_registry/agents.py

@@ -653,7 +653,6 @@ def _cmd_agents(session: ReplSession, console: Console, args: list[str]) -> bool
         return _cmd_agents_list(console)
 
     sub = args[0].lower().strip()
-
     if sub == "budget":
         return _cmd_agents_budget(session, console, args[1:])
     if sub == "bus":

app/cli/interactive_shell/command_registry/help.py

@@ -50,13 +50,14 @@ def _raw_help_sections() -> list[HelpSection]:
     from app.cli.interactive_shell.command_registry.session_cmds import COMMANDS as SESSION_CMDS
     from app.cli.interactive_shell.command_registry.system import COMMANDS as SYS_CMDS
     from app.cli.interactive_shell.command_registry.tasks_cmds import COMMANDS as TASK_CMDS
+    from app.cli.interactive_shell.command_registry.tools_cmds import COMMANDS as TOOLS_CMDS
     from app.cli.interactive_shell.command_registry.watch_cmds import COMMANDS as WATCH_CMDS
 
     return [
         _quick_access_section(),
         ("Help", list(COMMANDS)),
         ("Session", list(SESSION_CMDS)),
-        ("Integrations & Models", list(INT_CMDS) + list(MODEL_CMDS)),
+        ("Integrations, Models & Tools", list(INT_CMDS) + list(MODEL_CMDS) + list(TOOLS_CMDS)),
         ("Investigation", list(INV_CMDS)),
         ("Privacy", list(PRIVACY_CMDS)),
         ("Tasks", list(TASK_CMDS) + list(WATCH_CMDS)),

app/cli/interactive_shell/command_registry/integrations.py

@@ -1,4 +1,4 @@
-"""Slash commands for /list, /integrations, and /mcp."""
+"""Slash commands for /integrations and /mcp."""
 
 from __future__ import annotations
 
@@ -8,7 +8,6 @@
 from app.cli.interactive_shell.command_registry import repl_data
 from app.cli.interactive_shell.command_registry.cli_parity import run_cli_command
 from app.cli.interactive_shell.command_registry.types import ExecutionTier, SlashCommand
-from app.cli.interactive_shell.config.tool_catalog import build_tool_catalog
 from app.cli.interactive_shell.runtime import ReplSession
 from app.cli.interactive_shell.ui import (
     BOLD_BRAND,
@@ -19,8 +18,6 @@
     WARNING,
     render_integrations_table,
     render_mcp_table,
-    render_models_table,
-    render_tools_table,
     repl_table,
 )
 from app.cli.interactive_shell.ui.choice_menu import (
@@ -36,7 +33,6 @@
     repl_print,
 )
 
-_ROOT_LIST = "/list"
 _ROOT_INTEGRATIONS = "/integrations"
 _ROOT_MCP = "/mcp"
 
@@ -89,40 +85,6 @@ def _render_integration_show(console: Console, service: str) -> bool:
     return True
 
 
-def _interactive_list_menu(_session: ReplSession, console: Console) -> bool:
-    while True:
-        sub = repl_choose_one(
-            title="list",
-            breadcrumb=_ROOT_LIST,
-            choices=[
-                ("integrations", "/list integrations"),
-                ("models", "/list models"),
-                ("mcp", "/list mcp"),
-                ("tools", "/list tools"),
-                ("all", "all"),
-                ("done", "done"),
-            ],
-        )
-        if sub is None or sub == "done":
-            return True
-        if sub == "integrations":
-            results = repl_data.load_verified_integrations()
-            render_integrations_table(console, results)
-        elif sub == "mcp":
-            results = repl_data.load_verified_integrations()
-            render_mcp_table(console, results)
-        elif sub == "models":
-            render_models_table(console, repl_data.load_llm_settings())
-        elif sub == "all":
-            results = repl_data.load_verified_integrations()
-            render_integrations_table(console, results)
-            render_mcp_table(console, results)
-            render_models_table(console, repl_data.load_llm_settings())
-        elif sub == "tools":
-            render_tools_table(console, build_tool_catalog())
-        repl_section_break(console)
-
-
 def _cmd_integrations(session: ReplSession, console: Console, args: list[str]) -> bool:
     if not args and repl_tty_interactive():
         return _interactive_integrations_menu(session, console)
@@ -294,50 +256,6 @@ def _interactive_mcp_menu(session: ReplSession, console: Console) -> bool:
             repl_section_break(console)
 
 
-def _cmd_list(session: ReplSession, console: Console, args: list[str]) -> bool:
-    if not args and repl_tty_interactive():
-        return _interactive_list_menu(session, console)
-
-    sub = (args[0].lower() if args else "").strip()
-
-    if sub in ("integrations", "integration", "int"):
-        render_integrations_table(console, repl_data.load_verified_integrations())
-        return True
-
-    if sub in ("mcp", "mcps"):
-        render_mcp_table(console, repl_data.load_verified_integrations())
-        return True
-
-    if sub in ("models", "model", "llm", "llms"):
-        render_models_table(console, repl_data.load_llm_settings())
-        return True
-
-    if sub in ("tools", "tool"):
-        render_tools_table(console, build_tool_catalog())
-        return True
-
-    if sub and sub not in ("", "all"):
-        console.print(
-            f"[{ERROR}]unknown list target:[/] {escape(sub)}  "
-            "(try [bold]/list integrations[/bold], [bold]/list models[/bold], "
-            "[bold]/list mcp[/bold], or [bold]/list tools[/bold])"
-        )
-        return True
-
-    results = repl_data.load_verified_integrations()
-    render_integrations_table(console, results)
-    render_mcp_table(console, results)
-    render_models_table(console, repl_data.load_llm_settings())
-    return True
-
-
-_LIST_FIRST_ARGS: tuple[tuple[str, str], ...] = (
-    ("integrations", "alert-source integrations"),
-    ("models", "active LLM models"),
-    ("mcp", "connected MCP servers"),
-    ("tools", "registered tools (investigation + chat surfaces)"),
-)
-
 _INTEGRATIONS_FIRST_ARGS: tuple[tuple[str, str], ...] = (
     ("list", "list all configured integrations"),
     ("verify", "run health checks on all integrations"),
@@ -351,15 +269,6 @@ def _cmd_list(session: ReplSession, console: Console, args: list[str]) -> bool:
 )
 
 COMMANDS: list[SlashCommand] = [
-    SlashCommand(
-        "/list",
-        "Browse integrations, MCP servers, models, and tools.",
-        _cmd_list,
-        usage=("/list", "/list integrations", "/list models", "/list mcp", "/list tools"),
-        notes=("In a TTY, bare /list opens an interactive menu.",),
-        first_arg_completions=_LIST_FIRST_ARGS,
-        execution_tier=ExecutionTier.SAFE,
-    ),
     SlashCommand(
         "/integrations",
         "Manage integrations.",

app/cli/interactive_shell/command_registry/session_cmds.py

@@ -10,7 +10,10 @@
 from rich.markup import escape
 
 import app.cli.interactive_shell.command_registry.repl_data as repl_data
-from app.cli.interactive_shell.command_registry.types import ExecutionTier, SlashCommand
+from app.cli.interactive_shell.command_registry.types import (
+    ExecutionTier,
+    SlashCommand,
+)
 from app.cli.interactive_shell.runtime import ReplSession
 from app.cli.interactive_shell.ui import (
     BOLD_BRAND,
@@ -658,7 +661,12 @@ def _cmd_resume(session: ReplSession, console: Console, args: list[str]) -> bool
         first_arg_completions=_VERBOSE_FIRST_ARGS,
     ),
     SlashCommand("/compact", "Trim old session history to free memory.", _cmd_compact),
-    SlashCommand("/sessions", "List recent REPL sessions.", _cmd_sessions),
+    SlashCommand(
+        "/sessions",
+        "List recent REPL sessions.",
+        _cmd_sessions,
+        usage=("/sessions",),
+    ),
     SlashCommand(
         "/resume",
         "Resume a previous session by restoring its conversation context.",

app/cli/interactive_shell/command_registry/slash_catalog.py

@@ -15,7 +15,7 @@
     string_property,
 )
 
-_MAX_COMPACT_DESC_CHARS = 180
+_MAX_COMPACT_DESC_CHARS = 120
 
 
 @dataclass(frozen=True)
@@ -125,7 +125,7 @@ def _mcp(
         "User asks if OpenSRE is healthy, working, or connected",
         anti_examples=(
             "User asks what integrations OpenSRE supports in general (docs → assistant_handoff)",
-            "User asks to list connected integrations (use /list integrations)",
+            "User asks to list connected integrations (use /integrations list)",
         ),
     ),
     "/help": _mcp(
@@ -147,7 +147,7 @@ def _mcp(
         "User asks to show details for a configured integration",
         anti_examples=(
             "User asks which integrations OpenSRE supports without configuring (assistant_handoff)",
-            "User asks to list connected integrations (prefer /list integrations)",
+            "User asks to list connected integrations (prefer /integrations list)",
         ),
     ),
     "/investigate": _mcp(
@@ -163,14 +163,6 @@ def _mcp(
         "Reprint the most recent investigation report from this session.",
         "User asks to show the last investigation result or report again",
     ),
-    "/list": _mcp(
-        "Browse connected integrations, MCP servers, active LLM models, or registered tools.",
-        "User asks to show or list connected/configured integrations or services",
-        "User asks to list MCP servers (/list mcp)",
-        anti_examples=(
-            "User asks what integrations OpenSRE supports in general (assistant_handoff)",
-        ),
-    ),
     "/mcp": _mcp(
         "Manage connected MCP servers. Subcommands: list, connect, disconnect.",
         "User asks to list, connect, or disconnect MCP servers",
@@ -259,6 +251,11 @@ def _mcp(
         "Print a starter alert JSON template (generic, datadog, grafana, honeycomb, coralogix, splunk).",
         "User asks for an alert template or example payload format",
     ),
+    "/tools": _mcp(
+        "List registered investigation/chat tools wired into this OpenSRE build.",
+        "User asks what tools the REPL can use",
+        "User asks to list investigation or chat tools",
+    ),
     "/tests": _mcp(
         "Browse and run inventoried tests from the terminal. Subcommands: list, run, synthetic.",
         "User asks to list or run bundled tests via /tests",
@@ -383,7 +380,7 @@ def format_slash_catalog_text(
         if compact and len(desc) > _MAX_COMPACT_DESC_CHARS:
             desc = desc[: _MAX_COMPACT_DESC_CHARS - 1].rstrip() + "…"
         lines.append(f"- **{spec.name}** — {desc}")
-        if spec.use_cases:
+        if spec.use_cases and not compact:
             lines.append(f"  - use when: {spec.use_cases[0]}")
         if spec.anti_examples and not compact:
             lines.append(f"  - not for: {spec.anti_examples[0]}")
@@ -399,7 +396,10 @@ def slash_invoke_tool_description(specs: list[SlashCommandSpec] | None = None) -
         "Pick the command whose use-case best matches the user request, then supply "
         "positional args in the args array."
     )
-    body = format_slash_catalog_text(entries, compact=True)
+    # Keep planner payload intentionally tiny for live LLM runs with strict
+    # prompt budgets. The full rich catalog remains available via
+    # format_slash_catalog_text(..., compact=False).
+    body = "\n".join(f"- `{spec.name}`" for spec in entries)
     return f"{header}\n\n{body}"
 
 
@@ -410,8 +410,8 @@ def slash_invoke_input_schema(
     command_names = tuple(spec.name for spec in entries)
     args_description = (
         "Positional arguments after the command name. Valid values depend on the "
-        "chosen command — see the tool description catalog. Examples: "
-        '["integrations"] for /list, ["verify", "datadog"] for /integrations.'
+        "chosen command — see the slash_invoke tool description. Examples: "
+        '["list"] for /tools, ["verify", "datadog"] for /integrations.'
     )
     return object_schema(
         properties={

app/cli/interactive_shell/command_registry/tasks_cmds.py

@@ -7,7 +7,10 @@
 from rich.console import Console
 from rich.markup import escape
 
-from app.cli.interactive_shell.command_registry.types import ExecutionTier, SlashCommand
+from app.cli.interactive_shell.command_registry.types import (
+    ExecutionTier,
+    SlashCommand,
+)
 from app.cli.interactive_shell.history import load_command_history_entries
 from app.cli.interactive_shell.runtime import ReplSession, TaskKind, TaskRecord, TaskStatus
 from app.cli.interactive_shell.ui import (
@@ -217,7 +220,12 @@ def _cmd_cancel(session: ReplSession, console: Console, args: list[str]) -> bool
 
 COMMANDS: list[SlashCommand] = [
     SlashCommand("/history", "Show persisted command history.", _cmd_history),
-    SlashCommand("/tasks", "List recent and in-flight shell tasks.", _cmd_tasks),
+    SlashCommand(
+        "/tasks",
+        "List recent and in-flight shell tasks.",
+        _cmd_tasks,
+        usage=("/tasks",),
+    ),
     SlashCommand(
         "/cancel",
         "Cancel a running task by id.",

app/cli/interactive_shell/command_registry/tools_cmds.py

@@ -0,0 +1,43 @@
+"""Slash command /tools."""
+
+from __future__ import annotations
+
+from rich.console import Console
+
+from app.cli.interactive_shell.command_registry.types import (
+    ExecutionTier,
+    SlashCommand,
+    make_list_root_handler,
+)
+from app.cli.interactive_shell.config.tool_catalog import build_tool_catalog
+from app.cli.interactive_shell.runtime import ReplSession
+from app.cli.interactive_shell.ui import render_tools_table
+
+
+def _list_tools(_session: ReplSession, console: Console, _args: list[str]) -> bool:
+    render_tools_table(console, build_tool_catalog())
+    return True
+
+
+_cmd_tools = make_list_root_handler(
+    "/tools",
+    _list_tools,
+    list_aliases=("list", "ls", "tool", "tools"),
+)
+
+_TOOLS_FIRST_ARGS: tuple[tuple[str, str], ...] = (
+    ("list", "list registered tools (investigation + chat surfaces)"),
+)
+
+COMMANDS: list[SlashCommand] = [
+    SlashCommand(
+        "/tools",
+        "List registered tools.",
+        _cmd_tools,
+        usage=("/tools", "/tools list"),
+        first_arg_completions=_TOOLS_FIRST_ARGS,
+        execution_tier=ExecutionTier.SAFE,
+    )
+]
+
+__all__ = ["COMMANDS", "_TOOLS_FIRST_ARGS", "_cmd_tools"]