refactor: slash command rewrite (#223)

* checkpoint

* checkpoint 2

* checkpoint 3

* checkpoint 4

* checkpoint 5

* checkpoint 6

Author: chaynabors

crates/chat-cli/src/api_client/clients/streaming_client.rs

@@ -96,16 +96,13 @@ impl StreamingClient {
         Ok(Self { inner, profile })
     }
 
-    pub async fn send_message(
-        &self,
-        conversation_state: ConversationState,
-    ) -> Result<SendMessageOutput, ApiClientError> {
-        debug!("Sending conversation: {:#?}", conversation_state);
+    pub async fn send_message(&self, conversation: ConversationState) -> Result<SendMessageOutput, ApiClientError> {
+        debug!("Sending conversation: {:#?}", conversation);
         let ConversationState {
             conversation_id,
             user_input_message,
             history,
-        } = conversation_state;
+        } = conversation;
 
         match &self.inner {
             inner::Inner::Codewhisperer(client) => {
@@ -124,7 +121,8 @@ impl StreamingClient {
                             .transpose()?,
                     )
                     .build()
-                    .expect("building conversation_state should not fail");
+                    .expect("building conversation should not fail");
+
                 let response = client
                     .generate_assistant_response()
                     .conversation_state(conversation_state)

crates/chat-cli/src/cli/chat/cli/clear.rs

@@ -0,0 +1,65 @@
+use clap::Args;
+use crossterm::style::{
+    self,
+    Color,
+    Stylize,
+};
+use crossterm::{
+    cursor,
+    execute,
+};
+
+use crate::cli::chat::{
+    ChatError,
+    ChatSession,
+    ChatState,
+};
+
+#[deny(missing_docs)]
+#[derive(Debug, PartialEq, Args)]
+pub struct ClearArgs;
+
+impl ClearArgs {
+    pub async fn execute(self, session: &mut ChatSession) -> Result<ChatState, ChatError> {
+        execute!(
+            session.output,
+            style::SetForegroundColor(Color::DarkGrey),
+            style::Print(
+                "\nAre you sure? This will erase the conversation history and context from hooks for the current session. "
+            ),
+            style::Print("["),
+            style::SetForegroundColor(Color::Green),
+            style::Print("y"),
+            style::SetForegroundColor(Color::DarkGrey),
+            style::Print("/"),
+            style::SetForegroundColor(Color::Green),
+            style::Print("n"),
+            style::SetForegroundColor(Color::DarkGrey),
+            style::Print("]:\n\n"),
+            style::SetForegroundColor(Color::Reset),
+            cursor::Show,
+        )?;
+
+        // Setting `exit_on_single_ctrl_c` for better ux: exit the confirmation dialog rather than the CLI
+        let user_input = match session.read_user_input("> ".yellow().to_string().as_str(), true) {
+            Some(input) => input,
+            None => "".to_string(),
+        };
+
+        if ["y", "Y"].contains(&user_input.as_str()) {
+            session.conversation.clear(true);
+            if let Some(cm) = session.conversation.context_manager.as_mut() {
+                cm.hook_executor.global_cache.clear();
+                cm.hook_executor.profile_cache.clear();
+            }
+            execute!(
+                session.output,
+                style::SetForegroundColor(Color::Green),
+                style::Print("\nConversation history cleared.\n\n"),
+                style::SetForegroundColor(Color::Reset)
+            )?;
+        }
+
+        Ok(ChatState::default())
+    }
+}

crates/chat-cli/src/cli/chat/cli/compact.rs

@@ -0,0 +1,49 @@
+use clap::Args;
+
+use crate::cli::chat::{
+    ChatError,
+    ChatSession,
+    ChatState,
+};
+use crate::database::Database;
+use crate::platform::Context;
+use crate::telemetry::TelemetryThread;
+
+#[deny(missing_docs)]
+#[derive(Debug, PartialEq, Args)]
+#[command(
+    before_long_help = "/compact summarizes the conversation history to free up context space
+while preserving essential information. This is useful for long-running conversations
+that may eventually reach memory constraints.
+
+When to use
+• When you see the memory constraint warning message
+• When a conversation has been running for a long time
+• Before starting a new topic within the same session
+• After completing complex tool operations
+
+How it works
+• Creates an AI-generated summary of your conversation
+• Retains key information, code, and tool executions in the summary
+• Clears the conversation history to free up space
+• The assistant will reference the summary context in future responses"
+)]
+pub struct CompactArgs {
+    /// The prompt to use when generating the summary
+    prompt: Option<String>,
+    show_summary: bool,
+}
+
+impl CompactArgs {
+    pub async fn execute(
+        self,
+        ctx: &Context,
+        database: &mut Database,
+        telemetry: &TelemetryThread,
+        session: &mut ChatSession,
+    ) -> Result<ChatState, ChatError> {
+        session
+            .compact_history(ctx, database, telemetry, self.prompt, self.show_summary)
+            .await
+    }
+}