brightspace-mcp-server 4.0.6 → 4.0.7

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

@@ -22,8 +22,9 @@ const QUIZ_FEEDBACK_NOTE = "The gradebook has no comment for this item, but it i
     "browser such as Respondus LockDown Browser. Open feedbackUrl to check; do not report " +
     "this item as having no feedback.";
 /**
- * The course's quizzes, or none when the route refuses or fails. A missing
- * quiz list only costs the feedback links, never the grades themselves.
+ * The course's quizzes, or null when the route refuses or fails. A missing
+ * quiz list only costs the feedback links, never the grades themselves, but
+ * the caller is told so a gradebook link isn't read as "not a quiz" (#278).
  */
 async function fetchQuizzes(apiClient, courseId) {
     try {
@@ -34,7 +35,7 @@ async function fetchQuizzes(apiClient, courseId) {
     }
     catch (error) {
         log("DEBUG", `get_my_grades: quiz list unavailable for course ${courseId}`, error);
-        return [];
+        return null;
     }
 }
 /**
@@ -42,7 +43,7 @@ async function fetchQuizzes(apiClient, courseId) {
  * rows keep the quiz source URL and note; other rows link to the gradebook.
  */
 function toGradeItems(gradeValues, quizzes, baseUrl, courseId) {
-    const quizByGradeItem = new Map(quizzes
+    const quizByGradeItem = new Map((quizzes ?? [])
         .filter((q) => q.GradeItemId != null)
         .map((q) => [String(q.GradeItemId), q.QuizId]));
     return gradeValues.map((gv) => {
@@ -65,6 +66,11 @@ function toGradeItems(gradeValues, quizzes, baseUrl, courseId) {
         };
     });
 }
+function quizzesUnavailableNotice(courseIds) {
+    return (`The quiz list for ${courseIds.length} course(s) (${courseIds.join(", ")}) could not be read, so ` +
+        "quiz-scored grade items there link to the gradebook and lack the quiz feedback note. Quiz " +
+        "feedback may still exist on the quiz's own page; do not report those items as having no feedback.");
+}
 /**
  * Register get_my_grades tool
  */
@@ -122,8 +128,17 @@ export function registerGetMyGrades(server, apiClient, config) {
                     throw error;
                 }
                 // Map to clean objects
-                const grades = toGradeItems(gradeValues, await quizzesPromise, config.baseUrl, courseId);
+                const quizzes = await quizzesPromise;
+                const grades = toGradeItems(gradeValues, quizzes, config.baseUrl, courseId);
                 log("INFO", `get_my_grades: Retrieved ${grades.length} grade items for course ${courseId}`);
+                if (quizzes === null) {
+                    return toolResponse({
+                        courseId,
+                        grades,
+                        quizzesUnavailable: true,
+                        notice: quizzesUnavailableNotice([courseId]),
+                    });
+                }
                 return toolResponse({ courseId, grades });
             }
             // All courses case
@@ -160,6 +175,7 @@ export function registerGetMyGrades(server, apiClient, config) {
                         courseId: item.OrgUnit.Id,
                         courseName: item.OrgUnit.Name,
                         grades,
+                        ...(quizzes === null ? { quizzesUnavailable: true } : {}),
                     };
                 }
                 catch (error) {
@@ -216,7 +232,12 @@ export function registerGetMyGrades(server, apiClient, config) {
                 .map(({ failed: _failed, ...rest }) => rest);
             const courses = settled
                 .filter((c) => !c.authPending && !c.restricted && !c.failed)
-                .map(({ courseId, courseName, grades }) => ({ courseId, courseName, grades }));
+                .map(({ courseId, courseName, grades, quizzesUnavailable }) => ({
+                courseId,
+                courseName,
+                grades,
+                ...(quizzesUnavailable ? { quizzesUnavailable } : {}),
+            }));
             log("INFO", `get_my_grades: Retrieved grades for ${courses.length} courses, ` +
                 `${restrictedCourses.length} restricted (out of ${enrollmentItems.length} enrolled` +
                 `${pending.length > 0 ? `, ${pending.length} pending sign-in` : ""}` +
@@ -255,6 +276,9 @@ export function registerGetMyGrades(server, apiClient, config) {
                     "so they are listed in failedCourses instead of courses. Do not report these courses as " +
                     "having no grades: open each gradeUrl, or call get_my_grades again for that courseId.");
             }
+            const quizGaps = courses.filter((c) => c.quizzesUnavailable).map((c) => c.courseId);
+            if (quizGaps.length > 0)
+                notices.push(quizzesUnavailableNotice(quizGaps));
             if (notices.length > 0)
                 response.notice = notices.join(" ");
             return toolResponse(response);

package/package.json

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