fix: Add feedback after calling todo_list commands

Updates:
- The model now sees the updated todo list after invoking the todo_list tool
- Todo lists shouldn't get stuck on marking a task as complete (in theory)

Author: kiran-garre

crates/chat-cli/src/cli/chat/tools/todo.rs

@@ -1,5 +1,5 @@
-use std::io::Write;
 use std::collections::HashSet;
+use std::io::Write;
 use std::time::{
     SystemTime,
     UNIX_EPOCH,
@@ -21,13 +21,6 @@ use serde::{
 use super::InvokeOutput;
 use crate::os::Os;
 
-// Demo prompts:
-// Create a Python package layout with a blank main file and a blank utilities file. Start by making
-// a todo list. Design your own super simple programming task with 4 steps. Make a todo list for the
-// task, and begin executing those steps. Implement a basic input to Python type converter where the
-// user can input either a string or integer and it gets converted to the corresponding Python
-// object.
-
 #[derive(Debug, Clone, Deserialize)]
 #[serde(tag = "command")]
 pub enum TodoInput {
@@ -58,7 +51,7 @@ pub enum TodoInput {
     Remove {
         remove_indices: Vec<usize>,
         new_description: Option<String>,
-    }
+    },
 }
 
 #[derive(Debug, Default, Serialize, Deserialize, Clone)]
@@ -188,7 +181,11 @@ impl TodoInput {
                 TodoState::set_current_todo_id(os, id)?;
                 TodoState::load(os, id)?
             },
-            TodoInput::Add { new_tasks, insert_indices, new_description } => {
+            TodoInput::Add {
+                new_tasks,
+                insert_indices,
+                new_description,
+            } => {
                 let current_id = match TodoState::get_current_todo_id(os)? {
                     Some(id) => id,
                     None => bail!("No to-do list currently loaded"),
@@ -204,7 +201,10 @@ impl TodoInput {
                 state.save(os, &current_id)?;
                 state
             },
-            TodoInput::Remove { remove_indices, new_description } => {
+            TodoInput::Remove {
+                remove_indices,
+                new_description,
+            } => {
                 let current_id = match TodoState::get_current_todo_id(os)? {
                     Some(id) => id,
                     None => bail!("No to-do list currently loaded"),
@@ -219,12 +219,12 @@ impl TodoInput {
                 }
                 state.save(os, &current_id)?;
                 state
-            }
+            },
         };
         state.display_list(output)?;
         output.flush()?;
 
-        Ok(Default::default())
+        Ok(InvokeOutput { output: super::OutputKind::Json(serde_json::to_value(state)?) })
     }
 
     pub fn queue_description(&self, _os: &Os, _output: &mut impl Write) -> Result<()> {
@@ -267,7 +267,11 @@ impl TodoInput {
                     bail!("Loaded todo list is empty");
                 }
             },
-            TodoInput::Add { new_tasks, insert_indices, new_description } => {
+            TodoInput::Add {
+                new_tasks,
+                insert_indices,
+                new_description,
+            } => {
                 let current_id = match TodoState::get_current_todo_id(os)? {
                     Some(id) => id,
                     None => bail!("No todo list is currently loaded"),
@@ -285,7 +289,10 @@ impl TodoInput {
                     bail!("New description cannot be empty");
                 }
             },
-            TodoInput::Remove { remove_indices, new_description } => {
+            TodoInput::Remove {
+                remove_indices,
+                new_description,
+            } => {
                 let current_id = match TodoState::get_current_todo_id(os)? {
                     Some(id) => id,
                     None => bail!("No todo list is currently loaded"),
@@ -298,18 +305,17 @@ impl TodoInput {
                 } else if new_description.is_some() && new_description.as_ref().unwrap().trim().is_empty() {
                     bail!("New description cannot be empty");
                 }
-            }
+            },
         }
         Ok(())
     }
 }
 
-
 /// Generated by Q
-fn has_duplicates<T>(vec: &[T]) -> bool 
+fn has_duplicates<T>(vec: &[T]) -> bool
 where
     T: std::hash::Hash + Eq,
 {
     let mut seen = HashSet::with_capacity(vec.len());
     vec.iter().any(|item| !seen.insert(item))
-}
+}