revert(backend): 撤回 #2053 后端改动，让前端PR先合入

按方案1拆分：撤回 generation_cover.py 的 template_id 可选改动和 dependencies.py 的 get_draft_plan_id_optional 新增依赖。PR#2053 现在只包含前端修复（模板缩略图渲染 + 不传 "default" 魔法字符串 + 真实模板id解析）。后端改动（让 /generate-cover 支持 template_id 为空自动兜底默认模板）后续会在独立 PR 中跟进，配合单测更新一起合。前端已处理的向后兼容： - api/generation/cover.ts: templateId 为空或 'default' 时不传 template_id 参数。当前后端 template_id 是必填 Query，会返回 422 Validation Error，这比之前传 'default' 返回 404 '模板不存在' 更明确； - useSharedCover 挂载时会自动拉模板列表把 'default' 占位符解析为第一个系统模板的真实 id，因此绝大多数正常路径（选过模板/默认系统模板）不会触发空 template_id 分支； - useBatchCovers 同样使用解析后的真实模板 id 或空串，空串情况只发生在用户删除全部模板后的极端兜底场景，届时用户会看到 422 错误提示需要新建模板，属于可接受的退化行为。
2026-09-26 18:18:40 +08:00
parent a989670d83
commit 6becc7beca
2 changed files with 5 additions and 38 deletions
@@ -30,7 +30,7 @@ from packages.domain.config_schemas import normalize_plan_config
 from packages.middleware.points_gate import points_gate
 from packages.shared.storage import get_shared_storage_service

-from .templates_editor.dependencies import get_editor_services, resolve_draft_plan_id
+from .templates_editor.dependencies import get_draft_plan_id, get_editor_services

 logger = logging.getLogger(__name__)
 router = APIRouter(tags=["Generation"])
@@ -349,8 +349,8 @@ def _is_trusted_media_url(url: str) -> bool:
@points_gate("ai_cover")
 def generate_cover(
    body: GenerateCoverRequest,
-    template_id: str = Query("", description="模板 ID，为空时使用用户默认模板"),
-    plan_id: str = "",
+    template_id: str = Query(..., description="模板 ID"),
+    plan_id: str = Depends(get_draft_plan_id),
    services: tuple[EditTemplateService, EditPlanService] = Depends(get_editor_services),
    db: Session = Depends(get_db_session),
    current_user: AuthenticatedUser = Depends(get_current_user),
@@ -366,19 +366,7 @@ def generate_cover(

    MediaKit 的调用方式（strategy / max_frames / 轮询 / 重试 / 降级）不变。
    """
-    _tpl_svc, plan_svc = services
-    # 只有显式未传 plan_id（HTTP 调用默认空串）时才按 template_id 解析，
-    # 以便单测可直接传入 mock 的 plan_id 字符串。
-    # 注意：plan_id 不作为公开 query 参数暴露，直接传任意 plan_id 会被
-    # get_plan_or_raise 校验，不会造成越权（plan 归属校验在 service 层）。
-    if not plan_id:
-        plan_id = resolve_draft_plan_id(
-            template_id=template_id,
-            services=services,
-            current_user=current_user,
-            db=db,
-            auto_create_default=(not template_id),
-        )
+    _, plan_svc = services
    plan = plan_svc.get_plan_or_raise(plan_id)

    # ── upload 类型：直接保存前端上传的封面图片，不需要预览视频 ──────
@@ -3,7 +3,6 @@
 核心依赖：
 - get_editor_services: 获取模板+计划服务
 - get_draft_plan_id: Depends 形式的路径依赖（template_id 路径参数必填）
- get_draft_plan_id_optional: Depends 形式的查询依赖（template_id 可为空，自动兜底默认模板）
 - resolve_draft_plan_id: 纯函数版本，供 clips_standalone 等非路径参数场景复用
  （支持空 tid 时自动兜底创建默认模板）
 """
@@ -17,7 +16,7 @@ from app.auth import AuthenticatedUser, get_current_user
 from app.dependencies import get_db_session
 from app.services.edit_plan_service import EditPlanService
 from app.services.edit_template_service import EditTemplateService
-from fastapi import Depends, HTTPException, Query, status
+from fastapi import Depends, HTTPException, status
 from sqlalchemy.orm import Session

 from packages.adapters.sqlalchemy_impl.template_repository import (
@@ -148,23 +147,3 @@ def get_draft_plan_id(
        db=db,
        auto_create_default=False,
    )
-
-
-def get_draft_plan_id_optional(
-    template_id: str = Query("", description="模板 ID，为空时使用用户默认模板"),
-    services: tuple[EditTemplateService, EditPlanService] = Depends(get_editor_services),
-    current_user: AuthenticatedUser = Depends(get_current_user),
-    db: Session = Depends(get_db_session),
-) -> str:
-    """可选 Depends 版本：template_id 为空时自动创建/获取用户默认模板.
-
-    供 /generation/generate-cover 等「用户没选模板也应可用」的端点复用。
-    返回的 plan_id 一定非空（字符串）。
-    """
-    return resolve_draft_plan_id(
-        template_id=template_id,
-        services=services,
-        current_user=current_user,
-        db=db,
-        auto_create_default=True,
-    )