From bf401e49165ae7c6bcaee91e0618d4d5a0c1e8ef Mon Sep 17 00:00:00 2001
From: xfy <i@rua.plus>
Date: Wed, 10 Jun 2026 13:19:57 +0800
Subject: [PATCH] fix(api): correct AppError variant semantics for
 transaction/query errors

- client.transaction() failures: db_conn -> tx (logs correct category)
- auth session INSERT: db_conn -> query
- delete_post UPDATE: tx -> query (no transaction context)
---
 src/api/auth.rs  | 2 +-
 src/api/posts.rs | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/api/auth.rs b/src/api/auth.rs
index 6e78fac..ddd435c 100644
--- a/src/api/auth.rs
+++ b/src/api/auth.rs
@@ -196,7 +196,7 @@ pub async fn login(username: String, password: String) -> Result<AuthResponse, S
             &[&user_id, &token, &expires_at],
         )
         .await
-        .map_err(AppError::db_conn)?;
+        .map_err(AppError::query)?;
 
     let cookie = format!(
         "session={token}; HttpOnly; Path=/; Max-Age={}; SameSite=Lax",
diff --git a/src/api/posts.rs b/src/api/posts.rs
index ffd3e92..fb4ce39 100644
--- a/src/api/posts.rs
+++ b/src/api/posts.rs
@@ -263,7 +263,7 @@ pub async fn create_post(
         None
     };
 
-    let tx = client.transaction().await.map_err(AppError::db_conn)?;
+    let tx = client.transaction().await.map_err(AppError::tx)?;
 
     let row = tx
         .query_one(
@@ -411,7 +411,7 @@ pub async fn update_post(
     let post_status = PostStatus::from_str(&status).unwrap_or(PostStatus::Draft);
     let cover_image = cover_image.filter(|s| !s.trim().is_empty());
 
-    let tx = client.transaction().await.map_err(AppError::db_conn)?;
+    let tx = client.transaction().await.map_err(AppError::tx)?;
 
     let old_tags: Vec<String> = {
         let rows = tx
@@ -715,7 +715,7 @@ pub async fn delete_post(post_id: i32) -> Result<CreatePostResponse, ServerFnErr
             &[&post_id],
         )
         .await
-        .map_err(AppError::tx)?;
+        .map_err(AppError::query)?;
 
     if result == 0 {
         return Ok(CreatePostResponse {