sonance-brand-mcp 1.3.89 → 1.3.90

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

@@ -229,6 +229,139 @@ function findElementIdInFile(
   return null;
 }
 
+/**
+ * Find the line number of a focused element in the source code
+ * Uses multiple strategies in priority order:
+ * 1. DOM id (highest confidence)
+ * 2. Text content
+ * 3. ClassName patterns
+ * 
+ * This extends the basic ID matching to handle elements without IDs.
+ */
+function findElementLineInFile(
+  fileContent: string,
+  focusedElement: VisionFocusedElement
+): { lineNumber: number; snippet: string; confidence: 'high' | 'medium' | 'low'; matchedBy: string } | null {
+  if (!fileContent) return null;
+  
+  const lines = fileContent.split('\n');
+  
+  // PRIORITY 1: DOM id - highest confidence (exact match)
+  if (focusedElement.elementId) {
+    const idPattern = new RegExp(`id=["'\`]${focusedElement.elementId}["'\`]`);
+    for (let i = 0; i < lines.length; i++) {
+      if (idPattern.test(lines[i])) {
+        return {
+          lineNumber: i + 1,
+          snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+          confidence: 'high',
+          matchedBy: `DOM id="${focusedElement.elementId}"`
+        };
+      }
+    }
+  }
+  
+  // Also try child IDs with high confidence
+  if (focusedElement.childIds && focusedElement.childIds.length > 0) {
+    for (const childId of focusedElement.childIds) {
+      const idPattern = new RegExp(`id=["'\`]${childId}["'\`]`);
+      for (let i = 0; i < lines.length; i++) {
+        if (idPattern.test(lines[i])) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'high',
+            matchedBy: `child id="${childId}"`
+          };
+        }
+      }
+    }
+  }
+  
+  // PRIORITY 2: Exact text content in JSX
+  if (focusedElement.textContent && focusedElement.textContent.trim().length >= 2) {
+    const text = focusedElement.textContent.trim();
+    // Escape special regex characters in the text
+    const escapedText = text.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+    
+    for (let i = 0; i < lines.length; i++) {
+      const line = lines[i];
+      // Match patterns like: >Text<, >Text</tag, "Text", 'Text', {`Text`}
+      const textPatterns = [
+        `>${escapedText}<`,           // JSX content: >Text<
+        `"${escapedText}"`,           // String literal
+        `'${escapedText}'`,           // String literal
+        `\`${escapedText}\``,         // Template literal
+      ];
+      
+      for (const pattern of textPatterns) {
+        if (line.includes(pattern.replace(/\\/g, ''))) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'high',
+            matchedBy: `textContent="${text.substring(0, 30)}${text.length > 30 ? '...' : ''}"`
+          };
+        }
+      }
+    }
+    
+    // Try partial match for longer text (first 15+ chars)
+    if (text.length > 15) {
+      const partialText = text.substring(0, 15);
+      for (let i = 0; i < lines.length; i++) {
+        if (lines[i].includes(`>${partialText}`) || lines[i].includes(`"${partialText}`)) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'medium',
+            matchedBy: `partial textContent starting with "${partialText}..."`
+          };
+        }
+      }
+    }
+  }
+  
+  // PRIORITY 3: Distinctive className patterns
+  if (focusedElement.className) {
+    // Extract distinctive class names (long, not hover/focus pseudo-classes)
+    const classes = focusedElement.className.split(/\s+/)
+      .filter(c => c.length > 8 && !c.startsWith('hover:') && !c.startsWith('focus:') && !c.startsWith('active:'));
+    
+    for (const cls of classes) {
+      for (let i = 0; i < lines.length; i++) {
+        if (lines[i].includes(cls)) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'medium',
+            matchedBy: `className contains "${cls}"`
+          };
+        }
+      }
+    }
+    
+    // Try shorter but unique-looking class names (not utility classes)
+    const uniqueClasses = focusedElement.className.split(/\s+/)
+      .filter(c => c.length > 4 && !c.match(/^(p[xytblr]?-|m[xytblr]?-|w-|h-|bg-|text-|flex|grid|block|hidden)/));
+    
+    for (const cls of uniqueClasses) {
+      for (let i = 0; i < lines.length; i++) {
+        if (lines[i].includes(cls)) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'low',
+            matchedBy: `className contains "${cls}"`
+          };
+        }
+      }
+    }
+  }
+  
+  return null;
+}
+
 /**
  * PHASE 0: Deterministic Element ID Search (Cursor-style)
  * Grep entire codebase for the element ID. If found in multiple files,
@@ -1330,12 +1463,21 @@ User Request: "${userPrompt}"
       if (recommendedFileContent) {
         const content = recommendedFileContent.content;
         
-        // Search for element IDs in the file to enable precise targeting
-        let idMatch: { lineNumber: number; matchedId: string; snippet: string } | null = null;
-      if (focusedElements && focusedElements.length > 0) {
+        // Search for focused element in the file using multiple strategies
+        // Priority: DOM id > textContent > className patterns
+        let elementLocation: { lineNumber: number; snippet: string; confidence: 'high' | 'medium' | 'low'; matchedBy: string } | null = null;
+        if (focusedElements && focusedElements.length > 0) {
           for (const el of focusedElements) {
-            idMatch = findElementIdInFile(content, el.elementId, el.childIds);
-            if (idMatch) break;
+            elementLocation = findElementLineInFile(content, el);
+            if (elementLocation) {
+              debugLog("Found focused element in file", {
+                matchedBy: elementLocation.matchedBy,
+                lineNumber: elementLocation.lineNumber,
+                confidence: elementLocation.confidence,
+                file: recommendedFileContent.path,
+              });
+              break;
+            }
           }
         }
         
@@ -1350,22 +1492,36 @@ User Request: "${userPrompt}"
             textContent += `\n`;
           }
           
-          // Add precise targeting if we found an ID match
-          if (idMatch) {
+          // Add precise targeting with line number and snippet
+          if (elementLocation) {
             textContent += `
-PRECISE TARGET (found by element ID):
-  → ID: "${idMatch.matchedId}"
-  → Line: ${idMatch.lineNumber}
-  → Look for this ID in the code and modify the element that contains it.
+══════════════════════════════════════════════════════════════════════════════
+PRECISE TARGET LOCATION (${elementLocation.confidence} confidence)
+══════════════════════════════════════════════════════════════════════════════
+  → Matched by: ${elementLocation.matchedBy}
+  → Line: ${elementLocation.lineNumber}
+  
+THE USER CLICKED ON THE ELEMENT AT LINE ${elementLocation.lineNumber}.
+Here is the exact code around that element:
+\`\`\`
+${elementLocation.snippet}
+\`\`\`
+
+⚠️  IMPORTANT: Modify ONLY the element at line ${elementLocation.lineNumber}, NOT other similar elements in the file.
 
 `;
-            debugLog("Found element ID in file", {
-              matchedId: idMatch.matchedId,
-              lineNumber: idMatch.lineNumber,
-              file: recommendedFileContent.path,
-            });
           } else {
             textContent += `\n`;
+            debugLog("Could not locate focused element in file - no ID, textContent, or className match", {
+              file: recommendedFileContent.path,
+              focusedElements: focusedElements.map(el => ({
+                name: el.name,
+                type: el.type,
+                textContent: el.textContent?.substring(0, 30),
+                className: el.className?.substring(0, 50),
+                elementId: el.elementId,
+              }))
+            });
           }
         }
         

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

@@ -225,6 +225,139 @@ function findElementIdInFile(
   return null;
 }
 
+/**
+ * Find the line number of a focused element in the source code
+ * Uses multiple strategies in priority order:
+ * 1. DOM id (highest confidence)
+ * 2. Text content
+ * 3. ClassName patterns
+ * 
+ * This extends the basic ID matching to handle elements without IDs.
+ */
+function findElementLineInFile(
+  fileContent: string,
+  focusedElement: VisionFocusedElement
+): { lineNumber: number; snippet: string; confidence: 'high' | 'medium' | 'low'; matchedBy: string } | null {
+  if (!fileContent) return null;
+  
+  const lines = fileContent.split('\n');
+  
+  // PRIORITY 1: DOM id - highest confidence (exact match)
+  if (focusedElement.elementId) {
+    const idPattern = new RegExp(`id=["'\`]${focusedElement.elementId}["'\`]`);
+    for (let i = 0; i < lines.length; i++) {
+      if (idPattern.test(lines[i])) {
+        return {
+          lineNumber: i + 1,
+          snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+          confidence: 'high',
+          matchedBy: `DOM id="${focusedElement.elementId}"`
+        };
+      }
+    }
+  }
+  
+  // Also try child IDs with high confidence
+  if (focusedElement.childIds && focusedElement.childIds.length > 0) {
+    for (const childId of focusedElement.childIds) {
+      const idPattern = new RegExp(`id=["'\`]${childId}["'\`]`);
+      for (let i = 0; i < lines.length; i++) {
+        if (idPattern.test(lines[i])) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'high',
+            matchedBy: `child id="${childId}"`
+          };
+        }
+      }
+    }
+  }
+  
+  // PRIORITY 2: Exact text content in JSX
+  if (focusedElement.textContent && focusedElement.textContent.trim().length >= 2) {
+    const text = focusedElement.textContent.trim();
+    // Escape special regex characters in the text
+    const escapedText = text.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+    
+    for (let i = 0; i < lines.length; i++) {
+      const line = lines[i];
+      // Match patterns like: >Text<, >Text</tag, "Text", 'Text', {`Text`}
+      const textPatterns = [
+        `>${escapedText}<`,           // JSX content: >Text<
+        `"${escapedText}"`,           // String literal
+        `'${escapedText}'`,           // String literal
+        `\`${escapedText}\``,         // Template literal
+      ];
+      
+      for (const pattern of textPatterns) {
+        if (line.includes(pattern.replace(/\\/g, ''))) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'high',
+            matchedBy: `textContent="${text.substring(0, 30)}${text.length > 30 ? '...' : ''}"`
+          };
+        }
+      }
+    }
+    
+    // Try partial match for longer text (first 15+ chars)
+    if (text.length > 15) {
+      const partialText = text.substring(0, 15);
+      for (let i = 0; i < lines.length; i++) {
+        if (lines[i].includes(`>${partialText}`) || lines[i].includes(`"${partialText}`)) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'medium',
+            matchedBy: `partial textContent starting with "${partialText}..."`
+          };
+        }
+      }
+    }
+  }
+  
+  // PRIORITY 3: Distinctive className patterns
+  if (focusedElement.className) {
+    // Extract distinctive class names (long, not hover/focus pseudo-classes)
+    const classes = focusedElement.className.split(/\s+/)
+      .filter(c => c.length > 8 && !c.startsWith('hover:') && !c.startsWith('focus:') && !c.startsWith('active:'));
+    
+    for (const cls of classes) {
+      for (let i = 0; i < lines.length; i++) {
+        if (lines[i].includes(cls)) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'medium',
+            matchedBy: `className contains "${cls}"`
+          };
+        }
+      }
+    }
+    
+    // Try shorter but unique-looking class names (not utility classes)
+    const uniqueClasses = focusedElement.className.split(/\s+/)
+      .filter(c => c.length > 4 && !c.match(/^(p[xytblr]?-|m[xytblr]?-|w-|h-|bg-|text-|flex|grid|block|hidden)/));
+    
+    for (const cls of uniqueClasses) {
+      for (let i = 0; i < lines.length; i++) {
+        if (lines[i].includes(cls)) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'low',
+            matchedBy: `className contains "${cls}"`
+          };
+        }
+      }
+    }
+  }
+  
+  return null;
+}
+
 /**
  * PHASE 0: Deterministic Element ID Search (Cursor-style)
  * Grep entire codebase for the element ID. If found in multiple files,
@@ -1299,12 +1432,21 @@ User Request: "${userPrompt}"
       if (recommendedFileContent) {
         const content = recommendedFileContent.content;
         
-        // Search for element IDs in the file to enable precise targeting
-        let idMatch: { lineNumber: number; matchedId: string; snippet: string } | null = null;
-      if (focusedElements && focusedElements.length > 0) {
+        // Search for focused element in the file using multiple strategies
+        // Priority: DOM id > textContent > className patterns
+        let elementLocation: { lineNumber: number; snippet: string; confidence: 'high' | 'medium' | 'low'; matchedBy: string } | null = null;
+        if (focusedElements && focusedElements.length > 0) {
           for (const el of focusedElements) {
-            idMatch = findElementIdInFile(content, el.elementId, el.childIds);
-            if (idMatch) break;
+            elementLocation = findElementLineInFile(content, el);
+            if (elementLocation) {
+              debugLog("Found focused element in file", {
+                matchedBy: elementLocation.matchedBy,
+                lineNumber: elementLocation.lineNumber,
+                confidence: elementLocation.confidence,
+                file: recommendedFileContent.path,
+              });
+              break;
+            }
           }
         }
         
@@ -1319,22 +1461,36 @@ User Request: "${userPrompt}"
             textContent += `\n`;
           }
           
-          // Add precise targeting if we found an ID match
-          if (idMatch) {
+          // Add precise targeting with line number and snippet
+          if (elementLocation) {
             textContent += `
-PRECISE TARGET (found by element ID):
-  → ID: "${idMatch.matchedId}"
-  → Line: ${idMatch.lineNumber}
-  → Look for this ID in the code and modify the element that contains it.
+══════════════════════════════════════════════════════════════════════════════
+PRECISE TARGET LOCATION (${elementLocation.confidence} confidence)
+══════════════════════════════════════════════════════════════════════════════
+  → Matched by: ${elementLocation.matchedBy}
+  → Line: ${elementLocation.lineNumber}
+  
+THE USER CLICKED ON THE ELEMENT AT LINE ${elementLocation.lineNumber}.
+Here is the exact code around that element:
+\`\`\`
+${elementLocation.snippet}
+\`\`\`
+
+⚠️  IMPORTANT: Modify ONLY the element at line ${elementLocation.lineNumber}, NOT other similar elements in the file.
 
 `;
-            debugLog("Found element ID in file", {
-              matchedId: idMatch.matchedId,
-              lineNumber: idMatch.lineNumber,
-              file: recommendedFileContent.path,
-            });
           } else {
             textContent += `\n`;
+            debugLog("Could not locate focused element in file - no ID, textContent, or className match", {
+              file: recommendedFileContent.path,
+              focusedElements: focusedElements.map(el => ({
+                name: el.name,
+                type: el.type,
+                textContent: el.textContent?.substring(0, 30),
+                className: el.className?.substring(0, 50),
+                elementId: el.elementId,
+              }))
+            });
           }
         }
         

package/dist/assets/dev-tools/SonanceDevTools.tsx

@@ -1215,10 +1215,11 @@ export function SonanceDevTools() {
       console.warn("[Apply-First] Failed to persist session:", e);
     }
 
-    // After a brief delay, assume HMR has completed
+    // Force page refresh to ensure changes are visible
+    // Session is already persisted to localStorage, so it survives refresh
     setTimeout(() => {
-      setApplyFirstStatus("reviewing");
-    }, 1500);
+      window.location.reload();
+    }, 500);
   }, [visionFocusedElements]);
 
   // Accept changes - delete backups

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sonance-brand-mcp",
-  "version": "1.3.89",
+  "version": "1.3.90",
   "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",