fix @ mention edit path missing parent_id support

Author: ComputelessComputer

src/llm.rs

@@ -860,6 +860,7 @@ struct Enriched {
     priority: Option<String>,
     due_date: Option<String>,
     dependencies: Option<Vec<String>>,
+    parent_id: Option<String>,
     subtasks: Option<Vec<SubTaskEnriched>>,
 }
 
@@ -920,7 +921,7 @@ fn edit_task(cfg: &LlmConfig, job: &AiJob, instruction: &str) -> AiResult {
         .join("|");
 
     let user = format!(
-        "Current task:\n{}\n\nInstruction: {}\n\nExisting tasks (id_prefix [bucket] title):\n{}\nReturn JSON with ONLY fields that should change (set unchanged fields to null):\n{{\n  \"title\": string | null,\n  \"bucket\": {bucket_enum} | null,\n  \"description\": string | null,\n  \"progress\": \"Backlog\"|\"Todo\"|\"In progress\"|\"Done\" | null,\n  \"priority\": \"Low\"|\"Medium\"|\"High\"|\"Critical\" | null,\n  \"due_date\": \"YYYY-MM-DD\" | null,\n  \"dependencies\": [\"id_prefix\", ...] | null,\n  \"subtasks\": [{{\"title\": string, \"description\": string, \"bucket\": {bucket_enum}, \"priority\": \"Low\"|\"Medium\"|\"High\"|\"Critical\", \"progress\": \"Backlog\"|\"Todo\"|\"In progress\"|\"Done\", \"due_date\": \"YYYY-MM-DD\" | null, \"depends_on\": [0-based index, ...]}}] | null\n}}\nRules:\n- If the instruction asks to create sub-issues, sub-tasks, break down, or decompose the task, return them as entries in the \"subtasks\" array. NEVER write sub-task lists, numbered breakdowns, or step-by-step plans into the \"description\" field.\n- depends_on is an array of 0-based indices into the subtasks array representing execution order. Use it to express sequential dependencies between subtasks.\n- Subtasks inherit the parent task's bucket and priority unless the instruction specifies otherwise.\n- Only include fields that should change. Set unchanged fields to null.\n",
+        "Current task:\n{}\n\nInstruction: {}\n\nExisting tasks (id_prefix [bucket] title):\n{}\nReturn JSON with ONLY fields that should change (set unchanged fields to null):\n{{\n  \"title\": string | null,\n  \"bucket\": {bucket_enum} | null,\n  \"description\": string | null,\n  \"progress\": \"Backlog\"|\"Todo\"|\"In progress\"|\"Done\" | null,\n  \"priority\": \"Low\"|\"Medium\"|\"High\"|\"Critical\" | null,\n  \"due_date\": \"YYYY-MM-DD\" | null,\n  \"dependencies\": [\"id_prefix\", ...] | null,\n  \"parent_id\": \"id_prefix\" | \"none\" | null,\n  \"subtasks\": [{{\"title\": string, \"description\": string, \"bucket\": {bucket_enum}, \"priority\": \"Low\"|\"Medium\"|\"High\"|\"Critical\", \"progress\": \"Backlog\"|\"Todo\"|\"In progress\"|\"Done\", \"due_date\": \"YYYY-MM-DD\" | null, \"depends_on\": [0-based index, ...]}}] | null\n}}\nRules:\n- If the instruction asks to create sub-issues, sub-tasks, break down, or decompose the task, return them as entries in the \"subtasks\" array. NEVER write sub-task lists, numbered breakdowns, or step-by-step plans into the \"description\" field.\n- depends_on is an array of 0-based indices into the subtasks array representing execution order. Use it to express sequential dependencies between subtasks.\n- Subtasks inherit the parent task's bucket and priority unless the instruction specifies otherwise.\n- To move this task under another task as a sub-task, set \"parent_id\" to the target task's id_prefix. To promote to a root task, set \"parent_id\" to \"none\".\n- Only include fields that should change. Set unchanged fields to null.\n",
         snapshot,
         instruction,
         context_lines
@@ -1029,6 +1030,12 @@ fn edit_task(cfg: &LlmConfig, job: &AiJob, instruction: &str) -> AiResult {
         update.dependencies = out;
     }
 
+    update.parent_id = enriched
+        .parent_id
+        .as_deref()
+        .map(|s| s.trim().to_string())
+        .filter(|s| !s.is_empty());
+
     let sub_task_specs: Vec<SubTaskSpec> = enriched
         .subtasks
         .unwrap_or_default()

src/main.rs

@@ -339,7 +339,7 @@ fn spawn_email_poller(
                 Ok(t) => t,
                 Err(_) => continue,
             };
-            let emails = match google::get_recent_emails(&token, 10) {
+            let emails: Vec<google::Email> = match google::get_recent_emails(&token, 10) {
                 Ok(e) => e,
                 Err(_) => continue,
             };
@@ -3938,6 +3938,45 @@ fn poll_ai(app: &mut App) -> bool {
                 changed = true;
             }
         }
+        if let Some(ref new_parent_prefix) = result.update.parent_id {
+            let task_id = result.task_id;
+            let old_parent = app
+                .tasks
+                .iter()
+                .find(|t| t.id == task_id)
+                .and_then(|t| t.parent_id);
+            if new_parent_prefix.eq_ignore_ascii_case("none") {
+                if let Some(task) = app.tasks.iter_mut().find(|t| t.id == task_id) {
+                    task.parent_id = None;
+                    changed = true;
+                }
+            } else {
+                let new_parent_id = app.tasks.iter().find_map(|t| {
+                    let short = t.id.to_string().chars().take(8).collect::<String>();
+                    if short.eq_ignore_ascii_case(new_parent_prefix) {
+                        Some(t.id)
+                    } else {
+                        None
+                    }
+                });
+                if let Some(pid) = new_parent_id {
+                    if let Some(task) = app.tasks.iter_mut().find(|t| t.id == task_id) {
+                        task.parent_id = Some(pid);
+                        changed = true;
+                    }
+                }
+            }
+            if let Some(old_pid) = old_parent {
+                sync_parent_progress(&mut app.tasks, old_pid, Utc::now());
+            }
+            if !changed {
+                app.status = Some((
+                    format!("AI: parent task {} not found", new_parent_prefix),
+                    Instant::now(),
+                    false,
+                ));
+            }
+        }
         sync_parent_progress(&mut app.tasks, parent_id, Utc::now());
 
         // Create actual sub-task records when the edit response includes subtasks.
@@ -5172,6 +5211,15 @@ fn render_calendar_tab(stdout: &mut Stdout, app: &App, cols: u16, rows: u16) ->
                 ResetColor
             )?;
             y_cur += 2;
+        } else if !app.google_connected {
+            queue!(
+                stdout,
+                MoveTo(events_x, y_cur),
+                SetForegroundColor(Color::DarkGrey),
+                Print(clamp_text("Connect Google in Settings (0)", events_width)),
+                ResetColor
+            )?;
+            y_cur += 2;
         }
 
         queue!(