refactor: Add explicit HTTP status codes for API clarity

Implements the optional improvement suggested in PR review to make
HTTP status codes explicit rather than implicit.

Changes:
- Added explicit status_code=201 to POST /projects/projects decorator
- Added Response parameter to override status code for idempotent requests
- Returns 201 Created for new projects, 200 OK for existing (idempotent)
- Updated tests to verify correct status codes (201 vs 200)

This improves API documentation clarity and follows REST best practices
for idempotent operations.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: phernandez <paul@basicmachines.co>

Author: phernandez

src/basic_memory/api/routers/project_router.py

@@ -1,7 +1,7 @@
 """Router for project management."""
 
 import os
-from fastapi import APIRouter, HTTPException, Path, Body, BackgroundTasks
+from fastapi import APIRouter, HTTPException, Path, Body, BackgroundTasks, Response
 from typing import Optional
 from loguru import logger
 
@@ -180,8 +180,9 @@ async def list_projects(
 
 
 # Add a new project
-@project_resource_router.post("/projects", response_model=ProjectStatusResponse)
+@project_resource_router.post("/projects", response_model=ProjectStatusResponse, status_code=201)
 async def add_project(
+    response: Response,
     project_data: ProjectInfoRequest,
     project_service: ProjectServiceDep,
 ) -> ProjectStatusResponse:
@@ -205,6 +206,7 @@ async def add_project(
 
         if requested_path == existing_path:
             # Same name, same path - return 200 OK (idempotent)
+            response.status_code = 200
             return ProjectStatusResponse(  # pyright: ignore [reportCallIssue]
                 message=f"Project '{project_data.name}' already exists",
                 status="success",

tests/api/test_project_router.py

@@ -571,8 +571,8 @@ async def test_create_project_idempotent_same_path(test_config, client, project_
         json={"name": test_project_name, "path": test_project_path, "set_default": False},
     )
 
-    # Should succeed
-    assert response1.status_code == 200
+    # Should succeed with 201 Created
+    assert response1.status_code == 201
     data1 = response1.json()
     assert data1["status"] == "success"
     assert data1["new_project"]["name"] == test_project_name
@@ -610,8 +610,8 @@ async def test_create_project_fails_different_path(test_config, client, project_
         json={"name": test_project_name, "path": test_project_path1, "set_default": False},
     )
 
-    # Should succeed
-    assert response1.status_code == 200
+    # Should succeed with 201 Created
+    assert response1.status_code == 201
 
     # Try to create the same project with different path
     test_project_path2 = "/tmp/test-path-conflict-2"