optimize

src/ntqqapi/api/msg.ts

@@ -153,12 +153,7 @@ export class NTQQMsgApi extends Service {
       )
       msgList = data.msgList
     }
-    const retMsg = msgList.find(msgRecord => {
+    return msgList.find(msgRecord => msgRecord.guildId === msgId)
-      if (msgRecord.guildId === msgId) {
-        return true
-      }
-    })
-    return retMsg!
   }
 
   async forwardMsg(srcPeer: Peer, destPeer: Peer, msgIds: string[]) {

src/onebot11/action/msg/DeleteMsg.ts

@@ -11,16 +11,16 @@ class DeleteMsg extends BaseAction<Payload, void> {
 
   protected async _handle(payload: Payload) {
     if (!payload.message_id) {
-      throw Error('message_id不能为空')
+      throw new Error('参数message_id不能为空')
     }
     const msg = await MessageUnique.getMsgIdAndPeerByShortId(+payload.message_id)
     if (!msg) {
-      throw `消息${payload.message_id}不存在`
+      throw new Error(`消息${payload.message_id}不存在`)
     }
     const data = await this.ctx.ntMsgApi.recallMsg(msg.Peer, [msg.MsgId])
     if (data.result !== 0) {
       this.ctx.logger.error('delete_msg', payload.message_id, data)
-      throw `消息撤回失败`
+      throw new Error(`消息撤回失败`)
     }
   }
 }

src/onebot11/action/msg/SendMsg.ts

@@ -160,6 +160,9 @@ export class SendMsg extends BaseAction<OB11PostSendMsg, ReturnDataType> {
       }
     }
     const returnMsg = await sendMsg(this.ctx, peer, sendElements, deleteAfterSentFiles)
+    if (!returnMsg) {
+      throw new Error('消息发送失败')
+    }
     return { message_id: returnMsg.msgShortId! }
   }
 
@@ -251,9 +254,12 @@ export class SendMsg extends BaseAction<OB11PostSendMsg, ReturnDataType> {
           // log("分割后的转发节点", sendElementsSplit)
           for (const eles of sendElementsSplit) {
             const nodeMsg = await sendMsg(this.ctx, selfPeer, eles, [], true)
+            if (!nodeMsg) {
+              this.ctx.logger.warn('转发节点生成失败', eles)
+              continue
+            }
             nodeMsgIds.push(nodeMsg.msgId)
             await this.ctx.sleep(400)
-            this.ctx.logger.info('转发节点生成成功', nodeMsg.msgId)
           }
           deleteAfterSentFiles.map((f) => fs.unlink(f, () => {
           }))

src/onebot11/helper/createMessage.ts

@@ -270,8 +270,10 @@ export async function sendMsg(
   const timeout = 10000 + (totalSize / 1024 / 256 * 1000)  // 10s Basic Timeout + PredictTime( For File 512kb/s )
   //log('设置消息超时时间', timeout)
   const returnMsg = await ctx.ntMsgApi.sendMsg(peer, sendElements, waitComplete, timeout)
-  returnMsg.msgShortId = MessageUnique.createMsg(peer, returnMsg.msgId)
+  if (returnMsg) {
-  ctx.logger.info('消息发送', returnMsg.msgShortId)
+    returnMsg.msgShortId = MessageUnique.createMsg(peer, returnMsg.msgId)
-  deleteAfterSentFiles.map(path => fsPromise.unlink(path))
+    ctx.logger.info('消息发送', returnMsg.msgShortId)
-  return returnMsg
+    deleteAfterSentFiles.map(path => fsPromise.unlink(path))
+    return returnMsg
+  }
 }