Improve README and release notes; fix smithery scanning

Author: PV-Bhat

README.md

@@ -2,11 +2,13 @@
 
 <img src="https://github.com/PV-Bhat/vibe-check-mcp-server/blob/main/Attachments/vibelogov2.png" alt="Logo" width="300"/>
 
-## The Most Widely-Deployed Feedback Layer in the MCP Ecosystem
+## Widely Adopted Across the MCP Ecosystem
 
-> Used in 1,000+ real workflows.
-> Featured across 10+ orchestration platforms.
-> 6.5K+ developers already trust it to prevent agentic cascade errors.
+> 10k+ downloads on PulseMCP
+> 1k+ monthly tool calls on Smithery
+> Available on 12+ orchestration platforms
+> Verified security score 4.3 on MseeP
+> GitHub ★102 • 15 forks • 1 open issue
 
 
 [![Version](https://img.shields.io/badge/version-2.1-blue)](https://github.com/PV-Bhat/vibe-check-mcp-server)
@@ -37,6 +39,7 @@ misalignment, overengineering and wasted cycles.
 - **vibe_learn** – Optional logging tool that builds a history of mistakes and fixes for later review.
 - **History continuity** – Previous vibe_check responses are fed back in by default for a sense of memory.
 - **Multi-LLM flexibility** – Works with Gemini, OpenAI or OpenRouter via simple overrides.
+- **Test coverage** – Vitest suite verifies core logic and guards against regressions.
 
 ## What's New in v2.1
 
@@ -45,6 +48,7 @@ misalignment, overengineering and wasted cycles.
 - Supports Gemini, OpenAI and OpenRouter with easy overrides
 - vibe_learn is now optional and can be disabled entirely
 - Project restructured for stability and online deployment
+- Added Vitest CI with coverage reports
 
 
 

docs/changelog.md

@@ -6,3 +6,5 @@
 - Multi-provider support for Gemini, OpenAI and OpenRouter
 - Optional learning log can be disabled
 - Repository cleanup and CI via GitHub Actions
+- Vitest test suite with coverage
+- Fixed Smithery tool scanning via lazy API key validation

smithery.yaml

@@ -5,8 +5,7 @@ startCommand:
   configSchema:
     # JSON Schema defining the configuration options for the MCP.
     type: object
-    required:
-      - geminiApiKey
+    required: []
     properties:
       geminiApiKey:
         type: string
@@ -27,7 +26,13 @@ startCommand:
   (config) => ({
     command: 'node',
     args: ['build/index.js'],
-    env: { GEMINI_API_KEY: config.geminiApiKey }
+    env: {
+      GEMINI_API_KEY: config.geminiApiKey,
+      OPENAI_API_KEY: config.openaiApiKey,
+      OPENROUTER_API_KEY: config.openrouterApiKey,
+      DEFAULT_LLM_PROVIDER: config.defaultProvider,
+      DEFAULT_MODEL: config.defaultModel
+    }
   })
   exampleConfig:
     geminiApiKey: EXAMPLE_GEMINI_API_KEY_123

src/index.ts

@@ -156,6 +156,18 @@ async function main() {
           );
         }
 
+        // Determine provider for lazy auth validation
+        const provider = (args as any)?.modelOverride?.provider || process.env.DEFAULT_LLM_PROVIDER || 'gemini';
+        if (provider === 'gemini' && !process.env.GEMINI_API_KEY) {
+          throw new McpError(ErrorCode.InvalidParams, 'Missing GEMINI_API_KEY');
+        }
+        if (provider === 'openai' && !process.env.OPENAI_API_KEY) {
+          throw new McpError(ErrorCode.InvalidParams, 'Missing OPENAI_API_KEY');
+        }
+        if (provider === 'openrouter' && !process.env.OPENROUTER_API_KEY) {
+          throw new McpError(ErrorCode.InvalidParams, 'Missing OPENROUTER_API_KEY');
+        }
+
         // Fix type casting error - convert args to the correct interface
         const input: VibeCheckInput = {
           goal: args.goal,
@@ -260,46 +272,15 @@ async function main() {
     return output;
   }
 
-  let buffer = '';
-  process.stdin.on('data', async (data) => {
-    buffer += data.toString();
-    try {
-      const request = JSON.parse(buffer);
-      console.error('Received request:', request);
-
-      if (request.method === 'tools/call' && request.params.name === 'vibe_check') {
-        const result = await vibeCheckTool(request.params.arguments);
-        const response = {
-          jsonrpc: '2.0',
-          id: request.id,
-          result: {
-            content: [
-              {
-                type: 'text',
-                text: formatVibeCheckOutput(result),
-              },
-            ],
-          },
-        };
-        process.stdout.write(JSON.stringify(response));
-      }
-      buffer = ''; // Clear buffer after successful parse
-    } catch (e) {
-      // Incomplete JSON, wait for more data
-    }
-  });
-
   // Set up error handler
   server.onerror = (error) => {
     console.error("[Vibe Check Error]", error);
   };
 
   try {
-    // Connect to transport
     console.error('Connecting to transport...');
-    // const transport = new StdioServerTransport();
-    // await server.connect(transport);
-
+    const transport = new StdioServerTransport();
+    await server.connect(transport);
     console.error('Vibe Check MCP server running');
   } catch (error) {
     console.error("Error connecting to transport:", error);
@@ -311,4 +292,4 @@ async function main() {
 main().catch((error) => {
   console.error("Server startup error:", error);
   process.exit(1);
-});
+});