sonance-brand-mcp 1.3.95 → 1.3.97

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

@@ -516,6 +516,98 @@ function findElementLineInFile(
   return null;
 }
 
+/**
+ * Search imported component files for specific TEXT CONTENT
+ * This is used to redirect from parent components to child components
+ * when the actual visible text lives in an imported file.
+ * 
+ * Example: "What's the essence of this process?" might be in EssenceTab.tsx
+ * but the parent QuickAddProcessModal.tsx was selected by Phase 2a.
+ */
+/**
+ * Score imported files by how many discovered text strings they contain.
+ * This finds the TRUE file by matching ALL visible text, not just one substring.
+ * 
+ * Example: If annotation discovers ["What's the essence?", "Essence Quality", "Show Example"]
+ * and EssenceTab.tsx contains 2 of them while QuickAddProcessModal.tsx contains 1,
+ * EssenceTab.tsx wins with the higher score.
+ */
+function scoreFilesForTextContent(
+  textContents: string[],
+  importedFiles: { path: string; content: string }[]
+): { path: string; content: string; score: number; matchedTexts: string[]; firstMatchLine: number } | null {
+  if (!textContents || textContents.length === 0) return null;
+  
+  debugLog("Scoring imported files for text content", {
+    textCount: textContents.length,
+    texts: textContents.slice(0, 5).map(t => t.substring(0, 30)),
+    filesCount: importedFiles.length
+  });
+  
+  // Only search component files (where JSX text lives)
+  const componentFiles = importedFiles.filter(f => 
+    f.path.includes('components/') || f.path.includes('/ui/')
+  );
+  
+  const results: { path: string; content: string; score: number; matchedTexts: string[]; firstMatchLine: number }[] = [];
+  
+  for (const file of componentFiles) {
+    let score = 0;
+    const matchedTexts: string[] = [];
+    let firstMatchLine = 0;
+    
+    for (const text of textContents) {
+      if (!text || text.length < 5) continue;
+      
+      // Use first 20 chars for matching (handles dynamic suffixes)
+      const searchText = text.substring(0, 20);
+      
+      // Check if this text exists in the file
+      if (file.content.includes(searchText)) {
+        score++;
+        matchedTexts.push(text.substring(0, 30) + (text.length > 30 ? '...' : ''));
+        
+        // Find line number for first match (for element location)
+        if (firstMatchLine === 0) {
+          const lines = file.content.split('\n');
+          for (let i = 0; i < lines.length; i++) {
+            if (lines[i].includes(searchText)) {
+              firstMatchLine = i + 1;
+              break;
+            }
+          }
+        }
+      }
+    }
+    
+    if (score > 0) {
+      results.push({ 
+        path: file.path, 
+        content: file.content, 
+        score, 
+        matchedTexts,
+        firstMatchLine 
+      });
+    }
+  }
+  
+  // Sort by score descending (highest match count wins)
+  results.sort((a, b) => b.score - a.score);
+  
+  if (results.length > 0) {
+    debugLog("Text scoring results", {
+      topFile: results[0].path,
+      topScore: results[0].score,
+      topMatches: results[0].matchedTexts,
+      allResults: results.slice(0, 3).map(r => ({ path: r.path, score: r.score }))
+    });
+  } else {
+    debugLog("No text matches found in any imported component files");
+  }
+  
+  return results.length > 0 ? results[0] : null;
+}
+
 /**
  * Search imported component files for the focused element
  * Reuses findElementLineInFile() for consistent detection
@@ -1528,7 +1620,7 @@ export async function POST(request: Request) {
             const confirmedPath = phase2aMatches.find(p => 
               focusedElementHints.some(h => h.path === p && h.score > 0)
             );
-            recommendedFile = {
+              recommendedFile = {
               path: confirmedPath!,
               reason: `Phase 2a component-name match confirmed by element search`
             };
@@ -1552,7 +1644,7 @@ export async function POST(request: Request) {
               selectedPath: bestMatch,
               allCandidates: phase2aMatches 
             });
-          } else {
+            } else {
             // FALLBACK PRIORITY 4: Use the page file from the current route
             const routePageFile = discoverPageFile(pageRoute || "/", projectRoot);
             
@@ -1666,7 +1758,7 @@ User Request: "${userPrompt}"
         let elementLocation: { lineNumber: number; snippet: string; confidence: 'high' | 'medium' | 'low'; matchedBy: string } | null = null;
         let actualTargetFile = recommendedFileContent; // May change if we redirect
         
-        if (focusedElements && focusedElements.length > 0) {
+      if (focusedElements && focusedElements.length > 0) {
           for (const el of focusedElements) {
             elementLocation = findElementLineInFile(content, el);
             if (elementLocation) {
@@ -1680,6 +1772,72 @@ User Request: "${userPrompt}"
             }
           }
           
+          // TEXT SCORING REDIRECT: Score ALL imported files by how many discovered
+          // text strings they contain. The file with the highest score is the TRUE target.
+          // This handles parent/child component cases accurately.
+          if (elementLocation && elementLocation.confidence !== 'high') {
+            // Collect ALL text content from focused elements
+            const allTextContent = focusedElements
+              .filter(e => e.textContent && e.textContent.length > 5)
+              .map(e => e.textContent!);
+            
+            if (allTextContent.length > 0) {
+              debugLog("Medium/low confidence match - scoring imports for all text", {
+                currentFile: recommendedFileContent.path,
+                currentConfidence: elementLocation.confidence,
+                textCount: allTextContent.length,
+                texts: allTextContent.slice(0, 5).map(t => t.substring(0, 25))
+              });
+              
+              // Score all imported files
+              const bestMatch = scoreFilesForTextContent(
+                allTextContent,
+                pageContext.componentSources
+              );
+              
+              // Also score the current file for comparison
+              const currentFileScore = allTextContent.filter(text => 
+                recommendedFileContent.content.includes(text.substring(0, 20))
+              ).length;
+              
+              debugLog("Text scoring comparison", {
+                currentFile: recommendedFileContent.path,
+                currentScore: currentFileScore,
+                bestImport: bestMatch?.path || 'none',
+                bestImportScore: bestMatch?.score || 0
+              });
+              
+              // Redirect only if imported file has MORE matches than current file
+              if (bestMatch && bestMatch.score > currentFileScore) {
+                debugLog("TEXT REDIRECT: Imported file has more text matches", {
+                  originalFile: recommendedFileContent.path,
+                  originalScore: currentFileScore,
+                  redirectTo: bestMatch.path,
+                  redirectScore: bestMatch.score,
+                  matchedTexts: bestMatch.matchedTexts
+                });
+                
+                // Switch target file to where most text content lives
+                actualTargetFile = {
+                  path: bestMatch.path,
+                  content: bestMatch.content
+                };
+                
+                // Find a line with matched text for element location
+                const lines = bestMatch.content.split('\n');
+                const snippetStart = Math.max(0, bestMatch.firstMatchLine - 4);
+                const snippetEnd = Math.min(lines.length, bestMatch.firstMatchLine + 5);
+                
+                elementLocation = {
+                  lineNumber: bestMatch.firstMatchLine,
+                  snippet: lines.slice(snippetStart, snippetEnd).join('\n'),
+                  confidence: 'high',
+                  matchedBy: `text scoring: ${bestMatch.score} matches in imported file`
+                };
+              }
+            }
+          }
+          
           // DYNAMIC IMPORT SEARCH: If not found in main file, search imported components
           if (!elementLocation) {
             debugLog("Element not in main file, searching imported components...", {

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

@@ -512,6 +512,98 @@ function findElementLineInFile(
   return null;
 }
 
+/**
+ * Search imported component files for specific TEXT CONTENT
+ * This is used to redirect from parent components to child components
+ * when the actual visible text lives in an imported file.
+ * 
+ * Example: "What's the essence of this process?" might be in EssenceTab.tsx
+ * but the parent QuickAddProcessModal.tsx was selected by Phase 2a.
+ */
+/**
+ * Score imported files by how many discovered text strings they contain.
+ * This finds the TRUE file by matching ALL visible text, not just one substring.
+ * 
+ * Example: If annotation discovers ["What's the essence?", "Essence Quality", "Show Example"]
+ * and EssenceTab.tsx contains 2 of them while QuickAddProcessModal.tsx contains 1,
+ * EssenceTab.tsx wins with the higher score.
+ */
+function scoreFilesForTextContent(
+  textContents: string[],
+  importedFiles: { path: string; content: string }[]
+): { path: string; content: string; score: number; matchedTexts: string[]; firstMatchLine: number } | null {
+  if (!textContents || textContents.length === 0) return null;
+  
+  debugLog("Scoring imported files for text content", {
+    textCount: textContents.length,
+    texts: textContents.slice(0, 5).map(t => t.substring(0, 30)),
+    filesCount: importedFiles.length
+  });
+  
+  // Only search component files (where JSX text lives)
+  const componentFiles = importedFiles.filter(f => 
+    f.path.includes('components/') || f.path.includes('/ui/')
+  );
+  
+  const results: { path: string; content: string; score: number; matchedTexts: string[]; firstMatchLine: number }[] = [];
+  
+  for (const file of componentFiles) {
+    let score = 0;
+    const matchedTexts: string[] = [];
+    let firstMatchLine = 0;
+    
+    for (const text of textContents) {
+      if (!text || text.length < 5) continue;
+      
+      // Use first 20 chars for matching (handles dynamic suffixes)
+      const searchText = text.substring(0, 20);
+      
+      // Check if this text exists in the file
+      if (file.content.includes(searchText)) {
+        score++;
+        matchedTexts.push(text.substring(0, 30) + (text.length > 30 ? '...' : ''));
+        
+        // Find line number for first match (for element location)
+        if (firstMatchLine === 0) {
+          const lines = file.content.split('\n');
+          for (let i = 0; i < lines.length; i++) {
+            if (lines[i].includes(searchText)) {
+              firstMatchLine = i + 1;
+              break;
+            }
+          }
+        }
+      }
+    }
+    
+    if (score > 0) {
+      results.push({ 
+        path: file.path, 
+        content: file.content, 
+        score, 
+        matchedTexts,
+        firstMatchLine 
+      });
+    }
+  }
+  
+  // Sort by score descending (highest match count wins)
+  results.sort((a, b) => b.score - a.score);
+  
+  if (results.length > 0) {
+    debugLog("Text scoring results", {
+      topFile: results[0].path,
+      topScore: results[0].score,
+      topMatches: results[0].matchedTexts,
+      allResults: results.slice(0, 3).map(r => ({ path: r.path, score: r.score }))
+    });
+  } else {
+    debugLog("No text matches found in any imported component files");
+  }
+  
+  return results.length > 0 ? results[0] : null;
+}
+
 /**
  * Search imported component files for the focused element
  * Reuses findElementLineInFile() for consistent detection
@@ -1497,7 +1589,7 @@ export async function POST(request: Request) {
             const confirmedPath = phase2aMatches.find(p => 
               focusedElementHints.some(h => h.path === p && h.score > 0)
             );
-            recommendedFile = {
+              recommendedFile = {
               path: confirmedPath!,
               reason: `Phase 2a component-name match confirmed by element search`
             };
@@ -1521,7 +1613,7 @@ export async function POST(request: Request) {
               selectedPath: bestMatch,
               allCandidates: phase2aMatches 
             });
-          } else {
+            } else {
             // FALLBACK PRIORITY 4: Use the page file from the current route
             const routePageFile = discoverPageFile(pageRoute || "/", projectRoot);
             
@@ -1635,7 +1727,7 @@ User Request: "${userPrompt}"
         let elementLocation: { lineNumber: number; snippet: string; confidence: 'high' | 'medium' | 'low'; matchedBy: string } | null = null;
         let actualTargetFile = recommendedFileContent; // May change if we redirect
         
-        if (focusedElements && focusedElements.length > 0) {
+      if (focusedElements && focusedElements.length > 0) {
           for (const el of focusedElements) {
             elementLocation = findElementLineInFile(content, el);
             if (elementLocation) {
@@ -1649,6 +1741,72 @@ User Request: "${userPrompt}"
             }
           }
           
+          // TEXT SCORING REDIRECT: Score ALL imported files by how many discovered
+          // text strings they contain. The file with the highest score is the TRUE target.
+          // This handles parent/child component cases accurately.
+          if (elementLocation && elementLocation.confidence !== 'high') {
+            // Collect ALL text content from focused elements
+            const allTextContent = focusedElements
+              .filter(e => e.textContent && e.textContent.length > 5)
+              .map(e => e.textContent!);
+            
+            if (allTextContent.length > 0) {
+              debugLog("Medium/low confidence match - scoring imports for all text", {
+                currentFile: recommendedFileContent.path,
+                currentConfidence: elementLocation.confidence,
+                textCount: allTextContent.length,
+                texts: allTextContent.slice(0, 5).map(t => t.substring(0, 25))
+              });
+              
+              // Score all imported files
+              const bestMatch = scoreFilesForTextContent(
+                allTextContent,
+                pageContext.componentSources
+              );
+              
+              // Also score the current file for comparison
+              const currentFileScore = allTextContent.filter(text => 
+                recommendedFileContent.content.includes(text.substring(0, 20))
+              ).length;
+              
+              debugLog("Text scoring comparison", {
+                currentFile: recommendedFileContent.path,
+                currentScore: currentFileScore,
+                bestImport: bestMatch?.path || 'none',
+                bestImportScore: bestMatch?.score || 0
+              });
+              
+              // Redirect only if imported file has MORE matches than current file
+              if (bestMatch && bestMatch.score > currentFileScore) {
+                debugLog("TEXT REDIRECT: Imported file has more text matches", {
+                  originalFile: recommendedFileContent.path,
+                  originalScore: currentFileScore,
+                  redirectTo: bestMatch.path,
+                  redirectScore: bestMatch.score,
+                  matchedTexts: bestMatch.matchedTexts
+                });
+                
+                // Switch target file to where most text content lives
+                actualTargetFile = {
+                  path: bestMatch.path,
+                  content: bestMatch.content
+                };
+                
+                // Find a line with matched text for element location
+                const lines = bestMatch.content.split('\n');
+                const snippetStart = Math.max(0, bestMatch.firstMatchLine - 4);
+                const snippetEnd = Math.min(lines.length, bestMatch.firstMatchLine + 5);
+                
+                elementLocation = {
+                  lineNumber: bestMatch.firstMatchLine,
+                  snippet: lines.slice(snippetStart, snippetEnd).join('\n'),
+                  confidence: 'high',
+                  matchedBy: `text scoring: ${bestMatch.score} matches in imported file`
+                };
+              }
+            }
+          }
+          
           // DYNAMIC IMPORT SEARCH: If not found in main file, search imported components
           if (!elementLocation) {
             debugLog("Element not in main file, searching imported components...", {

package/package.json

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