feat(points): P2 step 1 - POINTS_ENABLED safety gate (default false) #1936

Merged

auto-approve-bot merged 5 commits from feat/points-gate-config into develop 2026-09-16 00:26:29 +08:00

Conversation 3 Commits 5 Files Changed 5

+172 -39

xiaoxia commented 2026-09-15 18:40:38 +08:00

Owner

Copy Link

Copy Source

#1895 P2 第 1 步：积分系统配置安全闸门

按要求先补总开关，再逐个路由接入 @points_gate，确保 P2 阶段合入后对现有用户零影响。

改动

1. packages/config/base.py SharedSettings 新增 points_enabled: bool = False（pydantic-settings 大小写不敏感，自动支持 POINTS_ENABLED=true 环境变量）
2. .env.example 新增 POINTS_ENABLED=false 配置项及注释说明
3. packages/middleware/points_gate.py 新增模块级函数 _points_gate_enabled() 读取开关：
   • 默认 false（开关未开）时，@points_gate 装饰器完全透传原函数，零开销零副作用，不读 user/db、不扣点、不抛 401/402
   • true 时执行原有扣费/401/402/异常自动退费逻辑
   • 配置加载失败默认关闭（fail-closed，安全）
4. 测试：
   • test_config_base.py 加 4 个用例：默认 false、环境变量 true、关闭时装饰器透传、开启时强制鉴权
   • test_points_gate.py 所有用例加 autouse fixture monkeypatch.setattr(_points_gate_enabled, lambda: True)，确保原有 10 个测试继续覆盖开启时的行为

验证

• 本地 15442 passed, 28 skipped（全量单测）
• black + ruff 全部通过
• 开关默认 false，staging 合入后对现有用户完全无影响

后续 PR 计划（按你给的优先级）：

• TTS 合成/试听（routes/tts.py + voice_clones.py preview）
• 口型同步（routes/lipsync.py）
• 文案 AI（routes/scripts_ai.py 三端点）
• 视频生成（routes/generation_tasks.py）
• 视频预览（routes/generation_preview.py）
• AI 数字人（routes/ai_avatar_render.py）
• 封面生成（routes/generation_cover.py）
• 注册送积分（routes/auth.py）

## #1895 P2 第 1 步：积分系统配置安全闸门 按要求先补总开关，再逐个路由接入 @points_gate，确保 P2 阶段合入后对现有用户零影响。 ### 改动 1. **`packages/config/base.py` SharedSettings** 新增 `points_enabled: bool = False`（pydantic-settings 大小写不敏感，自动支持 `POINTS_ENABLED=true` 环境变量） 2. **`.env.example`** 新增 `POINTS_ENABLED=false` 配置项及注释说明 3. **`packages/middleware/points_gate.py`** 新增模块级函数 `_points_gate_enabled()` 读取开关： - 默认 false（开关未开）时，`@points_gate` 装饰器完全透传原函数，零开销零副作用，不读 user/db、不扣点、不抛 401/402 - true 时执行原有扣费/401/402/异常自动退费逻辑 - 配置加载失败默认关闭（fail-closed，安全） 4. **测试**： - `test_config_base.py` 加 4 个用例：默认 false、环境变量 true、关闭时装饰器透传、开启时强制鉴权 - `test_points_gate.py` 所有用例加 autouse fixture `monkeypatch.setattr(_points_gate_enabled, lambda: True)`，确保原有 10 个测试继续覆盖开启时的行为 ### 验证 - 本地 15442 passed, 28 skipped（全量单测） - black + ruff 全部通过 - 开关默认 false，staging 合入后对现有用户完全无影响 后续 PR 计划（按你给的优先级）： - [ ] TTS 合成/试听（routes/tts.py + voice_clones.py preview） - [ ] 口型同步（routes/lipsync.py） - [ ] 文案 AI（routes/scripts_ai.py 三端点） - [ ] 视频生成（routes/generation_tasks.py） - [ ] 视频预览（routes/generation_preview.py） - [ ] AI 数字人（routes/ai_avatar_render.py） - [ ] 封面生成（routes/generation_cover.py） - [ ] 注册送积分（routes/auth.py）

xiaoxia added 1 commit 2026-09-15 18:40:39 +08:00

feat(points): P2 step 1 - add POINTS_ENABLED safety gate with default false ... f587b7a44d

- Add points_enabled: bool = False to SharedSettings (packages/config/base.py)
- Add POINTS_ENABLED=false to .env.example with documentation
- Add _points_gate_enabled() module-level helper in points_gate middleware
- When disabled (default), @points_gate decorator is pure pass-through (zero side effects)
- When enabled, existing deduct/refund/402/401/500 logic activates as before
- Add 4 unit tests covering: default false / env override / disabled passthrough / enabled enforcement
- Fix existing points_gate tests to monkeypatch _points_gate_enabled=True (since default is now off)
- Total: 15442 passed, 28 skipped; black/ruff clean

gitea-actions bot commented 2026-09-15 18:42:09 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

项目	详情
PR号	#1936
预览链接	https://pr-1936.preview.xiaoxiajianji.com
API环境	staging

💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。

🔄 每次提交新代码后预览环境会自动更新。

🗑️ PR 关闭或合并后，预览环境会自动清理。

🚀 **预览环境已部署** | 项目 | 详情 | |------|------| | PR号 | #1936 | | 预览链接 | [https://pr-1936.preview.xiaoxiajianji.com](https://pr-1936.preview.xiaoxiajianji.com) | | API环境 | staging | > 💡 预览环境使用 staging API 数据，请勿在预览环境中操作重要数据。 > > 🔄 每次提交新代码后预览环境会自动更新。 > > 🗑️ PR 关闭或合并后，预览环境会自动清理。

auto-approve-bot added 1 commit 2026-09-15 18:44:39 +08:00

style: auto-format with black + isort + ruff + prettier [skip ci-format-check] 7aa5e56cd6

xiaoxia closed this pull request 2026-09-15 18:44:47 +08:00

xiaoxia reopened this pull request 2026-09-15 18:44:52 +08:00

gitea-actions bot commented 2026-09-15 18:45:36 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #1936 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1936 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

xiaoxia added 1 commit 2026-09-15 18:53:17 +08:00

chore: trigger CI 8343a1211e

xiaoxia added 1 commit 2026-09-15 22:52:05 +08:00

fix(points): make @points_gate wrapper inherit decorated module globals ... 61d998bb01

Previously the async/sync wrappers were defined inside the points_gate
module, so wrapper.__globals__ pointed to packages.middleware.points_gate
instead of the route module. When FastAPI/Pydantic resolved ForwardRef
type annotations (e.g. Pydantic models imported only in the route
module), they hit PydanticUndefinedAnnotation in environments where
typing.get_type_hints didn't walk __wrapped__ (Python 3.12 fast path).

Fix: clone the wrapper code via types.FunctionType into the decorated
function's module globals (with internal helpers injected), and use
_filter_kwargs consistently in all call paths (async passthrough and
gate-enabled paths included).

auto-approve-bot added 1 commit 2026-09-15 23:19:14 +08:00

style: auto-format with black + isort + ruff + prettier [skip ci-format-check] c8b0425986

auto-approve-bot merged commit 00045131f6 into develop 2026-09-16 00:26:29 +08:00

auto-approve-bot deleted branch feat/points-gate-config 2026-09-16 00:26:30 +08:00

gitea-actions bot commented 2026-09-16 00:29:23 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

PR #1936 已关闭或合并，对应的预览环境已被清理。

如有需要，可以重新打开 PR 来重新生成预览环境。

🗑️ **预览环境已清理** PR #1936 已关闭或合并，对应的预览环境已被清理。 > 如有需要，可以重新打开 PR 来重新生成预览环境。

xiaoxia referenced this pull request 2026-09-16 06:31:35 +08:00

feat(#1895): 会员积分系统后端（P1-P4，默认关闭） #1923

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot frontend-dev xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#1936