Preview deployment for your docs. Learn more about Mintlify Previews.

💡 Tip: Enable Workflows to automatically generate PRs for you.

Greptile Summary

This PR reorganizes flat app/cli/ and app/deployment/ modules into focused subpackages (interactive_shell/, support/, investigation/, methods/, operations/) and renames the REPL to interactive_shell. It also introduces two new modules — cli_agent.py (a LangGraph-free terminal assistant) and cli_help.py (reference-grounded CLI Q&A) — along with a richer input classifier in router.py. All 97 files have been updated with no stale app.cli.repl imports remaining in the codebase. The migration is clean and backward-compatible through the re-exporting __init__.py files.

Confidence Score: 5/5

Safe to merge — refactoring is complete and all import paths are correctly updated with no broken references remaining.

All 97 files are updated consistently; no stale app.cli.repl imports remain anywhere in the codebase; backward-compatible re-exporting init.py files ensure downstream callers are unaffected. The three remaining findings are all P2 style/naming suggestions that do not affect runtime behavior.

No files require special attention. Minor terminology cleanup in router.py docstring and loop.py public symbol name would be nice-to-have follow-ups.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    Input["User input (text)"] --> R{router.classify_input}

    R -->|starts with /| Slash["dispatch_slash()\napp/cli/interactive_shell/commands.py"]
    R -->|bare alias e.g. help/exit| Slash
    R -->|CLI how-to pattern| CLIHelp["answer_cli_help()\napp/cli/interactive_shell/cli_help.py"]
    R -->|no prior state + incident vocab| NewAlert["_run_new_alert()\nLangGraph investigation pipeline"]
    R -->|short question + prior state| FollowUp["answer_follow_up()\napp/cli/interactive_shell/follow_up.py"]
    R -->|long/alert text + prior state| NewAlert
    R -->|chat / general| CLIAgent["answer_cli_agent()\napp/cli/interactive_shell/cli_agent.py"]

    CLIHelp -->|grounding=reference_only| CLIAgent
    CLIAgent --> LLM["get_llm_for_reasoning()\napp/services/llm_client.py"]
    NewAlert --> LangGraph["LangGraph pipeline\napp/cli/investigation/"]

    subgraph new_subpackages ["New subpackages (this PR)"]
        direction LR
        IS["app/cli/interactive_shell/\nloop · router · cli_agent\ncli_help · commands · session"]
        INV["app/cli/investigation/\ninvestigate · payload\nalert_templates"]
        SUP["app/cli/support/\nargs · context · errors\nexit_codes · health_view · update"]
        DM["app/deployment/methods/\nrailway · langsmith"]
        DO["app/deployment/operations/\nec2_config · health · provider_config"]
    end

_{Reviews (1): Last reviewed commit: "refactor(cli): rename REPL to interactiv..." | Re-trigger Greptile}

Hey @davincios, really impressive refactor — the separation into app/cli/interactive_shell/, app/cli/investigation/, and app/cli/support/ is a big readability win. The old flat layout made it hard to tell which module owned what, so grouping by concern (shell loop, investigation pipeline, shared support) makes the structure self-documenting.

A couple of small observations while reading through:

1. router.py in interactive_shell gained two new input kinds: cli_help and cli_agent on top of the original slash / new_alert / follow_up. The old app/cli/repl/router.py has been deleted. Just worth confirming that any external code that imported from app.cli.repl.router import InputKind or classify_input has been updated — the app/cli/repl/__init__.py compat shim might need a note or a deprecation warning for anyone importing directly from the old path.

2. In cli_agent.py the _MAX_CLI_AGENT_TURNS = 12 cap stores 2 entries per turn (user + assistant), so the actual history is 6 turns of back-and-forth. The comment already says this — just wondering if a docstring clarification like "stores up to 12 messages (6 user/assistant pairs)" would help future readers.

3. The _TERMINOLOGY_RULE and _MARKDOWN_RULE constants in cli_agent.py are strings that get appended to every system prompt — nice approach for enforcing consistency. Are these also applied in answer_cli_help, or only in answer_cli_agent? If they differ, it might be worth noting why.

Really clean work — the direction is clearly right. 🙏

really impressive refactor — the separation into app/cli/interactive_shell/, app/cli/investigation/, and app/cli/support/ is a big readability win. The old flat layout made it hard to tell which module owned what, so grouping by con

Thanks @Ghraven that is very kind of you :)

😤 @davincios said "I will fix this" and then actually fixed it. Legendary behavior.

👋 Join us on Discord - OpenSRE : hang out, contribute, or hunt for features and issues. Everyone's welcome.