brightspace-mcp-server 3.15.0 → 3.15.1

package/build/tools/get-calendar-events.js

@@ -37,6 +37,7 @@ export function registerGetCalendarEvents(server, apiClient, config) {
             // have gone out independently. Collect it rather than failing the
             // whole call and losing every course that *did* answer.
             const pendingCourseIds = [];
+            const failedCourseIds = [];
             let firstAuthError = null;
             const events = results
                 .flatMap((result, i) => {
@@ -48,23 +49,37 @@ export function registerGetCalendarEvents(server, apiClient, config) {
                     log("DEBUG", `get_calendar_events: sign-in pending for course ${courses[i].id}`, result.reason);
                     return [];
                 }
-                log("DEBUG", `get_calendar_events: skipping course ${courses[i].id} after fetch failure`, result.reason);
+                // Anything else (a 404, a 5xx, a dropped connection) still must
+                // not make the course vanish: an exam missing from a list that
+                // looks complete reads as "no exam" (#262).
+                failedCourseIds.push(courses[i].id);
+                log("DEBUG", `get_calendar_events: calendar unavailable for course ${courses[i].id}`, result.reason);
                 return [];
             })
                 .filter((event) => includeGenerated || !event.generatedFrom)
                 .sort((a, b) => new Date(a.start).getTime() - new Date(b.start).getTime());
             log("INFO", `get_calendar_events: Retrieved ${events.length} events across ${courses.length} courses` +
-                (pendingCourseIds.length > 0 ? ` (${pendingCourseIds.length} pending sign-in)` : ""));
+                (pendingCourseIds.length > 0 ? ` (${pendingCourseIds.length} pending sign-in)` : "") +
+                (failedCourseIds.length > 0 ? ` (${failedCourseIds.length} failed)` : ""));
             // The response is always a bare array, pending sign-in or not — a
-            // shape change would break every existing caller. A pending course
-            // instead adds a second content block carrying the notice, which
-            // names which course ids are unavailable.
-            if (pendingCourseIds.length === 0) {
+            // shape change would break every existing caller. A pending or failed
+            // course instead adds a second content block carrying the notice,
+            // which names which course ids are unavailable.
+            const notices = [];
+            if (pendingCourseIds.length > 0) {
+                notices.push("Sign-in to Brightspace is still in progress, so calendar events for " +
+                    `${pendingCourseIds.length} course(s) (${pendingCourseIds.join(", ")}) could not be fetched ` +
+                    `yet. ${authPendingNotice(firstAuthError)} Call get_calendar_events again once sign-in finishes.`);
+            }
+            if (failedCourseIds.length > 0) {
+                notices.push(`Calendar events for ${failedCourseIds.length} course(s) (${failedCourseIds.join(", ")}) ` +
+                    "could not be read from Brightspace, so they are missing from this list. Do not report " +
+                    "these courses as having no events: call get_calendar_events again with that courseId.");
+            }
+            if (notices.length === 0) {
                 return toolResponse(events);
             }
-            return toolResponseWithNotice(events, "Sign-in to Brightspace is still in progress, so calendar events for " +
-                `${pendingCourseIds.length} course(s) (${pendingCourseIds.join(", ")}) could not be fetched ` +
-                `yet. ${authPendingNotice(firstAuthError)} Call get_calendar_events again once sign-in finishes.`);
+            return toolResponseWithNotice(events, notices.join(" "));
         }
         catch (error) {
             return sanitizeError(error);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "brightspace-mcp-server",
-  "version": "3.15.0",
+  "version": "3.15.1",
   "mcpName": "io.github.RohanMuppa/brightspace",
   "description": "MCP server for Brightspace (D2L). Check grades, due dates, assignments, announcements, syllabus, rosters and more via Claude, ChatGPT, Cursor, Windsurf, or any MCP client.",
   "type": "module",