refactor(BA-5844): switch REST v2 Policy surface to id-keyed routes

Track BA-5814 / BA-5815 DTO refactor:
  • GET /v2/app-config-policies/{config_name}
    → GET /v2/app-config-policies/{policy_id}
  • Replace AppConfigPolicyConfigNamePathParam with
    AppConfigPolicyIdPathParam (policy_id: UUID).
  • Update bulk-purge handler docstring (purge keyed on row id).

Author: jopemachine

changes/11312.feature.md

@@ -1 +1 @@
-Add `AppConfigPolicy` REST v2 surface (`POST /v2/app-config-policies/search`, `GET /v2/app-config-policies/{config_name}`, admin `bulk-create` / `bulk-update` / `bulk-purge`) — pairs with the GraphQL surface from BA-5815.
+Add `AppConfigPolicy` REST v2 surface (`POST /v2/app-config-policies/search`, `GET /v2/app-config-policies/{policy_id}`, admin `bulk-create` / `bulk-update` / `bulk-purge`) — pairs with the GraphQL surface from BA-5815.

src/ai/backend/manager/api/rest/v2/app_config_policy/handler.py

@@ -19,7 +19,7 @@
     SearchAppConfigPoliciesInput,
 )
 from ai.backend.logging import BraceStyleAdapter
-from ai.backend.manager.api.rest.v2.path_params import AppConfigPolicyConfigNamePathParam
+from ai.backend.manager.api.rest.v2.path_params import AppConfigPolicyIdPathParam
 
 if TYPE_CHECKING:
     from ai.backend.manager.api.adapters.app_config_policy import AppConfigPolicyAdapter
@@ -37,10 +37,10 @@ def __init__(self, *, adapter: AppConfigPolicyAdapter) -> None:
 
     async def get(
         self,
-        path: PathParam[AppConfigPolicyConfigNamePathParam],
+        path: PathParam[AppConfigPolicyIdPathParam],
     ) -> APIResponse:
-        """Read a single policy by `config_name` (any authenticated user)."""
-        result = await self._adapter.get(path.parsed.config_name)
+        """Read a single policy by row id (any authenticated user)."""
+        result = await self._adapter.get(path.parsed.policy_id)
         return APIResponse.build(status_code=HTTPStatus.OK, response_model=result)
 
     async def search(
@@ -73,6 +73,6 @@ async def admin_bulk_purge(
         self,
         body: BodyParam[AdminBulkPurgeAppConfigPoliciesInput],
     ) -> APIResponse:
-        """Hard-delete (admin only); referenced `config_name`s fail per-item."""
+        """Hard-delete by row id (admin only); rows still referenced by fragments fail per-item."""
         result = await self._adapter.admin_bulk_purge(body.parsed)
         return APIResponse.build(status_code=HTTPStatus.OK, response_model=result)

src/ai/backend/manager/api/rest/v2/app_config_policy/registry.py

@@ -19,15 +19,15 @@ def register_v2_app_config_policy_routes(
 ) -> RouteRegistry:
     """Register all v2 app-config policy routes.
 
-    Reads (`GET /{config_name}`, `POST /search`) are available to any
+    Reads (`GET /{policy_id}`, `POST /search`) are available to any
     authenticated user. Writes are bulk-only and admin-only —
     `/bulk-create`, `/bulk-update`, `/bulk-purge`.
     """
     reg = RouteRegistry.create("app-config-policies", route_deps.cors_options)
 
     # Reads
     reg.add("POST", "/search", handler.search, middlewares=[auth_required])
-    reg.add("GET", "/{config_name}", handler.get, middlewares=[auth_required])
+    reg.add("GET", "/{policy_id}", handler.get, middlewares=[auth_required])
     # Admin bulk writes
     reg.add("POST", "/bulk-create", handler.admin_bulk_create, middlewares=[superadmin_required])
     reg.add("POST", "/bulk-update", handler.admin_bulk_update, middlewares=[superadmin_required])

src/ai/backend/manager/api/rest/v2/path_params.py

@@ -9,8 +9,8 @@
 from ai.backend.common.api_handlers import BaseRequestModel
 
 
-class AppConfigPolicyConfigNamePathParam(BaseRequestModel):
-    config_name: str = Field(description="App-config policy `config_name`")
+class AppConfigPolicyIdPathParam(BaseRequestModel):
+    policy_id: UUID = Field(description="App-config policy row UUID")
 
 
 class DomainNamePathParam(BaseRequestModel):