@studious-lms/server 1.5.0 → 1.6.0

package/src/workers/processors/studentProgressChat.ts

@@ -0,0 +1,28 @@
+import { chatAboutStudentProgress } from '../../services/studentProgress.js';
+import { createRunJobWorker } from './createRunJobWorker.js';
+
+export const studentProgressChatWorker = createRunJobWorker({
+  type: 'STUDENT_PROGRESS_CHAT',
+  workerOptions: {
+    concurrency: 4,
+    limiter: { max: 20, duration: 60_000 },
+  },
+  run: async ({ classId, studentId, message }, ctx) => {
+    // `chatAboutStudentProgress` runs an authorization check using the viewer
+    // id, so callers MUST pass `userId` via `enqueueRunJob` options for the
+    // service to receive it here.
+    if (!ctx.userId) {
+      throw new Error(
+        'STUDENT_PROGRESS_CHAT requires `userId` on the RunJob (set via enqueueRunJob options)',
+      );
+    }
+
+    const response = await chatAboutStudentProgress(ctx.userId, {
+      classId,
+      studentId,
+      message,
+    });
+
+    return response;
+  },
+});

package/src/workers/queues.ts

@@ -26,15 +26,30 @@ const DEFAULT_JOB_OPTIONS: JobsOptions = {
  * The queue name string must match what the corresponding processor passes
  * to `new Worker(<name>, ...)`.
  */
-type JobInputs = {
-  PDF_TO_WORKSHEET: { gcsUrl: string; classId: string };
+export type JobInputs = {
+  /**
+   * `worksheetId` is the placeholder Worksheet row created synchronously by
+   * the producer (so the client has an id to navigate to). The pipeline fills
+   * that row in asynchronously and broadcasts progress on the per-worksheet
+   * job channel.
+   */
+  PDF_TO_WORKSHEET: { gcsUrl: string; classId: string; worksheetId: string };
   GRADE_WORKSHEET: { worksheetResponseId: string };
-  AI_LAB_CHAT: { conversationId: string; messageId: string };
-  AI_NEWTON_CHAT: { conversationId: string; messageId: string };
+  /**
+   * `aiMessageId` is the placeholder AI Message row created synchronously by
+   * the producer before enqueue (so the client renders a PENDING bubble in
+   * the timeline). The pipeline fills in the content and flips the row to
+   * COMPLETED / FAILED. Used for both intros and replies — single shape.
+   */
+  AI_LAB_CHAT: { conversationId: string; aiMessageId: string };
+  /**
+   * `aiMessageId` placeholder pattern, same as AI_LAB_CHAT — see above.
+   */
+  AI_NEWTON_CHAT: { conversationId: string; aiMessageId: string };
   STUDENT_PROGRESS_CHAT: { classId: string; studentId: string; message: string };
 };
 
-const QUEUE_NAMES: Record<RunJobType, string> = {
+export const QUEUE_NAMES: Record<RunJobType, string> = {
   PDF_TO_WORKSHEET: 'pdfToWorksheet',
   GRADE_WORKSHEET: 'gradeWorksheet',
   AI_LAB_CHAT: 'aiLabChat',

package/tests/integration/routers/attendance.test.ts

@@ -117,6 +117,66 @@ describe('Attendance Router', () => {
       expect(updatedRecord?.late.some((l: any) => l.id === student2.id)).toBe(true);
     });
 
+    test('should notify students newly marked late or absent', async () => {
+      await prisma.notification.deleteMany({
+        where: { receiverId: student2.id },
+      });
+
+      await user1Caller.attendance.update({
+        classId: testClass.id,
+        eventId: testEvent.event.id,
+        attendance: {
+          present: [{ id: student1.id, username: student1.username }],
+          late: [{ id: student2.id, username: student2.username }],
+          absent: [],
+        },
+      });
+
+      let notifications = await user2Caller.notification.list();
+      expect(
+        notifications.some(
+          (notification: any) =>
+            notification.title === `Marked late in ${testClass.name}` &&
+            notification.content.includes(testEvent.event.name) &&
+            notification.content.includes(`/class/${testClass.id}/attendance/${testEvent.event.id}`)
+        )
+      ).toBe(true);
+
+      await user1Caller.attendance.update({
+        classId: testClass.id,
+        eventId: testEvent.event.id,
+        attendance: {
+          present: [{ id: student1.id, username: student1.username }],
+          late: [{ id: student2.id, username: student2.username }],
+          absent: [],
+        },
+      });
+
+      notifications = await user2Caller.notification.list();
+      expect(
+        notifications.filter(
+          (notification: any) => notification.title === `Marked late in ${testClass.name}`
+        )
+      ).toHaveLength(1);
+
+      await user1Caller.attendance.update({
+        classId: testClass.id,
+        eventId: testEvent.event.id,
+        attendance: {
+          present: [{ id: student1.id, username: student1.username }],
+          late: [],
+          absent: [{ id: student2.id, username: student2.username }],
+        },
+      });
+
+      notifications = await user2Caller.notification.list();
+      expect(
+        notifications.some(
+          (notification: any) => notification.title === `Marked absent in ${testClass.name}`
+        )
+      ).toBe(true);
+    });
+
     test('should update attendance without event', async () => {
       const result = await user1Caller.attendance.update({
         classId: testClass.id,
@@ -157,4 +217,3 @@ describe('Attendance Router', () => {
       })));
   });
 });
-