@sage-rsc/talking-head-react 1.7.6 → 1.7.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sage-rsc/talking-head-react",
-  "version": "1.7.6",
+  "version": "1.7.7",
   "description": "A reusable React component for 3D talking avatars with lip-sync and text-to-speech",
   "main": "./dist/index.cjs",
   "module": "./dist/index.js",

package/src/components/SimpleTalkingAvatar.jsx

@@ -118,11 +118,12 @@ const SimpleTalkingAvatar = forwardRef(({
       
       // Option 1: Load from manifest file only
       if (animations.manifest && !animations.auto) {
-        try {
-          const manifestAnimations = await loadAnimationsFromManifest(animations.manifest);
+        const manifestAnimations = await loadAnimationsFromManifest(animations.manifest);
+        // loadAnimationsFromManifest returns {} on error, so check if we got anything
+        const hasAnimations = Object.keys(manifestAnimations).length > 0;
+        if (hasAnimations) {
           setLoadedAnimations(manifestAnimations);
-        } catch (error) {
-          console.error('Failed to load animation manifest:', error);
+        } else {
           setLoadedAnimations(animations);
         }
       }
@@ -155,7 +156,7 @@ const SimpleTalkingAvatar = forwardRef(({
             folderPaths[`shared_talking`] = `${basePath}/shared/talking`;
             folderPaths[`shared_idle`] = `${basePath}/shared/idle`;
             
-            console.log(`📁 Loading animations from folders with genderKey="${genderKey}" for avatarBody="${avatarBody}"`);
+            // Loading animations from folders
             const discoveredAnimations = await autoLoadAnimationsFromFolders(folderPaths, avatarBody);
             
             // Check if we found any animations
@@ -406,33 +407,30 @@ const SimpleTalkingAvatar = forwardRef(({
     if (loadedAnimations._genderSpecific) {
       const genderKey = getGenderKey(avatarBody);
       
-      // Debug: Log gender detection
-      console.log(`🔍 Gender detection: avatarBody="${avatarBody}" -> genderKey="${genderKey}"`);
-      
       const genderGroups = loadedAnimations._genderSpecific[genderKey];
       
       // Try gender-specific first
       if (genderGroups && genderGroups[groupName]) {
         group = genderGroups[groupName];
-        console.log(`✅ Found ${genderKey} animations for "${groupName}": ${Array.isArray(group) ? group.length : 1} animation(s)`);
       } 
       // Fallback to shared gender-specific animations
       else if (loadedAnimations._genderSpecific.shared && loadedAnimations._genderSpecific.shared[groupName]) {
         group = loadedAnimations._genderSpecific.shared[groupName];
-        console.log(`✅ Found shared animations for "${groupName}": ${Array.isArray(group) ? group.length : 1} animation(s)`);
-      } else {
-        console.log(`⚠️ No ${genderKey} or shared animations found for "${groupName}"`);
       }
     }
     
     // Fallback to root-level animations if gender-specific not found
     if (!group && loadedAnimations[groupName]) {
       group = loadedAnimations[groupName];
-      console.log(`✅ Found root-level animations for "${groupName}": ${Array.isArray(group) ? group.length : 1} animation(s)`);
     }
     
     if (!group) {
-      console.log(`❌ No animations found for "${groupName}"`);
+      // Only log warning if animations were actually configured (not just empty object)
+      const hasAnyAnimations = Object.keys(loadedAnimations).length > 0 || 
+                               (loadedAnimations._genderSpecific && Object.keys(loadedAnimations._genderSpecific).length > 0);
+      if (hasAnyAnimations) {
+        console.warn(`⚠️ No animations found for group "${groupName}". Make sure animations are configured correctly.`);
+      }
       return [];
     }
     

package/src/utils/animationLoader.js

@@ -13,13 +13,35 @@ export async function loadAnimationsFromManifest(manifestPath) {
   try {
     const response = await fetch(manifestPath);
     if (!response.ok) {
+      // Don't log error for 404 - manifest is optional
+      if (response.status === 404) {
+        return {};
+      }
       throw new Error(`Failed to fetch manifest: ${response.status} ${response.statusText}`);
     }
-    const manifest = await response.json();
+    
+    // Check if response is actually JSON (not HTML error page)
+    const contentType = response.headers.get('content-type');
+    if (contentType && !contentType.includes('application/json')) {
+      // Response is not JSON (probably HTML error page)
+      return {};
+    }
+    
+    const text = await response.text();
+    // Try to parse JSON, but check if it looks like HTML first
+    if (text.trim().startsWith('<!')) {
+      // This is HTML, not JSON (likely a 404 page)
+      return {};
+    }
+    
+    const manifest = JSON.parse(text);
     const animations = manifest.animations || {};
     return animations;
   } catch (error) {
-    console.error('Failed to load animation manifest:', error);
+    // Only log if it's not a JSON parse error (which we handle above)
+    if (!(error instanceof SyntaxError)) {
+      console.warn('⚠️ Could not load animation manifest (this is optional):', manifestPath);
+    }
     return {};
   }
 }