refactor(BA-5836): rename my_bulk_*, drop BEP refs, log.warning, StrEnum match

- Rename `bulk_create_my` / `bulk_update_my` → `my_bulk_create` /
  `my_bulk_update` across action classes, files, service methods,
  processors, and the request DTO module to match the existing `my_`
  prefix convention used elsewhere in the v2 surface (e.g. `my_keypair`).
- Switch order-field dispatch in `AppConfigFragmentAdapter` from string
  `.value` matching to direct StrEnum matching against
  `AppConfigFragmentOrderField`.
- Promote per-item bulk failure logs from `log.debug` to `log.warning`.
- Drop BEP-1052 §X references from doctrings, comments, and inline
  notes across the service / adapter / DTO modules.

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

Author: jopemachine

src/ai/backend/common/dto/manager/v2/app_config_fragment/__init__.py

@@ -6,9 +6,9 @@
     AppConfigFragmentFilter,
     AppConfigFragmentKeyInput,
     AppConfigFragmentOrder,
-    BulkCreateMyAppConfigFragmentsInput,
-    BulkUpdateMyAppConfigFragmentsInput,
     MyAppConfigFragmentItemInput,
+    MyBulkCreateAppConfigFragmentsInput,
+    MyBulkUpdateAppConfigFragmentsInput,
     SearchAppConfigFragmentsInput,
 )
 from .response import (
@@ -42,8 +42,8 @@
     "AppConfigFragmentOrder",
     "AppConfigFragmentOrderField",
     "AppConfigScopeType",
-    "BulkCreateMyAppConfigFragmentsInput",
-    "BulkUpdateMyAppConfigFragmentsInput",
+    "MyBulkCreateAppConfigFragmentsInput",
+    "MyBulkUpdateAppConfigFragmentsInput",
     "GetAppConfigFragmentPayload",
     "MyAppConfigFragmentItemInput",
     "OrderDirection",

src/ai/backend/common/dto/manager/v2/app_config_fragment/request.py

@@ -21,8 +21,8 @@
     "AppConfigFragmentFilter",
     "AppConfigFragmentKeyInput",
     "AppConfigFragmentOrder",
-    "BulkCreateMyAppConfigFragmentsInput",
-    "BulkUpdateMyAppConfigFragmentsInput",
+    "MyBulkCreateAppConfigFragmentsInput",
+    "MyBulkUpdateAppConfigFragmentsInput",
     "MyAppConfigFragmentItemInput",
     "SearchAppConfigFragmentsInput",
 )
@@ -56,7 +56,7 @@ class AppConfigFragmentOrder(BaseRequestModel):
     direction: OrderDirection = Field(default=OrderDirection.ASC, description="Order direction.")
 
 
-# ── Bulk mutation inputs (BEP-1052 §3) ───────────────────────────
+# ── Bulk mutation inputs ─────────────────────────────────────────
 
 
 class AdminAppConfigFragmentItemInput(BaseRequestModel):
@@ -93,11 +93,11 @@ class MyAppConfigFragmentItemInput(BaseRequestModel):
     )
 
 
-class BulkCreateMyAppConfigFragmentsInput(BaseRequestModel):
+class MyBulkCreateAppConfigFragmentsInput(BaseRequestModel):
     items: list[MyAppConfigFragmentItemInput] = Field(description="USER-scope rows to create.")
 
 
-class BulkUpdateMyAppConfigFragmentsInput(BaseRequestModel):
+class MyBulkUpdateAppConfigFragmentsInput(BaseRequestModel):
     items: list[MyAppConfigFragmentItemInput] = Field(description="USER-scope rows to update.")
 
 

src/ai/backend/common/dto/manager/v2/app_config_fragment/response.py

@@ -55,7 +55,7 @@ class SearchAppConfigFragmentsPayload(BaseResponseModel):
     has_previous_page: bool = Field(default=False, description="Whether there is a previous page.")
 
 
-# ── Bulk mutation payloads (BEP-1052 §3, bulk-only writes) ───────
+# ── Bulk mutation payloads (bulk-only writes) ────────────────────
 
 
 class AppConfigFragmentBulkError(BaseResponseModel):
@@ -99,7 +99,7 @@ class AdminBulkPurgeAppConfigFragmentsPayload(BaseResponseModel):
     failed: list[AppConfigFragmentBulkError] = Field(description="Per-item failures.")
 
 
-# `BulkCreateMyAppConfigFragmentsPayload` / `BulkUpdateMyAppConfigFragmentsPayload`
+# `MyBulkCreateAppConfigFragmentsPayload` / `MyBulkUpdateAppConfigFragmentsPayload`
 # return recomputed merged `AppConfig` views — they live in
 # `common/dto/manager/v2/app_config/response.py` (added with the
 # merged-view DTO in the GQL/REST layer) to keep `AppConfigNode` as the

src/ai/backend/common/dto/manager/v2/app_config_fragment/types.py

@@ -16,7 +16,7 @@
 
 
 class AppConfigScopeType(StrEnum):
-    """Scope types for app-config fragments (BEP-1052 §1)."""
+    """Scope types for app-config fragments."""
 
     PUBLIC = "public"
     DOMAIN = "domain"

src/ai/backend/manager/api/adapters/app_config_fragment.py

@@ -21,10 +21,13 @@
     PurgeAppConfigFragmentKey,
     SearchAppConfigFragmentsPayload,
 )
+from ai.backend.common.dto.manager.v2.app_config_fragment.types import (
+    AppConfigFragmentOrderField,
+    OrderDirection,
+)
 from ai.backend.common.dto.manager.v2.app_config_fragment.types import (
     AppConfigScopeType as DTOAppConfigScopeType,
 )
-from ai.backend.common.dto.manager.v2.app_config_fragment.types import OrderDirection
 from ai.backend.manager.api.adapter_options.pagination.pagination import PaginationSpec
 from ai.backend.manager.data.app_config_fragment.bulk_types import (
     AppConfigFragmentBulkItem,
@@ -63,11 +66,11 @@
 
 
 class AppConfigFragmentAdapter(BaseAdapter):
-    """Adapter for AppConfigFragment domain operations (BEP-1052 §2).
+    """Adapter for AppConfigFragment domain operations.
 
-    Writes are bulk-only (BEP-1052 §3); single-item create / update /
-    purge entry points are intentionally absent. Self-service my_bulk
-    methods are added with the merged-view DTOs in BA-5829.
+    Writes are bulk-only; single-item create / update / purge entry
+    points are intentionally absent. Self-service my_bulk methods are
+    added with the merged-view DTOs in BA-5829.
     """
 
     async def get(self, key_input: AppConfigFragmentKeyInput) -> GetAppConfigFragmentPayload:
@@ -186,16 +189,16 @@ def _convert_orders(orders: list[AppConfigFragmentOrder]) -> list[QueryOrder]:
         result: list[QueryOrder] = []
         for order in orders:
             ascending = order.direction == OrderDirection.ASC
-            match order.field.value:
-                case "scope_type":
+            match order.field:
+                case AppConfigFragmentOrderField.SCOPE_TYPE:
                     result.append(AppConfigFragmentOrders.scope_type(ascending))
-                case "scope_id":
+                case AppConfigFragmentOrderField.SCOPE_ID:
                     result.append(AppConfigFragmentOrders.scope_id(ascending))
-                case "name":
+                case AppConfigFragmentOrderField.NAME:
                     result.append(AppConfigFragmentOrders.name(ascending))
-                case "created_at":
+                case AppConfigFragmentOrderField.CREATED_AT:
                     result.append(AppConfigFragmentOrders.created_at(ascending))
-                case "updated_at":
+                case AppConfigFragmentOrderField.UPDATED_AT:
                     result.append(AppConfigFragmentOrders.updated_at(ascending))
         return result
 
@@ -219,7 +222,7 @@ def _data_to_dto(data: AppConfigFragmentData) -> AppConfigFragmentNode:
             updated_at=data.updated_at,
         )
 
-    # ── Bulk mutations (BEP-1052 §3) ───────────────────────────────
+    # ── Bulk mutations ─────────────────────────────────────────────
     #
     # Each bulk processor returns a `BulkProcessResult[T]` whose
     # `.result` field is the underlying `*ActionResult` produced by the

src/ai/backend/manager/data/app_config_fragment/bulk_types.py

@@ -1,4 +1,4 @@
-"""Bulk-mutation service-layer dataclasses for app_config_fragments (BEP-1052 §3)."""
+"""Bulk-mutation service-layer dataclasses for app_config_fragments."""
 
 from __future__ import annotations
 
@@ -31,7 +31,7 @@ class MyAppConfigFragmentBulkItem:
 
 @dataclass(frozen=True)
 class AppConfigFragmentBulkItemError:
-    """Per-item failure carried through bulk action results (BEP-1052 §3).
+    """Per-item failure carried through bulk action results.
 
     `scope_type` / `scope_id` / `name` identify which input row failed;
     `index` preserves the caller's original list position.

…onfig_fragment/actions/bulk_create_my.py …onfig_fragment/actions/my_bulk_create.pysrc/ai/backend/manager/services/app_config_fragment/actions/bulk_create_my.py renamed to src/ai/backend/manager/services/app_config_fragment/actions/my_bulk_create.py src/ai/backend/manager/services/app_config_fragment/actions/bulk_create_my.py renamed to src/ai/backend/manager/services/app_config_fragment/actions/my_bulk_create.py

@@ -14,7 +14,7 @@
 
 
 @dataclass
-class BulkCreateMyAppConfigFragmentsAction(BaseBulkAction[MyAppConfigFragmentBulkItem]):
+class MyBulkCreateAppConfigFragmentsAction(BaseBulkAction[MyAppConfigFragmentBulkItem]):
     """Self-service bulk create — scope is `USER` / `current_user.user_id`.
 
     The owning `user_id` is resolved from the request `ContextVar`
@@ -41,7 +41,7 @@ def operation_type(cls) -> ActionOperationType:
 
 
 @dataclass
-class BulkCreateMyAppConfigFragmentsActionResult(BaseBulkActionResult):
+class MyBulkCreateAppConfigFragmentsActionResult(BaseBulkActionResult):
     """`created` carries the recomputed merged view per successfully
     created fragment; `failed` carries per-item errors.
     """

…onfig_fragment/actions/bulk_update_my.py …onfig_fragment/actions/my_bulk_update.pysrc/ai/backend/manager/services/app_config_fragment/actions/bulk_update_my.py renamed to src/ai/backend/manager/services/app_config_fragment/actions/my_bulk_update.py src/ai/backend/manager/services/app_config_fragment/actions/bulk_update_my.py renamed to src/ai/backend/manager/services/app_config_fragment/actions/my_bulk_update.py

@@ -14,7 +14,7 @@
 
 
 @dataclass
-class BulkUpdateMyAppConfigFragmentsAction(BaseBulkAction[MyAppConfigFragmentBulkItem]):
+class MyBulkUpdateAppConfigFragmentsAction(BaseBulkAction[MyAppConfigFragmentBulkItem]):
     """Self-service bulk update — see `BulkCreateMyAppConfigFragmentsAction`."""
 
     items: list[MyAppConfigFragmentBulkItem] = field(default_factory=list)
@@ -35,7 +35,7 @@ def operation_type(cls) -> ActionOperationType:
 
 
 @dataclass
-class BulkUpdateMyAppConfigFragmentsActionResult(BaseBulkActionResult):
+class MyBulkUpdateAppConfigFragmentsActionResult(BaseBulkActionResult):
     updated: list[AppConfigData]
     failed: list[AppConfigFragmentBulkItemError]
 

src/ai/backend/manager/services/app_config_fragment/processors.py

@@ -21,18 +21,18 @@
     AdminSearchAppConfigFragmentsAction,
     AdminSearchAppConfigFragmentsActionResult,
 )
-from ai.backend.manager.services.app_config_fragment.actions.bulk_create_my import (
-    BulkCreateMyAppConfigFragmentsAction,
-    BulkCreateMyAppConfigFragmentsActionResult,
-)
-from ai.backend.manager.services.app_config_fragment.actions.bulk_update_my import (
-    BulkUpdateMyAppConfigFragmentsAction,
-    BulkUpdateMyAppConfigFragmentsActionResult,
-)
 from ai.backend.manager.services.app_config_fragment.actions.get import (
     GetAppConfigFragmentAction,
     GetAppConfigFragmentActionResult,
 )
+from ai.backend.manager.services.app_config_fragment.actions.my_bulk_create import (
+    MyBulkCreateAppConfigFragmentsAction,
+    MyBulkCreateAppConfigFragmentsActionResult,
+)
+from ai.backend.manager.services.app_config_fragment.actions.my_bulk_update import (
+    MyBulkUpdateAppConfigFragmentsAction,
+    MyBulkUpdateAppConfigFragmentsActionResult,
+)
 from ai.backend.manager.services.app_config_fragment.actions.search import (
     SearchAppConfigFragmentsAction,
     SearchAppConfigFragmentsActionResult,
@@ -46,10 +46,10 @@ class AppConfigFragmentProcessors(AbstractProcessorPackage):
     admin_search: ActionProcessor[
         AdminSearchAppConfigFragmentsAction, AdminSearchAppConfigFragmentsActionResult
     ]
-    # Bulk mutations (BEP-1052 §3) — wrapped by BulkActionProcessor so
-    # validators (RBAC, etc.) can filter entity_ids per-item before the
-    # service runs. No bulk validators are wired today; the processor
-    # simply forwards to the service.
+    # Bulk mutations — wrapped by BulkActionProcessor so validators
+    # (RBAC, etc.) can filter entity_ids per-item before the service
+    # runs. No bulk validators are wired today; the processor simply
+    # forwards to the service.
     admin_bulk_create: BulkActionProcessor[
         AdminBulkCreateAppConfigFragmentsAction, AdminBulkCreateAppConfigFragmentsActionResult
     ]
@@ -59,11 +59,11 @@ class AppConfigFragmentProcessors(AbstractProcessorPackage):
     admin_bulk_purge: BulkActionProcessor[
         AdminBulkPurgeAppConfigFragmentsAction, AdminBulkPurgeAppConfigFragmentsActionResult
     ]
-    bulk_create_my: BulkActionProcessor[
-        BulkCreateMyAppConfigFragmentsAction, BulkCreateMyAppConfigFragmentsActionResult
+    my_bulk_create: BulkActionProcessor[
+        MyBulkCreateAppConfigFragmentsAction, MyBulkCreateAppConfigFragmentsActionResult
     ]
-    bulk_update_my: BulkActionProcessor[
-        BulkUpdateMyAppConfigFragmentsAction, BulkUpdateMyAppConfigFragmentsActionResult
+    my_bulk_update: BulkActionProcessor[
+        MyBulkUpdateAppConfigFragmentsAction, MyBulkUpdateAppConfigFragmentsActionResult
     ]
 
     def __init__(
@@ -78,8 +78,8 @@ def __init__(
         self.admin_bulk_create = BulkActionProcessor(service.admin_bulk_create, action_monitors)
         self.admin_bulk_update = BulkActionProcessor(service.admin_bulk_update, action_monitors)
         self.admin_bulk_purge = BulkActionProcessor(service.admin_bulk_purge, action_monitors)
-        self.bulk_create_my = BulkActionProcessor(service.bulk_create_my, action_monitors)
-        self.bulk_update_my = BulkActionProcessor(service.bulk_update_my, action_monitors)
+        self.my_bulk_create = BulkActionProcessor(service.my_bulk_create, action_monitors)
+        self.my_bulk_update = BulkActionProcessor(service.my_bulk_update, action_monitors)
 
     @override
     def supported_actions(self) -> list[ActionSpec]:
@@ -90,6 +90,6 @@ def supported_actions(self) -> list[ActionSpec]:
             AdminBulkCreateAppConfigFragmentsAction.spec(),
             AdminBulkUpdateAppConfigFragmentsAction.spec(),
             AdminBulkPurgeAppConfigFragmentsAction.spec(),
-            BulkCreateMyAppConfigFragmentsAction.spec(),
-            BulkUpdateMyAppConfigFragmentsAction.spec(),
+            MyBulkCreateAppConfigFragmentsAction.spec(),
+            MyBulkUpdateAppConfigFragmentsAction.spec(),
         ]