feat(vibe coding PR): 支持建卡完成后的可选生图 - #202
Conversation
There was a problem hiding this comment.
Review Summary
Found one merge-blocking validation issue. The full backend test suite passes locally (233 passed, 2 skipped), as do Ruff lint and ty, but the required Ruff formatting check fails on the current head. No additional high-confidence quality, performance, security, or documentation findings were identified.
| and self.portrait_image_provider == "dashscope" | ||
| and (self.dashscope_api_key is None or not secret_value(self.dashscope_api_key).strip()) | ||
| ): | ||
| raise ValueError("角色生图使用 DashScope 时必须设置 DASHSCOPE_API_KEY") |
There was a problem hiding this comment.
[P1] Restore the required backend formatting check
uv run ruff format --check . fails on this head and reports that app/adapters/deepseek_models.py, app/core/config.py, app/service/portrait_generation.py, and tests/test_portrait_generation.py would be reformatted. This is also why the backend test check is red. Please run Ruff formatting and commit the resulting changes so the required CI job can pass.
关联 Issue
Closes #199
主要改动
generatePortrait(...)及对应请求/响应类型。.env.example和 README 配置说明。采用该方案的原因
测试与验证
17 passed30 passed233 passed, 2 skipped16 passedruff check/ty check/git diff --check/ SDK lint-typecheck-build:通过风险与回滚
DEEPSEEK_API_KEY和DASHSCOPE_API_KEY。CHARACTER_PORTRAIT_ENABLED,默认行为不变。UI 证据
AI 使用情况
自检