feat(config): 增加两套引导视频奖励配置 #96

2026-07-29T14:49:37+08:00

zuochenyong commented

2026-07-29 14:49:37 +08:00

No description provided.

zuochenyong added 1 commit 2026-07-29 14:49:37 +08:00

feat(config): 增加两套引导视频奖励配置 b74bfcbf64

guke commented

2026-07-29 15:08:55 +08:00

🤖 review-pr 深审结论

🔴 必须改（后端契约缺失） · 置信度 0.9

前端把「引导视频」配置重构成 coupon / comparison 两套场景（每个 API 带 ?scene=），并重新开放 max_plays/reward_coin 编辑、展示 total_plays/granted_plays。但——

high（阻断） GuideVideoConfig.tsx — 后端 origin/main 的 admin API /admin/api/guide-video（GET/PATCH//video）是单份全局配置（get_config(db) / update_config(db, enabled/max_plays/reward_coin) 无 scene；配置存单个 AppConfig[_KEY] 行），不认 ?scene=。scene 只存在于播放表 GuideVideoPlay 与客户端 /api/v1。→ 两个 tab 读写同一份配置，保存 comparison 会覆盖 coupon（反之亦然）；FastAPI 忽略多余 query 参数、请求仍返回 200，静默错乱不报错。且当前无任何 open 后端 PR 补 admin 层 scene。
low src/lib/types.ts — max_plays/total_plays 等字段注释仍写「后台不开放调整 / 不再展示」，与本 PR 重新开放编辑+展示自相矛盾，建议同步更新注释。

正面：InputNumber 上限 50/10000 与后端 MAX_PLAYS_LIMIT=50 / REWARD_COIN_LIMIT=10_000 精确对齐；types.ts 字段齐、前端用法类型自洽；无运行时崩溃。

构建：未跑前端全量构建（隔离 worktree 无 node_modules；本 PR 缺陷是运行时契约，构建也测不出）。类型层已静态核对。

建议：后端先加 admin 层 scene 支持（每场景一份配置 + GET/PATCH//video/DELETE 带 scene），#96 再合并；否则两 tab 互相覆盖。或本期先只保留单场景（撤 comparison tab），待后端就绪再上第二套。

_{🔒 只读深审 · worktree 旁路隔离 · 已核验零残留}

## 🤖 review-pr 深审结论 🔴 **必须改（后端契约缺失）** · 置信度 0.9 前端把「引导视频」配置重构成 coupon / comparison 两套场景（每个 API 带 `?scene=`），并重新开放 `max_plays`/`reward_coin` 编辑、展示 `total_plays`/`granted_plays`。但—— - **high（阻断）** `GuideVideoConfig.tsx` — 后端 `origin/main` 的 admin API `/admin/api/guide-video`（GET/PATCH/`/video`）是**单份全局配置**（`get_config(db)` / `update_config(db, enabled/max_plays/reward_coin)` 无 scene；配置存单个 `AppConfig[_KEY]` 行），**不认 `?scene=`**。scene 只存在于播放表 `GuideVideoPlay` 与客户端 `/api/v1`。→ 两个 tab 读写同一份配置，**保存 comparison 会覆盖 coupon**（反之亦然）；FastAPI 忽略多余 query 参数、请求仍返回 200，**静默错乱不报错**。且当前无任何 open 后端 PR 补 admin 层 scene。 - **low** `src/lib/types.ts` — `max_plays`/`total_plays` 等字段注释仍写「后台不开放调整 / 不再展示」，与本 PR 重新开放编辑+展示自相矛盾，建议同步更新注释。 **正面**：InputNumber 上限 `50`/`10000` 与后端 `MAX_PLAYS_LIMIT=50` / `REWARD_COIN_LIMIT=10_000` 精确对齐；types.ts 字段齐、前端用法类型自洽；无运行时崩溃。 **构建**：未跑前端全量构建（隔离 worktree 无 node_modules；本 PR 缺陷是运行时契约，构建也测不出）。类型层已静态核对。 **建议**：后端先加 admin 层 scene 支持（每场景一份配置 + GET/PATCH/`/video`/DELETE 带 scene），#96 再合并；否则两 tab 互相覆盖。或本期先只保留单场景（撤 comparison tab），待后端就绪再上第二套。 <sub>🔒 只读深审 · worktree 旁路隔离 · 已核验零残留</sub>

guke added 1 commit 2026-07-29 15:18:19 +08:00

Merge branch 'main' into fix-guideVideoFix 767585cafa

guke merged commit c564e9df12 into main

2026-07-29 15:18:52 +08:00

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: WonderableAI/shaguabijia-admin-web#96