Kl/scrum 147 fix timetable integration bugs (#103)

Co-authored-by: kevin-lann <[email protected]>

Author: kevin-lann

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

@@ -267,6 +267,7 @@ export const chat = asyncHandler(async (req: Request, res: Response) => {
             - 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.
             - Do not make up fake courses or offerings. 
+            - You can only edit title of the timetable, nothing else. If a user tries to edit something else, acknowledge this limitation.
             - For delete timetable requests, if the user asks to delete an ambiguous timetable name (i.e many with similar name exist) then ask them to clarify which one
             - For delete timetable requests, first check that the timetable the user is refering to exists 
             - For delete timetable requests, ask for user confirmation with command "/timetable confirm" before proceeding. If their next message is anything other than "/timetable confim" then cancel the deletion. 
@@ -285,13 +286,10 @@ export const chat = asyncHandler(async (req: Request, res: Response) => {
             },
           }),
           updateTimetable: tool({
-            description: "Update a user's timetable by title and/or semester",
+            description: "Update a user's timetable's title",
             parameters: z.object({
               id: z.number().positive(),
               timetable_title: z.string().optional(),
-              semester: z
-                .enum(["Fall 2025", "Summer 2025", "Winter 2026"])
-                .optional(),
             }),
             execute: async (args) => {
               return await availableFunctions.updateTimetable(args, req);

course-matrix/frontend/src/pages/TimetableBuilder/TimetableBuilder.tsx

@@ -370,11 +370,13 @@ const TimetableBuilder = () => {
           <Form {...form}>
             <FormContext.Provider value={form}>
               <form
-                onSubmit={form.handleSubmit(handleGenerate)}
+                onSubmit={form.handleSubmit(handleGenerate, (errors) => {
+                  console.error("Form submission errors:", errors);
+                })}
                 className="space-y-8"
               >
                 <div className="flex gap-8 w-full">
-                  {/* <FormField
+                  <FormField
                     control={form.control}
                     name="semester"
                     render={({ field }) => (
@@ -383,11 +385,13 @@ const TimetableBuilder = () => {
                         <FormControl>
                           <Select
                             onValueChange={(value) => {
-                              form.reset({ offeringIds: [], courses: [] });
+                              form.setValue("offeringIds", []);
+                              form.setValue("courses", []);
                               form.setValue("semester", value);
                             }}
                             value={field.value}
                             defaultValue={field.value}
+                            disabled={isEditingTimetable}
                           >
                             <SelectTrigger className="w-[140px]">
                               <SelectValue placeholder="Select a semester" />
@@ -406,7 +410,7 @@ const TimetableBuilder = () => {
                         <FormMessage />
                       </FormItem>
                     )}
-                  /> */}
+                  />
 
                   <FormField
                     control={form.control}
@@ -437,7 +441,7 @@ const TimetableBuilder = () => {
                     <p className="text-sm">
                       Selected courses: {selectedCourses.length} (Max 8)
                     </p>
-                    {!isEditingTimetable && (
+                    {!isEditingTimetable && !isGeneratingTimetables && (
                       <div className="flex items-center gap-2">
                         <Checkbox
                           id="manual-selection"