fixing some things before video

mcp-server.js

@@ -193,10 +193,6 @@ class ReubenOSMCPServer {
                 type: 'string',
                 description: 'Song lyrics (will be used to generate music)',
               },
-              passkey: {
-                type: 'string',
-                description: 'Your passkey for authentication',
-              },
             },
             required: ['title', 'style', 'lyrics'],
           },
@@ -215,12 +211,26 @@ class ReubenOSMCPServer {
                 type: 'string',
                 description: 'Story content/text to be narrated (max 2000 characters for best performance)',
               },
+            },
+            required: ['title', 'content'],
+          },
+        },
+        {
+          name: 'analyze_quiz',
+          description: 'Analyze quiz answers from quiz_answers.json against the quiz.json questions and provide feedback on correctness',
+          inputSchema: {
+            type: 'object',
+            properties: {
               passkey: {
                 type: 'string',
-                description: 'Your passkey for authentication',
+                description: 'Your passkey for accessing the quiz files',
+              },
+              isPublic: {
+                type: 'boolean',
+                description: 'Set to true if quiz files are in public folder. Default: false',
               },
             },
-            required: ['title', 'content'],
+            required: ['passkey'],
           },
         },
       ],
@@ -245,6 +255,8 @@ class ReubenOSMCPServer {
             return await this.generateSongAudio(args);
           case 'generate_story_audio':
             return await this.generateStoryAudio(args);
+          case 'analyze_quiz':
+            return await this.analyzeQuiz(args);
           default:
             throw new Error(`Unknown tool: ${name}`);
         }
@@ -377,6 +389,19 @@ class ReubenOSMCPServer {
         };
       }
 
+      // Ensure content is properly stringified if it's an object
+      let fileContent = content;
+      if (typeof content === 'object' && content !== null) {
+        fileContent = JSON.stringify(content, null, 2);
+      }
+
+      // Handle special characters in content for code files
+      const ext = fileName.split('.').pop().toLowerCase();
+      const isCodeFile = ['dart', 'js', 'ts', 'jsx', 'tsx', 'py', 'java', 'cpp', 'c', 'h', 'hpp', 'cs', 'swift', 'kt', 'go', 'rs', 'rb', 'php'].includes(ext);
+
+      // Ensure content is a string
+      fileContent = String(fileContent);
+
       const response = await fetch(API_ENDPOINT, {
         method: 'POST',
         headers: { 'Content-Type': 'application/json' },
@@ -384,7 +409,7 @@ class ReubenOSMCPServer {
           passkey,
           action: 'save_file',
           fileName,
-          content,
+          content: fileContent,
           isPublic,
         }),
       });
@@ -393,12 +418,13 @@ class ReubenOSMCPServer {
 
       if (response.ok && data.success) {
         const location = isPublic ? 'Public Folder' : `Secure Data (passkey: ${passkey})`;
+        const fileType = isCodeFile ? `(${ext.toUpperCase()} code file)` : '';
 
         return {
           content: [
             {
               type: 'text',
-              text: `✅ File saved: ${fileName}\n📁 Saved to: ${location}`,
+              text: `✅ File saved: ${fileName} ${fileType}\n📁 Saved to: ${location}\n📊 Size: ${(new TextEncoder().encode(fileContent).length / 1024).toFixed(2)} KB`,
             },
           ],
         };
@@ -575,7 +601,9 @@ class ReubenOSMCPServer {
 
   async generateSongAudio(args) {
     try {
-      const { title, style, lyrics, passkey = 'demo' } = args;
+      const { title, style, lyrics } = args;
+      // Use a default passkey internally - users don't need to provide it
+      const passkey = 'voice_default';
 
       if (!title || !style || !lyrics) {
         return {
@@ -624,7 +652,9 @@ class ReubenOSMCPServer {
 
   async generateStoryAudio(args) {
     try {
-      const { title, content, passkey = 'demo' } = args;
+      const { title, content } = args;
+      // Use a default passkey internally - users don't need to provide it
+      const passkey = 'voice_default';
 
       if (!title || !content) {
         return {
@@ -671,6 +701,133 @@ class ReubenOSMCPServer {
     }
   }
 
+  async analyzeQuiz(args) {
+    try {
+      const { passkey, isPublic = false } = args;
+
+      if (!passkey) {
+        return {
+          content: [{ type: 'text', text: '❌ Passkey is required to access quiz files' }],
+        };
+      }
+
+      // Read quiz.json
+      const quizUrl = new URL(API_ENDPOINT);
+      quizUrl.searchParams.set('passkey', passkey);
+      if (isPublic) quizUrl.searchParams.set('isPublic', 'true');
+
+      const quizResponse = await fetch(quizUrl, {
+        method: 'GET',
+        headers: { 'Content-Type': 'application/json' },
+      });
+
+      const quizData = await quizResponse.json();
+
+      if (!quizResponse.ok || !quizData.success) {
+        return {
+          content: [{ type: 'text', text: `❌ Failed to read quiz.json: ${quizData.error || 'Unknown error'}` }],
+        };
+      }
+
+      const quizFile = quizData.files.find(f => f.name === 'quiz.json');
+      if (!quizFile) {
+        return {
+          content: [{ type: 'text', text: '❌ quiz.json not found in storage' }],
+        };
+      }
+
+      // Read quiz_answers.json
+      const answersFile = quizData.files.find(f => f.name === 'quiz_answers.json');
+      if (!answersFile) {
+        return {
+          content: [{ type: 'text', text: '❌ quiz_answers.json not found in storage' }],
+        };
+      }
+
+      // Parse the quiz and answers
+      const quiz = JSON.parse(quizFile.content);
+      const answers = JSON.parse(answersFile.content);
+
+      // Analyze the answers
+      let correctCount = 0;
+      let totalPoints = 0;
+      let maxPoints = 0;
+      const feedback = [];
+
+      quiz.questions.forEach((question, index) => {
+        const userAnswer = answers.answers?.[question.id] || answers[question.id];
+        const points = question.points || 1;
+        maxPoints += points;
+
+        // For multiple choice questions, check if answer matches any correct option
+        if (question.type === 'multiple_choice') {
+          const correctAnswer = question.correctAnswer || question.correct;
+          const isCorrect = userAnswer === correctAnswer ||
+                           (Array.isArray(correctAnswer) && correctAnswer.includes(userAnswer));
+
+          if (isCorrect) {
+            correctCount++;
+            totalPoints += points;
+            feedback.push(`✅ Question ${index + 1} (${question.id}): Correct! (+${points} points)`);
+          } else {
+            feedback.push(`❌ Question ${index + 1} (${question.id}): Incorrect. Your answer: "${userAnswer}"${question.explanation ? `. ${question.explanation}` : ''}`);
+          }
+        } else if (question.type === 'true_false') {
+          const correctAnswer = question.correctAnswer || question.correct;
+          const isCorrect = String(userAnswer).toLowerCase() === String(correctAnswer).toLowerCase();
+
+          if (isCorrect) {
+            correctCount++;
+            totalPoints += points;
+            feedback.push(`✅ Question ${index + 1} (${question.id}): Correct! (+${points} points)`);
+          } else {
+            feedback.push(`❌ Question ${index + 1} (${question.id}): Incorrect. Your answer: "${userAnswer}"${question.explanation ? `. ${question.explanation}` : ''}`);
+          }
+        } else {
+          // For other question types, do a simple string comparison
+          const correctAnswer = question.correctAnswer || question.answer || question.correct;
+          const isCorrect = userAnswer === correctAnswer;
+
+          if (isCorrect) {
+            correctCount++;
+            totalPoints += points;
+            feedback.push(`✅ Question ${index + 1} (${question.id}): Correct! (+${points} points)`);
+          } else {
+            feedback.push(`❌ Question ${index + 1} (${question.id}): Your answer: "${userAnswer}"${question.explanation ? `. ${question.explanation}` : ''}`);
+          }
+        }
+      });
+
+      const percentage = Math.round((totalPoints / maxPoints) * 100);
+      const grade = percentage >= 90 ? 'A' :
+                   percentage >= 80 ? 'B' :
+                   percentage >= 70 ? 'C' :
+                   percentage >= 60 ? 'D' : 'F';
+
+      return {
+        content: [
+          {
+            type: 'text',
+            text: `📊 Quiz Analysis Results for "${quiz.title}"
+
+📈 Score: ${totalPoints}/${maxPoints} points (${percentage}%)
+✅ Correct Answers: ${correctCount}/${quiz.questions.length}
+🎯 Grade: ${grade}
+
+📝 Question Feedback:
+${feedback.join('\n')}
+
+${percentage >= 70 ? '🎉 Great job!' : '📚 Keep studying and try again!'}`,
+          },
+        ],
+      };
+    } catch (error) {
+      return {
+        content: [{ type: 'text', text: `❌ Error analyzing quiz: ${error.message}` }],
+      };
+    }
+  }
+
   async run() {
     const transport = new StdioServerTransport();
     await this.server.connect(transport);

test-mcp-changes.js

@@ -0,0 +1,210 @@
+#!/usr/bin/env node
+// Test script for MCP server changes
+
+import fetch from 'node-fetch';
+
+const BASE_URL = process.env.REUBENOS_URL || 'http://localhost:3000';
+const API_ENDPOINT = `${BASE_URL}/api/mcp-handler`;
+
+async function testFlutterCodeSave() {
+  console.log('Testing Flutter code save...');
+
+  const flutterCode = `import 'package:flutter/material.dart';
+
+void main() {
+  runApp(MyApp());
+}
+
+class MyApp extends StatelessWidget {
+  @override
+  Widget build(BuildContext context) {
+    return MaterialApp(
+      title: 'Flutter Demo',
+      theme: ThemeData(
+        primarySwatch: Colors.blue,
+      ),
+      home: MyHomePage(title: 'Flutter Demo Home Page'),
+    );
+  }
+}
+
+class MyHomePage extends StatefulWidget {
+  MyHomePage({Key? key, required this.title}) : super(key: key);
+  final String title;
+
+  @override
+  _MyHomePageState createState() => _MyHomePageState();
+}
+
+class _MyHomePageState extends State<MyHomePage> {
+  int _counter = 0;
+
+  void _incrementCounter() {
+    setState(() {
+      _counter++;
+    });
+  }
+
+  @override
+  Widget build(BuildContext context) {
+    return Scaffold(
+      appBar: AppBar(
+        title: Text(widget.title),
+      ),
+      body: Center(
+        child: Column(
+          mainAxisAlignment: MainAxisAlignment.center,
+          children: <Widget>[
+            Text(
+              'You have pushed the button this many times:',
+            ),
+            Text(
+              '$_counter',
+              style: Theme.of(context).textTheme.headline4,
+            ),
+          ],
+        ),
+      ),
+      floatingActionButton: FloatingActionButton(
+        onPressed: _incrementCounter,
+        tooltip: 'Increment',
+        child: Icon(Icons.add),
+      ),
+    );
+  }
+}`;
+
+  try {
+    const response = await fetch(API_ENDPOINT, {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({
+        passkey: 'test123',
+        action: 'save_file',
+        fileName: 'main.dart',
+        content: flutterCode,
+        isPublic: false,
+      }),
+    });
+
+    const data = await response.json();
+    console.log('Flutter code save result:', data);
+
+    if (data.success) {
+      console.log('✅ Flutter code saved successfully!');
+
+      // Try to read it back
+      const readUrl = new URL(API_ENDPOINT);
+      readUrl.searchParams.set('passkey', 'test123');
+
+      const readResponse = await fetch(readUrl, {
+        method: 'GET',
+        headers: { 'Content-Type': 'application/json' },
+      });
+
+      const readData = await readResponse.json();
+      if (readData.success) {
+        const file = readData.files.find(f => f.name === 'main.dart');
+        if (file && file.content === flutterCode) {
+          console.log('✅ Flutter code verified - content matches!');
+        } else {
+          console.log('⚠️ Flutter code content mismatch or file not found');
+        }
+      }
+    } else {
+      console.log('❌ Failed to save Flutter code:', data.error);
+    }
+  } catch (error) {
+    console.error('Error testing Flutter code save:', error);
+  }
+}
+
+async function testAnalyzeQuiz() {
+  console.log('\nTesting analyze quiz functionality...');
+
+  // First, create a sample quiz
+  const quiz = {
+    title: "Sample Quiz",
+    description: "Test quiz for analysis",
+    questions: [
+      {
+        id: "q1",
+        type: "multiple_choice",
+        question: "What is 2+2?",
+        options: ["3", "4", "5", "6"],
+        correctAnswer: "4",
+        points: 1
+      },
+      {
+        id: "q2",
+        type: "multiple_choice",
+        question: "What is the capital of France?",
+        options: ["London", "Berlin", "Paris", "Madrid"],
+        correctAnswer: "Paris",
+        points: 1
+      },
+      {
+        id: "q3",
+        type: "true_false",
+        question: "The Earth is flat",
+        correctAnswer: false,
+        points: 1
+      }
+    ]
+  };
+
+  const quizAnswers = {
+    answers: {
+      "q1": "4",
+      "q2": "London",
+      "q3": false
+    }
+  };
+
+  try {
+    // Save quiz.json
+    await fetch(API_ENDPOINT, {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({
+        passkey: 'test123',
+        action: 'save_file',
+        fileName: 'quiz.json',
+        content: JSON.stringify(quiz, null, 2),
+        isPublic: false,
+      }),
+    });
+
+    // Save quiz_answers.json
+    await fetch(API_ENDPOINT, {
+      method: 'POST',
+      headers: { 'Content-Type': 'application/json' },
+      body: JSON.stringify({
+        passkey: 'test123',
+        action: 'save_file',
+        fileName: 'quiz_answers.json',
+        content: JSON.stringify(quizAnswers, null, 2),
+        isPublic: false,
+      }),
+    });
+
+    console.log('✅ Quiz and answers saved. Ready to test analyze function.');
+    console.log('Note: The analyze_quiz function needs to be called through the MCP server.');
+    console.log('Expected results: 2/3 correct (q1 ✅, q2 ❌, q3 ✅)');
+  } catch (error) {
+    console.error('Error setting up quiz test:', error);
+  }
+}
+
+// Run tests
+console.log('Starting MCP server tests...');
+console.log('Using server:', BASE_URL);
+
+testFlutterCodeSave().then(() => {
+  return testAnalyzeQuiz();
+}).then(() => {
+  console.log('\n✅ All tests completed!');
+  console.log('Note: To fully test the analyze_quiz function, use the MCP client with the analyze_quiz tool.');
+}).catch(error => {
+  console.error('Test failed:', error);
+});