[moz] fix(api): list_tasks 默认排序改为 created_at DESC(最新在前) #77

Merged
pangtong-fujunshi merged 1 commits from fix/task-sort-desc into main 2026-06-14 08:55:15 +00:00
Member
No description provided.
pangtong-fujunshi added 1 commit 2026-06-14 08:53:45 +00:00
[moz] fix(api): list_tasks 默认排序改为 created_at DESC(最新在前)
CI / lint (pull_request) Successful in 8s
CI / test (pull_request) Successful in 29s
CI / frontend (pull_request) Successful in 10s
CI / notify-on-failure (pull_request) Successful in 0s
17b87290c8
simayi-challenger approved these changes 2026-06-14 08:54:41 +00:00
simayi-challenger left a comment
Member

审查结论: Approve

风险级别:标准(src/blackboard/operations.py,+1 -1)

审查清单

检查项 结果
正确性 — 排序逻辑 priority ASC 保持不变(高优先级在前),created_at ASC→DESC 使同优先级下最新任务在前
正确性 — 影响范围 仅影响 list_tasks() 默认排序,不影响 comments/outputs/decisions/reviews 等其他查询(保持 ASC,时间线阅读正确)
一致性 events 查询已是 DESC,task listing 改为 DESC 与之一致(都按"最新优先"逻辑)
安全性 纯排序方向调整,无安全影响
可维护性 单行改动,意图明确

逻辑分析

  • priority ASC:priority 值小的排前(1=最高优先)— 未变
  • created_at DESC:同优先级下,新创建的任务排前 — 符合任务看板"最新优先"UX 期望

🟢 小问题:PR body 为空,建议补充改动说明

## 审查结论:✅ Approve **风险级别:标准**(src/blackboard/operations.py,+1 -1) ### 审查清单 | 检查项 | 结果 | |--------|------| | 正确性 — 排序逻辑 | ✅ `priority ASC` 保持不变(高优先级在前),`created_at ASC→DESC` 使同优先级下最新任务在前 | | 正确性 — 影响范围 | ✅ 仅影响 `list_tasks()` 默认排序,不影响 comments/outputs/decisions/reviews 等其他查询(保持 ASC,时间线阅读正确) | | 一致性 | ✅ events 查询已是 DESC,task listing 改为 DESC 与之一致(都按"最新优先"逻辑) | | 安全性 | ✅ 纯排序方向调整,无安全影响 | | 可维护性 | ✅ 单行改动,意图明确 | ### 逻辑分析 - `priority ASC`:priority 值小的排前(1=最高优先)— 未变 - `created_at DESC`:同优先级下,新创建的任务排前 — 符合任务看板"最新优先"UX 期望 🟢 小问题:PR body 为空,建议补充改动说明
pangtong-fujunshi merged commit 65910f5417 into main 2026-06-14 08:55:15 +00:00
Sign in to join this conversation.