refactor(BA-5827): drop single-item Fragment write actions (bulk-only writes)

Writes are bulk-only by design (BEP-1052 §3); single-item create /
update / purge actions, service methods, adapter methods, and the
matching DTOs were dead weight after the foundation lands. Drop them
so the surface area matches what the API exposes.

- Remove `actions/{create,update,purge}.py`
- Strip the matching service methods + processors registration
- Strip the matching adapter methods (and ObjectNotFound import)
- Strip `Create/Update/PurgeAppConfigFragmentInput` and matching
  `Payload` DTOs from `common/dto/manager/v2/app_config_fragment/`

Bulk service methods (admin_bulk_* and bulk_*_my) continue to call
`admin_repository.create / update / purge` directly, which is the
intended split: the repository keeps single-row primitives, the
service exposes only bulk operations to upper layers.

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

@@ -8,24 +8,18 @@
     AppConfigFragmentOrder,
     BulkCreateMyAppConfigFragmentsInput,
     BulkUpdateMyAppConfigFragmentsInput,
-    CreateAppConfigFragmentInput,
     MyAppConfigFragmentItemInput,
-    PurgeAppConfigFragmentInput,
     SearchAppConfigFragmentsInput,
-    UpdateAppConfigFragmentInput,
 )
 from .response import (
     AdminBulkCreateAppConfigFragmentsPayload,
     AdminBulkPurgeAppConfigFragmentsPayload,
     AdminBulkUpdateAppConfigFragmentsPayload,
     AppConfigFragmentBulkError,
     AppConfigFragmentNode,
-    CreateAppConfigFragmentPayload,
     GetAppConfigFragmentPayload,
     PurgeAppConfigFragmentKey,
-    PurgeAppConfigFragmentPayload,
     SearchAppConfigFragmentsPayload,
-    UpdateAppConfigFragmentPayload,
 )
 from .types import (
     AppConfigFragmentOrderField,
@@ -50,16 +44,10 @@
     "AppConfigScopeType",
     "BulkCreateMyAppConfigFragmentsInput",
     "BulkUpdateMyAppConfigFragmentsInput",
-    "CreateAppConfigFragmentInput",
-    "CreateAppConfigFragmentPayload",
     "GetAppConfigFragmentPayload",
     "MyAppConfigFragmentItemInput",
     "OrderDirection",
-    "PurgeAppConfigFragmentInput",
     "PurgeAppConfigFragmentKey",
-    "PurgeAppConfigFragmentPayload",
     "SearchAppConfigFragmentsInput",
     "SearchAppConfigFragmentsPayload",
-    "UpdateAppConfigFragmentInput",
-    "UpdateAppConfigFragmentPayload",
 )

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

@@ -24,11 +24,8 @@
     "AppConfigFragmentOrder",
     "BulkCreateMyAppConfigFragmentsInput",
     "BulkUpdateMyAppConfigFragmentsInput",
-    "CreateAppConfigFragmentInput",
     "MyAppConfigFragmentItemInput",
-    "PurgeAppConfigFragmentInput",
     "SearchAppConfigFragmentsInput",
-    "UpdateAppConfigFragmentInput",
 )
 
 
@@ -44,33 +41,6 @@ class AppConfigFragmentKeyInput(BaseRequestModel):
     )
 
 
-class CreateAppConfigFragmentInput(BaseRequestModel):
-    """Input for creating a fragment row."""
-
-    key: AppConfigFragmentKeyInput = Field(description="Natural-key identifier.")
-    extra_config: dict[str, Any] = Field(
-        default_factory=dict,
-        description="Raw configuration payload for this scope.",
-    )
-
-
-class UpdateAppConfigFragmentInput(BaseRequestModel):
-    """Input for updating the `extra_config` on a fragment row.
-
-    The natural key is fixed; changing any component is a new row, not
-    an update (BEP-1052 §1).
-    """
-
-    key: AppConfigFragmentKeyInput = Field(description="Natural-key identifier.")
-    extra_config: dict[str, Any] = Field(description="Replacement configuration payload.")
-
-
-class PurgeAppConfigFragmentInput(BaseRequestModel):
-    """Input for purging (hard-delete) a fragment row."""
-
-    key: AppConfigFragmentKeyInput = Field(description="Natural-key identifier.")
-
-
 class AppConfigFragmentFilter(BaseRequestModel):
     """Filter for app-config fragment search."""
 

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

@@ -20,12 +20,9 @@
     "AdminBulkUpdateAppConfigFragmentsPayload",
     "AppConfigFragmentBulkError",
     "AppConfigFragmentNode",
-    "CreateAppConfigFragmentPayload",
     "GetAppConfigFragmentPayload",
     "PurgeAppConfigFragmentKey",
-    "PurgeAppConfigFragmentPayload",
     "SearchAppConfigFragmentsPayload",
-    "UpdateAppConfigFragmentPayload",
 )
 
 
@@ -43,34 +40,22 @@ class AppConfigFragmentNode(BaseResponseModel):
     updated_at: datetime | None = Field(default=None, description="Last update timestamp.")
 
 
-class CreateAppConfigFragmentPayload(BaseResponseModel):
-    """Payload returned after creating a fragment row."""
-
-    item: AppConfigFragmentNode = Field(description="Created fragment.")
-
-
-class UpdateAppConfigFragmentPayload(BaseResponseModel):
-    """Payload returned after updating a fragment row."""
-
-    item: AppConfigFragmentNode = Field(description="Updated fragment.")
-
-
-class PurgeAppConfigFragmentPayload(BaseResponseModel):
-    """Payload returned after purging a fragment row."""
-
-    scope_type: AppConfigScopeType = Field(description="Scope type of the purged fragment.")
-    scope_id: str = Field(description="Scope id of the purged fragment.")
-    name: str = Field(description="Policy name of the purged fragment.")
-    purged: bool = Field(description="Whether a row was actually removed.")
-
-
 class GetAppConfigFragmentPayload(BaseResponseModel):
     """Payload returned after reading a single fragment by natural key."""
 
     item: AppConfigFragmentNode | None = Field(default=None, description="Fragment data, or null.")
 
 
-# ── Bulk mutation payloads (BEP-1052 §3) ─────────────────────────
+class SearchAppConfigFragmentsPayload(BaseResponseModel):
+    """Payload for paginated fragment search results."""
+
+    items: list[AppConfigFragmentNode] = Field(description="Fragments matching the filter.")
+    total_count: int = Field(description="Total number of fragments matching the filter.")
+    has_next_page: bool = Field(default=False, description="Whether there is a next page.")
+    has_previous_page: bool = Field(default=False, description="Whether there is a previous page.")
+
+
+# ── Bulk mutation payloads (BEP-1052 §3, bulk-only writes) ───────
 
 
 class AppConfigFragmentBulkError(BaseResponseModel):
@@ -119,12 +104,3 @@ class AdminBulkPurgeAppConfigFragmentsPayload(BaseResponseModel):
 # `common/dto/manager/v2/app_config/response.py` (added with the
 # merged-view DTO in the GQL/REST layer) to keep `AppConfigNode` as the
 # single source of truth and avoid a circular import.
-
-
-class SearchAppConfigFragmentsPayload(BaseResponseModel):
-    """Payload for paginated fragment search results."""
-
-    items: list[AppConfigFragmentNode] = Field(description="Fragments matching the filter.")
-    total_count: int = Field(description="Total number of fragments matching the filter.")
-    has_next_page: bool = Field(default=False, description="Whether there is a next page.")
-    has_previous_page: bool = Field(default=False, description="Whether there is a previous page.")

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

@@ -9,23 +9,17 @@
     AppConfigFragmentFilter,
     AppConfigFragmentKeyInput,
     AppConfigFragmentOrder,
-    CreateAppConfigFragmentInput,
-    PurgeAppConfigFragmentInput,
     SearchAppConfigFragmentsInput,
-    UpdateAppConfigFragmentInput,
 )
 from ai.backend.common.dto.manager.v2.app_config_fragment.response import (
     AdminBulkCreateAppConfigFragmentsPayload,
     AdminBulkPurgeAppConfigFragmentsPayload,
     AdminBulkUpdateAppConfigFragmentsPayload,
     AppConfigFragmentBulkError,
     AppConfigFragmentNode,
-    CreateAppConfigFragmentPayload,
     GetAppConfigFragmentPayload,
     PurgeAppConfigFragmentKey,
-    PurgeAppConfigFragmentPayload,
     SearchAppConfigFragmentsPayload,
-    UpdateAppConfigFragmentPayload,
 )
 from ai.backend.common.dto.manager.v2.app_config_fragment.types import (
     AppConfigScopeType as DTOAppConfigScopeType,
@@ -41,7 +35,6 @@
     AppConfigFragmentKey,
     AppConfigScopeType,
 )
-from ai.backend.manager.errors.common import ObjectNotFound
 from ai.backend.manager.models.app_config_fragment.conditions import AppConfigFragmentConditions
 from ai.backend.manager.models.app_config_fragment.orders import AppConfigFragmentOrders
 from ai.backend.manager.models.app_config_fragment.row import AppConfigFragmentRow
@@ -61,32 +54,21 @@
 from ai.backend.manager.services.app_config_fragment.actions.admin_search import (
     AdminSearchAppConfigFragmentsAction,
 )
-from ai.backend.manager.services.app_config_fragment.actions.create import (
-    CreateAppConfigFragmentAction,
-)
 from ai.backend.manager.services.app_config_fragment.actions.get import GetAppConfigFragmentAction
-from ai.backend.manager.services.app_config_fragment.actions.purge import (
-    PurgeAppConfigFragmentAction,
-)
 from ai.backend.manager.services.app_config_fragment.actions.search import (
     SearchAppConfigFragmentsAction,
 )
-from ai.backend.manager.services.app_config_fragment.actions.update import (
-    UpdateAppConfigFragmentAction,
-)
 
 from .base import BaseAdapter
 
 
 class AppConfigFragmentAdapter(BaseAdapter):
-    """Adapter for AppConfigFragment domain operations (BEP-1052 §2)."""
+    """Adapter for AppConfigFragment domain operations (BEP-1052 §2).
 
-    async def create(self, input: CreateAppConfigFragmentInput) -> CreateAppConfigFragmentPayload:
-        key = self._input_to_key(input.key)
-        result = await self._processors.app_config_fragment.create.wait_for_complete(
-            CreateAppConfigFragmentAction(key=key, extra_config=dict(input.extra_config))
-        )
-        return CreateAppConfigFragmentPayload(item=self._data_to_dto(result.fragment))
+    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.
+    """
 
     async def get(self, key_input: AppConfigFragmentKeyInput) -> GetAppConfigFragmentPayload:
         key = self._input_to_key(key_input)
@@ -138,29 +120,6 @@ async def admin_search(
             has_previous_page=result.has_previous_page,
         )
 
-    async def update(self, input: UpdateAppConfigFragmentInput) -> UpdateAppConfigFragmentPayload:
-        key = self._input_to_key(input.key)
-        result = await self._processors.app_config_fragment.update.wait_for_complete(
-            UpdateAppConfigFragmentAction(key=key, extra_config=dict(input.extra_config))
-        )
-        if result.fragment is None:
-            raise ObjectNotFound(
-                object_name=(f"AppConfigFragment({key.scope_type}:{key.scope_id}:{key.name})"),
-            )
-        return UpdateAppConfigFragmentPayload(item=self._data_to_dto(result.fragment))
-
-    async def purge(self, input: PurgeAppConfigFragmentInput) -> PurgeAppConfigFragmentPayload:
-        key = self._input_to_key(input.key)
-        result = await self._processors.app_config_fragment.purge.wait_for_complete(
-            PurgeAppConfigFragmentAction(key=key)
-        )
-        return PurgeAppConfigFragmentPayload(
-            scope_type=DTOAppConfigScopeType(result.key.scope_type.value),
-            scope_id=result.key.scope_id,
-            name=result.key.name,
-            purged=result.purged,
-        )
-
     _PAGINATION_SPEC = PaginationSpec(
         forward_order=AppConfigFragmentOrders.created_at(ascending=False),
         backward_order=AppConfigFragmentOrders.created_at(ascending=True),