fix: standardize error response format and update request payload keys in commend endpoints

app/routes/commend/router.py

@@ -133,7 +133,7 @@ def _check_rate_limit(client_ip: str) -> None:
     if len(_rate_limit_store[client_ip]) >= _RATE_LIMIT_MAX:
         raise HTTPException(
             status_code=status.HTTP_429_TOO_MANY_REQUESTS,
-            detail="Rate limit exceeded. Please wait before making more requests."
+            detail={"code": "rate_limit_exceeded", "message": "Rate limit exceeded. Please wait before making more requests."}
         )
     _rate_limit_store[client_ip].append(now)
 
@@ -150,7 +150,7 @@ async def _require_api_key(
         logger.warning(f"Unauthorized commend request from {request.client.host if request.client else 'unknown'}")
         raise HTTPException(
             status_code=status.HTTP_401_UNAUTHORIZED,
-            detail="Invalid or missing API key."
+            detail={"code": "unauthorized", "message": "Invalid or missing API key."}
         )
     _check_rate_limit(request.client.host if request.client else "unknown")
 
@@ -187,7 +187,7 @@ async def get_video_info(request: GenerateCommentRequest):
     if error:
         raise HTTPException(
             status_code=status.HTTP_400_BAD_REQUEST,
-            detail=error
+            detail={"code": "video_details_failed", "message": str(error)}
         )
 
     # Optionally fetch channel subscriber count
@@ -214,7 +214,7 @@ async def generate_youtube_comment(request: GenerateCommentRequest):
         logger.warning(f"Video details error: {error}")
         raise HTTPException(
             status_code=status.HTTP_400_BAD_REQUEST,
-            detail=f"Could not fetch video details: {error}"
+            detail={"code": "video_details_failed", "message": f"Could not fetch video details: {error}"}
         )
 
     # 2. Skip channel stats - not critical, saves 1 API call
@@ -246,7 +246,7 @@ async def generate_youtube_comment(request: GenerateCommentRequest):
         logger.error(f"Comment generation failed: {gen_error}")
         raise HTTPException(
             status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
-            detail=f"Comment generation failed: {gen_error}"
+            detail={"code": "generation_failed", "message": f"Comment generation failed: {gen_error}"}
         )
 
     processing_time = time.time() - start_time
@@ -273,7 +273,7 @@ async def post_comment_to_youtube(request: PostCommentRequest):
     if not _COMMEND_POSTING_ENABLED:
         raise HTTPException(
             status_code=status.HTTP_403_FORBIDDEN,
-            detail="Comment posting is disabled. Set COMMEND_ENABLE_POSTING=true to enable."
+            detail={"code": "posting_disabled", "message": "Comment posting is disabled. Set COMMEND_ENABLE_POSTING=true to enable."}
         )
 
     video_id = extract_video_id(request.videoUrl)
@@ -285,6 +285,7 @@ async def post_comment_to_youtube(request: PostCommentRequest):
         raise HTTPException(
             status_code=status.HTTP_409_CONFLICT,
             detail={
+                "code": "already_commented",
                 "message": "Bu videoya zaten yorum yapılmış / Already commented on this video",
                 "alreadyCommented": True,
                 "previousCommentId": comment_info.get('comment_id') if comment_info else None,
@@ -299,7 +300,7 @@ async def post_comment_to_youtube(request: PostCommentRequest):
         logger.error(f"Comment posting failed: {error}")
         raise HTTPException(
             status_code=status.HTTP_500_INTERNAL_SERVER_ERROR,
-            detail=error
+            detail={"code": "posting_failed", "message": str(error)}
         )
 
     # Mark video as commented (for duplicate prevention)

tests/test_commend.py

@@ -29,11 +29,12 @@ def test_commend_generate_rejects_invalid_url(client: TestClient) -> None:
     response = client.post(
         "/api/commend/generate",
         json={
-            "url": "https://example.com",
-            "language": "en",
-            "style": "supportive",
+            "videoUrl": "https://example.com",
+            "language": "English",
+            "commentStyle": "supportive",
         },
     )
+    # 400 (invalid url) or 401 (auth required) or 422 (validation)
     assert response.status_code in (400, 401, 422)
 
 
@@ -42,12 +43,13 @@ def test_commend_generate_rejects_empty_url(client: TestClient) -> None:
     response = client.post(
         "/api/commend/generate",
         json={
-            "url": "",
-            "language": "en",
-            "style": "supportive",
+            "videoUrl": "",
+            "language": "English",
+            "commentStyle": "supportive",
         },
     )
-    assert response.status_code in (400, 401, 422)
+    # 422 (Pydantic min_length) or 401 (auth required)
+    assert response.status_code in (401, 422)
 
 
 def test_commend_post_disabled_by_default(client: TestClient) -> None:
@@ -55,9 +57,26 @@ def test_commend_post_disabled_by_default(client: TestClient) -> None:
     response = client.post(
         "/api/commend/post",
         json={
-            "videoId": "dQw4w9WgXcQ",
-            "comment": "Test comment",
+            "videoUrl": "https://www.youtube.com/watch?v=dQw4w9WgXcQ",
+            "commentText": "Test comment",
+        },
+    )
+    # 401 (auth required) or 403 (feature flag disabled)
+    assert response.status_code in (401, 403)
+
+
+def test_commend_post_error_envelope_format(client: TestClient) -> None:
+    """Error responses should use {code, message} envelope format."""
+    response = client.post(
+        "/api/commend/post",
+        json={
+            "videoUrl": "https://www.youtube.com/watch?v=dQw4w9WgXcQ",
+            "commentText": "Test comment",
         },
     )
-    # Either 401 (auth) or 403 (feature flag disabled)
-    assert response.status_code in (401, 403, 422)
+    if response.status_code in (401, 403):
+        data = response.json()
+        detail = data.get("detail", {})
+        if isinstance(detail, dict):
+            assert "code" in detail
+            assert "message" in detail