sonance-brand-mcp 1.3.89 → 1.3.91

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

@@ -229,6 +229,277 @@ 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 2b: Look for icon patterns if type is button and no text (like X buttons)
+  if (focusedElement.type === 'button' && (!focusedElement.textContent || focusedElement.textContent.trim().length === 0)) {
+    // Common close/icon button patterns
+    const iconPatterns = ['<X ', '<X/', 'DialogClose', 'DialogPrimitive.Close', 'closeButton', 'CloseButton'];
+    for (const pattern of iconPatterns) {
+      for (let i = 0; i < lines.length; i++) {
+        if (lines[i].includes(pattern)) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'low',
+            matchedBy: `icon pattern "${pattern}"`
+          };
+        }
+      }
+    }
+  }
+  
+  // PRIORITY 3: Distinctive className patterns (ONLY semantic/custom classes)
+  if (focusedElement.className) {
+    // Filter OUT all Tailwind utility patterns - these are too generic and match wrong elements
+    const tailwindPatterns = [
+      /^p[xytblr]?-/,     // padding: px-4, py-2, pt-1
+      /^m[xytblr]?-/,     // margin: mx-auto, my-4
+      /^-?m[xytblr]?-/,   // negative margins: -mt-4
+      /^w-/,              // width: w-full, w-4
+      /^h-/,              // height: h-10, h-full
+      /^min-/,            // min-width/height
+      /^max-/,            // max-width/height
+      /^size-/,           // size utilities
+      /^bg-/,             // background
+      /^text-/,           // text color/size
+      /^font-/,           // font-weight
+      /^leading-/,        // line-height
+      /^tracking-/,       // letter-spacing
+      /^rounded/,         // border-radius: rounded-sm, rounded-lg
+      /^border/,          // border
+      /^outline/,         // outline
+      /^ring/,            // ring
+      /^shadow/,          // shadow
+      /^flex/,            // flex utilities
+      /^grow/,            // flex-grow
+      /^shrink/,          // flex-shrink
+      /^basis-/,          // flex-basis
+      /^grid/,            // grid utilities
+      /^col-/,            // grid columns
+      /^row-/,            // grid rows
+      /^gap-/,            // gap
+      /^items-/,          // align-items
+      /^justify-/,        // justify-content
+      /^self-/,           // align-self
+      /^place-/,          // place utilities
+      /^space-/,          // space-x, space-y
+      /^overflow/,        // overflow
+      /^scroll/,          // scroll utilities
+      /^cursor-/,         // cursor
+      /^pointer-/,        // pointer-events
+      /^select-/,         // user-select
+      /^opacity-/,        // opacity
+      /^visible/,         // visibility
+      /^invisible/,       // visibility
+      /^z-/,              // z-index
+      /^inset/,           // inset utilities
+      /^top-/,            // positioning
+      /^right-/,          // positioning
+      /^bottom-/,         // positioning
+      /^left-/,           // positioning
+      /^static/,          // position
+      /^fixed/,           // position
+      /^absolute/,        // position
+      /^relative/,        // position
+      /^sticky/,          // position
+      /^transition/,      // transitions
+      /^duration-/,       // duration
+      /^ease-/,           // timing function
+      /^delay-/,          // delay
+      /^animate-/,        // animations
+      /^transform/,       // transform
+      /^scale-/,          // scale
+      /^rotate-/,         // rotate
+      /^translate-/,      // translate
+      /^skew-/,           // skew
+      /^origin-/,         // transform-origin
+      /^appearance-/,     // appearance
+      /^accent-/,         // accent-color
+      /^caret-/,          // caret-color
+      /^fill-/,           // SVG fill
+      /^stroke-/,         // SVG stroke
+      /^object-/,         // object-fit/position
+      /^aspect-/,         // aspect-ratio
+      /^container/,       // container
+      /^columns-/,        // columns
+      /^break-/,          // word-break
+      /^truncate/,        // text-overflow
+      /^whitespace-/,     // whitespace
+      /^list-/,           // list-style
+      /^decoration-/,     // text-decoration
+      /^underline/,       // underline
+      /^overline/,        // overline
+      /^line-through/,    // line-through
+      /^no-underline/,    // no underline
+      /^antialiased/,     // font-smoothing
+      /^subpixel/,        // subpixel-antialiased
+      /^italic/,          // font-style
+      /^not-italic/,      // font-style
+      /^uppercase/,       // text-transform
+      /^lowercase/,       // text-transform
+      /^capitalize/,      // text-transform
+      /^normal-case/,     // text-transform
+      /^align-/,          // vertical-align
+      /^indent-/,         // text-indent
+      /^content-/,        // content
+      /^will-change/,     // will-change
+      /^hover:/,          // hover states
+      /^focus:/,          // focus states
+      /^focus-within:/,   // focus-within states
+      /^focus-visible:/,  // focus-visible states
+      /^active:/,         // active states
+      /^visited:/,        // visited states
+      /^disabled:/,       // disabled states
+      /^checked:/,        // checked states
+      /^group-/,          // group utilities
+      /^peer-/,           // peer utilities
+      /^dark:/,           // dark mode
+      /^light:/,          // light mode
+      /^motion-/,         // motion utilities
+      /^print:/,          // print utilities
+      /^portrait:/,       // orientation
+      /^landscape:/,      // orientation
+      /^sm:|^md:|^lg:|^xl:|^2xl:/,  // responsive prefixes
+      /^data-\[/,         // data attribute variants
+      /^aria-/,           // aria variants
+      /^supports-/,       // supports variants
+      /^has-/,            // has variants
+      /^group$/,          // group class itself
+      /^peer$/,           // peer class itself
+      /^sr-only/,         // screen reader only
+      /^not-sr-only/,     // not screen reader only
+      /^isolate/,         // isolation
+      /^block$/,          // display
+      /^inline/,          // display
+      /^hidden$/,         // display
+      /^table/,           // display table
+    ];
+    
+    const isTailwindUtility = (cls: string) => 
+      tailwindPatterns.some(pattern => pattern.test(cls));
+    
+    // Only use classes that are NOT Tailwind utilities (likely custom/semantic)
+    const semanticClasses = focusedElement.className.split(/\s+/)
+      .filter(c => c.length > 3 && !isTailwindUtility(c));
+    
+    // Only proceed if we found semantic classes
+    if (semanticClasses.length > 0) {
+      for (const cls of semanticClasses) {
+        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: `semantic className "${cls}"`
+            };
+          }
+        }
+      }
+    }
+    
+    // Log when we filtered out all classes (helpful for debugging)
+    if (semanticClasses.length === 0 && focusedElement.className.trim().length > 0) {
+      console.log('[findElementLineInFile] All classes filtered as Tailwind utilities:', 
+        focusedElement.className.substring(0, 100));
+    }
+  }
+  
+  return null;
+}
+
 /**
  * PHASE 0: Deterministic Element ID Search (Cursor-style)
  * Grep entire codebase for the element ID. If found in multiple files,
@@ -1330,12 +1601,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 +1630,49 @@ 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("WARNING: Could not locate focused element in file", {
+              file: recommendedFileContent.path,
+              hint: "The clicked element may be rendered by a child component (e.g., DialogContent, Button component). Check the component's source file instead.",
+              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,
+              }))
+            });
+            
+            // Add a hint to the LLM that we couldn't precisely locate the element
+            textContent += `
+⚠️  WARNING: Could not precisely locate the clicked element in this file.
+The element may be:
+  - Rendered by a child component (e.g., DialogContent, Button)
+  - Dynamically generated
+  - In a different file imported by this component
+
+Please proceed with caution and verify you're modifying the correct element.
+
+`;
           }
         }
         

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

@@ -225,6 +225,277 @@ 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 2b: Look for icon patterns if type is button and no text (like X buttons)
+  if (focusedElement.type === 'button' && (!focusedElement.textContent || focusedElement.textContent.trim().length === 0)) {
+    // Common close/icon button patterns
+    const iconPatterns = ['<X ', '<X/', 'DialogClose', 'DialogPrimitive.Close', 'closeButton', 'CloseButton'];
+    for (const pattern of iconPatterns) {
+      for (let i = 0; i < lines.length; i++) {
+        if (lines[i].includes(pattern)) {
+          return {
+            lineNumber: i + 1,
+            snippet: lines.slice(Math.max(0, i - 3), i + 5).join('\n'),
+            confidence: 'low',
+            matchedBy: `icon pattern "${pattern}"`
+          };
+        }
+      }
+    }
+  }
+  
+  // PRIORITY 3: Distinctive className patterns (ONLY semantic/custom classes)
+  if (focusedElement.className) {
+    // Filter OUT all Tailwind utility patterns - these are too generic and match wrong elements
+    const tailwindPatterns = [
+      /^p[xytblr]?-/,     // padding: px-4, py-2, pt-1
+      /^m[xytblr]?-/,     // margin: mx-auto, my-4
+      /^-?m[xytblr]?-/,   // negative margins: -mt-4
+      /^w-/,              // width: w-full, w-4
+      /^h-/,              // height: h-10, h-full
+      /^min-/,            // min-width/height
+      /^max-/,            // max-width/height
+      /^size-/,           // size utilities
+      /^bg-/,             // background
+      /^text-/,           // text color/size
+      /^font-/,           // font-weight
+      /^leading-/,        // line-height
+      /^tracking-/,       // letter-spacing
+      /^rounded/,         // border-radius: rounded-sm, rounded-lg
+      /^border/,          // border
+      /^outline/,         // outline
+      /^ring/,            // ring
+      /^shadow/,          // shadow
+      /^flex/,            // flex utilities
+      /^grow/,            // flex-grow
+      /^shrink/,          // flex-shrink
+      /^basis-/,          // flex-basis
+      /^grid/,            // grid utilities
+      /^col-/,            // grid columns
+      /^row-/,            // grid rows
+      /^gap-/,            // gap
+      /^items-/,          // align-items
+      /^justify-/,        // justify-content
+      /^self-/,           // align-self
+      /^place-/,          // place utilities
+      /^space-/,          // space-x, space-y
+      /^overflow/,        // overflow
+      /^scroll/,          // scroll utilities
+      /^cursor-/,         // cursor
+      /^pointer-/,        // pointer-events
+      /^select-/,         // user-select
+      /^opacity-/,        // opacity
+      /^visible/,         // visibility
+      /^invisible/,       // visibility
+      /^z-/,              // z-index
+      /^inset/,           // inset utilities
+      /^top-/,            // positioning
+      /^right-/,          // positioning
+      /^bottom-/,         // positioning
+      /^left-/,           // positioning
+      /^static/,          // position
+      /^fixed/,           // position
+      /^absolute/,        // position
+      /^relative/,        // position
+      /^sticky/,          // position
+      /^transition/,      // transitions
+      /^duration-/,       // duration
+      /^ease-/,           // timing function
+      /^delay-/,          // delay
+      /^animate-/,        // animations
+      /^transform/,       // transform
+      /^scale-/,          // scale
+      /^rotate-/,         // rotate
+      /^translate-/,      // translate
+      /^skew-/,           // skew
+      /^origin-/,         // transform-origin
+      /^appearance-/,     // appearance
+      /^accent-/,         // accent-color
+      /^caret-/,          // caret-color
+      /^fill-/,           // SVG fill
+      /^stroke-/,         // SVG stroke
+      /^object-/,         // object-fit/position
+      /^aspect-/,         // aspect-ratio
+      /^container/,       // container
+      /^columns-/,        // columns
+      /^break-/,          // word-break
+      /^truncate/,        // text-overflow
+      /^whitespace-/,     // whitespace
+      /^list-/,           // list-style
+      /^decoration-/,     // text-decoration
+      /^underline/,       // underline
+      /^overline/,        // overline
+      /^line-through/,    // line-through
+      /^no-underline/,    // no underline
+      /^antialiased/,     // font-smoothing
+      /^subpixel/,        // subpixel-antialiased
+      /^italic/,          // font-style
+      /^not-italic/,      // font-style
+      /^uppercase/,       // text-transform
+      /^lowercase/,       // text-transform
+      /^capitalize/,      // text-transform
+      /^normal-case/,     // text-transform
+      /^align-/,          // vertical-align
+      /^indent-/,         // text-indent
+      /^content-/,        // content
+      /^will-change/,     // will-change
+      /^hover:/,          // hover states
+      /^focus:/,          // focus states
+      /^focus-within:/,   // focus-within states
+      /^focus-visible:/,  // focus-visible states
+      /^active:/,         // active states
+      /^visited:/,        // visited states
+      /^disabled:/,       // disabled states
+      /^checked:/,        // checked states
+      /^group-/,          // group utilities
+      /^peer-/,           // peer utilities
+      /^dark:/,           // dark mode
+      /^light:/,          // light mode
+      /^motion-/,         // motion utilities
+      /^print:/,          // print utilities
+      /^portrait:/,       // orientation
+      /^landscape:/,      // orientation
+      /^sm:|^md:|^lg:|^xl:|^2xl:/,  // responsive prefixes
+      /^data-\[/,         // data attribute variants
+      /^aria-/,           // aria variants
+      /^supports-/,       // supports variants
+      /^has-/,            // has variants
+      /^group$/,          // group class itself
+      /^peer$/,           // peer class itself
+      /^sr-only/,         // screen reader only
+      /^not-sr-only/,     // not screen reader only
+      /^isolate/,         // isolation
+      /^block$/,          // display
+      /^inline/,          // display
+      /^hidden$/,         // display
+      /^table/,           // display table
+    ];
+    
+    const isTailwindUtility = (cls: string) => 
+      tailwindPatterns.some(pattern => pattern.test(cls));
+    
+    // Only use classes that are NOT Tailwind utilities (likely custom/semantic)
+    const semanticClasses = focusedElement.className.split(/\s+/)
+      .filter(c => c.length > 3 && !isTailwindUtility(c));
+    
+    // Only proceed if we found semantic classes
+    if (semanticClasses.length > 0) {
+      for (const cls of semanticClasses) {
+        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: `semantic className "${cls}"`
+            };
+          }
+        }
+      }
+    }
+    
+    // Log when we filtered out all classes (helpful for debugging)
+    if (semanticClasses.length === 0 && focusedElement.className.trim().length > 0) {
+      console.log('[findElementLineInFile] All classes filtered as Tailwind utilities:', 
+        focusedElement.className.substring(0, 100));
+    }
+  }
+  
+  return null;
+}
+
 /**
  * PHASE 0: Deterministic Element ID Search (Cursor-style)
  * Grep entire codebase for the element ID. If found in multiple files,
@@ -1299,12 +1570,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 +1599,49 @@ 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("WARNING: Could not locate focused element in file", {
+              file: recommendedFileContent.path,
+              hint: "The clicked element may be rendered by a child component (e.g., DialogContent, Button component). Check the component's source file instead.",
+              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,
+              }))
+            });
+            
+            // Add a hint to the LLM that we couldn't precisely locate the element
+            textContent += `
+⚠️  WARNING: Could not precisely locate the clicked element in this file.
+The element may be:
+  - Rendered by a child component (e.g., DialogContent, Button)
+  - Dynamically generated
+  - In a different file imported by this component
+
+Please proceed with caution and verify you're modifying the correct element.
+
+`;
           }
         }
         

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.91",
   "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",