brightspace-mcp-server 5.1.32 → 5.1.33

package/build/tools/get-roster.js

@@ -21,6 +21,15 @@ const TEACHING_ROLE_NAME = /\b(instructor|professor|lecturer|teaching assistant|
 function isTeachingRoleByName(user) {
     return TEACHING_ROLE_NAME.test(user.ClasslistRoleDisplayName ?? "");
 }
+/**
+ * Whether a failed role route must be reported rather than skipped. A 403
+ * means the classlist is restricted for that role: nothing was missed.
+ * Anything else (a 404, a 5xx, a dropped connection) leaves a hole the caller
+ * would read as "this course has no TAs".
+ */
+function isReportableFailure(error) {
+    return error?.status !== 403;
+}
 /**
  * Fetch every classlist user matching the optional filters, across all pages.
  *
@@ -60,6 +69,7 @@ export function registerGetRoster(server, apiClient) {
                 const allUsers = [];
                 let authFailure;
                 let roleFallbackUsed = false;
+                let unavailable = [];
                 if (!includeStudents) {
                     // Fetch instructors and TAs in parallel
                     const [instructorResult, taResult] = await Promise.allSettled([
@@ -87,6 +97,10 @@ export function registerGetRoster(server, apiClient) {
                         allUsers.push(...instructorResult.value);
                     }
                     else {
+                        if (!isAuthUnavailable(instructorResult.reason) &&
+                            isReportableFailure(instructorResult.reason)) {
+                            unavailable.push("instructors");
+                        }
                         log("WARN", "get_roster: Failed to fetch instructors", {
                             error: instructorResult.reason,
                         });
@@ -95,6 +109,9 @@ export function registerGetRoster(server, apiClient) {
                         allUsers.push(...taResult.value);
                     }
                     else {
+                        if (!isAuthUnavailable(taResult.reason) && isReportableFailure(taResult.reason)) {
+                            unavailable.push("TAs");
+                        }
                         log("WARN", "get_roster: Failed to fetch TAs", {
                             error: taResult.reason,
                         });
@@ -112,6 +129,9 @@ export function registerGetRoster(server, apiClient) {
                         });
                         allUsers.push(...everyone.filter(isTeachingRoleByName));
                         roleFallbackUsed = true;
+                        // The unfiltered fetch covered every role, so nothing the
+                        // failed role routes would have returned is missing any more.
+                        unavailable = [];
                     }
                 }
                 else {
@@ -139,6 +159,17 @@ export function registerGetRoster(server, apiClient) {
                 }));
                 log("INFO", `get_roster: Retrieved ${users.length} users for course ${courseId}` +
                     (authFailure ? " (sign-in pending for one route)" : ""));
+                const notices = [];
+                if (unavailable.length > 0) {
+                    notices.push(`The ${unavailable.join(" and ")} for this course could not be read from ` +
+                        "Brightspace, so they may be missing from this list. Do not report this " +
+                        `course as having no ${unavailable.join(" or ")}; call get_roster again to retry.`);
+                }
+                if (authFailure) {
+                    notices.push("Sign-in to Brightspace is still in progress, so part of the roster for this " +
+                        `course could not be fetched yet. ${authPendingNotice(authFailure.reason)} Call ` +
+                        "get_roster again once sign-in finishes.");
+                }
                 return toolResponse({
                     courseId,
                     total,
@@ -153,14 +184,9 @@ export function registerGetRoster(server, apiClient) {
                         }
                         : {}),
                     users,
-                    ...(authFailure
-                        ? {
-                            authPending: true,
-                            notice: "Sign-in to Brightspace is still in progress, so part of the roster for this " +
-                                `course could not be fetched yet. ${authPendingNotice(authFailure.reason)} Call ` +
-                                "get_roster again once sign-in finishes.",
-                        }
-                        : {}),
+                    ...(unavailable.length > 0 ? { unavailable } : {}),
+                    ...(authFailure ? { authPending: true } : {}),
+                    ...(notices.length > 0 ? { notice: notices.join(" ") } : {}),
                 });
             }
             catch (error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "brightspace-mcp-server",
-  "version": "5.1.32",
+  "version": "5.1.33",
   "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",