brightspace-mcp-server 5.1.31 → 5.1.33

package/build/tools/get-my-groups.js

@@ -62,6 +62,7 @@ export function registerGetMyGroups(server, apiClient) {
             // though it looks numeric, so it must be coerced before it can be
             // used as a key alongside the numeric Enrollments ids.
             let classlistById = new Map();
+            let namesUnavailable = false;
             try {
                 const classlist = await fetchClasslistUsers(apiClient, courseId);
                 classlistById = new Map(classlist.map((u) => [Number(u.Identifier), u]));
@@ -69,6 +70,7 @@ export function registerGetMyGroups(server, apiClient) {
             catch (error) {
                 if (isMissingOrForbidden(error)) {
                     log("INFO", "get_my_groups: Classlist not accessible, member names will be null", { courseId });
+                    namesUnavailable = true;
                 }
                 else {
                     throw error;
@@ -90,7 +92,14 @@ export function registerGetMyGroups(server, apiClient) {
                 };
             });
             log("INFO", `get_my_groups: Found ${groups.length} groups for course ${courseId}`);
-            return toolResponse({ courseId, groups, ...(unreadNote ? { note: unreadNote } : {}) });
+            // Null names are only meaningful with a reason attached: without one,
+            // a caller can't tell "the classlist couldn't be read" from "these
+            // members have no names", and may report the group as anonymous.
+            const namesNote = namesUnavailable
+                ? "Couldn't read this course's classlist, so member names are null; only user ids are listed."
+                : undefined;
+            const note = [unreadNote, namesNote].filter(Boolean).join(" ") || undefined;
+            return toolResponse({ courseId, groups, ...(note ? { note } : {}) });
         }
         catch (error) {
             return sanitizeError(error);

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.31",
+  "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",