@mixio-pro/kalaasetu-mcp 2.0.11-beta → 2.1.1-beta

package/src/tools/perplexity.ts

@@ -12,13 +12,13 @@ export const perplexityImages = {
     query: z
       .string()
       .describe(
-        "Descriptive search terms (e.g., 'SpaceX Starship launch photos')."
+        "Descriptive search terms (e.g., 'SpaceX Starship launch photos').",
       ),
     image_domain_filter: z
       .array(z.string())
       .optional()
       .describe(
-        "Filter results by domain. Use 'domain.com' to include, or '-domain.com' to exclude. (e.g., ['wikimedia.org', '-pinterest.com'])."
+        "Filter results by domain. Use 'domain.com' to include, or '-domain.com' to exclude. (e.g., ['wikimedia.org', '-pinterest.com']).",
       ),
     image_format_filter: z
       .array(z.string())
@@ -31,94 +31,102 @@ export const perplexityImages = {
     image_domain_filter?: string[];
     image_format_filter?: string[];
   }) => {
-    return safeToolExecute(async () => {
-      const apiKey = process.env.PERPLEXITY_API_KEY;
-      if (!apiKey) {
-        throw new Error("PERPLEXITY_API_KEY environment variable is not set.");
-      }
-
-      const url = "https://api.perplexity.ai/chat/completions";
-      const headers: Record<string, string> = {
-        Authorization: `Bearer ${apiKey}`,
-        "Content-Type": "application/json",
-        accept: "application/json",
-      };
-
-      const payload = {
-        model: "sonar",
-        messages: [
-          { role: "user", content: `Show me images of ${args.query}` },
-        ],
-        return_images: true,
-        ...(args.image_domain_filter
-          ? { image_domain_filter: args.image_domain_filter }
-          : {}),
-        ...(args.image_format_filter
-          ? { image_format_filter: args.image_format_filter }
-          : {}),
-      };
-
-      const res = await fetch(url, {
-        method: "POST",
-        headers: headers,
-        body: JSON.stringify(payload),
-      });
-
-      if (!res.ok) {
-        const text = await res.text();
-        throw new Error(`Perplexity API request failed: ${res.status} ${text}`);
-      }
-
-      const data = (await res.json()) as any;
-      let content = data.choices?.[0]?.message?.content;
-      const images = (data.images || []) as any[];
-      const citations = (data.citations || []) as string[];
-
-      if (images.length === 0) {
-        return `No direct image URLs found in the API response. The text content was: ${content}`;
-      }
-
-      // Create a map of origin_url -> new 1-based index
-      const originUrlToImageIndex: Record<string, number> = {};
-      images.forEach((img, index) => {
-        if (img.origin_url) {
-          originUrlToImageIndex[img.origin_url] = index + 1;
+    return safeToolExecute(
+      async () => {
+        const apiKey = process.env.PERPLEXITY_API_KEY;
+        if (!apiKey) {
+          throw new Error(
+            "PERPLEXITY_API_KEY environment variable is not set.",
+          );
         }
-      });
 
-      // Create a map of old citation index -> new image index
-      const oldToNewCitationMap: Record<number, number> = {};
-      citations.forEach((citationUrl, index) => {
-        if (originUrlToImageIndex[citationUrl]) {
-          oldToNewCitationMap[index + 1] = originUrlToImageIndex[citationUrl];
+        const url = "https://api.perplexity.ai/chat/completions";
+        const headers: Record<string, string> = {
+          Authorization: `Bearer ${apiKey}`,
+          "Content-Type": "application/json",
+          accept: "application/json",
+        };
+
+        const payload = {
+          model: "sonar",
+          messages: [
+            { role: "user", content: `Show me images of ${args.query}` },
+          ],
+          return_images: true,
+          ...(args.image_domain_filter
+            ? { image_domain_filter: args.image_domain_filter }
+            : {}),
+          ...(args.image_format_filter
+            ? { image_format_filter: args.image_format_filter }
+            : {}),
+        };
+
+        const res = await fetch(url, {
+          method: "POST",
+          headers: headers,
+          body: JSON.stringify(payload),
+        });
+
+        if (!res.ok) {
+          const text = await res.text();
+          throw new Error(
+            `Perplexity API request failed: ${res.status} ${text}`,
+          );
         }
-      });
-
-      // Replace citations in the content
-      if (content && typeof content === "string") {
-        content = content
-          .replace(/\[(\d+)\]/g, (_match, oldIndexStr) => {
-            const oldIndex = parseInt(oldIndexStr, 10);
-            const newIndex = oldToNewCitationMap[oldIndex];
-            if (newIndex) {
-              return `[${newIndex}]`;
-            }
-            return "";
-          })
-          .replace(/(\s\s+)/g, " ")
-          .trim();
-      }
-
-      // Build the final formatted output
-      let output = content + "\n\n--- Images ---\n";
-      images.forEach((img, index) => {
-        output += `${index + 1}. ${img.image_url}\n   (Source: ${
-          img.origin_url
-        })\n`;
-      });
-
-      return output;
-    }, "perplexityImages");
+
+        const data = (await res.json()) as any;
+        let content = data.choices?.[0]?.message?.content;
+        const images = (data.images || []) as any[];
+        const citations = (data.citations || []) as string[];
+
+        if (images.length === 0) {
+          return `No direct image URLs found in the API response. The text content was: ${content}`;
+        }
+
+        // Create a map of origin_url -> new 1-based index
+        const originUrlToImageIndex: Record<string, number> = {};
+        images.forEach((img, index) => {
+          if (img.origin_url) {
+            originUrlToImageIndex[img.origin_url] = index + 1;
+          }
+        });
+
+        // Create a map of old citation index -> new image index
+        const oldToNewCitationMap: Record<number, number> = {};
+        citations.forEach((citationUrl, index) => {
+          if (originUrlToImageIndex[citationUrl]) {
+            oldToNewCitationMap[index + 1] = originUrlToImageIndex[citationUrl];
+          }
+        });
+
+        // Replace citations in the content
+        if (content && typeof content === "string") {
+          content = content
+            .replace(/\[(\d+)\]/g, (_match, oldIndexStr) => {
+              const oldIndex = parseInt(oldIndexStr, 10);
+              const newIndex = oldToNewCitationMap[oldIndex];
+              if (newIndex) {
+                return `[${newIndex}]`;
+              }
+              return "";
+            })
+            .replace(/(\s\s+)/g, " ")
+            .trim();
+        }
+
+        // Build the final formatted output
+        let output = content + "\n\n--- Images ---\n";
+        images.forEach((img, index) => {
+          output += `${index + 1}. ${img.image_url}\n   (Source: ${
+            img.origin_url
+          })\n`;
+        });
+
+        return output;
+      },
+      "perplexityImages",
+      { toolName: "perplexityImages" },
+    );
   },
 };
 
@@ -134,97 +142,105 @@ export const perplexityVideos = {
       .array(z.string())
       .optional()
       .describe(
-        "Optional: Restrict search to specific domains (e.g., ['youtube.com']) or exclude them with '-' prefix."
+        "Optional: Restrict search to specific domains (e.g., ['youtube.com']) or exclude them with '-' prefix.",
       ),
   }),
   timeoutMs: 300000,
   execute: async (args: { query: string; search_domain_filter?: string[] }) => {
-    return safeToolExecute(async () => {
-      const apiKey = process.env.PERPLEXITY_API_KEY;
-      if (!apiKey) {
-        throw new Error("PERPLEXITY_API_KEY environment variable is not set.");
-      }
-
-      const url = "https://api.perplexity.ai/chat/completions";
-      const headers: Record<string, string> = {
-        Authorization: `Bearer ${apiKey}`,
-        "Content-Type": "application/json",
-        accept: "application/json",
-      };
-
-      const payload = {
-        model: "sonar-pro",
-        messages: [
-          { role: "user", content: `Show me videos of ${args.query}` },
-        ],
-        media_response: { overrides: { return_videos: true } },
-        ...(args.search_domain_filter
-          ? { search_domain_filter: args.search_domain_filter }
-          : {}),
-      };
-
-      const res = await fetch(url, {
-        method: "POST",
-        headers: headers,
-        body: JSON.stringify(payload),
-      });
-
-      if (!res.ok) {
-        const text = await res.text();
-        throw new Error(`Perplexity API request failed: ${res.status} ${text}`);
-      }
-
-      const data = (await res.json()) as any;
-      let content = data.choices?.[0]?.message?.content;
-      const videos = (data.videos || []) as any[];
-      const citations = (data.citations || []) as string[];
-
-      if (videos.length === 0) {
-        return `No direct video URLs found in the API response. Full API Response: ${JSON.stringify(
-          data,
-          null,
-          2
-        )}`;
-      }
-
-      // Create a map of video url -> new 1-based index
-      const urlToVideoIndex: Record<string, number> = {};
-      videos.forEach((video, index) => {
-        if (video.url) {
-          urlToVideoIndex[video.url] = index + 1;
+    return safeToolExecute(
+      async () => {
+        const apiKey = process.env.PERPLEXITY_API_KEY;
+        if (!apiKey) {
+          throw new Error(
+            "PERPLEXITY_API_KEY environment variable is not set.",
+          );
         }
-      });
 
-      // Create a map of old citation index -> new video index
-      const oldToNewCitationMap: Record<number, number> = {};
-      citations.forEach((citationUrl, index) => {
-        if (urlToVideoIndex[citationUrl]) {
-          oldToNewCitationMap[index + 1] = urlToVideoIndex[citationUrl];
+        const url = "https://api.perplexity.ai/chat/completions";
+        const headers: Record<string, string> = {
+          Authorization: `Bearer ${apiKey}`,
+          "Content-Type": "application/json",
+          accept: "application/json",
+        };
+
+        const payload = {
+          model: "sonar-pro",
+          messages: [
+            { role: "user", content: `Show me videos of ${args.query}` },
+          ],
+          media_response: { overrides: { return_videos: true } },
+          ...(args.search_domain_filter
+            ? { search_domain_filter: args.search_domain_filter }
+            : {}),
+        };
+
+        const res = await fetch(url, {
+          method: "POST",
+          headers: headers,
+          body: JSON.stringify(payload),
+        });
+
+        if (!res.ok) {
+          const text = await res.text();
+          throw new Error(
+            `Perplexity API request failed: ${res.status} ${text}`,
+          );
         }
-      });
-
-      // Replace citations in the content
-      if (content && typeof content === "string") {
-        content = content
-          .replace(/\[(\d+)\]/g, (_match, oldIndexStr) => {
-            const oldIndex = parseInt(oldIndexStr, 10);
-            const newIndex = oldToNewCitationMap[oldIndex];
-            if (newIndex) {
-              return `[${newIndex}]`;
-            }
-            return "";
-          })
-          .replace(/(\s\s+)/g, " ")
-          .trim();
-      }
-
-      // Build the final formatted output
-      let output = content + "\n\n--- Videos ---\n";
-      videos.forEach((video, index) => {
-        output += `${index + 1}. ${video.url}\n`;
-      });
-
-      return output;
-    }, "perplexityVideos");
+
+        const data = (await res.json()) as any;
+        let content = data.choices?.[0]?.message?.content;
+        const videos = (data.videos || []) as any[];
+        const citations = (data.citations || []) as string[];
+
+        if (videos.length === 0) {
+          return `No direct video URLs found in the API response. Full API Response: ${JSON.stringify(
+            data,
+            null,
+            2,
+          )}`;
+        }
+
+        // Create a map of video url -> new 1-based index
+        const urlToVideoIndex: Record<string, number> = {};
+        videos.forEach((video, index) => {
+          if (video.url) {
+            urlToVideoIndex[video.url] = index + 1;
+          }
+        });
+
+        // Create a map of old citation index -> new video index
+        const oldToNewCitationMap: Record<number, number> = {};
+        citations.forEach((citationUrl, index) => {
+          if (urlToVideoIndex[citationUrl]) {
+            oldToNewCitationMap[index + 1] = urlToVideoIndex[citationUrl];
+          }
+        });
+
+        // Replace citations in the content
+        if (content && typeof content === "string") {
+          content = content
+            .replace(/\[(\d+)\]/g, (_match, oldIndexStr) => {
+              const oldIndex = parseInt(oldIndexStr, 10);
+              const newIndex = oldToNewCitationMap[oldIndex];
+              if (newIndex) {
+                return `[${newIndex}]`;
+              }
+              return "";
+            })
+            .replace(/(\s\s+)/g, " ")
+            .trim();
+        }
+
+        // Build the final formatted output
+        let output = content + "\n\n--- Videos ---\n";
+        videos.forEach((video, index) => {
+          output += `${index + 1}. ${video.url}\n`;
+        });
+
+        return output;
+      },
+      "perplexityVideos",
+      { toolName: "perplexityVideos" },
+    );
   },
 };

package/src/tools/youtube.ts

@@ -17,60 +17,64 @@ export const analyzeYoutubeVideo = {
     youtube_url: z
       .string()
       .describe(
-        "The full URL of the YouTube video (e.g., 'https://www.youtube.com/watch?v=dQw4w9WgXcQ')."
+        "The full URL of the YouTube video (e.g., 'https://www.youtube.com/watch?v=dQw4w9WgXcQ').",
       ),
     prompt: z
       .string()
       .describe(
-        "Instruction or question about the video content (e.g., 'Summarize the main points' or 'What color was the car?')."
+        "Instruction or question about the video content (e.g., 'Summarize the main points' or 'What color was the car?').",
       ),
   }),
   timeoutMs: 300000,
   execute: async (args: { youtube_url: string; prompt: string }) => {
-    return safeToolExecute(async () => {
-      try {
-        // Validate YouTube URL format
-        if (
-          !args.youtube_url ||
-          (!args.youtube_url.includes("youtube.com/watch") &&
-            !args.youtube_url.includes("youtu.be"))
-        ) {
-          throw new Error(
-            "Invalid YouTube URL format. Expected: https://www.youtube.com/watch?v=VIDEO_ID"
-          );
-        }
+    return safeToolExecute(
+      async () => {
+        try {
+          // Validate YouTube URL format
+          if (
+            !args.youtube_url ||
+            (!args.youtube_url.includes("youtube.com/watch") &&
+              !args.youtube_url.includes("youtu.be"))
+          ) {
+            throw new Error(
+              "Invalid YouTube URL format. Expected: https://www.youtube.com/watch?v=VIDEO_ID",
+            );
+          }
 
-        // Create content using the correct FileData approach with fileUri
-        const response = await ai.models.generateContent({
-          model: "models/gemini-2.5-flash",
-          contents: {
-            parts: [
-              {
-                fileData: {
-                  fileUri: args.youtube_url,
+          // Create content using the correct FileData approach with fileUri
+          const response = await ai.models.generateContent({
+            model: "models/gemini-2.5-flash",
+            contents: {
+              parts: [
+                {
+                  fileData: {
+                    fileUri: args.youtube_url,
+                  },
                 },
-              },
-              { text: args.prompt },
-            ],
-          },
-        });
+                { text: args.prompt },
+              ],
+            },
+          });
 
-        let result = "";
-        if (response.candidates && response.candidates[0]?.content?.parts) {
-          for (const part of response.candidates[0].content.parts) {
-            if (part.text) {
-              result += part.text;
+          let result = "";
+          if (response.candidates && response.candidates[0]?.content?.parts) {
+            for (const part of response.candidates[0].content.parts) {
+              if (part.text) {
+                result += part.text;
+              }
             }
           }
-        }
 
-        return (
-          result ||
-          "YouTube video analysis completed but no text response received"
-        );
-      } catch (error: any) {
-        throw new Error(`YouTube video analysis failed: ${error.message}`);
-      }
-    }, "analyzeYoutubeVideo");
+          return (
+            result ||
+            "YouTube video analysis completed but no text response received"
+          );
+        } catch (error: any) {
+          throw new Error(`YouTube video analysis failed: ${error.message}`);
+        }
+      },
+      "analyzeYoutubeVideo",
+      { toolName: "analyzeYoutubeVideo" },
+    );
   },
 };

package/src/utils/llm-prompt-enhancer.ts

@@ -6,6 +6,7 @@
  */
 
 import { GoogleGenAI } from "@google/genai";
+import { logger } from "./logger";
 
 const ai = new GoogleGenAI({
   apiKey: process.env.GEMINI_API_KEY || "",
@@ -236,7 +237,7 @@ export const LLM_ENHANCER_CONFIGS: Record<string, LLMEnhancerConfig> = {
 export async function enhancePromptWithLLM(
   prompt: string,
   configOrName: string | LLMEnhancerConfig = "ltx2",
-  images?: string[]
+  images?: string[],
 ): Promise<string> {
   // Resolve config - ltx2 is always available as default
   let config: LLMEnhancerConfig;
@@ -288,7 +289,7 @@ export async function enhancePromptWithLLM(
 
     return enhancedPrompt;
   } catch (error: any) {
-    console.error(`LLM prompt enhancement failed: ${error.message}`);
+    logger.error(`LLM prompt enhancement failed: ${error.message}`);
     // Fall back to original prompt on error
     return prompt;
   }

package/src/utils/logger.ts

@@ -0,0 +1,71 @@
+/**
+ * Winston + Sentry logging for MCP server.
+ * All logs go to stderr (MCP-safe). Errors are also sent to Sentry if configured.
+ */
+
+import winston from "winston";
+import Sentry from "winston-transport-sentry-node";
+
+const { combine, timestamp, printf, colorize } = winston.format;
+
+// Default Sentry DSN - can be overridden via SENTRY_DSN env var
+const DEFAULT_SENTRY_DSN =
+  "https://349c280f4b1c731bc1ac1a1189cae5e7@o4510770034245632.ingest.us.sentry.io/4510770047025152";
+
+// Custom format for console output (cleaner for MCP inspector)
+const consoleFormat = printf(({ level, message, timestamp, tags }) => {
+  const clientId = tags?.client_id ? ` [${tags.client_id}]` : "";
+  return `${timestamp} [${level}]${clientId} ${message}`;
+});
+
+// Determine log level from environment
+const LOG_LEVEL = process.env.LOG_LEVEL || "info";
+
+// Create transports array
+const transports: winston.transport[] = [
+  // Console transport - ALL levels go to stderr (MCP-safe)
+  new winston.transports.Console({
+    stderrLevels: [
+      "error",
+      "warn",
+      "info",
+      "http",
+      "verbose",
+      "debug",
+      "silly",
+    ],
+    format: combine(
+      colorize(),
+      timestamp({ format: "HH:mm:ss" }),
+      consoleFormat,
+    ),
+  }),
+];
+
+// Add Sentry transport - uses default DSN if not overridden
+const SENTRY_DSN = process.env.SENTRY_DSN || DEFAULT_SENTRY_DSN;
+transports.push(
+  new Sentry({
+    sentry: {
+      dsn: SENTRY_DSN,
+      environment: process.env.NODE_ENV || "development",
+    },
+    level: "error", // Only send errors and above to Sentry
+  }),
+);
+
+// Create the logger instance
+export const logger = winston.createLogger({
+  level: LOG_LEVEL,
+  format: combine(timestamp(), winston.format.json()),
+  defaultMeta: {
+    service: "kalaasetu-mcp",
+    tags: {
+      client_id: process.env.CLIENT_ID || "unknown",
+    },
+  },
+  transports,
+});
+
+// Log startup info
+logger.info("Sentry error tracking enabled");