refactor: asegurar created_by y usuarios al asignarlos en TaskService

Co-authored-by: aider (openrouter/openai/gpt-5) <aider@aider.chat>

src/tasks/service.ts

@@ -1,5 +1,5 @@
 import type { Database } from 'bun:sqlite';
-import { db } from '../db';
+import { db, ensureUserExists } from '../db';
 
 type CreateTaskInput = {
   description: string;
@@ -23,11 +23,16 @@ export class TaskService {
         VALUES (?, ?, ?, ?)
       `);
 
+      const ensuredCreator = ensureUserExists(task.created_by, this.dbInstance);
+      if (!ensuredCreator) {
+        throw new Error('No se pudo asegurar created_by');
+      }
+
       const runResult = insertTask.run(
         task.description,
         task.due_date ?? null,
         task.group_id ?? null,
-        task.created_by
+        ensuredCreator
       );
       const taskId = Number((runResult as any).lastInsertRowid);
 
@@ -37,12 +42,18 @@ export class TaskService {
           VALUES (?, ?, ?)
         `);
 
-        // Evitar duplicados por (task_id, user_id)
+        // Evitar duplicados por (task_id, user_id) tras asegurar usuarios
         const seen = new Set<string>();
         for (const a of assignments) {
-          if (seen.has(a.user_id)) continue;
+          const ensuredUser = ensureUserExists(a.user_id, this.dbInstance);
-          seen.add(a.user_id);
+          if (!ensuredUser) continue;
-          insertAssignment.run(taskId, a.user_id, a.assigned_by);
+          if (seen.has(ensuredUser)) continue;
+          seen.add(ensuredUser);
+
+          const ensuredAssigner =
+            ensureUserExists(a.assigned_by || ensuredCreator, this.dbInstance) || ensuredCreator;
+
+          insertAssignment.run(taskId, ensuredUser, ensuredAssigner);
         }
       }