[All] Various fixes (#801)

1. In some cases, @ALL cannot be successfully parsed as MentionEntity
2. @ALL qq is not all
3. BounceFaceEntity's ToPreviewString is incorrectly written

Author: DarkRRb

Lagrange.Core/Internal/Packets/Message/Element/Implementation/Extra/MentionExtra.cs

@@ -1,5 +1,7 @@
 using ProtoBuf;
 
+#pragma warning disable CS8618
+
 namespace Lagrange.Core.Internal.Packets.Message.Element.Implementation.Extra;
 
 /// <summary>
@@ -8,11 +10,11 @@ namespace Lagrange.Core.Internal.Packets.Message.Element.Implementation.Extra;
 [ProtoContract]
 internal class MentionExtra
 {
-    [ProtoMember(3)] public int? Type { get; set; } // 1 for All Member, 2 for Specific Member
-    
-    [ProtoMember(4)] public uint? Uin { get; set; }
-    
-    [ProtoMember(5)] public int? Field5 { get; set; } // Const Zero
-    
-    [ProtoMember(9)] public string? Uid { get; set; } // set this string to "all" to mention everyone in the Group
+    [ProtoMember(3)] public int Type { get; set; } // 2
+
+    [ProtoMember(4)] public uint Uin { get; set; }
+
+    [ProtoMember(5)] public int Field5 { get; set; } // Const Zero
+
+    [ProtoMember(9)] public string Uid { get; set; } // set this string to "all" to mention everyone in the Group
 }

Lagrange.Core/Message/Entity/BounceFaceEntity.cs

@@ -104,7 +104,7 @@ IEnumerable<Elem> IMessageEntity.PackElement()
         return new BounceFaceEntity(extra.Face.FaceId, extra.Count, extra.Name);
     }
 
-    public string ToPreviewString() => "$[BounceFace | Name: {Name}({FaceId}) | Count: {Count}]";
+    public string ToPreviewString() => $"[BounceFace | Name: {Name}({FaceId}) | Count: {Count}]";
 
     public string ToPreviewText() => $"[{Name}]x{Count}";
 }

Lagrange.Core/Message/Entity/MentionEntity.cs

@@ -10,18 +10,18 @@ namespace Lagrange.Core.Message.Entity;
 public class MentionEntity : IMessageEntity
 {
     public uint Uin { get; set; }
-    
+
     public string Uid { get; set; }
-    
+
     public string? Name { get; set; }
-    
+
     public MentionEntity()
     {
         Uin = 0;
         Uid = "";
         Name = "";
     }
-    
+
     /// <summary>
     /// Set target to 0 to mention everyone
     /// </summary>
@@ -43,7 +43,7 @@ IEnumerable<Elem> IMessageEntity.PackElement()
         };
         using var stream = new MemoryStream();
         Serializer.Serialize(stream, reserve);
-        
+
         return new Elem[]
         {
             new()
@@ -56,15 +56,18 @@ IEnumerable<Elem> IMessageEntity.PackElement()
             }
         };
     }
-    
+
     IMessageEntity? IMessageEntity.UnpackElement(Elem elems)
     {
-        if (elems.Text is { Str: not null, Attr6Buf: { Length: >= 11 } attr })
+        if (elems.Text is { Str: not null, PbReserve: { Length: > 0 } extra })
         {
+            MentionExtra info = Serializer.Deserialize<MentionExtra>(extra.AsSpan());
+            if (info.Type != 2) return null;
+
             return new MentionEntity
             {
                 Name = elems.Text.Str,
-                Uin = BitConverter.ToUInt32(attr.AsSpan()[7..11], false),
+                Uin = info.Uin,
                 Uid = ""
             };
         }

Lagrange.OneBot/Message/Entity/AtSegment.cs

@@ -11,7 +11,7 @@ public AtSegment() : this(0, null) { }
 
     public AtSegment(uint at) : this(at, null) { }
 
-    [JsonPropertyName("qq")] [CQProperty] public string At { get; set; } = at.ToString();
+    [JsonPropertyName("qq")] [CQProperty] public string At { get; set; } = at == 0 ? "all" : at.ToString();
 
     [JsonPropertyName("name")] [CQProperty] public string? Name { get; set; } = name;
 }