Fix: DaysOfWeekEnum, and improve system prompt for function caller

Author: kevin-lann

course-matrix/backend/src/controllers/aiController.ts

@@ -246,13 +246,14 @@ export const chat = asyncHandler(async (req: Request, res: Response) => {
             - Include course codes when referencing specific courses
             - If information is missing from the context but likely exists, try to use info from web to answer. If still not able to form a decent response, acknowledge the limitation
             - For unrelated questions, politely explain that you're specialized in UTSC academic information
+            - Format long lists of timetables as a table
 
             ## Tool call guidelines
             - Include the timetable ID in all getTimetables tool call responses
-            - For every tool call, for each timetable that it gets/deletes/modifies/creates, include a link underneath it displayed as "View timetable" to ${process.env.CLIENT_APP_URL}/dashboard/timetable?edit=[[TIMETABLE_ID]] , where TIMETABLE_ID is the id of the respective timetable.
+            - Link: For every tool call, for each timetable that it gets/deletes/modifies/creates, include a link with it displayed as "View Timetable" to ${process.env.CLIENT_APP_URL}/dashboard/timetable?edit=[[TIMETABLE_ID]] , where TIMETABLE_ID is the id of the respective timetable.
             - If the user provides a course code of length 6 like CSCA08, then assume they mean CSCA08H3 (H3 appended)
             - If the user wants to create a timetable, first call getCourses to get course information on the requested courses, then call generateTimetable.
-            - For the response to create timetable, format as a table.
+            - Do not make up fake courses or offerings. 
             `,
         messages,
         tools: {
@@ -430,6 +431,7 @@ export const chat = asyncHandler(async (req: Request, res: Response) => {
           ## Response Guidelines
           - Be concise and direct when answering course-related questions
           - Use bullet points for listing multiple pieces of information
+          - Use tables for listing multiple offerings, courses, or other information that could be better viewed in tabular fashion
           - Include course codes when referencing specific courses
           - If information is missing from the context but likely exists, try to use info from web to answer. If still not able to form a decent response, acknowledge the limitation
           - For unrelated questions, politely explain that you're specialized in UTSC academic information

course-matrix/backend/src/models/timetable-form.ts

@@ -45,7 +45,7 @@ export const daysOfWeek = [
   },
 ] as const;
 
-export const DayOfWeekEnum = z.enum(["SU", "MO", "TU", "WE", "TH", "FR", "SA"]);
+export const DayOfWeekEnum = z.enum(["MO", "TU", "WE", "TH", "FR"]);
 
 export const SemesterEnum = z.enum(["Summer 2025", "Fall 2025", "Winter 2026"]);
 
@@ -56,7 +56,7 @@ export const CourseSchema = z.object({
     .max(8, "Invalid course code")
     .min(1, "Course code is required")
     .describe(
-      "The course code. Formatted like: CSCA08H3. Course codes cannot be provided without the H3 at the end.",
+      "The course code. Formatted like: CSCA08H3. Course codes cannot be provided without the H3 at the end."
     ),
   name: z.string().describe("The name of the course"),
 });
@@ -70,30 +70,32 @@ export const RestrictionSchema = z.object({
       "Restrict Day",
       "Days Off",
     ])
-    .describe("The type of restriction being applied"),
+    .describe(
+      "The type of restriction being applied. Restrict before restricts all times before 'endTime', Restrict Before restricts all times after 'startTime', Restrict Between restricts all times between 'startTime' and 'endTime', Restrict Day restricts the entirety of each day in field 'days', and Days Off enforces as least 'numDays' days off per week."
+    ),
   days: z
     .array(DayOfWeekEnum)
-    .default(["SU", "MO", "TU", "WE", "TH", "FR", "SA"])
+    .default(["MO", "TU", "WE", "TH", "FR"])
     .describe("Specific days of the week this restriction applies to"),
   numDays: z
     .number()
     .positive()
     .max(4, "Cannot block all days of the week")
     .optional()
     .describe(
-      "If type is Days Off, then this field is used and describes min number of days off per week.",
+      "If type is Days Off, then this field is used and describes min number of days off per week. For example, if set to 2, and 'type' is Days Off, then this means we want at least 2 days off per week."
     ),
   startTime: z
     .string()
     .optional()
     .describe(
-      "If type is Restrict After, or Restrict Between, then this field describes the start time of the restricted time. Formatted HH:mm:ss",
+      "If type is Restrict After, or Restrict Between, then this field describes the start time of the restricted time. Formatted HH:mm:ss"
     ),
   endTime: z
     .string()
     .optional()
     .describe(
-      "If type is Restrict Before, or Restrict Between, then this field describes the end time of the restricted time. Formatted HH:mm:ss",
+      "If type is Restrict Before, or Restrict Between, then this field describes the end time of the restricted time. Formatted HH:mm:ss"
     ),
   disabled: z
     .boolean()