🍻 ignore at_sender when send to private target

Author: RF-Tar-Railt

src/nonebot_plugin_alconna/uniseg/message.py

@@ -1380,12 +1380,14 @@ async def send(
                 except LookupError as e:
                     raise SerializeFailed(lang.require("nbp-uniseg", "bot_missing")) from e
         if at_sender:
-            if isinstance(at_sender, str):
-                self.insert(0, At("user", at_sender))  # type: ignore
-            elif isinstance(target, Event):
-                self.insert(0, At("user", target.get_user_id()))  # type: ignore
-            else:
-                raise TypeError("at_sender must be str when target is not Event")
+            _target = target if isinstance(target, Target) else get_target(target, bot)
+            if not _target.private:
+                if isinstance(at_sender, str):
+                    self.insert(0, At("user", at_sender))  # type: ignore
+                elif isinstance(target, Event):
+                    self.insert(0, At("user", target.get_user_id()))  # type: ignore
+                else:
+                    raise TypeError("at_sender must be str when target is not Event")
         if reply_to:
             if isinstance(reply_to, Reply):
                 self.insert(0, reply_to)  # type: ignore

tests/test_uniseg.py

@@ -6,7 +6,7 @@
 from nonebot.compat import model_dump, type_validate_python
 from nonebot.adapters.onebot.v11 import Bot, Adapter, Message, MessageSegment
 
-from tests.fake import fake_group_message_event_v11
+from tests.fake import fake_group_message_event_v11, fake_private_message_event_v11
 
 
 def ansi(*code) -> str:
@@ -277,6 +277,17 @@ async def handle(msg: MsgId):
             MessageSegment.reply(event.message_id + 1) + MessageSegment.at(123) + MessageSegment.text("world!"),
         )
         ctx.should_call_api("delete_msg", {"message_id": event.message_id + 1})
+        event1 = fake_private_message_event_v11(message=Message("test_unimsg_send"), user_id=123)
+        ctx.receive_event(bot, event1)
+        ctx.should_call_send(
+            event1,
+            MessageSegment.reply(event1.message_id) + MessageSegment.text("hello!"),
+        )
+        ctx.should_call_send(
+            event1,
+            MessageSegment.reply(event1.message_id + 1) + MessageSegment.text("world!"),
+        )
+        ctx.should_call_api("delete_msg", {"message_id": event1.message_id + 1})
 
     async with app.test_api() as ctx1:
         adapter = get_adapter(Adapter)