Add detailed logging and fix read-only filesystem issues

- Add verbose logging to agent creation process
- Fix VectorStoreClient to handle read-only filesystem
- Fix EmbeddingsClient cache creation with proper error handling
- Only create directories if they don't exist and we have write permissions
- Better traceback logging for database connection errors

agent/a11y_agent.py

@@ -175,22 +175,27 @@ def create_agent(language: Optional[str] = None) -> A11yExpertAgent:
     """Factory function to create and initialize agent."""
     language = language or "en"
     
+    logger.info(f"Creating agent with language: {language}")
     settings = get_settings()
     
-    # Create vector store with lazy connection
+    # Create vector store with lazy connection (no DB access yet)
+    logger.info("Initializing vector store client...")
     vector_store = VectorStoreClient(uri=settings.lancedb_uri)
     
     api_key = settings.openai_api_key
     
+    logger.info("Initializing OpenAI client...")
     client_args = {"api_key": api_key}
     if settings.llm_base_url:
         client_args["base_url"] = settings.llm_base_url
         
     llm_client = OpenAI(**client_args)
     
+    logger.info("Creating A11yExpertAgent instance...")
     agent = A11yExpertAgent(
         vector_store=vector_store,
         llm_client=llm_client,
         language=language
     )
+    logger.info("Agent creation complete")
     return agent

database/vector_store_client.py

@@ -60,16 +60,21 @@ class VectorStoreClient:
             import os
             logger.info(f"Connecting to LanceDB at: {self.uri}")
             
-            try:
-                os.makedirs(self.uri, exist_ok=True)
-            except Exception as e:
-                logger.warning(f"Could not create directory {self.uri}: {e}")
+            # Only try to create directory if it doesn't exist and we can write
+            if not os.path.exists(self.uri):
+                try:
+                    os.makedirs(self.uri, exist_ok=True)
+                    logger.debug(f"Created directory: {self.uri}")
+                except (OSError, PermissionError) as e:
+                    logger.warning(f"Could not create directory (read-only filesystem?): {e}")
             
             try:
                 self._db = lancedb.connect(self.uri)
-                logger.info("✅ Connected to LanceDB")
+                logger.info("✅ Connected to LanceDB (read-only mode)")
             except Exception as e:
                 logger.error(f"Failed to connect to LanceDB: {e}")
+                import traceback
+                logger.error(traceback.format_exc())
                 raise
         return self._db
     

models/embeddings.py

@@ -68,8 +68,15 @@ class EmbeddingsClient:
         
         # Initialize cache if available
         if CACHE_AVAILABLE:
-            self.cache = Cache(cache_dir)
-            logger.info(f"✅ EmbeddingsClient initialized (model: {self.model}, cache: enabled)")
+            try:
+                import os
+                os.makedirs(cache_dir, exist_ok=True)
+                self.cache = Cache(cache_dir)
+                logger.info(f"✅ EmbeddingsClient initialized (model: {self.model}, cache: enabled)")
+            except (OSError, PermissionError) as e:
+                logger.warning(f"Could not create cache directory (read-only?): {e}")
+                self.cache = None
+                logger.info(f"✅ EmbeddingsClient initialized (model: {self.model}, cache: disabled)")
         else:
             self.cache = None
             logger.info(f"✅ EmbeddingsClient initialized (model: {self.model}, cache: disabled)")