sonance-brand-mcp 1.3.48 → 1.3.49

package/dist/assets/api/sonance-vision-apply/route.ts

@@ -545,6 +545,7 @@ export async function POST(request: Request) {
       return NextResponse.json({
         success: result.success,
         message: result.message,
+        error: result.success ? undefined : result.message, // Include error for client compatibility
         filesReverted: result.filesReverted,
       });
     }
@@ -846,6 +847,14 @@ CRITICAL: Edit the TARGET COMPONENT (marked with ***), not the page wrapper.`;
         // If this is a retry, add feedback about what went wrong
         if (retryCount > 0 && lastPatchErrors.length > 0) {
           debugLog("Retry attempt with feedback", { retryCount, errorCount: lastPatchErrors.length });
+          
+          // Get a snippet of the actual file content to help AI find correct code
+          let fileSnippet = "";
+          if (recommendedFileContent) {
+            // Show first 800 chars to give AI context about what code actually exists
+            fileSnippet = recommendedFileContent.content.substring(0, 800).replace(/\n/g, "\n");
+          }
+          
           currentMessages.push({
             role: "assistant",
             content: "I'll analyze the screenshot and generate the patches now.",
@@ -857,10 +866,20 @@ CRITICAL: Edit the TARGET COMPONENT (marked with ***), not the page wrapper.`;
 Failed patches:
 ${lastPatchErrors.join("\n\n")}
 
-IMPORTANT: You must copy the "search" string EXACTLY from the file content I provided. Do NOT invent or guess code.
-Look carefully at the ACTUAL file content in the TARGET COMPONENT section above and try again.
+${fileSnippet ? `Here is the ACTUAL beginning of the file you're trying to edit:
+\`\`\`tsx
+${fileSnippet}...
+\`\`\`
+` : ""}
+CRITICAL INSTRUCTIONS:
+1. Your "search" string MUST be copied EXACTLY from the file content I provided above
+2. Do NOT invent, guess, or imagine code that might exist
+3. Look for the ACTUAL code in the TARGET COMPONENT section and copy it exactly
 
-If you cannot find the exact code to modify, return an empty modifications array with an explanation.`,
+If you still cannot find the exact code to modify after reviewing the file:
+- Return {"modifications": [], "explanation": "Could not locate the exact code. Please specify which element you want to change more precisely."}
+
+This is better than generating patches with made-up code.`,
           });
         }
 
@@ -1849,6 +1868,7 @@ function searchFilesForKeywords(
 let cachedPathAliases: Map<string, string> | null = null;
 let cachedProjectRoot: string | null = null;
 let cachedTsconfigMtime: number | null = null;
+let cachedParseError: string | null = null; // Cache parse errors to avoid log spam
 
 /**
  * Clean tsconfig.json content to make it valid JSON
@@ -1921,21 +1941,23 @@ function getPathAliases(projectRoot: string): Map<string, string> {
         cachedTsconfigMtime = null;
       }
     } catch (e) {
-      // Log the error with more context for debugging
       const errorStr = String(e);
-      const posMatch = errorStr.match(/position (\d+)/);
-      let context = "";
-      if (posMatch) {
-        const pos = parseInt(posMatch[1], 10);
-        const content = fs.readFileSync(tsconfigPath, "utf-8");
-        context = `Near: "${content.substring(Math.max(0, pos - 20), pos + 20)}"`;
+      
+      // Only log parse error once to avoid log spam
+      if (!cachedParseError) {
+        const posMatch = errorStr.match(/position (\d+)/);
+        let context = "";
+        if (posMatch) {
+          const pos = parseInt(posMatch[1], 10);
+          const content = fs.readFileSync(tsconfigPath, "utf-8");
+          context = `Near: "${content.substring(Math.max(0, pos - 20), pos + 20)}"`;
+        }
+        debugLog("[apply] Failed to parse tsconfig.json (will use defaults, logging once)", { error: errorStr, context });
+        cachedParseError = errorStr;
       }
-      debugLog("[apply] Failed to parse tsconfig.json", { error: errorStr, context });
       
-      // Clear cache on error so we retry next time
-      cachedPathAliases = null;
-      cachedProjectRoot = null;
-      cachedTsconfigMtime = null;
+      // Don't clear cache - we'll use the fallback aliases
+      // cachedPathAliases will remain null, triggering fallback below
     }
   }
   
@@ -1950,13 +1972,16 @@ function getPathAliases(projectRoot: string): Map<string, string> {
     } else {
       aliases.set("@/", "");
     }
-    debugLog("[apply] Using default @/ alias", { alias: aliases.get("@/") });
+    // Only log default alias once (not when using cached error fallback)
+    if (!cachedParseError) {
+      debugLog("[apply] Using default @/ alias", { alias: aliases.get("@/") });
+    }
   }
   
-  // Only cache if we parsed successfully or there's no tsconfig
-  if (parsedSuccessfully || !fs.existsSync(tsconfigPath)) {
-  cachedPathAliases = aliases;
-  cachedProjectRoot = projectRoot;
+  // Cache aliases if parsed successfully, no tsconfig exists, OR we have a parse error (cache fallback)
+  if (parsedSuccessfully || !fs.existsSync(tsconfigPath) || cachedParseError) {
+    cachedPathAliases = aliases;
+    cachedProjectRoot = projectRoot;
   }
   
   return aliases;

package/dist/assets/api/sonance-vision-edit/route.ts

@@ -852,6 +852,14 @@ CRITICAL: Edit the TARGET COMPONENT (marked with ***), not the page wrapper.`;
         // If this is a retry, add feedback about what went wrong
         if (retryCount > 0 && lastPatchErrors.length > 0) {
           debugLog("Retry attempt with feedback", { retryCount, errorCount: lastPatchErrors.length });
+          
+          // Get a snippet of the actual file content to help AI find correct code
+          let fileSnippet = "";
+          if (recommendedFileContent) {
+            // Show first 800 chars to give AI context about what code actually exists
+            fileSnippet = recommendedFileContent.content.substring(0, 800).replace(/\n/g, "\n");
+          }
+          
           currentMessages.push({
             role: "assistant",
             content: "I'll analyze the screenshot and generate the patches now.",
@@ -863,10 +871,20 @@ CRITICAL: Edit the TARGET COMPONENT (marked with ***), not the page wrapper.`;
 Failed patches:
 ${lastPatchErrors.join("\n\n")}
 
-IMPORTANT: You must copy the "search" string EXACTLY from the file content I provided. Do NOT invent or guess code.
-Look carefully at the ACTUAL file content in the TARGET COMPONENT section above and try again.
+${fileSnippet ? `Here is the ACTUAL beginning of the file you're trying to edit:
+\`\`\`tsx
+${fileSnippet}...
+\`\`\`
+` : ""}
+CRITICAL INSTRUCTIONS:
+1. Your "search" string MUST be copied EXACTLY from the file content I provided above
+2. Do NOT invent, guess, or imagine code that might exist
+3. Look for the ACTUAL code in the TARGET COMPONENT section and copy it exactly
 
-If you cannot find the exact code to modify, return an empty modifications array with an explanation.`,
+If you still cannot find the exact code to modify after reviewing the file:
+- Return {"modifications": [], "explanation": "Could not locate the exact code. Please specify which element you want to change more precisely."}
+
+This is better than generating patches with made-up code.`,
           });
         }
 
@@ -1755,6 +1773,7 @@ function searchFilesForKeywords(
 let cachedPathAliases: Map<string, string> | null = null;
 let cachedProjectRoot: string | null = null;
 let cachedTsconfigMtime: number | null = null;
+let cachedParseError: string | null = null; // Cache parse errors to avoid log spam
 
 /**
  * Clean tsconfig.json content to make it valid JSON
@@ -1827,21 +1846,23 @@ function getPathAliases(projectRoot: string): Map<string, string> {
         cachedTsconfigMtime = null;
       }
     } catch (e) {
-      // Log the error with more context for debugging
       const errorStr = String(e);
-      const posMatch = errorStr.match(/position (\d+)/);
-      let context = "";
-      if (posMatch) {
-        const pos = parseInt(posMatch[1], 10);
-        const content = fs.readFileSync(tsconfigPath, "utf-8");
-        context = `Near: "${content.substring(Math.max(0, pos - 20), pos + 20)}"`;
+      
+      // Only log parse error once to avoid log spam
+      if (!cachedParseError) {
+        const posMatch = errorStr.match(/position (\d+)/);
+        let context = "";
+        if (posMatch) {
+          const pos = parseInt(posMatch[1], 10);
+          const content = fs.readFileSync(tsconfigPath, "utf-8");
+          context = `Near: "${content.substring(Math.max(0, pos - 20), pos + 20)}"`;
+        }
+        debugLog("[edit] Failed to parse tsconfig.json (will use defaults, logging once)", { error: errorStr, context });
+        cachedParseError = errorStr;
       }
-      debugLog("[edit] Failed to parse tsconfig.json", { error: errorStr, context });
       
-      // Clear cache on error so we retry next time
-      cachedPathAliases = null;
-      cachedProjectRoot = null;
-      cachedTsconfigMtime = null;
+      // Don't clear cache - we'll use the fallback aliases
+      // cachedPathAliases will remain null, triggering fallback below
     }
   }
   
@@ -1856,13 +1877,16 @@ function getPathAliases(projectRoot: string): Map<string, string> {
     } else {
       aliases.set("@/", "");
     }
-    debugLog("[edit] Using default @/ alias", { alias: aliases.get("@/") });
+    // Only log default alias once (not when using cached error fallback)
+    if (!cachedParseError) {
+      debugLog("[edit] Using default @/ alias", { alias: aliases.get("@/") });
+    }
   }
   
-  // Only cache if we parsed successfully or there's no tsconfig
-  if (parsedSuccessfully || !fs.existsSync(tsconfigPath)) {
-  cachedPathAliases = aliases;
-  cachedProjectRoot = projectRoot;
+  // Cache aliases if parsed successfully, no tsconfig exists, OR we have a parse error (cache fallback)
+  if (parsedSuccessfully || !fs.existsSync(tsconfigPath) || cachedParseError) {
+    cachedPathAliases = aliases;
+    cachedProjectRoot = projectRoot;
   }
   
   return aliases;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sonance-brand-mcp",
-  "version": "1.3.48",
+  "version": "1.3.49",
   "description": "MCP Server for Sonance Brand Guidelines and Component Library - gives Claude instant access to brand colors, typography, and UI components.",
   "main": "dist/index.js",
   "type": "module",