brightspace-mcp-server 3.13.11 → 3.13.14

package/build/tools/get-announcements.js

@@ -10,6 +10,7 @@ import { toolResponse, toolResponseWithNotice, sanitizeError, isAuthUnavailable,
 import { log } from "../utils/logger.js";
 import { applyCourseFilter } from "../utils/course-filter.js";
 import { matchesModifiedSince } from "../utils/modified-since.js";
+import { convertHtmlToMarkdown } from "../utils/html-converter.js";
 function pageInfo(offset, count, returned, total, unavailableCourseIds = []) {
     const nextOffset = total !== null && offset + returned < total ? offset + returned : null;
     return { offset, count, returned, total, nextOffset, truncated: nextOffset !== null,
@@ -73,6 +74,20 @@ export async function fetchCourseNews(apiClient, courseId) {
         ttl: DEFAULT_CACHE_TTLS.announcements,
     });
 }
+/**
+ * The announcement text a reader should see. D2L's RichText.Text is only "the
+ * plaintext form of the text": it carries no link targets, and the API makes
+ * no promise that it is present or equal to Html. The Html is the post as the
+ * instructor wrote it, so it leads (as markdown, keeping links, lists and
+ * tables); Text is the fallback for a text-only post or Html with nothing
+ * readable in it.
+ */
+export function newsBodyText(body) {
+    if (!body)
+        return "";
+    const markdown = body.Html ? convertHtmlToMarkdown(body.Html).markdown.trim() : "";
+    return markdown || body.Text || "";
+}
 /**
  * Map a raw D2L news item to a clean announcement object.
  */
@@ -85,7 +100,7 @@ export function mapNewsItem(item) {
     return {
         id: item.Id,
         title: item.Title,
-        body: item.Body?.Text ?? "",
+        body: newsBodyText(item.Body),
         createdBy: item.CreatedBy?.DisplayName ?? "Unknown",
         date: effectiveDate(item),
         isPinned: item.IsPinned,
@@ -100,7 +115,7 @@ export function registerGetAnnouncements(server, apiClient, config) {
     server.registerTool("get_announcements", {
         title: "Get Announcements",
         annotations: { readOnlyHint: true },
-        description: "Fetch announcements from one course with courseId, or all courses when omitted, newest first. count is 1-50 per page. Continue with offset= pagination.nextOffset until it is null. Pagination metadata is in a separate JSON text block for array responses, or the pagination field with modifiedSince. A false coverageComplete means some courses were not fetched: retry from offset 0 after recovery. Offsets reflect the current filtered list; new posts can shift pages. Attachments are listed per announcement; fetch them with download_file (newsId + fileId) or read them with get_announcement_files.",
+        description: "Fetch announcements from one course with courseId, or all courses when omitted, newest first. count is 1-50 per page. Each body is the full announcement text as markdown (links kept); pagination.truncated only means more announcements remain, never that a body was cut. Continue with offset= pagination.nextOffset until it is null. Pagination metadata is in a separate JSON text block for array responses, or the pagination field with modifiedSince. A false coverageComplete means some courses were not fetched: retry from offset 0 after recovery. Offsets reflect the current filtered list; new posts can shift pages. Attachments are listed per announcement; fetch them with download_file (newsId + fileId) or read them with get_announcement_files.",
         inputSchema: GetAnnouncementsSchema,
     }, async (args) => {
         try {

package/build/tools/get-upcoming-due-dates.js

@@ -101,6 +101,12 @@ async function fetchCourseDueItems(apiClient, baseUrl, course, from, to) {
                 continue;
             if (!folder.DueDate)
                 continue;
+            // Check the window before resolving the link: a group folder's link
+            // costs group-membership requests, wasted on an item the handler's
+            // window filter would drop anyway (issue #230).
+            const due = new Date(folder.DueDate).getTime();
+            if (!Number.isFinite(due) || due < from || due > to)
+                continue;
             const link = await assignmentLink(folder);
             items.push({
                 type: "assignment",

package/build/utils/html-converter.js

@@ -15,6 +15,15 @@ function getTurndownService() {
         turndownService = new TurndownService({
             headingStyle: "atx",
             codeBlockStyle: "fenced",
+            // Turndown routes any element with no text (an <object>, say) to the
+            // blank rule before the rules above are consulted, so an embedded
+            // object has to be recognised here to keep its address.
+            blankReplacement: (_content, node) => {
+                const el = node;
+                if (EMBEDDED_TAGS.includes(el.nodeName))
+                    return embeddedLink(el);
+                return node.isBlock ? "\n\n" : "";
+            },
         });
         // Turndown has no rule for <script>, <style> or <noscript>, so it falls
         // back to emitting their text: a D2L description carrying an inline
@@ -81,6 +90,14 @@ function getTurndownService() {
                 return `[${content}](${cleaned}${titlePart})`;
             },
         });
+        // Embedded frames and media carry no text, so Turndown dropped them
+        // whole: a post whose details sat in an embedded PDF or video read as if
+        // that part did not exist. Keep the address as a link so the model can
+        // see that something was embedded and where it lives.
+        turndownService.addRule("embeddedContent", {
+            filter: ["iframe", "object", "embed", "video", "audio"],
+            replacement: (_content, node) => embeddedLink(node),
+        });
         turndownService.addRule("imageSessionTokens", {
             filter: "img",
             replacement: (_content, node) => {
@@ -95,6 +112,16 @@ function getTurndownService() {
     }
     return turndownService;
 }
+const EMBEDDED_TAGS = ["IFRAME", "OBJECT", "EMBED", "VIDEO", "AUDIO"];
+/** A link to an embedded frame or media file; empty when it has no usable address. */
+function embeddedLink(el) {
+    const raw = el.getAttribute("src") ?? el.getAttribute("data") ?? "";
+    if (!raw.trim() || isUnsafeHref(raw))
+        return "";
+    const label = (el.getAttribute("title") || el.getAttribute("aria-label") || "Embedded content")
+        .replace(/[\[\]]/g, "");
+    return `[${label}](${stripD2lSessionParams(raw).replace(/([()])/g, "\\$1")})`;
+}
 /** True for `javascript:`/`data:` hrefs -- never a legitimate content link. */
 function isUnsafeHref(href) {
     return /^\s*(javascript|data):/i.test(href);

package/package.json

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