fix: 素材列表默认按created_at倒序，新上传素材在最前面 #505

Merged

auto-approve-bot merged 3 commits from fix/asset-list-order-by into develop 2026-07-18 15:05:08 +08:00

Conversation 5 Commits 3 Files Changed 2

+106 -1

xiaoxia commented 2026-07-18 13:57:24 +08:00

Owner

Copy Link

Copy Source

问题：素材列表三个查询接口（find_by_library / find_by_project / find_by_library_and_file_type）全部缺少 ORDER BY 子句，PostgreSQL 按物理存储顺序返回数据。素材多过一页时，新上传的素材可能被排到后面，用户看不到。

修复：三个接口统一加 ORDER BY created_at DESC，最新上传的在最前面。

测试：新增3个单元测试验证排序正确性，全部通过。

关联：#406

**问题**：素材列表三个查询接口（find_by_library / find_by_project / find_by_library_and_file_type）全部缺少 ORDER BY 子句，PostgreSQL 按物理存储顺序返回数据。素材多过一页时，新上传的素材可能被排到后面，用户看不到。 **修复**：三个接口统一加 `ORDER BY created_at DESC`，最新上传的在最前面。 **测试**：新增3个单元测试验证排序正确性，全部通过。 **关联**：#406

xiaoxia added the bugbackend labels 2026-07-18 13:57:24 +08:00

xiaoxia added 1 commit 2026-07-18 13:57:24 +08:00

fix: 素材列表默认按created_at倒序排列，新上传的在最前面 ... 29d0ee5ba1

三个列表查询接口全部缺少ORDER BY子句，PostgreSQL按物理存储顺序返回，
导致新上传的素材可能不出现在第一页。

修复：
- find_by_library: 加 ORDER BY created_at DESC
- find_by_project: 加 ORDER BY created_at DESC
- find_by_library_and_file_type: 加 ORDER BY created_at DESC

新增3个单元测试验证排序正确性。

xiaoxia added 1 commit 2026-07-18 14:40:17 +08:00

style: black格式化测试文件 2a7565ea7e

gitea-actions bot commented 2026-07-18 14:41:19 +08:00

Copy Link

Copy Source

🚀 预览环境已部署

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

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

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

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

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

auto-approve-bot commented 2026-07-18 14:45:13 +08:00

Collaborator

Copy Link

Copy Source

代码审查结果 - PR #505

⚠️ 问题（0个需要修改）

无

💡 建议（1个可选）

1. packages/adapters/sqlalchemy_impl/asset_repository.py 第23, 37, 56行：建议确认 AssetModel.created_at 字段在数据库层面已建立索引。
   • 原因：在数据量较大的情况下，对非索引字段进行排序（ORDER BY）会导致数据库出现 Using filesort（MySQL）或类似的排序操作，消耗大量内存和CPU，严重降低查询性能。
   • 建议：检查对应的 Migration 文件或 Model 定义，确保 created_at 字段上有索引（例如 Index('idx_created_at', 'created_at')）。

---

✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能

---

_{🤖 由 AI 代码审查机器人自动生成 | 2026-07-18 14:45:13 | 模型:}

## 代码审查结果 - PR #505 ### ⚠️ 问题（0个需要修改） 无 ### 💡 建议（1个可选） 1. **packages/adapters/sqlalchemy_impl/asset_repository.py 第23, 37, 56行**：建议确认 `AssetModel.created_at` 字段在数据库层面已建立索引。 - **原因**：在数据量较大的情况下，对非索引字段进行排序（`ORDER BY`）会导致数据库出现 `Using filesort`（MySQL）或类似的排序操作，消耗大量内存和CPU，严重降低查询性能。 - **建议**：检查对应的 Migration 文件或 Model 定义，确保 `created_at` 字段上有索引（例如 `Index('idx_created_at', 'created_at')`）。 --- ✅ 格式检查通过 | ✅ 逻辑审查通过 | ⚠️ 建议关注性能 --- _{🤖 由 AI 代码审查机器人自动生成 | 2026-07-18 14:45:13 | 模型:} <!-- AI_CODE_REVIEW_AUTO_COMMENT -->

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

style: isort修复import排序 d4c7dcbd11

auto-approve-bot approved these changes 2026-07-18 15:03:12 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot approved these changes 2026-07-18 15:03:12 +08:00

auto-approve-bot left a comment

Collaborator

Copy Link

Copy Source

CI全绿，自动审批通过。

CI全绿，自动审批通过。

auto-approve-bot merged commit e724225f8a into develop 2026-07-18 15:05:08 +08:00

auto-approve-bot deleted branch fix/asset-list-order-by 2026-07-18 15:05:08 +08:00

gitea-actions bot commented 2026-07-18 15:05:36 +08:00

Copy Link

Copy Source

🗑️ 预览环境已清理

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

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

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

Sign in to join this conversation.

Reviewers

No Reviewers

auto-approve-bot

Labels

Clear labels

P0

阻塞性，必须立即处理

P1

高优先级

P2

中优先级

P3

低优先级/后考虑

architecture

架构设计

backend

后端相关

blocked

阻塞中

bug

Bug修复

ci-bug

CI本身的bug

ci-optimization

CI优化类任务

ci-pipeline

CI流水线/workflow相关

cleanup

代码清理

devops

运维/部署/CI

documentation

文档相关

feature

新功能

frontend

前端相关

question

需要讨论确认

ready

PR CI通过，可合并

refactor

代码重构

No Label backend bug

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

auto-approve-bot xiaoxia

No Assignees

3 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: xiaoxia/xiaoxia-saas#505