Upload 2 files

app.py

@@ -10,39 +10,68 @@ import urllib.parse
 
 
 # Configuration
-SPACE_NAME = "AI Assistant"
-SPACE_DESCRIPTION = "A customizable AI assistant"
+SPACE_NAME = 'AI Assistant'
+SPACE_DESCRIPTION = 'A customizable AI assistant'
 
-# Default configuration values
-DEFAULT_SYSTEM_PROMPT = """You are a research aid specializing in academic literature search and analysis. Your expertise spans discovering peer-reviewed sources, assessing research methodologies, synthesizing findings across studies, and delivering properly formatted citations. When responding, anchor claims in specific sources from provided URL contexts, differentiate between direct evidence and interpretive analysis, and note any limitations or contradictory results. Employ clear, accessible language that demystifies complex research, and propose connected research directions when appropriate. Your purpose is to serve as an informed research tool supporting users through initial concept development, exploratory investigation, information collection, and source compilation."""
-DEFAULT_TEMPERATURE = 0.7
-DEFAULT_MAX_TOKENS = 750
+# Default configuration values (used only if config.json is missing)
+DEFAULT_CONFIG = {{
+    'name': SPACE_NAME,
+    'description': SPACE_DESCRIPTION,
+    'system_prompt': "You are a pedagogically-minded academic assistant designed for introductory courses. Your approach follows constructivist learning principles: build on students' prior knowledge, scaffold complex concepts through graduated questioning, and use Socratic dialogue to guide discovery. Provide concise, evidence-based explanations that connect theory to lived experiences. Each response should model critical thinking by acknowledging multiple perspectives, identifying assumptions, and revealing conceptual relationships. Conclude with open-ended questions that promote higher-order thinking—analysis, synthesis, or evaluation—rather than recall.",
+    'temperature': 0.7,
+    'max_tokens': 750,
+    'model': 'google/gemini-2.0-flash-001',
+    'api_key_var': 'API_KEY',
+    'theme': 'Default',
+    'grounding_urls': '[]',
+    'enable_dynamic_urls': True,
+    'examples': ['Can you help me understand why the sky is blue?'],
+    'locked': False
+}}
 
-# Try to load configuration from file (if modified by faculty)
-try:
-    with open('config.json', 'r') as f:
-        saved_config = json.load(f)
-        SYSTEM_PROMPT = saved_config.get('system_prompt', DEFAULT_SYSTEM_PROMPT)
-        temperature = saved_config.get('temperature', DEFAULT_TEMPERATURE)
-        max_tokens = saved_config.get('max_tokens', DEFAULT_MAX_TOKENS)
-        print("✅ Loaded configuration from config.json")
-except:
-    # Use defaults if no config file or error
-    SYSTEM_PROMPT = DEFAULT_SYSTEM_PROMPT
-    temperature = DEFAULT_TEMPERATURE
-    max_tokens = DEFAULT_MAX_TOKENS
-    print("ℹ️ Using default configuration")
+# Load configuration from file - this is the single source of truth
+def load_config():
+    """Load configuration from config.json with fallback to defaults"""
+    try:
+        with open('config.json', 'r') as f:
+            config = json.load(f)
+            print("✅ Loaded configuration from config.json")
+            return config
+    except FileNotFoundError:
+        print("ℹ️ No config.json found, using default configuration")
+        # Save default config for future use
+        try:
+            with open('config.json', 'w') as f:
+                json.dump(DEFAULT_CONFIG, f, indent=2)
+            print("✅ Created config.json with default values")
+        except:
+            pass
+        return DEFAULT_CONFIG
+    except Exception as e:
+        print(f"⚠️ Error loading config.json: {{e}}, using defaults")
+        return DEFAULT_CONFIG
+
+# Load configuration
+config = load_config()
+
+# Initial load of configuration values
+SPACE_NAME = config.get('name', DEFAULT_CONFIG['name'])
+SPACE_DESCRIPTION = config.get('description', DEFAULT_CONFIG['description'])
+SYSTEM_PROMPT = config.get('system_prompt', DEFAULT_CONFIG['system_prompt'])
+temperature = config.get('temperature', DEFAULT_CONFIG['temperature'])
+max_tokens = config.get('max_tokens', DEFAULT_CONFIG['max_tokens'])
+MODEL = config.get('model', DEFAULT_CONFIG['model'])
+THEME = config.get('theme', DEFAULT_CONFIG['theme'])
+GROUNDING_URLS = config.get('grounding_urls', DEFAULT_CONFIG['grounding_urls'])
+ENABLE_DYNAMIC_URLS = config.get('enable_dynamic_urls', DEFAULT_CONFIG['enable_dynamic_urls'])
 
-MODEL = "google/gemini-2.0-flash-001"
-THEME = "Default"  # Gradio theme name
-GROUNDING_URLS = []
 # Get access code from environment variable for security
 # If ACCESS_CODE is not set, no access control is applied
 ACCESS_CODE = os.environ.get("ACCESS_CODE")
-ENABLE_DYNAMIC_URLS = True
 
 # Get API key from environment - customizable variable name with validation
-API_KEY = os.environ.get("API_KEY")
+API_KEY_VAR = config.get('api_key_var', DEFAULT_CONFIG['api_key_var'])
+API_KEY = os.environ.get(API_KEY_VAR)
 if API_KEY:
     API_KEY = API_KEY.strip()  # Remove any whitespace
     if not API_KEY:  # Check if empty after stripping
@@ -53,21 +82,21 @@ def validate_api_key():
     """Validate API key configuration with detailed logging"""
     if not API_KEY:
         print(f"⚠️  API KEY CONFIGURATION ERROR:")
-        print(f"   Variable name: API_KEY")
+        print(f"   Variable name: {API_KEY_VAR}")
         print(f"   Status: Not set or empty")
-        print(f"   Action needed: Set 'API_KEY' in HuggingFace Space secrets")
+        print(f"   Action needed: Set '{API_KEY_VAR}' in HuggingFace Space secrets")
         print(f"   Expected format: sk-or-xxxxxxxxxx")
         return False
     elif not API_KEY.startswith('sk-or-'):
         print(f"⚠️  API KEY FORMAT WARNING:")
-        print(f"   Variable name: API_KEY")
-        print(f"   Current value: {API_KEY[:10]}..." if len(API_KEY) > 10 else API_KEY)
+        print(f"   Variable name: {API_KEY_VAR}")
+        print(f"   Current value: {API_KEY[:10]}..." if len(API_KEY) > 10 else "{API_KEY}")
         print(f"   Expected format: sk-or-xxxxxxxxxx")
         print(f"   Note: OpenRouter keys should start with 'sk-or-'")
         return True  # Still try to use it
     else:
         print(f"✅ API Key configured successfully")
-        print(f"   Variable: API_KEY")
+        print(f"   Variable: {API_KEY_VAR}")
         print(f"   Format: Valid OpenRouter key")
         return True
 
@@ -181,7 +210,10 @@ def get_grounding_context():
             context_parts.append(f"[{priority_label}] Context from URL {i} ({url}):\n{content}")
     
     if context_parts:
-        result = "\n\n" + "\n\n".join(context_parts) + "\n\n"
+        result = "\n\
+" + "\n\
+".join(context_parts) + "\n\
+"
     else:
         result = ""
     
@@ -195,7 +227,7 @@ def export_conversation_to_markdown(conversation_history):
         return "No conversation to export."
     
     markdown_content = f"""# Conversation Export
-Generated on: {datetime.now().strftime('%Y-%m-%d %H:%M:%S')}
+Generated on: {datetime.now().strftime('%%Y-%%m-%%d %%H:%%M:%%S')}
 
 ---
 
@@ -209,17 +241,21 @@ Generated on: {datetime.now().strftime('%Y-%m-%d %H:%M:%S')}
             
             if role == 'user':
                 message_pair_count += 1
-                markdown_content += f"## User Message {message_pair_count}\n\n{content}\n\n"
+                markdown_content += f"## User Message {{message_pair_count}}\n\n{{content}}\n\
+"
             elif role == 'assistant':
-                markdown_content += f"## Assistant Response {message_pair_count}\n\n{content}\n\n---\n\n"
+                markdown_content += f"## Assistant Response {{message_pair_count}}\n\n{{content}}\n\n---\n\
+"
         elif isinstance(message, (list, tuple)) and len(message) >= 2:
             # Handle legacy tuple format: ["user msg", "assistant msg"]
             message_pair_count += 1
             user_msg, assistant_msg = message[0], message[1]
             if user_msg:
-                markdown_content += f"## User Message {message_pair_count}\n\n{user_msg}\n\n"
+                markdown_content += f"## User Message {{message_pair_count}}\n\n{{user_msg}}\n\
+"
             if assistant_msg:
-                markdown_content += f"## Assistant Response {message_pair_count}\n\n{assistant_msg}\n\n---\n\n"
+                markdown_content += f"## Assistant Response {{message_pair_count}}\n\n{{assistant_msg}}\n\n---\n\
+"
     
     return markdown_content
 
@@ -229,14 +265,20 @@ def generate_response(message, history):
     
     # Enhanced API key validation with helpful messages
     if not API_KEY:
-        error_msg = f"🔑 **API Key Required**\n\n"
-        error_msg += f"Please configure your OpenRouter API key:\n"
-        error_msg += f"1. Go to Settings (⚙️) in your HuggingFace Space\n"
-        error_msg += f"2. Click 'Variables and secrets'\n"
-        error_msg += f"3. Add secret: **API_KEY**\n"
-        error_msg += f"4. Value: Your OpenRouter API key (starts with `sk-or-`)\n\n"
+        error_msg = f"🔑 **API Key Required**\n\
+"
+        error_msg += f"Please configure your OpenRouter API key:\
+"
+        error_msg += f"1. Go to Settings (⚙️) in your HuggingFace Space\
+"
+        error_msg += f"2. Click 'Variables and secrets'\
+"
+        error_msg += f"3. Add secret: **{API_KEY_VAR}**\
+"
+        error_msg += f"4. Value: Your OpenRouter API key (starts with `sk-or-`)\n\
+"
         error_msg += f"Get your API key at: https://openrouter.ai/keys"
-        print(f"❌ API request failed: No API key configured for API_KEY")
+        print(f"❌ API request failed: No API key configured for {API_KEY_VAR}")
         return error_msg
     
     # Get grounding context
@@ -253,7 +295,8 @@ def generate_response(message, history):
                 content = fetch_url_content(url)
                 dynamic_context_parts.append(f"\n\nDynamic context from {url}:\n{content}")
             if dynamic_context_parts:
-                grounding_context += "\n".join(dynamic_context_parts)
+                grounding_context += "\
+".join(dynamic_context_parts)
     
     # Build enhanced system prompt with grounding context
     enhanced_system_prompt = SYSTEM_PROMPT + grounding_context
@@ -331,19 +374,28 @@ def generate_response(message, history):
                 print(f"❌ Failed to parse API response: {str(e)}")
                 return f"API Error: Failed to parse response - {str(e)}"
         elif response.status_code == 401:
-            error_msg = f"🔐 **Authentication Error**\n\n"
-            error_msg += f"Your API key appears to be invalid or expired.\n\n"
-            error_msg += f"**Troubleshooting:**\n"
-            error_msg += f"1. Check that your **API_KEY** secret is set correctly\n"
-            error_msg += f"2. Verify your API key at: https://openrouter.ai/keys\n"
-            error_msg += f"3. Ensure your key starts with `sk-or-`\n"
+            error_msg = f"🔐 **Authentication Error**\n\
+"
+            error_msg += f"Your API key appears to be invalid or expired.\n\
+"
+            error_msg += f"**Troubleshooting:**\
+"
+            error_msg += f"1. Check that your **{API_KEY_VAR}** secret is set correctly\
+"
+            error_msg += f"2. Verify your API key at: https://openrouter.ai/keys\
+"
+            error_msg += f"3. Ensure your key starts with `sk-or-`\
+"
             error_msg += f"4. Check that you have credits on your OpenRouter account"
             print(f"❌ API authentication failed: {response.status_code} - {response.text[:200]}")
             return error_msg
         elif response.status_code == 429:
-            error_msg = f"⏱️ **Rate Limit Exceeded**\n\n"
-            error_msg += f"Too many requests. Please wait a moment and try again.\n\n"
-            error_msg += f"**Troubleshooting:**\n"
+            error_msg = f"⏱️ **Rate Limit Exceeded**\n\
+"
+            error_msg += f"Too many requests. Please wait a moment and try again.\n\
+"
+            error_msg += f"**Troubleshooting:**\
+"
             error_msg += f"1. Wait 30-60 seconds before trying again\n"
             error_msg += f"2. Check your OpenRouter usage limits\n"
             error_msg += f"3. Consider upgrading your OpenRouter plan"
@@ -356,17 +408,21 @@ def generate_response(message, history):
             except:
                 error_message = response.text
             
-            error_msg = f"⚠️ **Request Error**\n\n"
+            error_msg = f"⚠️ **Request Error**\n\
+"
             error_msg += f"The API request was invalid:\n"
-            error_msg += f"`{error_message}`\n\n"
+            error_msg += f"`{error_message}`\n\
+"
             if "model" in error_message.lower():
                 error_msg += f"**Model Issue:** The model `{MODEL}` may not be available.\n"
                 error_msg += f"Try switching to a different model in your Space configuration."
             print(f"❌ Bad request: {response.status_code} - {error_message}")
             return error_msg
         else:
-            error_msg = f"🚫 **API Error {response.status_code}**\n\n"
-            error_msg += f"An unexpected error occurred. Please try again.\n\n"
+            error_msg = f"🚫 **API Error {response.status_code}**\n\
+"
+            error_msg += f"An unexpected error occurred. Please try again.\n\
+"
             error_msg += f"If this persists, check:\n"
             error_msg += f"1. OpenRouter service status\n"
             error_msg += f"2. Your API key and credits\n"
@@ -375,8 +431,10 @@ def generate_response(message, history):
             return error_msg
             
     except requests.exceptions.Timeout:
-        error_msg = f"⏰ **Request Timeout**\n\n"
-        error_msg += f"The API request took too long (30s limit).\n\n"
+        error_msg = f"⏰ **Request Timeout**\n\
+"
+        error_msg += f"The API request took too long (30s limit).\n\
+"
         error_msg += f"**Troubleshooting:**\n"
         error_msg += f"1. Try again with a shorter message\n"
         error_msg += f"2. Check your internet connection\n"
@@ -384,8 +442,10 @@ def generate_response(message, history):
         print(f"❌ Request timeout after 30 seconds")
         return error_msg
     except requests.exceptions.ConnectionError:
-        error_msg = f"🌐 **Connection Error**\n\n"
-        error_msg += f"Could not connect to OpenRouter API.\n\n"
+        error_msg = f"🌐 **Connection Error**\n\
+"
+        error_msg += f"Could not connect to OpenRouter API.\n\
+"
         error_msg += f"**Troubleshooting:**\n"
         error_msg += f"1. Check your internet connection\n"
         error_msg += f"2. Check OpenRouter service status\n"
@@ -393,10 +453,12 @@ def generate_response(message, history):
         print(f"❌ Connection error to OpenRouter API")
         return error_msg
     except Exception as e:
-        error_msg = f"❌ **Unexpected Error**\n\n"
-        error_msg += f"An unexpected error occurred:\n"
-        error_msg += f"`{str(e)}`\n\n"
-        error_msg += f"Please try again or contact support if this persists."
+        error_msg = "❌ **Unexpected Error**\n\
+"
+        error_msg += "An unexpected error occurred:\n"
+        error_msg += f"`{str(e)}`\n\
+"
+        error_msg += "Please try again or contact support if this persists."
         print(f"❌ Unexpected error: {str(e)}")
         return error_msg
 
@@ -480,46 +542,53 @@ def export_conversation(history):
 
 # Configuration status display
 def get_configuration_status():
-    """Generate a configuration status message for display"""
+    """Generate a clean configuration status message for display"""
     status_parts = []
     
-    # API Key status
-    status_parts.append("### 🔑 API Configuration")
-    if API_KEY_VALID:
-        status_parts.append("✅ **API Key:** Ready")
-    else:
-        status_parts.append("❌ **API Key:** Not configured")
-        status_parts.append("   Set `API_KEY` in Space secrets")
-    
-    # Model and parameters
-    status_parts.append("")  # Blank line
-    status_parts.append("### 🤖 Model Settings")
-    status_parts.append(f"**Model:** {MODEL.split('/')[-1]}")
+    # Basic configuration info (without redundant "Configuration:" header)
+    status_parts.append(f"**Name:** {SPACE_NAME}")
+    status_parts.append(f"**Model:** {MODEL}")
+    status_parts.append(f"**Theme:** {THEME}")
     status_parts.append(f"**Temperature:** 0.7")
-    status_parts.append(f"**Max Tokens:** 750")
+    status_parts.append(f"**Max Response Tokens:** 750")
+    status_parts.append("")
+    
+    # Example prompts
+    status_parts.append("")
+    examples_list = config.get('examples', [])
+    if isinstance(examples_list, str):
+        try:
+            import ast
+            examples_list = ast.literal_eval(examples_list)
+        except:
+            examples_list = []
+    
+    if examples_list and len(examples_list) > 0:
+        status_parts.append("**Example Prompts:**")
+        for example in examples_list[:5]:  # Show up to 5 examples
+            status_parts.append(f"• {example}")
+        if len(examples_list) > 5:
+            status_parts.append(f"• ... and {len(examples_list) - 5} more")
+    else:
+        status_parts.append("**Example Prompts:** No example prompts configured")
     
     # URL Context if configured
-    if GROUNDING_URLS:
-        status_parts.append("")  # Blank line
-        status_parts.append("### 🔗 Context Sources")
-        status_parts.append(f"**URLs Configured:** {len(GROUNDING_URLS)}")
-        for i, url in enumerate(GROUNDING_URLS[:2], 1):
-            status_parts.append(f"  {i}. {url[:50]}{'...' if len(url) > 50 else ''}")
-        if len(GROUNDING_URLS) > 2:
-            status_parts.append(f"  ... and {len(GROUNDING_URLS) - 2} more")
-    
-    # Access control
-    if ACCESS_CODE is not None:
-        status_parts.append("")  # Blank line
-        status_parts.append("### 🔐 Access Control")
-        status_parts.append("**Status:** Password protected")
-    
-    # System prompt
-    status_parts.append("")  # Blank line
-    status_parts.append("### 📝 System Prompt")
-    # Show first 200 chars of system prompt
-    prompt_preview = SYSTEM_PROMPT[:200] + "..." if len(SYSTEM_PROMPT) > 200 else SYSTEM_PROMPT
-    status_parts.append(f"```\n{prompt_preview}\n```")
+    if GROUNDING_URLS and len(GROUNDING_URLS) > 0:
+        status_parts.append("")
+        status_parts.append("**Grounding URLs:**")
+        for i, url in enumerate(GROUNDING_URLS[:5], 1):  # Show first 5 URLs
+            status_parts.append(f"{i}. {url}")
+        if len(GROUNDING_URLS) > 5:
+            status_parts.append(f"... and {len(GROUNDING_URLS) - 5} more URLs")
+    
+    # System prompt at the end
+    status_parts.append("")
+    status_parts.append(f"**System Prompt:** {SYSTEM_PROMPT}")
+    
+    # API Key status (minimal, at the end)
+    status_parts.append("")
+    if not API_KEY_VALID:
+        status_parts.append(f"**Note:** API key ({API_KEY_VAR}) not configured in Space secrets")
     
     return "\n".join(status_parts)
 
@@ -545,11 +614,20 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
     
     # Main chat interface (hidden until access granted)
     with gr.Column(visible=(ACCESS_CODE is None)) as chat_section:
+        # Get examples from config
+        examples = config.get('examples', [])
+        if isinstance(examples, str):
+            try:
+                import ast
+                examples = ast.literal_eval(examples)
+            except:
+                examples = []
+        
         chat_interface = gr.ChatInterface(
             fn=store_and_generate_response,  # Use wrapper function to store history
             title="",  # Title already shown above
             description="",  # Description already shown above
-            examples=['Hello! How can you help me?', 'Tell me something interesting', 'What can you do?'],
+            examples=examples if examples else None,
             type="messages"  # Use modern message format for better compatibility
         )
         
@@ -564,6 +642,9 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
             outputs=[export_file]
         )
         
+        # Configuration status
+        with gr.Accordion("Configuration", open=False):
+            gr.Markdown(get_configuration_status())
     
     # Connect access verification
     if ACCESS_CODE is not None:
@@ -610,29 +691,11 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
                     with open('config.json', 'r') as f:
                         current_config = json.load(f)
                 except:
-                    current_config = {
-                        'system_prompt': SYSTEM_PROMPT,
-                        'temperature': 0.7,
-                        'max_tokens': 750,
-                        'locked': False
-                    }
-                
-                # Editable fields - Order matches the Configuration tab
-                # 1. Assistant Identity
-                edit_name = gr.Textbox(
-                    label="Assistant Name",
-                    value=current_config.get('name', SPACE_NAME),
-                    placeholder="My AI Assistant"
-                )
-                
-                edit_description = gr.Textbox(
-                    label="Assistant Description",
-                    value=current_config.get('description', SPACE_DESCRIPTION),
-                    lines=2,
-                    placeholder="A helpful AI assistant for..."
-                )
+                    # Use DEFAULT_CONFIG as fallback
+                    current_config = DEFAULT_CONFIG.copy()
                 
-                # 2. System Prompt
+                # Editable fields
+                # System Prompt
                 edit_system_prompt = gr.Textbox(
                     label="System Prompt",
                     value=current_config.get('system_prompt', SYSTEM_PROMPT),
@@ -654,8 +717,8 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
                     ],
                     value=current_config.get('model', MODEL)
                 )
-                
-                # 4. Example Prompts
+            
+                # 4. Example prompts field
                 examples_value = current_config.get('examples', [])
                 if isinstance(examples_value, list):
                     examples_text_value = "\n".join(examples_value)
@@ -686,7 +749,7 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
                         step=50
                     )
                 
-                # 6. URL Grounding
+                # URL Grounding fields
                 gr.Markdown("### URL Grounding")
                 grounding_urls_value = current_config.get('grounding_urls', [])
                 if isinstance(grounding_urls_value, str):
@@ -713,8 +776,8 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
                 )
                 
                 with gr.Row():
-                    save_config_btn = gr.Button("💾 Save Configuration", variant="primary")
-                    reset_config_btn = gr.Button("↩️ Reset to Defaults", variant="secondary")
+                    save_config_btn = gr.Button("Save Configuration", variant="primary")
+                    reset_config_btn = gr.Button("Reset to Defaults", variant="secondary")
                 
                 config_status = gr.Markdown("")
             
@@ -722,39 +785,30 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
             def verify_faculty_password(password):
                 if password == FACULTY_PASSWORD:
                     return (
-                        gr.update(value="✅ Authentication successful!"),
+                        gr.update(value="Authentication successful!"),
                         gr.update(visible=False),  # Hide auth row
                         gr.update(visible=True),    # Show config section
                         True  # Update auth state
                     )
                 else:
                     return (
-                        gr.update(value="❌ Invalid password"),
+                        gr.update(value="Invalid password"),
                         gr.update(visible=True),   # Keep auth row visible
                         gr.update(visible=False),  # Keep config hidden
                         False  # Auth failed
                     )
             
             # Save configuration function
-            def save_configuration(new_name, new_description, new_prompt, new_model, new_examples, new_temp, new_tokens, *args):
-                # Extract URL values, lock_config, and is_authenticated from args
-                # args should contain: url1, url2, ..., url10, lock_config, is_authenticated
-                if len(args) < 12:  # Need at least 10 URLs + lock_config + is_authenticated
-                    return "❌ Invalid number of parameters"
-                
-                url_values = args[:10]  # First 10 are URLs
-                lock_config = args[10]  # 11th is lock_config
-                is_authenticated = args[11]  # 12th is is_authenticated
-                
+            def save_configuration(new_prompt, new_model, new_examples, new_temp, new_tokens, *url_values, lock_config, is_authenticated):
                 if not is_authenticated:
-                    return "❌ Not authenticated"
+                    return "Not authenticated"
                 
                 # Check if configuration is already locked
                 try:
                     with open('config.json', 'r') as f:
                         existing_config = json.load(f)
                         if existing_config.get('locked', False):
-                            return "🔒 Configuration is locked and cannot be modified"
+                            return "Configuration is locked and cannot be modified"
                 except:
                     pass
                 
@@ -763,26 +817,27 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
                     with open('config.json', 'r') as f:
                         current_full_config = json.load(f)
                 except:
-                    # If config.json doesn't exist, use global config
-                    current_full_config = config.copy()
+                    # If config.json doesn't exist, use default configuration
+                    current_full_config = DEFAULT_CONFIG.copy()
                 
                 # Process example prompts
                 examples_list = [ex.strip() for ex in new_examples.split('\n') if ex.strip()]
                 
-                # Process URL values
-                grounding_urls = [url.strip() for url in url_values if url and url.strip()]
+                # Process URL values - lock_config is the last parameter
+                urls = list(url_values[:-1])  # All but last are URLs
+                lock_config_from_args = url_values[-1]  # Last is lock_config
+                # Filter out empty URLs
+                grounding_urls = [url.strip() for url in urls if url.strip()]
                 
                 # Update all editable fields while preserving everything else
                 current_full_config.update({
-                    'name': new_name,
-                    'description': new_description,
                     'system_prompt': new_prompt,
                     'model': new_model,
                     'examples': examples_list,
                     'temperature': new_temp,
                     'max_tokens': int(new_tokens),
                     'grounding_urls': grounding_urls,
-                    'locked': lock_config,
+                    'locked': lock_config_from_args,
                     'last_modified': datetime.now().isoformat(),
                     'last_modified_by': 'faculty'
                 })
@@ -791,61 +846,34 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
                     with open('config.json', 'w') as f:
                         json.dump(current_full_config, f, indent=2)
                     
-                    # Reload all configuration values from the saved config
-                    global SPACE_NAME, SPACE_DESCRIPTION, SYSTEM_PROMPT, MODEL, temperature, max_tokens, GROUNDING_URLS
-                    SPACE_NAME = current_full_config.get('name', SPACE_NAME)
-                    SPACE_DESCRIPTION = current_full_config.get('description', SPACE_DESCRIPTION)
-                    SYSTEM_PROMPT = current_full_config.get('system_prompt', SYSTEM_PROMPT)
-                    MODEL = current_full_config.get('model', MODEL)
-                    temperature = current_full_config.get('temperature', temperature)
-                    max_tokens = current_full_config.get('max_tokens', max_tokens)
-                    GROUNDING_URLS = current_full_config.get('grounding_urls', GROUNDING_URLS)
+                    # Optional: Auto-commit to HuggingFace if token is available
+                    hf_token = os.environ.get("HF_TOKEN")
+                    space_id = os.environ.get("SPACE_ID") 
                     
-                    # Determine which changes require restart
-                    restart_required = False
-                    restart_reasons = []
-                    
-                    # Check if theme changed (requires restart)
-                    if 'theme' in current_full_config and current_full_config.get('theme') != THEME:
-                        restart_required = True
-                        restart_reasons.append("Theme change")
-                    
-                    # Check if name/description changed (requires restart for interface title)
-                    if new_name != SPACE_NAME or new_description != SPACE_DESCRIPTION:
-                        restart_required = True
-                        restart_reasons.append("Assistant name or description")
-                    
-                    # Check if examples changed (requires restart for ChatInterface)
-                    old_examples = config.get('examples', [])
-                    if examples_list != old_examples:
-                        restart_required = True
-                        restart_reasons.append("Example prompts")
-                    
-                    # Build response message
-                    response_msg = f"✅ Configuration saved successfully at {datetime.now().strftime('%Y-%m-%d %H:%M:%S')}"
-                    
-                    # Add restart instructions if needed
-                    if restart_required:
-                        response_msg += f"\n\n🔄 **Restart Required**\n"
-                        response_msg += f"The following changes require a Space restart to take effect:\n"
-                        for reason in restart_reasons:
-                            response_msg += f"• {reason}\n"
-                        response_msg += "\n**To restart your Space:**\n"
-                        response_msg += "1. Go to your Space settings (⚙️ icon)\n"
-                        response_msg += "2. Click 'Factory reboot' for a complete restart\n"
-                        response_msg += "3. Wait ~30 seconds for the Space to reload\n"
-                        response_msg += "\n*Note: System prompt, model, temperature, and URL changes take effect immediately for new conversations.*"
+                    if hf_token and space_id:
+                        try:
+                            from huggingface_hub import HfApi
+                            api = HfApi()
+                            api.upload_file(
+                                path_or_fileobj="config.json",
+                                path_in_repo="config.json", 
+                                repo_id=space_id,
+                                repo_type="space",
+                                commit_message=f"Update configuration by faculty at {datetime.now().strftime('%Y-%m-%d %H:%M:%S')}"
+                            )
+                            return f"✅ Configuration saved and committed to repository at {datetime.now().strftime('%Y-%m-%d %H:%M:%S')}\n\n🔄 **Space will restart automatically** to apply changes."
+                        except Exception as commit_error:
+                            print(f"Note: Could not auto-commit to repository: {commit_error}")
+                            return f"✅ Configuration saved locally at {datetime.now().strftime('%Y-%m-%d %H:%M:%S')}\n\n🔄 **Manual Restart Required**\nFor changes to take effect:\n1. Go to Settings (⚙️)\n2. Click 'Factory reboot'\n3. Wait ~30 seconds for restart"
                     else:
-                        response_msg += "\n\n✅ All changes will take effect immediately for new conversations."
-                    
-                    return response_msg
+                        return f"✅ Configuration saved at {datetime.now().strftime('%Y-%m-%d %H:%M:%S')}\n\n🔄 **Manual Restart Required**\nFor changes to take effect:\n1. Go to Settings (⚙️)\n2. Click 'Factory reboot'\n3. Wait ~30 seconds for restart"
                 except Exception as e:
                     return f"❌ Error saving configuration: {str(e)}"
             
             # Reset configuration function
             def reset_configuration(is_authenticated):
                 if not is_authenticated:
-                    updates = ["❌ Not authenticated"] + [gr.update() for _ in range(17)]  # 1 status + 7 main fields + 10 URLs
+                    updates = ["Not authenticated"] + [gr.update() for _ in range(14)]  # 1 status + 14 fields (prompt, model, examples, temp, tokens + 10 urls)
                     return tuple(updates)
                 
                 # Check if locked
@@ -853,7 +881,7 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
                     with open('config.json', 'r') as f:
                         existing_config = json.load(f)
                         if existing_config.get('locked', False):
-                            updates = ["🔒 Configuration is locked"] + [gr.update() for _ in range(17)]
+                            updates = ["Configuration is locked"] + [gr.update() for _ in range(14)]
                             return tuple(updates)
                 except:
                     pass
@@ -865,20 +893,20 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
                 else:
                     examples_text = ""
                 
-                # Get default URLs
+                # Get default URLs - parse from JSON string if needed
                 default_urls = DEFAULT_CONFIG.get('grounding_urls', [])
                 if isinstance(default_urls, str):
                     try:
-                        import ast
-                        default_urls = ast.literal_eval(default_urls)
+                        import json
+                        default_urls = json.loads(default_urls)
                     except:
                         default_urls = []
+                elif not isinstance(default_urls, list):
+                    default_urls = []
                 
                 # Reset to original default values
                 updates = [
-                    "✅ Reset to default values",
-                    gr.update(value=DEFAULT_CONFIG.get('name', SPACE_NAME)),
-                    gr.update(value=DEFAULT_CONFIG.get('description', SPACE_DESCRIPTION)),
+                    "Reset to default values",
                     gr.update(value=DEFAULT_CONFIG.get('system_prompt', SYSTEM_PROMPT)),
                     gr.update(value=DEFAULT_CONFIG.get('model', MODEL)),
                     gr.update(value=examples_text),
@@ -909,17 +937,17 @@ with gr.Blocks(title=SPACE_NAME, theme=theme_class()) as demo:
             # Connect configuration buttons
             save_config_btn.click(
                 save_configuration,
-                inputs=[edit_name, edit_description, edit_system_prompt, edit_model, edit_examples, edit_temperature, edit_max_tokens] + url_fields + [config_locked, faculty_auth_state],
+                inputs=[edit_system_prompt, edit_model, edit_examples, edit_temperature, edit_max_tokens] + url_fields + [config_locked, faculty_auth_state],
                 outputs=[config_status]
             )
             
             reset_config_btn.click(
                 reset_configuration,
                 inputs=[faculty_auth_state],
-                outputs=[config_status, edit_name, edit_description, edit_system_prompt, edit_model, edit_examples, edit_temperature, edit_max_tokens] + url_fields
+                outputs=[config_status, edit_system_prompt, edit_model, edit_examples, edit_temperature, edit_max_tokens] + url_fields
             )
         else:
-            gr.Markdown("ℹ️ Faculty configuration is not enabled. Set CONFIG_CODE in Space secrets to enable.")
+            gr.Markdown("Faculty configuration is not enabled. Set CONFIG_CODE in Space secrets to enable.")
 
 if __name__ == "__main__":
     demo.launch()

config.json

@@ -1,13 +1,15 @@
 {
   "name": "AI Assistant",
   "description": "A customizable AI assistant",
-  "system_prompt": "You are a research aid specializing in academic literature search and analysis. Your expertise spans discovering peer-reviewed sources, assessing research methodologies, synthesizing findings across studies, and delivering properly formatted citations. When responding, anchor claims in specific sources from provided URL contexts, differentiate between direct evidence and interpretive analysis, and note any limitations or contradictory results. Employ clear, accessible language that demystifies complex research, and propose connected research directions when appropriate. Your purpose is to serve as an informed research tool supporting users through initial concept development, exploratory investigation, information collection, and source compilation.",
+  "system_prompt": "You are a pedagogically-minded academic assistant designed for introductory courses. Your approach follows constructivist learning principles: build on students' prior knowledge, scaffold complex concepts through graduated questioning, and use Socratic dialogue to guide discovery. Provide concise, evidence-based explanations that connect theory to lived experiences. Each response should model critical thinking by acknowledging multiple perspectives, identifying assumptions, and revealing conceptual relationships. Conclude with open-ended questions that promote higher-order thinking\u2014analysis, synthesis, or evaluation\u2014rather than recall.",
   "model": "google/gemini-2.0-flash-001",
   "api_key_var": "API_KEY",
   "temperature": 0.7,
   "max_tokens": 750,
-  "examples": "['Hello! How can you help me?', 'Tell me something interesting', 'What can you do?']",
-  "grounding_urls": "[]",
+  "examples": [
+    "Can you help me understand why the sky is blue?"
+  ],
+  "grounding_urls": [],
   "enable_dynamic_urls": true,
   "theme": "Default"
 }