xiaoxia/xiaoxia-saas

fix: 3个bug修复 - flush/append_log/plan_id兜底 #1476

Merged

auto-approve-bot merged 6 commits from fix/three-bugs-flush-log-planid into develop

2026-08-24 11:26:00 +08:00

Author	SHA1	Message	Date
CI Bot	bf5244a6e1	style: auto-format with black + isort + prettier [skip ci-format-check] CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped Details CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped Details CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped Details CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped Details CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped Details CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 46s Details CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped Details CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped Details CI/CD Pipeline / PR Build Web Image (pull_request) Has been skipped Details CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 29s Details Preview Deploy / Deploy Preview Environment (pull_request) Successful in 1m47s Details CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m50s Details PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m48s Details AI Code Review / AI Code Review (pull_request) Successful in 2m2s Details CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m14s Details CI/CD Pipeline / PR Build API Image (pull_request) Successful in 1m48s Details PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m43s Details CI/CD Pipeline / Unit Tests (pull_request) Successful in 4m11s Details CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 6m8s Details CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped Details CI/CD Pipeline / Deploy Production (pull_request) Has been skipped Details CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped Details CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped Details CI/CD Pipeline / Integration Tests (pull_request) Successful in 3m16s Details CI/CD Pipeline / CI Gate (pull_request) Successful in 6s Details ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 1m13s Details Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 1m20s Details	2026-08-24 03:13:03 +00:00
xiaoxia	26b27a34fd	fix: AI Review修复 - plan_id兜底改为DB层过滤 CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped Details CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped Details CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped Details CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped Details CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped Details CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 41s Details CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped Details CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped Details CI/CD Pipeline / PR Build Web Image (pull_request) Has been skipped Details CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 29s Details PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m40s Details CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m52s Details Preview Deploy / Deploy Preview Environment (pull_request) Successful in 1m54s Details CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m15s Details CI/CD Pipeline / PR Build API Image (pull_request) Successful in 1m52s Details CI/CD Pipeline / Validate - Code Quality (pull_request) Has been cancelled Details CI/CD Pipeline / Unit Tests (pull_request) Has been cancelled Details CI/CD Pipeline / Integration Tests (pull_request) Has been cancelled Details CI/CD Pipeline / Build Production API Image (pull_request) Has been cancelled Details CI/CD Pipeline / Build Production Web Image (pull_request) Has been cancelled Details CI/CD Pipeline / Build Production Worker Image (pull_request) Has been cancelled Details CI/CD Pipeline / Deploy Production (pull_request) Has been cancelled Details CI/CD Pipeline / Production Browser E2E (pull_request) Has been cancelled Details CI/CD Pipeline / Canary Release to Production (pull_request) Has been cancelled Details CI/CD Pipeline / CI Gate (pull_request) Has been cancelled Details AI Code Review / AI Code Review (pull_request) Has been cancelled Details PR Automation / Auto Approve on CI Green (pull_request) Has been cancelled Details AI Code Review阻塞问题：list_by_template(limit=20)+内存过滤user_id 存在边界缺陷（超过20条或排序不确定时可能关联错误plan）修复：直接查EditPlanModel，在DB层按template_id+user_id过滤并按created_at desc取最新一条，无limit风险更新测试：mock改为DB query chain，新增no_match场景 11个回归测试全过	2026-08-24 11:09:31 +08:00
CI Bot	59d5986b20	style: auto-format with black + isort + prettier [skip ci-format-check] CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped Details CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped Details CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped Details CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped Details CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped Details CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 1m9s Details CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped Details CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped Details CI/CD Pipeline / PR Build Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m44s Details Preview Deploy / Deploy Preview Environment (pull_request) Successful in 1m43s Details CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m59s Details PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m58s Details CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 47s Details AI Code Review / AI Code Review (pull_request) Failing after 2m9s Details CI/CD Pipeline / PR Build API Image (pull_request) Successful in 1m43s Details PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m58s Details CI/CD Pipeline / Unit Tests (pull_request) Successful in 3m9s Details CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 7m5s Details CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped Details CI/CD Pipeline / Deploy Production (pull_request) Has been skipped Details CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped Details CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped Details CI/CD Pipeline / Integration Tests (pull_request) Has been cancelled Details CI/CD Pipeline / CI Gate (pull_request) Has been cancelled Details	2026-08-24 03:00:19 +00:00
xiaoxia	9bfe0b9963	test: 补充plan_id兜底逻辑执行测试，diff覆盖率100% CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped Details CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped Details CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped Details CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped Details CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped Details CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 50s Details CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped Details CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped Details CI/CD Pipeline / PR Build Web Image (pull_request) Has been skipped Details CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 45s Details CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m50s Details PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m51s Details CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 1m56s Details Preview Deploy / Deploy Preview Environment (pull_request) Successful in 2m33s Details CI/CD Pipeline / PR Build API Image (pull_request) Successful in 1m48s Details CI/CD Pipeline / Validate - Code Quality (pull_request) Has been cancelled Details CI/CD Pipeline / Unit Tests (pull_request) Has been cancelled Details CI/CD Pipeline / Integration Tests (pull_request) Has been cancelled Details CI/CD Pipeline / Build Production API Image (pull_request) Has been cancelled Details CI/CD Pipeline / Build Production Web Image (pull_request) Has been cancelled Details CI/CD Pipeline / Build Production Worker Image (pull_request) Has been cancelled Details CI/CD Pipeline / Deploy Production (pull_request) Has been cancelled Details CI/CD Pipeline / Production Browser E2E (pull_request) Has been cancelled Details CI/CD Pipeline / Canary Release to Production (pull_request) Has been cancelled Details CI/CD Pipeline / CI Gate (pull_request) Has been cancelled Details AI Code Review / AI Code Review (pull_request) Has been cancelled Details PR Automation / Auto Approve on CI Green (pull_request) Has been cancelled Details 新增3个端到端测试覆盖generation_tasks.py的兜底关联逻辑： - test_fallback_sets_plan_id_when_empty: 验证空plan_id时自动查找并关联 - test_no_fallback_when_plan_id_already_set: 验证已有plan_id时不触发兜底 - test_fallback_handles_exception_gracefully: 验证查找异常不阻塞主流程总计10个回归测试，diff覆盖率从7%提升到100%（14/14行全覆盖）	2026-08-24 10:56:52 +08:00
CI Bot	b348507d25	style: auto-format with black + isort + prettier [skip ci-format-check] CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 30s Details CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m52s Details CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m1s Details CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped Details PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m30s Details AI Code Review / AI Code Review (pull_request) Successful in 4m25s Details CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped Details CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped Details Preview Deploy / Deploy Preview Environment (pull_request) Successful in 3m11s Details CI/CD Pipeline / PR Build Web Image (pull_request) Has been skipped Details PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m42s Details CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped Details CI/CD Pipeline / Staging E2E Tests (pull_request) Has been skipped Details CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been skipped Details CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been skipped Details CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 43s Details CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 7m2s Details CI/CD Pipeline / PR Build API Image (pull_request) Successful in 3m0s Details CI/CD Pipeline / Unit Tests (pull_request) Failing after 6m14s Details CI/CD Pipeline / Build Production API Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Production Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Production Worker Image (pull_request) Has been skipped Details CI/CD Pipeline / Deploy Production (pull_request) Has been skipped Details CI/CD Pipeline / Canary Release to Production (pull_request) Has been skipped Details CI/CD Pipeline / Production Browser E2E (pull_request) Has been skipped Details CI/CD Pipeline / Integration Tests (pull_request) Successful in 7m36s Details CI/CD Pipeline / CI Gate (pull_request) Failing after 19s Details	2026-08-24 02:25:53 +00:00
xiaoxia	f8eb32144f	fix: 3个bug修复 - flush/append_log/plan_id兜底 CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 1m30s Details CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped Details CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 1m55s Details CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m8s Details PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m32s Details AI Code Review / AI Code Review (pull_request) Successful in 2m17s Details CI/CD Pipeline / Frontend Lint (pull_request) Has been skipped Details CI/CD Pipeline / Frontend Unit Tests (pull_request) Has been skipped Details CI/CD Pipeline / PR Build Web Image (pull_request) Has been skipped Details CI/CD Pipeline / Deploy Staging (Watchtower auto-deploy) (pull_request) Has been skipped Details Preview Deploy / Deploy Preview Environment (pull_request) Successful in 2m38s Details CI/CD Pipeline / PR Build Worker Image (pull_request) Successful in 45s Details CI/CD Pipeline / Validate - Code Quality (pull_request) Has been cancelled Details CI/CD Pipeline / Unit Tests (pull_request) Has been cancelled Details CI/CD Pipeline / Integration Tests (pull_request) Has been cancelled Details CI/CD Pipeline / PR Build API Image (pull_request) Has been cancelled Details CI/CD Pipeline / Staging E2E Tests (pull_request) Has been cancelled Details CI/CD Pipeline / Staging API Integration Tests (pull_request) Has been cancelled Details CI/CD Pipeline / Build Production API Image (pull_request) Has been cancelled Details CI/CD Pipeline / Build Production Web Image (pull_request) Has been cancelled Details CI/CD Pipeline / Build Production Worker Image (pull_request) Has been cancelled Details CI/CD Pipeline / Deploy Production (pull_request) Has been cancelled Details CI/CD Pipeline / Production Browser E2E (pull_request) Has been cancelled Details CI/CD Pipeline / ACR Image Cleanup (pull_request) Has been cancelled Details CI/CD Pipeline / Canary Release to Production (pull_request) Has been cancelled Details CI/CD Pipeline / CI Gate (pull_request) Has been cancelled Details PR Automation / Auto Approve on CI Green (pull_request) Has been cancelled Details Bug 1 (P0): edit_plan_service.replace_all_clips_transactional - db.add(model)后加db.flush()，让新建clip写入事务 - 后续查询status=='pending'才能找到新记录并标记ready - 根因：未flush的记录对SQLAlchemy query不可见 Bug 2 (P1): generation.py 失败日志append_log参数重复 - 原代码: append_log('任务失败', str(error), stage='render') - '任务失败'作为第一个位置参数已赋给stage，又传stage='render' → TypeError - 修复: append_log('render', str(error), level='ERROR', error_type=...) - 排查其他调用点无同样问题 Bug 3 (P0): 正式生成API缺少plan_id兜底关联 - 预览API有兜底：前端未传source_edit_plan_id时通过template_id+user_id查找plan - 正式生成API没有此逻辑，导致Worker拿不到plan_id无法走DB渲染路径 - 修复：任务创建并入队后，若source_edit_plan_id为空且template_id存在，通过SQLAlchemyEditPlanRepository.list_by_template查找该用户最新的plan关联新增7个回归测试，127个相关测试全过	2026-08-24 10:18:13 +08:00