@trops/dash-core 0.1.343 → 0.1.345

package/dist/electron/index.js

@@ -18,7 +18,7 @@ var require$$8$1 = require('https');
 var require$$0$6 = require('@modelcontextprotocol/sdk/client/index.js');
 var require$$1$4 = require('@modelcontextprotocol/sdk/client/stdio.js');
 var require$$0$5 = require('pkce-challenge');
-var require$$5$1 = require('child_process');
+var require$$6$1 = require('child_process');
 var require$$2$2 = require('algoliasearch');
 var require$$3$3 = require('node:path');
 var require$$0$7 = require('openai');
@@ -4601,7 +4601,7 @@ function stableHash(obj) {
   return hash.toString(36);
 }
 
-const responseCache$2 = {
+const responseCache$3 = {
   async get(key, fetcher, options = {}) {
     const { ttl = 30000, forceRefresh = false } = options;
 
@@ -4686,7 +4686,7 @@ const responseCache$2 = {
   },
 };
 
-var responseCache_1 = responseCache$2;
+var responseCache_1 = responseCache$3;
 
 /**
  * clientCache
@@ -25620,6 +25620,52 @@ const {
 } = streamableHttp$1;
 const path$b = require$$1$2;
 const fs$7 = require$$0$2;
+const responseCache$2 = responseCache_1;
+
+/**
+ * Tool name prefixes considered safe to cache (read-only).
+ * Writes/mutations are NOT cached so they always hit the source.
+ */
+const READ_ONLY_PREFIXES = [
+  "list_",
+  "get_",
+  "read_",
+  "search_",
+  "resolve_",
+  "find_",
+  "fetch_",
+  "describe_",
+];
+
+const WRITE_PREFIXES = [
+  "create_",
+  "write_",
+  "update_",
+  "delete_",
+  "remove_",
+  "append_",
+  "set_",
+  "put_",
+  "post_",
+  "patch_",
+  "send_",
+  "execute_",
+  "run_",
+];
+
+function isReadOnlyTool(toolName) {
+  if (!toolName) return false;
+  const lower = toolName.toLowerCase();
+  // Explicit write prefix wins (in case a tool starts with read_ but actually writes)
+  if (WRITE_PREFIXES.some((p) => lower.startsWith(p))) return false;
+  return READ_ONLY_PREFIXES.some((p) => lower.startsWith(p));
+}
+
+/**
+ * Default TTL for cached MCP tool calls (milliseconds).
+ * Short enough to keep data fresh, long enough to dedupe concurrent widget loads.
+ */
+const DEFAULT_TOOL_CACHE_TTL = 5000;
 
 /**
  * Cached shell PATH result (resolved once, reused for all spawns).
@@ -25657,7 +25703,7 @@ function isNodeEsmError(errorText) {
 function getShellPath$1() {
   if (_shellPath$1 !== null) return _shellPath$1;
 
-  const { execSync } = require$$5$1;
+  const { execSync } = require$$6$1;
   const fallbackDirs = ["/usr/local/bin", "/opt/homebrew/bin"];
 
   // Scan nvm versions, tracking both latest and best compatible version
@@ -26241,17 +26287,31 @@ const mcpController$2 = {
         );
       }
 
-      console.log(`[mcpController] Calling tool: ${serverName}/${toolName}`);
+      const doCall = async () => {
+        console.log(`[mcpController] Calling tool: ${serverName}/${toolName}`);
+        const result = await server.client.callTool({
+          name: toolName,
+          arguments: args || {},
+        });
+        return {
+          success: true,
+          result,
+        };
+      };
 
-      const result = await server.client.callTool({
-        name: toolName,
-        arguments: args || {},
-      });
+      // Cache read-only tool calls with in-flight dedup.
+      // Writes always hit the source (and we invalidate the server's cache).
+      if (isReadOnlyTool(toolName)) {
+        const key = `mcp:${serverName}:${toolName}:${JSON.stringify(args || {})}`;
+        return responseCache$2.get(key, doCall, {
+          ttl: DEFAULT_TOOL_CACHE_TTL,
+        });
+      }
 
-      return {
-        success: true,
-        result,
-      };
+      // Write/mutation: invalidate any cached reads for this server
+      // (safest default — broad invalidation when state changes)
+      responseCache$2.invalidatePrefix(`mcp:${serverName}:`);
+      return doCall();
     } catch (error) {
       console.error(
         `[mcpController] Error calling tool ${serverName}/${toolName}:`,
@@ -26473,7 +26533,7 @@ const mcpController$2 = {
    * @returns {{ success } | { error, message }}
    */
   runAuth: async (win, mcpConfig, credentials, authCommand) => {
-    const { spawn } = require$$5$1;
+    const { spawn } = require$$6$1;
 
     const env = cleanEnvForChildProcess();
 
@@ -27725,7 +27785,7 @@ var pluginController_1 = pluginController$1;
  * can use the Chat widget without a separate API key.
  */
 
-const { spawn, execSync } = require$$5$1;
+const { spawn, execSync } = require$$6$1;
 const {
   LLM_STREAM_DELTA: LLM_STREAM_DELTA$2,
   LLM_STREAM_TOOL_CALL: LLM_STREAM_TOOL_CALL$2,