fix: address review comments — deduplicate URL prefix, preserve response status codes

- Move transformation router to dedicated prefix
  project/<project_id>/no_code_model/ instead of duplicate "project" prefix
- Remove status code override in handle_http_request — let views return
  their intended status codes (201, 204, etc.) instead of forcing 200
- Normalize generate_formula param from model_name to file_name for
  consistency with URL pattern

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: abhizipstack

backend/backend/core/routers/transformation/urls.py

@@ -12,50 +12,15 @@
     validate_model_file,
 )
 
+# Mounted at: project/<project_id>/no_code_model/
 urlpatterns = [
-    path(
-        "/<str:project_id>/no_code_model/<str:file_name>/validate",
-        validate_model_file,
-        name="validate-no-code-model-file",
-    ),
-    path(
-        "/<str:project_id>/no_code_model/<str:file_name>/set-model",
-        set_model_config_and_reference,
-        name="set-no-code-model-config",
-    ),
-    path(
-        "/<str:project_id>/no_code_model/<str:file_name>/set-transform",
-        set_model_transformation,
-        name="set-no-code-model-transformation",
-    ),
-    path(
-        "/<str:project_id>/no_code_model/<str:file_name>/delete-transform",
-        delete_model_transformation,
-        name="delete-no-code-model-transformation",
-    ),
-    path(
-        "/<str:project_id>/no_code_model/<str:file_name>/set-presentation",
-        set_model_presentation,
-        name="set-no-code-model-presentation",
-    ),
-    path(
-        "/<str:project_id>/no_code_model/<str:file_name>/columns",
-        get_transformation_columns,
-        name="get-transformation-columns",
-    ),
-    path(
-        "/<str:project_id>/no_code_model/<str:file_name>/supported_references",
-        get_supported_models,
-        name="get-supported-reference-models",
-    ),
-    path(
-        "/<str:project_id>/no_code_model/<str:file_name>",
-        save_model_file,
-        name="save-no-code-model-file",
-    ),
-    path(
-        "/<str:project_id>/no_code_model/<str:model_name>/generate_formula",
-        generate_formula,
-        name="generate-formula",
-    ),
+    path("<str:file_name>/validate", validate_model_file, name="validate-no-code-model-file"),
+    path("<str:file_name>/set-model", set_model_config_and_reference, name="set-no-code-model-config"),
+    path("<str:file_name>/set-transform", set_model_transformation, name="set-no-code-model-transformation"),
+    path("<str:file_name>/delete-transform", delete_model_transformation, name="delete-no-code-model-transformation"),
+    path("<str:file_name>/set-presentation", set_model_presentation, name="set-no-code-model-presentation"),
+    path("<str:file_name>/columns", get_transformation_columns, name="get-transformation-columns"),
+    path("<str:file_name>/supported_references", get_supported_models, name="get-supported-reference-models"),
+    path("<str:file_name>/generate_formula", generate_formula, name="generate-formula"),
+    path("<str:file_name>", save_model_file, name="save-no-code-model-file"),
 ]

backend/backend/core/routers/transformation/views.py

@@ -211,13 +211,13 @@ def get_supported_models(request: Request, project_id: str, file_name: str) -> R
 @api_view([HTTPMethods.POST])
 @handle_http_request
 @handle_permission
-def generate_formula(request: Request, project_id: str, model_name: str) -> Response:
+def generate_formula(request: Request, project_id: str, file_name: str) -> Response:
     # Generate Excel Formula based on the User prompt with OpenAi support
     user_prompt = request.data["user_prompt"]
     app = FormulaContext(project_id=project_id)
 
     # Get schema details and construct the prompt
-    schema_details = app.get_schema_details(model_name)
+    schema_details = app.get_schema_details(file_name)
     constructed_prompt = app.construct_prompt(user_prompt, schema_details)
 
     # Generate the formula

backend/backend/core/urls.py

@@ -19,8 +19,11 @@
     path("token", include("backend.core.routers.api_tokens.urls")),
     path("connection", include("backend.core.routers.connection.urls")),
     path("environment", include("backend.core.routers.environment.urls")),
-    path("project", include("backend.core.routers.transformation.urls")),
     path("project", include("backend.core.routers.projects.urls")),
+    path(
+        "project/<str:project_id>/no_code_model/",
+        include("backend.core.routers.transformation.urls"),
+    ),
     path("onboarding/", include("backend.core.routers.onboarding.urls")),
     path(
         "project/<str:project_id>/connection",

backend/backend/core/utils.py

@@ -37,8 +37,6 @@ def handle_exceptions(*args, **kwargs) -> Response:
                     )
                 cache.set(lock_key, True, timeout=120)
             response: Response = func(*args, **kwargs)
-            if response.status_code < 400:
-                response.status_code = status.HTTP_200_OK
             logging.info(f"Deleting lock - {lock_key}")
             cache.delete(lock_key)
             return response