fix: Make todo lists an experimental feature

Author: kiran-garre

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

@@ -44,6 +44,11 @@ static AVAILABLE_EXPERIMENTS: &[Experiment] = &[
         description: "Enables entering into a temporary mode for sending isolated conversations (/tangent)",
         setting_key: Setting::EnabledTangentMode,
     },
+    Experiment {
+        name: "Todo Lists",
+        description: "Enables Q to create todo lists that can be viewed and managed using /todos",
+        setting_key: Setting::EnabledTodoLists,
+    },
 ];
 
 #[derive(Debug, PartialEq, Args)]

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

@@ -17,6 +17,7 @@ use crate::cli::chat::{
     ChatSession,
     ChatState,
 };
+use crate::database::settings::Setting;
 use crate::os::Os;
 
 /// Defines subcommands that allow users to view and manage todo lists
@@ -65,6 +66,23 @@ impl std::fmt::Display for TodoDisplayEntry {
 
 impl TodoSubcommand {
     pub async fn execute(self, os: &mut Os, session: &mut ChatSession) -> Result<ChatState, ChatError> {
+        // Check if todo lists are enabled
+        if !os
+            .database
+            .settings
+            .get_bool(Setting::EnabledTodoLists)
+            .unwrap_or(false)
+        {
+            execute!(
+                session.stderr,
+                style::SetForegroundColor(style::Color::Red),
+                style::Print("Todo lists are disabled. Enable them with: q settings chat.enableTodoLists true\n"),
+                style::SetForegroundColor(style::Color::Reset)
+            )?;
+            return Ok(ChatState::PromptUser {
+                skip_printing_tools: true,
+            });
+        }
         TodoListState::init_dir(os)
             .await
             .map_err(|e| ChatError::Custom(format!("Could not create todos directory: {e}").into()))?;

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

@@ -24,6 +24,7 @@ use serde::{
 };
 
 use super::InvokeOutput;
+use crate::database::settings::Setting;
 use crate::os::Os;
 
 #[derive(Debug, Default, Serialize, Deserialize, Clone)]
@@ -214,6 +215,23 @@ pub enum TodoList {
 
 impl TodoList {
     pub async fn invoke(&self, os: &Os, output: &mut impl Write) -> Result<InvokeOutput> {
+        // Check if todo lists are enabled
+        if !os
+            .database
+            .settings
+            .get_bool(Setting::EnabledTodoLists)
+            .unwrap_or(false)
+        {
+            queue!(
+                output,
+                style::SetForegroundColor(style::Color::Red),
+                style::Print("Todo lists are disabled. Enable them with: q settings chat.enableTodoLists true"),
+                style::SetForegroundColor(style::Color::Reset)
+            )?;
+            return Ok(InvokeOutput {
+                output: super::OutputKind::Text("Todo lists are disabled.".to_string()),
+            });
+        }
         if let Some(id) = self.get_id() {
             if !os.fs.exists(id_to_path(os, &id)?) {
                 let error_string = "No todo list exists with the given ID";

crates/chat-cli/src/database/settings.rs

@@ -75,6 +75,8 @@ pub enum Setting {
     ChatDisableAutoCompaction,
     #[strum(message = "Show conversation history hints (boolean)")]
     ChatEnableHistoryHints,
+    #[strum(message = "Enable the todo list feature (boolean)")]
+    EnabledTodoLists,
 }
 
 impl AsRef<str> for Setting {
@@ -109,6 +111,7 @@ impl AsRef<str> for Setting {
             Self::ChatDefaultAgent => "chat.defaultAgent",
             Self::ChatDisableAutoCompaction => "chat.disableAutoCompaction",
             Self::ChatEnableHistoryHints => "chat.enableHistoryHints",
+            Self::EnabledTodoLists => "chat.enableTodoLists",
         }
     }
 }
@@ -153,6 +156,7 @@ impl TryFrom<&str> for Setting {
             "chat.defaultAgent" => Ok(Self::ChatDefaultAgent),
             "chat.disableAutoCompaction" => Ok(Self::ChatDisableAutoCompaction),
             "chat.enableHistoryHints" => Ok(Self::ChatEnableHistoryHints),
+            "chat.enableTodoLists" => Ok(Self::EnabledTodoLists),
             _ => Err(DatabaseError::InvalidSetting(value.to_string())),
         }
     }