From 6becc7beca3e0056ef7e1624db7fbfc3b0638a91 Mon Sep 17 00:00:00 2001
From: xiaoxia <xiaoxia@xiaoxiajianji.com>
Date: Sat, 26 Sep 2026 18:18:40 +0800
Subject: [PATCH] =?UTF-8?q?revert(backend):=20=E6=92=A4=E5=9B=9E=20#2053?=
 =?UTF-8?q?=20=E5=90=8E=E7=AB=AF=E6=94=B9=E5=8A=A8=EF=BC=8C=E8=AE=A9?=
 =?UTF-8?q?=E5=89=8D=E7=AB=AFPR=E5=85=88=E5=90=88=E5=85=A5?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

按方案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 错误提示需要
  新建模板，属于可接受的退化行为。
---
 apps/api/app/api/routes/generation_cover.py   | 20 ++++------------
 .../routes/templates_editor/dependencies.py   | 23 +------------------
 2 files changed, 5 insertions(+), 38 deletions(-)

diff --git a/apps/api/app/api/routes/generation_cover.py b/apps/api/app/api/routes/generation_cover.py
index 06fba29eb..c147c0f93 100644
--- a/apps/api/app/api/routes/generation_cover.py
+++ b/apps/api/app/api/routes/generation_cover.py
@@ -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 类型：直接保存前端上传的封面图片，不需要预览视频 ──────
diff --git a/apps/api/app/api/routes/templates_editor/dependencies.py b/apps/api/app/api/routes/templates_editor/dependencies.py
index b6c7014dd..1f70dd25f 100755
--- a/apps/api/app/api/routes/templates_editor/dependencies.py
+++ b/apps/api/app/api/routes/templates_editor/dependencies.py
@@ -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,
-    )