sonance-brand-mcp 1.3.107 → 1.3.109

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

@@ -2035,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)
@@ -2141,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;
 
@@ -2510,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

@@ -2004,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)
@@ -2110,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;
 
@@ -2479,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/dist/index.js

@@ -1397,33 +1397,145 @@ function getImageMimeType(filePath) {
         default: return 'application/octet-stream';
     }
 }
+/**
+ * Centralized logo path mapping for all brands
+ * Light = logo for light backgrounds (typically darker logo)
+ * Dark = logo for dark backgrounds (typically lighter/reversed logo)
+ */
+const LOGO_MAP = {
+    default: {
+        light: "/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Dark.png",
+        dark: "/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Light.png",
+    },
+    sonance: {
+        light: "/logos/sonance/Sonance_Logo_2C_Dark_RGB.png",
+        dark: "/logos/sonance/Sonance_Logo_2C_Light_RGB.png",
+    },
+    iport: {
+        light: "/logos/iport/IPORT_Sonance_LockUp_2C_Dark_RGB.png",
+        dark: "/logos/iport/IPORT_Sonance_LockUp_2C_Light_RGB.png",
+    },
+    blaze: {
+        light: "/logos/blaze/BlazeBySonance_Logo_Lockup_3C_Dark_RGB_05162025.png",
+        dark: "/logos/blaze/BlazeBySonance_Logo_Lockup_2C_Light_RGB_05162025.png",
+    },
+    james: {
+        light: "/logos/james/James_Logo_Black_RGB.png",
+        dark: "/logos/james/James_Logo_Reverse_RGB.png",
+    },
+};
+// Validation state for logo map
+let logoMapValidated = false;
+let logoMapValidationErrors = [];
+/**
+ * Validate that all logos in LOGO_MAP exist
+ * Called once on first logo-using tool invocation
+ */
+function validateLogoMap() {
+    if (logoMapValidated) {
+        return { valid: logoMapValidationErrors.length === 0, errors: logoMapValidationErrors };
+    }
+    const errors = [];
+    for (const [key, paths] of Object.entries(LOGO_MAP)) {
+        for (const [theme, logoPath] of Object.entries(paths)) {
+            const resolvedPath = resolveLogoPath(logoPath);
+            if (!resolvedPath || !fs.existsSync(resolvedPath)) {
+                errors.push(`LOGO_MAP["${key}"].${theme}: "${logoPath}" not found`);
+            }
+        }
+    }
+    logoMapValidated = true;
+    logoMapValidationErrors = errors;
+    if (errors.length > 0) {
+        console.error(`[Logo Validation] ${errors.length} logo(s) not found:`);
+        errors.forEach(e => console.error(`  - ${e}`));
+    }
+    return { valid: errors.length === 0, errors };
+}
 /**
  * Embed a logo as base64 image content for MCP responses
- * Returns an MCP-compatible image content block or null if logo not found
+ * Returns either a success result with image data or an error result with details
  */
 async function embedLogo(logoPath) {
+    // Validate input
+    if (!logoPath || typeof logoPath !== 'string') {
+        return {
+            success: false,
+            error: "invalid_path",
+            message: "Logo path is empty or invalid",
+            requestedPath: String(logoPath),
+        };
+    }
+    const resolvedPath = resolveLogoPath(logoPath);
+    if (!resolvedPath) {
+        return {
+            success: false,
+            error: "invalid_path",
+            message: `Failed to resolve logo path: ${logoPath}`,
+            requestedPath: logoPath,
+        };
+    }
+    // Check if file exists
+    if (!fs.existsSync(resolvedPath)) {
+        return {
+            success: false,
+            error: "not_found",
+            message: `Logo file does not exist`,
+            requestedPath: logoPath,
+            resolvedPath,
+        };
+    }
     try {
-        const resolvedPath = resolveLogoPath(logoPath);
-        if (!resolvedPath)
-            return null;
-        // Check if file exists
-        if (!fs.existsSync(resolvedPath)) {
-            console.error(`Logo not found: ${resolvedPath}`);
-            return null;
-        }
         // Read file as base64
         const fileBuffer = fs.readFileSync(resolvedPath);
         const base64Data = fileBuffer.toString('base64');
         const mimeType = getImageMimeType(resolvedPath);
         return {
+            success: true,
             type: "image",
             data: base64Data,
             mimeType,
         };
     }
     catch (error) {
-        console.error(`Error embedding logo: ${error}`);
-        return null;
+        return {
+            success: false,
+            error: "read_error",
+            message: `Failed to read logo file: ${error instanceof Error ? error.message : String(error)}`,
+            requestedPath: logoPath,
+            resolvedPath,
+        };
+    }
+}
+/**
+ * Embed a logo and add appropriate content blocks with warnings if it fails
+ * Returns true if logo was embedded successfully, false otherwise
+ */
+async function embedLogoWithFeedback(logoPath, label, contentBlocks, options = {}) {
+    const result = await embedLogo(logoPath);
+    if (result.success) {
+        contentBlocks.push({
+            type: "text",
+            text: `## ${label}\n**Path:** \`${logoPath}\``
+        });
+        contentBlocks.push({
+            type: "image",
+            data: result.data,
+            mimeType: result.mimeType,
+        });
+        return true;
+    }
+    else {
+        // Add warning to response unless silent mode
+        if (!options.silent) {
+            contentBlocks.push({
+                type: "text",
+                text: `> **Warning:** Could not embed ${label.toLowerCase()}: ${result.message}\n> Requested path: \`${logoPath}\`${result.resolvedPath ? `\n> Resolved to: \`${result.resolvedPath}\`` : ''}\n> Use \`list_logos\` to see available logos.`,
+            });
+        }
+        // Always log to console for debugging
+        console.error(`[Logo Embed Failed] ${label}: ${result.message} (path: ${logoPath})`);
+        return false;
     }
 }
 /**
@@ -2427,7 +2539,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
             },
             {
                 name: "get_logo",
-                description: "Returns a brand logo image as base64-encoded data that AI can view and analyze. Use list_logos first to see available logos, then use this tool to retrieve the actual image.",
+                description: "Returns a brand logo with ready-to-use implementation code. USE THIS when user asks to 'add logo', 'put the logo', 'include the logo', or needs a specific logo for their code. Returns: the logo image, React/Next.js code snippets, sizing guidelines, and recommendations for light/dark mode variants. Use list_logos first to see available logos.",
                 inputSchema: {
                     type: "object",
                     properties: {
@@ -2439,6 +2551,15 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                     required: ["logo_path"],
                 },
             },
+            {
+                name: "diagnose_logos",
+                description: "Diagnose logo loading issues. Returns validation status, path resolution examples, and environment info. Use this when logos fail to embed or appear in responses.",
+                inputSchema: {
+                    type: "object",
+                    properties: {},
+                    required: [],
+                },
+            },
             {
                 name: "get_full_library",
                 description: "RECOMMENDED FOR APP REDESIGNS & FULL APPLICATIONS: Returns the complete component library including brand guidelines, CSS theme, utilities, and all component source code. USE THIS TOOL when the user asks to: redesign an application, rebrand an app, build a full application, update an entire codebase to use Sonance/IPORT/Blaze brand, or any multi-file/multi-component project. This is the go-to tool for comprehensive brand implementation across an entire project.",
@@ -2890,6 +3011,87 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 const base64Data = imageBuffer.toString('base64');
                 const mimeType = getImageMimeType(resolvedPath);
                 const fileName = path.basename(resolvedPath);
+                // Determine logo variant info for recommendations
+                const isLightLogo = /light|reverse|white/i.test(fileName);
+                const isDarkLogo = /dark|black/i.test(fileName);
+                const backgroundGuidance = isLightLogo
+                    ? "**Use on:** Dark backgrounds (charcoal, dark gray, black)"
+                    : isDarkLogo
+                        ? "**Use on:** Light backgrounds (white, light gray)"
+                        : "**Use on:** Check filename for light/dark variant guidance";
+                // Suggest complementary variant
+                let variantSuggestion = "";
+                if (isLightLogo) {
+                    const darkVariant = fileName.replace(/light/gi, "Dark").replace(/reverse/gi, "Dark").replace(/white/gi, "Black");
+                    variantSuggestion = `**For light backgrounds:** Look for \`${darkVariant}\` or a "Dark" variant`;
+                }
+                else if (isDarkLogo) {
+                    const lightVariant = fileName.replace(/dark/gi, "Light").replace(/black/gi, "Reverse");
+                    variantSuggestion = `**For dark backgrounds:** Look for \`${lightVariant}\` or a "Light/Reverse" variant`;
+                }
+                // Determine brand for alt text
+                const brandMatch = logoPath.match(/\/(sonance|iport|blaze|james|trufig)/i);
+                const brandName = brandMatch ? brandMatch[1].charAt(0).toUpperCase() + brandMatch[1].slice(1) : "Brand";
+                // Normalize the path for code examples (ensure it starts with /logos/)
+                const normalizedPath = logoPath.startsWith('/') ? logoPath : `/logos/${logoPath.replace(/^logos\//, '')}`;
+                const implementationGuide = `## Logo: ${fileName}
+
+${backgroundGuidance}
+${variantSuggestion}
+
+---
+
+### Implementation (Next.js with next/image)
+\`\`\`tsx
+import Image from 'next/image';
+
+// In your component:
+<Image
+  src="${normalizedPath}"
+  alt="${brandName}"
+  width={150}
+  height={40}
+  className="h-10 w-auto"
+  priority // Add if logo is above the fold
+/>
+\`\`\`
+
+### Implementation (React/HTML)
+\`\`\`tsx
+<img
+  src="${normalizedPath}"
+  alt="${brandName}"
+  className="h-10 w-auto"
+/>
+\`\`\`
+
+### With Dark Mode Support
+\`\`\`tsx
+{/* Use CSS to swap logos based on theme */}
+<img
+  src="${normalizedPath}"
+  alt="${brandName}"
+  className="h-10 w-auto dark:hidden"  {/* Light mode */}
+/>
+<img
+  src="${normalizedPath.replace(/dark/gi, 'Light').replace(/black/gi, 'Reverse')}"
+  alt="${brandName}"
+  className="h-10 w-auto hidden dark:block"  {/* Dark mode */}
+/>
+\`\`\`
+
+### Sizing Guidelines
+| Location | Tailwind Class | Pixels |
+|----------|---------------|--------|
+| Header/Navbar | \`h-8\` to \`h-10\` | 32-40px |
+| Hero Section | \`h-12\` to \`h-16\` | 48-64px |
+| Footer | \`h-6\` to \`h-8\` | 24-32px |
+| Mobile Header | \`h-6\` to \`h-8\` | 24-32px |
+
+---
+
+**File Info:** ${imageBuffer.length} bytes, ${mimeType}
+**Full Path:** \`${normalizedPath}\``;
                 return {
                     content: [
                         {
@@ -2899,7 +3101,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                         },
                         {
                             type: "text",
-                            text: `**Logo:** ${fileName}\n**Path:** ${logoPath}\n**Size:** ${imageBuffer.length} bytes\n**Format:** ${mimeType}`,
+                            text: implementationGuide,
                         },
                     ],
                 };
@@ -2911,6 +3113,88 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 };
             }
         }
+        case "diagnose_logos": {
+            // Run validation and collect diagnostic info
+            const validation = validateLogoMap();
+            const samplePath = "/logos/sonance/Sonance_Logo_2C_Dark_RGB.png";
+            const resolvedSample = resolveLogoPath(samplePath);
+            // Count logos in manifest or directory
+            let logoCount = "Unknown";
+            try {
+                if (IS_BUNDLED) {
+                    const manifestPath = getAssetPath("logos");
+                    if (fs.existsSync(manifestPath)) {
+                        const manifest = JSON.parse(fs.readFileSync(manifestPath, "utf-8"));
+                        logoCount = `${manifest.length} logos in manifest`;
+                    }
+                    else {
+                        logoCount = "Manifest file not found";
+                    }
+                }
+                else {
+                    const logosDir = getAssetPath("logos");
+                    if (fs.existsSync(logosDir)) {
+                        // Recursive count
+                        const countFiles = (dir) => {
+                            let count = 0;
+                            const entries = fs.readdirSync(dir, { withFileTypes: true });
+                            for (const entry of entries) {
+                                if (entry.isDirectory()) {
+                                    count += countFiles(path.join(dir, entry.name));
+                                }
+                                else if (/\.(png|svg|jpg|jpeg)$/i.test(entry.name)) {
+                                    count++;
+                                }
+                            }
+                            return count;
+                        };
+                        logoCount = `${countFiles(logosDir)} logos in directory`;
+                    }
+                    else {
+                        logoCount = "Logos directory not found";
+                    }
+                }
+            }
+            catch (e) {
+                logoCount = `Error counting: ${e}`;
+            }
+            const diagnostics = `# Logo System Diagnostics
+
+## Environment
+- **Bundled Mode:** ${IS_BUNDLED}
+- **Assets Path:** ${BUNDLED_ASSETS}
+- **Assets Exist:** ${fs.existsSync(BUNDLED_ASSETS)}
+- **Dev Root:** ${DEV_PROJECT_ROOT}
+
+## Logo Map Validation
+- **Status:** ${validation.valid ? "PASS - All logos found" : "FAIL - Some logos missing"}
+${validation.errors.length > 0 ? `- **Errors:**\n${validation.errors.map(e => `  - ${e}`).join('\n')}` : "- **Errors:** None"}
+
+## Path Resolution Test
+- **Input:** \`${samplePath}\`
+- **Resolved:** \`${resolvedSample || "NULL - Resolution failed"}\`
+- **Exists:** ${resolvedSample ? (fs.existsSync(resolvedSample) ? "Yes" : "No - File not found") : "N/A"}
+
+## Logo Inventory
+- **Count:** ${logoCount}
+- **Source:** ${IS_BUNDLED ? getAssetPath("logos") : getAssetPath("logos")}
+
+## Centralized LOGO_MAP
+${Object.entries(LOGO_MAP).map(([key, paths]) => `- **${key}:**
+  - light: \`${paths.light}\`
+  - dark: \`${paths.dark}\``).join('\n')}
+
+## Recommendations
+${!validation.valid ? `
+1. Check that logo files exist at the expected paths
+2. Run \`npm run mcp:build\` to rebundle assets if in bundled mode
+3. Verify \`public/logos/\` directory structure in development mode
+` : "Logo system is functioning correctly. If you still experience issues, check the specific logo path being requested."}
+`;
+            return {
+                content: [{ type: "text", text: diagnostics }],
+            };
+        }
         case "get_full_library": {
             try {
                 let fullLibrary = "# Sonance Component Library\n\n";
@@ -2957,8 +3241,60 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 catch (e) {
                     fullLibrary += "*Components directory not found*\n";
                 }
+                // Add logo section with implementation guidance
+                fullLibrary += "\n\n---\n\n## 5. Brand Logos\n\n";
+                fullLibrary += "Use these logos in your application. The Sonance+James+IPORT lockup is the default.\n\n";
+                fullLibrary += `### Logo Paths
+| Variant | Path | Use On |
+|---------|------|--------|
+| **Light Backgrounds** | \`/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Dark.png\` | White, light gray backgrounds |
+| **Dark Backgrounds** | \`/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Light.png\` | Charcoal, dark backgrounds |
+
+### Implementation (Next.js)
+\`\`\`tsx
+import Image from 'next/image';
+
+// Light mode logo (for light backgrounds)
+<Image
+  src="/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Dark.png"
+  alt="Sonance"
+  width={150}
+  height={40}
+  className="h-10 w-auto dark:hidden"
+  priority
+/>
+
+// Dark mode logo (for dark backgrounds)
+<Image
+  src="/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Light.png"
+  alt="Sonance"
+  width={150}
+  height={40}
+  className="h-10 w-auto hidden dark:block"
+  priority
+/>
+\`\`\`
+
+### Sizing Guidelines
+- Header/Navbar: \`h-8\` to \`h-10\` (32-40px)
+- Hero Section: \`h-12\` to \`h-16\` (48-64px)
+- Footer: \`h-6\` to \`h-8\` (24-32px)
+
+### Other Brand Logos
+Use \`list_logos\` to see all available logos, or \`get_logo\` to retrieve a specific logo with implementation code.
+`;
+                // Build content blocks with embedded logos
+                const contentBlocks = [];
+                // Validate and embed logos
+                validateLogoMap();
+                const lightLogoPath = LOGO_MAP.default.light;
+                const darkLogoPath = LOGO_MAP.default.dark;
+                await embedLogoWithFeedback(lightLogoPath, "Brand Logo (for Light Backgrounds)", contentBlocks);
+                await embedLogoWithFeedback(darkLogoPath, "Brand Logo (for Dark Backgrounds)", contentBlocks);
+                // Add the full library text
+                contentBlocks.push({ type: "text", text: fullLibrary });
                 return {
-                    content: [{ type: "text", text: fullLibrary }],
+                    content: contentBlocks,
                 };
             }
             catch (e) {
@@ -3000,31 +3336,8 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
             else if (!rawArgs.theme) {
                 defaultsUsed.push("theme → Light");
             }
-            // Logo path mapping based on preference and theme
-            // Light theme = dark logo (for light backgrounds), Dark theme = light logo (for dark backgrounds)
-            const logoMap = {
-                default: {
-                    light: "/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Dark.png",
-                    dark: "/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Light.png",
-                },
-                sonance: {
-                    light: "/logos/sonance/Sonance_Logo_2C_Dark_RGB.png",
-                    dark: "/logos/sonance/Sonance_Logo_2C_Light_RGB.png",
-                },
-                iport: {
-                    light: "/logos/iport/IPORT_Sonance_LockUp_2C_Dark_RGB.png",
-                    dark: "/logos/iport/IPORT_Sonance_LockUp_2C_Light_RGB.png",
-                },
-                blaze: {
-                    light: "/logos/blaze/BlazeBySonance_Logo_Lockup_3C_Dark_RGB_05162025.png",
-                    dark: "/logos/blaze/BlazeBySonance_Logo_Lockup_2C_Light_RGB_05162025.png",
-                },
-                james: {
-                    light: "/logos/james/James_Logo_Black_RGB.png",
-                    dark: "/logos/james/James_Logo_Reverse_RGB.png",
-                },
-            };
-            const logoPath = logoMap[logoPreference]?.[theme] || logoMap.default[theme];
+            // Use centralized LOGO_MAP for logo paths
+            const logoPath = LOGO_MAP[logoPreference]?.[theme] || LOGO_MAP.default[theme];
             const logoPreferenceLabel = logoPreference === "default"
                 ? "Sonance + James + IPORT Lockup"
                 : logoPreference === "james"
@@ -3366,8 +3679,8 @@ const example = "Sonance";
             let response;
             if (isDualThemeMode) {
                 // DUAL-THEME MODE: Provide both light and dark tokens for UI
-                const lightLogoPath = logoMap[logoPreference]?.light || logoMap.default.light;
-                const darkLogoPath = logoMap[logoPreference]?.dark || logoMap.default.dark;
+                const lightLogoPath = LOGO_MAP[logoPreference]?.light || LOGO_MAP.default.light;
+                const darkLogoPath = LOGO_MAP[logoPreference]?.dark || LOGO_MAP.default.dark;
                 response = `# Design Context for: ${component_description}
 
 **Brand**: ${brand.toUpperCase()}
@@ -3525,30 +3838,26 @@ ${COMPONENT_CATEGORIES[detectedComponentType]?.slice(0, 8).join(", ") || "See co
 
 Now design the **${component_description}** following these tokens and principles.`;
             }
-            // Embed logos in response
+            // Embed logos in response with proper error feedback
             const contentBlocks = [];
+            // Validate logo map on first use
+            validateLogoMap();
             if (isDualThemeMode) {
                 // Dual-theme mode: embed both light and dark theme logos
-                const lightLogoPath = logoMap[logoPreference]?.light || logoMap.default.light;
-                const darkLogoPath = logoMap[logoPreference]?.dark || logoMap.default.dark;
-                const lightLogo = await embedLogo(lightLogoPath);
-                const darkLogo = await embedLogo(darkLogoPath);
-                if (lightLogo) {
-                    contentBlocks.push({ type: "text", text: `## Logo for Light Backgrounds\n**Path:** \`${lightLogoPath}\`` });
-                    contentBlocks.push(lightLogo);
-                }
-                if (darkLogo) {
-                    contentBlocks.push({ type: "text", text: `## Logo for Dark Backgrounds\n**Path:** \`${darkLogoPath}\`` });
-                    contentBlocks.push(darkLogo);
+                const lightLogoPath = LOGO_MAP[logoPreference]?.light || LOGO_MAP.default.light;
+                const darkLogoPath = LOGO_MAP[logoPreference]?.dark || LOGO_MAP.default.dark;
+                const lightEmbedded = await embedLogoWithFeedback(lightLogoPath, "Logo for Light Backgrounds", contentBlocks);
+                const darkEmbedded = await embedLogoWithFeedback(darkLogoPath, "Logo for Dark Backgrounds", contentBlocks);
+                if (!lightEmbedded && !darkEmbedded) {
+                    contentBlocks.push({
+                        type: "text",
+                        text: `> **Note:** No logos were embedded. The design guidance below remains valid, but you'll need to manually obtain the logo files. Use \`get_logo\` with paths from \`list_logos\`.`,
+                    });
                 }
             }
             else {
                 // Single-theme mode: embed the appropriate logo
-                const embeddedLogo = await embedLogo(logoPath);
-                if (embeddedLogo) {
-                    contentBlocks.push({ type: "text", text: `## Brand Logo (${theme === "light" ? "for Light Backgrounds" : "for Dark Backgrounds"})\n**Path:** \`${logoPath}\`` });
-                    contentBlocks.push(embeddedLogo);
-                }
+                await embedLogoWithFeedback(logoPath, `Brand Logo (${theme === "light" ? "for Light Backgrounds" : "for Dark Backgrounds"})`, contentBlocks);
             }
             // Add the main response text
             contentBlocks.push({ type: "text", text: response });
@@ -4303,8 +4612,8 @@ h2_style.paragraph_format.space_after = Pt(8)
 # ============================================
 # ADD CONTENT
 # ============================================
-# Add logo (if available)
-# doc.add_picture('path/to/logo.png', width=Inches(2))
+# Add logo - save the embedded logo above to 'logo.png', then:
+doc.add_picture('logo.png', width=Inches(2))
 
 # Title
 doc.add_paragraph('${templateType === 'proposal' ? 'Project Proposal' : templateType === 'spec-sheet' ? 'Product Specification' : templateType === 'invoice' ? 'Invoice' : 'Report Title'}', style='${palette.name}Title')
@@ -4422,8 +4731,8 @@ doc = SimpleDocTemplate(
 # Content
 content = []
 
-# Logo (optional)
-# content.append(Image('logo.png', width=2*inch, height=0.5*inch))
+# Logo - save the embedded logo above to 'logo.png', then:
+content.append(Image('logo.png', width=2*inch, height=0.5*inch))
 
 # Title
 content.append(Paragraph('${templateType === 'proposal' ? 'Project Proposal' : templateType === 'spec-sheet' ? 'Product Specification' : templateType === 'invoice' ? 'Invoice' : 'Report Title'}', title_style))
@@ -4565,8 +4874,32 @@ ${palette.preferredTheme === "dark" ? `- **Note**: ${palette.name} prefers dark
                     isError: true,
                 };
             }
+            // Embed logos for document templates using centralized LOGO_MAP
+            const templateContentBlocks = [];
+            // Validate logo map on first use
+            validateLogoMap();
+            // Map brand to centralized LOGO_MAP (sonance uses default lockup)
+            const brandLogoKey = targetBrand === "sonance" ? "default" : targetBrand;
+            const lightLogoPath = LOGO_MAP[brandLogoKey]?.light || LOGO_MAP.default.light;
+            const darkLogoPath = LOGO_MAP[brandLogoKey]?.dark || LOGO_MAP.default.dark;
+            // Embed logos with guidance
+            const lightEmbedded = await embedLogoWithFeedback(lightLogoPath, `## Logo for Light Backgrounds\n**Save this image as \`logo_dark.png\` for use in your document**\nUse on white/light backgrounds.`, templateContentBlocks);
+            const darkEmbedded = await embedLogoWithFeedback(darkLogoPath, `## Logo for Dark Backgrounds\n**Save this image as \`logo_light.png\` for use in your document**\nUse on dark/colored backgrounds.`, templateContentBlocks);
+            if (!lightEmbedded && !darkEmbedded) {
+                templateContentBlocks.push({
+                    type: "text",
+                    text: `> **Logo Note:** Brand logos could not be embedded automatically. Use the \`get_logo\` tool to retrieve ${palette.name} logos.`,
+                });
+            }
+            // Add logo usage instructions
+            templateContentBlocks.push({
+                type: "text",
+                text: `## Logo Usage\n\nTo add the logo to your document:\n1. Save the appropriate logo image above (right-click → Save as)\n2. Use the saved path in your code:\n\n**Word (python-docx):**\n\`\`\`python\ndoc.add_picture('logo_dark.png', width=Inches(2))  # For light backgrounds\n\`\`\`\n\n**PDF (ReportLab):**\n\`\`\`python\ncontent.append(Image('logo_dark.png', width=2*inch, height=0.5*inch))\n\`\`\`\n`,
+            });
+            // Add the main template text
+            templateContentBlocks.push({ type: "text", text: template });
             return {
-                content: [{ type: "text", text: template }],
+                content: templateContentBlocks,
             };
         }
         case "rebrand_document": {
@@ -4837,35 +5170,22 @@ ${rebrandArgs.source_format === "word" ? `
 
 **Remember:** The goal is to make the document unmistakably ${palette.name} while preserving its original structure and content.
 `;
-            // Embed logos for rebrand_document
-            // Use the Sonance+James+IPORT lockup as default for Sonance brand
-            const rebrandLogoMap = {
-                sonance: {
-                    light: "/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Dark.png",
-                    dark: "/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Light.png",
-                },
-                iport: {
-                    light: "/logos/iport/IPORT_Sonance_LockUp_2C_Dark_RGB.png",
-                    dark: "/logos/iport/IPORT_Sonance_LockUp_2C_Light_RGB.png",
-                },
-                blaze: {
-                    light: "/logos/blaze/BlazeBySonance_Logo_Lockup_3C_Dark_RGB_05162025.png",
-                    dark: "/logos/blaze/BlazeBySonance_Logo_Lockup_2C_Light_RGB_05162025.png",
-                },
-            };
+            // Embed logos for rebrand_document using centralized LOGO_MAP
             const rebrandContentBlocks = [];
+            // Validate logo map on first use
+            validateLogoMap();
             // For documents, provide both light and dark logo variants
-            const lightLogoPath = rebrandLogoMap[targetBrand].light;
-            const darkLogoPath = rebrandLogoMap[targetBrand].dark;
-            const lightLogo = await embedLogo(lightLogoPath);
-            const darkLogo = await embedLogo(darkLogoPath);
-            if (lightLogo) {
-                rebrandContentBlocks.push({ type: "text", text: `## Logo for Light Backgrounds\n**Use on white/light document backgrounds**\n**Path:** \`${lightLogoPath}\`` });
-                rebrandContentBlocks.push(lightLogo);
-            }
-            if (darkLogo) {
-                rebrandContentBlocks.push({ type: "text", text: `## Logo for Dark Backgrounds\n**Use on dark/colored document backgrounds**\n**Path:** \`${darkLogoPath}\`` });
-                rebrandContentBlocks.push(darkLogo);
+            // Map brand to centralized LOGO_MAP (sonance uses default lockup)
+            const brandLogoKey = targetBrand === "sonance" ? "default" : targetBrand;
+            const lightLogoPath = LOGO_MAP[brandLogoKey]?.light || LOGO_MAP.default.light;
+            const darkLogoPath = LOGO_MAP[brandLogoKey]?.dark || LOGO_MAP.default.dark;
+            const lightEmbedded = await embedLogoWithFeedback(lightLogoPath, "Logo for Light Backgrounds\n**Use on white/light document backgrounds**", rebrandContentBlocks);
+            const darkEmbedded = await embedLogoWithFeedback(darkLogoPath, "Logo for Dark Backgrounds\n**Use on dark/colored document backgrounds**", rebrandContentBlocks);
+            if (!lightEmbedded && !darkEmbedded) {
+                rebrandContentBlocks.push({
+                    type: "text",
+                    text: `> **Logo Note:** Brand logos could not be embedded automatically. Download the appropriate ${palette.name} logo from your brand assets folder or use the \`get_logo\` tool.`,
+                });
             }
             // Add the main guide text
             rebrandContentBlocks.push({ type: "text", text: rebrandGuide });
@@ -5060,31 +5380,21 @@ ${layout.template}
 
 Now you have everything needed to build a ${palette.name}-branded ${appType} application!
 `;
-            // Embed logos
-            const logoMap = {
-                sonance: {
-                    light: "/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Dark.png",
-                    dark: "/logos/sonance-james-iport/Sonance_James_IPORT_Lockup_Light.png",
-                },
-                iport: {
-                    light: "/logos/iport/IPORT_Sonance_LockUp_2C_Dark_RGB.png",
-                    dark: "/logos/iport/IPORT_Sonance_LockUp_2C_Light_RGB.png",
-                },
-                blaze: {
-                    light: "/logos/blaze/BlazeBySonance_Logo_Lockup_3C_Dark_RGB_05162025.png",
-                    dark: "/logos/blaze/BlazeBySonance_Logo_Lockup_2C_Light_RGB_05162025.png",
-                },
-            };
+            // Embed logos using centralized LOGO_MAP with proper error feedback
             const contentBlocks = [];
-            const lightLogo = await embedLogo(logoMap[brand].light);
-            const darkLogo = await embedLogo(logoMap[brand].dark);
-            if (lightLogo) {
-                contentBlocks.push({ type: "text", text: `## Logo for Light Backgrounds\n**Path:** \`${logoMap[brand].light}\`` });
-                contentBlocks.push(lightLogo);
-            }
-            if (darkLogo) {
-                contentBlocks.push({ type: "text", text: `## Logo for Dark Backgrounds\n**Path:** \`${logoMap[brand].dark}\`` });
-                contentBlocks.push(darkLogo);
+            // Validate logo map on first use
+            validateLogoMap();
+            // Map brand to centralized LOGO_MAP (sonance uses default lockup)
+            const brandLogoKey = brand === "sonance" ? "default" : brand;
+            const lightLogoPath = LOGO_MAP[brandLogoKey]?.light || LOGO_MAP.default.light;
+            const darkLogoPath = LOGO_MAP[brandLogoKey]?.dark || LOGO_MAP.default.dark;
+            const lightEmbedded = await embedLogoWithFeedback(lightLogoPath, "Logo for Light Backgrounds", contentBlocks);
+            const darkEmbedded = await embedLogoWithFeedback(darkLogoPath, "Logo for Dark Backgrounds", contentBlocks);
+            if (!lightEmbedded && !darkEmbedded) {
+                contentBlocks.push({
+                    type: "text",
+                    text: `> **Logo Note:** Brand logos could not be embedded. Use \`list_logos\` and \`get_logo\` to retrieve logo assets manually.`,
+                });
             }
             contentBlocks.push({ type: "text", text: response });
             return {
@@ -5359,9 +5669,54 @@ className="hover:bg-primary/90 focus-visible:ring-2 focus-visible:ring-primary"
 - [ ] Dark mode support added
 - [ ] Hover states on interactive elements
 - [ ] Focus states for accessibility
+
+---
+
+## Brand Logos
+
+Use these logos when adding branding to your application:
+
+### Logo Paths
+| Variant | Path | Use On |
+|---------|------|--------|
+| **Light Backgrounds** | \`${LOGO_MAP[brand === "sonance" ? "default" : brand]?.light || LOGO_MAP.default.light}\` | White, light gray |
+| **Dark Backgrounds** | \`${LOGO_MAP[brand === "sonance" ? "default" : brand]?.dark || LOGO_MAP.default.dark}\` | Charcoal, dark |
+
+### Quick Implementation
+\`\`\`tsx
+import Image from 'next/image';
+
+<Image
+  src="${LOGO_MAP[brand === "sonance" ? "default" : brand]?.light || LOGO_MAP.default.light}"
+  alt="${palette.name}"
+  width={150}
+  height={40}
+  className="h-10 w-auto dark:hidden"
+  priority
+/>
+<Image
+  src="${LOGO_MAP[brand === "sonance" ? "default" : brand]?.dark || LOGO_MAP.default.dark}"
+  alt="${palette.name}"
+  width={150}
+  height={40}
+  className="h-10 w-auto hidden dark:block"
+  priority
+/>
+\`\`\`
 `;
+            // Build content blocks with embedded logos
+            const contentBlocks = [];
+            // Validate and embed logos for the target brand
+            validateLogoMap();
+            const brandLogoKey = brand === "sonance" ? "default" : brand;
+            const lightLogoPath = LOGO_MAP[brandLogoKey]?.light || LOGO_MAP.default.light;
+            const darkLogoPath = LOGO_MAP[brandLogoKey]?.dark || LOGO_MAP.default.dark;
+            await embedLogoWithFeedback(lightLogoPath, `${palette.name} Logo (for Light Backgrounds)`, contentBlocks);
+            await embedLogoWithFeedback(darkLogoPath, `${palette.name} Logo (for Dark Backgrounds)`, contentBlocks);
+            // Add the analysis text
+            contentBlocks.push({ type: "text", text: response });
             return {
-                content: [{ type: "text", text: response }],
+                content: contentBlocks,
             };
         }
         case "analyze_for_redesign": {

package/package.json

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