From c115414f5afe23cff2ce555fa272aea2a074e79e Mon Sep 17 00:00:00 2001
From: "Nathan.fooo" <86001920+appflowy@users.noreply.github.com>
Date: Sun, 5 May 2024 10:52:56 +0800
Subject: [PATCH] chore: return no content (#523)

---
 src/api/workspace.rs | 27 ++++++++++++++++++++-------
 1 file changed, 20 insertions(+), 7 deletions(-)

diff --git a/src/api/workspace.rs b/src/api/workspace.rs
index 582acb0b..c47b2716 100644
--- a/src/api/workspace.rs
+++ b/src/api/workspace.rs
@@ -1019,14 +1019,27 @@ async fn summary_row_handler(
       return Err(AppError::InvalidRequest("Identity data is not supported".to_string()).into());
     },
     SummarizeRowData::Content(content) => {
-      let text = state
-        .ai_client
-        .summarize_row(&content)
-        .await
-        .map_err(|err| AppError::InvalidRequest(err.to_string()))?
-        .text;
+      if content.is_empty() {
+        return Ok(
+          AppResponse::Ok()
+            .with_data(SummarizeRowResponse {
+              text: "No content".to_string(),
+            })
+            .into(),
+        );
+      }
+
+      let result = state.ai_client.summarize_row(&content).await;
+      let resp = match result {
+        Ok(resp) => SummarizeRowResponse { text: resp.text },
+        Err(err) => {
+          error!("Failed to summarize row: {:?}", err);
+          SummarizeRowResponse {
+            text: "No content".to_string(),
+          }
+        },
+      };
 
-      let resp = SummarizeRowResponse { text };
       Ok(AppResponse::Ok().with_data(resp).into())
     },
   }