@nbakka/mcp-appium 2.0.54 → 2.0.55

package/lib/server.js

@@ -580,7 +580,7 @@ tool(
 
 
 // ----------------------
-// TOOL: Generate Manual Test Cases from PDF + JIRA
+// FIXED TOOL: Generate Manual Test Cases from PDF + JIRA
 // ----------------------
 tool(
   "upload_pdf_to_openai",
@@ -602,7 +602,9 @@ tool(
       const client = new OpenAI({ apiKey: openaiKey });
 
       // Validate file exists
-      if (!await fs.access(pdfPath).then(() => true).catch(() => false)) {
+      try {
+        await fs.access(pdfPath);
+      } catch (error) {
         throw new Error(`PDF file not found at path: ${pdfPath}`);
       }
 
@@ -611,9 +613,12 @@ tool(
 
       console.log(`📄 Processing ${fileName} for test case generation...`);
 
+      // Use the require() fs for createReadStream (CommonJS style)
+      const fsSync = require('fs');
+
       // Upload PDF to OpenAI
       const fileUpload = await client.files.create({
-        file: fs.createReadStream(pdfPath),
+        file: fsSync.createReadStream(pdfPath), // Use fsSync.createReadStream
         purpose: "assistants"
       });
 
@@ -629,7 +634,7 @@ tool(
         - Edge cases and negative scenarios
         - Cross-browser/device compatibility tests
         - Accessibility testing scenarios`,
-        model: "gpt-4o", // Best model for complex reasoning
+        model: "gpt-4o",
         tools: [{ type: "file_search" }],
         tool_resources: {
           file_search: {
@@ -681,7 +686,7 @@ Please structure your response as follows:
 [Same format as above]
 
 **IMPORTANT GUIDELINES:**
-1. Create 15-25 test cases total covering all aspects
+1. Create 15-20 test cases total covering all aspects
 2. Reference specific UI elements from the PDF design
 3. Include both positive and negative test scenarios
 4. Consider different user roles if applicable
@@ -714,7 +719,7 @@ Generate detailed test cases now based on the PDF design and JIRA requirements.
       // Wait for completion with progress tracking
       let runStatus = await client.beta.threads.runs.retrieve(thread.id, run.id);
       let attempts = 0;
-      const maxAttempts = 60; // 60 seconds for complex test case generation
+      const maxAttempts = 60;
 
       while (runStatus.status !== 'completed' && attempts < maxAttempts) {
         if (runStatus.status === 'failed') {
@@ -741,7 +746,7 @@ Generate detailed test cases now based on the PDF design and JIRA requirements.
         .map(msg => msg.content[0]?.text?.value)
         .join('\n\n');
 
-      // Parse and structure the response
+      // Parse test case count
       const testCaseCount = (testCases.match(/\*\*TC-/g) || []).length;
 
       // Clean up assistant
@@ -751,7 +756,7 @@ Generate detailed test cases now based on the PDF design and JIRA requirements.
         console.warn('⚠️ Assistant cleanup failed:', cleanupError.message);
       }
 
-      // Save test cases to file for easy reference
+      // Save test cases to file
       const timestamp = new Date().toISOString().replace(/[:.]/g, '-');
       const outputFileName = `test_cases_${timestamp.substring(0, 19)}.md`;
       const outputPath = path.join(path.dirname(pdfPath), outputFileName);
@@ -782,96 +787,46 @@ ${testCases}
 
       console.log(`✅ Test cases generated and saved!`);
 
-      return {
-        success: true,
-        message: `✅ Successfully generated ${testCaseCount} manual test cases!`,
-        results: {
-          testCases: testCases,
-          summary: {
-            totalTestCases: testCaseCount,
-            categories: [
-              "Functional Test Cases",
-              "UI/UX Test Cases",
-              "Edge Case & Negative Test Cases",
-              "Compatibility Test Cases",
-              "Accessibility Test Cases"
-            ],
-            generatedFrom: {
-              jiraSummary: jiraSummary,
-              designFile: fileName,
-              fileSize: `${(stats.size / 1024).toFixed(2)} KB`
-            }
-          },
-          outputs: {
-            savedToFile: outputPath,
-            fileName: outputFileName,
-            generatedAt: new Date().toISOString()
-          },
-          execution: {
-            processingTime: `${attempts} seconds`,
-            model: "gpt-4o",
-            pdfAnalyzed: true
-          }
-        }
-      };
+      // CRITICAL: Return only a STRING, not an object
+      // The MCP wrapper will handle the proper formatting
+      return `✅ Successfully generated ${testCaseCount} manual test cases!
+
+📁 Test cases saved to: ${outputFileName}
+📂 Full path: ${outputPath}
+
+📊 Generation Summary:
+- Total Test Cases: ${testCaseCount}
+- Categories: Functional, UI/UX, Edge Cases, Compatibility, Accessibility
+- Source File: ${fileName} (${(stats.size / 1024).toFixed(2)} KB)
+- Processing Time: ${attempts} seconds
+- Model Used: gpt-4o
+
+🎯 Test Case Preview:
+${testCases.substring(0, 1000)}...
+
+[Complete test cases saved to markdown file for easy reference and sharing]`;
 
     } catch (err) {
       console.error("❌ Test Case Generation Error:", err);
 
-      let errorMessage = "❌ Failed to generate test cases: ";
+      // Return only error string, not object
+      let errorMessage = `❌ Failed to generate test cases: ${err.message}`;
 
       if (err.message?.includes('quota')) {
-        errorMessage += "OpenAI quota exceeded. Check billing.";
+        errorMessage += "\n\n💡 Troubleshooting: OpenAI quota exceeded. Check your billing at platform.openai.com";
       } else if (err.message?.includes('timeout')) {
-        errorMessage += "Generation took too long. Try with smaller PDF.";
+        errorMessage += "\n\n💡 Troubleshooting: Generation took too long. Try with a smaller PDF file.";
       } else if (err.message?.includes('file')) {
-        errorMessage += "PDF processing error. Check file format.";
+        errorMessage += "\n\n💡 Troubleshooting: PDF processing error. Check file format and accessibility.";
       } else if (err.message?.includes('API key')) {
-        errorMessage += "Invalid OpenAI API key.";
-      } else {
-        errorMessage += err.message || "Unknown error occurred.";
+        errorMessage += "\n\n💡 Troubleshooting: Invalid OpenAI API key. Check ~/Desktop/openai.json file.";
       }
 
-      return {
-        success: false,
-        message: errorMessage,
-        error: {
-          message: err.message,
-          timestamp: new Date().toISOString(),
-          attempted: {
-            jiraSummary: jiraSummary,
-            pdfPath: pdfPath
-          }
-        },
-        troubleshooting: [
-          "Verify OpenAI API key in ~/Desktop/openai.json",
-          "Check PDF file exists and is readable",
-          "Ensure sufficient OpenAI credits",
-          "Try with smaller PDF if timeout occurs"
-        ]
-      };
+      return errorMessage;
     }
   }
 );
 
-// ----------------------
-// USAGE EXAMPLE
-// ----------------------
-
-/*
-const result = await tools.uploadPdfToOpenai({
-  jiraSummary: "User Login Page Implementation",
-  jiraDescription: "Create a responsive login page with email/password fields, remember me checkbox, forgot password link, and social login options. Must support mobile and desktop views.",
-  pdfPath: "/path/to/login-design.pdf"
-});
-
-// Expected Output:
-// - 15-25 detailed manual test cases
-// - Categories: Functional, UI/UX, Edge Cases, Compatibility, Accessibility
-// - Saved to markdown file for easy reference
-// - Structured format ready for test management tools
-*/
-
     return server;
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nbakka/mcp-appium",
-  "version": "2.0.54",
+  "version": "2.0.55",
   "description": "Appium MCP",
   "engines": {
     "node": ">=18"