brightspace-mcp-server 5.0.1 → 5.0.3

package/README.md

@@ -69,7 +69,7 @@ School-specific quirks: [docs/sign-in.md](docs/sign-in.md#per-school-notes).
 |-------|-----|
 | Grades | "Am I passing all my classes?" · "Compare my grades across courses" · "What did my professor say about Mini-Assignment 1?" (own assignment feedback is read from every route Brightspace documents for students) · "Is there feedback on my midterm?" (quiz-scored grades link to the quiz, since quiz feedback, including feedback only viewable in LockDown Browser, isn't readable through the API) |
 | Due dates | "What's due in the next 48 hours?" · "Build me a study schedule for the week" |
-| Assignments and rubrics | "What does the lab 4 spec actually ask for?" · "Why did I lose points on the analysis criterion?" |
+| Assignments and rubrics | "What does the lab 4 spec actually ask for?" · "Why did I lose points on the analysis criterion?" · "Did I turn in HW 3?" (if Brightspace won't give students their own submission list, the assignment's feedback page is checked instead; when neither shows a submission it's reported as unknown, never as missing) |
 | Quizzes and exams | "Which quizzes close this week?" · "Is there a midterm in the gradebook that isn't on my assignments list?" (quizzes are collected from the quiz list, the course content, and the gradebook, so a closed quiz or one the list leaves out still shows up; each course says whether its quiz sources all answered, and if Brightspace refused one it's marked incomplete rather than passed off as everything; "complete" means every source answered, not that every quiz is guaranteed to be listed. A gradebook column linked to a quiz the list omits now shows up as a quiz with its dates and attempts instead of a bare grade-only row when the quiz can be read. Gradebook-only rows now also appear on tenants that send only the documented `GradeType` on grade columns; hidden grade columns, hidden assignments and inactive quizzes never produce one. Content quizzes beyond the first 30 per course are listed from the course content alone and the course is marked incomplete. Quiz rows may also say whether they are open, closed, or not yet open, and mark a quiz whose details Brightspace would not return; gradebook lookups for unlisted quizzes are capped at 20 per course, or 5 when the course's quiz tool can't be identified) |
 | Announcements | "Did any professor post something important today?" · "Read the file attached to today's announcement" |
 | Course content | "Find the midterm review slides" · "Download every PDF from Module 5" · "Search this course for office hours" · or just read a file inline instead of saving it |

package/build/tools/get-assignments.js

@@ -390,6 +390,23 @@ function quizCoverage(answered) {
 export async function fetchCourseAssignments(apiClient, courseId, baseUrl) {
     return (await fetchCourseInventory(apiClient, courseId, baseUrl)).assignments;
 }
+/**
+ * True when a student's own dropbox feedback page lists at least one
+ * submission. Brightspace renders each submission as a row of its
+ * submitted-files grid: the numeric submission ID in a <label>, then the
+ * submitted file links (span.dfl), then the date. This reads that structure,
+ * not the column titles, which are translated and are present even as headers
+ * of an otherwise empty grid. A page with "No submissions made for this
+ * assignment", a login page, or any other page has no such row.
+ */
+function feedbackPageListsSubmission(html) {
+    for (const row of html.matchAll(/<tr\b[^>]*>([\s\S]*?)<\/tr>/gi)) {
+        const cells = row[1];
+        if (/^\s*<td\b[^>]*>\s*<label>\s*\d+\s*<\/label>\s*<\/td>/i.test(cells) && /class="dfl"/i.test(cells))
+            return true;
+    }
+    return false;
+}
 /** A course's assignments and quizzes, with whether its quiz list is complete. */
 async function fetchCourseInventory(apiClient, courseId, baseUrl) {
     const assignments = [];
@@ -456,6 +473,22 @@ async function fetchCourseInventory(apiClient, courseId, baseUrl) {
                     log("DEBUG", `Could not determine submission state for folder ${folder.Id}`, error);
                 }
             }
+            // When the API refuses the list, the student's own feedback page may
+            // still show it. Only a listed submission counts: a page with none, or
+            // no page at all, leaves the state unknown rather than "not submitted".
+            let submissionSource = null;
+            if (!submissionsKnown && (folder.GroupTypeId === null || folder.GroupTypeId === undefined)) {
+                try {
+                    const html = await apiClient.getPage(`/d2l/lms/dropbox/user/folder_user_view_feedback.d2l?db=${folder.Id}&grpid=0&ou=${courseId}`);
+                    if (html !== null && feedbackPageListsSubmission(html))
+                        submissionSource = "feedbackPage";
+                }
+                catch (error) {
+                    if (isAuthUnavailable(error))
+                        throw error;
+                    log("DEBUG", `Feedback-page fallback failed for folder ${folder.Id}`, error);
+                }
+            }
             // Fetch feedback independently of submissions
             let feedback = null;
             let feedbackStatus = "unavailable";
@@ -550,12 +583,18 @@ async function fetchCourseInventory(apiClient, courseId, baseUrl) {
                 // when nothing is known, but submissionStatus tells apart "nothing
                 // was submitted" (404, a known answer) from "the answer is denied"
                 // (any other error), so a caller never states real work as missing.
-                submissionStatus: !submissionsKnown
-                    ? "unknown"
-                    : submissions.length > 0
-                        ? "submitted"
-                        : "not_submitted",
-                ...(submissionsKnown ? {} : {
+                submissionStatus: submissionSource
+                    ? "submitted"
+                    : !submissionsKnown
+                        ? "unknown"
+                        : submissions.length > 0
+                            ? "submitted"
+                            : "not_submitted",
+                ...(submissionSource ? {
+                    submissionSource,
+                    submissionStatusNote: "Brightspace did not return this assignment's submission list, but its feedback page lists a " +
+                        "submission, so it was submitted. File details are only shown on that page.",
+                } : submissionsKnown ? {} : {
                     submissionStatusNote: "Brightspace did not return this assignment's submission list, so whether it was submitted " +
                         "is unknown. Do not report it as missing or unsubmitted - tell the user to check Brightspace directly.",
                 }),

package/build/utils/transcript/lti.js

@@ -99,6 +99,10 @@ export function readLtiLaunchPage(html, origin) {
     const consumerKey = inputs.find(input => input.get("name") === "oauth_consumer_key")?.get("value");
     const candidates = [
         ...tagAttributes(html, "form").map(form => form.get("action")),
+        // Brightspace's own quickLink page renders the tool as a web component
+        // whose lti-launch-url is the tool launch (/d2l/le/lti/…/toolLaunch/…),
+        // not a form or an iframe.
+        ...tagAttributes(html, "d2l-lti-launch").map(element => element.get("lti-launch-url")),
         ...tagAttributes(html, "iframe").map(frame => frame.get("src")),
         ...inputs.map(input => input.get("value")),
     ].filter((candidate) => Boolean(candidate));

package/build/utils/transcript/platform.js

@@ -62,6 +62,16 @@ export function detectVideoPlatform(url) {
         return "kaltura";
     return "unknown";
 }
+/**
+ * Kaltura's hosted KAF front ends are named after the school's partner ID:
+ * `983291-6.kaf.kaltura.com` is partner 983291 (instance 6). An LTI 1.3
+ * launch's target_link_uri points there with only the entry ID in the path,
+ * so the host is the one place the partner ID appears.
+ */
+function kafHostPartnerId(hostname) {
+    const match = /^(\d+)(?:-\d+)?\.kaf\.kaltura\.com$/i.exec(hostname);
+    return match ? match[1] : null;
+}
 /** Kaltura embeds carry the entry and partner ID as query params or path segments; naming varies by embed generator. */
 export function extractKalturaIds(url) {
     let parsed;
@@ -85,7 +95,7 @@ export function extractKalturaIds(url) {
         params.get("pid") ??
         pathSegmentParam(pathname, "wid") ??
         pathSegmentParam(pathname, "partner_id");
-    const partnerId = widParam ? widParam.replace(/^_/, "") : null;
+    const partnerId = widParam ? widParam.replace(/^_/, "") : kafHostPartnerId(parsed.hostname);
     if (!entryId || !partnerId)
         return null;
     return { partnerId, entryId };

package/package.json

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