brightspace-mcp-server 5.1.43 → 5.2.0

package/build/runtime-relay.js

@@ -35,6 +35,13 @@ class Unreachable extends Error {
  */
 export async function relaySharedRuntime(transport, { dir, startRuntime, startTimeoutMs = DEFAULT_START_TIMEOUT_MS }) {
     const disconnected = new AbortController();
+    /**
+     * Each request is its own POST, so the runtime cannot match a later
+     * notifications/cancelled to it. The relay can: it keeps the POST of every
+     * request in progress and aborts it when the client cancels that request.
+     * The runtime ends the tool's own signal when its HTTP request goes away.
+     */
+    const inProgress = new Map();
     let current;
     let locating;
     /** The address to use, after `failed` (if given) did not answer. */
@@ -71,6 +78,15 @@ export async function relaySharedRuntime(transport, { dir, startRuntime, startTi
     };
     const forward = async (message) => {
         const id = "method" in message && "id" in message ? message.id : undefined;
+        if ("method" in message && message.method === "notifications/cancelled") {
+            const target = message.params?.requestId;
+            if (target !== undefined)
+                inProgress.get(target)?.abort();
+        }
+        const cancelled = new AbortController();
+        if (id !== undefined)
+            inProgress.set(id, cancelled);
+        const signal = AbortSignal.any([disconnected.signal, cancelled.signal]);
         let answered = false;
         const reply = (out) => {
             if (id !== undefined && "id" in out && out.id === id && !("method" in out))
@@ -82,7 +98,7 @@ export async function relaySharedRuntime(transport, { dir, startRuntime, startTi
             for (let attempt = 1;; attempt += 1) {
                 const address = await locate(failed);
                 try {
-                    await post(address, message, disconnected.signal, reply);
+                    await post(address, message, signal, reply);
                     if (id !== undefined && !answered)
                         throw new Error("the runtime ended the call without answering it");
                     return;
@@ -98,10 +114,17 @@ export async function relaySharedRuntime(transport, { dir, startRuntime, startTi
         catch (error) {
             if (disconnected.signal.aborted)
                 return;
+            // The client cancelled this request: it is no longer waiting for an answer.
+            if (cancelled.signal.aborted)
+                return;
             log("ERROR", "Relaying to the shared runtime failed", error);
             if (id !== undefined && !answered)
                 reply(failure(id, error));
         }
+        finally {
+            if (id !== undefined && inProgress.get(id) === cancelled)
+                inProgress.delete(id);
+        }
     };
     transport.onmessage = (message) => { void forward(message); };
     transport.onclose = () => disconnected.abort();

package/build/server.js

@@ -9,7 +9,7 @@ import { D2LApiClient } from "./api/index.js";
 import { trackToolActivity } from "./utils/dev-activity.js";
 import { log } from "./utils/logger.js";
 import { envCredentialRejectedMessage, ENV_COOKIE_REJECTED } from "./utils/commands.js";
-import { registerGetMyCourses, registerGetUpcomingDueDates, registerGetCalendarEvents, registerGetMyGrades, registerGetAnnouncements, registerGetAssignments, registerGetAssignmentRubric, registerGetAssignmentFiles, registerGetAnnouncementFiles, registerGetCourseContent, registerDownloadFile, COURSE_FILE_NEEDS_COOKIE, registerGetClasslistEmails, registerGetRoster, registerGetMyGroups, registerGetAvailableGroups, registerGetSyllabus, registerGetDiscussions, registerGetVideoTranscript, registerGetServerInfo, registerSearchCourse, registerSignOut, registerSignIn, registerGetDropboxFolders, registerGetDropboxSubmissions, registerGetDropboxUserSubmissions, registerGetDropboxFeedback, registerGetRubricsForObject, registerDownloadDropboxSubmissionFile, acceptOmittedArguments, relayMfaChallenges, currentMfaAnnouncer, } from "./tools/index.js";
+import { registerGetMyCourses, registerGetUpcomingDueDates, registerGetCalendarEvents, registerGetMyGrades, registerGetAnnouncements, registerGetAssignments, registerGetAssignmentRubric, registerGetAssignmentFiles, registerGetAnnouncementFiles, registerGetCourseContent, registerDownloadFile, COURSE_FILE_NEEDS_COOKIE, registerDownloadCourseFiles, registerGetClasslistEmails, registerGetRoster, registerGetMyGroups, registerGetAvailableGroups, registerGetSyllabus, registerGetDiscussions, registerGetVideoTranscript, registerGetServerInfo, registerSearchCourse, registerSignOut, registerSignIn, registerGetDropboxFolders, registerGetDropboxSubmissions, registerGetDropboxUserSubmissions, registerGetDropboxFeedback, registerGetRubricsForObject, registerDownloadDropboxSubmissionFile, acceptOmittedArguments, relayMfaChallenges, currentMfaAnnouncer, } from "./tools/index.js";
 import { registerWeeklyBriefingPrompt, registerGradeAuditPrompt, registerStudyPlannerPrompt, registerCourseSummaryPrompt, } from "./prompts/index.js";
 /**
  * Build the shared runtime. Nothing here reaches Brightspace: API versions
@@ -85,6 +85,7 @@ export function createMcpServer(runtime) {
     registerGetAnnouncementFiles(server, apiClient);
     registerGetCourseContent(server, apiClient);
     registerDownloadFile(server, apiClient, config.envSessionCookie ? ENV_COOKIE_REJECTED : config.envAccessToken ? COURSE_FILE_NEEDS_COOKIE : undefined);
+    registerDownloadCourseFiles(server, apiClient);
     registerGetClasslistEmails(server, apiClient);
     registerGetRoster(server, apiClient);
     registerGetMyGroups(server, apiClient);
@@ -103,7 +104,7 @@ export function createMcpServer(runtime) {
     registerGetDropboxFeedback(server, apiClient);
     registerGetRubricsForObject(server, apiClient);
     registerDownloadDropboxSubmissionFile(server, apiClient);
-    log("DEBUG", "MCP tools registered (28 tools)");
+    log("DEBUG", "MCP tools registered (29 tools)");
     // Register MCP prompts — surfaced in clients (e.g. Claude Desktop) as a
     // picker, distinct from tools. Each one is a canned user message that
     // names our own tools by name, so a client with no idea what to ask

package/build/sync/cleanup.js

@@ -0,0 +1,61 @@
+/**
+ * Purdue Brightspace MCP Server
+ * Copyright (c) 2026 Rohan Muppa. All rights reserved.
+ * Licensed under MIT — see LICENSE file for details.
+ */
+import fs from "node:fs/promises";
+import path from "node:path";
+import { log } from "../utils/logger.js";
+/** A partial download older than this belongs to a run that was killed, not one still going. */
+export const PARTIAL_MAX_AGE_MS = 60 * 60 * 1000;
+/** Exactly what secureStreamDownload names its temporary file. */
+const PARTIAL = /^\.download-[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}\.part$/;
+const MAX_VISITED = 50_000;
+const MAX_DEPTH = 64;
+/**
+ * Remove the `.download-<uuid>.part` files a killed run (kill -9, power loss)
+ * left in the destination tree. Only that exact name, only regular files
+ * older than an hour, and nothing is followed through a symbolic link or into
+ * a hidden folder. Returns how many were removed.
+ */
+export async function removeStalePartials(root, now = Date.now()) {
+    let removed = 0;
+    let visited = 0;
+    const walk = async (dir, depth) => {
+        if (depth > MAX_DEPTH || visited > MAX_VISITED)
+            return;
+        let entries;
+        try {
+            entries = await fs.readdir(dir, { withFileTypes: true });
+        }
+        catch {
+            return;
+        }
+        for (const entry of entries) {
+            if (++visited > MAX_VISITED)
+                return;
+            const full = path.join(dir, entry.name);
+            if (entry.isDirectory()) {
+                if (!entry.name.startsWith("."))
+                    await walk(full, depth + 1);
+            }
+            else if (entry.isFile() && PARTIAL.test(entry.name)) {
+                try {
+                    // lstat again: the entry type above could have changed, and a link is never touched.
+                    const stat = await fs.lstat(full);
+                    if (stat.isFile() && now - stat.mtimeMs > PARTIAL_MAX_AGE_MS) {
+                        await fs.rm(full, { force: true });
+                        removed++;
+                    }
+                }
+                catch {
+                    // Gone already, or not ours to remove.
+                }
+            }
+        }
+    };
+    await walk(root, 0);
+    if (removed > 0)
+        log("INFO", `Course sync: removed ${removed} partial download(s) left by an interrupted run`);
+    return removed;
+}