sonance-brand-mcp 1.3.106 → 1.3.108

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

@@ -825,14 +825,21 @@ function findElementInImportedFiles(
         file.path.includes('.d.ts')) continue;
     
     const result = findElementLineInFile(file.content, focusedElement);
-    if (result && result.confidence !== 'low') {
+    // Accept ALL matches including low confidence - let the scoring decide
+    // Low confidence matches from fallback logic (text/class fingerprint) are still useful
+    if (result) {
       let score = 0;
       
       // Text content match is highest priority (+100)
-      if (result.matchedBy.includes('textContent') || result.matchedBy.includes('word match')) {
+      if (result.matchedBy.includes('textContent') || result.matchedBy.includes('word match') || result.matchedBy.includes('flexible text')) {
         score += 100;
       }
       
+      // Class fingerprint match gets medium score (+60)
+      if (result.matchedBy.includes('class fingerprint')) {
+        score += 60;
+      }
+      
       // Directory proximity - same directory tree as page file (+50)
       // e.g., for page src/app/typography/page.tsx, prefer src/app/typography/_components/
       if (pageDir && file.path.includes(pageDir)) {
@@ -845,8 +852,8 @@ function findElementInImportedFiles(
         score += 30;
       }
       
-      // Confidence bonus
-      score += result.confidence === 'high' ? 20 : 10;
+      // Confidence bonus - low confidence still gets some points
+      score += result.confidence === 'high' ? 30 : result.confidence === 'medium' ? 20 : 5;
       
       matches.push({
         path: file.path,
@@ -2028,6 +2035,7 @@ export async function POST(request: Request) {
       let recommendedFile: { path: string; reason: string } | null = null;
       let deterministicMatch: { path: string; lineNumber: number } | null = null;
       let phase1VisibleText: string[] = []; // Store Phase 1 visible text for use in file scoring
+      let smartSearchTopPath: string | null = null; // Top smart search result for fallback
 
       // ========================================================================
       // PHASE 0: Deterministic Element ID Search (Cursor-style explicit selection)
@@ -2134,7 +2142,7 @@ export async function POST(request: Request) {
           // IMPROVED: Relative scoring - compare focused element vs smart search
           // This prevents false positives from generic element IDs in wrong files
           const smartSearchTopScore = searchResults[0]?.score || 0;
-          const smartSearchTopPath = searchResults[0]?.path || null;
+          smartSearchTopPath = searchResults[0]?.path || null; // Assign to outer scope variable
           const focusedTopScore = focusedElementHints[0]?.score || 0;
           const focusedTopPath = focusedElementHints[0]?.path || null;
 
@@ -2299,9 +2307,25 @@ export async function POST(request: Request) {
       
       let usedContext = 0;
       
+      // ========== PHASE 0 PRIORITY: If Phase 0 found a file, use it directly ==========
+      // Phase 0 is deterministic (ID match) so it should ALWAYS take priority
+      let phase0FileContent: { path: string; content: string } | null = null;
+      if (deterministicMatch) {
+        const fullPath = path.join(projectRoot, deterministicMatch.path);
+        if (fs.existsSync(fullPath)) {
+          const content = fs.readFileSync(fullPath, 'utf-8');
+          phase0FileContent = { path: deterministicMatch.path, content };
+          debugLog("PHASE 0 PRIORITY: Using deterministic match as target file", {
+            path: deterministicMatch.path,
+            lineNumber: deterministicMatch.lineNumber,
+            contentLength: content.length
+          });
+        }
+      }
+      
       // Extract recommended file from component sources (to show first, avoid duplication)
       let recommendedFileContent: { path: string; content: string } | null = null;
-      if (recommendedFile) {
+      if (recommendedFile && !phase0FileContent) {
         // Check componentSources first
         const idx = pageContext.componentSources.findIndex(c => c.path === recommendedFile.path);
         if (idx !== -1) {
@@ -2317,10 +2341,28 @@ export async function POST(request: Request) {
       // ========== FILE REDIRECT LOGIC (runs BEFORE building instructions) ==========
       // This determines the ACTUAL target file by checking for text matches in imported components
       // Must run first so that all subsequent code uses the correct file path
-      let actualTargetFile = recommendedFileContent || (pageContext.pageContent ? { path: pageContext.pageFile, content: pageContext.pageContent } : null);
+      // PHASE 0 FILES SKIP REDIRECT - they're already definitively correct
+      let actualTargetFile = phase0FileContent || recommendedFileContent || (pageContext.pageContent ? { path: pageContext.pageFile, content: pageContext.pageContent } : null);
       let elementLocation: { lineNumber: number; snippet: string; confidence: 'high' | 'medium' | 'low'; matchedBy: string } | null = null;
       
-      if (actualTargetFile && focusedElements && focusedElements.length > 0) {
+      // If Phase 0 succeeded, we already know the exact line - use it directly
+      if (phase0FileContent && deterministicMatch) {
+        const lines = phase0FileContent.content.split('\n');
+        const lineNum = deterministicMatch.lineNumber;
+        elementLocation = {
+          lineNumber: lineNum,
+          snippet: lines.slice(Math.max(0, lineNum - 4), lineNum + 5).join('\n'),
+          confidence: 'high',
+          matchedBy: 'Phase 0 deterministic ID match'
+        };
+        debugLog("PHASE 0: Set element location from deterministic match", {
+          lineNumber: lineNum,
+          confidence: 'high'
+        });
+      }
+      
+      // SKIP REDIRECT LOGIC if Phase 0 already determined the file (it's already correct)
+      if (!phase0FileContent && actualTargetFile && focusedElements && focusedElements.length > 0) {
         const content = actualTargetFile.content;
         
         // Search for focused element in the file using multiple strategies
@@ -2469,6 +2511,22 @@ export async function POST(request: Request) {
         }
       }
       
+      // ========== SMART SEARCH FALLBACK ==========
+      // If we still don't have a good target file (just the page wrapper), 
+      // use the smart search top result - it already found relevant files!
+      if (actualTargetFile && actualTargetFile.path === pageContext.pageFile && smartSearchTopPath) {
+        const smartSearchFullPath = path.join(projectRoot, smartSearchTopPath);
+        if (fs.existsSync(smartSearchFullPath)) {
+          const smartSearchContent = fs.readFileSync(smartSearchFullPath, 'utf-8');
+          actualTargetFile = { path: smartSearchTopPath, content: smartSearchContent };
+          debugLog("SMART SEARCH FALLBACK: Using top smart search result as target", {
+            originalFile: pageContext.pageFile,
+            redirectTo: smartSearchTopPath,
+            contentLength: smartSearchContent.length
+          });
+        }
+      }
+      
       debugLog("File redirect complete", {
         originalRecommended: recommendedFileContent?.path || 'none',
         actualTarget: actualTargetFile?.path || 'none',

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

@@ -821,14 +821,21 @@ function findElementInImportedFiles(
         file.path.includes('.d.ts')) continue;
     
     const result = findElementLineInFile(file.content, focusedElement);
-    if (result && result.confidence !== 'low') {
+    // Accept ALL matches including low confidence - let the scoring decide
+    // Low confidence matches from fallback logic (text/class fingerprint) are still useful
+    if (result) {
       let score = 0;
       
       // Text content match is highest priority (+100)
-      if (result.matchedBy.includes('textContent') || result.matchedBy.includes('word match')) {
+      if (result.matchedBy.includes('textContent') || result.matchedBy.includes('word match') || result.matchedBy.includes('flexible text')) {
         score += 100;
       }
       
+      // Class fingerprint match gets medium score (+60)
+      if (result.matchedBy.includes('class fingerprint')) {
+        score += 60;
+      }
+      
       // Directory proximity - same directory tree as page file (+50)
       // e.g., for page src/app/typography/page.tsx, prefer src/app/typography/_components/
       if (pageDir && file.path.includes(pageDir)) {
@@ -841,8 +848,8 @@ function findElementInImportedFiles(
         score += 30;
       }
       
-      // Confidence bonus
-      score += result.confidence === 'high' ? 20 : 10;
+      // Confidence bonus - low confidence still gets some points
+      score += result.confidence === 'high' ? 30 : result.confidence === 'medium' ? 20 : 5;
       
       matches.push({
         path: file.path,
@@ -1997,6 +2004,7 @@ export async function POST(request: Request) {
       let recommendedFile: { path: string; reason: string } | null = null;
       let deterministicMatch: { path: string; lineNumber: number } | null = null;
       let phase1VisibleText: string[] = []; // Store Phase 1 visible text for use in file scoring
+      let smartSearchTopPath: string | null = null; // Top smart search result for fallback
 
       // ========================================================================
       // PHASE 0: Deterministic Element ID Search (Cursor-style explicit selection)
@@ -2103,7 +2111,7 @@ export async function POST(request: Request) {
           // IMPROVED: Relative scoring - compare focused element vs smart search
           // This prevents false positives from generic element IDs in wrong files
           const smartSearchTopScore = searchResults[0]?.score || 0;
-          const smartSearchTopPath = searchResults[0]?.path || null;
+          smartSearchTopPath = searchResults[0]?.path || null; // Assign to outer scope variable
           const focusedTopScore = focusedElementHints[0]?.score || 0;
           const focusedTopPath = focusedElementHints[0]?.path || null;
 
@@ -2268,9 +2276,25 @@ export async function POST(request: Request) {
       
       let usedContext = 0;
       
+      // ========== PHASE 0 PRIORITY: If Phase 0 found a file, use it directly ==========
+      // Phase 0 is deterministic (ID match) so it should ALWAYS take priority
+      let phase0FileContent: { path: string; content: string } | null = null;
+      if (deterministicMatch) {
+        const fullPath = path.join(projectRoot, deterministicMatch.path);
+        if (fs.existsSync(fullPath)) {
+          const content = fs.readFileSync(fullPath, 'utf-8');
+          phase0FileContent = { path: deterministicMatch.path, content };
+          debugLog("PHASE 0 PRIORITY: Using deterministic match as target file", {
+            path: deterministicMatch.path,
+            lineNumber: deterministicMatch.lineNumber,
+            contentLength: content.length
+          });
+        }
+      }
+      
       // Extract recommended file from component sources (to show first, avoid duplication)
       let recommendedFileContent: { path: string; content: string } | null = null;
-      if (recommendedFile) {
+      if (recommendedFile && !phase0FileContent) {
         // Check componentSources first
         const idx = pageContext.componentSources.findIndex(c => c.path === recommendedFile.path);
         if (idx !== -1) {
@@ -2286,10 +2310,28 @@ export async function POST(request: Request) {
       // ========== FILE REDIRECT LOGIC (runs BEFORE building instructions) ==========
       // This determines the ACTUAL target file by checking for text matches in imported components
       // Must run first so that all subsequent code uses the correct file path
-      let actualTargetFile = recommendedFileContent || (pageContext.pageContent ? { path: pageContext.pageFile, content: pageContext.pageContent } : null);
+      // PHASE 0 FILES SKIP REDIRECT - they're already definitively correct
+      let actualTargetFile = phase0FileContent || recommendedFileContent || (pageContext.pageContent ? { path: pageContext.pageFile, content: pageContext.pageContent } : null);
       let elementLocation: { lineNumber: number; snippet: string; confidence: 'high' | 'medium' | 'low'; matchedBy: string } | null = null;
       
-      if (actualTargetFile && focusedElements && focusedElements.length > 0) {
+      // If Phase 0 succeeded, we already know the exact line - use it directly
+      if (phase0FileContent && deterministicMatch) {
+        const lines = phase0FileContent.content.split('\n');
+        const lineNum = deterministicMatch.lineNumber;
+        elementLocation = {
+          lineNumber: lineNum,
+          snippet: lines.slice(Math.max(0, lineNum - 4), lineNum + 5).join('\n'),
+          confidence: 'high',
+          matchedBy: 'Phase 0 deterministic ID match'
+        };
+        debugLog("PHASE 0: Set element location from deterministic match", {
+          lineNumber: lineNum,
+          confidence: 'high'
+        });
+      }
+      
+      // SKIP REDIRECT LOGIC if Phase 0 already determined the file (it's already correct)
+      if (!phase0FileContent && actualTargetFile && focusedElements && focusedElements.length > 0) {
         const content = actualTargetFile.content;
         
         // Search for focused element in the file using multiple strategies
@@ -2438,6 +2480,22 @@ export async function POST(request: Request) {
         }
       }
       
+      // ========== SMART SEARCH FALLBACK ==========
+      // If we still don't have a good target file (just the page wrapper), 
+      // use the smart search top result - it already found relevant files!
+      if (actualTargetFile && actualTargetFile.path === pageContext.pageFile && smartSearchTopPath) {
+        const smartSearchFullPath = path.join(projectRoot, smartSearchTopPath);
+        if (fs.existsSync(smartSearchFullPath)) {
+          const smartSearchContent = fs.readFileSync(smartSearchFullPath, 'utf-8');
+          actualTargetFile = { path: smartSearchTopPath, content: smartSearchContent };
+          debugLog("SMART SEARCH FALLBACK: Using top smart search result as target", {
+            originalFile: pageContext.pageFile,
+            redirectTo: smartSearchTopPath,
+            contentLength: smartSearchContent.length
+          });
+        }
+      }
+      
       debugLog("File redirect complete", {
         originalRecommended: recommendedFileContent?.path || 'none',
         actualTarget: actualTargetFile?.path || 'none',

package/package.json

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