brightspace-mcp-server 5.2.4 → 5.2.5

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

@@ -14,6 +14,8 @@ import { log } from "../utils/logger.js";
 const SELF_ENROLLMENT_STYLES = new Set([3, 4, 5]);
 /** Without a courseId, courses that started longer ago than this are skipped unless includeOlder is set. */
 const RECENT_COURSE_MS = 365 * 24 * 60 * 60 * 1000;
+/** Why a category's groups couldn't be read, in words an agent can relay. */
+const unreadReason = (status) => status === 403 ? "Brightspace refused access (403)" : "Brightspace found no group list (404)";
 const inaccessible = (error) => error instanceof ApiError && (error.status === 403 || error.status === 404);
 const text = (value) => value?.Text?.trim() || undefined;
 /**
@@ -27,7 +29,7 @@ function signUpOpen(category, now) {
     const end = category.SelfEnrollmentExpiryDate ? Date.parse(category.SelfEnrollmentExpiryDate) : Infinity;
     return start <= now && now <= end;
 }
-async function fetchCourseGroups(apiClient, course, myId, now) {
+async function fetchCourseGroups(apiClient, course, myId, now, unreadOut) {
     const result = { courseId: course.id, courseName: course.name, categories: [], alreadyInGroup: [] };
     let categories;
     try {
@@ -47,6 +49,7 @@ async function fetchCourseGroups(apiClient, course, myId, now) {
         catch (error) {
             if (inaccessible(error)) {
                 unread.push(category.Name);
+                unreadOut.push({ courseId: course.id, name: category.Name, status: error.status });
                 continue;
             }
             throw error;
@@ -115,7 +118,8 @@ export function registerGetAvailableGroups(server, apiClient, config) {
                 return Number.isFinite(start) && start >= now - RECENT_COURSE_MS;
             });
             const olderSkipped = enrolled.length - courses.length;
-            const results = await Promise.allSettled(courses.map((course) => fetchCourseGroups(apiClient, course, myId, now)));
+            const unreadCategories = [];
+            const results = await Promise.allSettled(courses.map((course) => fetchCourseGroups(apiClient, course, myId, now, unreadCategories)));
             const pendingCourseIds = [];
             const failedCourseIds = [];
             let firstAuthError = null;
@@ -149,6 +153,16 @@ export function registerGetAvailableGroups(server, apiClient, config) {
             if (failedCourseIds.length > 0) {
                 notes.push(`Couldn't read groups for course(s) ${failedCourseIds.join(", ")}, so open sign-ups there aren't listed.`);
             }
+            // Without a reason attached, a bare unreadCategories list reads like a
+            // category with nothing open, so name each one and say why it's there (#433).
+            const shownCourseIds = new Set(withChoices.map((c) => c.courseId));
+            const unread = unreadCategories.filter((u) => shownCourseIds.has(u.courseId));
+            if (unread.length > 0) {
+                const where = unread.map((u) => `"${u.name}" in course ${u.courseId}: ${unreadReason(u.status)}`).join("; ");
+                notes.push(`Couldn't read the groups in these open sign-ups, so any open groups there aren't listed ` +
+                    `(see unreadCategories): ${where}. Don't report them as full or closed; ` +
+                    "check Groups > Available Groups in Brightspace.");
+            }
             const data = {
                 coursesChecked: courses.length,
                 courses: withChoices,

package/package.json

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