sonance-brand-mcp 1.3.26 → 1.3.27

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

@@ -50,7 +50,11 @@ 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";
@@ -512,6 +516,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 +533,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 +543,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
       })),
     };
 
@@ -616,24 +630,51 @@ async function revertFromBackups(
     );
 
     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);
 
-      if (fs.existsSync(backupPath)) {
-        fs.copyFileSync(backupPath, originalPath);
-        filesReverted++;
+      // Handle new files - delete them
+      if (file.isNewFile) {
+        if (fs.existsSync(originalPath)) {
+          fs.unlinkSync(originalPath);
+          filesDeleted++;
+          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);

package/package.json

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