Merge pull request #35 from dacharyc/final-cleanup-pass

Final cleanup pass

Author: dacharyc

mflix/README-JAVASCRIPT-EXPRESS.md

@@ -18,7 +18,7 @@ This is a full-stack movie browsing application built with Express.js and Next.j
 
 - **Node.js 22** or higher
 - **MongoDB Atlas cluster or local deployment** with the `sample_mflix` dataset loaded
-  - [Load sample data](https://www.mongodb.com/docs/atlas/sample-data/) 
+  - [Load sample data](https://www.mongodb.com/docs/atlas/sample-data/)
 - **npm** (included with Node.js)
 - **Voyage AI API key** (For MongoDB Vector Search)
   - [Get a Voyage AI API key](https://www.voyageai.com/)
@@ -54,7 +54,6 @@ VOYAGE_API_KEY=your_voyage_api_key
 PORT=3001
 NODE_ENV=development
 
-
 # CORS Configuration
 # Allowed origin for cross-origin requests (frontend URL)
 # For multiple origins, separate with commas

mflix/README-PYTHON-FASTAPI.md

@@ -22,7 +22,7 @@ This is a full-stack movie browsing application built with Python FastAPI and Ne
 - **Python 3.10** to **Python 3.13**
 - **Node.js 20** or higher
 - **MongoDB Atlas cluster or local deployment** with the `sample_mflix` dataset loaded
-  - [Load sample data](https://www.mongodb.com/docs/atlas/sample-data/) 
+  - [Load sample data](https://www.mongodb.com/docs/atlas/sample-data/)
 - **pip** for Python package management
 - **Voyage AI API key** (For MongoDB Vector Search)
   - [Get a Voyage AI API key](https://www.voyageai.com/)
@@ -56,7 +56,7 @@ VOYAGE_API_KEY=your_voyage_api_key
 
 # CORS Configuration
 # Comma-separated list of allowed origins for CORS
-CORS_ORIGINS=http://localhost:3000,http://localhost:8000
+CORS_ORIGINS=http://localhost:3000,http://localhost:3001
 ```
 
 **Note:** Replace `username`, `password`, and `cluster` with your actual MongoDB Atlas
@@ -69,7 +69,7 @@ python -m venv .venv
 ```
 
 Activate the virtual environment:
-    
+
 ```bash
 source .venv/bin/activate
 ```
@@ -85,12 +85,13 @@ pip install -r requirements.txt
 From the `server/` directory, run:
 
 ```bash
-fastapi dev main.py --reload
+uvicorn main:app --reload --port 3001
 ```
 
-The server will start on `http://localhost:8000`. You can verify it's running by visiting:
-- API root: http://localhost:8000/api/movies
-- API documentation (Swagger UI): http://localhost:8000/docs
+The server will start on `http://localhost:3001`. You can verify it's running by visiting:
+- API root: http://localhost:3001/api/movies
+- API documentation (Swagger UI): http://localhost:3001/docs
+- Interactive API documentation (ReDoc): http://localhost:3001/redoc
 
 ### 3. Configure and Start the Frontend
 
@@ -118,8 +119,8 @@ The Next.js application will start on `http://localhost:3000`.
 
 Open your browser and navigate to:
 - **Frontend:** http://localhost:3000
-- **Backend API:** http://localhost:8000
-- **API Documentation:** http://localhost:8000/docs
+- **Backend API:** http://localhost:3001
+- **API Documentation:** http://localhost:3001/docs
 
 ## Features
 
@@ -204,4 +205,3 @@ If you have problems running the sample app, please check the following:
 If you have verified the above and still have issues, please
 [open an issue](https://github.com/mongodb/docs-sample-apps/issues/new/choose)
 on the source repository `mongodb/docs-sample-apps`.
-

mflix/client/.env.example

@@ -9,4 +9,4 @@ VOYAGE_API_KEY=your_voyage_api_key
 
 # CORS Configuration
 # Comma-separated list of allowed origins for CORS
-CORS_ORIGINS="http://localhost:3000,http://localhost:8000"
+CORS_ORIGINS="http://localhost:3000,http://localhost:3001"

mflix/client/README.md

@@ -17,6 +17,14 @@ async def lifespan(app: FastAPI):
     # Startup: Create search indexes
     await ensure_search_index()
     await vector_search_index()
+
+    # Print server information
+    print(f"\n{'='*60}")
+    print(f"  Server started at http://127.0.0.1:3001")
+    print(f"  Documentation at http://127.0.0.1:3001/docs")
+    print(f"  Interactive API docs at http://127.0.0.1:3001/redoc")
+    print(f"{'='*60}\n")
+
     yield
     # Shutdown: Clean up resources if needed
     # Add any cleanup code here

mflix/server/python-fastapi/.env.example

@@ -8,7 +8,6 @@
 client = AsyncMongoClient(os.getenv("MONGO_URI"))
 db = client[os.getenv("MONGO_DB")]
 
-# Set the API key but don't instantiate the client here
 voyage_api_key = os.getenv("VOYAGE_API_KEY")
 if voyage_api_key:
     voyageai.api_key = voyage_api_key

mflix/server/python-fastapi/main.py

@@ -38,17 +38,6 @@ class Movie(BaseModel):
         "populate_by_name" : True
     }
 
-
-'''
-So this an interesting conversion. Pydanic doesn't cleanly support constructing
-models that have MongoDB query operators as field names. This becomes an issue when
-we want to convert the validated model back to a dictionary to use as a MongoDB query filter.
-For example, if a user leaves the 'q' parameter blank, we don't want to include the '$text' operator
-in the filter at all but validation will send an empty value for it and that causes errors. So
-I am handling the validation in the query router itself, but leaving this here as an example of how
-it could be done, if I am wrong about Pydantic's capabilities.
-'''
-
 class TextFilter(BaseModel):
     search: str = Field(..., alias="$search")