From ccd72146cd2e62391b2bf47d9de585c557aa0073 Mon Sep 17 00:00:00 2001
From: "brobert (aider)" <borja@brobert.net>
Date: Sun, 23 Mar 2025 12:11:09 +0100
Subject: [PATCH] fix: Correct syntax errors and add error handling in task
 command

---
 src/bot/commands/task.ts | 32 ++++++++++++++++++--------------
 1 file changed, 18 insertions(+), 14 deletions(-)

diff --git a/src/bot/commands/task.ts b/src/bot/commands/task.ts
index 9b43773..43cf5c2 100644
--- a/src/bot/commands/task.ts
+++ b/src/bot/commands/task.ts
@@ -64,21 +64,25 @@ export function handleTaskCommand(body: string, sender: string, groupId: string,
         try {
           const taskId = validateTaskId(args[0]);
           const task = getTaskById(taskId);
-        
-        if (!task) {
-          sendMessage(sender, `Tarea ${taskId} no encontrada`);
-          return;
-        }
-        
-        // Check if task is assigned to current user
-        const currentUser = `@${sender.split('@')[0]}`;
-        if (task.assignedTo !== currentUser) {
-          sendMessage(sender, `No puedes completar la tarea ${taskId} porque no está asignada a ti`);
-          return;
+          
+          if (!task) {
+            sendMessage(sender, `Tarea ${taskId} no encontrada`);
+            return;
+          }
+          
+          // Check if task is assigned to current user
+          const currentUser = `@${sender.split('@')[0]}`;
+          if (task.assignedTo !== currentUser) {
+            sendMessage(sender, `No puedes completar la tarea ${taskId} porque no está asignada a ti`);
+            return;
+          }
+          
+          completeTask(taskId);
+          sendMessage(sender, `✅ Tarea ${taskId} completada: ${task.description}`);
+        } catch (error) {
+          console.error('Error completing task:', error);
+          sendMessage(sender, `❌ Error al completar tarea: ${error.message}`);
         }
-        
-        completeTask(parseInt(taskId));
-        sendMessage(sender, `✅ Tarea ${taskId} completada: ${task.description}`);
       } else if (action === 'mostrar' || !action) {
         // Handle /tarea or /tarea mostrar
         const user = `@${sender.split('@')[0]}`; // Default to the sender