Clean up old Redis configuration references and simplify config parsing

README.md

@@ -91,22 +91,24 @@ Configure with OPENAI_API_KEY environment variable.
 ### For Local Development (Windows/Ollama):
 
 1. Install Ollama: 
-bash
+```bash
 # Download from https://ollama.com/download/OllamaSetup.exe
-
 Pull and run models:
+
+
 ollama pull mistral
 ollama pull llama3
 ollama serve
-
 Start ngrok tunnel:
-ngrok http 11434
 
+
+ngrok http 11434
 Update environment variables:
+
+
 OLLAMA_HOST=https://your-ngrok-url.ngrok-free.app
 LOCAL_MODEL_NAME=mistral
 USE_FALLBACK=false
-
 For Production Deployment:
 
 The application automatically handles provider fallback:
@@ -117,44 +119,42 @@ Tertiary: OpenAI (if configured)
 Architecture
 
 This application consists of:
-- Streamlit frontend (app.py)
-- Core LLM abstraction (core/llm.py)
-- Memory management (core/memory.py)
-- Configuration management (utils/config.py)
-- API endpoints (in api/ directory for future expansion)
 
+Streamlit frontend (app.py)
+Core LLM abstraction (core/llm.py)
+Memory management (core/memory.py)
+Configuration management (utils/config.py)
+API endpoints (in api/ directory for future expansion)
 Built with Python, Streamlit, FastAPI, and Redis.
 
-## Troubleshooting Common Issues
-
+Troubleshooting Common Issues
 503 Errors with Hugging Face Fallback:
-- Wait 4 minutes for cold start initialization
-- Retry request after endpoint warms up
 
+Wait 4 minutes for cold start initialization
+Retry request after endpoint warms up
 Ollama Connection Issues:
-- Verify ollama serve is running locally
-- Check ngrok tunnel status
-- Confirm ngrok URL matches OLLAMA_HOST
-- Test with test_ollama_connection.py
 
+Verify ollama serve is running locally
+Check ngrok tunnel status
+Confirm ngrok URL matches OLLAMA_HOST
+Test with test_ollama_connection.py
 Redis Connection Problems:
-- The Redis configuration is now hardcoded for maximum reliability
-- If issues persist, check network connectivity to Redis Cloud
 
+The Redis configuration is now hardcoded for maximum reliability
+If issues persist, check network connectivity to Redis Cloud
 Model Not Found:
-- Pull required model: ollama pull <model-name>
-- Check available models: ollama list
 
+Pull required model: ollama pull <model-name>
+Check available models: ollama list
 Diagnostic Scripts:
-- Run python test_ollama_connection.py to verify Ollama connectivity.
-- Run python diagnose_ollama.py for detailed connection diagnostics.
-- Run python test_hardcoded_redis.py to verify Redis connectivity with hardcoded configuration.
-
-## New Redis Database Configuration
 
+Run python test_ollama_connection.py to verify Ollama connectivity.
+Run python diagnose_ollama.py for detailed connection diagnostics.
+Run python test_hardcoded_redis.py to verify Redis connectivity with hardcoded configuration.
+New Redis Database Configuration
 The Redis connection now uses the new database credentials:
 
-```python
+
 import redis
 r = redis.Redis(
   host='redis-16717.c85.us-east-1-2.ec2.redns.redis-cloud.com',

utils/config.py

@@ -20,33 +20,22 @@ class Config:
         # Fallback settings
         self.use_fallback = os.getenv("USE_FALLBACK", "true").lower() == "true"
         
-        # Redis configuration (optional for HF)
-        self.redis_host = self._sanitize_host(os.getenv("REDIS_HOST", ""))
-        self.redis_port = int(os.getenv("REDIS_PORT", "6379"))
-        self.redis_username = os.getenv("REDIS_USERNAME", "")
-        self.redis_password = os.getenv("REDIS_PASSWORD", "")
-        self.redis_retries = int(os.getenv("REDIS_RETRIES", "3"))
-        self.redis_retry_delay = int(os.getenv("REDIS_RETRY_DELAY", "1"))
-        self.redis_disable_ssl = os.getenv("REDIS_DISABLE_SSL", "false").lower() == "true"
-        
         # Local model configuration
         self.local_model_name = os.getenv("LOCAL_MODEL_NAME", "mistral:latest")
         self.ollama_host = self._sanitize_url(os.getenv("OLLAMA_HOST", ""))
         
         # OpenWeather API
         self.openweather_api_key = os.getenv("OPENWEATHER_API_KEY")
-    
+        
     def _sanitize_url(self, url: str) -> str:
         """Sanitize URL by removing whitespace and control characters"""
         if not url:
             return ""
-        
+            
         # Remove leading/trailing whitespace and control characters
         url = url.strip()
-        
         # Remove any newlines, carriage returns, tabs, and null bytes
         url = re.sub(r'[\r\n\t\0\x0b\x0c]+', '', url)
-        
         # Remove any trailing URL encoding artifacts
         url = url.rstrip('%0a').rstrip('%0d')
         
@@ -59,20 +48,18 @@ class Config:
             else:
                 # Otherwise default to http
                 url = 'http://' + url
-        
+                
         return url
-    
+        
     def _sanitize_host(self, host: str) -> str:
         """Sanitize host by removing whitespace and control characters"""
         if not host:
             return ""
-        
+            
         # Remove leading/trailing whitespace and control characters
         host = host.strip()
-        
         # Remove any newlines, carriage returns, tabs, and null bytes
         host = re.sub(r'[\r\n\t\0\x0b\x0c]+', '', host)
-        
         # Remove any trailing URL encoding artifacts
         host = host.rstrip('%0a').rstrip('%0d')
         

utils/config_validator.py

@@ -20,8 +20,6 @@ class ConfigValidator:
         # Log current configuration for debugging
         logger.info(f"Current configuration:")
         logger.info(f"  OLLAMA_HOST: '{config.ollama_host}'")
-        logger.info(f"  REDIS_HOST: '{config.redis_host}'")
-        logger.info(f"  REDIS_PORT: {config.redis_port}")
         
         # Validate Ollama host
         if config.ollama_host:
@@ -33,31 +31,23 @@ class ConfigValidator:
                 report['valid'] = False
         else:
             report['warnings'].append("OLLAMA_HOST not set, local Ollama won't be available")
-        
-        # Validate Redis configuration
-        if config.redis_host:
-            sanitized_host = config._sanitize_host(config.redis_host)
-            if sanitized_host != config.redis_host:
-                logger.info(f"Sanitized REDIS_HOST from '{config.redis_host}' to '{sanitized_host}'")
-        else:
-            report['warnings'].append("Redis not configured, using in-memory storage")
-        
+            
         # Validate Hugging Face token
         if config.hf_token:
             if len(config.hf_token) < 10:  # Basic sanity check
                 report['warnings'].append("HF_TOKEN seems too short to be valid")
         else:
             report['warnings'].append("HF_TOKEN not set, Hugging Face models won't be available")
-        
+            
         # Validate OpenAI API key
         if config.openai_api_key:
             if not re.match(r'^sk-[a-zA-Z0-9]{32,}', config.openai_api_key):
                 report['warnings'].append("OPENAI_API_KEY format looks invalid")
         else:
             report['warnings'].append("OPENAI_API_KEY not set, OpenAI models won't be available")
-        
+            
         return report
-    
+        
     @staticmethod
     def _is_valid_url(url: str) -> bool:
         """Check if URL is valid"""
@@ -66,13 +56,12 @@ class ConfigValidator:
             return all([result.scheme, result.netloc])
         except:
             return False
-    
+            
     @staticmethod
     def _is_valid_host(host: str) -> bool:
         """Check if host is valid"""
         if not host:
             return False
-        
         # Basic regex for hostname validation
         pattern = r'^[a-zA-Z0-9.-]+$'
         return bool(re.match(pattern, host))
@@ -85,13 +74,13 @@ def validate_configuration():
     if report['errors']:
         logger.error("Configuration validation errors:")
         for error in report['errors']:
-            logger.error(f"  - {error}")
-    
+            logger.error(f" - {error}")
+            
     if report['warnings']:
         logger.warning("Configuration warnings:")
         for warning in report['warnings']:
-            logger.warning(f"  - {warning}")
-    
+            logger.warning(f" - {warning}")
+            
     return report['valid']
 
 # Run validation on import