Skip to content

Commit e7ba59c

Browse files
jopemachineclaude
andcommitted
refactor(BA-5815): drop REST v2 surface — split into BA-5844
The combined GraphQL + REST landing didn't match the AppConfigFragment review pattern (BA-5829 GraphQL, BA-5830 REST as separate PRs). Move the REST handler / registry / path-param wiring out to BA-5844 and narrow this PR (and its news fragment) to the AppConfigPolicy GraphQL surface only. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent dde96d8 commit e7ba59c

6 files changed

Lines changed: 1 addition & 125 deletions

File tree

changes/11269.feature.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Add `AppConfigPolicy` vertical (service / GraphQL types/queries/mutations / REST endpoints / SDK / CLI) per BEP-1052 §3 — admins can create / update / purge / list policies that gate AppConfigFragment writes. Scaffold landing in this PR; full surface follows.
1+
Add `AppConfigPolicy` GraphQL surface (types / queries / bulk mutations) — admins can create / update / purge / list policies that gate AppConfigFragment writes via Strawberry GQL.

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

Whitespace-only changes.

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

Lines changed: 0 additions & 78 deletions
This file was deleted.

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

Lines changed: 0 additions & 36 deletions
This file was deleted.

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

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,6 @@
99
from ai.backend.common.api_handlers import BaseRequestModel
1010

1111

12-
class AppConfigPolicyConfigNamePathParam(BaseRequestModel):
13-
config_name: str = Field(description="App-config policy `config_name`")
14-
15-
1612
class DomainNamePathParam(BaseRequestModel):
1713
domain_name: str = Field(description="Domain name")
1814

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

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,6 @@ def build_v2_routes(
2828
# Lazy imports to avoid circular dependencies at module level
2929
from .agent.handler import V2AgentHandler
3030
from .agent.registry import register_v2_agent_routes
31-
from .app_config_policy.handler import V2AppConfigPolicyHandler
32-
from .app_config_policy.registry import register_v2_app_config_policy_routes
3331
from .artifact.handler import V2ArtifactHandler
3432
from .artifact.registry import register_v2_artifact_routes
3533
from .artifact_registry.handler import V2ArtifactRegistryHandler
@@ -117,7 +115,6 @@ def build_v2_routes(
117115

118116
# Build all handlers (each takes its individual adapter)
119117
agent_handler = V2AgentHandler(adapter=adapters.agent)
120-
app_config_policy_handler = V2AppConfigPolicyHandler(adapter=adapters.app_config_policy)
121118
artifact_handler = V2ArtifactHandler(adapter=adapters.artifact)
122119
artifact_registry_handler = V2ArtifactRegistryHandler(adapter=adapters.artifact_registry)
123120
audit_log_handler = V2AuditLogHandler(adapter=adapters.audit_log)
@@ -174,9 +171,6 @@ def build_v2_routes(
174171

175172
# Add all domain sub-registries
176173
v2_reg.add_subregistry(register_v2_agent_routes(agent_handler, route_deps))
177-
v2_reg.add_subregistry(
178-
register_v2_app_config_policy_routes(app_config_policy_handler, route_deps)
179-
)
180174
v2_reg.add_subregistry(register_v2_artifact_routes(artifact_handler, route_deps))
181175
v2_reg.add_subregistry(
182176
register_v2_artifact_registry_routes(artifact_registry_handler, route_deps)

0 commit comments

Comments
 (0)