🐛 fix Onebot11 message_id get

resolve #99

Author: RF-Tar-Railt

src/nonebot_plugin_alconna/uniseg/adapters/onebot11/builder.py

@@ -6,7 +6,7 @@
 
 from nonebot_plugin_alconna.uniseg.constraint import SupportAdapter
 from nonebot_plugin_alconna.uniseg.builder import MessageBuilder, build
-from nonebot_plugin_alconna.uniseg.segment import At, AtAll, Emoji, Hyper, Image, Reply, Video, Voice, Reference
+from nonebot_plugin_alconna.uniseg.segment import At, File, AtAll, Emoji, Hyper, Image, Reply, Video, Voice, Reference
 
 
 class Onebot11MessageBuilder(MessageBuilder):
@@ -54,6 +54,12 @@ def json(self, seg: MessageSegment):
     def xml(self, seg: MessageSegment):
         return Hyper("xml", seg.data["data"])
 
+    @build("file")
+    def file(self, seg: MessageSegment):
+        url = seg.data.get("url") or seg.data.get("file")
+        name = seg.data.get("file_name") or seg.data.get("file")
+        return File(id=seg.data["file_id"], name=name or "file.bin", url=url)
+
     async def extract_reply(self, event: Event, bot: Bot):
         if TYPE_CHECKING:
             assert isinstance(event, MessageEvent)

src/nonebot_plugin_alconna/uniseg/adapters/onebot11/exporter.py

@@ -3,7 +3,6 @@
 
 from tarina import lang
 from nonebot.adapters import Bot, Event
-from nonebot.adapters.onebot.v11.event import MessageEvent
 from nonebot.adapters.onebot.v11.bot import Bot as OnebotBot
 from nonebot.adapters.onebot.v11.message import Message, MessageSegment
 
@@ -53,8 +52,9 @@ def get_target(self, event: Event, bot: Union[Bot, None] = None) -> Target:
         raise NotImplementedError
 
     def get_message_id(self, event: Event) -> str:
-        assert isinstance(event, MessageEvent)
-        return str(event.message_id)
+        if message_id := getattr(event, "message_id", None):
+            return str(message_id)
+        raise NotImplementedError
 
     @export
     async def text(self, seg: Text, bot: Union[Bot, None]) -> "MessageSegment":
@@ -197,8 +197,8 @@ async def recall(self, mid: Any, bot: Bot, context: Union[Target, Event]):
             await bot.delete_msg(message_id=mid["message_id"])
         elif isinstance(mid, (str, int)):
             await bot.delete_msg(message_id=int(mid))
-        elif not mid and isinstance(context, MessageEvent):
-            await bot.delete_msg(message_id=context.message_id)
+        elif not mid and hasattr(context, "message_id"):
+            await bot.delete_msg(message_id=context.message_id)  # type: ignore
 
     async def reaction(self, emoji: Emoji, mid: Any, bot: Bot, context: Union[Target, Event], delete: bool = False):
         assert isinstance(bot, OnebotBot)
@@ -217,9 +217,9 @@ async def reaction(self, emoji: Emoji, mid: Any, bot: Bot, context: Union[Target
                     return
                 group_id = int(context.id)
             else:
-                if not hasattr(context, "group_id"):
+                if not (group_id := getattr(context, "group_id", None)):
                     return
-                group_id = int(context.group_id)  # type: ignore
+                group_id = int(group_id)
             await bot.call_api(
                 "set_group_reaction",
                 group_id=group_id,