fix: allow closing draft CLs and fix draft list ordering (#1982)

* fix: allow closing draft CLs and fix draft list ordering

- Allow /api/v1/cl/{link}/close to close both Open and Draft change lists so draft CLs can be closed from the UI
- Remove default ORDER BY id ASC in CL list query and respect created_at/updated_at sorting
- Ensure newly created draft CLs appear on the first page when sorted by latest creation time

Signed-off-by: Hongze Gao <15101764808@163.com>

* fix: make CL list fallback sort deterministic

- Ensure CL list falls back to ORDER BY id DESC when no valid sort field is provided
- Treat unknown sort_by values the same as missing ones to keep pagination stable
- Preserve created_at/updated_at based ordering when a recognized sort field is passed

Signed-off-by: Hongze Gao <15101764808@163.com>

* fix: format cl_storage.rs with cargo fmt

Signed-off-by: Hongze Gao <15101764808@163.com>

---------

Signed-off-by: Hongze Gao <15101764808@163.com>

Author: WiedersehenM

jupiter/src/storage/cl_storage.rs

@@ -100,14 +100,20 @@ impl ClStorage {
                 q.filter(mega_cl::Column::Username.eq(author))
             })
             .filter(cond)
-            .distinct()
-            .order_by_asc(mega_cl::Column::Id);
+            .distinct();
 
         let mut sort_map = HashMap::new();
         sort_map.insert("created_at", mega_cl::Column::CreatedAt);
         sort_map.insert("updated_at", mega_cl::Column::UpdatedAt);
 
-        let sorted_query = apply_sort(base_query, params.sort_by.as_deref(), params.asc, &sort_map);
+        let sort_field = params.sort_by.as_deref();
+        let has_valid_sort = sort_field.and_then(|field| sort_map.get(field)).is_some();
+
+        let mut sorted_query = apply_sort(base_query, sort_field, params.asc, &sort_map);
+
+        if !has_valid_sort {
+            sorted_query = sorted_query.order_by_desc(mega_cl::Column::Id);
+        }
 
         let paginator = sorted_query.paginate(self.get_connection(), page.per_page);
         let total = paginator.num_items().await?;

mono/src/api/router/cl_router.rs

@@ -108,7 +108,7 @@ async fn close_cl(
     let res = state.cl_stg().get_cl(&link).await?;
     let model = res.ok_or(MegaError::Other("Not Found".to_string()))?;
 
-    if model.status == MergeStatusEnum::Open {
+    if matches!(model.status, MergeStatusEnum::Open | MergeStatusEnum::Draft) {
         let link = model.link.clone();
         state.cl_stg().close_cl(model).await?;
         state