brightspace-mcp-server 5.1.4 → 5.1.5

package/build/api/client.js

@@ -11,7 +11,7 @@ import { ApiError, RateLimitError, NetworkError } from "./errors.js";
 import { withRetry, isRetryableFailure, retryAfterMsFrom, parseRetryAfter, } from "./retry.js";
 import { log } from "../utils/logger.js";
 import { AUTH_COMMAND } from "../utils/commands.js";
-import { devActivity } from "../utils/dev-activity.js";
+import { devActivity, httpOperation } from "../utils/dev-activity.js";
 /**
  * True when a followed redirect ended on this Brightspace's sign-in page. A
  * web route asked with a dead cookie 302s there rather than serving the
@@ -426,8 +426,8 @@ export class D2LApiClient {
         return send(loggedIn);
     }
     /** Buckets one observed HTTP status into statsData.statusClasses. */
-    recordStatus(status) {
-        devActivity("http_response", { status });
+    recordStatus(status, path) {
+        devActivity("http_response", { status, operation: httpOperation(path) });
         const classes = this.statsData.statusClasses;
         if (status >= 200 && status < 300)
             classes["2xx"]++;
@@ -490,7 +490,7 @@ export class D2LApiClient {
                 headers,
                 signal: AbortSignal.timeout(this.timeoutMs),
             });
-            this.recordStatus(response.status);
+            this.recordStatus(response.status, path);
             // Preserve cookie material: a 401 only rejects this access token.
             if (response.status === 401) {
                 throw new ApiError(401, path, "Brightspace rejected the access token.");
@@ -558,7 +558,7 @@ export class D2LApiClient {
                 headers,
                 signal: idle.signal,
             });
-            this.recordStatus(response.status);
+            this.recordStatus(response.status, path);
             // Preserve cookie material for the shared HTTP refresh path.
             if (response.status === 401) {
                 throw new ApiError(401, path, "Brightspace rejected the access token.");

package/build/utils/dev-activity.js

@@ -4,6 +4,46 @@ import { appendFileSync, mkdirSync, statSync } from "node:fs";
 import { join } from "node:path";
 import { AUTH_PHASES } from "../auth/auth-phases.js";
 import { ERROR_KINDS, errorKind, resultErrorKind } from "./error-kind.js";
+const API = String.raw `^/d2l/api/l[pe]/[^/]+/`;
+const COURSE = API + String.raw `\d+/`;
+/** First match wins; a path that matches none is "other". Matched on the path alone, query dropped. */
+const HTTP_OPERATIONS = [
+    ["versions", "^/d2l/api/versions/"],
+    ["whoami", API + "users/whoami"],
+    ["enrollments", API + "enrollments/"],
+    ["groups", COURSE + String.raw `groupcategories/\d+/groups/`],
+    ["groupCategories", COURSE + "groupcategories/"],
+    ["ltiLinks", API + "lti/link/"],
+    ["quizList", COURSE + "quizzes/?$"],
+    ["quizAttempts", COURSE + String.raw `quizzes/\d+/attempts/`],
+    ["quiz", COURSE + "quizzes/"],
+    ["dropboxFolders", COURSE + "dropbox/folders/?$"],
+    ["dropboxSubmissions", COURSE + String.raw `dropbox/folders/\d+/submissions/`],
+    ["dropboxFeedback", COURSE + String.raw `dropbox/folders/\d+/feedback/`],
+    ["dropboxAttachment", COURSE + String.raw `dropbox/folders/\d+/attachments/`],
+    ["dropboxFolder", COURSE + "dropbox/folders/"],
+    ["dropboxFeedbackPage", "^/d2l/lms/dropbox/user/folder_user_view_feedback"],
+    ["rubrics", COURSE + "rubrics"],
+    ["grades", COURSE + "grades/"],
+    ["contentFile", COURSE + String.raw `content/topics/\d+/file`],
+    ["content", COURSE + "content/"],
+    ["newsAttachment", COURSE + String.raw `news/\d+/attachments/`],
+    ["news", COURSE + "news/"],
+    ["discussions", COURSE + "discussions/"],
+    ["overviewAttachment", COURSE + "overview/attachment"],
+    ["overview", COURSE + "overview"],
+    ["classlist", COURSE + "classlist/"],
+    ["quickLink", "^/d2l/common/dialogs/quickLink/"],
+    // The LTI launch page a video quickLink leads to (get_video_transcript).
+    ["ltiLaunch", String.raw `^/d2l/le/lti/\d+/toolLaunch/`],
+];
+const OPERATION_PATTERNS = HTTP_OPERATIONS.map(([name, pattern]) => [name, new RegExp(pattern)]);
+const OPERATION_NAMES = [...HTTP_OPERATIONS.map(([name]) => name), "other"];
+/** Names what a request was for without keeping its endpoint, IDs or query. */
+export function httpOperation(path) {
+    const bare = path.split(/[?#]/, 1)[0];
+    return OPERATION_PATTERNS.find(([, pattern]) => pattern.test(bare))?.[0] ?? "other";
+}
 const calls = new AsyncLocalStorage();
 const runId = randomUUID();
 const MAX_BYTES = 5 * 1024 * 1024;
@@ -45,6 +85,8 @@ function write(event, fields, context = calls.getStore(), idleMs, sinceSuccessMs
         record.phase = fields.phase;
     if (ERROR_KINDS.includes(fields.errorKind ?? ""))
         record.errorKind = fields.errorKind;
+    if (OPERATION_NAMES.includes(fields.operation ?? ""))
+        record.operation = fields.operation;
     try {
         mkdirSync(directory, { recursive: true, mode: 0o700 });
         const file = join(directory, `activity-${at.slice(0, 10)}-${process.pid}-${runId}.jsonl`);

package/package.json

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