brightspace-mcp-server 3.7.4 → 3.7.6

package/build/api/client.js

@@ -8,7 +8,7 @@ import { TTLCache } from "./cache.js";
 import { TokenBucket } from "./rate-limiter.js";
 import { discoverVersions } from "./version-discovery.js";
 import { ApiError, RateLimitError, NetworkError } from "./errors.js";
-import { withRetry, isRetryableFailure, retryAfterMsFrom } from "./retry.js";
+import { withRetry, isRetryableFailure, retryAfterMsFrom, parseRetryAfter, } from "./retry.js";
 import { log } from "../utils/logger.js";
 import { AUTH_COMMAND } from "../utils/commands.js";
 /** An ordinary course HTML link to the login page is not an expired session. */
@@ -306,9 +306,7 @@ export class D2LApiClient {
             }
             // Handle 429 rate limiting
             if (response.status === 429) {
-                const retryAfter = response.headers.get("Retry-After");
-                const retryAfterSeconds = retryAfter ? parseInt(retryAfter, 10) : undefined;
-                throw new RateLimitError(path, retryAfterSeconds);
+                throw new RateLimitError(path, parseRetryAfter(response.headers.get("Retry-After")));
             }
             // Handle 403 (common for past-semester courses)
             if (response.status === 403) {
@@ -371,9 +369,7 @@ export class D2LApiClient {
             }
             // Handle 429 rate limiting
             if (response.status === 429) {
-                const retryAfter = response.headers.get("Retry-After");
-                const retryAfterSeconds = retryAfter ? parseInt(retryAfter, 10) : undefined;
-                throw new RateLimitError(path, retryAfterSeconds);
+                throw new RateLimitError(path, parseRetryAfter(response.headers.get("Retry-After")));
             }
             // Handle 403 (common for past-semester courses or no access)
             if (response.status === 403) {

package/build/api/retry.js

@@ -40,6 +40,28 @@ export function isRetryableFailure(error) {
         return error.status >= 500 && error.status <= 599;
     return error instanceof NetworkError;
 }
+/**
+ * Read a Retry-After header as whole seconds, or undefined when it asks for
+ * no wait. RFC 9110 allows delta-seconds ("120") or an HTTP-date; a date is
+ * measured from `now` and rounded up. Anything else is ignored so the normal
+ * backoff applies.
+ */
+export function parseRetryAfter(value, now = Date.now()) {
+    const trimmed = value?.trim();
+    if (!trimmed)
+        return undefined;
+    let seconds;
+    if (/^\d+$/.test(trimmed)) {
+        seconds = Number(trimmed);
+    }
+    else {
+        const at = Date.parse(trimmed);
+        if (Number.isNaN(at))
+            return undefined;
+        seconds = Math.ceil((at - now) / 1000);
+    }
+    return seconds > 0 ? seconds : undefined;
+}
 /** The Retry-After a 429 carried, in milliseconds, or undefined. */
 export function retryAfterMsFrom(error) {
     if (error instanceof RateLimitError && typeof error.retryAfter === "number" && error.retryAfter > 0) {

package/build/utils/config.js

@@ -26,9 +26,9 @@ export async function loadConfig() {
             ? expandTilde(store.sessionDir)
             : path.join(os.homedir(), ".d2l-session");
     // Code-entry and other interactive MFA methods need a visible browser.
-    const headless = process.env.D2L_HEADLESS !== undefined
-        ? process.env.D2L_HEADLESS !== "false"
-        : store?.headless ?? true;
+    const headless = envBoolean(process.env.D2L_HEADLESS, "D2L_HEADLESS")
+        ?? store?.headless
+        ?? true;
     // Resolve tokenTtl: env > store > default (3600)
     const tokenTtl = positiveSeconds(process.env.D2L_TOKEN_TTL, "D2L_TOKEN_TTL")
         ?? positiveSeconds(store?.tokenTtl, "tokenTtl in config.json")
@@ -42,10 +42,9 @@ export async function loadConfig() {
         ? process.env.D2L_EXCLUDE_COURSES.split(',').map(s => parseInt(s.trim(), 10)).filter(n => !isNaN(n))
         : store?.excludeCourses;
     // Resolve activeOnly: env > store > default (true)
-    let activeOnly = store?.activeOnly ?? true;
-    if (process.env.D2L_ACTIVE_ONLY !== undefined) {
-        activeOnly = process.env.D2L_ACTIVE_ONLY !== 'false';
-    }
+    const activeOnly = envBoolean(process.env.D2L_ACTIVE_ONLY, "D2L_ACTIVE_ONLY")
+        ?? store?.activeOnly
+        ?? true;
     const configuredUrl = new URL(process.env.D2L_BASE_URL || store?.baseUrl || "https://purdue.brightspace.com");
     if (configuredUrl.protocol !== "https:" || configuredUrl.username || configuredUrl.password) {
         throw new Error("The Brightspace URL must be an HTTPS school URL without embedded credentials.");
@@ -93,6 +92,24 @@ function positiveSeconds(value, source) {
     console.error(`[config] Ignoring ${source}=${JSON.stringify(value)}: expected a positive whole number of seconds`);
     return undefined;
 }
+/**
+ * An on/off environment variable. Comparing against the exact string "false"
+ * read "0", "no", "False" and a typo as true, so D2L_HEADLESS=0 kept the
+ * browser hidden from a user who needed it to enter an MFA code. An empty
+ * value counts as unset, and anything unrecognized is ignored with a warning
+ * so config.json or the default applies, as positiveSeconds does.
+ */
+function envBoolean(value, source) {
+    const text = value?.trim().toLowerCase();
+    if (!text)
+        return undefined;
+    if (["true", "1", "yes", "on"].includes(text))
+        return true;
+    if (["false", "0", "no", "off"].includes(text))
+        return false;
+    console.error(`[config] Ignoring ${source}=${JSON.stringify(value)}: expected true or false`);
+    return undefined;
+}
 function expandTilde(filePath) {
     if (filePath.startsWith("~")) {
         return path.join(os.homedir(), filePath.slice(1));

package/package.json

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