Greptile code review

This repo uses Greptile for automated review. Before merge, aim for Confidence Score: 5/5 with zero unresolved review threads — see CONTRIBUTING.md.

Run a review — add a PR comment with:

@greptile review

Give it ~5-10 minutes (sometimes longer) for results, then fix feedback and re-trigger until you reach Confidence Score: 5/5.

Optional: automate with the greploop skill.

Greptile Summary

This PR adds a session-resume hint to all CLI exit paths (ctrl+c double-press, /exit, /quit, and ctrl+d), showing the user Resume this session with: /resume <session_id> before the farewell message.

• system.py: /exit and /quit now print the resume hint (guarded by if session.session_id:) before printing goodbye.
• prompt_support.py: repl_prompt_note_ctrl_c gains a session_id parameter and prints the resume hint before Goodbye! on the second ctrl+c; the guard around the resume lines is missing, so a None session_id will render /resume None on screen
• loop.py: The ctrl+d (EOFError) block now prints the resume hint and Goodbye!, but does so with unstyled plain strings rather than the [{DIM}] Rich markup used by every other exit path

Confidence Score: 4/5

Safe to merge once the missing session_id guard in the ctrl+c double-press path is added; without it, the terminal outputs the literal text /resume None when no session is active.

The ctrl+c exit path in prompt_support.py prints the resume line unconditionally — if session_id is None, the user sees /resume None on screen, which is misleading. Every other changed exit path wraps the resume output in a session_id check, so this one path is out of step with the rest of the change.

app/cli/support/prompt_support.py — the repl_prompt_note_ctrl_c function needs a if session_id: guard around the resume lines (lines 197-198)

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[User triggers exit] --> B{Exit path}
    B -->|/exit or /quit| C[system.py: _cmd_exit]
    B -->|Ctrl+C x 2| D[prompt_support.py: repl_prompt_note_ctrl_c]
    B -->|Ctrl+D / EOFError| E[loop.py: EOFError handler]

    C --> F{session.session_id?}
    F -->|Yes| G[print resume hint - DIM styled]
    F -->|No| H[skip resume hint]
    G --> I[print goodbye. - DIM styled]
    H --> I

    D --> J{last ctrl+c within 2s?}
    J -->|Yes| K[print resume hint - no session_id guard!]
    K --> L[print Goodbye! - DIM styled]
    J -->|No| M[print Press Ctrl+C again to exit]

    E --> N{session.session_id?}
    N -->|Yes| O[print resume hint + Goodbye! - plain text no DIM]
    N -->|No| P[silent exit]

_{Reviews (2): Last reviewed commit: "Update app/cli/support/prompt_support.py" | Re-trigger Greptile}

🎯 Bullseye. @larsspinetta12 opened a PR, kept the vibes clean, and got it merged. Absolute cinema. 🎬

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

@greptile review