Katrina feedback: more informative error message

src/backend/db/migrations/2_add-uniqueness-constraint-on-tasks.sql

@@ -1,7 +1,7 @@
 -- Each task should have a unique subgoal_id - assignee_id combination
 -- which corresponds to a unique benchmark / para combo
 ALTER TABLE task
-ADD CONSTRAINT UC_Task UNIQUE (subgoal_id, assignee_id);
+ADD CONSTRAINT subgoal_assignee_unique UNIQUE (subgoal_id, assignee_id);
 
 -- Add index to allow easy queries of tasks by assignee
 CREATE INDEX idx_task_assignee ON task(assignee_id);

src/backend/routers/iep.test.ts

@@ -155,7 +155,7 @@ test("addTask - no duplicate subgoal_id + assigned_id combo", async (t) => {
 
   t.is(
     error?.message,
-    'duplicate key value violates unique constraint "uc_task"'
+    "Task already exists: This subgoal has already been assigned to the same para"
   );
 });
 

src/backend/routers/iep.ts

@@ -135,6 +135,20 @@ export const iep = router({
     .mutation(async (req) => {
       const { subgoal_id, assignee_id, due_date, trial_count } = req.input;
 
+      // make sure that this goal belongs to this case manager
+      const existingTask = await req.ctx.db
+        .selectFrom("task")
+        .where("subgoal_id", "=", subgoal_id)
+        .where("assignee_id", "=", assignee_id)
+        .selectAll()
+        .executeTakeFirst();
+
+      if (existingTask) {
+        throw new Error(
+          "Task already exists: This subgoal has already been assigned to the same para"
+        );
+      }
+
       const result = await req.ctx.db
         .insertInto("task")
         .values({