perf: ⚡️ 报错过滤有价值信息提示到接口

danni-cool · May 20, 2024 · 7429a1c · 7429a1c
1 parent 9a24f20
commit 7429a1c
Show file tree

Hide file tree

Showing 4 changed files with 35 additions and 15 deletions.
diff --git a/src/service/core.js b/src/service/core.js
@@ -93,7 +93,7 @@ module.exports.formatAndSendMsg = async function ({
     }
     success = true
   } catch (/** @type {any} */ e) {
-    error = e
+    error = Utils.filterUseFulHttpError(e)
     Utils.logger.error(e)
   }
 

diff --git a/src/service/msgSender.js b/src/service/msgSender.js
@@ -396,7 +396,7 @@ const handleMsg2Single = async function (body, { bot, messageReceiver }) {
         })
         msgArr[i].success = success
         if (!success) {
-          msgArr[i].error = error.toString()
+          msgArr[i].error = error
         }
       }
 
@@ -419,7 +419,7 @@ const handleMsg2Single = async function (body, { bot, messageReceiver }) {
           : null
       }
     } else if (!Array.isArray(payload.data)) {
-      const { success } = await formatAndSendMsg({
+      const { success, error } = await formatAndSendMsg({
         isRoom,
         bot,
         msgData: payload.data,
@@ -435,7 +435,7 @@ const handleMsg2Single = async function (body, { bot, messageReceiver }) {
           : {
               to,
               ...(isRoom !== undefined ? { isRoom } : {}),
-              data: [payload.data]
+              data: [{ ...payload.data, error }]
             }
       }
     }

diff --git a/src/utils/index.js b/src/utils/index.js
@@ -238,15 +238,31 @@ const sleep = async (ms) => {
 }
 
 /**
- * 删除登录缓存文件
+ * 过滤有价值的发送消息报错信息
+ * @param {*} error
  */
-// const deleteMemoryCard = () => {
-//   //@ts-expect-errors 必定是 pathlike
-//   if (fs.existsSync(memoryCardPath)) {
-//     //@ts-expect-errors 必定是 pathlike
-//     fs.unlinkSync(memoryCardPath)
-//   }
-// }
+const filterUseFulHttpError = (error) => {
+  let newErrorObj = {}
+
+  if (error.tips) {
+    newErrorObj.tips = error.tips
+    newErrorObj.code = error.code
+    newErrorObj.stack = error.stack
+
+    if (error.response) {
+      newErrorObj.response = {}
+      newErrorObj.response.status = error.response.status
+      newErrorObj.response.statusText = error.response.statusText
+      newErrorObj.response.url = error.response.url
+      newErrorObj.response.method = error.response.method
+      newErrorObj.response.method = error.response.method
+      newErrorObj.response.data = error.response.data
+    }
+    return newErrorObj
+  } else {
+    return error
+  }
+}
 
 module.exports = {
   ...require('./msg.js'),
@@ -262,5 +278,6 @@ module.exports = {
   parseJsonLikeStr,
   tryConvertCnCharToUtf8Char,
   sleep,
-  Defer
+  Defer,
+  filterUseFulHttpError
 }
diff --git a/typings/global.d.ts b/typings/global.d.ts
@@ -39,7 +39,7 @@ type pushMsgUnitTypeOpt = {
 
 type msgData = {
   success?: boolean
-  error?: string
+  error?: never
 } & pushMsgUnitTypeOpt
 
 type toType = string | { alias: string }
@@ -104,7 +104,10 @@ type msg2SingleRejectReason = {
 type failedTaskType = {
   to: toType
   isRoom?: boolean
-  data: pushMsgUnitTypeOpt | pushMsgUnitTypeOpt[] | []
+  data:
+    | (pushMsgUnitTypeOpt & { error?: never })
+    | (pushMsgUnitTypeOpt & { error?: never })[]
+    | []
 }
 
 type sendingTaskType = {