sad-mcp 0.1.7 → 0.1.9

package/dist/tools.js

@@ -46,7 +46,7 @@ export function registerToolHandlers(server) {
         tools: [
             {
                 name: "search_materials",
-                description: "Search across all course materials (lecture transcripts, presentations, exams) for relevant content. Use this when a student asks about a specific topic.",
+                description: "Search across all course materials for a topic. Returns a SHORT summary list of matching files (name, category, match count). To read the actual content, use get_material on the most relevant file(s) from the results.",
                 inputSchema: {
                     type: "object",
                     properties: {
@@ -58,6 +58,20 @@ export function registerToolHandlers(server) {
                     required: ["query"],
                 },
             },
+            {
+                name: "get_material",
+                description: "Get the full text content of a specific course material file. Use this AFTER search_materials to read the content of a relevant file.",
+                inputSchema: {
+                    type: "object",
+                    properties: {
+                        name: {
+                            type: "string",
+                            description: "The file name (or partial name) to retrieve. Matched against file names from search_materials or list_materials results.",
+                        },
+                    },
+                    required: ["name"],
+                },
+            },
             {
                 name: "list_materials",
                 description: "List all available course materials, optionally filtered by category.",
@@ -103,18 +117,38 @@ export function registerToolHandlers(server) {
                 };
             }
             await ensureTextCache();
+            const queryLower = query.toLowerCase();
             const results = [];
             for (const [, { file, text }] of textCache) {
+                const nameMatch = file.name.toLowerCase().includes(queryLower) || file.path.toLowerCase().includes(queryLower);
                 const matches = searchInText(text, query);
-                if (matches.length > 0) {
+                if (matches.length > 0 || nameMatch) {
                     results.push({
                         fileName: file.name,
-                        path: file.path,
                         category: categorizeFile(file),
-                        matches: matches.slice(0, 5), // Top 5 matches per file
+                        matchCount: nameMatch ? matches.length + 100 : matches.length, // Boost file-name matches
+                        preview: matches.length > 0
+                            ? matches[0].line.trim().substring(0, 120)
+                            : `(file name matches "${query}")`,
                     });
                 }
             }
+            // Also search files NOT in textCache (failed extraction) by name
+            const allFiles = await listAllFiles();
+            for (const file of allFiles) {
+                if (textCache.has(file.id))
+                    continue; // Already checked
+                if (file.name.toLowerCase().includes(queryLower) || file.path.toLowerCase().includes(queryLower)) {
+                    results.push({
+                        fileName: file.name,
+                        category: categorizeFile(file),
+                        matchCount: 100,
+                        preview: `(file name matches "${query}" — use get_material to read)`,
+                    });
+                }
+            }
+            // Sort by match count descending (most relevant first)
+            results.sort((a, b) => b.matchCount - a.matchCount);
             if (results.length === 0) {
                 return {
                     content: [
@@ -126,18 +160,79 @@ export function registerToolHandlers(server) {
                 };
             }
             const formatted = results
-                .map((r) => {
-                const matchLines = r.matches
-                    .map((m) => `  Line ${m.lineNumber}: ${m.line.substring(0, 200)}`)
-                    .join("\n");
-                return `📄 ${r.fileName} [${r.category}]\n   Path: ${r.path}\n${matchLines}`;
-            })
-                .join("\n\n");
+                .map((r) => `- ${r.fileName} [${r.category}] (${r.matchCount} matches) — "${r.preview}"`)
+                .join("\n");
+            return {
+                content: [
+                    {
+                        type: "text",
+                        text: `Found "${query}" in ${results.length} file(s). Use get_material to read the most relevant one(s):\n\n${formatted}`,
+                    },
+                ],
+            };
+        }
+        if (name === "get_material") {
+            const queryName = args.name;
+            if (!queryName) {
+                return {
+                    content: [{ type: "text", text: "Error: name parameter is required" }],
+                };
+            }
+            await ensureTextCache();
+            const queryLower = queryName.toLowerCase();
+            // First: check text cache
+            let bestMatch = null;
+            for (const [, entry] of textCache) {
+                if (entry.file.name.toLowerCase().includes(queryLower)) {
+                    bestMatch = entry;
+                    break;
+                }
+            }
+            // Fallback: search all files by name and attempt fresh extraction
+            if (!bestMatch) {
+                const allFiles = await listAllFiles();
+                const matchedFile = allFiles.find(f => f.name.toLowerCase().includes(queryLower));
+                if (matchedFile && isExtractable(matchedFile)) {
+                    try {
+                        const buffer = await downloadFile(matchedFile);
+                        const text = await extractText(matchedFile, buffer);
+                        textCache.set(matchedFile.id, { file: matchedFile, text });
+                        saveTextEntry(matchedFile.id, { modifiedTime: matchedFile.modifiedTime, text });
+                        bestMatch = { file: matchedFile, text };
+                    }
+                    catch {
+                        // Extraction failed — return what we know
+                        return {
+                            content: [
+                                {
+                                    type: "text",
+                                    text: `Found file "${matchedFile.name}" but could not extract its text content. It may be an image-heavy presentation. Try searching for a transcript of the same lecture instead (e.g., search for the lecture name in transcripts).`,
+                                },
+                            ],
+                        };
+                    }
+                }
+            }
+            if (!bestMatch) {
+                return {
+                    content: [
+                        {
+                            type: "text",
+                            text: `No file found matching "${queryName}". Use search_materials or list_materials to find available files.`,
+                        },
+                    ],
+                };
+            }
+            // Truncate very large files
+            const maxLen = 30000;
+            const text = bestMatch.text.length > maxLen
+                ? bestMatch.text.substring(0, maxLen) + "\n...[truncated]"
+                : bestMatch.text;
             return {
                 content: [
                     {
                         type: "text",
-                        text: `Found matches in ${results.length} file(s) for "${query}":\n\n${formatted}`,
+                        text: `📄 ${bestMatch.file.name} [${categorizeFile(bestMatch.file)}]\n\n${text}`,
                     },
                 ],
             };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sad-mcp",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "MCP server for Software Analysis and Design course materials at BGU",
   "type": "module",
   "bin": {