fix: CI failures — ruff format, docker ENCRYPTION_KEY

- Run ruff format on 4 backend files
- Add ENCRYPTION_KEY to docker-compose.yml (required since Phase 1)

Author: MoonMao42

apps/api/app/core/config.py

@@ -111,17 +111,17 @@ def validate_secrets(self) -> None:
         if len(self.ENCRYPTION_KEY) < 32:
             raise ValueError(
                 "ENCRYPTION_KEY must be at least 32 bytes long for Fernet encryption. "
-                "Generate one with: python -c \"from cryptography.fernet import Fernet; "
-                "print(Fernet.generate_key().decode())\" and set as export ENCRYPTION_KEY=<key>"
+                'Generate one with: python -c "from cryptography.fernet import Fernet; '
+                'print(Fernet.generate_key().decode())" and set as export ENCRYPTION_KEY=<key>'
             )
 
         # Production and staging environments must use explicit key (not default)
         if (self.is_production or self.is_staging) and self.is_using_default_secrets:
             raise ValueError(
                 f"Cannot use default encryption key in {self.ENVIRONMENT} environment. "
                 "Please set ENCRYPTION_KEY environment variable explicitly. "
-                "Generate with: python -c \"from cryptography.fernet import Fernet; "
-                "print(Fernet.generate_key().decode())\" and export ENCRYPTION_KEY=<generated_key>"
+                'Generate with: python -c "from cryptography.fernet import Fernet; '
+                'print(Fernet.generate_key().decode())" and export ENCRYPTION_KEY=<generated_key>'
             )
 
 

apps/api/app/main.py

@@ -250,7 +250,11 @@ async def global_exception_handler(request: Request, exc: Exception) -> JSONResp
     if settings.DEBUG:
         # In debug mode: include request path but NOT full stack trace
         response_data["debug_detail"] = str(exc)
-        logger.debug("Debug error details", request_path=str(request.url.path), full_exception=traceback.format_exc())
+        logger.debug(
+            "Debug error details",
+            request_path=str(request.url.path),
+            full_exception=traceback.format_exc(),
+        )
 
     return JSONResponse(
         status_code=status_code,

apps/api/app/services/result_processor.py

@@ -106,15 +106,15 @@ async def extract_results(
                 chart_config = extract_chart_config(ai_content)
                 if chart_config:
                     result["chart_config"] = chart_config
-                    logger.info("Extracted chart configuration", chart_type=chart_config.get("type"))
+                    logger.info(
+                        "Extracted chart configuration", chart_type=chart_config.get("type")
+                    )
             except Exception as exc:
                 logger.warning("Failed to extract chart config", error=str(exc))
                 result["errors"].append(f"Chart extraction: {exc}")
 
             artifact_count = sum(
-                1
-                for k, v in result.items()
-                if k != "errors" and k != "thinking" and v is not None
+                1 for k, v in result.items() if k != "errors" and k != "thinking" and v is not None
             )
             logger.info(
                 "Result extraction completed",

apps/api/tests/test_services.py

@@ -90,9 +90,7 @@ async def test_execute_sql_programming_error(self, executor):
         with patch("app.services.database.create_database_manager") as mock_create_db:
             mock_db = MagicMock()
             mock_db.execute_query = MagicMock(
-                side_effect=ProgrammingError(
-                    "syntax error", None, None
-                )
+                side_effect=ProgrammingError("syntax error", None, None)
             )
             mock_create_db.return_value = mock_db
 
@@ -109,9 +107,7 @@ async def test_execute_sql_value_error(self, executor):
         """Test SQL execution with ValueError (validation error)"""
         with patch("app.services.database.create_database_manager") as mock_create_db:
             mock_db = MagicMock()
-            mock_db.execute_query = MagicMock(
-                side_effect=ValueError("Read-only check failed")
-            )
+            mock_db.execute_query = MagicMock(side_effect=ValueError("Read-only check failed"))
             mock_create_db.return_value = mock_db
 
             result, count = await executor.execute_sql(
@@ -152,9 +148,7 @@ async def test_inject_sql_data_custom_variable_name(self, executor):
         sql = "SELECT * FROM users"
         data = [{"id": 1}]
 
-        code = await executor.inject_sql_data(
-            sql, data, variable_name="custom_var"
-        )
+        code = await executor.inject_sql_data(sql, data, variable_name="custom_var")
 
         assert "custom_var" in code
         assert isinstance(code, str)
@@ -194,10 +188,7 @@ async def test_execute_safe_code(self, sandbox):
                 mock_runtime.inject_sql_data = MagicMock()
                 mock_runtime_class.return_value = mock_runtime
 
-                with patch.object(
-                    sandbox, "_execute_with_timeout",
-                    return_value=("3", [])
-                ):
+                with patch.object(sandbox, "_execute_with_timeout", return_value=("3", [])):
                     output, images = await sandbox.execute(safe_code)
 
                     assert output == "3"
@@ -260,8 +251,7 @@ async def test_execute_with_sql_data(self, sandbox):
                 mock_runtime_class.return_value = mock_runtime
 
                 with patch.object(
-                    sandbox, "_execute_with_timeout",
-                    return_value=("[{'id': 1}]", [])
+                    sandbox, "_execute_with_timeout", return_value=("[{'id': 1}]", [])
                 ):
                     output, images = await sandbox.execute(code, sql_data=sql_data)
 
@@ -373,7 +363,9 @@ async def test_extract_results_malformed_response(self, processor):
         with patch("app.services.engine_content.extract_sql_block", return_value=None):
             with patch("app.services.engine_content.extract_python_block", return_value=None):
                 with patch("app.services.engine_content.extract_chart_config", return_value=None):
-                    with patch("app.services.engine_content.parse_thinking_markers", return_value=[]):
+                    with patch(
+                        "app.services.engine_content.parse_thinking_markers", return_value=[]
+                    ):
                         result = await processor.extract_results(ai_content)
 
                         assert result["sql_code"] is None
@@ -397,7 +389,6 @@ async def test_extract_chart_config_valid(self, processor):
                 "yKeys": ["revenue"],
             }
 
-
             await processor.extract_results(ai_content)
 
             # Since extract_chart_config is mocked, verify the mock is set up
@@ -423,7 +414,9 @@ def test_build_chart_payload(self, processor):
             "data": data,
         }
 
-        with patch("app.services.engine_visualization.build_chart_from_config", return_value=expected_chart):
+        with patch(
+            "app.services.engine_visualization.build_chart_from_config", return_value=expected_chart
+        ):
             result = processor.build_chart_payload(chart_config, data)
 
             assert result is not None

docker-compose.yml

@@ -28,6 +28,7 @@ services:
       PORT: "8000"
       ENVIRONMENT: development
       DEBUG: "true"
+      ENCRYPTION_KEY: "docker-dev-key-min-32-bytes-long!!"
       LOG_FORMAT: console
       CORS_ORIGINS_STR: http://localhost:3000,http://127.0.0.1:3000
       MPLBACKEND: Agg