fix: Regressions from PR #711

Author: devanshi14malhotra

backend/config.py

@@ -24,10 +24,10 @@ def load_environment():
     env_path = os.path.join(base_dir, 'config', f'.env.{env}')
 
     if os.path.exists(env_path):
-        load_dotenv(env_path, override=True)
+        load_dotenv(env_path)
     else:
         # Fallback to standard .env in root
-        load_dotenv(override=True)
+        load_dotenv()
 
 load_environment()
 

backend/price_tracker/price_tracker.py

@@ -13,7 +13,7 @@
 from core.validators.validators import validate_google_books_id
 
 # Load environment variables
-load_dotenv(override=True)
+load_dotenv()
 
 # Configure logging
 logger = logging.getLogger(__name__)

frontend/js/config.js

@@ -60,17 +60,40 @@ if (typeof window !== 'undefined') {
         async fetchVolumes(query, options = {}) {
             const maxResults = options.maxResults || 5;
             const extraParams = options.extraParams || '';
-            const apiKey = CONFIG.GOOGLE_BOOKS_API_KEY || CONFIG.GOOGLE_BOOKS_API_KEYS[0] || '';
-            if (!apiKey) {
-                throw new Error('Google Books API key is not configured');
+            let keys = this.getKeys();
+            if (keys.length === 0 && CONFIG.GOOGLE_BOOKS_API_KEY) {
+                keys = [CONFIG.GOOGLE_BOOKS_API_KEY];
             }
-            const url = `${CONFIG.API_BASE}?q=${encodeURIComponent(query)}&maxResults=${encodeURIComponent(maxResults)}&key=${encodeURIComponent(apiKey)}${extraParams}`;
+            const candidates = keys.length > 0 ? keys : [null];
+            let lastError = null;
 
-            const response = await fetch(url);
-            if (response.ok) {
-                return await response.json();
+            for (let index = 0; index < candidates.length; index += 1) {
+                const key = candidates[index];
+                const keyParam = key ? `&key=${encodeURIComponent(key)}` : '';
+                const url = `${CONFIG.API_BASE}?q=${encodeURIComponent(query)}&maxResults=${maxResults}${extraParams}${keyParam}`;
+
+                try {
+                    const response = await fetch(url);
+                    if (response.ok) {
+                        return await response.json();
+                    }
+
+                    const retryableStatuses = [429, 403, 503];
+                    if (retryableStatuses.includes(response.status) && index < candidates.length - 1) {
+                        lastError = new Error(`Google Books API returned ${response.status}`);
+                        continue;
+                    }
+
+                    throw new Error(`Google Books API returned ${response.status}`);
+                } catch (error) {
+                    lastError = error;
+                    if (index < candidates.length - 1) {
+                        continue;
+                    }
+                }
             }
-            throw new Error(`Google Books API returned ${response.status}`);
+
+            throw lastError || new Error('Google Books request failed');
         }
     };
 }

tests/test_llm.py

@@ -2,7 +2,7 @@
 import os
 
 # Load environment variables
-loaded = load_dotenv(override=True)
+loaded = load_dotenv()
 print(f".env loaded: {loaded}")
 
 if os.path.exists(".env"):