brightspace-mcp-server 5.1.13 → 5.1.14

package/build/utils/config.js

@@ -102,13 +102,11 @@ export async function loadConfig({ credentials = true } = {}) {
         ?? positiveSeconds(store?.tokenTtl, "tokenTtl in config.json")
         ?? 3600;
     // Resolve includeCourseIds: env > store > undefined
-    const includeCourseIds = process.env.D2L_INCLUDE_COURSES
-        ? process.env.D2L_INCLUDE_COURSES.split(',').map(s => parseInt(s.trim(), 10)).filter(n => !isNaN(n))
-        : store?.includeCourses;
+    const includeCourseIds = courseIdList(process.env.D2L_INCLUDE_COURSES, "D2L_INCLUDE_COURSES")
+        ?? store?.includeCourses;
     // Resolve excludeCourseIds: env > store > undefined
-    const excludeCourseIds = process.env.D2L_EXCLUDE_COURSES
-        ? process.env.D2L_EXCLUDE_COURSES.split(',').map(s => parseInt(s.trim(), 10)).filter(n => !isNaN(n))
-        : store?.excludeCourses;
+    const excludeCourseIds = courseIdList(process.env.D2L_EXCLUDE_COURSES, "D2L_EXCLUDE_COURSES")
+        ?? store?.excludeCourses;
     // Resolve activeOnly: env > store > default (true)
     const activeOnly = envBoolean(process.env.D2L_ACTIVE_ONLY, "D2L_ACTIVE_ONLY")
         ?? store?.activeOnly
@@ -195,6 +193,30 @@ function positiveSeconds(value, source) {
     console.error(`[config] Ignoring ${source}=${JSON.stringify(value)}: expected a positive whole number of seconds`);
     return undefined;
 }
+/**
+ * A comma-separated list of course (org unit) ids. parseInt read "123x" as
+ * 123 and dropped "CS180" without a word, so a whitelist of course codes
+ * became empty and an empty whitelist filters nothing: every course showed.
+ * Each entry must be a positive whole number or it is ignored with a warning,
+ * and a list with no valid id is ignored outright so config.json applies, as
+ * positiveSeconds does. Empty entries from a stray comma are skipped quietly.
+ */
+function courseIdList(value, source) {
+    const entries = (value ?? "").split(",").map((entry) => entry.trim()).filter((entry) => entry.length > 0);
+    if (entries.length === 0)
+        return undefined;
+    const ids = [];
+    for (const entry of entries) {
+        if (/^\d+$/.test(entry) && Number(entry) > 0)
+            ids.push(Number(entry));
+        else
+            console.error(`[config] Ignoring ${JSON.stringify(entry)} in ${source}: expected a numeric course id`);
+    }
+    if (ids.length > 0)
+        return ids;
+    console.error(`[config] Ignoring ${source}=${JSON.stringify(value)}: no valid course id`);
+    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

package/package.json

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