perf(projects): eliminate preview request waterfall - #462
Conversation
Rebuild the conflict-resolved PR content directly on the current main base.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
|
Replay the project preview aggregation changes on the latest main and retain the card-thumbnail fallback test.
Resolve the project page test conflict by preserving the single-request preview flow and the card-thumbnail fallback coverage.
![fennoai[bot]](https://avatars.githubusercontent.com/in/1924301?s=60&v=4){kind=small}
There was a problem hiding this comment.
检查了项目预览的后端聚合契约、前端消费路径，以及该替代提交与当前 main 的实际三方合并结果。批量接口消除了请求瀑布，但当前分支仍无法合并，并且预览搜索范围比原实现缩窄，会把仍有可用素材的项目显示为空。
验证：GitHub 返回 mergeable=false / mergeable_state=dirty，本地固定 SHA 三方合并复现了冲突；原前端实现与新 SQL 排名条件的调用链已逐项比对。测试未能在此运行环境启动：后端缺少 uv/pytest，前端未安装 node_modules。
| @@ -1,42 +1,24 @@ | |||
| import { useEffect, useRef, useState, type CSSProperties } from 'react' | |||
| import { useEffect, useState, type CSSProperties } from 'react' | |||
There was a problem hiding this comment.
[P1] 将替代提交真正建立在当前 main 上
此 PR 的说明称提交已直接重建在当前 main，但 13ca792 的父提交仍是 57aca58，而不是本次基线 5ebe10a。GitHub 当前也返回 mergeable=false / dirty；按这两个固定 SHA 做三方合并时，frontend/src/pages/projects/index.tsx、对应测试和 openapi.json 都产生冲突。因此这个用于解决冲突的替代 PR 仍然无法合并；请从 5ebe10a 重新创建/拣选该提交并解决这些重叠改动。
| ranked.c.character_data, | ||
| ranked.c.project_rank, | ||
| ) | ||
| .where(ranked.c.project_rank <= character_limit) |
There was a problem hiding this comment.
[P1] 不要只检查最近六个角色的预览
原来的前端逻辑以每页 6 个角色持续翻页，直到找到预览或遍历完整个项目；这里却在 SQL 层永久丢弃 project_rank > 6 的角色。只要最新六个角色都没有图片、较早角色仍有造型/参考图/帧，列表就会返回 preview_url: null 并错误显示“等待第一份角色资产”。请在单次批量查询中搜索全部角色（例如直接选出每项目首个非空预览），而不是用固定数量截断语义。
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
1 participant
Why
项目列表直接交付每个项目的可用资产预览，并移除项目中心逐项目读取 Character 的请求队列，使一页项目只需一次业务 API 请求。
Conflict resolution
This replacement preserves the exact resolved source tree from #431, but rebuilds it as a single commit directly on current
mainto avoid GitHub's persistent falsedirtymergeability state.Supersedes #431 for merging purposes.