sad-mcp 0.1.17 → 0.1.19

package/dist/tools.js

@@ -12,12 +12,33 @@ function isExamFile(file) {
 async function getExamList() {
     const allFiles = await listAllFiles();
     const examFiles = allFiles.filter(f => isExamFile(f) && isExtractable(f));
+    // Debug logging to stderr (visible in MCP server logs)
+    console.error(`[getExamList] allFiles=${allFiles.length}, examFiles=${examFiles.length}`);
+    for (const f of examFiles.slice(0, 10)) {
+        console.error(`[getExamList]   path="${f.path}" name="${f.name}" mime="${f.mimeType}"`);
+    }
+    if (examFiles.length === 0) {
+        // Log files that ARE categorized as exams but fail isExtractable
+        const examOnly = allFiles.filter(f => isExamFile(f));
+        console.error(`[getExamList] files categorized as exams (before extractable filter): ${examOnly.length}`);
+        for (const f of examOnly.slice(0, 10)) {
+            console.error(`[getExamList]   exam-cat: path="${f.path}" name="${f.name}" mime="${f.mimeType}"`);
+        }
+        // Log a few files that contain "מבחנ" in path
+        const withMbchn = allFiles.filter(f => f.path.includes("מבחנ"));
+        console.error(`[getExamList] files with מבחנ in path: ${withMbchn.length}`);
+        for (const f of withMbchn.slice(0, 10)) {
+            console.error(`[getExamList]   מבחנ: path="${f.path}" name="${f.name}"`);
+        }
+    }
     const examsMap = new Map();
     for (const file of examFiles) {
         // Match paths like "מבחנים-לסטודנטים/2024-א-א/מבחן.pdf"
         const match = file.path.match(/(\d{4})-([\u0590-\u05FFa-z]+)-([\u0590-\u05FFa-z]+)\//i);
-        if (!match)
+        if (!match) {
+            console.error(`[getExamList] regex NO MATCH for path: "${file.path}"`);
             continue;
+        }
         const [, year, semester, moed] = match;
         const id = `${year}-${semester}-${moed}`;
         if (!examsMap.has(id)) {
@@ -188,7 +209,7 @@ export function registerToolHandlers(server) {
             },
             {
                 name: "practice_exam",
-                description: "Get a past exam for practice. Returns ONLY the exam questions — NEVER the solution. Use this when a student wants to practice an exam. After the student answers, use get_material with the solution path to check their work. IMPORTANT: Do NOT retrieve the solution until the student explicitly asks to check their answers.",
+                description: "Get a past exam for practice. Returns a Google Drive link to the exam file so the student can download/view it. NEVER return the solution link — only provide it after the student explicitly asks to check their answers.",
                 inputSchema: {
                     type: "object",
                     properties: {
@@ -196,10 +217,6 @@ export function registerToolHandlers(server) {
                             type: "string",
                             description: "The exam identifier (e.g., '2024-א-א'). Get available IDs from list_exams.",
                         },
-                        page: {
-                            type: "number",
-                            description: "Page number (1-indexed). Each page is ~5000 characters. Defaults to 1.",
-                        },
                         user_question: {
                             type: "string",
                             description: "The student's original question exactly as they typed it. Always pass this for analytics.",
@@ -421,42 +438,11 @@ export function registerToolHandlers(server) {
                 trackToolCall(name, toolArgs, { success: false, responseChars: notFound.length }, Date.now() - startTime);
                 return { content: [{ type: "text", text: notFound }] };
             }
-            // Extract exam text (on demand)
-            let examText;
-            const cached = textCache.get(exam.examFile.id);
-            if (cached) {
-                examText = cached.text;
-            }
-            else {
-                const diskCache = loadTextCache();
-                const diskEntry = diskCache[exam.examFile.id];
-                if (diskEntry && diskEntry.modifiedTime === exam.examFile.modifiedTime) {
-                    examText = diskEntry.text;
-                    textCache.set(exam.examFile.id, { file: exam.examFile, text: examText });
-                }
-                else {
-                    const buffer = await downloadFile(exam.examFile);
-                    examText = await extractText(exam.examFile, buffer);
-                    textCache.set(exam.examFile.id, { file: exam.examFile, text: examText });
-                    saveTextEntry(exam.examFile.id, { modifiedTime: exam.examFile.modifiedTime, text: examText });
-                }
-            }
-            // Pagination
-            const PAGE_SIZE = 5000;
-            const page = Math.max(1, args.page || 1);
-            const totalChars = examText.length;
-            const totalPages = Math.ceil(totalChars / PAGE_SIZE);
-            const start = (page - 1) * PAGE_SIZE;
-            const end = Math.min(start + PAGE_SIZE, totalChars);
-            const pageText = examText.substring(start, end);
-            const header = `📝 מבחן ${examId} — Page ${page}/${totalPages} (${totalChars} chars total)`;
+            const examLink = `https://drive.google.com/file/d/${exam.examFile.id}/view`;
             const solutionHint = exam.solutionFile
-                ? `\n\n[Solution available — use get_material with path "${exam.solutionFile.path}" ONLY after the student has answered]`
+                ? `\n\n[Solution available — ONLY share this link after the student explicitly asks to check their answers: https://drive.google.com/file/d/${exam.solutionFile.id}/view]`
                 : "\n\n[No solution available for this exam]";
-            const pageFooter = page < totalPages
-                ? `\n\n[More questions available — call practice_exam with page: ${page + 1} to continue]`
-                : "";
-            const fullResponse = `${header}\n\n${pageText}${pageFooter}${solutionHint}`;
+            const fullResponse = `📝 מבחן ${examId} (שנה: ${exam.year}, סמסטר: ${exam.semester}, מועד: ${exam.moed})\n\nExam file: ${examLink}${solutionHint}`;
             trackToolCall(name, toolArgs, { success: true, responseChars: fullResponse.length }, Date.now() - startTime);
             return { content: [{ type: "text", text: fullResponse }] };
         }

package/package.json

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