fix: address Copilot documentation and CI suggestions

- Add required env vars (OPENAI_API_KEY, DATABASE_URL) to GitHub Actions
- Clarify DATABASE_URL priority over individual DB variables in README
- Add comment explaining STORAGE_PATH usage
- Remove non-existent config/settings.py from project structure
- Fixes all remaining Copilot documentation suggestions

Author: SparkZou

.github/workflows/code-quality.yml

@@ -40,6 +40,9 @@ jobs:
         uv run ruff check .
 
     - name: Run tests
+      env:
+        OPENAI_API_KEY: "test-openai-key"
+        DATABASE_URL: "sqlite:///:memory:"
       run: |
         uv run pytest -v --cov=app --cov-report=xml --cov-report=term
 

README.md

@@ -101,7 +101,10 @@ The API will be available at:
 Key environment variables in \`.env\`:
 
 ```bash
-# Database
+# Database (DATABASE_URL takes priority if set)
+DATABASE_URL=postgresql+psycopg://memu_user:memu_pass@localhost:54320/memu_db
+
+# Alternative: Individual database variables (used by app/database.py)
 DATABASE_HOST=localhost
 DATABASE_PORT=54320
 DATABASE_USER=memu_user
@@ -276,8 +279,6 @@ memU-server/
 │   ├── utils/        # Utilities
 │   ├── database.py   # Database configuration
 │   └── main.py       # FastAPI application
-├── config/
-│   └── settings.py   # Configuration management
 ├── tests/            # Test suite
 ├── alembic/          # Database migrations
 ├── docker-compose.yml
@@ -349,7 +350,7 @@ See [LICENSE](LICENSE) file for details.
 
 ## 🙏 Acknowledgments
 
-- [memu-py](https://github.com/mem0ai/memu-py) - Core memory management engine
+- [memu-py](https://github.com/NevaMind-AI/memu-py) - Core memory management engine
 - [FastAPI](https://fastapi.tiangolo.com/) - Modern web framework
 - [Temporal](https://temporal.io/) - Workflow orchestration
 - [pgvector](https://github.com/pgvector/pgvector) - Vector similarity search

app/database.py

@@ -35,7 +35,7 @@
             f"Database configuration is incomplete. Missing environment variables: {', '.join(missing_vars)}"
         )
 
-    DATABASE_URL = f"postgresql+psycopg://{db_user}:{db_pass}@{db_host}:{db_port}/{db_name}"
+    DATABASE_URL = f"postgresql+asyncpg://{db_user}:{db_pass}@{db_host}:{db_port}/{db_name}"
 
 # Create SQLAlchemy async engine
 engine = create_async_engine(

app/main.py

@@ -40,6 +40,7 @@
     database_config={"url": database_url},
 )
 
+# Storage directory for conversation files
 storage_dir = Path(os.getenv("STORAGE_PATH", "./data"))
 storage_dir.mkdir(parents=True, exist_ok=True)