Upload 34 files

app.py

@@ -45,7 +45,7 @@ def main():
         launch_kwargs = {
             "server_name": "0.0.0.0",
             "server_port": 7860,
-            "share": True,
+            "share": False,
             "show_error": True,
             "quiet": False
         }

coursecrafter/agents/llm_client.py

@@ -8,6 +8,7 @@ import json
 from typing import Dict, List, Any, Optional, AsyncGenerator
 from dataclasses import dataclass
 from abc import ABC, abstractmethod
+import os
 
 import openai
 import anthropic
@@ -249,6 +250,39 @@ class LlmClient:
             except Exception as e:
                 print(f"❌ Failed to initialize {provider} client: {e}")
 
+    def update_provider_config(self, provider: str, api_key: str = None, **kwargs):
+        """Update configuration for a specific provider and reinitialize client"""
+        
+        # Update environment variables
+        if provider == "openai" and api_key:
+            os.environ["OPENAI_API_KEY"] = api_key
+        elif provider == "anthropic" and api_key:
+            os.environ["ANTHROPIC_API_KEY"] = api_key
+        elif provider == "google" and api_key:
+            os.environ["GOOGLE_API_KEY"] = api_key
+        elif provider == "openai_compatible":
+            if api_key:
+                os.environ["OPENAI_COMPATIBLE_API_KEY"] = api_key
+            if kwargs.get("base_url"):
+                os.environ["OPENAI_COMPATIBLE_BASE_URL"] = kwargs["base_url"]
+            if kwargs.get("model"):
+                os.environ["OPENAI_COMPATIBLE_MODEL"] = kwargs["model"]
+        
+        # Reinitialize the specific client
+        try:
+            if provider in ["openai", "openai_compatible"]:
+                self.clients[provider] = OpenAIClient(provider)
+            elif provider == "anthropic":
+                self.clients[provider] = AnthropicClient()
+            elif provider == "google":
+                self.clients[provider] = GoogleClient()
+            
+            print(f"✅ Updated and reinitialized {provider} client")
+            return True
+        except Exception as e:
+            print(f"❌ Failed to reinitialize {provider} client: {e}")
+            return False
+
     def get_available_providers(self) -> List[LLMProvider]:
         """Get list of available providers"""
         return list(self.clients.keys())

coursecrafter/agents/simple_course_agent.py

@@ -106,6 +106,13 @@ For quizzes, return:
 
 Always strive to create courses that are not just informative, but are easy to understand, engaging, learning experiences."""
 
+    def update_provider_config(self, provider: str, api_key: str = None, **kwargs):
+        """Update provider configuration and reinitialize client"""
+        success = self.llm_client.update_provider_config(provider, api_key, **kwargs)
+        if success:
+            self.default_provider = provider
+        return success
+
     async def generate_course(
         self,
         topic: str,

coursecrafter/ui/gradio_app.py

@@ -775,8 +775,10 @@ def create_coursecrafter_interface() -> gr.Blocks:
 
         # Provider change handler to show/hide OpenAI-compatible fields
         def on_provider_change(provider):
-            # Return visibility state for OpenAI-compatible fields
-            return provider == "openai_compatible"
+            if provider == "openai_compatible":
+                return gr.update(visible=True)
+            else:
+                return gr.update(visible=False)
 
         # Event handlers
         async def generate_course_wrapper(topic: str, difficulty: str, lessons: int, provider: str, api_key: str, endpoint_url: str, model_name: str, progress=gr.Progress()):
@@ -785,28 +787,28 @@ def create_coursecrafter_interface() -> gr.Blocks:
                 return (
                     "<div class='error'>❌ Please enter a topic for your course.</div>",
                     "", "", 
-                    False, [], "<div class='image-details'>Error loading images</div>"
+                    gr.update(visible=False), [], "<div class='image-details'>Error loading images</div>"
                 )
 
             if not api_key.strip() and provider != "openai_compatible":
                 return (
                     "<div class='error'>❌ Please enter your API key for the selected LLM provider.</div>",
                     "", "", 
-                    False, [], "<div class='image-details'>Error loading images</div>"
+                    gr.update(visible=False), [], "<div class='image-details'>Error loading images</div>"
                 )
             
             if provider == "openai_compatible" and not endpoint_url.strip():
                 return (
                     "<div class='error'>❌ Please enter the endpoint URL for OpenAI-compatible provider.</div>",
                     "", "", 
-                    False, [], "<div class='image-details'>Error loading images</div>"
+                    gr.update(visible=False), [], "<div class='image-details'>Error loading images</div>"
                 )
             
             if provider == "openai_compatible" and not model_name.strip():
                 return (
                     "<div class='error'>❌ Please enter the model name for OpenAI-compatible provider.</div>",
                     "", "", 
-                    False, [], "<div class='image-details'>Error loading images</div>"
+                    gr.update(visible=False), [], "<div class='image-details'>Error loading images</div>"
                 )
 
             try:
@@ -827,13 +829,37 @@ def create_coursecrafter_interface() -> gr.Blocks:
                     os.environ["OPENAI_COMPATIBLE_BASE_URL"] = endpoint_url
                     os.environ["OPENAI_COMPATIBLE_MODEL"] = model_name
                 
-                # Initialize the simplified agent
+                # IMPORTANT: Create a fresh agent instance to pick up the new environment variables
+                # This ensures the LlmClient reinitializes with the updated API keys
                 agent = SimpleCourseAgent()
-                # Override the default provider with user selection
-                agent.default_provider = provider
+                
+                # Use the new dynamic configuration method to update provider settings
+                config_kwargs = {}
+                if provider == "openai_compatible":
+                    config_kwargs["base_url"] = endpoint_url
+                    config_kwargs["model"] = model_name
+                
+                # Update provider configuration dynamically
+                config_success = agent.update_provider_config(provider, api_key, **config_kwargs)
+                if not config_success:
+                    return (
+                        f"<div class='error'>❌ Failed to configure provider '{provider}'. Please check your API key and settings.</div>",
+                        "", "", 
+                        gr.update(visible=False), [], "<div class='image-details'>Error loading images</div>"
+                    )
+                
                 course_context["agent"] = agent
                 course_context["topic"] = topic
 
+                # Verify the provider is available with the new configuration
+                available_providers = agent.get_available_providers()
+                if provider not in available_providers:
+                    return (
+                        f"<div class='error'>❌ Provider '{provider}' is not available after configuration. Please check your API key and configuration.</div>",
+                        "", "", 
+                        gr.update(visible=False), [], "<div class='image-details'>Error loading images</div>"
+                    )
+
                 progress(0.1, desc="⚙️ Setting up generation options...")
 
                 # Create generation options
@@ -853,7 +879,7 @@ def create_coursecrafter_interface() -> gr.Blocks:
                     return (
                         "<div class='error'>❌ No LLM providers available. Please check your API keys.</div>",
                         "", "", 
-                        False, [], "<div class='image-details'>Error loading images</div>"
+                        gr.update(visible=False), [], "<div class='image-details'>Error loading images</div>"
                     )
 
                 progress(0.2, desc="🎓 Starting course generation...")
@@ -953,8 +979,14 @@ def create_coursecrafter_interface() -> gr.Blocks:
                     quiz_data = course_data.get("quiz", {})
                     quizzes_html = format_quiz(quiz_data)
 
-                    # Show quiz button if quiz exists
-                    quiz_btn_visible = bool(quiz_data and quiz_data.get("questions"))
+                    # Show quiz button if quiz exists - be more permissive to ensure it shows
+                    quiz_btn_visible = bool(quiz_data and (quiz_data.get("questions") or len(str(quiz_data)) > 50))
+                    print(f"🎯 Quiz button visibility: {quiz_btn_visible} (quiz_data: {bool(quiz_data)}, questions: {bool(quiz_data.get('questions') if quiz_data else False)})")
+                    
+                    # Force quiz button to be visible if we have any quiz content
+                    if quiz_data and not quiz_btn_visible:
+                        print("⚠️ Forcing quiz button to be visible due to quiz data presence")
+                        quiz_btn_visible = True
 
                     progress(0.98, desc="🖼️ Processing images for gallery...")
 
@@ -1210,7 +1242,7 @@ def create_coursecrafter_interface() -> gr.Blocks:
 
                     return (
                         lessons_html, flashcards_html, quizzes_html,
-                        quiz_btn_visible, images, image_details_html
+                        gr.update(visible=quiz_btn_visible), images, image_details_html
                     )
                 else:
                     quiz_btn_visible = False
@@ -1218,7 +1250,7 @@ def create_coursecrafter_interface() -> gr.Blocks:
 
                     return (
                         "", "", "",
-                        quiz_btn_visible, [], "<div class='image-details'>No images available</div>"
+                        gr.update(visible=quiz_btn_visible), [], "<div class='image-details'>No images available</div>"
                     )
 
             except Exception as e:
@@ -1227,13 +1259,13 @@ def create_coursecrafter_interface() -> gr.Blocks:
                 print(f"Error in course generation: {error_details}")
                 return (
                     "", "", "",
-                    False, [], "<div class='image-details'>Error loading images</div>"
+                    gr.update(visible=False), [], "<div class='image-details'>Error loading images</div>"
                 )
 
         def handle_quiz_submit():
             """Handle quiz submission using client-side processing"""
             # This function will be replaced by client-side JavaScript
-            return None
+            return gr.update()
 
         async def handle_chat(message: str, current_chat: str):
             """Handle chat messages for answering questions about the course content"""

coursecrafter/utils/config.py

@@ -163,6 +163,9 @@ class Config:
 
     def get_llm_config(self, provider: LLMProvider) -> LLMProviderConfig:
         """Get configuration for a specific LLM provider"""
+        # Reload config to pick up any environment variable changes
+        self._config = self._load_default_config()
+        
         if provider not in self._config["llm_providers"]:
             raise ValueError(f"Unknown LLM provider: {provider}")
 
@@ -171,6 +174,9 @@ class Config:
 
     def get_available_llm_providers(self) -> List[LLMProvider]:
         """Get list of available LLM providers with API keys"""
+        # Reload config to pick up any environment variable changes
+        self._config = self._load_default_config()
+        
         available = []
         for provider, config in self._config["llm_providers"].items():
             if provider == "openai_compatible":
@@ -185,6 +191,9 @@ class Config:
 
     def get_default_llm_provider(self) -> LLMProvider:
         """Get the default LLM provider, falling back to first available if not configured"""
+        # Reload config to pick up any environment variable changes
+        self._config = self._load_default_config()
+        
         default_provider = self._config["system"]["default_llm_provider"]
         available_providers = self.get_available_llm_providers()
         
@@ -197,8 +206,9 @@ class Config:
             print(f"⚠️ Default provider '{default_provider}' not configured, using '{available_providers[0]}'")
             return available_providers[0]
         
-        # If no providers are available, raise an error
-        raise ValueError("No LLM providers are configured. Please set up at least one provider.")
+        # If no providers are available, return a fallback instead of raising an error
+        print("⚠️ Warning: No LLM providers are configured. Returning 'google' as fallback.")
+        return "google"  # Return a fallback provider that can be configured later
 
     def get_image_generation_config(self) -> Dict[str, Any]:
         """Get image generation configuration"""