xiaoxia/xiaoxia-saas

fix: 修复 AI Code Review 阻塞级问题 v3（useCanvasPlayer 解码策略+资源泄漏+useStep6Cover 轮询超时） #1450

Merged

auto-approve-bot merged 3 commits from fix/ai-code-review-blocking-v3 into develop

2026-08-20 12:01:01 +08:00

Author	SHA1	Message	Date
xiaoxia	9cdbe6e97d	fix: 修复 AI Code Review v5（轮询请求重叠+重播黑屏） CI/CD Pipeline / Check if frontend-only change (pull_request) Successful in 1m0s 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 2m28s Details CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m29s Details PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m25s Details CI/CD Pipeline / Unit Tests (pull_request) Has been skipped Details PR Automation / Auto Approve on CI Green (pull_request) Successful in 2m51s Details CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped Details CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 1m7s Details Preview Deploy / Deploy Preview Environment (pull_request) Successful in 3m1s Details CI/CD Pipeline / PR Build 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 / Frontend Lint (pull_request) Successful in 2m20s Details CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 2m18s Details AI Code Review / AI Code Review (pull_request) Successful in 7m2s Details CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 9m42s Details CI/CD Pipeline / Integration Tests (pull_request) Has been skipped 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 / Canary Release to Production (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 / CI Gate (pull_request) Successful in 14s Details ACR Cleanup / ACR Image Cleanup (pull_request_target) Successful in 45s Details Preview Cleanup / Cleanup Preview Environment (pull_request) Successful in 1m9s Details - useStep6Cover: setInterval 改为递归 setTimeout，确保上一次请求完成后再发起下一次，避免请求风暴 - useCanvasPlayer play(): 检测重播场景（currentTime 接近0但 decodedSegmentsRef 非空）时清空标记并递增 generation，让 decodeAroundPosition 重新解码	2026-08-20 11:49:24 +08:00
xiaoxia	e3e57d236a	fix: 修复 AI Code Review v4（VideoFrame 所有权+解码并发竞态） CI/CD Pipeline / Build Staging API Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Worker Image (pull_request) Has been skipped Details CI/CD Pipeline / Build Staging Web 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 / Unit Tests (pull_request) Has been skipped Details CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped Details CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 50s Details CI/CD Pipeline / PR Build Worker Image (pull_request) Has been skipped Details Preview Deploy / Deploy Preview Environment (pull_request) Successful in 1m51s Details PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 1m58s Details CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 2m25s Details AI Code Review / AI Code Review (pull_request) Failing after 2m26s Details CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 2m36s Details CI/CD Pipeline / Frontend Lint (pull_request) Successful in 2m12s Details PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m16s Details CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 2m34s Details CI/CD Pipeline / Validate - Code Quality (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 - 移除 lastDrawnFrameRef：getCurrentFrame 返回 FrameQueue 内部引用，帧生命周期由 FrameQueue 管理（push 淘汰/close 时释放），渲染层不应 close - 引入 decodeGenerationRef 计数器：seek/destroy 时递增，decodeAroundPosition 在 await 前后检查代数是否匹配，避免并发竞态 - 初始化解码循环同样加入 generation 检查	2026-08-20 11:44:32 +08:00
xiaoxia	f729524ad0	fix: 修复 AI Code Review 阻塞级问题 v3（解码策略+资源泄漏+轮询超时） 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 1m10s Details Preview Deploy / Deploy Preview Environment (pull_request) Successful in 1m45s Details PR Automation / Auto Merge on CI Green + Approved (pull_request) Successful in 2m12s Details CI/CD Pipeline / Validate - Type Check (mypy) (pull_request) Successful in 3m36s Details CI/CD Pipeline / Validate - Migration (alembic) (pull_request) Successful in 3m48s Details PR Automation / Auto Approve on CI Green (pull_request) Successful in 3m59s Details CI/CD Pipeline / Unit Tests (pull_request) Has been skipped Details CI/CD Pipeline / PR Build API Image (pull_request) Has been skipped Details CI/CD Pipeline / PR Build Worker Image (pull_request) Has been skipped Details CI/CD Pipeline / Frontend Lint (pull_request) Successful in 2m36s Details CI/CD Pipeline / Frontend Unit Tests (pull_request) Successful in 1m53s Details AI Code Review / AI Code Review (pull_request) Failing after 6m50s Details CI/CD Pipeline / PR Build Web Image (pull_request) Successful in 2m45s Details CI/CD Pipeline / Validate - Code Quality (pull_request) Successful in 11m14s Details CI/CD Pipeline / Integration Tests (pull_request) Has been skipped 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 / Canary Release to Production (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 / CI Gate (pull_request) Successful in 13s Details useCanvasPlayer (PR #1437 + #1438): - 初始化解码改为按需模式：仅解码前3片段 + decodeAroundPosition 动态加载 ±1 片段 - 解码标记采用先add再decode模式（防并发重入），catch中delete允许失败重试 - 新增 lastDrawnFrameRef 追踪上一帧，绘制新帧前 close() 释放防止内存泄漏 - seek 时清空帧队列+释放上一帧+重置解码标记+重新解码目标区域 - destroy 增加帧清理和解码状态重置 useStep6Cover (PR #1439): - 轮询增加 pollCount(max 60) + setTimeout 120s 双重超时保护 - 所有退出路径均正确 clearTimeout + clearInterval - 400错误精确匹配预览缺失模式(errCode/message正则)，避免误触发自动修复 Closes: #1437, #1438, #1439	2026-08-20 11:28:10 +08:00