sonance-brand-mcp 1.3.26 → 1.3.28

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

@@ -50,10 +50,34 @@ interface ApplyFirstRequest {
 interface BackupManifest {
   sessionId: string;
   timestamp: number;
-  files: { original: string; backup: string }[];
+  files: { 
+    original: string; 
+    backup: string;
+    isNewFile: boolean;  // Track if file was newly created (needs deletion on revert)
+  }[];
 }
 
 const BACKUP_ROOT = ".sonance-backups";
+const DEBUG_LOG_FILE = "sonance-debug.log";
+
+/**
+ * Debug logging utility - writes to sonance-debug.log in project root
+ * This helps diagnose issues with file discovery, validation, and revert
+ */
+function debugLog(message: string, data?: unknown) {
+  const timestamp = new Date().toISOString();
+  const dataStr = data !== undefined ? `\n${JSON.stringify(data, null, 2)}` : "";
+  const logEntry = `[${timestamp}] [apply] ${message}${dataStr}\n`;
+  
+  try {
+    const logPath = path.join(process.cwd(), DEBUG_LOG_FILE);
+    fs.appendFileSync(logPath, logEntry);
+    // Also log to console for terminal visibility
+    console.log(`[Sonance] ${message}`, data !== undefined ? data : "");
+  } catch (e) {
+    console.error("[Sonance] Failed to write debug log:", e);
+  }
+}
 
 const VISION_SYSTEM_PROMPT = `You are an expert React/TypeScript developer with vision capabilities. You can see screenshots and modify code.
 
@@ -373,6 +397,12 @@ CRITICAL: Your modified file should have approximately the same number of lines
         validFilePaths.add(comp.path);
       }
       
+      debugLog("VALIDATION: Valid file paths from page context", { 
+        pageFile: pageContext.pageFile,
+        validFilePaths: Array.from(validFilePaths),
+        aiRequestedFiles: aiResponse.modifications.map(m => m.filePath)
+      });
+      
       // Process modifications - apply patches to get modified content
       const modifications: VisionFileModification[] = [];
       const patchErrors: string[] = [];
@@ -380,7 +410,15 @@ CRITICAL: Your modified file should have approximately the same number of lines
       for (const mod of aiResponse.modifications) {
         // Validate that the file path is in the page context
         // This prevents the AI from creating new files
-        if (!validFilePaths.has(mod.filePath)) {
+        const isValidPath = validFilePaths.has(mod.filePath);
+        debugLog(`VALIDATION CHECK: ${mod.filePath}`, { 
+          isValidPath, 
+          fileExists: fs.existsSync(path.join(projectRoot, mod.filePath)),
+          validPaths: Array.from(validFilePaths)
+        });
+        
+        if (!isValidPath) {
+          debugLog(`REJECTED: File not in page context`, { filePath: mod.filePath });
           console.warn(`[Apply-First] Rejected modification to unknown file: ${mod.filePath}`);
           console.warn(`[Apply-First] Valid files are: ${Array.from(validFilePaths).join(", ")}`);
           patchErrors.push(`${mod.filePath}: This file was not found in the page context. The AI can only modify existing files that are part of the current page.`);
@@ -512,6 +550,15 @@ async function applyChangesWithBackup(
   const backupDir = path.join(projectRoot, BACKUP_ROOT, sessionId);
   const backupPaths: string[] = [];
 
+  // Track which files exist before we modify them
+  const existingFiles = new Set<string>();
+  for (const mod of modifications) {
+    const fullPath = path.join(projectRoot, mod.filePath);
+    if (fs.existsSync(fullPath)) {
+      existingFiles.add(mod.filePath);
+    }
+  }
+
   try {
     // Step 1: Create backup directory
     fs.mkdirSync(backupDir, { recursive: true });
@@ -520,7 +567,7 @@ async function applyChangesWithBackup(
     for (const mod of modifications) {
       const fullPath = path.join(projectRoot, mod.filePath);
 
-      if (fs.existsSync(fullPath)) {
+      if (existingFiles.has(mod.filePath)) {
         const backupPath = path.join(backupDir, mod.filePath);
         const backupDirForFile = path.dirname(backupPath);
 
@@ -530,13 +577,14 @@ async function applyChangesWithBackup(
       }
     }
 
-    // Step 3: Write manifest
+    // Step 3: Write manifest (track which files are new)
     const manifest: BackupManifest = {
       sessionId,
       timestamp: Date.now(),
       files: modifications.map(m => ({
         original: m.filePath,
         backup: path.join(backupDir, m.filePath),
+        isNewFile: !existingFiles.has(m.filePath),  // Track if file was newly created
       })),
     };
 
@@ -589,9 +637,11 @@ async function revertFromBackups(
   sessionId: string,
   projectRoot: string
 ): Promise<{ success: boolean; message: string; filesReverted: number }> {
+  debugLog("revertFromBackups called", { sessionId, projectRoot });
   const backupDir = path.join(projectRoot, BACKUP_ROOT, sessionId);
 
   if (!fs.existsSync(backupDir)) {
+    debugLog("REVERT ERROR: Backup directory not found", { backupDir });
     return {
       success: false,
       message: "Backup session not found",
@@ -603,7 +653,7 @@ async function revertFromBackups(
     const manifestPath = path.join(backupDir, "manifest.json");
 
     if (!fs.existsSync(manifestPath)) {
-      // If no manifest, try to restore any backup files we find
+      debugLog("REVERT ERROR: Manifest not found", { manifestPath });
       return {
         success: false,
         message: "Backup manifest not found",
@@ -614,26 +664,66 @@ async function revertFromBackups(
     const manifest: BackupManifest = JSON.parse(
       fs.readFileSync(manifestPath, "utf-8")
     );
+    
+    debugLog("REVERT: Loaded manifest", { 
+      sessionId: manifest.sessionId,
+      fileCount: manifest.files.length,
+      files: manifest.files.map(f => ({ path: f.original, isNewFile: f.isNewFile }))
+    });
 
     let filesReverted = 0;
+    let filesDeleted = 0;
 
     for (const file of manifest.files) {
-      const backupPath = path.join(backupDir, file.original);
       const originalPath = path.join(projectRoot, file.original);
+      debugLog(`REVERT: Processing file`, { 
+        filePath: file.original, 
+        isNewFile: file.isNewFile,
+        fileExists: fs.existsSync(originalPath)
+      });
 
-      if (fs.existsSync(backupPath)) {
-        fs.copyFileSync(backupPath, originalPath);
-        filesReverted++;
+      // Handle new files - delete them
+      if (file.isNewFile) {
+        debugLog(`REVERT: Attempting to delete new file`, { originalPath });
+        if (fs.existsSync(originalPath)) {
+          fs.unlinkSync(originalPath);
+          filesDeleted++;
+          debugLog(`REVERT: Successfully deleted new file`, { originalPath });
+          console.log(`[Revert] Deleted new file: ${file.original}`);
+          
+          // Clean up empty parent directories
+          try {
+            const parentDir = path.dirname(originalPath);
+            const entries = fs.readdirSync(parentDir);
+            if (entries.length === 0) {
+              fs.rmdirSync(parentDir);
+              console.log(`[Revert] Removed empty directory: ${path.dirname(file.original)}`);
+            }
+          } catch {
+            // Ignore errors when cleaning up directories
+          }
+        }
+      } else {
+        // Handle existing files - restore from backup
+        const backupPath = path.join(backupDir, file.original);
+        if (fs.existsSync(backupPath)) {
+          fs.copyFileSync(backupPath, originalPath);
+          filesReverted++;
+        }
       }
     }
 
     // Delete backup directory after successful revert
     fs.rmSync(backupDir, { recursive: true });
 
+    const message = filesDeleted > 0 
+      ? `Reverted ${filesReverted} file(s), deleted ${filesDeleted} new file(s)`
+      : `Reverted ${filesReverted} file(s)`;
+
     return {
       success: true,
-      message: `Reverted ${filesReverted} file(s)`,
-      filesReverted,
+      message,
+      filesReverted: filesReverted + filesDeleted,
     };
   } catch (error) {
     console.error("Error reverting from backups:", error);
@@ -812,8 +902,11 @@ function gatherPageContext(
 }
 
 function discoverPageFile(route: string, projectRoot: string): string | null {
+  debugLog("discoverPageFile called", { route, projectRoot });
+  
   // Handle root route
   if (route === "/" || route === "") {
+    debugLog("Handling root route, checking patterns...");
     const rootPatterns = [
       // App Router patterns
       "src/app/page.tsx",
@@ -860,18 +953,26 @@ function discoverPageFile(route: string, projectRoot: string): string | null {
   ];
 
   for (const pattern of patterns) {
-    if (fs.existsSync(path.join(projectRoot, pattern))) {
+    const fullPath = path.join(projectRoot, pattern);
+    const exists = fs.existsSync(fullPath);
+    debugLog(`Checking exact pattern: ${pattern}`, { exists });
+    if (exists) {
+      debugLog("Found exact match!", { pattern });
       return pattern;
     }
   }
 
+  debugLog("No exact match found, trying dynamic route matching...");
+  
   // If exact match not found, try dynamic route matching
   // e.g., /processes/123 -> src/pages/processes/[id].tsx
   const dynamicResult = findDynamicRoute(cleanRoute, projectRoot);
   if (dynamicResult) {
+    debugLog("Found dynamic route match!", { dynamicResult });
     return dynamicResult;
   }
 
+  debugLog("discoverPageFile: NO FILE FOUND for route", { route });
   return null;
 }
 
@@ -881,6 +982,7 @@ function discoverPageFile(route: string, projectRoot: string): string | null {
  */
 function findDynamicRoute(cleanRoute: string, projectRoot: string): string | null {
   const segments = cleanRoute.split("/");
+  debugLog("findDynamicRoute called", { cleanRoute, segments });
   
   // Try replacing the last segment with dynamic patterns
   const baseDirs = [
@@ -895,7 +997,10 @@ function findDynamicRoute(cleanRoute: string, projectRoot: string): string | nul
   
   for (const baseDir of baseDirs) {
     const basePath = path.join(projectRoot, baseDir);
-    if (!fs.existsSync(basePath)) continue;
+    if (!fs.existsSync(basePath)) {
+      debugLog(`Base dir does not exist: ${baseDir}`);
+      continue;
+    }
     
     // Build path with all segments except the last one
     const parentSegments = segments.slice(0, -1);
@@ -903,6 +1008,8 @@ function findDynamicRoute(cleanRoute: string, projectRoot: string): string | nul
       ? path.join(basePath, ...parentSegments)
       : basePath;
     
+    debugLog(`Checking parent path for dynamic routes`, { baseDir, parentPath, exists: fs.existsSync(parentPath) });
+    
     if (!fs.existsSync(parentPath)) continue;
     
     // Check for dynamic route files
@@ -933,31 +1040,43 @@ function findDynamicRoute(cleanRoute: string, projectRoot: string): string | nul
     // Also scan directory for any dynamic segment pattern [...]
     try {
       const entries = fs.readdirSync(parentPath, { withFileTypes: true });
+      const dynamicEntries = entries.filter(e => e.name.startsWith("[") && e.name.includes("]"));
+      debugLog(`Scanning directory for dynamic segments`, { parentPath, dynamicEntries: dynamicEntries.map(e => e.name) });
+      
       for (const entry of entries) {
         if (entry.name.startsWith("[") && entry.name.includes("]")) {
+          debugLog(`Found dynamic segment: ${entry.name}`, { isDirectory: entry.isDirectory() });
           for (const ext of extensions) {
             if (entry.isDirectory()) {
               // App Router or Pages Router with folder
               const pagePath = path.join(parentPath, entry.name, `page${ext}`);
               const indexPath = path.join(parentPath, entry.name, `index${ext}`);
+              debugLog(`Checking dynamic folder paths`, { pagePath, indexPath, pageExists: fs.existsSync(pagePath), indexExists: fs.existsSync(indexPath) });
               if (fs.existsSync(pagePath)) {
-                return path.relative(projectRoot, pagePath);
+                const result = path.relative(projectRoot, pagePath);
+                debugLog(`Found dynamic route (folder/page)!`, { result });
+                return result;
               }
               if (fs.existsSync(indexPath)) {
-                return path.relative(projectRoot, indexPath);
+                const result = path.relative(projectRoot, indexPath);
+                debugLog(`Found dynamic route (folder/index)!`, { result });
+                return result;
               }
             } else if (entry.isFile() && entry.name.endsWith(ext)) {
               // Pages Router file-based
-              return path.relative(projectRoot, path.join(parentPath, entry.name));
+              const result = path.relative(projectRoot, path.join(parentPath, entry.name));
+              debugLog(`Found dynamic route (file)!`, { result });
+              return result;
             }
           }
         }
       }
-    } catch {
-      // Skip if directory can't be read
+    } catch (e) {
+      debugLog(`Error scanning directory: ${parentPath}`, { error: String(e) });
     }
   }
   
+  debugLog("findDynamicRoute: NO DYNAMIC ROUTE FOUND");
   return null;
 }
 

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

@@ -56,6 +56,27 @@ interface VisionEditResponse {
   error?: string;
 }
 
+const DEBUG_LOG_FILE = "sonance-debug.log";
+
+/**
+ * Debug logging utility - writes to sonance-debug.log in project root
+ * This helps diagnose issues with file discovery, validation, and revert
+ */
+function debugLog(message: string, data?: unknown) {
+  const timestamp = new Date().toISOString();
+  const dataStr = data !== undefined ? `\n${JSON.stringify(data, null, 2)}` : "";
+  const logEntry = `[${timestamp}] [edit] ${message}${dataStr}\n`;
+  
+  try {
+    const logPath = path.join(process.cwd(), DEBUG_LOG_FILE);
+    fs.appendFileSync(logPath, logEntry);
+    // Also log to console for terminal visibility
+    console.log(`[Sonance] ${message}`, data !== undefined ? data : "");
+  } catch (e) {
+    console.error("[Sonance] Failed to write debug log:", e);
+  }
+}
+
 const VISION_SYSTEM_PROMPT = `You are an expert React/TypeScript developer with vision capabilities. You can see screenshots and modify code.
 
 ═══════════════════════════════════════════════════════════════════════════════
@@ -381,12 +402,19 @@ CRITICAL: Only use file paths from the VALID FILES list above. Do NOT create new
         validPaths.add(comp.path);
       }
 
+      debugLog("VALIDATION: Valid file paths from page context", { 
+        pageFile: pageContext.pageFile,
+        validPaths: Array.from(validPaths),
+        aiRequestedFiles: (aiResponse.modifications || []).map(m => m.filePath)
+      });
+
       // Validate AI response - reject any file paths not in our valid list
       const invalidMods = (aiResponse.modifications || []).filter(
         (mod) => !validPaths.has(mod.filePath)
       );
 
       if (invalidMods.length > 0) {
+        debugLog("REJECTED: AI attempted to create new files", { invalidMods: invalidMods.map(m => m.filePath) });
         console.error(
           "AI attempted to create new files:",
           invalidMods.map((m) => m.filePath)
@@ -761,8 +789,11 @@ function gatherPageContext(
  * Supports both App Router (src/app/) and Pages Router (src/pages/, pages/)
  */
 function discoverPageFile(route: string, projectRoot: string): string | null {
+  debugLog("discoverPageFile called", { route, projectRoot });
+  
   // Handle root route
   if (route === "/" || route === "") {
+    debugLog("Handling root route, checking patterns...");
     const rootPatterns = [
       // App Router patterns
       "src/app/page.tsx",
@@ -810,18 +841,26 @@ function discoverPageFile(route: string, projectRoot: string): string | null {
   ];
 
   for (const pattern of patterns) {
-    if (fs.existsSync(path.join(projectRoot, pattern))) {
+    const fullPath = path.join(projectRoot, pattern);
+    const exists = fs.existsSync(fullPath);
+    debugLog(`Checking exact pattern: ${pattern}`, { exists });
+    if (exists) {
+      debugLog("Found exact match!", { pattern });
       return pattern;
     }
   }
 
+  debugLog("No exact match found, trying dynamic route matching...");
+  
   // If exact match not found, try dynamic route matching
   // e.g., /processes/123 -> src/pages/processes/[id].tsx
   const dynamicResult = findDynamicRoute(cleanRoute, projectRoot);
   if (dynamicResult) {
+    debugLog("Found dynamic route match!", { dynamicResult });
     return dynamicResult;
   }
 
+  debugLog("discoverPageFile: NO FILE FOUND for route", { route });
   return null;
 }
 
@@ -831,6 +870,7 @@ function discoverPageFile(route: string, projectRoot: string): string | null {
  */
 function findDynamicRoute(cleanRoute: string, projectRoot: string): string | null {
   const segments = cleanRoute.split("/");
+  debugLog("findDynamicRoute called", { cleanRoute, segments });
   
   // Try replacing the last segment with dynamic patterns
   const baseDirs = [
@@ -845,7 +885,10 @@ function findDynamicRoute(cleanRoute: string, projectRoot: string): string | nul
   
   for (const baseDir of baseDirs) {
     const basePath = path.join(projectRoot, baseDir);
-    if (!fs.existsSync(basePath)) continue;
+    if (!fs.existsSync(basePath)) {
+      debugLog(`Base dir does not exist: ${baseDir}`);
+      continue;
+    }
     
     // Build path with all segments except the last one
     const parentSegments = segments.slice(0, -1);
@@ -853,6 +896,8 @@ function findDynamicRoute(cleanRoute: string, projectRoot: string): string | nul
       ? path.join(basePath, ...parentSegments)
       : basePath;
     
+    debugLog(`Checking parent path for dynamic routes`, { baseDir, parentPath, exists: fs.existsSync(parentPath) });
+    
     if (!fs.existsSync(parentPath)) continue;
     
     // Check for dynamic route files
@@ -883,31 +928,43 @@ function findDynamicRoute(cleanRoute: string, projectRoot: string): string | nul
     // Also scan directory for any dynamic segment pattern [...]
     try {
       const entries = fs.readdirSync(parentPath, { withFileTypes: true });
+      const dynamicEntries = entries.filter(e => e.name.startsWith("[") && e.name.includes("]"));
+      debugLog(`Scanning directory for dynamic segments`, { parentPath, dynamicEntries: dynamicEntries.map(e => e.name) });
+      
       for (const entry of entries) {
         if (entry.name.startsWith("[") && entry.name.includes("]")) {
+          debugLog(`Found dynamic segment: ${entry.name}`, { isDirectory: entry.isDirectory() });
           for (const ext of extensions) {
             if (entry.isDirectory()) {
               // App Router or Pages Router with folder
               const pagePath = path.join(parentPath, entry.name, `page${ext}`);
               const indexPath = path.join(parentPath, entry.name, `index${ext}`);
+              debugLog(`Checking dynamic folder paths`, { pagePath, indexPath, pageExists: fs.existsSync(pagePath), indexExists: fs.existsSync(indexPath) });
               if (fs.existsSync(pagePath)) {
-                return path.relative(projectRoot, pagePath);
+                const result = path.relative(projectRoot, pagePath);
+                debugLog(`Found dynamic route (folder/page)!`, { result });
+                return result;
               }
               if (fs.existsSync(indexPath)) {
-                return path.relative(projectRoot, indexPath);
+                const result = path.relative(projectRoot, indexPath);
+                debugLog(`Found dynamic route (folder/index)!`, { result });
+                return result;
               }
             } else if (entry.isFile() && entry.name.endsWith(ext)) {
               // Pages Router file-based
-              return path.relative(projectRoot, path.join(parentPath, entry.name));
+              const result = path.relative(projectRoot, path.join(parentPath, entry.name));
+              debugLog(`Found dynamic route (file)!`, { result });
+              return result;
             }
           }
         }
       }
-    } catch {
-      // Skip if directory can't be read
+    } catch (e) {
+      debugLog(`Error scanning directory: ${parentPath}`, { error: String(e) });
     }
   }
   
+  debugLog("findDynamicRoute: NO DYNAMIC ROUTE FOUND");
   return null;
 }
 

package/package.json

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