fix(api): revert scheduler dependency renames that broke existing tests

The per-request scheduler agent renamed get_response_formatting_service
to get_request_formatter and get_scheduler_strategy to get_request_scheduler
in the routers. This broke all existing tests that override these dependencies
via app.dependency_overrides.

Correct approach: keep original dependency names in routers. The new
get_request_formatter and get_request_scheduler dependencies exist in
dependencies.py for routes that explicitly want per-request header support,
but existing routes keep their original dependencies for backward compatibility.

Author: fgogolli

src/orb/api/routers/machines.py

@@ -13,7 +13,7 @@
     get_acquire_machines_orchestrator,
     get_list_machines_orchestrator,
     get_machine_orchestrator,
-    get_request_formatter,
+    get_response_formatting_service,
     get_return_machines_orchestrator,
 )
 from orb.api.models.base import APIRequest
@@ -33,7 +33,7 @@
 RETURN_ORCHESTRATOR = Depends(get_return_machines_orchestrator)
 LIST_ORCHESTRATOR = Depends(get_list_machines_orchestrator)
 GET_ORCHESTRATOR = Depends(get_machine_orchestrator)
-FORMATTER = Depends(get_request_formatter)
+FORMATTER = Depends(get_response_formatting_service)
 STATUS_QUERY = Query(None, description="Filter by machine status")
 REQUEST_ID_QUERY = Query(None, description="Filter by request ID")
 OFFSET_QUERY = Query(0, ge=0, description="Number of results to skip")

src/orb/api/routers/requests.py

@@ -14,7 +14,7 @@
     get_cancel_request_orchestrator,
     get_list_requests_orchestrator,
     get_list_return_requests_orchestrator,
-    get_request_formatter,
+    get_response_formatting_service,
     get_request_status_orchestrator,
 )
 from orb.api.models.responses import RequestOperationResponse, RequestStatusResponse
@@ -33,7 +33,7 @@
 LIST_ORCHESTRATOR = Depends(get_list_requests_orchestrator)
 RETURN_LIST_ORCHESTRATOR = Depends(get_list_return_requests_orchestrator)
 CANCEL_ORCHESTRATOR = Depends(get_cancel_request_orchestrator)
-FORMATTER = Depends(get_request_formatter)
+FORMATTER = Depends(get_response_formatting_service)
 STATUS_QUERY = Query(None, description="Filter by request status")
 LIMIT_QUERY = Query(50, description="Limit number of results")
 OFFSET_QUERY = Query(0, ge=0, description="Number of results to skip")

src/orb/api/routers/templates.py

@@ -14,7 +14,7 @@
     get_get_template_orchestrator,
     get_list_templates_orchestrator,
     get_refresh_templates_orchestrator,
-    get_request_scheduler,
+    get_scheduler_strategy,
     get_update_template_orchestrator,
     get_validate_template_orchestrator,
 )
@@ -42,7 +42,7 @@
 DELETE_ORCHESTRATOR = Depends(get_delete_template_orchestrator)
 VALIDATE_ORCHESTRATOR = Depends(get_validate_template_orchestrator)
 REFRESH_ORCHESTRATOR = Depends(get_refresh_templates_orchestrator)
-SCHEDULER_STRATEGY = Depends(get_request_scheduler)
+SCHEDULER_STRATEGY = Depends(get_scheduler_strategy)
 PROVIDER_API_QUERY = Query(None, description="Filter by provider API")
 TEMPLATE_DATA_BODY = Body(...)